From 14ac9830f01e4cbe74118f7837c2e01ab28582c7 Mon Sep 17 00:00:00 2001 From: Ed_ Date: Tue, 3 Mar 2026 01:11:09 -0500 Subject: [PATCH] conductor(checkpoint): Checkpoint end of Phase 3 --- .../tracks/test_stabilization_20260302/plan.md | 15 ++++++++++----- 1 file changed, 10 insertions(+), 5 deletions(-) diff --git a/conductor/tracks/test_stabilization_20260302/plan.md b/conductor/tracks/test_stabilization_20260302/plan.md index a137537..f603ae8 100644 --- a/conductor/tracks/test_stabilization_20260302/plan.md +++ b/conductor/tracks/test_stabilization_20260302/plan.md @@ -48,11 +48,16 @@ - [ ] WHAT: Fix `AssertionError: Expected at least 2 entries, found 0`. - [ ] HOW: Update simulation flow to properly wait for the `User` and `AI` entries to populate the GUI history before asserting. - [ ] SAFETY: Use dynamic wait (`ApiHookClient.wait_for_event`) instead of static sleeps. -- [~] Task: Remove Legacy `gui_legacy` Test Imports & File - - [~] WHERE: `tests/test_gui_events.py`, `tests/test_gui_updates.py`, `tests/test_gui_diagnostics.py`, and project root. - - [~] WHAT: Change `from gui_legacy import App` to `from gui_2 import App`. Fix any breaking UI locators. Then delete `gui_legacy.py`. - - [~] HOW: String replacement and standard `os.remove`. - - [~] SAFETY: Verify no remaining imports exist across the suite using `grep_search`. +- [x] Task: Remove Legacy `gui_legacy` Test Imports & File [4d171ff] + - [x] WHERE: `tests/test_gui_events.py`, `tests/test_gui_updates.py`, `tests/test_gui_diagnostics.py`, and project root. + - [x] WHAT: Change `from gui_legacy import App` to `from gui_2 import App`. Fix any breaking UI locators. Then delete `gui_legacy.py`. + - [x] HOW: String replacement and standard `os.remove`. + - [x] SAFETY: Verify no remaining imports exist across the suite using `grep_search`. +- [x] Task: Resolve `pytest.fail` in `tests/test_agent_tools_wiring.py` [20b2e2d] + - [x] WHERE: `tests/test_agent_tools_wiring.py`. + - [x] WHAT: Implement actual assertions for `test_set_agent_tools`. + - [x] HOW: Verify that `ai_client.set_agent_tools` correctly updates the active tool set. + - [x] SAFETY: Use mocks for `ai_client` if necessary. - [ ] Task: Conductor - User Manual Verification 'Phase 3: Assertions & Legacy Cleanup' (Protocol in workflow.md) ## Phase 4: Documentation & Final Verification