Content
92%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.
An exemplary instruction-only skill body: lean, policy-dense, with a clearly sequenced review workflow, an explicit pre-response validation loop on findings, and correctly split one-level-deep reference files with conditional loading guidance. The only notable gap is the absence of a worked example showing the expected finding format.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The 40-line body is dense and padded nowhere: it explains no framework basics and every sentence adds review-specific policy (e.g. "Treat compatibility and safety claims as proof obligations", "accepting an old input alone does not prove backward compatibility"), fully assuming Claude's competence. | 5 / 5 |
Actionability | Concrete, executable instruction-only guidance — numbered boundary steps, an explicit output spec ("name the affected file and tight line range, explain the failure path and impact, and propose a proportionate correction"), and specific validation behaviors ("test ordering and boundary conditions") — with the minor gap that no worked example of a formatted finding is provided, so it stops short of anchor 5's copy-paste-ready bar. | 4 / 5 |
Workflow Clarity | The sequence (establish boundary → review by risk → load references conditionally → output) is clear and capped by an explicit validation checkpoint with recovery actions: "Before responding, challenge and deduplicate the candidate findings. Drop any claim defeated by a guard, ownership rule, version constraint, test, or framework guarantee." Error paths are handled ("If there are no actionable findings, say so directly") and the read-only scope means the destructive/batch cap does not apply. | 5 / 5 |
Progressive Disclosure | A lean overview with two well-signaled, one-level-deep references (both verified to exist as substantive files with no nested links), each carrying an explicit loading condition ("Read [Flutter review checklist]... only for the technical domains touched by the change"), matching the clear-overview anchor exactly. | 5 / 5 |
Total | 19 / 20 Passed |