Content
57%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 maximally concise and cleanly structured, but it is so sparse that it provides almost no actionable detail: no commands, no review criteria, no output format. Its companion reference file exists in the bundle yet is never linked, so prioritization guidance it contains is undiscoverable.
Suggestions
Add the concrete mechanics of Step 1, e.g. the command to obtain the diff ('git diff HEAD', 'gh pr diff <n>') and how much surrounding context to read.
Link the existing reference from the body, e.g. 'Prioritize findings per [review-guide.md](references/review-guide.md)', so the bundle's guidance is discoverable.
Specify the report format for Step 2 — severity levels, finding structure (file:line, issue, why it matters), and what to do when no findings exist.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean with zero padding — 'Process steps in order. Do not skip ahead.' plus two two-line steps — and assumes Claude's competence, explaining nothing Claude already knows. It matches 'lean and efficient; every token earns its place'. | 5 / 5 |
Actionability | Guidance is high-level: 'Inspect the complete diff and the surrounding code' and 'Report correctness, security, and test-coverage findings with file locations' give no concrete commands (how to obtain the diff), no criteria for severity or what counts as a finding, and no output format. This matches 'minimal concrete guidance; high-level hints but missing the specific steps to execute' — above level 1 only because the instructions are imperative and name specific finding categories and locations. | 2 / 5 |
Workflow Clarity | The two-step sequence is present and explicitly ordered ('Proceed immediately to Step 2', 'Finish here'), but there are no checkpoints — nothing telling Claude how to confirm the diff was fully read or how findings should be structured before reporting. It matches 'steps listed but... checkpoints missing or implicit'; it is above level 2 because the sequence is coherent and unambiguous in ordering. | 3 / 5 |
Progressive Disclosure | The body itself is short and well-sectioned, but the bundle's reference file (references/review-guide.md, which contains prioritization guidance like 'Prioritize behavior, security, and missing outcome coverage') is never mentioned or linked from the body. Scoring against the actual bundle structure, this is 'references present but not clearly signaled' — an orphaned reference and a clear organization gap; it stays at 3 rather than 2 because the inline content itself is appropriately placed and minimal. | 3 / 5 |
Total | 13 / 20 Passed |