Content
53%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 technically strong, example-rich ClickHouse reference with concrete executable SQL and TypeScript snippets, undermined by a monolithic single-file structure that inlines ~430 lines with zero progressive disclosure, a textbook overview section Claude doesn't need, and no validation feedback loops for its batch/insert/CDC operations. Splitting the analytics cookbook, monitoring, and pipeline patterns into reference files would address the biggest weaknesses.
Suggestions
Move the analytics query cookbook (retention/funnel/cohort), monitoring queries, and data-pipeline patterns into references/ files (e.g., ANALYTICS.md, MONITORING.md, PIPELINES.md), leaving SKILL.md as a concise overview with clearly signaled one-level-deep links.
Delete the introductory "概要" section and "主な機能" list — Claude already knows what ClickHouse and columnar OLAP databases are — and open with the table-design patterns directly.
Add validation/verification steps for batch operations: check insert results, verify row counts after bulk loads, and handle the CDC notification handler's failure path before continuing.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The bulk is ClickHouse-specific code, which earns its place, but the opening explains concepts Claude already knows — "ClickHouseは、オンライン分析処理(OLAP)用のカラム指向データベース管理システム(DBMS)です" plus a "主な機能" list of column storage, compression, and parallel execution — textbook knowledge that should be cut. At ~430 lines in a single file, several sections (monitoring queries, analytics cookbook) could be tightened; this sits at the "mostly efficient but includes some unnecessary explanation" anchor rather than clearly verbose (2). | 3 / 5 |
Actionability | Mostly executable, copy-paste-ready code: complete CREATE TABLE statements with engines/partitioning, concrete aggregate and window-function SQL, and working @clickhouse/client insert snippets. Not 5 because of minor gaps — the ETL example calls undefined helpers (`extractFromPostgres()`, `bulkInsertToClickHouse()`), and some queries reference columns (`trader_id`, `trade_size`) not present in any defined table DDL. | 4 / 5 |
Workflow Clarity | The skill is pattern-organized with clear PASS/FAIL contrasts rather than a step sequence, which is acceptable for a reference skill, but it includes batch and streaming insert operations, ETL pipelines, and CDC sync with no validation or verification steps (no insert-result checks, row-count verification, or error-recovery guidance). Per the rubric's cap, batch operations without feedback loops cap workflow clarity at 3; it is not 2 because the sections are individually coherent and clearly sequenced within each example. | 3 / 5 |
Progressive Disclosure | There are no bundle files at all (no references/, scripts/, or assets/), so the entire ~430-line reference — including a full analytics query cookbook (retention, funnel, cohort), monitoring queries, and pipeline code that clearly belongs in separate reference files — is inlined in SKILL.md. This matches the anchor for content that clearly belongs in separate files being inlined; it is not 3 because there are no references to signal or reorganize, and the inlining is more extensive than that anchor's example. | 2 / 5 |
Total | 12 / 20 Passed |