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.
A tightly written, highly actionable instruction-only skill: a six-step workflow with embedded verification and a recovery-oriented self-check, executable git/gh commands, concrete output shapes, and clean progressive disclosure into two well-signaled reference files that exist and deliver what the links promise. No dimension shows more than trivial room for improvement.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body never explains what Claude already knows (no 'what a PR is', no tool tutorials — it explicitly instructs the opposite: "Explain this change, not the reviewer's tools"), and nearly every sentence states a rule, a concrete example, or an exemption ("Record decisions with their rejected alternatives. This is the highest-value content in most descriptions and it cannot be recovered from the diff"). It assumes competence ("Skip it when a competent reviewer will know where to look") and dedicates an entire step to cutting. This matches the 5 anchor (lean, every token earns its place) rather than the 4, where instances of over-explanation could be trimmed — I could not find any sentence whose only job is padding. | 5 / 5 |
Actionability | Guidance is concrete and executable throughout: copy-paste-ready commands ("git --no-pager log <base>..HEAD", "gh pr view <n> --repo <owner/repo> --json title,commits,files,baseRefName", "gcloud iam roles describe"), a default body shape with exact headings ("## Review guide", "## What changed", "## Validation"), word-count anchors per change size, and quoted example phrasings for the validation section ("Expect one destroy and one create, nothing else"). Per the code_vs_instruction note, an instruction-only skill with this density of specific guidance warrants 5 rather than 4. | 5 / 5 |
Workflow Clarity | A clear numbered sequence (1 collect facts → 2 follow template → 3 write body → 4 reviewer guidance → 5 cut → 6 self-check) with verification built into the workflow (step 1: "Do not forward a claim you have not checked" with a list of specific cheap checks) and a closing self-check checklist that includes an error-recovery loop ("Did the cut take a decision...? ...Put them back"). This matches the 5 anchor (explicit validation steps, feedback loops, checklist); no gaps keep it at 4. | 5 / 5 |
Progressive Disclosure | The bundle structure is clean: two reference files (references/plain-language.md, references/review-guide.md), both real, both one level deep, both linked exactly at their point of use with their content signaled ("See references/plain-language.md for the rules and the exemptions"; "See references/review-guide.md, which opens with a short map of which of its sections you need" — verified true: that file opens with a 'Which part of this file you need' section). The bulky material (plain-language rules, review-guide details) is split out rather than inlined, and the body keeps only the decisions. This is the 5 anchor (clear overview, well-signaled one-level-deep references), not the 4. | 5 / 5 |
Total | 20 / 20 Passed |