Content
65%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 is highly actionable with comprehensive executable migration examples, but it is weighed down by generic boilerplate, missing validation feedback loops on destructive operations, and broken/nonexistent bundle references that should hold the inlined detail.
Suggestions
Move the bulk of ORM/schema/data/rollback/zero-downtime code examples into the referenced reference files and keep SKILL.md as a lean overview, so progressive disclosure is real rather than dangling.
Create the missing bundle files (references/orm-switching.md, schema-migration.md, etc., assets/*, scripts/*) and fix the 'resources/' vs 'references/' path mismatch for implementation-playbook.md.
Replace the generic Instructions/Do-not-use boilerplate with an explicit migration workflow that includes validation checkpoints (migrate → verify on staging → on failure, rollback) to satisfy the destructive-operations feedback-loop requirement.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient code-forward content, but it includes generic boilerplate ("Clarify goals, constraints, and required inputs", "Apply relevant best practices") and the large volume of inlined examples could be tightened or offloaded to references. | 3 / 5 |
Actionability | Provides fully executable, copy-paste-ready migration code for Sequelize, TypeORM, and Prisma with concrete run/rollback commands, covering the common migration cases comprehensively. | 5 / 5 |
Workflow Clarity | Phased sequences exist (zero-downtime Phase 1-5, column-rename Step 1-3) and one checkpoint example verifies a count, but the primary destructive migration workflow lacks explicit validate→fix→retry feedback loops, capping the score at 3 per the destructive-operations rule. | 3 / 5 |
Progressive Disclosure | Has clear section headers and a Resources section, but large bodies of example code that belong in separate reference files are inlined, and the referenced files (references/*.md, assets/*, scripts/*) do not exist; the body also points to a misnamed 'resources/implementation-playbook.md'. | 3 / 5 |
Total | 14 / 20 Passed |