Skip to content

Fix template/doc accuracy: CLAUDE_MD table entry, README steps, unconditional self-review - #8

Merged
dmccoystephenson merged 1 commit into
mainfrom
fix/template-and-doc-accuracy
May 17, 2026
Merged

dmccoystephenson merged 1 commit into
mainfrom
fix/template-and-doc-accuracy

Conversation

@dmccoystephenson

@dmccoystephenson dmccoystephenson commented May 17, 2026 •

Copy link
Copy Markdown
Member

Summary

  • Add CLAUDE_MD row to the Step 4 substitution table — the {{#if CLAUDE_MD}} conditional in the template had no guidance on how to resolve it
  • Update README "What it does" to list all 7 steps; previously omitted Steps 6 (create GitHub repo) and 7 (record in example-skills-catalog)
  • Move Phase 4 self-review outside {{#if REVIEWER}} so it is always present; repos with no configured reviewer previously had no code review step at all

Test plan

  • Open create-dev-loop.md and verify CLAUDE_MD appears in the substitution table
  • Verify Phase 4 template shows Perform a self-review: unconditionally, with the reviewer-addition block above it conditionally
  • Open README.md and verify "What it does" lists 6 items including create-repo and record-skill
  • No unresolved {{placeholders}} remain (every placeholder in the template now has a substitution table row)

Closes #5
Closes #6
Closes #7


drafted by Claude on behalf of Daniel Stephenson

…ditional self-review

- Add CLAUDE_MD row to substitution table (closes #5)
- Update README to list all 7 steps including create-repo and record-skill (closes #6)
- Move Phase 4 self-review outside {{#if REVIEWER}} so it always runs (closes #7)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

@dmccoystephenson dmccoystephenson left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Self-review: no issues found. All three changes are correct — CLAUDE_MD table row is accurate, README step list matches the 7 actual steps, and Phase 4 self-review is cleanly separated from the conditional reviewer block.

@dmccoystephenson
dmccoystephenson merged commit 89b995a into main May 17, 2026
@dmccoystephenson
dmccoystephenson deleted the fix/template-and-doc-accuracy branch May 17, 2026 21:32
dmccoystephenson added a commit that referenced this pull request Aug 3, 2026
…ditional self-review (#8)

- Add CLAUDE_MD row to substitution table (closes #5)
- Update README to list all 7 steps including create-repo and record-skill (closes #6)
- Move Phase 4 self-review outside {{#if REVIEWER}} so it always runs (closes #7)

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

1 participant