Content
70%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 dense with genuinely non-obvious operational knowledge — pitfalls, lockout thresholds, tool verdicts — and the workflow/checklist structure is excellent. Weaknesses are token efficiency (repeated legal content and motivational padding) and progressive disclosure (a long monolithic body where tool-ladder detail belongs in reference files).
Suggestions
Consolidate the three overlapping legal sections (LEGAL GUARDRAILS, the DeHashed row in the Stage 2 tool table, and 'Legal red lines') into one authoritative section — the plaintext-breach, stop-on-first-hit, and notify-on-lockout points are each stated multiple times.
Trim motivational framing (Verizon DBIR stats, 'that's not testing, it's grinding for lulz') — it argues for the skill's value rather than instructing, and Claude doesn't need convincing.
Move the ~70-line TOOL LADDER & ALTERNATIVES tables to a references/tool-ladder.md and keep the 'Decision shortcuts' inline, so the SKILL.md body stays a lean overview.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly high-value operational content, but with real duplication: the plaintext-breach warning appears in LEGAL GUARDRAILS, the Stage 2 tool table, and 'Legal red lines'; 'stop on first hit' and 'notify the program on lockouts' each appear twice; plus motivational padding (Verizon DBIR stats, 'grinding for lulz'). Not 4: the repetition is more than minor trimming — the same three legal points are stated three times. | 3 / 5 |
Actionability | Concrete commands with exact flags throughout — '--delay 1800 --jitter 60', '--filter strict', '--max-count 1000000', '--with-linkedin', '--dry-run' — plus mode/filter tables and a working theHarvester wrapper fix. Not 5: some invocations are abbreviated ('theHarvester ... -f theharvester') and full copy-paste-ready sample commands per stage are not provided. | 4 / 5 |
Workflow Clarity | Clear 4-stage pipeline with a sequencing diagram, parallelization guidance, pre-flight checklist with '--dry-run' validation, explicit abort criteria during spray ('If status codes get weird... assume detection and abort'), and before/during/after checklists. The destructive/batch cap does not apply because validation checkpoints are present; matches the 5 anchor (explicit validation, feedback loops, checklists). | 5 / 5 |
Progressive Disclosure | No bundle files exist (no references/, scripts/, or assets/), so everything is inlined in a ~380-line body: the ~70-line Tool Ladder tables and deep pitfall detail are content that clearly belongs in separate reference files. Section structure itself is good with clear headers. Not 4: the split into reference files hasn't been made at all. | 3 / 5 |
Total | 15 / 20 Passed |