Content
87%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 lean, highly actionable rules-and-gotchas reference with executable commands, copy-paste code, and specific failure modes. Its one weakness is workflow clarity: destructive database operations lack an integrated validation checkpoint and feedback loop — 'prisma validate' is buried in an 'Other commands' list rather than sequenced into the migration workflow.
Suggestions
Add a numbered migration workflow with an explicit validation checkpoint, e.g.: 1. Edit schema 2. `npx prisma validate` 3. Fix errors and re-validate 4. `npx prisma migrate dev` 5. `npx prisma generate`.
Turn the 'inspect migration.sql before applying' rule into a feedback loop: if destructive operations (drops, column rewrites) are found, add the backfill step, regenerate, and re-inspect before applying.
Promote `prisma validate` out of the trailing 'Other commands' line into the Migration rules section as a required pre-apply step so the destructive-operation safeguard is impossible to miss.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean bullets and one copy-paste snippet; it assumes Claude's Prisma competence and adds only non-obvious rules ("Never edit an already-applied migration file — the checksum is recorded and will fail", "db push skips migration history entirely — prototyping only"). No padding found; every token earns its place. | 5 / 5 |
Actionability | Fully executable guidance throughout: exact commands ("npx prisma migrate dev ... in development only — never in production", "npx prisma migrate deploy", "npx prisma generate after every schema change"), concrete decorators (@@index, @relation onDelete: Cascade, @map/@@map), a copy-paste PrismaClient singleton snippet, and a specific error code (P2002) with handling direction. Covers the common cases; matches the level-5 anchor. | 5 / 5 |
Workflow Clarity | Migration rules imply a sequence (inspect migration.sql before applying; generate after every schema change) but no explicit validate→fix→retry loop exists for these destructive database operations. Per the rubric's cap ("database operations ... missing feedback loops cap workflow_clarity at 3"), this cannot score above 3: "prisma validate" appears only in the trailing 'Other commands' line, not as an integrated checkpoint. Not level 2 because the destructive-op safeguards (never edit applied migrations, inspect before applying, dev/deploy split) are real and well-stated. | 3 / 5 |
Progressive Disclosure | The body is under 50 lines with no external references needed and no bundle files (references/, scripts/, assets/ are absent), so the simple-skill exception applies: well-organized sections (Migration rules, Schema gotchas, Client and queries) plus one-level pointers to project config and docs earn a 5. | 5 / 5 |
Total | 18 / 20 Passed |