fix(bill): add requestId to webhook processing

[waleedlatif1]

Summary

• add requestId to webhook processing

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 5:17am

[greptile-apps]

Greptile Overview

Greptile Summary

removed the unused testMode parameter from checkWebhookPreprocessing function and all its call sites, completing the cleanup from PR #2143 which removed the usage of this parameter

Note: The PR title "add requestId to webhook processing" is misleading - the requestId parameter was already present. The actual change removes the testMode parameter that became unused after PR #2143.

Confidence Score: 5/5

• This PR is safe to merge with no risk
• Simple parameter removal refactoring with all call sites properly updated. The changes are straightforward, complete, and follow the cleanup from PR fix(webhooks): count test webhooks towards usage limit #2143.
• No files require special attention

Important Files Changed

File Analysis

Filename	Score	Overview
apps/sim/lib/webhooks/processor.ts	5/5	removed unused testMode parameter from checkWebhookPreprocessing function signature
apps/sim/app/api/webhooks/test/[id]/route.ts	5/5	updated call to checkWebhookPreprocessing to remove unused testMode argument
apps/sim/app/api/webhooks/trigger/[path]/route.ts	5/5	updated call to checkWebhookPreprocessing to remove unused testMode argument

Sequence Diagram

sequenceDiagram
    participant Route as Webhook Route
    participant Processor as checkWebhookPreprocessing
    participant PreprocessExecution as preprocessExecution
    
    Route->>Processor: checkWebhookPreprocessing(foundWorkflow, foundWebhook, requestId)
    Note over Route,Processor: testMode parameter removed
    Processor->>PreprocessExecution: preprocessExecution({workflowId, userId, triggerType, executionId, requestId, ...})
    Note over Processor,PreprocessExecution: No longer passes skipUsageLimits
    PreprocessExecution-->>Processor: PreprocessExecutionResult
    Processor-->>Route: NextResponse | null

Loading

[greptile-apps]

_{3 files reviewed, no comments}

_{Edit Code Review Agent Settings | Greptile}