fix(change-detection): copilot diffs have extra field

[icecrasher321]

Summary

is_diff and field_diffs should not be compared

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	Review	Updated (UTC)
docs	Skipped		Jan 21, 2026 5:57am

[greptile-apps]

Greptile Summary

Fixed false positive change detection by excluding copilot-specific metadata fields (is_diff, field_diffs) from workflow state comparisons.

Key changes:

• Added type definitions BlockWithDiffMarkers and SubBlockWithDiffMarker to handle optional copilot diff markers
• Excluded is_diff and field_diffs from block-level comparisons (lines 91-92, 102-103)
• Excluded is_diff from subBlock-level comparisons (lines 187-188)
• Also excluded outputs field from block comparison as it's derived from subBlocks and already compared via subBlock logic (lines 90, 101)

Confidence Score: 5/5

• This PR is safe to merge with minimal risk
• The change correctly addresses a specific bug where copilot diff markers were causing false positive change detection. The implementation properly uses TypeScript type guards, follows existing destructuring patterns in the function, and maintains backward compatibility since the excluded fields are optional
• No files require special attention

Important Files Changed

Filename	Overview
apps/sim/lib/workflows/comparison/compare.ts	Excluded copilot diff markers (is_diff, field_diffs) from change detection to prevent false positives when comparing workflow states

Sequence Diagram

sequenceDiagram
    participant User
    participant hasWorkflowChanged
    participant BlockState
    participant SubBlockState
    
    User->>hasWorkflowChanged: Compare current vs deployed state
    hasWorkflowChanged->>hasWorkflowChanged: Compare edges
    hasWorkflowChanged->>hasWorkflowChanged: Compare block IDs
    
    loop For each block
        hasWorkflowChanged->>BlockState: Cast to BlockWithDiffMarkers
        BlockState-->>hasWorkflowChanged: Block with optional is_diff, field_diffs
        hasWorkflowChanged->>hasWorkflowChanged: Destructure & exclude:<br/>position, layout, height, outputs,<br/>is_diff, field_diffs
        hasWorkflowChanged->>hasWorkflowChanged: Compare normalized blocks
        
        loop For each subBlock
            hasWorkflowChanged->>SubBlockState: Cast to SubBlockWithDiffMarker
            SubBlockState-->>hasWorkflowChanged: SubBlock with optional is_diff
            hasWorkflowChanged->>hasWorkflowChanged: Destructure & exclude:<br/>value, is_diff
            hasWorkflowChanged->>hasWorkflowChanged: Compare normalized subBlocks
        end
    end
    
    hasWorkflowChanged->>hasWorkflowChanged: Compare loops
    hasWorkflowChanged->>hasWorkflowChanged: Compare parallels
    hasWorkflowChanged->>hasWorkflowChanged: Compare variables
    hasWorkflowChanged-->>User: Return true/false

Loading