Content
88%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 high-quality, immediately actionable skill: executable SQL and commands throughout, a clearly sequenced four-step workflow with a real verify-and-measure feedback loop, and disciplined conciseness. The only structural gap is the absence of any progressive-disclosure layer — the anti-pattern catalog and the neon.ts IaC section are inline monoliths that could be split into reference files.
Suggestions
Move the five anti-pattern before/after examples into a references/anti-patterns.md file, keeping one-line summaries in SKILL.md, so the overview stays lean.
Move the neon.ts infrastructure-as-code section into the parent neon skill or a references file and keep a single pointer in SKILL.md, since it addresses compute/storage costs rather than egress.
Trim restated conclusions (e.g., the 'No duplication.' recap after the JOIN-split example) to lift conciseness to fully lean.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is code-first and lean: diagnostic SQL, before/after pairs, and a tight neon.ts snippet with no padding or explanation of concepts Claude already knows. It is not a 5 because a few sentences restate the obvious ("Two queries instead of one JOIN. The product data is fetched once. The reviews are fetched once. No duplication.") and the neon.ts infrastructure section repeats its rationale twice. | 4 / 5 |
Actionability | Everything is executable: ready-to-run pg_stat_statements queries ("SELECT query, calls, rows AS total_rows, rows / calls AS avg_rows_per_call FROM pg_stat_statements WHERE calls > 0 ORDER BY rows DESC LIMIT 10"), CREATE EXTENSION, reset/re-measure commands, bash install/apply commands, and a complete neon.ts config. Before/after examples cover the common anti-patterns (SELECT *, missing pagination, JOIN duplication, application-side aggregation). | 5 / 5 |
Workflow Clarity | The four steps are explicitly sequenced ("Work the four steps in order: diagnose... analyze... fix... verify") and Step 4 is a genuine validation feedback loop: run existing tests, check response shapes, then "reset it (SELECT pg_stat_statements_reset();), let traffic run, then re-run the diagnostic queries to compare before and after". Edge cases (missing extension, empty stats after scale-to-zero) are handled with explicit recovery paths. | 5 / 5 |
Progressive Disclosure | The single-file body is well organized into clear sections with one-level-deep, clearly signaled references to the parent `neon` skill and two external doc URLs; no nested-reference indirection exists. It is not a 5 because there is no reference layer at all — the anti-pattern catalog and the neon.ts infrastructure section are sizeable bodies of content that could live in separate reference files, and the neon.ts section is tangential to egress diagnosis. | 4 / 5 |
Total | 18 / 20 Passed |