The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		May 16, 2026 2:14am

PR Summary

Medium Risk
Adjusts workflow drift/diff detection to treat locked as UI-only metadata, which could change when diffs are reported and suppress prior alerts. Scope is limited and covered by updated/added unit tests around comparison, normalization, and diff-engine behavior.

Overview
Prevents toggling a block’s locked state from being treated as a workflow change/diff.

This removes locked from block field comparisons in both the client-side comparison path (generateWorkflowDiffSummary / normalization hashing) and the server-side diff engine (hasBlockChanged/computeFieldDiff), and updates tests to assert that only locked changes yield no edited blocks, no locked in changed_fields, and no diff-summary changes while still detecting real edits when locked changes alongside functional fields.

^{Reviewed by Cursor Bugbot for commit 261dda2. Configure here.}

Greptile Summary

This PR excludes the block locked field from all workflow diff and comparison paths so that toggling a block's locked state is treated as UI metadata rather than a meaningful workflow change.

• Removes locked from hasBlockChanged and computeFieldDiff.fieldsToCheck in diff-engine.ts, from blockFields in compare.ts, and strips it via destructuring in extractBlockFieldsForComparison in normalize.ts — covering every comparison entry point including the normalised-hash path used for drift detection.
• Adds/updates tests in all three suites, including a combined test asserting that a real field edit alongside a locked toggle still surfaces the real edit (with locked absent from changed_fields).

Confidence Score: 5/5

Safe to merge — the change is a targeted, consistent exclusion of a UI-only field across every comparison entry point, with no risk to real-field change detection.

The exclusion of locked is applied symmetrically in all four places that compare block state: the hasBlockChanged gate, computeFieldDiff, the blockFields guard in generateWorkflowDiffSummary, and the extractBlockFieldsForComparison normalisation used for both the diff summary and the workflow hash. The combined test confirms genuine changes are still detected correctly.

No files require special attention.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Block locked toggle] --> B{Diff detection path}
    B --> C[hasBlockChanged\ndiff-engine.ts]
    B --> D[extractBlockFieldsForComparison\nnormalize.ts]
    C -->|locked check removed| E[Block NOT in editedBlocks]
    D -->|locked stripped from blockRest| G[normalizedStringify comparison]
    G -->|outer guard does not fire| I[locked NOT in diff summary]
    D --> J[normalizeWorkflowState hashing]
    J -->|locked excluded from hash| K[Drift hash unchanged for locked-only toggle]

_{Reviews (1): Last reviewed commit: "fix(workflows): exclude block locked fro..." | Re-trigger Greptile}