Skip to content

Commit 055d7f9

Browse files
claude[bot]claude
andauthored
docs(devx): re-aim two stale comments in cross-package-test-inputs.mjs (#14807)
`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 Claude-Session: https://claude.ai/code/session_01WLJQhde67SeTccsmnBVarV Co-authored-by: Claude <noreply@anthropic.com>
1 parent 5a22459 commit 055d7f9

1 file changed

Lines changed: 20 additions & 11 deletions

File tree

scripts/cross-package-test-inputs.mjs

Lines changed: 20 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -144,18 +144,27 @@ export const CROSS_PACKAGE_TEST_INPUTS = {
144144
// schema files it inventories, so the ledger IS an input to the ratchet.
145145
'content/docs/api/error-catalog.mdx',
146146
'docs/audits/2026-07-unknown-key-strictness-ledger.md',
147-
// src/shared/retired-key-migrate-sentence.test.ts judges the ONE
148-
// governed markdown file its population was widened by (#10848,
149-
// maintainer-ruled): the retirement playbook that teaches authors the
150-
// prescription sentence the pin holds. One file, not `.claude/**`.
147+
// src/shared/retired-key-migrate-sentence.test.ts judges two corpora
148+
// (#10848, maintainer-ruled): this ONE governed `.claude` markdown
149+
// file — the retirement playbook that teaches authors the
150+
// prescription sentence the pin holds — plus every `skills/**/*.md`
151+
// file, declared separately below since that root has its own holder
152+
// too. One file for THIS entry, not `.claude/**`.
151153
'.claude/skills/spec-property-retirement/SKILL.md',
152-
// scripts/export-list.test.ts ends in a corpus gate over the PUBLISHED
153-
// skill references — it enumerates `skills/` and reads every
154-
// `<skill>/references/_index.md`, asserting none of their `Exports:` rows
155-
// names a machine constant (#12201). The artifacts are what a customer
156-
// agent actually loads, so they are the population that gate must judge;
157-
// checking the generator's output in memory instead would re-assert the
158-
// rule and see nothing about what is checked in.
154+
// Two holders. scripts/export-list.test.ts ends in a corpus gate over
155+
// the PUBLISHED skill references — it enumerates `skills/` and reads
156+
// every `<skill>/references/_index.md`, asserting none of their
157+
// `Exports:` rows names a machine constant (#12201). The artifacts are
158+
// what a customer agent actually loads, so they are the population
159+
// that gate must judge; checking the generator's output in memory
160+
// instead would re-assert the rule and see nothing about what is
161+
// checked in.
162+
//
163+
// src/shared/retired-key-migrate-sentence.test.ts also reads this
164+
// root (`PUBLISHED_SKILLS_ROOT`, `:102`) as the second corpus for its
165+
// widened population (#10848, see the `.claude` entry above) — "plus
166+
// every `.md` file under `skills/`" (`:92`), emitted as
167+
// `skills:`-prefixed entries (`:484-485`).
159168
//
160169
// The whole subtree rather than `skills/*/references/_index.md`: the test
161170
// reads the DIRECTORY too (a new skill dir changes its verdict), and a

0 commit comments

Comments
 (0)