Content
88%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, operational security-audit skill: dense domain-specific guidance, concrete commands and output templates, and a methodology with explicit validation gates and a fix-approval checkpoint. The only weakness is length — a few passages could be tightened, and some exhaustive bug-pattern catalogs could optionally live in reference files to ease navigation.
Suggestions
Tighten the longest bullets in sections 1 and 3 (e.g., the mass-assignment and OAuth entries) to one clause each; several currently restate the same idea twice.
Consider moving the per-category exhaustive bug-pattern checklists (sections 1-7) into a reference file (e.g., reference/bug-patterns.md) and keeping SKILL.md as a calibrated overview + methodology, improving progressive disclosure.
A few entries explain framework mechanics Claude already knows (e.g., that PrimaryKeyRelatedField is auto-generated); trim these to just the audit signal ('FK in Meta.fields not redeclared and not in read_only_fields => auto PrimaryKeyRelatedField with no tenant filter').
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is information-dense and largely assumes Claude's competence (specific DRF gotchas, exfil chains, framework-specific bypass patterns rather than generic 'what is XSS' explanations), but at ~270 lines some bullets could be tightened and a few entries restate neighboring ideas. It is above the midpoint (3) because padding is minimal, but not level 5 because a few passages could be trimmed without losing clarity. | 4 / 5 |
Actionability | Provides copy-paste-ready commands ('git diff $(git merge-base HEAD origin/main...)', 'gh pr diff <ref>'), a concrete finding output template with an example exploit request, and a precise four-step methodology. Not level 4 because the examples are executable and cover the common audit cases end-to-end. | 5 / 5 |
Workflow Clarity | Clear sequenced methodology with an explicit gate ('If any of those four steps fails, the finding is not real — drop it'), reproducer-test validation ('Run the test before applying any fix and confirm it fails for the expected reason', then re-run after fix), and an approval checkpoint before destructive fixes. Not level 4 because validation and feedback loops are explicit, not merely present. | 5 / 5 |
Progressive Disclosure | Well-organized into clear, prioritized sections (Input, Calibration, seven numbered audit categories, Methodology, Output format) with no dead or nested references; the inline checklist is appropriate for an audit skill that needs the whole catalog in context. Not level 5 because the volume inlined could support a one-level-deep reference split for some exhaustive categories; not level 3 because structure is genuinely good and navigable. | 4 / 5 |
Total | 18 / 20 Passed |