Content
88%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 well-engineered skill body: highly actionable phased workflow with explicit verification gates, a concrete severity model, and exemplary one-level-deep reference structure gated by phase and target domain. The main defects are the ~15 lines of Base Framework / workflow-integration scaffolding that could be trimmed, and 11 dangling `shared/` links pointing at files absent from the bundle.
Suggestions
Fix or remove the 11 dangling links to `shared/base/*.md` policy files and `shared/workflow-contract.md` — none exist in the bundle, so a reader following the "Required packaged policies" section hits dead paths; either package those files or drop the section.
Trim the "Base Framework" block (lines 13-16): its precedence and untrusted-content rules duplicate the "Evidence, secrets, and execution safety" section, so keeping one authoritative version would save tokens without losing guidance.
Move the "Optional Workflow Integration" and "Portability note" sections into existing references (e.g., portability.md) to shorten the always-loaded SKILL.md body.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and mostly earns its tokens: "Verify before you report — false positives are the cardinal sin", the severity table with concrete examples ("auth bypass (`jwt.decode` not `verify`)"), and the report template all add non-obvious guidance with no explanation of concepts Claude already knows. Not 5 because the "Base Framework" block largely duplicates the "Evidence, secrets, and execution safety" section, and the "Optional Workflow Integration" and "Portability note" sections are peripheral scaffolding that could be trimmed or offloaded to references. | 4 / 5 |
Actionability | For an instruction-only skill the guidance is fully concrete: 11 sequenced phases each enumerating exact checks ("missing object-level checks (IDOR)... every query scoped to the session's tenant, RLS correctness"), worked micro-examples ("an unauthenticated attacker can set `role=admin` in the signup body because the handler passes `req.body` straight to `User.create`"), named tools per task (Semgrep, Gitleaks, `npm audit`, Trivy), and a complete copy-paste-ready report template. Specific examples cover the common audit cases. | 5 / 5 |
Workflow Clarity | Clear ordered sequence ("Work the phases in order — understand and model before you judge, verify before you report") with explicit validation checkpoints and feedback loops: trace source→sink before reporting, "If you cannot verify... report it as a question", "Each CRITICAL and HIGH finding must be individually traced", assumptions made explicit with conditional findings, and "Read the target end-to-end once before flagging anything". Error-recovery paths (downgrade or drop unprovable findings, conditional-on-absent-mitigation framing) are explicit throughout. | 5 / 5 |
Progressive Disclosure | Structure against the actual bundle is excellent for the 13 references/*.md files: one level deep, each inline-linked at its phase ("read a reference when its phase begins and the target actually touches that domain, and skip references the target never touches") plus a consolidated "Reference map" table, and spot-checks confirm the files exist with real content. Not 5 because the "Required packaged policies" section links 10 `shared/base/*.md` files plus `shared/workflow-contract.md` that do not exist anywhere in the bundle — 11 dangling links that break navigation. | 4 / 5 |
Total | 18 / 20 Passed |