Review Skill
Run specialized reviewers against a skill document to find correctness, clarity, and conformance issues.
Core Philosophy
Every issue demands a document change: real issue → fix it; false positive → add clarifying text.
Reviewers
Correctness — will this do the wrong thing?
| Reviewer | Question | Finds |
|---|---|---|
| execution | "Would this cause wrong behavior?" | Logic errors, missing steps, broken flows (all treated equally; no severity levels) |
| contradictions | "Does A contradict B?" | Conflicting instructions |
| coverage | "Is every option/branch handled?" | Unhandled branches, missing handlers |
Clarity — will this be misunderstood?
| Reviewer | Question | Finds |
|---|---|---|
| adversarial | "Where might an LLM misinterpret this?" | Fixable ambiguities, missing info |
| terminology | "Is term X used consistently?" | Naming inconsistencies |
| conciseness | "Is everything here necessary?" | Verbosity, redundancy, over-explanation |
Conformance — does this follow conventions?
| Reviewer | Question | Finds |
|---|---|---|
| checklist | "Do these specific checks pass?" | Structural issues, missing sections |
| portability | "Would this break on other LLM providers?" | Provider-specific assumptions |
Phases
@lib/... references are expanded inline by the skill loader.
@lib/001_INITIALIZE.md @lib/002_FAN_OUT.md @lib/003_COLLECT.md @lib/004_SYNTHESIZE.md @lib/005_TRIAGE.md @lib/006_HIL_PLAN_APPROVAL.md @lib/007_ADDRESS.md @lib/008_VERIFY.md @lib/009_HIL_CHANGE_CONFIRMATION.md @lib/010_STAGE.md @lib/011_COMMIT.md @lib/012_LOOP_GATE.md @lib/013_EPILOGUE.md
Quick Reference
| Phase | Purpose |
|---|---|
| Initialize | Parse args, validate target |
| Fan Out | Launch all reviewers in parallel |
| Collect | Gather and merge results |
| Synthesize | Group into themes |
| Triage | Propose fixes |
| HIL: Plan Approval | User approves proposed fixes |
| Address | Make edits |
| Verify | Confirm changes |
| HIL: Change Confirmation | User confirms edits |
| Stage | Review and stage changes |
| Commit | Create commit with proper message |
| Loop Gate | Check pass count, loop or exit |
| Epilogue | Report and end |
Flags
| Flag | Behavior |
|---|---|
--auto | Skip HIL checkpoints (still displays plan/summary) |
-n N | Do N passes (default: 1) |
--reviewer <category> | Run only correctness, clarity, or conformance reviewers |