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 strong, well-sequenced investigation methodology with genuinely concrete tooling (grep commands, a worked example, good/bad fix code, a report template) and explicit validation gates — workflow clarity is excellent. Its weaknesses are structural rather than substantive: a malformed duplicated frontmatter block at the top of the body, an inlined patterns section that re-teaches standard Django/DRF knowledge, and a monolithic single-file layout where reference material (patterns, report template, OWASP-derived content) could be split into bundle files.
Suggestions
Delete the stray second YAML frontmatter block and "--- LICENSE" fragment at the top of the body; move that license/attribution info into a proper LICENSE or frontmatter field.
Trim or move the "Common Django Authorization Patterns" section into a references/ file (e.g. references/django-patterns.md) — it restates standard Django/DRF mechanisms Claude already knows and costs ~40 lines of context on every load.
Consider splitting the report format template and OWASP-derived reference material into a references/ directory, leaving SKILL.md as the investigation workflow itself.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The core methodology (investigation over pattern matching, the five phases) is tight and earns its tokens, but there is unnecessary content: a stray duplicated YAML frontmatter block with a "--- LICENSE" fragment at the top of the body, and a ~40-line "Common Django Authorization Patterns" section that re-explains standard Django/DRF mechanisms (permission_classes, @permission_required, has_object_permission) Claude already knows. This matches anchor 3 (mostly efficient but includes some unnecessary explanation or could be tightened) rather than 4, since the duplicated frontmatter is pure noise and the patterns section is padding. | 3 / 5 |
Actionability | The body provides copy-paste-ready grep commands for discovering the authorization model, a fully worked IDOR investigation example on a concrete endpoint, explicit good-vs-bad fix code, and a complete report template. This matches anchor 4 (mostly executable guidance with minor gaps): the worked example traces a hypothetical codebase rather than being runnable, and some guidance ("trace the flow") is necessarily process-level rather than executable code. | 4 / 5 |
Workflow Clarity | Five phases are clearly sequenced (understand the model → map attack surface → investigate → trace → report) with explicit validation checkpoints: a hard gate ("Do not proceed until you understand the authorization model"), a confidence-level table that prevents reporting unconfirmed findings, a feedback loop ("If you can't find a check, is there one you missed?" with specific places to re-check), and a closing investigation checklist. This matches anchor 5 (clear sequence, explicit validation steps, feedback loops, checklist). | 5 / 5 |
Progressive Disclosure | This is a single ~340-line file with no bundle files at all (no references/, scripts/, or assets/ directories exist), despite a comment attributing material to the OWASP Cheat Sheet Series. Section headers are clear, but content that would naturally live in separate reference files (the Common Django Authorization Patterns reference, the report format template) is inlined, and the stray duplicated frontmatter adds structural noise. This matches anchor 3 (some structure, content that should be separate is inline) rather than 4, which would require most content appropriately split. | 3 / 5 |
Total | 15 / 20 Passed |