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 |

---