Content
56%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 content is well-structured and highly actionable, with concrete commands, code patterns, and a sensible diagnosis-to-fix workflow. Its weaknesses are token efficiency — large portions restate Prisma knowledge Claude already has plus one filler section — and the absence of any progressive disclosure: all playbook detail is inlined in a single long file with no references.
Suggestions
Move each playbook (schema, migrations, query optimization, connections, transactions) into a separate references/ file and keep only a one-line index per topic in SKILL.md, loading detail only when the issue category matches.
Cut content that restates standard Prisma knowledge (the include/select/$queryRaw ladder, $transaction API usage, the globalForPrisma singleton) in favor of project-specific judgments, and delete the circular 'When to Use' section.
Replace placeholder diagnosis steps ('use database-specific command', comment-only logging block) with the actual commands per provider, and add explicit post-fix verification steps after destructive operations (e.g., re-run 'npx prisma migrate status' after 'migrate resolve').
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~350-line body is noticeably verbose: it restates the persona from the frontmatter, inlines standard Prisma knowledge Claude already has (include/select patterns, $transaction usage, the globalForPrisma singleton), and ends with the pure filler sentence 'This skill is applicable to execute the workflow or actions described in the overview.' It exceeds the 'mostly efficient with some tightening' anchor at 3 because several whole sections add no knowledge Claude lacks. | 2 / 5 |
Actionability | Mostly executable guidance: concrete commands ('npx prisma validate', 'npx prisma migrate diff --from-schema-datamodel ...', 'npx prisma migrate deploy') and complete TypeScript/Prisma code blocks for transactions, query logging, and connection config. Minor gaps keep it below 5: the migration-history check is a placeholder ('use database-specific command') and the query-logging 'diagnosis' block is only comments. | 4 / 5 |
Workflow Clarity | A clear sequence is present: environment detection (version, provider, migrations, client generation) → identify issue category → check anti-patterns → progressive fixes (minimal/better/complete) → 'Validate with Prisma CLI and testing'. Validation checkpoints exist (prisma validate, migrate status, migrate diff), so the destructive-operation cap does not apply; the gap is the absence of explicit re-verify loops after destructive fixes (migrate reset, migrate resolve), which keeps it below 5. | 4 / 5 |
Progressive Disclosure | The body has good section structure (consistent playbook format with Common Issues / Diagnosis / Prioritized Fixes / Resources), but it is a ~350-line monolith with no bundle files at all — each playbook (schema, migrations, queries, connections, transactions) is content that clearly belongs in separate reference files loaded on demand. This matches 'some structure but content that should be separate is inline'. | 3 / 5 |
Total | 13 / 20 Passed |