From fe2387bcf47b72ce3652a10ee58766f7ecf436fe Mon Sep 17 00:00:00 2001 From: Stella Test Date: Thu, 6 Aug 2026 04:22:26 -0700 Subject: [PATCH] =?UTF-8?q?fix(gate):=20finish=20the=20count-check=20remov?= =?UTF-8?q?al=20=E2=80=94=20the=20guard's=20header=20still=20advertised=20?= =?UTF-8?q?it?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit #1894 (via #1891) deleted check-gate-parity.sh's spelled-out step-count check — the one shared cell two concurrent guard-adding PRs collided on twice on 2026-08-06 (#1883) — but left the header's 'What it checks' list naming the deleted check as item 3. Stale comments are bugs: drop the ghost item, renumber, and record the total under 'deliberately does NOT check', pointing at the rationale block #1894 added in the body. Refs #1883 Refs #1437 --- scripts/check-gate-parity.sh | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/scripts/check-gate-parity.sh b/scripts/check-gate-parity.sh index de0963c80..fc917dbdc 100755 --- a/scripts/check-gate-parity.sh +++ b/scripts/check-gate-parity.sh @@ -26,13 +26,14 @@ # 2. Every step is named in CONTRIBUTING.md's gate fence, via the alias table # below (that fence lists raw commands, on purpose — its reader wants to # run them without make). -# 3. Both documents' spelled-out count matches the real number of steps. -# 4. CONTRIBUTING.md's fence does not run a `check-*.sh` that is no longer a +# 3. CONTRIBUTING.md's fence does not run a `check-*.sh` that is no longer a # gate step — which is how a removed guard leaves a ghost behind. Only # that fence is checked this way: it is a delimited list of commands, # whereas AGENTS.md's block is prose with parenthetical glosses. # -# What it deliberately does NOT check: the prose *around* the lists. Whether +# What it deliberately does NOT check: the spelled-out step total — that check +# existed and was removed; see "The count is deliberately NOT checked any +# more" below (#1883) — and the prose *around* the lists. Whether # "ci.yml's required job runs everything except invariants and doc-links" is # still true is a claim about a workflow, not about this list, and pretending a # grep could settle it would be worse than leaving it to review.