Skip to content

Commit 9219437

Browse files
committed
fix(gates): move check-ci-filter-parity's rollback pin to the post-#10848 measurement (10 -> 11, new member pinned by name)
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01MsbKEG4LtERSLaDrbehM3e
1 parent 61adbc2 commit 9219437

1 file changed

Lines changed: 14 additions & 4 deletions

File tree

scripts/check-ci-filter-parity.mjs

Lines changed: 14 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -563,11 +563,21 @@ export async function selfTest() {
563563
'the checked-in `crosspkg` still names the same-root-different-file entry #10015 added',
564564
);
565565
// The pre-#10015 list, as the measurement that motivated this gate: with the
566-
// four roots removed, the ten declarations #10015 fixed go uncovered here.
566+
// four roots removed, the ten declarations #10015 fixed go uncovered here —
567+
// plus, since #10848, the one post-#10015 declaration none of those roots
568+
// ever covered (the retirement skill's SKILL.md, a `.claude/` literal), so
569+
// the rollback now uncovers eleven. This pin is judged over the LIVE
570+
// declaration table on purpose: a declaration added under a root the
571+
// rollback keeps leaves the count alone, one under a new root moves it and
572+
// is recorded here by name.
567573
const preFix = judge(fixtureWorkflow({ core: real.filters?.core, crosspkg: ['scripts/**'] }), CROSS_PACKAGE_TEST_INPUTS);
568574
assert(
569-
new Set(uncoveredGlobs(preFix)).size === 10,
570-
`rolling \`crosspkg\` back to its pre-#10015 list uncovers exactly the ten -- got ${new Set(uncoveredGlobs(preFix)).size}`,
575+
new Set(uncoveredGlobs(preFix)).size === 11,
576+
`rolling \`crosspkg\` back to its pre-#10015 list uncovers the ten it fixed plus #10848's one -- got ${new Set(uncoveredGlobs(preFix)).size}`,
577+
);
578+
assert(
579+
uncoveredGlobs(preFix).includes('.claude/skills/spec-property-retirement/SKILL.md'),
580+
`-- and the post-#10015 member is #10848's declaration, by name`,
571581
);
572582

573583
// ── (7) WIRING: the gate and its self-test really run in CI ──────────────
@@ -593,7 +603,7 @@ export async function selfTest() {
593603
`same-root-different-file case observed failing and then covered by naming the file, a glob covered by ` +
594604
`\`core\`, one covered only by \`crosspkg\` and one covered by neither judged separately in one table, the ` +
595605
`stale-entry direction, seven refusals over subjects that could not be read, the checked-in ci.yml, the ` +
596-
`pre-#10015 rollback uncovering exactly ten, and the CI wiring read out of lint.yml.`,
606+
`pre-#10015 rollback uncovering the ten it fixed plus #10848's one, and the CI wiring read out of lint.yml.`,
597607
);
598608
return 0;
599609
}

0 commit comments

Comments
 (0)