Content
77%Weight 40%Scale 1-3Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
The body is highly actionable with complete executable commands and a well-validated install workflow, and sequences its destructive operations with real checkpoints. Its main weaknesses are verbosity (explains security basics Claude knows and inlines non-essential sections) and broken/missing bundle references (lib/, hooks/, test/) alongside content that belongs in separate files.
Suggestions
Remove or trim content that explains concepts Claude already knows (shell-injection prevention rationale, generic severity-prioritization rundown) and move roadmap/contributing/license out of the core skill body.
Resolve the broken bundle references — either add the referenced lib/, hooks/clawsec-scanner-hook/, and test/ files, or correct the architecture diagram and Verification/Development sections to reflect what actually ships.
Split the ScanReport/Vulnerability schema and the long manual install script into a separate reference file referenced one level deep from SKILL.md.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Largely actionable and technical, but the ~485-line body explains concepts Claude already knows (e.g. 'Subprocess arguments use arrays to prevent shell injection', the Vulnerability Prioritization severity rundown) and inlines roadmap/contributing/license content that could be trimmed. Not a 3 due to this padding; not a 1 because it is not a wall of introductory explanation. | 2 / 3 |
Actionability | Provides fully executable, copy-paste-ready guidance throughout — install commands, a complete checksum/signature-verified install script, runner.sh invocations with flags, env-var exports, and YAML/JS examples for custom rules — matching the 'fully executable code/commands; copy-paste ready' anchor. | 3 / 3 |
Workflow Clarity | The destructive install (Option B) is an explicit sequence with validation checkpoints — signature verify, checksum verify with `exit 1` on mismatch, then 'Checksums verified. Installing' — plus a verification checklist and troubleshooting/error-recovery feedback loops. The primary scan flow is more implicit, but the validation and feedback-loop presence meets the score-3 anchor. | 3 / 3 |
Progressive Disclosure | Sections are well-organized and real scripts references are one level deep, but the architecture diagram and sections reference lib/ (report.mjs, utils.mjs, types.ts), hooks/clawsec-scanner-hook/ (HOOK.md, handler.ts), and test/*.test.mjs that do not exist in the bundle, and large inline blocks (roadmap, contributing, license, full install script, security considerations) could be split out. Fits 'some structure but content that should be separate is inline'; not a 1 because it is sectioned and not deeply nested. | 2 / 3 |
Total | 10 / 12 Passed |