Content
71%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 an unusually operational, command-dense runbook with good sequencing and explicit attack-confirmation validation. Its weaknesses are the single-file monolithic layout (CVE maps and vendor matrices belong in reference files), one broken repository URL, and the lack of an error-recovery path for failed tests.
Suggestions
Move the KRACK/FragAttacks CVE maps and the per-vendor residual-exposure matrix into a references/ file (e.g. references/cve-map.md) linked from the body, keeping SKILL.md as a lean overview.
Correct the KRACK test-script repository URL (github.com/vanhoef/krackattacks-scripts) and verify fragattacks.py subtest names against the actual tool.
Add a short error-recovery note: what to do when a test reports FAILED, when the channel-based MITM position cannot be established, or when the target disconnects mid-test.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and operational — commands, CVE maps, and vendor notes with almost no filler or explanation of concepts Claude already knows. Minor trimmable padding exists: the 'How it works' prose re-explains KRACK mechanics Claude largely knows, and the git clone/pip/apt install boilerplate could be compressed. The version numbers and dates are functionally essential (patch-detection matrix) rather than gratuitous time-sensitive content, so they don't drag it below the level-4 anchor. | 4 / 5 |
Actionability | Nearly everything is copy-paste-ready: airodump-ng/tshark recon pipelines, concrete `krack-test-client.py` and `fragattacks.py` invocations with named subtests, an awk-based nonce-reuse detector, and evidence-recording code. Falls short of 5 because the KRACK repo URL (`github.com/vanhoej/krackattacks-scripts`) does not match Vanhoef's actual repository, so that command fails as written, and a couple of fragattacks.py subtest names are unverified against the real tool. | 4 / 5 |
Workflow Clarity | There is a clear sequence — viability gate → scope-viability assessment before any active test → per-family test execution → confirmation of success (per-test SUCCESS/FAILED output, pcap nonce-reuse and injected-ICMP evidence) → evidence recording → OPSEC gating — with validation checkpoints at the attack-confirmation stage. It sits at level 4 rather than 5 because there is no error-recovery loop (what to do on a FAILED test, or when the MITM position isn't achieved) and the recon-to-test handoff (how recon output feeds the chosen test) is implicit. | 4 / 5 |
Progressive Disclosure | The single SKILL.md carries everything inline: ~80 lines of CVE maps and per-vendor residual-exposure tables plus both attack families, with no references/ bundle (only external links and sibling-skill mentions). Section headers are clear, but reference-grade material (the CVE tables, vendor patch matrix) is inlined where the level-4/5 anchors expect it split into well-signaled one-level-deep files — matching 'some structure but content that should be separate is inline' better than 'most content is appropriately placed'. | 3 / 5 |
Total | 15 / 20 Passed |