Content
78%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-architected umbrella skill: the engine-generic workflow is genuinely engine-generic, per-engine depth is correctly pushed to real, verified reference files, and navigation is unambiguous. The main costs are a redundant ASCII routing diagram that repeats the routing table, a short concept-explainer opener, and no inline harness example or explicit verify-before-campaign checkpoint.
Suggestions
Drop the ASCII routing diagram in 'Choosing your fuzzer' - the characteristic-to-engine table and the routing-rationale bullets immediately below it already carry the same information in less space (conciseness).
Include one minimal complete harness example (e.g. a 10-line libFuzzer LLVMFuzzerTestOneInput) inline in Step 1 so the core authoring task is executable before jumping to the reference files (actionability).
Add an explicit pre-campaign validation checkpoint in Step 4, such as verifying the instrumented binary reports coverage on the seed corpus before starting the long run, plus what to check when a sanitizer build fails (workflow_clarity).
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and assumes competence, but the opening paragraph explains the coverage-guided mutation loop (a concept Claude already knows) and the ASCII routing diagram substantially duplicates the routing table immediately below it. This matches the 4 anchor (efficient with minor over-explanation to trim); it is above 3 because no section is padded, and below 5 because the diagram/table redundancy and concept-explainer could be cut. | 4 / 5 |
Actionability | Real, executable specifics throughout: `-fsanitize=fuzzer,address,undefined -fno-sanitize-recover=all`, `-merge=1` / `afl-cmin`, `./fuzz_target -max_total_time=300 fuzz/corpus fuzz/seeds`, a copy-paste CI YAML block, and named structured-input helpers per engine. Matches the 4 anchor (mostly executable with minor gaps); the gap is that the core authoring task defers entirely - "Exact harness syntax per engine is in the reference files" - leaving no complete inline harness example, which keeps it from 5. | 4 / 5 |
Workflow Clarity | Five explicitly sequenced steps with concrete per-step guidance and some checkpoints ("Run locally until coverage plateaus", "minimise every crash input before filing it", 3-5 min CI smoke bound). Matches the 4 anchor (clear sequence, most checkpoints present); it falls short of 5 because there are no explicit verify/recover feedback loops, e.g. confirming the harness builds and collects coverage before launching a campaign, and above 3 because the sequence and stopping criteria are explicit rather than implicit. | 4 / 5 |
Progressive Disclosure | The umbrella file stays at routing/workflow altitude while all nine referenced files exist, are one level deep (siblings link to each other and back to SKILL.md, never deeper), and are signaled three ways: the engine table, inline at the relevant workflow step, and a closing References list with per-file purpose notes. Matches the 5 anchor (clear overview, well-signaled one-level-deep references, appropriate split). | 5 / 5 |
Total | 17 / 20 Passed |