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.
An exemplarily lean and fully executable quick reference whose only meaningful gap is the absence of validation/verification steps for the database changes it recommends, which caps workflow clarity. Structure is good but the full content sits in a single inline file rather than splitting deep material into references.
Suggestions
Add verification steps to the index and policy patterns: e.g., verify index usage with `EXPLAIN ANALYZE` before/after creating an index, and test RLS policies by querying as a non-owner role.
Add a validation checkpoint to the Configuration Template (e.g., check `pg_settings` or current connection counts after `ALTER SYSTEM` + `pg_reload_conf()`) since those are system-wide changes.
Move the Anti-Pattern Detection queries and Configuration Template into a references/ file, keeping SKILL.md as a lean overview that points to them.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is a lean quick reference of tables and code snippets with terse instructive comments ("Wrap in SELECT!", "O(1) vs OFFSET which is O(n)") and no re-explanation of concepts Claude already knows; every section earns its place. | 5 / 5 |
Actionability | Every pattern is copy-paste-ready executable SQL (GIN/BRIN/composite indexes, `ON CONFLICT` upsert, `FOR UPDATE SKIP LOCKED` queue, cursor pagination, pg_stat_statements diagnostics, ALTER SYSTEM config), covering the common database cases. | 5 / 5 |
Workflow Clarity | Database operations are an explicitly feedback-loop-critical context, and the skill recommends consequential changes (CREATE INDEX, ALTER SYSTEM, CREATE POLICY) with no verification steps such as EXPLAIN ANALYZE after indexing, testing RLS policies under a role, or confirming config changes. The rubric's destructive/batch-operation cap applies despite the skill being a single-purpose reference. | 3 / 5 |
Progressive Disclosure | Well-organized sections with clear headers and a pointer to the `database-reviewer` agent for detail, but at ~140 lines everything (anti-pattern queries, configuration template) is inlined in one file with no reference split; the under-50-lines exception for a top score does not apply. | 4 / 5 |
Total | 17 / 20 Passed |