Content
75%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 highly actionable, code-dense reference whose examples are immediately executable across PostgreSQL and four migration tools. Its weaknesses are structural: duplicated when-to-use sections, a twice-explained expand-contract pattern, and no explicit post-migration validation/feedback loops for destructive or batch operations.
Suggestions
Merge the duplicate 'When to Activate' and 'When to Use This Skill' sections and drop one of the two expand-contract explanations to cut ~40 lines of redundancy.
Add explicit validation feedback loops for destructive/batch operations, e.g. check `prisma migrate status` / `migrate ... version` after applying, and an 'if dirty: force VERSION then re-run' recovery step.
Move the per-ORM workflow sections (Prisma, Drizzle, Django, golang-migrate) into one-level-deep references/ files, keeping SKILL.md as the overview of core patterns and the safety checklist.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly dense, code-first content, but it duplicates whole sections — "When to Activate" and "When to Use This Skill" repeat nearly the same bullet list, and the expand-contract pattern is fully explained twice ("Renaming a Column (Zero-Downtime)" and "Zero-Downtime Migration Strategy"). This is more than minor trimming, matching anchor 3 ('could be tightened') rather than 4. | 3 / 5 |
Actionability | Fully executable, copy-paste-ready guidance throughout: exact SQL for every pattern (including a batching loop with FOR UPDATE SKIP LOCKED and GET DIAGNOSTICS), per-tool CLI commands ("npx prisma migrate dev --create-only", "migrate -path migrations -database \"$DATABASE_URL\" up"), and complete Django migration classes. This matches the anchor-5 example of executable code covering common cases. | 5 / 5 |
Workflow Clarity | The expand-contract strategy has a clear phased sequence with a day-by-day timeline, and the Migration Safety Checklist plus "Verify data consistency" in Phase 2 act as checkpoints. It is capped below 5 for destructive/batch database operations because there are no explicit fail-and-retry validation loops — only a passing "force VERSION (fix dirty state)" hint — whereas anchor 5 requires explicit validate/fix/revalidate steps. | 4 / 5 |
Progressive Disclosure | No bundle files exist and the single-file body is well-sectioned (per-DB patterns, per-ORM workflows, strategy, anti-pattern table), with each subsection compact enough to justify inlining. It is a 4 rather than 5 because at ~340 lines the four ORM-specific workflow sections could be split into one-level-deep reference files, and the two 'when to use' sections fragment navigation. | 4 / 5 |
Total | 16 / 20 Passed |