Skip to content

Correct stale CI scope and version-comment claims in RESEARCH.md - #174

Merged
dmccoystephenson merged 1 commit into
mainfrom
fix/research-stale-ci-and-header-claims
Nov 4, 2026
Merged

dmccoystephenson merged 1 commit into
mainfrom
fix/research-stale-ci-and-header-claims

Conversation

@dmccoystephenson

Copy link
Copy Markdown
Member

Summary

Two stale claims in RESEARCH.md were corrected, both found by a Stage A documentation sweep.

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, and scripts/check_docs.py, with CLAUDE.md read for restated conventions. All ship dates cited in RESEARCH.md were also compared with GitHub's mergedAt, and no mismatch was found. The README.md, CONTRIBUTING.md, SECURITY.md, .github/ templates, and scripts/README.md needed no change.

Not covered: RESEARCH.md citation 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:

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.
  • CI scope was confirmed by reading .github/workflows/ci.yml, and the comment placement by reading create-dev-loop.md:124-127.
  • Behavioral validation is not applicable: neither create-dev-loop.md nor the template was changed, so no /create-dev-loop run was needed.

Deferred backlog

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

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@dmccoystephenson

Copy link
Copy Markdown
Member Author

Self-review rubric, scored against head 6c16067 with doc-consistency green. This review was performed inline in the same session that wrote the change, so it is not independent.

Universal items:

Repo-specific items:

  • Placeholder parity: PASS. check_docs.py passed, and the Step 4 rows were also compared with the template by hand in both directions, since the script checks only one. The two placeholder mentions in the diff are unchanged context.
  • README Step parity: PASS (not applicable). No Step was changed.
  • Fence escaping: PASS (not applicable). create-dev-loop.md is untouched.
  • Phase numbering: PASS (not applicable). create-dev-loop.md is untouched.
  • Version comments: PASS. The new note was checked against create-dev-loop.md:124-127.
  • Research grounding: PASS. The PR body states that no finding applies. No finding was shipped, so no Implementations entry was added.
  • No back-ported specifics: PASS (not applicable). create-dev-loop.md is untouched.
  • gardener interface: PASS (not applicable). Steps 3, 5, and 6 are untouched.
  • Retrofit note: PASS (not applicable). No rule was promoted.
  • Behavioral validation: PASS (not applicable). Neither the Steps nor the template were changed.

Weakest evidence: the PR #33 note attributes the move to #36 on the authority of CLAUDE.md, not from the commit history. The PR #54 title lists #36 in its range, which is consistent with that attribution. No do-not-auto-merge path is matched.

This self-review was drafted during a Gardener session (https://github.com/Stephenson-Software/gardener).


drafted by Claude on behalf of Daniel Stephenson

@dmccoystephenson
dmccoystephenson merged commit 627482d into main Nov 4, 2026
2 checks passed
@dmccoystephenson
dmccoystephenson deleted the fix/research-stale-ci-and-header-claims branch November 4, 2026 02:42

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 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.py and 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

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.

2 participants