Content
65%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 dense, command-first reference that is highly actionable for the common active-recon cases, with strong error-handling and pitfalls sections. Its weaknesses are the missing explicit validation checkpoints in the scan workflow (only implicit conditionals), the absence of any progressive-disclosure layering for a ~400-line document, a duplicated quick-reference block, and one incorrect flag (`-On` instead of `-Pn`) in the error-handling table.
Suggestions
Add explicit validation checkpoints to the section-12 workflow, e.g. "Step 0: confirm every target is in the documented scope before sending any packet" and "after port discovery, verify the output file lists open ports before running -sV".
Split the per-service NSE script catalogs (sections 3 and 8) and the nuclei/nikto/testssl flag details (sections 5–6) into references/ files (e.g. references/nse-scripts.md, references/vuln-scanners.md) and link them from a leaner SKILL.md.
Fix the error-handling table: "Add `-On` (skip host discovery)" should be `-Pn`; also deduplicate the Quick Reference block against sections 2, 4, and 5 to trim tokens.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dominated by lean, executable command blocks and reference tables with very little concept explanation Claude already knows, fitting the score-4 anchor ("Efficient; minor instances of over-explanation that could be trimmed"). Not a 5 because of trimmable content: the "Quick Reference — Common Scan Patterns" block duplicates commands that reappear verbatim in sections 2, 4, and 5, and the "Why IPv6 matters" / "Key TLS Findings" prose states well-known security fundamentals. | 4 / 5 |
Actionability | Nearly all guidance is copy-paste-ready with concrete flags ("nmap -sS -sV --top-ports 1000", "ffuf ... -mc 200,301,302,403", "nuclei -u ... -severity critical,high"), matching the score-4 anchor "mostly executable guidance with minor gaps". It falls short of 5 because of at least one incorrect concrete instruction: the error-handling table prescribes "Add `-On` (skip host discovery)" — the correct nmap flag is `-Pn` (`-oN` is an output flag), so that remediation command fails as written. | 4 / 5 |
Workflow Clarity | Section 12 lays out a clear 11-step sequence (Target Selection → Port Discovery → ... → Synthesis) with some conditional logic ("Scan AAAA records if found"), so a sequence is plainly present. However, checkpoints are implicit rather than explicit — there is no verify-scope step before sending packets, no check that the port scan succeeded before version detection, and no validate-results loop inside the workflow; per the rubric's cap guidance for batch operations (network scanning is inherently batch), workflow clarity is capped at 3 ("steps listed but validation gaps; checkpoints missing or implicit"). The pre-scan "Scan Justification" documentation requirement exists in section 1 but is never integrated into the section-12 workflow. | 3 / 5 |
Progressive Disclosure | No bundle files exist (no references/, scripts/, or assets/ directories), so all ~390 lines live inline in SKILL.md. Section structure is good (numbered sections, quick reference, tables), which keeps it above the score-2 anchor, but content that clearly belongs in separate reference files — the per-service NSE script catalogs (sections 3, 8), the nuclei/nikto/testssl flag references (sections 5–6) — is inlined, matching the score-3 anchor "some structure... content that should be separate is inline". The single external pointer ("hand off to `web-recon`") is to another skill, not a bundle file, and is not a markdown link. | 3 / 5 |
Total | 14 / 20 Passed |