Content
75%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 well-structured, lean overview: a compact catalogue table, scenario-driven guidance with concrete tool invocations and example arguments, and a clear 5-step workflow ending in a validation-minded checkpoint. Remaining improvements are trimming a few motivational sentences and moving the catalogue/reference-agent detail into one-level-deep bundle files.
Suggestions
Trim motivational flourishes (e.g. "Don't reinvent wheels..." and "Copy the report structure that got paid...") to pure guidance.
Add failure-path guidance to the workflow: what to do when ref_suggest returns nothing relevant or a ref_fetch fails or the grepped repo lacks the expected content.
Move the 13-entry catalogue table and the reference-agent descriptions into a references/catalogue.md file, keeping SKILL.md as a lean overview with one-level-deep pointers.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean: a 13-row table instead of prose, one-line tool invocations, and no explanation of concepts Claude already knows (no 'what SSRF is', no library comparisons). Minor padding remains — e.g. "Don't reinvent wheels — if one of them solved a problem elegantly, borrow the pattern..." and "Copy the report structure that got paid — it's validated by triagers" are motivational flourishes that could be trimmed. Fits anchor 4 (efficient, minor trimmable instances); not 5 because a few sentences don't earn their tokens, not 2/3 because there is no real over-explanation. | 4 / 5 |
Actionability | Concrete, copy-pasteable tool calls with realistic arguments appear throughout: ref_suggest(vuln_class="...", goal="recon"), payload_search(vuln_class="ssrf", keyword="imds"), ref_fetch("payloads-all-the-things") with target path /workspace/.references/... and a ~30s timing note, ref_grep with example queries ("DNS rebinding", "CVE-2024-1234"), plus the covered vuln-class list. Mostly executable with minor gaps — e.g. no note on the size/cost of ref_fetch("hackerone-reports") versus the ~30s PayloadsAllTheThings note. Anchor 4, not 5, for those small gaps; not 3 because the commands are concrete and complete rather than pseudocode. | 4 / 5 |
Workflow Clarity | A clear 5-step workflow (identify class → ref_suggest → pick tier → ref_grep → record borrowed patterns as hypothesis nodes) is present, and it ends with an explicit checkpoint: "Record any borrowed payloads / patterns as `hypothesis` nodes in the graph until validated". The operations are read-only lookups, not destructive/batch, so the cap of 3 does not apply. Minor validation gaps (no guidance on what to do when ref_suggest returns nothing relevant, or when a fetched repo lacks the expected content) hold this at anchor 4 rather than 5; sequence and one checkpoint are present, so not 3. | 4 / 5 |
Progressive Disclosure | The skill is a single self-contained file (no references/, scripts/, or assets/ bundle exists, and the body points only to external repos fetched at runtime, not to nested files) — so there is no multi-level reference nesting to penalize. Sections (catalogue, when-to-use per scenario, workflow, reference architectures) are well organized and navigable, and the catalogue is compact enough to justify inlining. It sits at anchor 4 (good structure, most content appropriately placed, minor gaps): the 13-entry catalogue table and the six reference-agent descriptions could arguably live in separate one-level-deep files for a leaner SKILL.md; not 5 because the file is ~75 lines, beyond the under-50-line simple-skill exception. | 4 / 5 |
Total | 16 / 20 Passed |