Skip to content

Commit 0c98edf

Browse files
committed
docs(gate): re-measure the pinned-set churn over the full commit population
The shipped comment justified this ratchet with "the 269 commits on `main` in the month to 2026-08-18 ... changed in 7 commits (2.6%) ... zero left". 269 was the depth of the shallow clone the measuring agent ran in, not the month's traffic (#9878). A denominator swap would not have been honest: the numerator was computed over the same 269 visible commits, so `7/3,110` would have replaced one unverified number with another inside the comment whose whole job is to justify a merge-blocking gate. Re-measured instead, over the full population. Method (stated in the comment so it can be redone): membership is proxied by `assertEngine…Dispatch(` call sites in test files under the scan roots, calibrated against this gate's own ledger at HEAD — 0 false negatives on the file sets, 309 of 310 (file, verb) rows agreeing on the exact count. Over 3,103 first-parent commits (2026-07-18..2026-08-18): delete changed in 111 commits (3.6%) 162 entered 1 left update changed in 103 commits (3.3%) 160 entered 1 left The rate is HIGHER than the sample reported, which is the direction swapping only the denominator would have hidden. The single departure per verb is one commit, f16e54e, that DELETED a test file and added a replacement carrying both pins in the same commit — repo-wide coverage never dropped. Zero instances of the shape this ratchet catches (file present, verb gone), and zero per-file counts that shrank without reaching zero. Also corrects "308 entries today" to 310 in the same paragraph, which the merge's ledger regeneration moved. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01XqDQYVU5smx29ts9pAErja
1 parent c3f11e2 commit 0c98edf

1 file changed

Lines changed: 37 additions & 8 deletions

File tree

scripts/check-engine-double-contract.mjs

Lines changed: 37 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -177,14 +177,43 @@
177177
// ratchet itself.
178178
//
179179
// The maintenance objection to an enumeration is real but was measured and is
180-
// small. Over the 269 commits on `main` in the month to 2026-08-18, membership
181-
// of the pinned set changed in 7 commits (2.6%): 8 files entered per verb, and
182-
// **zero left**. So the nuisance case an identity ledger is accused of -- a
183-
// correct change reddened by a legitimate decrease -- fired 0 times in a month,
184-
// while the additions it does redden on are already in conversation with this
185-
// gate (they are new fakes that had to write `assert…Dispatch` because PINNED
186-
// demanded it). 308 entries today against the 135 the DEBT ledger already
187-
// carries: same file format, same order of magnitude, same reconciliation shape.
180+
// small. Measured over the FULL population: 3,103 first-parent commits on
181+
// `main` in the month to 2026-08-18 (`git log --first-parent
182+
// --since=2026-07-18 --until=2026-08-18 origin/main`). Membership of the pinned
183+
// set changed in 111 of them for `delete` (3.6%) and 103 for `update` (3.3%);
184+
// 162 and 160 files ENTERED, and exactly ONE left per verb.
185+
//
186+
// ⛔ An earlier revision of this comment read "the 269 commits ... changed in 7
187+
// commits (2.6%) ... zero left". 269 was the depth of the shallow clone the
188+
// measuring agent ran in, not the month's traffic, and the 7 and the "zero"
189+
// were themselves computed over only those 269 commits -- so every number in
190+
// the sentence, numerator included, described a ~9% sample. It is re-measured
191+
// here rather than rescaled: the true rate is HIGHER than the sample reported,
192+
// which is the direction that swapping only the denominator would have hidden.
193+
//
194+
// The single departure per verb is the case worth stating precisely, because it
195+
// is the nuisance an identity ledger is accused of and it did not behave like
196+
// one: f16e54e1d deleted `protocol-delete-object-package-binding-guard.test.ts`
197+
// and added a replacement carrying both pins in the SAME commit, so repo-wide
198+
// coverage never dropped. It lands in the "file gone from disk" world below,
199+
// whose remedy is mechanical and carries no judgement call. Across the whole
200+
// month there were ZERO instances of the shape this ratchet exists to catch --
201+
// file present, verb gone -- and zero per-file counts that shrank without
202+
// reaching zero.
203+
//
204+
// Method, stated so this can be redone rather than trusted: membership is
205+
// proxied by `assertEngine…Dispatch(` call sites in test files under the scan
206+
// roots, calibrated against this gate's own ledger at HEAD -- 0 false negatives
207+
// on the file sets, and 309 of 310 (file, verb) rows agreeing on the exact
208+
// count. Anyone re-running it must first prove their clone actually covers the
209+
// window; a shallow one silently answers for its own depth, which is the whole
210+
// reason this paragraph had to be rewritten.
211+
//
212+
// So the additions the ratchet does redden on are already in conversation with
213+
// this gate (they are new fakes that had to write `assert…Dispatch` because
214+
// PINNED demanded it). 310 entries today against the 135 the DEBT ledger
215+
// already carries: same file format, same order of magnitude, same
216+
// reconciliation shape.
188217
//
189218
// ## How a LEGITIMATE decrease is expressed (the anti-nuisance half)
190219
//

0 commit comments

Comments
 (0)