docs(modules): correct F3 in the acceptance record #148

Merged
whitlocktech merged 1 commits from docs/kit-acceptance-f3-correction into main 2026-08-12 22:42:17 +00:00
Member

Follow-up to #147, which merged while the Integration-kit half was still being written.

The correction. F3 said both release.yml flavours carry the publishing placeholders. Only the Gitea one does — the GitHub flavour uses GITHUB_REPOSITORY, GITHUB_SERVER_URL and friends and has nothing to rename. Found while actually fixing it (Integration-kit#4), where it became one checklist row rather than two.

And what that implies, recorded in the document. Every finding in the report was re-derived before it was acted on, and that was not ceremony: this one was wrong in a detail, and F5 turned out to understate its problem in a way that changed the fix (a " does not raise the parse error the report assumed — it silently truncates the value while the generator prints Success). A cold agent's report is evidence, not a verdict, which is the standard the rest of this plan already applies to a passing test suite.

Separate PR because #147 was already merged, and a push to a merged branch is silent — the trap that produced docs#142 in Phase 4.


  • AI-assisted: written with Claude Code (Claude Opus 5); commits carry a Co-Authored-By trailer.
Follow-up to #147, which merged while the `Integration-kit` half was still being written. **The correction.** F3 said *both* `release.yml` flavours carry the publishing placeholders. Only the Gitea one does — the GitHub flavour uses `GITHUB_REPOSITORY`, `GITHUB_SERVER_URL` and friends and has nothing to rename. Found while actually fixing it (`Integration-kit#4`), where it became one checklist row rather than two. **And what that implies, recorded in the document.** Every finding in the report was re-derived before it was acted on, and that was not ceremony: this one was wrong in a detail, and F5 turned out to *understate* its problem in a way that changed the fix (a `"` does not raise the parse error the report assumed — it silently truncates the value while the generator prints `Success`). A cold agent's report is **evidence, not a verdict**, which is the standard the rest of this plan already applies to a passing test suite. Separate PR because #147 was already merged, and a push to a merged branch is silent — the trap that produced `docs#142` in Phase 4. --- - [x] AI-assisted: written with Claude Code (Claude Opus 5); commits carry a `Co-Authored-By` trailer.
wtclaude added 1 commit 2026-08-12 22:41:56 +00:00
Only the Gitea release workflow carries the publishing placeholders. The report
said both flavours; the GitHub one uses GITHUB_REPOSITORY and friends and has
nothing to rename. Found while fixing it in the kit (Integration-kit#4), which is
where the single checklist row landed.

Also records what that implies about a run like this: every finding was
re-derived before being acted on, and it was not ceremony — this one was wrong in
a detail and F5 understated its problem in a way that changed the fix. A cold
agent's report is evidence, not a verdict, which is the standard the rest of the
plan already applies to a passing suite.

Separate PR because #147 was already merged; a push to a merged branch is silent.

Co-Authored-By: Claude <noreply@anthropic.com>
whitlocktech merged commit 33838bdf10 into main 2026-08-12 22:42:17 +00:00
whitlocktech deleted branch docs/kit-acceptance-f3-correction 2026-08-12 22:42:17 +00:00
Sign in to join this conversation.
No description provided.