Content
75%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 high-quality, operationally specific reference that assumes expertise, gives executable commands with validation gates, and cleanly defers deep detail to AGENTS.md. The main gaps are minor: a few placeholder code templates and some dense inline tables that could be pushed deeper.
Suggestions
Replace the `SQL_FUNCTION()` / `node_roles=[...]` placeholders in the migration-structure example with a minimal real snippet so the canonical example is copy-pasteable.
Move the full run_mode predicate table and engine reference into AGENTS.md, keeping only the most common predicate (`is_deployed_cloud`) inline, to tighten the quick reference and deepen progressive disclosure.
Add a short numbered 'write a migration' checklist (pick node role → write operations → gate with run_mode → update schema.py tuple → update HCL → check.sh) so the end-to-end workflow is explicit rather than dispersed across sections.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and assumes Claude's competence — no padding explaining what ClickHouse is — and most rationale (CODEC, Kafka settings) is PostHong-specific knowledge Claude lacks; a few prose passages could be trimmed. | 4 / 5 |
Actionability | Provides concrete, copy-pasteable commands (`bash $HCL/gen-golden.sh`, `bash $HCL/check.sh`) and real code (run_mode gating, Kafka SETTINGS baseline); the migration-structure example uses placeholders like `SQL_FUNCTION()` and `node_roles=[...]`, leaving minor gaps. | 4 / 5 |
Workflow Clarity | Destructive/batch operations carry explicit validation checkpoints (HCL `check.sh must exit 0`, CI `DRIFT:` failure, the two-step DROP COLUMN process), and the HCL section is a clear sequenced workflow; the skill is reference-organized rather than one end-to-end flow, so it stops short of full workflow maturity. | 4 / 5 |
Progressive Disclosure | Clearly defers comprehensive detail one level deep to `posthog/clickhouse/migrations/AGENTS.md` and `hcl/README.md` with well-signaled 'Read X for...' framing, and sections are well-organized; some dense inline reference (engine tables, run_mode predicates) could arguably live behind those references. | 4 / 5 |
Total | 16 / 20 Passed |