Skip to content

docs: add skill review guidance to CLAUDE.md#460

Open
vanceingalls wants to merge 1 commit intomainfrom
vance/skill-review-guidance
Open

docs: add skill review guidance to CLAUDE.md#460
vanceingalls wants to merge 1 commit intomainfrom
vance/skill-review-guidance

Conversation

@vanceingalls
Copy link
Copy Markdown
Collaborator

@vanceingalls vanceingalls commented Apr 23, 2026

Summary

  • Adds a "Reviewing skill changes" section to CLAUDE.md with guidance for reviewing skills/ diffs
  • Frames review through three lenses: where agents misinterpret, silently skip, or produce wrong output on first attempt
  • Reminds reviewers that agents have no memory between compositions and must survive in 300+ line instruction files

Why

Skill review requires a different mindset than code review — instructions that look correct to a human reader can still fail when an agent processes them. This codifies the review heuristics we've been applying manually.

Test plan

  • Read the added section for clarity and completeness
  • Verified CLAUDE.md renders correctly

🤖 Generated with Claude Code

Copy link
Copy Markdown
Collaborator Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

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.

1 participant