Repository navigation
docs(webhooks): pricing and per-plan webhook limits - #5051
Conversation
Webhooks have no separate charge: the runs a delivery starts are billed as normal compute, and filtered deliveries start no run. Adds the plan limits for webhook waiters, processing concurrency, delivery retention and subscribers per endpoint to the limits page, links them from the webhooks pages, and adds a webhook filter tip to the spend guide and the bundled cost-savings skill.
|
No actionable comments were generated in the recent review.
 |
| Check name | Status | Explanation |
|---|---|---|
| Title check |  |
The title clearly and concisely describes the main documentation changes for webhook pricing and per-plan limits. |
| Description check | |
The description explains the pricing, affected documentation, testing performed, and remaining merge checks. It does not follow the template exactly because it omits a Closes issue line, the standard … |
| Docstring Coverage | |
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0… |
| Linked Issues check | |
Check skipped because no linked issues were found for this pull request. |
| Out of Scope Changes check | |
Check skipped because no linked issues were found for this pull request. |
Full details: Description check
-
Explanation
The description explains the pricing, affected documentation, testing performed, and remaining merge checks. It does not follow the template exactly because it omits a Closes issue line, the standard checklist, and a Screenshots section, but it is sufficiently complete and directly relevant.
Full details: Docstring Coverage
-
Explanation
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (7 skipped: 7 unsupported.)
Finishing Touches
-
🧪 Generate unit tests (beta)
-
- Commit to this branch
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
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.
Comment @coderabbitai help to get the list of available commands.
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 7
ℹ️ Review info
⚙️ Run configuration
- Source mode: Auto
- Repository: Unavailable
- Branch: Unavailable
- File: No configuration file from this repository applied
- Commit: Unavailable
- Layers: Repository UI
- Review profile: CHILL
- Plan: Advanced
- Run ID:
735eb1fe-95d4-45b6-9a5e-bc1f29b5101a
📒 Files selected for processing: 7
.changeset/cost-skill-webhook-filters.mddocs/how-to-reduce-your-spend.mdxdocs/limits.mdxdocs/webhooks/deliveries.mdxdocs/webhooks/overview.mdxdocs/webhooks/waiters.mdxpackages/trigger-sdk/skills/trigger-cost-savings/SKILL.md
Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 9 remain after this review.
📜 Review details
⏰ Context from checks skipped due to timeout: 12
- GitHub Check: sdk-compat / Node.js 20.20 (warp-ubuntu-latest-x64-4x)
- GitHub Check: webapp / 🧪 Unit Tests: Webapp (2, 6)
- GitHub Check: webapp / 🧪 Unit Tests: Webapp (6, 6)
- GitHub Check: sdk-compat / Node.js 26.12.1 (warp-ubuntu-latest-x64-4x)
- GitHub Check: webapp / 🧪 Unit Tests: Webapp (3, 6)
- GitHub Check: webapp / 🧪 Unit Tests: Webapp (1, 6)
- GitHub Check: internal / 🧪 Unit Tests: Internal (1)
- GitHub Check: packages / 🧪 Unit Tests: Packages (2, 3)
- GitHub Check: internal / 🧪 Unit Tests: Internal (2)
- GitHub Check: e2e-webapp / 🧪 E2E Tests: Webapp (1, 1)
- GitHub Check: packages / 🧪 Unit Tests: Packages (3, 3)
- GitHub Check: Analyze (javascript-typescript)
🧰 Additional context used
📚 Code guidelines: 4
packages/trigger-sdk/AGENTS.md — auto-discovered
.claude/rules/sdk-packages.md — auto-discovered
.claude/rules/docs-writing.md — auto-discovered
docs/AGENTS.md — auto-discovered
📓 Path-based instructions: 4
Source excerpt: Always import from `@trigger.dev/sdk`.
📄 CodeRabbit inference engine (packages/trigger-sdk/AGENTS.md)
Files:
packages/trigger-sdk/skills/trigger-cost-savings/SKILL.md
Source excerpt: Changes to `packages/` are **customer-facing**.
📄 CodeRabbit inference engine (.claude/rules/sdk-packages.md)
Files:
packages/trigger-sdk/skills/trigger-cost-savings/SKILL.md
Source excerpt: Use Mintlify MDX format.
📄 CodeRabbit inference engine (.claude/rules/docs-writing.md)
Files:
docs/webhooks/deliveries.mdxdocs/webhooks/waiters.mdxdocs/how-to-reduce-your-spend.mdxdocs/limits.mdxdocs/webhooks/overview.mdx
Applicability reference: AGENTS.md: # AGENTS.md Docs live in `docs/` as a Mintlify site (MDX format).
📄 CodeRabbit inference engine (docs/AGENTS.md)
Files:
docs/webhooks/deliveries.mdxdocs/webhooks/waiters.mdxdocs/how-to-reduce-your-spend.mdxdocs/limits.mdxdocs/webhooks/overview.mdx
🪛 SkillSpector (2.12.0)
packages/trigger-sdk/skills/trigger-cost-savings/SKILL.md
[warning] 43: [RP1] MCP server referenced without pinned version: 'npx trigger.dev'.: npx commands without a version suffix (e.g. @1.0.0) create a rug-pull risk if the upstream server is compromised and publishes a malicious update.
Remediation: Pin the version: npx @scope/server@1.2.3
(MCP Rug Pull (RP1))
🔇 Additional comments: 2
docs/webhooks/overview.mdx: 1
92-92: 🗄️ Data Integrity & IntegrationConfirm the webhook-limit rollout before publishing these values.
The page presents the plan limits as active behavior. The implementation resolves limits from organization configuration over environment defaults, but the available source does not show which production organizations have these values or whether
webhook-limits-backfillhas run. Confirm the deployed configuration and backfill state. Otherwise, label the limits as planned or beta.docs/limits.mdx: 1
185-190: 🗄️ Data Integrity & IntegrationThe active Cloud values and
webhook-limits-backfillstatus cannot be determined from the repository. The resolver reads environment defaults and organization overrides, but no deployed Cloud configuration or backfill record is present. The publication concern therefore remains undecidable from the available evidence.
- Filters only skip subscribers: a waiting run still resumes - Runs are billed an invocation cost plus compute - Enterprise rows for processing concurrency and delivery retention - Qualify the 30-day storage claim for strict retention - Scope the waiter table's live-waiter defaults to self-hosted - Grammar fixes in limits and the changeset
🦋 Changeset detectedLatest commit: d2bad11 The changes in this PR will be included in the next version bump. This PR includes changesets to release 30 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
@trigger.dev/build
trigger.dev
@trigger.dev/core
@trigger.dev/linear
@trigger.dev/python
@trigger.dev/react-hooks
@trigger.dev/redis-worker
@trigger.dev/rsc
@trigger.dev/schema-to-json
@trigger.dev/slack
@trigger.dev/sdk
commit: |
Adds webhooks pricing and per-plan limits to the docs ahead of the webhooks launch on Mon Oct 26 (TRI-14179).
Pricing
Webhooks have no separate charge. The runs a delivery starts or resumes are billed as normal compute (Eric in #project-webhooks, Oct 20). Filtered deliveries start no run, and waiting runs are suspended, so neither uses compute.
Changes
docs/limits.mdx: new sections for webhook waiters, processing concurrency, delivery retention and subscribers per endpoint (values fromenterprise/apps/billing/app/plans/v3.server.tsandMAX_WEBHOOK_SUBSCRIBERS_PER_ENDPOINT). The existing ingest-rate tables and the#webhook-ingest-rate-limitsanchor are unchanged.docs/webhooks/overview.mdx: new "Pricing and limits" section.docs/webhooks/deliveries.mdx: new "Retention" section.docs/webhooks/waiters.mdx: links the plan-dependent limits to the new table.docs/how-to-reduce-your-spend.mdx: new tip, "Filter webhooks before they start runs".trigger-cost-savingsskill: one bullet on webhook filters, plus a patch changeset for@trigger.dev/sdk.mint broken-linkspasses.To confirm before merging
webhook-limits-backfillhas run).