Content
100%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 an exemplar of operational skill writing: terse, fully executable, and structured as a clear four-step delegation workflow with validation checkpoints, fallbacks, exact output formats, and ready-to-run gh/git commands. It also discloses depth correctly by delegating the review policy to REVIEW.md/AGENTS.md rather than inlining it.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and purely operational: it never explains what code review is or how gh/git work, and every section (steps, prompt template, output format, posting commands) is directly load-bearing. The one explanatory paragraph ("Why a subagent") exists to enforce a behavioral constraint — running in a fresh context — rather than to teach, so every token earns its place per anchor 5. Not 4: there is no over-explanation to trim. | 5 / 5 |
Actionability | Guidance is copy-paste executable throughout: concrete commands (`gh pr view <n>`, `git rev-parse <branch>`, `gh pr diff <N>`, `git diff main...<branch>`, `gh api repos/{owner}/{repo}/pulls/{pr}/reviews`), a complete subagent prompt template, and an exact output format covering both the findings and no-findings cases. This matches anchor 5 ('fully executable; copy-paste ready; covers the common cases') better than 4, which anticipates gaps. | 5 / 5 |
Workflow Clarity | The four steps are clearly sequenced with explicit checkpoints: step 1 verifies the PR/branch exists (`gh pr view <n>` / `git rev-parse <branch>`) before delegating; step 2 includes a fallback when the purpose-built subagent type is unavailable; and the subagent template embeds self-validation ("is this definitely a real issue a senior engineer would flag? Discard if uncertain"). This matches anchor 5's 'explicit validation steps; feedback loops for error recovery' — not 4, since no checkpoint is merely implicit. The skill is read-only (posting is a separate opt-in step), so the destructive/batch cap does not apply. | 5 / 5 |
Progressive Disclosure | The body is a well-organized single file (~90 lines) with clearly headed sections, and it externalizes depth exactly where it should: the full review policy lives in REVIEW.md and AGENTS.md files, referenced one level deep and clearly signaled ("the review policy lives in REVIEW.md", "read REVIEW.md first for the policy"). No bundle files exist to misplace, and nothing that belongs in a separate file is inlined — anchor 5. Not 4: navigation and placement have no gaps. | 5 / 5 |
Total | 20 / 20 Passed |