Content
77%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 strong, disciplined audit workflow whose validation machinery (confidence gate, FP filter, pre-emit quote gate) is the best part — sequence and checkpoints are exemplary. The main weakness is structure: a ~330-line monolith whose exclusion lists, precedents, and OWASP table would serve better as one-level-deep reference files, and phases beyond secrets archaeology rely on pattern checklists rather than executable commands.
Suggestions
Move the 22 hard exclusions, 12 precedents, and OWASP Top 10 mapping into references/ files (e.g. references/fp-rules.md, references/owasp.md) with one-line pointers from Phase 12 and Phase 9, keeping SKILL.md as an overview.
Add copy-paste-ready grep/audit commands for the other high-signal phases (CI/CD workflow YAML checks, webhook signature-verification greps, LLM sink patterns) in the style of Phase 2's git log commands.
Tighten narrative passages such as the 'Framework-meta nudge' paragraph and the Phase 8 rationale into single-sentence rules to reclaim tokens.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and instruction-only — no explanations of concepts Claude already knows, and severity/FP rules are stated as terse clauses ('devDependency CVEs cap at MEDIUM'). It matches the 4 anchor ('efficient; minor instances of over-explanation that could be trimmed'): passages like the 'Framework-meta nudge' paragraph and the Phase 8 rationale ('Installed agent skills are executable prompt code, not documentation') could be tightened. Not 5 because a few narrative paragraphs pad beyond what the rules themselves require. | 4 / 5 |
Actionability | Phase 2 provides copy-paste-ready git log/grep commands, and other phases name exact search patterns ('InsecureSkipVerify', 'dangerouslySetInnerHTML', 'pull_request_target', 'sk-ant-'), plus a concrete report template. It matches the 4 anchor ('mostly executable guidance with minor gaps'): only Phase 2 has literal commands, and Phases 3-8 give pattern checklists rather than executable audit commands. Not 5 because the common cases beyond secrets archaeology lack copy-paste command examples. | 4 / 5 |
Workflow Clarity | Phases 0-13 are clearly sequenced, modes are tabulated with gate thresholds, and validation is the skill's spine: a confidence gate, a pre-emit verification gate ('If you cannot quote it, the finding is unverified'), a per-finding-type verification table, independent verifier subagents, and explicit skip reporting. This matches the 5 anchor — clear sequence with explicit validation steps and feedback loops for error recovery. | 5 / 5 |
Progressive Disclosure | The single ~330-line file has clear section headers, but content that would benefit from being in separate reference files is fully inlined: the 22-item hard-exclusion list, the 12 precedents, the OWASP mapping table, and the data classification scheme. No references/ or scripts/ bundle files exist at all. This matches the 3 anchor ('some structure but could be better organized; content that should be separate is inline') — better than 2 because the structure is strong and nothing is buried, but short of 4 because substantial reference material is monolithically inlined. | 3 / 5 |
Total | 16 / 20 Passed |