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 CLI-oriented skill body with excellent token efficiency and a well-sequenced caching workflow. Its main defects are the two missing reference/ files the body links to, and the absence of an explicit error-recovery checkpoint in the workflow.
Suggestions
Add the missing reference/discovery-warehouse.md and reference/common-patterns.md files (or remove/inline the links) — both are cited in the workflow and References section but do not exist in the bundle.
Add an explicit error-recovery step to the workflow, e.g. 'If a query times out, re-run with exec "..." -t 600; if it errors, check the warehouse size with warehouse list before retrying'.
State when to use `table cache` in the workflow (e.g. after discovering a schema in step 3) so the table-schema cache commands in the CLI reference are anchored to a step.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and dense: a 6-step workflow, a kernel-functions table, and terse CLI command listings with no padding and no explanations of concepts Claude already knows (e.g. 'the kernel self-terminates after 2h idle' and fail-fast semantics of run_sql_many are genuinely non-obvious facts). Every section earns its tokens. | 5 / 5 |
Actionability | Every instruction is a copy-paste-ready `uv run scripts/cli.py ...` command with concrete flags (-k <KEY_COL>, -t 600, --stale-only), and the kernel API is given as an executable snippet ("dfs = run_sql_many(['SELECT ...', 'SELECT ...']); print(dfs[0])"). The scripts/ bundle (cli.py, cache.py, etc.) actually exists, so the commands are real, not pseudocode. | 5 / 5 |
Workflow Clarity | The 6-step workflow is clearly sequenced with conditional branching ('If a pattern exists, follow its strategy', 'If cache misses') and a built-in feedback loop via `pattern record <name> --success/--failure`. It falls short of a 5 because there is no explicit checkpoint for handling a failed or timed-out query (e.g. retry with a higher -t, inspect the error) — only the timeout mechanics are described, not the recovery step. | 4 / 5 |
Progressive Disclosure | Structure is good — a References section clearly signals two one-level-deep files ('discovery-warehouse.md — Large table handling...', 'common-patterns.md — SQL templates...'), and SQL templates/discovery details are appropriately deferred. However, neither referenced file exists in the bundle (no reference/ or references/ directory), so the disclosure navigation is broken and a following agent hits dead links; this is worse than the 'minor organization gaps' of a 4. | 3 / 5 |
Total | 17 / 20 Passed |