From e431207aa492b1bc3f861153c55d5182959258f6 Mon Sep 17 00:00:00 2001 From: Claude Date: Thu, 3 Sep 2026 02:55:24 +0000 Subject: [PATCH] docs(devx): re-aim two stale comments in cross-package-test-inputs.mjs MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit `scripts/cross-package-test-inputs.mjs`'s `@objectstack/spec` declaration block carried two comments PR #14557 made stale: - the `.claude/skills/spec-property-retirement/SKILL.md` entry said the retirement pin (`retired-key-migrate-sentence.test.ts`) judges "the ONE governed markdown file" — after #14557 the pin judges that file PLUS every `skills/**/*.md`. "One file, not `.claude/**`" was still literally true about the `.claude` entry while reading as an exhaustive scope claim about the test's population, so the sentence needed re-aiming, not a bumped count: it now names the two-corpus structure and points at the `skills/**` entry below for the second corpus. - the `skills/**` entry named `scripts/export-list.test.ts` as its only holder. `retired-key-migrate-sentence.test.ts` reads the same root (`PUBLISHED_SKILLS_ROOT`, `:102`) as its second corpus — "plus every `.md` file under `skills/`" (`:92`), emitted as `skills:`-prefixed entries (`:484-485`). The comment now names both holders. Comments only: every glob is unchanged, `check:cross-package-test-inputs` was already green on #14557's head, and it stays green here. Fixes #14560 Co-Authored-By: Claude Fable 5.1 Claude-Session: https://claude.ai/code/session_01WLJQhde67SeTccsmnBVarV --- scripts/cross-package-test-inputs.mjs | 31 +++++++++++++++++---------- 1 file changed, 20 insertions(+), 11 deletions(-) diff --git a/scripts/cross-package-test-inputs.mjs b/scripts/cross-package-test-inputs.mjs index 1a6c0ce3b9..4efb8bfbbb 100644 --- a/scripts/cross-package-test-inputs.mjs +++ b/scripts/cross-package-test-inputs.mjs @@ -144,18 +144,27 @@ export const CROSS_PACKAGE_TEST_INPUTS = { // schema files it inventories, so the ledger IS an input to the ratchet. 'content/docs/api/error-catalog.mdx', 'docs/audits/2026-07-unknown-key-strictness-ledger.md', - // src/shared/retired-key-migrate-sentence.test.ts judges the ONE - // governed markdown file its population was widened by (#10848, - // maintainer-ruled): the retirement playbook that teaches authors the - // prescription sentence the pin holds. One file, not `.claude/**`. + // src/shared/retired-key-migrate-sentence.test.ts judges two corpora + // (#10848, maintainer-ruled): this ONE governed `.claude` markdown + // file — the retirement playbook that teaches authors the + // prescription sentence the pin holds — plus every `skills/**/*.md` + // file, declared separately below since that root has its own holder + // too. One file for THIS entry, not `.claude/**`. '.claude/skills/spec-property-retirement/SKILL.md', - // scripts/export-list.test.ts ends in a corpus gate over the PUBLISHED - // skill references — it enumerates `skills/` and reads every - // `/references/_index.md`, asserting none of their `Exports:` rows - // names a machine constant (#12201). The artifacts are what a customer - // agent actually loads, so they are the population that gate must judge; - // checking the generator's output in memory instead would re-assert the - // rule and see nothing about what is checked in. + // Two holders. scripts/export-list.test.ts ends in a corpus gate over + // the PUBLISHED skill references — it enumerates `skills/` and reads + // every `/references/_index.md`, asserting none of their + // `Exports:` rows names a machine constant (#12201). The artifacts are + // what a customer agent actually loads, so they are the population + // that gate must judge; checking the generator's output in memory + // instead would re-assert the rule and see nothing about what is + // checked in. + // + // src/shared/retired-key-migrate-sentence.test.ts also reads this + // root (`PUBLISHED_SKILLS_ROOT`, `:102`) as the second corpus for its + // widened population (#10848, see the `.claude` entry above) — "plus + // every `.md` file under `skills/`" (`:92`), emitted as + // `skills:`-prefixed entries (`:484-485`). // // The whole subtree rather than `skills/*/references/_index.md`: the test // reads the DIRECTORY too (a new skill dir changes its verdict), and a