Content
46%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 cleanly sectioned and concise enough, but it offers almost no executable guidance and frames a deliverable list as a workflow without validation checkpoints for destructive migration operations.
Suggestions
Replace the generic Instructions with at least one concrete executable pattern, e.g., an expand-contract migration snippet (add column -> backfill -> switch -> drop) with real SQL for PostgreSQL/MySQL/SQL Server.
Turn the Output Format into a true sequenced workflow with explicit validation checkpoints (e.g., run pre-migration checks, apply, validate row counts, only then promote, else roll back), including a validate->fix->retry loop.
Remove the redundant 'Context' and 'Resources' sections (they restate the Overview) and either supply the referenced 'implementation playbooks' as bundle files or drop the dangling reference.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly efficient and does not explain concepts Claude already knows, but the role preamble, the 'Context' section, and the one-line 'Resources' section each restate the Overview, and the Instructions are generic template bullets that could be trimmed. | 3 / 5 |
Actionability | Guidance is largely abstract: the Instructions are generic ('Clarify goals', 'Apply relevant best practices') and the Output Format names deliverables without any executable SQL, commands, or concrete expand-contract/rollback patterns, and it defers 'detailed examples' to nonexistent playbooks. | 2 / 5 |
Workflow Clarity | The Output Format gives a rough sequence of deliverables (analysis, plan, scripts, validation, rollback) but no validate->fix->retry checkpoint, and because database migrations are destructive/batch operations the missing validation loop caps this score at 3. | 3 / 5 |
Progressive Disclosure | The skill is under 50 lines with well-organized sections and no bundle files, which suits a single-file overview, but the inline mention of 'implementation playbooks' is a dangling reference to files that do not exist, a minor organization gap. | 4 / 5 |
Total | 12 / 20 Passed |