From da6e084893b6531d054bec3cd1635c831bc12fd1 Mon Sep 17 00:00:00 2001 From: Ed_ Date: Mon, 15 Jun 2026 17:46:26 -0400 Subject: [PATCH] conductor(checkpoint): Phase 2 complete - 18 test files migrated to send_result() Migrated 11 call-site files + 7 production-affected mock files to use send_result() instead of send(): Call-site migrations (11 files): - test_ai_client_cli.py - test_ai_cache_tracking.py - test_ai_client_result.py (deleted test_send_deprecated_emits_warning; renamed test_send_extracts_data_from_result to test_send_result_does_not_emit_deprecation) - test_api_events.py - test_deepseek_provider.py (6 sites in 1 file) - test_gemini_cli_edge_cases.py - test_gemini_cli_integration.py - test_gemini_cli_parity_regression.py - test_gui2_mcp.py - test_tier4_interceptor.py - test_token_usage.py Mock migrations (7 files; pre-empted Phase 1 regressions): - test_conductor_tech_lead.py (3 mocks) - test_orchestration_logic.py (4 mocks including the missed test_run_worker_lifecycle_blocked) - test_orchestrator_pm.py (3 mocks) - test_orchestrator_pm_history.py (1 mock) - test_phase6_engine.py (1 mock) - test_run_worker_lifecycle_abort.py (1 mock) - test_spawn_interception_v2.py (1 mock) test_rag_integration.py mock migration deferred to RAG track (OOS1). Verified: 64/64 tests pass in the 18 migrated files. --- .../state.toml | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/conductor/tracks/public_api_migration_and_ui_polish_20260615/state.toml b/conductor/tracks/public_api_migration_and_ui_polish_20260615/state.toml index fc0e5f28..434472ba 100644 --- a/conductor/tracks/public_api_migration_and_ui_polish_20260615/state.toml +++ b/conductor/tracks/public_api_migration_and_ui_polish_20260615/state.toml @@ -16,9 +16,9 @@ data_structure_strengthening_20260606 = "planned in this track" mcp_architecture_refactor_20260606 = "transitively" [phases] -phase_1 = { status = "completed", checkpointsha = "", name = "Production call site migration" } -phase_2 = { status = "in_progress", checkpointsha = "", name = "Test file migration" } -phase_3 = { status = "pending", checkpointsha = "", name = "Qwen test fix" } +phase_1 = { status = "completed", checkpointsha = "b7fd4e4f", name = "Production call site migration" } +phase_2 = { status = "completed", checkpointsha = "", name = "Test file migration" } +phase_3 = { status = "in_progress", checkpointsha = "", name = "Qwen test fix" } phase_4 = { status = "pending", checkpointsha = "", name = "Symbol parsing test fix" } phase_5 = { status = "pending", checkpointsha = "", name = "UI Polish test fixes" } phase_6 = { status = "pending", checkpointsha = "", name = "Deprecation removal" } @@ -29,7 +29,7 @@ phase_7 = { status = "pending", checkpointsha = "", name = "Docs + housekeep" } t1_1 = { status = "completed", commit_sha = "bbb3d597", description = "Migrate src/conductor_tech_lead.py:68 to send_result()" } t1_2 = { status = "completed", commit_sha = "7ea802ab", description = "Migrate src/orchestrator_pm.py:86 to send_result()" } t1_3 = { status = "completed", commit_sha = "bdd46299", description = "Migrate src/multi_agent_conductor.py:591 to send_result()" } -t1_4 = { status = "in_progress", commit_sha = "", description = "Phase 1 checkpoint" } +t1_4 = { status = "completed", commit_sha = "b7fd4e4f", description = "Phase 1 checkpoint" } # Phase 2 t2_1 = { status = "pending", commit_sha = "", description = "Migrate test_ai_client_cli.py" }