Repository navigation
Drop the site-count words from the COMPILE_CMD and LINT_CMD rows - #153
Conversation
Closes #152 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Self-review rubric (Phase 4, performed inline — no independent reviewer was available in this headless dispatch; Copilot reviewer request failed with "Could not resolve user with login 'copilot'"):
Out-of-diff observation: the parenthetical site lists in all three rows ( This review comment was drafted during a Gardener session (https://github.com/Stephenson-Software/gardener). drafted by Claude on behalf of Daniel Stephenson |
|
Merge-readiness status:
The PR is left open, not merged, the same way #151 was. The change edits This comment was drafted during a Gardener session (https://github.com/Stephenson-Software/gardener). drafted by Claude on behalf of Daniel Stephenson |
Summary
COMPILE_CMDandLINT_CMDrows are reworded to drop the count words ("both places", "in both") and instead say "everywhere{{COMPILE_CMD}}appears" / "everywhere{{LINT_CMD}}appears", matching the form theTEST_CMDrow took in Enumerate all three TEST_CMD substitution sites in the Step 4 row #151. This is option 1 from the issue.TEST_CMDrow drifted after Require unique scratch filenames and define the structurally-red-anchor path #121.<!-- sites: N -->check inscripts/check_docs.py) was not taken: it introduces a new table convention that is a maintainer decision, andscripts/check_docs.pyis on this loop's do-not-auto-merge list. The parenthetical site lists can still go stale; only option 2 would catch that class.Closes #152
Research grounding
No
RESEARCH.mdfinding applies. This is a documentation-accuracy rewording with no empirical claim behind it, as the issue itself states.Doc sync check
{{placeholder}}added or changed has a corresponding Step 4 substitution-table row — no placeholders were added or removed;python3 scripts/check_docs.pypasses.README.mdis unaffected (no Step added, removed, or renamed). No finding is shipped, so noRESEARCH.mdentry is needed.Test plan
python3 scripts/check_docs.py— "Doc consistency check passed."python3 -m unittest discover -s tests— Ran 13 tests, OK{{COMPILE_CMD}}and{{LINT_CMD}}in the template body re-confirmed (two each)/create-dev-loopbehavioral run — not performed. The change edits Step 4 instruction prose only; no template-body text, placeholder, or Step 3/5/6 path is touched, and the instruction's meaning for the current template is unchanged. The behavioral half is UNVERIFIED in the ordinary sense for this repo.Skipped issues (recorded per Phase 1)
CLAUDE.md, which is harness-blocked in this loop and needs separate maintainer authorization.template-rule) — each proposes a new rule in the template body, which has no automated test and would need a real/create-dev-loopvalidation run this headless session cannot perform. A fully verifiable documentation fix ranked higher on the Phase 2 tiebreaker.This PR description was drafted during a Gardener session (https://github.com/Stephenson-Software/gardener).
🤖 Generated with Claude Code
drafted by Claude on behalf of Daniel Stephenson