README headless-verification list is stale — `xp.js` and `strategic.js` not mentioned #8

New issue

Closed

opened 2026-09-21 22:20:39 +00:00 by Mark · 1 comment

Mark commented

2026-09-21 22:20:39 +00:00

Owner

The README's ## Headless verification section enumerates 5 of the 6 node-based smokes (smoke 0/1/2, saveload, teams, research). It does not mention tools/xp.js or tools/strategic.js. CI (.forgejo/workflows/ci.yml) runs both, plus the others. Real verification coverage is broader than the README claims; either update README to match CI, or trim CI to match README.

What to do

Update the ## Headless verification section to list all seven tools verbatim from ci.yml. Include a one-liner per tool about what it asserts (already in each tool's header comment — copy from tools/*.js first lines). Keep the node tools/smoke.js 2 call with its current comment about intentionally losing by design.

Acceptance

Diff ## Headless verification against the actual .forgejo/workflows/ci.yml run: lines: every CI invocation appears in the section with a brief assertion summary.

Labels

documentation

The README's `## Headless verification` section enumerates 5 of the 6 node-based smokes (smoke 0/1/2, saveload, teams, research). It does not mention `tools/xp.js` or `tools/strategic.js`. CI (`.forgejo/workflows/ci.yml`) runs both, plus the others. Real verification coverage is broader than the README claims; either update README to match CI, or trim CI to match README. ## What to do Update the `## Headless verification` section to list all seven tools verbatim from `ci.yml`. Include a one-liner per tool about what it asserts (already in each tool's header comment — copy from `tools/*.js` first lines). Keep the `node tools/smoke.js 2` call with its current comment about intentionally losing by design. ## Acceptance Diff `## Headless verification` against the actual `.forgejo/workflows/ci.yml` `run:` lines: every CI invocation appears in the section with a brief assertion summary. ## Labels `documentation`

Mark added the

documentation

label

2026-09-21 22:20:39 +00:00

Mark added the

triage-approved

label

2026-09-22 01:47:32 +00:00

Mark referenced this issue from a commit

2026-09-22 15:53:08 +00:00

cybertactics#8: refresh README headless-verification list

Mark closed this issue

2026-09-22 15:56:01 +00:00

Mark commented

2026-09-22 15:56:07 +00:00

Author

Owner

Direct-fix landed at b6f0211 on origin/main. The README's ## Headless verification section now mirrors every run: line in .forgejo/workflows/ci.yml (smoke 0/1/2, saveload, teams, research, strategic, xp, playwright, godot_import_check). Each one-liner was refreshed from the matching tool's own header comment. The smoke 2 line is preserved with its 'loses by design' comment per the issue body. Closing the closing-paragraph 'all seven' wording too.

Direct-fix landed at b6f0211 on origin/main. The README's `## Headless verification` section now mirrors every `run:` line in `.forgejo/workflows/ci.yml` (smoke 0/1/2, saveload, teams, research, strategic, xp, playwright, godot_import_check). Each one-liner was refreshed from the matching tool's own header comment. The smoke 2 line is preserved with its 'loses by design' comment per the issue body. Closing the closing-paragraph 'all seven' wording too.