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 |
