Content
50%Weight 40%Scale 1-3Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
The skill provides a well-sequenced, concrete comparison workflow with a validation step and a useful error-handling table, but it stops short on the generation deliverables, lacks a failure-recovery feedback loop for destructive database ops, and fails to integrate its own bundled script while listing phantom bundle files. The truncated Overview is an additional content defect.
Suggestions
Add the missing failure-recovery branch to step 10: "If the second diff is non-empty, revise the migration SQL and re-run against the copy until it reports zero differences; only then apply to the target."
Reference and actually use the bundled script (or replace it): either point step 8–9 to scripts/generate_migration_script.py with invocation examples, or remove it; and delete the phantom entries from scripts/README.md and assets/README.md that have no corresponding files.
Provide concrete, copy-paste-ready SQL scaffolding for the migration-generation step (CREATE/ALTER/DROP templates with transaction wrapping) instead of only describing the statements to generate, and complete the truncated Overview sentence.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The instructions are mostly lean and assume Claude's competence (concrete SQL and CLI commands, no basic-concept explanations), but the narrative Examples section and the truncated Overview ("staging vs.") add padding and a defect, so it does not reach the every-token-earns-its-place bar of level 3. | 2 / 3 |
Actionability | Steps 1–7 give concrete, executable SQL and pg_dump/mysqldump commands, but the core deliverable steps (8–9: generating the diff report and migration SQL) only describe output rather than providing complete executable code, and the bundled generate_migration_script.py — a generic placeholder — is never referenced, leaving key details missing per the level-2 anchor. | 2 / 3 |
Workflow Clarity | The 10-step sequence is clearly ordered with an explicit validation step (step 10: apply to a copy and re-run the diff expecting zero differences), but for destructive database operations the rubric requires a full validate->fix->retry loop; only the validate half is present with no failure-recovery branch or "only proceed when valid" gate, capping it at 2. | 2 / 3 |
Progressive Disclosure | The body is organized into clear sections rather than a wall of text, but the existing bundle file generate_migration_script.py is never referenced from the body, and the scripts/assets READMEs list files (rollback_migration.py, validate_schema_changes.py, migration_template.sql) that do not exist, so references are poorly signaled and partially broken per the level-2 anchor. | 2 / 3 |
Total | 8 / 12 Passed |