Content
96%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 content is a lean, highly actionable runbook with a clearly sequenced workflow and explicit validation feedback loops for destructive migration operations. Its only weakness is progressive disclosure: with no bundle files, a large amount of operational detail is inlined in SKILL.md rather than split into referenced files.
Suggestions
Extract the hot-table and cross-language NOT NULL hazard sections into separate reference files (e.g. references/hot-tables.md, references/cross-language-not-null.md) and link to them from SKILL.md to reduce inlined bulk.
Add a brief 'Quick reference' summary at the top mapping each risky pattern to its helper class and CI guard so Claude can scan before reading the detail.
Move the per-command verification snippets (sqlmigrate / --dry-run checks) into a short consolidated validation checklist section for faster lookup.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and high-signal with no padding explaining what Django or migrations are; it assumes Claude's competence and every section (CI guard names, incident-causing specifics, exact commands) earns its tokens. | 5 / 5 |
Actionability | Provides copy-paste-ready commands (`DEBUG=1 ./manage.py makemigrations`, `sqlmigrate`, `rebase_migration`), concrete code (the `db_default=` field, `RunSQL` example), named helper classes, and named CI/allowlist files covering the common risky cases. | 5 / 5 |
Workflow Clarity | The numbered Workflow (Classify → Generate → Apply safety rules → Validate) is explicitly sequenced with validation checkpoints (sqlmigrate, tests, linear-sequence confirmation) and feedback loops (re-verify no stray `DROP DEFAULT`, `makemigrations --dry-run` state-drift check) for destructive operations. | 5 / 5 |
Progressive Disclosure | Clear section headers and well-signaled one-level references to repo docs at the top, but substantial inlined operational detail (hot table hazard, cross-language NOT NULL hazard) that a bundle could split into separate reference files; no bundle files exist to offload it. | 4 / 5 |
Total | 19 / 20 Passed |