Skip to content

✨ feat(docs): gate example pack and format claims against dogfood (EX-S01) - #58

Merged
konih merged 5 commits into
mainfrom
lane/ex-s01
Aug 15, 2026
Merged

✨ feat(docs): gate example pack and format claims against dogfood (EX-S01)#58
konih merged 5 commits into
mainfrom
lane/ex-s01

Conversation

@konih

@konih konih commented Aug 15, 2026

Copy link
Copy Markdown
Contributor

EX-S01 from openspec/specs/p5-ex-complex-examples/spec.md. Independent review: REQUEST_CHANGES → fixes → fresh re-review APPROVE (no P0/P1). Gates green on merged head.

konih added 5 commits August 15, 2026 17:52
Paper-gate both polarities so README cannot name a pack or input format
that is not shipped under examples/packs, and a dogfooded pack cannot
go undocumented. Wired through task docs-gates.
… tests-dir mutation (EX-S01 review)

P1-F1: readme_formats tokenized only a hard-coded whitelist, so an
unrecognised claim (e.g. toml) was silently dropped and the gate stayed
green (REQ-EX-S01-04 fail-open). Now the ENTIRE comma-separated list
after 'Input formats:' is tokenized and any token with no matching pack
class match.paths extension reddens with a dedicated message. New toml
polarity assertion added red-first.

P1-F2: the orphan (no .assent/tests/) mutation used GOOD_README, which
omits orphan, so deleting the tests-dir check still reddened via the
packs-list mismatch — the mutation could not detect removal of the
check. The mutation now uses a README variant that lists orphan and
asserts the redden reason is the tests-dir message. Verified: deleting
the check now flips this assertion to FAIL.

P3: deleted the dead bare-tf branch after the exact FORMATS match — the
match guarantees tfvars is present, so the branch could never fire.
@konih
konih merged commit 240cf1f into main Aug 15, 2026
7 checks passed
@konih
konih deleted the lane/ex-s01 branch August 15, 2026 23:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant