Content
87%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-signal, domain-specific reference that is lean and highly actionable with one clearly signaled external pointer. The one weakness is workflow clarity: destructive migration operations are governed by rules but lack an explicit validation/retry feedback loop, which the rubric caps at 3.
Suggestions
Add a short ordered migration workflow (write migration -> run locally -> verify table created in local dev -> check cloud guard parity) with explicit validation checkpoints before shipping.
Include a validate-and-retry loop for the destructive cases, e.g. after DROP TABLE IF EXISTS ... SYNC, verify the table is gone before recreating, and how to recover if the drop stalls.
Surface a brief verification step for the testing section beyond the single re-run hint (e.g. how to confirm a migration applied cleanly in local dev).
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is a dense reference with no padding of concepts Claude already knows; explanatory prose (CODEC ordering, DROP COLUMN two-step, kafka no-go zone) covers genuinely non-obvious ClickHouse internals where every token earns its place. Not a 4 because there is no trimmable over-explanation. | 5 / 5 |
Actionability | Provides copy-paste-ready operation scaffolding, exact node-role lists, concrete engine signatures, and the specific schema.py tuples to edit; specific examples cover the common sharded/replicated/ingestion cases. Not a 4 because guidance is executable rather than leaving gaps. | 5 / 5 |
Workflow Clarity | Content is rule/reference style rather than a sequenced multi-step workflow, and the destructive/batch operations (DROP COLUMN, DROP TABLE SYNC, schema migrations) lack an explicit validate-then-fix-then-retry feedback loop; per the rubric's destructive-operation cap, workflow clarity is capped at 3. | 3 / 5 |
Progressive Disclosure | The top line delegates the deep reference to posthog/clickhouse/migrations/AGENTS.md as a clearly signaled one-level-deep pointer, while the body keeps a well-organized quick reference; no bundle files exist to over-nest, matching the well-organized single-file anchor. | 5 / 5 |
Total | 18 / 20 Passed |