Greptile Overview

Greptile Summary

Fixed a bug where duplicating trigger blocks would cause the duplicate to reference the original webhook database row, leading to incorrect webhook associations.

The fix introduces a constant WEBHOOK_SUBBLOCK_FIELDS containing ['webhookId', 'triggerPath'] and filters these fields out in three key locations:

• Duplication flow (use-collaborative-workflow.ts): Excludes webhook fields when creating duplicate block payload and merging subblock values
• Collaborative operations (use-collaborative-workflow.ts): Prevents webhook fields from being applied when receiving 'add' and 'duplicate' socket operations
• State merging (utils.ts, server-utils.ts): Skips webhook fields in both client and server-side merge operations

This ensures duplicated trigger blocks generate new webhooks instead of pointing to the original block's webhook.

Confidence Score: 5/5

• This PR is safe to merge with minimal risk
• The fix is surgical and targeted - it consistently excludes webhook-specific fields across all relevant code paths using a well-defined constant. The implementation is defensive (using includes() checks) and follows the existing patterns in the codebase. The changes were manually tested and don't introduce any breaking changes to existing functionality.
• No files require special attention

Important Files Changed

File Analysis

Sequence Diagram

sequenceDiagram
    participant User
    participant UI as Workflow UI
    participant Hook as useCollaborativeWorkflow
    participant SubBlockStore
    participant WorkflowStore
    participant Socket as Socket Server

    User->>UI: Duplicate trigger block
    UI->>Hook: duplicateBlock(sourceId)
    
    Note over Hook: Get source block & subblock values
    Hook->>SubBlockStore: Get subblock values for sourceId
    SubBlockStore-->>Hook: All subblock values (including webhookId, triggerPath)
    
    Note over Hook: Filter webhook-specific fields
    Hook->>Hook: Filter out webhookId & triggerPath from subBlockValues
    Hook->>Hook: Delete webhookId & triggerPath from mergedSubBlocks
    
    Note over Hook: Create duplicate with new ID
    Hook->>Socket: Emit 'duplicate' operation
    Socket->>WorkflowStore: Create new block (without webhook fields)
    Socket->>SubBlockStore: Set subblock values (excluding webhook fields)
    
    Note over Socket: Apply subblock values (collaborative)
    Socket->>SubBlockStore: Skip webhookId & triggerPath during setValue
    
    WorkflowStore-->>UI: New block created
    Note over UI: Duplicated block has no webhook reference<br/>New webhook will be created when saved

_{3 files reviewed, no comments}

_{Edit Code Review Agent Settings | Greptile}