Content
52%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.
The body is well-organized and concise with a sensible workflow and a useful output template, but as delivered the skill is a hollow shell: all substantive decision content lives in bundle files that are absent. Actionability and progressive disclosure are the casualties, since every pointer is a dead end.
Suggestions
Ship the referenced bundle files (rules/decision-*.md, mappings/doc_links.yaml, AGENTS.md, examples/) alongside SKILL.md, or remove the references — as delivered, every path in the body is broken.
Inline at least one worked example (workload shape → key decisions → recommendation with official/derived/field label and a concrete SQL validation) so the skill remains actionable even when the rule files are unavailable.
Replace vague pointers like 'Relevant best-practices insert rules' with the specific rule file names or IDs from clickhouse-best-practices, and show a sample doc_links.yaml entry so step 3 is executable.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean with no padding and no explanations of concepts Claude already knows; the workload list, provenance rules, and output template each earn their tokens. Not 5 because a few lines ('Relevant best-practices insert rules') are vague filler that could be tightened or made specific. | 4 / 5 |
Actionability | Beyond the output-format template there is no executable guidance — no SQL, commands, or worked example — and the core decision logic is deferred to 'rules/decision-*.md' and 'mappings/doc_links.yaml' files that are not present in the bundle. Matches anchor 2 (high-level hints, missing the specific steps to execute); not 3 because the pseudocode-equivalent template alone doesn't supply concrete executable guidance. | 2 / 5 |
Workflow Clarity | A clear numbered sequence (identify workload → read rule files → attach docs → classify provenance → flag uncertainty) plus a scenario-to-file index, with a 'Validation' field acting as an output checkpoint. Not 5 because validation of the skill's own intermediate steps is left implicit and depends on the missing rule files. | 4 / 5 |
Progressive Disclosure | The one-level-deep reference design is right in principle, but scored against the actual bundle every referenced path is broken: 'rules/', 'mappings/doc_links.yaml', 'AGENTS.md', and 'examples/' do not exist, and no references/, scripts/, or assets/ directories ship with the skill. Matches anchor 2; not 3 because the navigation cannot actually lead anywhere as delivered. | 2 / 5 |
Total | 12 / 20 Passed |