Content
67%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 standards skill: concrete, named technologies and commands, a sequenced workflow with a CI verification checkpoint, and a clean one-level-deep reference bundle. The main issues are redundancy between the Workflow and Context-Specific sections and a dangling 'Remediation anchors' list that belongs in a reference file.
Suggestions
Deduplicate the Zero Trust / least-privilege guidance between the Workflow and Data Safeguarding sections so each concept appears once.
Move the bare 'Remediation anchors' term list into the VULNERABILITY_REMEDIATION reference (or expand it with context inline) instead of leaving it as an unexplained section.
Add a short feedback loop to the workflow for what to do when SAST/DAST scanners flag issues before merge (fix, re-scan, only merge when clean).
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly efficient rule statements, but there is avoidable duplication: "Zero Trust: Never trust external input. Sanitize and validate every data boundary" repeats Workflow step 2 ("Validate and sanitize all external input at each boundary"), "Least Privilege" appears both as Workflow step 3 and as a Data Safeguarding bullet, and the "Remediation anchors" section is a bare term list ("Argon2id, parameterized queries or ORM, rate limiting, HttpOnly Secure cookies") duplicating concepts already covered. This fits the 'mostly efficient but could be tightened' anchor — not 4 because the duplication and the dangling anchor list are real slack; not 2 because there is no padding or explanation of concepts Claude already knows. | 3 / 5 |
Actionability | The guidance is concrete and executable for an instruction-only skill: "AES-256 for data-at-rest; TLS 1.3 for data-in-transit", "`npm audit`, `pip audit`", "`WHERE id = ${userId}` always wrong", "Argon2id", "HttpOnly Secure cookies", "MFA". It is not 5 because inline examples of the correct patterns (e.g., a parameterized query) are deferred entirely to references, leaving minor gaps; it is well above 3 because these are specific technologies and commands, not vague direction. | 4 / 5 |
Workflow Clarity | The four-step workflow (identify trust boundaries → validate/sanitize → least privilege → "Verify with SAST/DAST scanners in CI before merge") has a clear sequence with a terminal verification checkpoint, and the 'Always-Apply on every code write' framing is unambiguous. It is not 5 because there is no feedback loop (what to do when a scanner flags an issue before merge) and not 3 because an explicit validation checkpoint is present and the steps are well defined. | 4 / 5 |
Progressive Disclosure | All three referenced files exist in references/ (INJECTION_TESTING.md, VULNERABILITY_REMEDIATION.md, implementation.md), are one level deep with no hidden chains, and are clearly signaled with descriptive titles under a References section plus one contextual inline link. It is not 5 because the "Remediation anchors" section duplicates reference-level detail inline and the cross-link inside INJECTION_TESTING.md makes navigation slightly less clean; it is above 3 because the split between always-apply rules and detailed reference material is well judged. | 4 / 5 |
Total | 15 / 20 Passed |