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.
This is a strong, token-efficient quick-reference skill: every snippet is executable, the tables are dense with actionable guidance, and navigation to the full database-reviewer workflow is clearly signaled. The only meaningful gap is the Configuration Template, which applies destructive/system-wide changes (ALTER SYSTEM, REVOKE ALL) without any verification or rollback guidance.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is a lean cheat sheet — tables and copy-paste SQL with terse, load-bearing comments like "Wrap in SELECT!" and "O(1) vs OFFSET which is O(n)" — with no padding and no explanations of concepts Claude already knows. Every section earns its tokens as a quick reference. | 5 / 5 |
Actionability | All examples are executable, copy-paste-ready SQL covering the common cases: index selection by query pattern, type choices, composite/covering/partial indexes, RLS policy, upsert, cursor pagination, queue processing with FOR UPDATE SKIP LOCKED, and diagnostic queries against pg_stat_statements. Minor placeholders ("adjust for RAM") are appropriately flexible rather than pseudocode. | 5 / 5 |
Workflow Clarity | As a lookup reference there is no multi-step workflow to sequence, but the Configuration Template performs risky system-wide operations — "ALTER SYSTEM SET max_connections", "REVOKE ALL ON SCHEMA public FROM public" — with only pg_reload_conf() and no verification, rollback, or check-existing-grants steps. Per the rubric's cap for database operations lacking validation/feedback loops, workflow clarity is capped at 3. | 3 / 5 |
Progressive Disclosure | The body is well-organized with clearly headed sections (cheat sheets, common patterns, anti-pattern detection, configuration) and an explicit pointer to the deeper "database-reviewer" agent for full workflows. It is not a 5 because at ~140 lines all content is inline — the anti-pattern queries and configuration template are candidates for one-level-deep reference files — though as a self-contained quick reference the placement is defensible. | 4 / 5 |
Total | 17 / 20 Passed |