Content
77%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 a dense, fully actionable verification runbook: every command is copy-paste ready, the failure-triage workflow has explicit validation and feedback loops, and nothing wastes tokens on general knowledge. Its weaknesses are density and monolithic structure — bug-history narrative, a findings catalog, and a coverage table all sit inline in one long file with no progressive disclosure.
Suggestions
Move the Findings catalog, the Coverage record table, and the What a pass proves planted-bug details into a references/ file, keeping SKILL.md to the property, how to run, and how to read a failure, and link to them with clearly signaled one-level-deep references.
Tighten 'The harm' and 'What it checks' by splitting their long compound sentences; the prose often packs three or more facts per sentence, which costs the reader more parsing than the token savings are worth.
Add a brief run-order summary near the top (check → read failure → classify → fix → --lower → re-run) so the workflow sequence is visible before the dense detail sections.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Nothing explains concepts Claude already knows — every sentence carries project-specific detail — but sections like 'The harm' (bug-history narrative) and 'What a pass proves' (planted-bug count catalog) run long and could be tightened, matching the mostly-efficient-but-could-be-tightened anchor rather than the clearly verbose one. | 3 / 5 |
Actionability | Fully executable guidance: copy-paste commands ('pnpm check:autoformat-rules', '--length=4', "--typed='<keys>'", '--jobs=<n>', '--lower', '--init'), concrete expected/got failure examples, and exact git commands for the coverage resync cover the common cases. | 5 / 5 |
Workflow Clarity | The workflow has an explicit loop with validation checkpoints: run the gate, read a failure, classify real bug vs modeling error vs intended change before changing code, fix, then 'the gate fails until you run --lower'; the coverage resync names four confirmations and a gate run before moving the sync commit. | 5 / 5 |
Progressive Disclosure | Headers are clear and sections well ordered, but the skill is a single 22 KB file with no bundle: the coverage-record table, the Findings catalog, and the What-a-pass-proves section are sizable reference material inlined in SKILL.md that could live in separate files, matching the some-structure-but-should-be-separate anchor. | 3 / 5 |
Total | 16 / 20 Passed |