Content
70%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.
The body is a well-sequenced, actionable workflow with a strong feedback and triage loop, undermined by redundancy across three sections that repeat the same review triggers. Its single external dependency is broken — code-reviewer.md is absent from the bundle — which weakens both actionability and navigation at the critical dispatch step.
Suggestions
Ship code-reviewer.md in the bundle (e.g. under references/) or inline the reviewer prompt template so the dispatch step is self-contained and the dangling reference is resolved.
Merge 'Integration with Workflows' and 'Red Flags' into 'When to Request Review' and 'Act on feedback' to eliminate the repeated trigger lists and duplicated triage rules.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | When-to-review triggers are stated three times ('When to Request Review', 'Integration with Workflows', and the worked example), and 'Red Flags' repeats the triage rules from 'Act on feedback' (e.g. Critical/Important handling, push-back guidance). The body is mostly efficient but these redundant sections mean it could be tightened noticeably — anchor 3 rather than 4. | 3 / 5 |
Actionability | Concrete bash commands (git rev-parse for BASE/HEAD SHAs), named placeholders with definitions, a specific tool/type ('task' tool, 'general-purpose' type), and severity-based triage make the guidance mostly executable. The gap keeping it from 5: the pivotal reviewer prompt template lives in 'code-reviewer.md', which is not present in the bundle, so the dispatch step cannot be executed as written. | 4 / 5 |
Workflow Clarity | The three-step sequence (get SHAs → dispatch reviewer → act on feedback) is explicit and numbered, with severity checkpoints (fix Critical immediately, Important before proceeding), fix-before-continue rules, and a disagreement/recovery protocol — matching the clear-sequence-with-explicit-checkpoints-and-feedback-loops anchor. | 5 / 5 |
Progressive Disclosure | Section headers are clear and the single external reference is plainly signaled ('fill template at code-reviewer.md', 'See template at: requesting-code-review/code-reviewer.md'), but that file does not exist anywhere in the bundle — navigation dead-ends at exactly the point where the needed detail lives. A dangling reference is more than the 'minor organization gaps' of anchor 4, so it sits at anchor 3. | 3 / 5 |
Total | 15 / 20 Passed |