Content
70%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
This is a well-structured security skill with excellent progressive disclosure and clear workflow definitions for different modes of operation. Its main weakness is that actionable code examples are almost entirely deferred to reference files (which aren't provided in the bundle), leaving the main skill with tables and descriptions rather than copy-paste-ready Go code. The content could also be tightened by removing explanations of concepts Claude already understands (e.g., why hardcoded secrets are bad, what STRIDE letters stand for).
Suggestions
Add 2-3 inline executable Go code examples for the most critical patterns (parameterized SQL, safe exec.Command, crypto/rand token generation) so the main skill is actionable without requiring reference file lookups.
Trim explanatory text that Claude already knows — e.g., remove 'Output is predictable — attacker can reproduce the sequence' from the math/rand row, and the STRIDE acronym expansion. Trust Claude's existing security knowledge and focus on Go-specific guidance.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The skill is reasonably efficient but includes some unnecessary explanatory text that Claude already knows (e.g., explaining what STRIDE stands for, explaining why MD5 is bad, explaining what defense in depth means). The tables and quick reference are efficient, but the 'Security Thinking Model' and 'Research Before Reporting' sections could be tighter. The 'Why It Fails' column in anti-patterns sometimes over-explains obvious concepts. | 2 / 3 |
Actionability | The skill provides concrete tool commands (gosec, govulncheck, race detector) and a useful quick reference table with standard library solutions, but lacks executable code examples for the most critical patterns (e.g., no actual Go code showing parameterized SQL queries, no code showing proper exec.Command usage, no crypto/rand token generation example). The detailed code is deferred to reference files which are not provided in the bundle. | 2 / 3 |
Workflow Clarity | The three modes (Review, Audit, Coding) are clearly defined with distinct workflows. The audit mode specifies parallel sub-agents with explicit domains. The 'Research Before Reporting' section provides a clear 4-step investigation workflow with severity adjustment guidance. The review checklist and severity levels provide clear decision frameworks. For a skill that primarily guides security thinking and review processes rather than destructive operations, the validation checkpoints (trace data flow, check upstream validation, document decisions) are appropriate. | 3 / 3 |
Progressive Disclosure | Excellent structure with a clear overview, quick reference table, and well-signaled one-level-deep references to 11 detailed topic files (cryptography, injection, filesystem, etc.). The main file serves as a navigational hub with enough context to act on common cases while pointing to detailed guides for deep dives. Cross-references to related skills are also included. | 3 / 3 |
Total | 10 / 12 Passed |