Content
83%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 SQL cheat sheet that excels at conciseness and actionability. Its main weakness is workflow clarity: as a reference catalog it lacks explicit validation checkpoints or feedback loops for the database operations it describes.
Suggestions
Add a short "Verify before shipping" step (e.g. EXPLAIN the query, confirm the index is used, test the RLS policy with a sample role) to turn isolated patterns into a validate→fix→retry workflow.
Consider moving the anti-pattern detection and configuration template blocks into separate reference files referenced one level deep, keeping SKILL.md as a tighter overview.
Add a brief pointer on when to run VACUUM/ANALYZE after schema/index changes so the catalog covers post-change validation.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Dense tables and copy-paste SQL with terse inline comments assume Claude's competence; no padded explanations of what PostgreSQL or an index is, so every token earns its place. | 5 / 5 |
Actionability | All examples are executable SQL covering the common cases — composite/covering/partial indexes, RLS, upsert, cursor pagination, SKIP LOCKED queues, and diagnostic anti-pattern queries — with no pseudocode. | 5 / 5 |
Workflow Clarity | It is a pattern catalog rather than a sequenced process, so there is no explicit multi-step workflow with validation checkpoints; the "When to Activate" list and diagnostic queries provide loose guidance but no validate→fix→retry feedback loop. | 3 / 5 |
Progressive Disclosure | Well-organized sections (When to Activate, Quick Reference, Related) with one-level pointers to the database-reviewer agent and no nested references, but the anti-pattern and configuration blocks are substantial enough that splitting them into reference files would improve navigation. | 4 / 5 |
Total | 17 / 20 Passed |