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 well-structured, highly actionable review skill with an excellent validated-only workflow, strong code examples, and thoughtful false-positive guidance. Its weaknesses are moderate redundancy across the validation sections and a monolithic 400-line body with no reference files to keep SKILL.md as a lean overview.
Suggestions
Move the detailed per-category pattern catalogs (Priorities 1-4) into reference files (e.g., references/n-plus-1.md, references/indexes.md, references/write-loops.md), leaving SKILL.md as a lean overview with the priority table, validation requirements, and output format.
Merge the 'Validation Requirements' section with the per-category checklists (or the closing sanity check) to eliminate the repeated validation guidance.
Complete the unfinished examples: show actual Prefetch/annotate code for the model-property rule and a runnable fix for the ORDER BY indexing rule.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is largely efficient: compact PROBLEM/SOLUTION code pairs and impact tables assume Claude's competence without teaching Django basics. However, there is noticeable duplication — the 'Validation Requirements' section largely restates the per-category checklists, and the closing sanity-check questions repeat the 'What NOT to Report' list — keeping it at anchor 4 ('minor instances of over-explanation that could be trimmed') rather than the lean anchor 5. | 4 / 5 |
Actionability | Most rules ship executable, copy-paste PROBLEM/SOLUTION code (select_related, annotate/Count, bulk_create, bulk_update, db_index, Meta.indexes) and a concrete output-format template. A few sections are incomplete — the model-property rule's fix is a one-line hint ('Use Prefetch with custom queryset, or annotate') with no code, and the ORDER BY example references code without a runnable fix — matching anchor 4 ('concrete code with minor gaps') rather than anchor 5's fully copy-paste-ready coverage. | 4 / 5 |
Workflow Clarity | The multi-step review workflow is clearly sequenced with explicit validation checkpoints: a 4-step Review Approach, per-category validation checklists, 5 numbered Validation Requirements, and a feedback loop in the output format ('If the answer to any is no - remove the finding') plus a pre-submission sanity check. This matches anchor 5's 'clear sequence with explicit validation steps; feedback loops; checklists for complex processes'. | 5 / 5 |
Progressive Disclosure | There are no bundle files at all; the ~400-line body inlines detailed per-category pattern catalogs (N+1 rules, index rules, write-loop rules) that are natural candidates for one-level-deep reference files, with only internal headers for navigation. This matches anchor 3 ('content that should be separate is inline') — better than anchor 2's structureless dump thanks to the priority table and clean sections, but short of anchor 4's appropriate content split. | 3 / 5 |
Total | 16 / 20 Passed |