Content
56%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.
The body is highly actionable with concrete, executable security code and useful validation checklists, but it is over-long for a SKILL.md and undermines its own progressive disclosure by repeatedly pointing to a `references/security-checklist.md` that is absent from the bundle.
Suggestions
Create the missing references/security-checklist.md referenced on lines 77, 303, and 451 (OWASP 2021 ordering, package-manager matrix, pre-commit verification steps), or remove the dangling ../../references/security-checklist.md pointers so navigation is not broken.
Move the bulk OWASP Top 10 code patterns and the Security Review Checklist into the referenced file so SKILL.md stays a lean overview with one-level-deep references, improving both conciseness and progressive disclosure.
Trim rhetorical framing (e.g. "every secret as sacred", "Security isn't a phase") and condense the large Common Rationalizations table to reduce token load while retaining the actionable checklists and code.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly efficient actionable reference material (code, tables, checklists), but includes rhetorical framing ("every secret as sacred", "Security isn't a phase") and a large Common Rationalizations table that could be trimmed, fitting the "mostly efficient but includes some unnecessary explanation or could be tightened" anchor at 3 rather than the minor-trims-only anchor at 4. | 3 / 5 |
Actionability | It provides extensive copy-paste-ready TypeScript covering the common cases (parameterized queries, bcrypt hashing, httpOnly/secure/sameSite session config, DOMPurify XSS handling, owner-scoped access-control checks, helmet/CSP/CORS, SSRF host allowlist with DNS-resolution checks, zod boundary schemas, rate limiting), fitting the fully-executable anchor at 5. | 5 / 5 |
Workflow Clarity | The "Process: Threat Model First" section gives a clear 4-step sequence and both the Security Review Checklist and final Verification section provide explicit validation checkpoints, fitting the "clear sequence with most checkpoints present" anchor at 4; not 5 because the skill is reference-shaped rather than a single fragile workflow with explicit validate→fix→retry feedback loops. | 4 / 5 |
Progressive Disclosure | Headers give reasonable structure and references to `../../references/security-checklist.md` are clearly signaled, but that referenced file does not exist (no references/ directory is present) and the ~500-line body inlines bulk OWASP patterns and checklists that belong in separate files, fitting the "some structure but content that should be separate is inline" anchor at 3 rather than the well-split anchor at 4. | 3 / 5 |
Total | 15 / 20 Passed |