Content
81%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 strong, safety-first workflow guide: fully sequenced steps, executable commands, and genuine validation checkpoints and checklists appropriate for destructive database operations. The main gaps are directional (non-executable) guidance in the multi-phase column-add and rename patterns and mild redundancy around lock_timeout handling.
Suggestions
Add concrete SQL for Patterns B and C (e.g., ALTER TABLE ... ADD COLUMN, adding NOT NULL via NOT VALID + VALIDATE CONSTRAINT, and a backfill UPDATE template) so all patterns are executable, not just described.
Consolidate the lock_timeout guidance that currently appears in both Section 3's checklist and Section 6's emergency checklist to trim a little redundancy.
Consider moving the extended zero-downtime patterns and rollback procedures into a references/ file to keep SKILL.md a leaner overview.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dominated by lean, purposeful commands, checklists, and code blocks with no padding explaining concepts Claude already knows. Minor tightening is possible — lock_timeout guidance appears in both the Section 3 checklist and the Section 6 emergency checklist, and the Core Principles prose restates points made later — so it fits 'efficient; minor instances that could be trimmed' rather than the every-token-earns-its-place anchor at 5. | 4 / 5 |
Actionability | Mostly executable guidance: copy-paste-ready dotnet ef commands with real project paths and flags, concrete SQL (SET lock_timeout, CREATE INDEX CONCURRENTLY, the pg_index validity query), and a C# builder annotation. It falls short of 5 because Patterns B and C ('Deploy app code writing to both old and new schema. Backfill existing records in background batch', 'Dual write to both columns') describe phases directionally without concrete SQL or command examples. | 4 / 5 |
Workflow Clarity | A clearly sequenced 5-step workflow (create migration → generate/inspect SQL → apply zero-downtime patterns → deploy across environments → rollback) with explicit validation checkpoints: the SQL Safety Inspection Checklist before deployment, PITR/WAL backup verification before production, and a post-rollback index-validity query. Error-recovery feedback is present (lock_timeout expiry → check pg_stat_activity before forcing), so the destructive-operation cap at 3 does not apply — this matches the 'explicit validation steps; feedback loops; checklists' anchor. | 5 / 5 |
Progressive Disclosure | Well-organized single-file guide with clearly signaled sections and no nested or buried references (no bundle files exist). It fits 'good structure; most content appropriately placed' rather than 5, because at ~150 lines the zero-downtime patterns and rollback detail could plausibly live in separate reference files, and the under-50-lines exception does not apply. | 4 / 5 |
Total | 17 / 20 Passed |