Content
86%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-built skill body: fully executable SQL at every step, a clear four-step workflow with an embedded validation checkpoint, and exemplary one-level-deep progressive disclosure to a real, well-organized reference file. The only gaps are slight - a trimmed rationale sentence or two, and an explicit what-if-validation-fails loop for the post-fix re-EXPLAIN step.
Suggestions
Tighten Step 1's justification paragraph ('Always use ANALYZE... can be wildly wrong') to a single sentence, since the anti-patterns table in plan-signatures.md already covers reading EXPLAIN without ANALYZE.
Add an explicit failure-path step after the validation re-EXPLAIN: what to do when the plan does not improve (e.g., check stale statistics with ANALYZE <table>, or verify the index is actually usable for the predicate) to earn a full feedback loop.
Consider moving the sibling-skill cross-references (flame-graph-analyzer, k6-load-testing) into a one-line 'Related skills' note or the reference file, trimming the main body's token budget.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is efficient - a five-case overview, per-engine EXPLAIN commands, and a worked output template with no padding or explanation of basics Claude already knows. Minor trimming opportunities exist (the rationale for "Always use ANALYZE... plain EXPLAIN gives only planner estimates" and the trailing cross-skill references to flame-graph-analyzer / k6-load-testing partially restate what the reference file or description already covers), which fits anchor 4 rather than the lean anchor 5. | 4 / 5 |
Actionability | Every step ships copy-paste-ready SQL: `EXPLAIN (ANALYZE, BUFFERS, FORMAT JSON, SETTINGS) SELECT ...`, five concrete CREATE INDEX variants (single, composite, partial, functional, covering) with inline rationale, and a fully worked output-format example including a validation re-run. This matches anchor 5 - executable, specific examples covering the common cases. | 5 / 5 |
Workflow Clarity | Steps 1-4 are clearly sequenced (capture ANALYZE plan -> find dominant node reading innermost-outward -> match diagnosis to fix via lookup table -> emit index/rewrite), and the output template embeds an explicit validation checkpoint ("Confirm the new plan uses idx_orders_customer_created and the actual time is below the budget") with expected impact. It stops short of anchor 5's explicit error-recovery feedback loop (what to do if the plan does NOT improve, e.g., re-check stats or drop the speculative index), so anchor 4 fits best. | 4 / 5 |
Progressive Disclosure | SKILL.md is a concise overview that delegates all lookup tables to one level-deep, well-signaled references - [references/plan-signatures.md](references/plan-signatures.md) is cited four times, each time naming the specific table it contains (cost-signature table, diagnosis-to-fix table, index-choice principles, anti-patterns), and that file exists and contains exactly those sections. This matches anchor 5: clear overview, appropriately split content, easy navigation. | 5 / 5 |
Total | 18 / 20 Passed |