fix(webhooks): count test webhooks towards usage limit

[waleedlatif1]

Summary

• count test webhooks towards usage limit

Type of Change

• Bug fix

Testing

Tested manually

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]

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

1 Skipped Deployment

Project	Deployment	Preview	Comments	Updated (UTC)
docs	Skipped			Dec 1, 2025 4:04am

[greptile-apps]

Greptile Overview

Greptile Summary

Removed the skipUsageLimits parameter that was being set based on testMode in webhook preprocessing, ensuring test webhooks now count towards usage limits.

Key Changes:

• Test webhooks previously bypassed usage limit checks by passing skipUsageLimits: testMode to preprocessExecution()
• After this change, test webhooks go through the full usage validation in checkServerSideUsageLimits()
• Test webhooks still have testMode flag set for other processing logic and logging purposes
• Rate limiting and deployment checks remain active for test webhooks (unchanged)

Confidence Score: 5/5

• This PR is safe to merge with minimal risk - it's a simple, well-scoped bug fix that correctly enforces usage limits on test webhooks
• The change is a single-line removal that closes a loophole where test webhooks could bypass usage limits. The logic is straightforward: removing skipUsageLimits: testMode ensures the default behavior (usage checks enabled) applies to all webhook executions including tests. This aligns with the PR's stated goal and doesn't introduce any breaking changes to the existing flow.
• No files require special attention

Important Files Changed

File Analysis

Filename	Score	Overview
apps/sim/lib/webhooks/processor.ts	5/5	Removed skipUsageLimits: testMode parameter from preprocessing to ensure test webhooks count towards usage limits

Sequence Diagram

sequenceDiagram
participant Client
participant TestAPI as Test Webhook API
participant Processor as Webhook Processor
participant Preprocessing as preprocessExecution
participant UsageMonitor as Usage Monitor
participant Queue as Execution Queue
Client->>TestAPI: POST /api/webhooks/test/{id}
TestAPI->>TestAPI: verifyTestWebhookToken()
TestAPI->>Processor: checkWebhookPreprocessing(testMode=true)
Note over Processor: Before PR: skipUsageLimits=testMode<br/>After PR: skipUsageLimits removed
Processor->>Preprocessing: preprocessExecution({<br/>checkRateLimit: true,<br/>checkDeployment: true<br/>})
Preprocessing->>Preprocessing: Validate workflow exists
Preprocessing->>Preprocessing: Check deployment status
Preprocessing->>Preprocessing: Resolve billing actor
Preprocessing->>Preprocessing: Get user subscription
Preprocessing->>Preprocessing: Check rate limits
alt Before PR (skipUsageLimits=true)
Preprocessing-->>Preprocessing: Skip usage check
else After PR (skipUsageLimits not set)
Preprocessing->>UsageMonitor: checkServerSideUsageLimits(actorUserId)
UsageMonitor-->>Preprocessing: Usage status
alt Usage exceeded
Preprocessing-->>TestAPI: Error 402: Usage limit exceeded
TestAPI-->>Client: 402 Payment Required
end
end
Preprocessing-->>Processor: Success
Processor->>Queue: queueWebhookExecution(testMode=true)
Queue-->>Client: 200 OK: Webhook processed

Loading

[greptile-apps]

_{1 file reviewed, no comments}

_{Edit Code Review Agent Settings | Greptile}