Content
86%Weight 40%Scale 1-5Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
A tight, well-structured overview that defers operational detail to the frontmatter composition and explains only trigger surface and fallback behavior. It is concise and well-organized; the only soft spot is that the body's actionability and workflow steps are described rather than given as a checkpointed procedure, since the executable content is in the composition block.
Suggestions
Add a brief '## How it runs' sequence in the body (read_diff → review_safety/review_tests/review_style → arbitrate) with the arbitrate priority rule restated as a numbered checklist, so the workflow is checkpointed without duplicating the frontmatter prompts.
Surface the primary trigger phrase ('codereview my diff') in the description text itself, not only in the frontmatter, to strengthen trigger-term coverage.
Clarify the failed-reviewer recovery as an explicit validate→fix→retry step (e.g. 'inspect step_outputs → re-run the failed reviewer step → re-arbitrate') to lift workflow clarity to a 5.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and assumes Claude's competence — it never explains what a code review is or how reviewers work; every sentence ('combinator + arbitrate pattern', the G1.6 contract reference) earns its place. | 5 / 5 |
Actionability | Concrete commands and tokens are present ('git diff --cached HEAD', 'git diff HEAD', 'NO_DIFF', 'PASS_WITH_NOTES: clean'), but the executable review logic itself lives in the frontmatter composition rather than the body, leaving minor gaps for an operator reading only the body. | 4 / 5 |
Workflow Clarity | The sequence (read diff → three parallel reviewers → arbitrate) is clear and the Fallback section gives explicit error-recovery for both NO_DIFF and a failed reviewer step, but the validate/fix/retry loop is framed in prose rather than as an explicit checkpointed checklist. | 4 / 5 |
Progressive Disclosure | A short, well-organized body (~30 lines) with clear Trigger surface and Fallback sections and no need for external references, satisfying the under-50-lines simple-skill exception for progressive disclosure. | 5 / 5 |
Total | 18 / 20 Passed |