Repository navigation
Correct stale CI scope and version-comment claims in RESEARCH.md - #174
Conversation
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Self-review rubric, scored against head Universal items:
Repo-specific items:
Weakest evidence: the PR #33 note attributes the move to #36 on the authority of This self-review was drafted during a Gardener session (https://github.com/Stephenson-Software/gardener). drafted by Claude on behalf of Daniel Stephenson |
There was a problem hiding this comment.
🟢 Approval recommended
There are no active review issues for the documentation-only updates to RESEARCH.md.
No open issues 🎉
What changed in this PR
This documentation-only PR updates two implementation notes in RESEARCH.md to reflect current CI coverage and generated-skill formatting, while preserving historical context.
Changes:
- Clarifies that CI runs both
scripts/check_docs.pyand its unit tests, but does not validate template behavior. - Notes that generated skills’ version comments now appear directly below the H1 heading.
| File | Description |
|---|---|
RESEARCH.md |
Corrects CI scope and clarifies current version-comment placement in historical implementation entries. |
Review effort: Balanced
Summary
Two stale claims in
RESEARCH.mdwere corrected, both found by a Stage A documentation sweep.scripts/check_docs.py. The unit tests of that script are also run by.github/workflows/ci.yml, so that scope was added. The conclusion drawn in the entry is unaffected, since CI still does not exercise the template's behavior.No tracking issue exists for these corrections; they were found during triage.
Sweep coverage
Every Phase 7 source was checked against
create-dev-loop.md,.github/workflows/ci.yml, andscripts/check_docs.py, withCLAUDE.mdread for restated conventions. All ship dates cited inRESEARCH.mdwere also compared with GitHub'smergedAt, and no mismatch was found. TheREADME.md,CONTRIBUTING.md,SECURITY.md,.github/templates, andscripts/README.mdneeded no change.Not covered:
RESEARCH.mdcitation URLs were not re-resolved, and the existing confidence levels were not re-assessed. "Last reviewed" was left unchanged for that reason.Drift that could not be corrected in this PR was filed:
TEMPLATE_VERSIONrow and atCLAUDE.md:53. Neither file can be changed by an autonomous documentation PR.Research grounding
No finding applies. Only existing entries were corrected, and the template was not touched. Finding §1's accuracy is preserved, and no finding was shipped by this PR.
Test plan
python3 scripts/check_docs.py: "Doc consistency check passed."python3 -m unittest discover -s tests -v: 42 tests ran, all passing..github/workflows/ci.yml, and the comment placement by readingcreate-dev-loop.md:124-127.create-dev-loop.mdnor the template was changed, so no/create-dev-looprun was needed.Deferred backlog
CLAUDE.mdedit, for which separate authorization is required.template-ruleissue changecreate-dev-loop.md. Each would be handed off UNVERIFIED and would block later cycles while held open, so a documentation sweep that can be merged was preferred.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