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 tight, highly actionable skill body: executable commands throughout, a clearly sequenced cache-first workflow, and exemplary token efficiency with no filler. The two real defects are that both linked reference files are missing from the bundle, breaking progressive disclosure, and that no validation checkpoint exists before presenting query results.
Suggestions
Create reference/discovery-warehouse.md and reference/common-patterns.md (or fix the link paths) so the References section's clearly-signaled links actually resolve — both files are absent from the bundle, leaving the detailed material unreachable.
Add a validation step between 'Execute query' and 'Present findings', e.g. sanity-check result row counts / NULLs and re-run with adjusted filters when a result looks wrong, before answering the user.
Consider moving the bulk of the CLI Reference section into a reference file, keeping only the most-used commands (exec, concept/pattern lookup and learn) inline to reduce duplication with the Workflow section.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and command-first: a six-step workflow of exact commands, a table for kernel functions, and one-line operational notes ("fail-fast", timeout, idle kernel) with zero explanation of concepts Claude already knows — "pl (Polars) and pd (Pandas) are pre-imported" is a necessary fact, not padding. The only redundancy is that pattern/concept commands appear both in the Workflow and the CLI Reference, but that duplication serves distinct purposes (usage in context vs. syntax reference), so every token still earns its place. | 5 / 5 |
Actionability | Guidance is fully executable: exact commands like `uv run scripts/cli.py exec "df = run_sql('SELECT ...')""`, a kernel-function table with return types, and concrete flags (-t 600, --stale-only); every listed subcommand was verified to exist in scripts/cli.py. Placeholders appear only where content is inherently user-determined (the SQL itself, the question text), which is justified flexibility, so this matches the copy-paste-ready top anchor. | 5 / 5 |
Workflow Clarity | The six steps are clearly sequenced with cache hit/miss branching ("If a pattern exists, follow its strategy", "If cache misses") and an outcome-recording loop ("pattern record <name> --success # or --failure"). It is not a 5 because there is no explicit validation checkpoint on query results before "Present findings" (e.g., sanity-check counts/NULLs before answering), and error recovery mid-workflow is only implicit; since queries are read-only ad-hoc analysis rather than destructive or batch operations, the cap-at-3 rule does not apply. | 4 / 5 |
Progressive Disclosure | The two references are clearly signaled and one level deep with accurate topic descriptions ("Large table handling, warehouse exploration, INFORMATION_SCHEMA queries"; "SQL templates for trends, comparisons, top-N, distributions, cohorts"), but neither reference/discovery-warehouse.md nor reference/common-patterns.md exists in the bundle — no reference directory is present — so navigation to the detailed material is broken. With 100% of the reference links dead and ~50 lines of CLI reference inlined, this exceeds the 'minor organization gaps' of the level-4 anchor. | 3 / 5 |
Total | 17 / 20 Passed |