conductor(checkpoint): Checkpoint end of Phase 3
This commit is contained in:
@@ -48,11 +48,16 @@
|
|||||||
- [ ] WHAT: Fix `AssertionError: Expected at least 2 entries, found 0`.
|
- [ ] 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.
|
- [ ] 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.
|
- [ ] SAFETY: Use dynamic wait (`ApiHookClient.wait_for_event`) instead of static sleeps.
|
||||||
- [~] Task: Remove Legacy `gui_legacy` Test Imports & File
|
- [x] Task: Remove Legacy `gui_legacy` Test Imports & File [4d171ff]
|
||||||
- [~] WHERE: `tests/test_gui_events.py`, `tests/test_gui_updates.py`, `tests/test_gui_diagnostics.py`, and project root.
|
- [x] 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`.
|
- [x] 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`.
|
- [x] HOW: String replacement and standard `os.remove`.
|
||||||
- [~] SAFETY: Verify no remaining imports exist across the suite using `grep_search`.
|
- [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)
|
- [ ] Task: Conductor - User Manual Verification 'Phase 3: Assertions & Legacy Cleanup' (Protocol in workflow.md)
|
||||||
|
|
||||||
## Phase 4: Documentation & Final Verification
|
## Phase 4: Documentation & Final Verification
|
||||||
|
|||||||
Reference in New Issue
Block a user