Content
93%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, dense with concrete detection indicators, explicit about scope exclusions, severity, and output format, with sound prompt-injection defenses for reviewing untrusted input. The only real improvement is making the implicit scan→triage→report sequence explicit with a verification step before high-severity findings.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and assumes Claude's competence: it names 'base64, hex, char codes', '~/.npmrc', 'pnpm overrides', 'onlyBuiltDependencies', 'GITHUB_*' without explaining what any of them are, and every section (untrusted-input framing, threat categories, exclusions, severity, output format) earns its place. Nothing here is padding or a concept Claude already needs explained, so it matches the anchor-5 example's economy rather than the 'could be tightened' anchor 3 or 'minor trimming' anchor 4. | 5 / 5 |
Actionability | For an instruction-only skill the guidance is fully actionable: exact APIs ('eval, new Function, vm, dynamic require/import of computed names'), exact paths ('~/.npmrc', '~/.config', '/proc'), exact hooks ('postinstall', 'prepare'), exact CI variables ('CI', 'GITHUB_*'), a defined three-level severity scale, and a concrete output spec ('file and line, what is hidden or risky, and what a maintainer should check'). Per the code_vs_instruction scoring note, absence of code is not penalized when guidance is this specific, and the indicator list covers the common cases comprehensively — the anchor-5 condition. | 5 / 5 |
Workflow Clarity | The skill is single-purpose with an unambiguous framing question ('Answer one question: could this change hide behavior a reviewer reading the diff would not see...') and a coherent implicit flow (scan categories → exclude non-qualifying issues → assign severity → report), which fits anchor 4's 'clear sequence with minor gaps'. It is not 5 because the sequence is implicit (organized by topic, not steps), there is no verification checkpoint (e.g., 'confirm the encoded blob is actually decoded before rating it high'), and the under-50-line simple-skill exception does not apply to this 63-line body; it is not 3 because the destructive/batch validation cap is inapplicable to this read-only task and the reporting contract is explicit. | 4 / 5 |
Progressive Disclosure | There are no bundle files (references/, scripts/, assets/ are absent) and the body references none, so there is nothing misfiled or nested: the ~63 lines of screening criteria are appropriately self-contained in SKILL.md with clear sections (Untrusted input, look-for categories, scope exclusions, severity, output) and easy navigation. Splitting this into reference files would add indirection without benefit, so it matches anchor 5's 'content appropriately split; easy navigation' rather than anchor 4, which requires organization gaps that are not present. | 5 / 5 |
Total | 19 / 20 Passed |