Content
78%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.
A well-structured security skill: dense severity-ranked reference tables, executable tooling commands, a clear three-mode operating model, and an excellent one-level-deep reference bundle that keeps the SKILL.md body lean. The main improvements are trimming concept explanations Claude already knows and adding an explicit post-fix verification loop.
Suggestions
Trim or move to references the background explanation Claude already knows (the defense-in-depth Overview paragraph and the STRIDE acronym expansion), keeping the body to skill-specific guidance.
Add an explicit verification loop to Coding mode, e.g. 'after fixing a finding, re-run gosec/govulncheck/go test -race on the changed package to confirm remediation before reporting it resolved'.
Inline 2-3 short copy-paste fix snippets for the Critical categories (parameterized query, exec.Command with separate args, AES-GCM) so the most common cases are actionable without opening reference files.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly tight, high-signal tables (Quick Reference, Common Mistakes, Anti-Patterns) with details correctly delegated to 12 reference files, but includes a few explanations Claude already knows — 'Security in Go follows the principle of defense in depth… vigilance is still required' and the STRIDE acronym expansion — that could be trimmed. Not a 3 because padding is minor; not a 5 because the Overview paragraph, persona/thinking/orchestration preamble, and known-concept expansions spend tokens without adding skill-specific value. | 4 / 5 |
Actionability | Concrete, executable guidance is present throughout — 'go tool gosec ./...', 'go tool govulncheck ./...', 'go test -race ./...', 'go test -fuzz=Fuzz', and named stdlib solutions like 'crypto/subtle.ConstantTimeCompare', 'os.Root', 'filepath.IsLocal + filepath.Rel' — plus a documented decision-comment convention. Not a 5 because the vulnerability-fix guidance in the body is mostly one-line table entries; the copy-paste-ready fix examples live only in the reference files. | 4 / 5 |
Workflow Clarity | Modes (Review/Audit/Coding) are clearly defined, 'Research Before Reporting' is a numbered 4-step sequence acting as a validation gate on findings, and audit-mode fan-out with per-fix worktree isolation and DREAD scoring is well specified. Not a 5 because there is no explicit re-verify step after a fix (e.g., re-run gosec/-race to confirm remediation) in Coding mode; not a 3 because checkpoints (trace data flow, check upstream validation, severity-adjust-not-dismiss) are explicit rather than missing. | 4 / 5 |
Progressive Disclosure | The body is a genuine overview hub — every one of the 12 referenced files (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) exists in ./references/, is linked with a one-line description, and is exactly one level deep with no nested-reference chains. Navigation is easy and content is appropriately split. | 5 / 5 |
Total | 17 / 20 Passed |