Content
72%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
A well-structured security skill body with concrete, executable defenses per vulnerability and excellent progressive disclosure via verified one-level-deep references. Its main weaknesses are table redundancy that hurts token efficiency and a missing validate-fix-retry feedback loop in the batch Audit workflow.
Suggestions
Consolidate the overlapping Quick Reference, Common Mistakes, and Security Anti-Patterns tables to remove duplicated entries (e.g. SQL injection, hardcoded secrets), keeping the highest-value framing and linking out for the rest — this would lift conciseness.
Add an explicit validation feedback loop to Audit mode, e.g. 'after applying a fix in its worktree, re-run govulncheck/gosec/-race on the changed package and only mark resolved when clean; if a finding reappears, reopen', so the batch workflow has a checkpoint.
Trim the inline STRIDE/DREAD explanation in the body since the Threat Modeling reference already covers it; a one-line pointer would reduce token cost without losing navigation.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient — tables of concrete fixes with detailed material offloaded to references — but there is redundancy across the Quick Reference, Common Mistakes, and Security Anti-Patterns tables (e.g. SQL injection and hardcoded secrets recur), and STRIDE/DREAD are explained inline before being pointed to. Not 3 because it could be tightened by deduplicating overlapping tables; not 1 because it assumes Claude's competence and avoids padding with basics. | 2 / 3 |
Actionability | Provides specific, executable solutions per vulnerability — 'database/sql with ? placeholders', 'exec.Command with separate args', 'os.Root', 'crypto/subtle.ConstantTimeCompare' — plus runnable commands like 'go tool gosec ./...', 'go tool govulncheck ./...', 'go test -race ./...', 'go test -fuzz=Fuzz'. Not below 3 because guidance is concrete and copy-paste ready rather than abstract. | 3 / 3 |
Workflow Clarity | Modes (Review/Audit/Coding) and the four-step 'Research Before Reporting' are clearly sequenced, with verification tooling (gosec, govulncheck, -race, fuzz) present. Capped at 2 because the batch Audit workflow — the most destructive/large-scale operation — lacks an explicit validate-fix-retry feedback loop for findings (it aggregates and reports, with no 're-verify the fix' checkpoint), which the rubric penalizes for batch operations. | 2 / 3 |
Progressive Disclosure | The body is a concise overview pointing to 12 one-level-deep reference files, each clearly signaled with a named link and one-line description; all referenced paths (cryptography.md, injection.md, filesystem.md, network.md, cookies.md, third-party.md, memory-safety.md, secrets.md, logging.md, threat-modeling.md, architecture.md, checklist.md) exist in ./references/. Not below 3 because navigation is explicit and content is appropriately split. | 3 / 3 |
Total | 10 / 12 Passed |