Content
90%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 tight, highly actionable migration runbook with executable commands, a real validation step, and a checklist — well-matched to a destructive database operation. The only meaningful gap is that the fix-and-retry feedback loop after SQL review is implied rather than explicit.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Lean throughout: no preamble explaining what EF Core or migrations are, and every line earns its place — the Step 1 'snapshot footgun' warning and the BaseDbContext edge-case note are non-obvious knowledge Claude would not already have. Matches anchor 5's 'every token earns its place'. | 5 / 5 |
Actionability | Every step is a copy-paste-ready command with all required flags spelled out ('dotnet ef migrations add {MigrationName} --project ... --startup-project ... --context {X}DbContext --output-dir {X}'); placeholders are necessary parameterization, not pseudocode, matching anchor 5. | 5 / 5 |
Workflow Clarity | Clear Step 0–4 sequence with an explicit validation step (Step 3 reviews generated SQL for data loss) and a closing checklist, so it clears the destructive-operation cap; held at 4 rather than 5 because the feedback loop is only implied ('Adjust the model or hand-edit the migration if needed') without an explicit re-validate/re-review step. | 4 / 5 |
Progressive Disclosure | Well-organized into clearly labeled sections (Steps 0–4, Notes, Checklist) with one-level-deep, clearly signaled external pointers ('see .agents/rules/database.md', 'see add-module'); no bundle files exist and all content is core workflow that belongs inline, but it is slightly over the ~50-line simple-skill threshold, so it sits at 4 rather than 5. | 4 / 5 |
Total | 18 / 20 Passed |