Content
81%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 thorough, highly actionable security skill with executable code, clear sequenced workflows, and explicit validation checkpoints for destructive and batch operations. Its main weakness is token efficiency from redundant restating across the checklist, verification, red-flags, and rationalizations sections, and content that is largely inlined rather than split into reference files.
Suggestions
Consolidate the overlapping Security Review Checklist, Verification, Red Flags, and Common Rationalizations sections into a single checklist to remove redundant restating and reclaim tokens.
Move the detailed pattern catalog (OWASP Top 10 Prevention Patterns, Input Validation, Rate Limiting examples) into a referenced reference file linked from the body, keeping SKILL.md as a leaner overview with quick-start examples.
Verify the referenced '../../references/security-checklist.md' is present in the bundle or adjust the path so the one-level-deep reference resolves within the skill's own directory.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly high-signal and skill-specific (TOCTOU gaps, DNS rebinding, marker self-attestation) rather than generic concept explanation, but the ~520-line body restates similar points across the Security Review Checklist, Verification, Red Flags, and Common Rationalizations sections, and includes motivational prose ('Security isn't a phase — it's a constraint on every line of code') that could be tightened. | 3 / 5 |
Actionability | Copy-paste-ready executable TypeScript examples with BAD/GOOD contrasts cover the common cases — parameterized queries, bcrypt hashing, session cookies, helmet/CSP/CORS, zod validation, SSRF allowlisting, rate limiting, and LLM output handling — with specific library calls and configuration values. | 5 / 5 |
Workflow Clarity | Multi-step processes are clearly sequenced with explicit validation checkpoints: the four-step threat-model process, the dependency-triage decision tree, the supply-chain install-boundary sequence ('stop on disagreement or competing lockfiles'), and the destructive-operations sequence (resolve symlinks, verify allowlisted root + minimum depth + ownership before running), plus a final Verification checklist. | 5 / 5 |
Progressive Disclosure | Good internal structure with clear section headers and a well-signaled one-level-deep reference to '../../references/security-checklist.md', but no bundle references/ directory exists in the skill to verify that file, and the bulk of detailed content is inlined in a single large file rather than appropriately split across reference files. | 4 / 5 |
Total | 17 / 20 Passed |