Content
76%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-first skill body with executable migration examples across three ORMs and correctly structured bundle references. Its main weakness is workflow safety: the multi-step destructive workflows (column renames, type changes, batch data migrations) lack validation/verification checkpoints between steps, which the rubric caps at 3.
Suggestions
Add explicit validation checkpoints between the steps of destructive multi-step workflows — e.g. after copying data in the rename workflow, verify row counts or NULL counts ('SELECT COUNT(*) FROM users WHERE full_name IS NULL') before removing the old column.
Add a verification step to the batch data-transformation loop — commit in batches and check updated row counts, or run the transformation inside a transaction with a post-migration verification query before dropping address_string.
Move the rollback-strategy and data-transformation sections into references/details.md so SKILL.md reads as a leaner overview, mirroring how blue-green patterns are already externalized.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and code-first with almost no prose padding — headers like '## ORM Migrations' go straight to executable code with only '// Run:' style comments, and no concepts Claude already knows are explained. It misses 5 because at ~330 lines some sections (rollback strategies, data transformation loop) duplicate what a references file could hold and could be tightened. | 4 / 5 |
Actionability | Fully executable, copy-paste-ready migration files for Sequelize, TypeORM, and Prisma, each with run/rollback commands ('npx sequelize-cli db:migrate:undo', 'npm run typeorm migration:revert', 'npx prisma migrate dev --name create_users'), covering the common cases: create table, add column, rename, type change, data backfill, rollback. Matches the 5 anchor. | 5 / 5 |
Workflow Clarity | Multi-step workflows are clearly sequenced (the 3-step zero-downtime rename, the 4-step type change, the 5-phase blue-green in references) but destructive and batch operations mostly lack validation checkpoints — e.g. '// Step 2: Update application to use new column' and the per-row UPDATE loop in 'Complex Data Migration' have no verify step between phases. Per the rubric's cap, database (destructive/batch) workflows missing validation cannot score above 3, despite the verification in the checkpoint-based rollback example. | 3 / 5 |
Progressive Disclosure | Good section structure (When to Use, ORM Migrations, Schema Transformations, Data Transformations, Rollback Strategies) with a real, one-level-deep, clearly signaled reference — 'references/details.md' exists and is properly linked once at the end. Not 5 because substantial detail (rollback strategies, data transformations, blue-green patterns) is inlined in SKILL.md that could be split out, leaving the overview leaner. | 4 / 5 |
Total | 16 / 20 Passed |