Content
57%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 well-organized, example-rich body with executable SQL/TypeScript/Mermaid snippets and useful pitfalls and best-practices lists. Its main weaknesses are the dangling reference to references/full-schema-examples.md, the absence of validation checkpoints in a database workflow that the rubric requires, and inline bulk (full seed and RLS code) that inflates token cost without teaching much Claude doesn't know.
Suggestions
Create references/full-schema-examples.md (or remove the pointer) so the 'Full Schema Example' section doesn't dead-end, and move the full seed script and RLS policy set there, leaving only short canonical fragments inline.
Add explicit validation checkpoints to the workflow: run migrations against a throwaway copy first, then 'test RLS with a non-superuser role' as a numbered step, mirroring the validation→fix→retry pattern.
Trim known-by-Claude content: drop the Tier/Category/Domain metadata header and the 'When to Use' section that duplicates the frontmatter description, and merge the overlapping soft-delete/index advice repeated across Step 3, Common Pitfalls, and Best Practices.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient with clear sections, but includes content Claude already knows or could derive: a ~60-line faker seed script and a ~40-line standard RLS policy set inlined in full, a Tier/Category/Domain metadata header that serves no runtime purpose, and a 'When to Use' section that duplicates the frontmatter description. Matches 'Mostly efficient but includes some unnecessary explanation or could be tightened'; not 4 because the two large code blocks plus redundant sections are more than minor trimming. | 3 / 5 |
Actionability | Concrete, near-executable guidance throughout: full SQL RLS policies, a complete Drizzle-style seed script, Mermaid ERD syntax, and runnable commands ('npx prisma-erd-generator'). Not 5 because of minor gaps: the RLS examples compare uuid columns to current_setting(...)::text without a uuid cast and omit grants for app_user, and the 'Full Schema Example' section points to a file that does not exist. | 4 / 5 |
Workflow Clarity | The 3-step design process (requirements → entities → relationships → cross-cutting concerns) is clearly sequenced, but there are no validation checkpoints anywhere — no verify-migration-on-a-copy step, no RLS test procedure integrated into the flow. The scoring note explicitly caps database-operation workflows without feedback loops at 3; the 'RLS not tested' pitfall mentions testing but never wires it into the process. Not 2 because the sequence itself is coherent and well-defined. | 3 / 5 |
Progressive Disclosure | Good section structure and one clearly signaled reference ('→ See references/full-schema-examples.md for details'), but that file does not exist — no references/, scripts/, or assets/ directory is present, so the pointer is dead. Additionally, bulky content that belongs in reference files (the full seed script and full RLS policy set) is inlined in SKILL.md. Matches 'Some structure but could be better organized; references present but not clearly signaled; content that should be separate is inline'; not 4 because a broken reference and the inlined bulk exceed 'minor organization gaps'. | 3 / 5 |
Total | 13 / 20 Passed |