Freegle / Iznik / 24680

73%

push

circleci

fix(ci): don't let a plans-only commit cancel the real build below it

A plans-only / yesterday-only commit's own build is halted (skipped) by the
orb's "Check if only plans or yesterday files changed" step - that's intended,
and correct on its own. The bug was in the auto-cancel logic: the check-runner
"Cancel older redundant pipelines" passes cancelled ALL older running workflows
for the branch, including a still-running real build for the code the plans sit
on top of. So when a plans/ commit landed seconds after a merge whose
build-test was still running, that real build was cancelled and the plans-only
build reported success without ever running the tests - master went green
without being tested.

Guard both cancel passes: if this commit changes only plans/ or yesterday/
(same detection the orb's skip uses), skip cancellation so the older real build
survives and finishes testing the code. Plans-only commits still skip their own
build, so the optimisation is preserved - they just no longer take a real build
down with them.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_014ZeG2zmg2JCR8pYEjAUaAk

12329 of 16289 branches covered (75.69%)

Branch coverage included in aggregate %.

130928 of 183406 relevant lines covered (71.39%)

37.86 hits per line