diff --git a/conductor/tracks/result_migration_app_controller_20260618/plan.md b/conductor/tracks/result_migration_app_controller_20260618/plan.md index 34600fb2..0a7eba93 100644 --- a/conductor/tracks/result_migration_app_controller_20260618/plan.md +++ b/conductor/tracks/result_migration_app_controller_20260618/plan.md @@ -75,12 +75,16 @@ Focus: register the sub-track, then immediately fix the 2 known regressions (tes - **COMMIT:** `test(app_controller): offloading - verify Result unwrap in success and error paths` - **GIT NOTE:** Tests for FR5; covers the regression from task 1.3. +- [x] **Task 1.5: Run the regression test and confirm both fixes** [7b823fd0] + ### Task 1.5: Run the regression test and confirm both fixes - **COMMAND:** `uv run python -m pytest tests/test_tool_presets_execution.py::test_tool_ask_approval tests/test_extended_sims.py::test_execution_sim_live -v` - **EXPECT:** Both pass. - **COMMIT:** No commit (verification only). - **NOTE:** If `test_execution_sim_live` still fails, investigate the failure mode (may be a separate issue from Regression 1). +- [x] **Task 1.6: Phase 1 checkpoint commit** [7b823fd0] + ### Task 1.6: Phase 1 checkpoint commit - **COMMIT:** `conductor(plan): mark Phase 1 complete (regression fix)` - **GIT NOTE:** Phase 1 = 2 known regressions fixed; verified by `test_tool_ask_approval` + `test_execution_sim_live`. Now safe to proceed with the bulk migration.