Content
53%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 delivers executable, ORM-specific migration examples with real commands, but it is padded with generic instruction boilerplate, its multi-step workflows lack explicit validation checkpoints, and its reference structure is broken: every listed bundle file is missing and the one in-instruction reference uses an inconsistent path. The skill reads as a code cookbook rather than a guided, navigable playbook.
Suggestions
Create the advertised bundle files (references/orm-switching.md, schema-migration.md, etc., assets/schema-migration-template.sql, scripts/test-migration.sh) or remove the Resources section, and fix the 'resources/implementation-playbook.md' path to match the actual layout.
Add explicit validation checkpoints to the destructive workflows (e.g. verify row counts after backfill, test down() on staging, confirm backup exists before running), so the blue-green and rename flows include validate-and-recover loops.
Delete the boilerplate 'Instructions' and 'Do not use this skill when' filler in favor of concrete, skill-specific guidance, and move the bulk of per-ORM examples into the referenced files to slim SKILL.md into an overview.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient code examples, but padded by generic boilerplate instructions ('Clarify goals, constraints, and required inputs', 'Apply relevant best practices and validate outcomes') and repeated Sequelize migration scaffolding across sections. It is not 2 because it never explains concepts Claude already knows; not 4 because the boilerplate and duplicated module.exports shells should be trimmed. | 3 / 5 |
Actionability | Concrete, executable code with real run/rollback commands ('npx sequelize-cli db:migrate:undo', 'npm run typeorm migration:revert', 'npx prisma migrate deploy') covering common cases. Minor gaps keep it below 5: no Prisma rollback command, and 'npm run typeorm' assumes an unstated package.json script. | 4 / 5 |
Workflow Clarity | Multi-step sequences exist (renaming columns in 3 steps, blue-green in 5 phases) but interim steps are only comment placeholders and lack validation checkpoints. Per the rubric's feedback-loop note, destructive database workflows without explicit validate-and-recover steps cap at 3; the transaction and checkpoint examples partially mitigate but do not sequence the main flows. | 3 / 5 |
Progressive Disclosure | The Resources section lists seven bundle files (references/, assets/, scripts/), yet none exist, and the Instructions point to an inconsistent path 'resources/implementation-playbook.md'. The bulk of per-ORM detail is inlined in SKILL.md where the referenced files say it should live — content that clearly belongs in separate files is inlined with broken navigation, matching anchor 2. | 2 / 5 |
Total | 12 / 20 Passed |