Content
78%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, highly actionable body: concrete executable patterns for every tool, clear expand-contract sequencing, and a genuine anti-pattern table. Its main weaknesses are structural (everything inlined in one long file with no reference files) and minor token waste on boilerplate examples, plus a missing verification loop on the batched backfill.
Suggestions
Split the five per-tool workflow sections into one-level-deep reference files (e.g., references/prisma.md, references/django.md), keeping SKILL.md as the safety principles, PostgreSQL patterns, and expand-contract overview with clearly signaled links.
Trim content Claude already knows — the Prisma model and Drizzle table schema examples and the Kysely programmatic migrator boilerplate — down to just the non-obvious warnings (e.g., the Kysely<any> rule and allowUnorderedMigrations caveat).
Add an explicit verification step after batched backfills (e.g., SELECT COUNT(*) WHERE normalized_email IS NULL must return 0) before proceeding to the contract phase.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The safety patterns, SQL snippets, and tool commands are dense and earn their tokens, but sections like the Prisma/Drizzle schema examples and the ~35-line Kysely programmatic migrator restate boilerplate Claude already knows. This matches the 4 anchor (efficient, minor over-explanation that could be trimmed) rather than 5; it is not 3 because the padding is localized rather than pervasive. | 4 / 5 |
Actionability | Nearly everything is copy-paste ready: executable ALTER/CREATE INDEX SQL with GOOD/BAD contrasts, a runnable batched-backfill DO block with FOR UPDATE SKIP LOCKED, exact CLI commands per tool, and complete Django/Kysely migration files. This matches the 5 anchor of fully executable guidance covering the common cases. | 5 / 5 |
Workflow Clarity | The expand-contract pattern is clearly sequenced across phases with a concrete day-by-day timeline, and the pre-flight safety checklist plus "Verify data consistency" in Phase 2 provide real checkpoints. However, the batched backfill — a batch operation — has progress notices but no explicit post-run verification (e.g., confirm zero remaining NULLs), which is the 'minor validation gaps' of the 4 anchor rather than a clean 5. | 4 / 5 |
Progressive Disclosure | Headers are clear and navigation within the file is easy, but this is a ~430-line monolith: the five per-tool workflow sections are exactly the content that belongs in one-level-deep reference files (e.g., references/prisma.md). This matches the 3 anchor (some structure, content that should be separate is inline); it is not 2 because the inlined content is well sectioned rather than a formless dump. | 3 / 5 |
Total | 16 / 20 Passed |