address review comments

Author: icecrasher321

apps/sim/lib/webhooks/processor.ts

@@ -1038,7 +1038,7 @@ export async function queueWebhookExecution(
       }
     }
 
-    const headers = Object.fromEntries(request.headers.entries())
+    const { 'x-sim-idempotency-key': _, ...headers } = Object.fromEntries(request.headers.entries())
 
     // For Microsoft Teams Graph notifications, extract unique identifiers for idempotency
     if (
@@ -1216,7 +1216,8 @@ export async function queueWebhookExecution(
     }
 
     if (foundWebhook.provider === 'generic' && providerConfig.responseMode === 'custom') {
-      const statusCode = Number(providerConfig.responseStatusCode) || 200
+      const rawCode = Number(providerConfig.responseStatusCode) || 200
+      const statusCode = rawCode >= 100 && rawCode <= 599 ? rawCode : 200
       const responseBody = (providerConfig.responseBody as string | undefined)?.trim()
 
       if (!responseBody) {

apps/sim/lib/workflows/comparison/compare.test.ts

@@ -433,7 +433,7 @@ describe('hasWorkflowChanged', () => {
       expect(hasWorkflowChanged(state1, state2)).toBe(true)
     })
 
-    it.concurrent('should detect subBlock type changes', () => {
+    it.concurrent('should ignore subBlock type changes', () => {
       const state1 = createWorkflowState({
         blocks: {
           block1: createBlock('block1', {
@@ -448,7 +448,7 @@ describe('hasWorkflowChanged', () => {
           }),
         },
       })
-      expect(hasWorkflowChanged(state1, state2)).toBe(true)
+      expect(hasWorkflowChanged(state1, state2)).toBe(false)
     })
 
     it.concurrent('should handle null/undefined subBlock values consistently', () => {

apps/sim/triggers/generic/webhook.ts

@@ -67,14 +67,16 @@ export const genericWebhookTrigger: TriggerConfig = {
         { label: 'Default', id: 'default' },
         { label: 'Custom', id: 'custom' },
       ],
+      defaultValue: 'default',
       mode: 'trigger',
     },
     {
       id: 'responseStatusCode',
       title: 'Response Status Code',
       type: 'short-input',
-      placeholder: '200',
-      description: 'HTTP status code to return to the webhook caller.',
+      placeholder: '200 (default)',
+      description:
+        'HTTP status code (100–599) to return to the webhook caller. Defaults to 200 if empty or invalid.',
       required: false,
       mode: 'trigger',
       condition: { field: 'responseMode', value: 'custom' },