Content
77%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
Highly actionable and clearly sequenced for a read-only SQL analysis tool, but the body is longer than necessary: it teaches generic SQL Claude already knows and keeps all detail inline in one file rather than splitting reference material out. Trimming the SQL tutorial and externalizing examples would improve token efficiency and structure.
Suggestions
Cut or heavily trim the 'Analysis Patterns' section — Claude already knows GROUP BY, window functions, CTEs, and CASE-based pivots; keep only patterns specific to this skill's table-naming rules or DuckDB quirks.
Consolidate the repeated script-invocation examples: the Complete Example and Multi-file Example restate the same parameter pattern, so one worked example plus the parameters table suffices.
Move the detailed SQL pattern library and full worked examples into a separate reference file (e.g. references/examples.md) linked from SKILL.md, keeping the main file a lean overview plus parameter reference.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Script-usage guidance is lean, but the 'Analysis Patterns' section (~60 lines) teaches SQL Claude already knows (GROUP BY, window functions, CTEs, CASE pivots) and the Complete/Multi-file examples restate the same invocation pattern — matching the score-2 anchor for mostly efficient but padded. | 2 / 3 |
Actionability | Provides fully executable bash commands with exact flags, a precise parameters table, copy-paste SQL, and output paths, matching the score-3 anchor for copy-paste-ready executable guidance. | 3 / 3 |
Workflow Clarity | Clearly numbered steps (understand -> inspect -> analyze) plus a full Steps 1-4 worked example, with inspect-before-query acting as the schema-discovery checkpoint; operations are read-only so the destructive/batch validation cap does not apply. | 3 / 3 |
Progressive Disclosure | A single ~240-line monolithic file holds everything inline (overview, SQL pattern library, full worked examples); the only bundle file (analyze.py) is an opaque executable ('Do NOT read the Python file') rather than a navigable reference, matching the score-2 anchor where split-able content stays inline. | 2 / 3 |
Total | 10 / 12 Passed |