chore(webhooks): remove implementation-detail comments

Author: waleedlatif1

apps/sim/app/api/webhooks/trigger/[path]/route.ts

@@ -80,7 +80,6 @@ export async function POST(
     }
 
     // Parallelize auth verification with preprocessing — they are independent
-    // checkWebhookPreprocessing has its own try/catch and always returns WebhookPreprocessingResult
     const [authError, preprocessResult] = await Promise.all([
       verifyProviderAuth(foundWebhook, foundWorkflow, request, rawBody, requestId),
       checkWebhookPreprocessing(foundWorkflow, foundWebhook, requestId),

apps/sim/background/webhook-execution.ts

@@ -140,7 +140,7 @@ export async function executeWebhookJob(payload: WebhookExecutionPayload) {
 }
 
 /**
- * Resolve the account userId for a credential (deferred from API route to background job)
+ * Resolve the account userId for a credential
  */
 async function resolveCredentialAccountUserId(credentialId: string): Promise<string | undefined> {
   const resolved = await resolveOAuthAccountId(credentialId)
@@ -167,8 +167,7 @@ async function executeWebhookJobInternal(
     requestId
   )
 
-  // Use preprocessExecution to resolve workflow record, billing actor, subscription, and timeout
-  // Rate limits and deployment checks were already done in the API route
+  // Resolve workflow record, billing actor, subscription, and timeout
   const preprocessResult = await preprocessExecution({
     workflowId: payload.workflowId,
     userId: payload.userId,

apps/sim/lib/webhooks/processor.ts

@@ -1086,7 +1086,6 @@ export async function queueWebhookExecution(
       }
     }
 
-    // actorUserId is pre-resolved by checkWebhookPreprocessing and threaded via options
     const actorUserId = options.actorUserId
     if (!actorUserId) {
       logger.error(`[${options.requestId}] No actorUserId provided for webhook ${foundWebhook.id}`)