Content
65%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
The body is highly actionable with executable SQL and a clearly sequenced workflow, but it is monolithic — ignoring available bundle scripts/assets — and could be tighter and more explicit about validation checkpoints for impactful database changes.
Suggestions
Reference the bundled scripts in the workflow (e.g., 'Run scripts/explain_plan_parser.py on the captured plan') and link to assets/example_explain_plans/ instead of keeping everything inline.
Add an explicit validation gate before recommending production changes, e.g., 'Verify the improved plan on staging; only recommend indexes whose staging EXPLAIN shows a lower cost.'
Trim inferred parentheticals like '(indicates missing index)' and '(consider hash join or merge join)' to rely on Claude's existing knowledge.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient with executable SQL throughout and no basic-concept padding, but includes inferable parentheticals ('indicates missing index', 'consider hash join or merge join', 'adding a covering index eliminates the sort') that could be trimmed. | 2 / 3 |
Actionability | Provides copy-paste-ready, executable SQL for every analysis step (pg_stat_statements query, EXPLAIN commands, cache-hit-ratio and bloat queries) plus concrete CREATE INDEX recommendations with exact columns. | 3 / 3 |
Workflow Clarity | Ten steps are clearly sequenced and step 8 verifies via before/after EXPLAIN on staging with an error-handling table, but there are no explicit validate-then-proceed checkpoints for impactful database changes, so per the database-operations feedback-loop guideline workflow clarity is capped at 2. | 2 / 3 |
Progressive Disclosure | Well-organized into clear sections, but the body is monolithic and never references the bundled scripts (explain_plan_parser.py, index_advisor.py, query_rewriter.py) or the example_explain_plans assets; content that could be split out remains inline. | 2 / 3 |
Total | 9 / 12 Passed |