Content
70%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 a well-structured, actionable analysis methodology with a strong sequenced workflow and validation loop. Its weaknesses are noticeable padding (the rationalizations table and repeated detection-pattern lists) and a progressive-disclosure promise that is broken because none of the referenced files actually exist in the bundle.
Suggestions
Ship the referenced files under references/ (crypto-apis.md, config-patterns.md, auth-patterns.md, case-studies.md, lang-*.md, language-specific.md) or remove the dead references from the body.
Trim or relocate the 'Rationalizations to Reject' table and consolidate the repeated per-category 'Detection patterns' lists to reduce tokens spent on persuasion Claude does not need.
Move the long language-specific footgun tables entirely into the referenced language files so the SKILL.md overview stays lean.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient domain guidance, but it carries padding that Claude does not need: the argumentative 'Rationalizations to Reject' table, repeated 'Detection patterns' lists, and rhetorical 'Questions to ask' sections could be tightened. | 3 / 5 |
Actionability | Concrete detection patterns (parameters like 'algorithm', 'mode', 'cipher'), real code examples in PHP/Python/Go/YAML contrasting safe vs unsafe usage, and specific probing questions give mostly executable guidance with minor gaps. | 4 / 5 |
Workflow Clarity | A clear four-phase workflow (Surface Identification, Edge Case Probing, Threat Modeling, Validate Findings) with explicit validation in Phase 4 (reproduce misuse, verify exploitability), a feedback loop ('return to Phase 2'), and a final Quality Checklist. | 5 / 5 |
Progressive Disclosure | The References section is well organized (by category and by language, one level deep), but the bundle contains no references/ directory, so all ~15 referenced files (crypto-apis.md, config-patterns.md, lang-*.md, etc.) are dead links and navigation is broken. | 3 / 5 |
Total | 15 / 20 Passed |