GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — improvement(concurrency): limits configurable, docs updates by icecrasher321 · Pull Request #5640 · simstudioai/sim

Conversation

@icecrasher321

@icecrasher321 icecrasher321 commented Jul 13, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor

Summary

Make concurrency limits configurable with env vars. Enterprise metadata supports custom limits by plan.

No limits for self hosted env vars. Update the docs.

Type of Change

  • Documentation
  • Other: Dev improvment

Testing

Tested on Staging.

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)

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

greptile-apps[bot] reacted with thumbs up emoji
All reactions
  • 👍 1 reaction
Vikhyath Mondreti
improvement(concurrency): limits configurable, docs updates
c2ebd04
@vercel

vercel Bot commented Jul 13, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview, Comment Jul 13, 2026 8:09pm

Request Review

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@cursor

cursor Bot commented Jul 13, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

PR Summary

Medium Risk
Changes admission concurrency caps, billing attribution, Enterprise Stripe provisioning, and default unlimited behavior on self-hosted installs—areas that affect whether runs are admitted, timed out, or retained.

Overview
Execution concurrency is now a first-class, configurable limit: hosted defaults are 10 / 50 / 200 / 1,000 (free / Pro / Max / Enterprise), overridable via BILLING_CONCURRENCY_LIMIT_* and wired through Redis usage reservation instead of hardcoded caps. Enterprise can set concurrencyLimit in subscription metadata (admin dashboard issue/limits APIs); attribution and admission forward that override into reserveExecutionSlot.

Plan limit bucketing is aligned with paid tiers: Pro and Pro for Teams share Pro limits; Max and Max for Teams share Max limits (getPlanTypeForLimits), affecting concurrency, storage, rate limits, and timeouts.

Self-hosted / billing disabled: rate limits, sync/async timeouts, tables, storage, and retention cleanup dispatch are off by default; each cap can be re-enabled by explicitly setting the corresponding free-tier env var (documented in self-hosting docs, .env.example, and Helm defaults for chart installs).

Enterprise provisioning gains optional pausePaymentCollection (draft invoice freeze + indefinite keep_as_draft), concurrency on Stripe metadata sync (including unset via empty string), and related admin contract/view fields.

Upgrade comparison/plan copy and platform cost docs add a Concurrent Executions section; MCP tool execution treats zero timeout as unlimited when billing is disabled.

Reviewed by Cursor Bugbot for commit 791fbfb. Configure here.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

Vikhyath Mondreti
remove dead tests
8a46744
@greptile-apps

greptile-apps Bot commented Jul 13, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR makes billing limits configurable and updates Enterprise limit handling. The main changes are:

  • Added configurable concurrency defaults and Enterprise overrides.
  • Updated usage admission, rate, timeout, table, and storage limit behavior for billing-disabled deployments.
  • Extended Enterprise admin and Stripe provisioning metadata.
  • Updated MCP execution timeout handling and public billing docs.

Confidence Score: 5/5

This looks safe to merge.

  • No blocking issues found in the changed code.

Important Files Changed

Filename Overview
apps/sim/lib/billing/concurrency-defaults.ts Adds shared concurrency defaults and parsing bounds for billing plans.
apps/sim/lib/billing/concurrency-limits.ts Resolves plan concurrency limits from deployment settings and Enterprise metadata.
apps/sim/lib/billing/calculations/usage-reservation.ts Uses the new concurrency resolver when reserving execution slots.
apps/sim/lib/billing/enterprise-provisioning.ts Adds Enterprise concurrency metadata and optional paused collection setup.
apps/sim/lib/admin/dashboard.ts Adds Enterprise concurrency limits to dashboard summaries and updates.
apps/sim/app/api/mcp/tools/execute/route.ts Treats a zero execution timeout as no timeout for MCP tool execution.

Reviews (2): Last reviewed commit: "limits self hosted vars" | Re-trigger Greptile

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@vercel
vercel Bot deployed to Preview July 13, 2026 18:52
View deployment
Vikhyath Mondreti
limits self hosted vars
791fbfb
@icecrasher321

icecrasher321 commented Jul 13, 2026

Copy link
Copy Markdown
Contributor Author

@greptile

greptile-apps[bot] reacted with thumbs up emoji
All reactions
  • 👍 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@icecrasher321

icecrasher321 commented Jul 13, 2026

Copy link
Copy Markdown
Contributor Author

bugbot run

cursor[bot] reacted with thumbs up emoji
All reactions
  • 👍 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@vercel
vercel Bot deployed to Preview July 13, 2026 20:09
View deployment
cursor[bot]
cursor Bot reviewed Jul 13, 2026
View reviewed changes

@cursor cursor Bot left a comment

Copy link
Copy Markdown

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit 791fbfb. Configure here.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@icecrasher321
icecrasher321 merged commit 2d41360 into staging Jul 13, 2026
19 checks passed

Uh oh!

There was an error while loading. Please reload this page.

@icecrasher321
icecrasher321 deleted the imrpovement/concurrency-limits branch July 13, 2026 20:11
@waleedlatif1 waleedlatif1 mentioned this pull request Jul 13, 2026
v0.7.32: concurrency control, seo/geo, new lib #5646
Merged

This branch was successfully deployed

1 active deployment
Preview — 791fbfba Deployed Jul 13, 2026 by vercel[bot]
View deployment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cursor cursor[bot] cursor[bot] left review comments

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

1 participant

@icecrasher321
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402520 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.