Skip to content

docs: update batch trigger concurrency limits#3171

Merged
samejr merged 1 commit intomainfrom
ea-branch-124
Mar 4, 2026
Merged

docs: update batch trigger concurrency limits#3171
samejr merged 1 commit intomainfrom
ea-branch-124

Conversation

@ericallam
Copy link
Member

No description provided.

@changeset-bot
Copy link

changeset-bot bot commented Mar 4, 2026

⚠️ No Changeset found

Latest commit: 290884a

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 4, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: a83fcd00-3dfa-4612-ab93-10dc1afee821

📥 Commits

Reviewing files that changed from the base of the PR and between c5ce297 and 290884a.

📒 Files selected for processing (1)
  • docs/limits.mdx
📜 Recent review details
🧰 Additional context used
📓 Path-based instructions (2)
docs/**/*.{md,mdx}

📄 CodeRabbit inference engine (CLAUDE.md)

Docs in docs/ directory should use Mintlify MDX format following conventions in docs/CLAUDE.md

Files:

  • docs/limits.mdx
docs/**/*.mdx

📄 CodeRabbit inference engine (docs/CLAUDE.md)

docs/**/*.mdx: MDX documentation pages must include frontmatter with title (required), description (required), and sidebarTitle (optional) in YAML format
Use Mintlify components for structured content: , , , , , , /, /
Always import from @trigger.dev/sdk in code examples (never from @trigger.dev/sdk/v3)
Code examples must be complete and runnable where possible
Use language tags in code fences: typescript, bash, json

Files:

  • docs/limits.mdx
🧠 Learnings (2)
📓 Common learnings
Learnt from: CR
Repo: triggerdotdev/trigger.dev PR: 0
File: .cursor/rules/writing-tasks.mdc:0-0
Timestamp: 2025-11-27T16:27:35.304Z
Learning: Applies to **/trigger/**/*.{ts,tsx,js,jsx} : Control concurrency using the `queue` property with `concurrencyLimit` option
📚 Learning: 2025-11-27T16:27:35.304Z
Learnt from: CR
Repo: triggerdotdev/trigger.dev PR: 0
File: .cursor/rules/writing-tasks.mdc:0-0
Timestamp: 2025-11-27T16:27:35.304Z
Learning: Applies to **/trigger/**/*.{ts,tsx,js,jsx} : Control concurrency using the `queue` property with `concurrencyLimit` option

Applied to files:

  • docs/limits.mdx
🔇 Additional comments (1)
docs/limits.mdx (1)

134-136: Looks good — batch concurrency limits update is correctly reflected.

The updated Free and Pro values are clear and appropriately applied in the limits table.


Walkthrough

The pull request updates the batch trigger processing concurrency limits documentation in docs/limits.mdx. The Free tier limit is increased from 1 to 5 concurrent batches, and the Pro tier limit is increased from 10 to 50 concurrent batches. The change consists of two line modifications updating numeric values in a documentation table.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The pull request description is entirely missing, failing to provide required sections like Testing, Changelog, and the completion checklist. Add a complete pull request description following the template, including the checklist, testing steps, and changelog entry.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately and specifically describes the main change: updating batch trigger concurrency limits in the documentation.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch ea-branch-124

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Contributor

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

Devin Review found 1 potential issue.

Open in Devin Review

Comment on lines +134 to +136
| Free | 5 concurrent batches |
| Hobby | 10 concurrent batches |
| Pro | 10 concurrent batches |
| Pro | 50 concurrent batches |
Copy link
Contributor

Choose a reason for hiding this comment

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

🚩 Hobby tier now has lower concurrency than Free tier's old value — verify intended ordering

After this change, the Free tier (5 concurrent batches) is still below the Hobby tier (10), and Pro (50) is above both, so the tier ordering is consistent. However, it's worth noting the Free tier jumped 5x (1→5) while Hobby stayed at 10, narrowing the gap significantly. The Pro tier jumped 5x (10→50), creating a much larger gap between Hobby and Pro. This may be intentional, but the reviewer should confirm that the Hobby tier value wasn't also meant to be updated (e.g., to 25 or similar) to maintain proportional spacing between tiers.

Open in Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.

@samejr samejr merged commit c013322 into main Mar 4, 2026
12 checks passed
@samejr samejr deleted the ea-branch-124 branch March 4, 2026 11:51
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