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.
An exemplary skill body: lean, fully executable, scale-aware strategy selection, and explicit validation loops. Fix details are correctly pushed one level deep into two real reference files that match the cited sections.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Every section is instruction or non-obvious operational knowledge (e.g., the "Running eslint on all of static/app/ can take 2+ minutes" tip); there is no padding and no explanation of concepts Claude already knows. Lean and efficient throughout. | 5 / 5 |
Actionability | Fully executable: copy-paste-ready commands parameterized by $0/$1 ("pnpm exec eslint --fix --rule '@sentry/scraps/$0: error' \"$1\""), concrete per-scale strategies, a specific PR title convention, and the exact prek invocation. Not score 4, since no minor gaps in executable guidance exist. | 5 / 5 |
Workflow Clarity | Clear sequence (understand rule → assess scale → choose strategy → fix loop → coordinate → verify) with explicit validation checkpoints: "Re-run after each batch to verify", "Always review the diff after --fix before committing", and a final verification command that "Should report 0 problems". Feedback loops are present for this batch operation, so the batch-cap of 3 does not apply. | 5 / 5 |
Progressive Disclosure | Per-rule fix details are appropriately externalized to references/fix-patterns.md and references/token-taxonomy.md (both exist, one level deep, with the § sections the table cites), surfaced via a well-signaled rule-to-reference table and a MUST-load instruction for the taxonomy case. | 5 / 5 |
Total | 20 / 20 Passed |