Content
80%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
The body is highly actionable and concise, packed with executable SQL for partitioning, migration, and maintenance across PostgreSQL and MySQL. Its weaknesses are incomplete validation/rollback checkpoints around destructive operations and a monolithic structure that ignores the stubbed bundle files.
Suggestions
Add explicit validation/rollback checkpoints to the destructive workflows: before the partition drop/detach in step 9, require verifying the partition is past retention and backed up; for the migration rename in step 6, document a rollback path if the rename fails.
Move reusable material into the bundled files and reference them from the body — e.g. put the partition DDL templates into assets/partition_template.sql and the design/maintenance logic into scripts/partition_design.py, partition_maintenance.py, then link to them so progressive disclosure is real rather than stubbed READMEs.
Tighten the redundancy between the Overview and step 3 so the four strategies are described once with the 'when to use' rationale, avoiding the duplicate enumeration.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and actionable, assuming Claude's competence with no padding or explanation of basic database concepts; the only mild waste is that the Overview and step 3 both enumerate the four strategies, but every section earns its place, keeping it at 3 rather than 2. | 3 / 3 |
Actionability | It provides concrete, executable SQL throughout (e.g. 'CREATE TABLE orders (...) PARTITION BY RANGE (created_at)', 'ALTER TABLE orders DETACH PARTITION orders_2022_01', 'INSERT INTO orders_partitioned SELECT * FROM orders_old WHERE...') that is copy-paste ready, matching the fully-executable anchor. | 3 / 3 |
Workflow Clarity | Steps 1-10 are clearly sequenced and the migration includes a verify checkpoint ('Verify row counts match') and step 10 verifies pruning, but destructive/batch operations have validation gaps (no rollback plan for the atomic rename, no pre-drop backup/retention verification before detaching/deleting partitions), triggering the cap-at-2 rule for destructive workflows; not a 1 because the sequence and some checkpoints are present. | 2 / 3 |
Progressive Disclosure | The body is well-sectioned (so not a monolithic wall of text), but it is a single monolithic file with no signaled references to the bundle, and the bundle directories contain only README stubs listing files that do not actually exist; content that should be split out (SQL templates, design/maintenance scripts) is described inline rather than in the stubbed assets/scripts, matching the 'some structure but content that should be separate is inline' anchor. | 2 / 3 |
Total | 10 / 12 Passed |