Content
57%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 a dense, largely executable ClickHouse cookbook with strong SQL examples, but it front-loads knowledge Claude already has, carries generic boilerplate sections, and provides no validation feedback loops for batch/database operations. Everything is inlined in one large file with no progressive disclosure into reference files.
Suggestions
Delete the Overview/Key Features explanations and the vacuous 'When to Use' line; Claude already knows what ClickHouse and OLAP are.
Add validation checkpoints around batch operations, e.g. verify inserted row counts against the source or query system.query_log after bulk loads before proceeding.
Split the analytics query cookbook (retention, funnel, cohort) and pipeline patterns into reference files linked one level deep from SKILL.md to reduce the monolithic inline content.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The Overview ('ClickHouse is a column-oriented database management system (DBMS) for online analytical processing (OLAP)') and the 'Key Features' list re-explain knowledge Claude already has, and 'This skill is applicable to execute the workflow or actions described in the overview' is vacuous filler. Mostly efficient code examples with some unnecessary explanation → anchor 3. Not 4 because the boilerplate Overview, Key Features, and generic When to Use/Limitations sections are clearly trimmable; not 2 because the bulk of the file is dense, useful code. | 3 / 5 |
Actionability | Extensive concrete, executable SQL (MergeTree/ReplacingMergeTree/AggregatingMergeTree DDL, quantile aggregations, window functions, system.query_log monitoring) plus TypeScript insert patterns. Minor gaps: the streaming example references undefined 'dataSource' and 'clickhouse' variables, and the bulk insert builds SQL via string interpolation rather than parameterized values. Not 5 because of these incomplete/inadvisable snippets; not 3 because nearly all examples are copy-paste-ready. | 4 / 5 |
Workflow Clarity | The ETL pattern lists a rough 1-2-3 sequence and sections are thematically ordered, but no validation or verification checkpoints exist for batch/bulk inserts or pipeline operations — database operations the rubric explicitly requires feedback loops for, capping workflow clarity at 3. Not 4 because validation is absent, not implicit-but-present; not 2 because sequences (ETL, CDC, monitoring) are listed and coherent. | 3 / 5 |
Progressive Disclosure | The single ~430-line file has clear section headers, but no bundle files exist and content that clearly belongs in separate references (the analytics query cookbook, data pipeline patterns, monitoring queries) is fully inlined. Anchor 3 ('some structure but content that should be separate is inline'). Not 4 because nothing is split out at all; not 2 because the section structure keeps it navigable. | 3 / 5 |
Total | 13 / 20 Passed |