dot-iu-cutter v0.2 — P0-6 + P0-5 Remainder DDL r2 GPT Review

Date: 2026-05-16
Reviewer: GPT
Files reviewed: r2 DDL draft, r2 verification plan, r2 rollback draft, r2 risk note, r2 authoring report
Scope: DDL r2 re-review only. No DDL executed, no mutation, no dry-run, no production action.

---

1. Verdict

p0_6_p0_5_remainder_DDL_r2_status: PASS
A_1_status: resolved
agent_revision_needed: false
risk_class: STANDARD
r1_status: superseded_do_not_execute
r2_DDL_authoring_package_accepted: true
dry_run_allowed_after_explicit_prompt: true
production_migration_allowed: false

The r2 DDL package is accepted. The dry-run lane may reopen under explicit prompt.

---

2. A-1 Fix Accepted

live_parent_table: cutter_governance.decision_backlog_entry
live_parent_PK: entry_id_uuid
wrong_r1_assumption: decision_id_uuid
r2_fix_strategy: child_columns_renamed_to_entry_id_family

Accepted r2 column-name changes:

decision_backlog_history:
  entry_id: replaces_decision_id
  FK_target: decision_backlog_entry.entry_id

decision_backlog_dependency:
  from_entry_id: replaces_from_decision_id
  to_entry_id: replaces_to_decision_id
  FK_targets:
    - decision_backlog_entry.entry_id
    - decision_backlog_entry.entry_id

The names decision_id, from_decision_id, and to_decision_id must not appear in the executable r2 DDL.

---

3. Accepted r2 DDL Shape

schema: cutter_governance
new_tables:
  - review_decision
  - decision_backlog_history
  - decision_backlog_dependency
  - decision_backlog_sweep_log
expected_cutter_governance_table_count_after_later_execution: 12
transaction: single_BEGIN_COMMIT
FK_count_expected: 7
cross_schema_FK: 0
CHECK: 0
trigger: 0
DEFAULT: 0
PG_enum: 0
lookup_table: 0
D24_lookup_FK: 0
INSERT_seed_backfill: 0
IF_NOT_EXISTS: 0

---

4. Expected FK Surface After r2

FKs:
  - review_decision.manifest_id_to_manifest_envelope.envelope_id
  - review_decision.manifest_id_manifest_unit_local_id_to_manifest_unit_block.envelope_id_unit_local_id_nullable_composite
  - review_decision.prior_review_decision_id_self
  - review_decision.superseded_by_review_decision_id_self
  - decision_backlog_history.entry_id_to_decision_backlog_entry.entry_id
  - decision_backlog_dependency.from_entry_id_to_decision_backlog_entry.entry_id
  - decision_backlog_dependency.to_entry_id_to_decision_backlog_entry.entry_id

---

5. Mandatory Dry-Run Gates

Dry-run must verify:

pre_DDL:
  cutter_governance_table_count: 8
  decision_backlog_entry_PK: entry_id_uuid
  manifest_envelope_PK: envelope_id_uuid
  manifest_unit_block_PK: [envelope_id_uuid, unit_local_id_text]
  four_new_tables_absent: true

post_DDL:
  cutter_governance_table_count: 12
  four_new_tables_exist: true
  all_four_row_count: 0
  FK_count: 7
  no_old_child_column_names: true
  no_cross_schema_FK_CHECK_trigger_DEFAULT_PG_enum_lookup: true

Rollback must drop only the four new empty tables and must return cutter_governance to 8 tables.

---

6. Gate Effect

ready_for_dry_run_prompt: true
ready_for_production_prompt: false

Agent may retry dry-run using r2 only. Production remains forbidden.

---

7. Status

ddl_written_by_this_review: false
mutation_performed: false
dry_run_started: false
production_migration_allowed: false