Content
82%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, high-signal instruction skill: every line carries project-specific review standards, concrete challenge procedures, and explicit escalation rules, with no filler. Its main gaps are the absence of executable command examples and an explicit stepwise sequence with feedback loops, plus an unverifiable external reference.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and entirely project-specific — e.g., 'Existing code and tests show behavior, not intent' and cited standards like 'Fix the cause that makes the bug possible... (fm #5236; pf #11998 #17266)' — with no padded explanation of concepts Claude already knows; the one time-sensitive date (2026-07-28) is confined to the 'Known smells' old-patterns section, so anchor 5 applies rather than 4. | 5 / 5 |
Actionability | Concrete, executable procedures appear throughout — 'apply the exact regression patch in a clean checkout, then run the focused regression and a neighboring control', 'Return `maintainer` instead of deciding these yourself' — but some checks remain directional ('Establish what FastMCP promises from the protocol, released docs, history') and there are no copy-paste commands, matching anchor 4; per the scoring notes code absence is not penalized for an instruction-only skill, so it stays above anchor 3. | 4 / 5 |
Workflow Clarity | The review procedure is well structured: a categorized challenge checklist ('The contract', 'The evidence', 'Closures', ...), an explicit escalation checkpoint ('Return `maintainer` instead of deciding these yourself'), and an embedded verification procedure for reproductions; however there is no explicit end-to-end sequence or output feedback loop, and the body (~57 lines) exceeds the under-50-line simple-skill exception, so anchor 4 fits better than 5 or 3. | 4 / 5 |
Progressive Disclosure | A self-contained single file with well-organized sections and one clearly signaled one-level reference ([review-issue](../review-issue/SKILL.md), which does not resolve within this bundle); the 14-item maintainer-standards catalog is inline reference-style material that could arguably live in a separate file, so anchor 4 fits — good structure with minor organization gaps — rather than 5. | 4 / 5 |
Total | 17 / 20 Passed |