diff --git a/conductor/tracks/result_migration_gui_2_20260619/plan.md b/conductor/tracks/result_migration_gui_2_20260619/plan.md index f37d73af..ec69f2d8 100644 --- a/conductor/tracks/result_migration_gui_2_20260619/plan.md +++ b/conductor/tracks/result_migration_gui_2_20260619/plan.md @@ -136,15 +136,15 @@ git commit -m "conductor(state): mark Phase 0 complete" **Focus:** Run the audit, walk every finding, classify each of the 42 migration-target sites. -### Task 1.1: Run the audit + capture JSON output +### Task 1.1: Run the audit + capture JSON output [a068934] -- [ ] **Step 1: Run audit and save JSON** +- [x] **Step 1: Run audit and save JSON** ```bash uv run python scripts/audit_exception_handling.py --src src/gui_2.py --json > tests/artifacts/PHASE1_AUDIT.json ``` -- [ ] **Step 2: Verify JSON was generated** +- [x] **Step 2: Verify JSON was generated** ```bash ls -la tests/artifacts/PHASE1_AUDIT.json @@ -152,12 +152,12 @@ ls -la tests/artifacts/PHASE1_AUDIT.json Expected: file exists, size > 1KB -### Task 1.2: Walk the audit + write the inventory +### Task 1.2: Walk the audit + write the inventory [a068934] **Files:** - Create: `tests/artifacts/PHASE1_SITE_INVENTORY.md` -- [ ] **Step 1: Extract the 42 migration-target sites** +- [x] **Step 1: Extract the 42 migration-target sites** ```bash uv run python -c " @@ -222,13 +222,13 @@ git add tests/artifacts/PHASE1_AUDIT.json tests/artifacts/PHASE1_MIGRATION_TARGE git commit -m "conductor(plan): Phase 1 site inventory — 42 migration-target sites classified" ``` -### Task 1.3: Phase 1 invariant test + checkpoint +### Task 1.3: Phase 1 invariant test + checkpoint [554fbbd] **Files:** - Create: `tests/test_gui_2_result.py` (initial creation; will be extended each phase) - Modify: `conductor/tracks/result_migration_gui_2_20260619/state.toml` -- [ ] **Step 1: Create the test file with Phase 1 invariant test** +- [x] **Step 1: Create the test file with Phase 1 invariant test** ```python """Tests for gui_2.py Result[T] migration (sub-track 4 of result_migration_20260616). @@ -275,7 +275,7 @@ def test_phase_1_invariant_baseline_captured(): assert gui["unclear_count"] >= 2, f"Expected ≥2 unclear, found {gui['unclear_count']}" ``` -- [ ] **Step 2: Run the test — it should PASS (the inventory was committed in Task 1.2)** +- [x] **Step 2: Run the test — it should PASS (the inventory was committed in Task 1.2)** ```bash uv run python -m pytest tests/test_gui_2_result.py::test_phase_1_invariant_inventory_complete tests/test_gui_2_result.py::test_phase_1_invariant_baseline_captured -v @@ -283,13 +283,13 @@ uv run python -m pytest tests/test_gui_2_result.py::test_phase_1_invariant_inven Expected: 2 PASSED -- [ ] **Step 3: Update state.toml Phase 1** +- [x] **Step 3: Update state.toml Phase 1** ```toml phase_1 = { status = "completed", checkpointsha = "", name = "Site inventory + classification (42 sites)" } ``` -- [ ] **Step 4: Commit** +- [x] **Step 4: Commit** ```bash git add tests/test_gui_2_result.py conductor/tracks/result_migration_gui_2_20260619/state.toml diff --git a/conductor/tracks/result_migration_gui_2_20260619/state.toml b/conductor/tracks/result_migration_gui_2_20260619/state.toml index c90dff9d..f316a043 100644 --- a/conductor/tracks/result_migration_gui_2_20260619/state.toml +++ b/conductor/tracks/result_migration_gui_2_20260619/state.toml @@ -19,7 +19,7 @@ result_migration_baseline_cleanup = "blocked by this track; date TBD when this t [phases] phase_0 = { status = "completed", checkpointsha = "62188d6", name = "Setup + styleguide re-read (3 tasks)" } -phase_1 = { status = "pending", checkpointsha = "", name = "Site inventory + classification (3 tasks; 42 sites in PHASE1_SITE_INVENTORY.md)" } +phase_1 = { status = "completed", checkpointsha = "554fbbd", name = "Site inventory + classification (3 tasks; 42 sites in PHASE1_SITE_INVENTORY.md)" } phase_2 = { status = "pending", checkpointsha = "", name = "Drain plane wiring (4 tasks; 3 new render functions + 2 invariant tests)" } phase_3 = { status = "pending", checkpointsha = "", name = "INTERNAL_BROAD_CATCH Batch A — render-loop sites (<=10 sites)" } phase_4 = { status = "pending", checkpointsha = "", name = "INTERNAL_BROAD_CATCH Batch B — modal/dialog sites (<=10 sites)" } @@ -40,9 +40,9 @@ t0_2 = { status = "completed", commit_sha = "62188d6", description = "Tier 2 rea t0_3 = { status = "in_progress", commit_sha = "", description = "Phase 0 checkpoint commit; update state.toml Phase 0 status" } # Phase 1: Site inventory + classification (3 tasks) -t1_1 = { status = "pending", commit_sha = "", description = "Run audit --src src/gui_2.py --json > tests/artifacts/PHASE1_AUDIT.json" } -t1_2 = { status = "pending", commit_sha = "", description = "Walk the audit + write tests/artifacts/PHASE1_SITE_INVENTORY.md (42 rows)" } -t1_3 = { status = "pending", commit_sha = "", description = "Create tests/test_gui_2_result.py with 2 Phase 1 invariant tests; Phase 1 checkpoint" } +t1_1 = { status = "completed", commit_sha = "a068934", description = "Run audit --src src/gui_2.py --json > tests/artifacts/PHASE1_AUDIT.json" } +t1_2 = { status = "completed", commit_sha = "a068934", description = "Walk the audit + write tests/artifacts/PHASE1_SITE_INVENTORY.md (42 rows)" } +t1_3 = { status = "in_progress", commit_sha = "", description = "Create tests/test_gui_2_result.py with 2 Phase 1 invariant tests; Phase 1 checkpoint" } # Phase 2: Drain plane wiring (4 tasks) t2_1 = { status = "pending", commit_sha = "", description = "Add render_controller_error_modal(app) — reads 8 controller attributes; renders popups" } @@ -166,7 +166,7 @@ t13_5 = { status = "pending", commit_sha = "", description = "Final checkpoint + [verification] phase_0_complete = true -phase_1_complete = false +phase_1_complete = true phase_2_complete = false phase_3_complete = false phase_4_complete = false @@ -181,7 +181,7 @@ phase_12_complete = false phase_13_complete = false audit_strict_exits_0 = false batched_suite_11_of_11_pass = false -site_inventory_has_42_rows = false +site_inventory_has_42_rows = true drain_plane_render_functions_exist = false silent_swallow_count_zero = false rethrow_count_zero = false