Skip to content

Conversation

@icecrasher321
Copy link
Collaborator

Summary

RATE_LIMIT_FREE_SYNC=50
RATE_LIMIT_FREE_ASYNC=200
RATE_LIMIT_PRO_SYNC=150
RATE_LIMIT_PRO_ASYNC=1000
RATE_LIMIT_TEAM_SYNC=300
RATE_LIMIT_TEAM_ASYNC=2500
RATE_LIMIT_ENTERPRISE_SYNC=600
RATE_LIMIT_ENTERPRISE_ASYNC=5000

Type of Change

  • Other: Rate Limits

Testing

N/A

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@vercel
Copy link

vercel bot commented Jan 31, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview, Comment Jan 31, 2026 4:01am

Request Review

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Jan 31, 2026

Greptile Overview

Greptile Summary

Increased rate limits across all subscription plans to better serve growing usage patterns. The changes are comprehensive and consistent across configuration, implementation, documentation, and UI.

Changes:

  • Free tier: 5x increase (10→50 sync, 50→200 async)
  • Pro tier: 6x sync increase (25→150), 5x async increase (200→1,000)
  • Team tier: 4x increase (75→300 sync, 500→2,500 async)
  • Enterprise tier: 4x increase (150→600 sync, 1,000→5,000 async)
  • API endpoint limits: Also increased proportionally (Free: 10→30, Pro: 30→100, Team: 60→200, Enterprise: 120→500)

All configuration files, documentation, UI components, and email templates were updated consistently to reflect the new limits.

Confidence Score: 5/5

  • This PR is safe to merge - it's a straightforward configuration change with no code logic modifications
  • The changes are purely numerical configuration updates that increase rate limits. All affected files (environment config, rate limiter types, documentation, UI text, and email templates) have been updated consistently. No logic changes, no new dependencies, and no security implications.
  • No files require special attention

Important Files Changed

Filename Overview
apps/sim/lib/core/config/env.ts Updated default rate limit values for all tiers - Free (5x increase), Pro (6x sync, 5x async), Team (4x both), Enterprise (4x both)
apps/sim/lib/core/rate-limiter/types.ts Updated rate limit configurations and fallback values to match new limits, also increased apiEndpoint limits (Free: 3x, Pro: 3.3x, Team: 3.3x, Enterprise: 4.2x)
apps/docs/content/docs/en/execution/costs.mdx Updated plan limits table and example responses to show new rate limits for all tiers

Sequence Diagram

sequenceDiagram
    participant User
    participant API
    participant RateLimiter
    participant TokenBucket
    participant Config

    Note over Config: Rate Limit Configuration<br/>env.ts → types.ts
    
    User->>API: Request (sync/async execution)
    API->>RateLimiter: Check rate limit
    RateLimiter->>Config: Get plan limits
    
    alt Environment Variables Set
        Config-->>RateLimiter: Use env values
    else No Environment Variables
        Config-->>RateLimiter: Use defaults<br/>(Free: 50/200, Pro: 150/1000,<br/>Team: 300/2500, Enterprise: 600/5000)
    end
    
    RateLimiter->>TokenBucket: Check tokens available
    TokenBucket->>TokenBucket: Calculate burst capacity<br/>(requestsPerMinute × 2)
    
    alt Tokens Available
        TokenBucket-->>RateLimiter: Tokens consumed
        RateLimiter-->>API: Allow request
        API-->>User: Response + rate limit headers
    else No Tokens Available
        TokenBucket-->>RateLimiter: Rate limited
        RateLimiter-->>API: Rate limit error (429)
        API-->>User: 429 Too Many Requests + Retry-After
    end
    
    Note over User,Config: Documentation & UI updated to reflect new limits
Loading

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

3 files reviewed, no comments

Edit Code Review Agent Settings | Greptile

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants