ed/manual_slop
Private
Public Access
0
0
Fork 0
Code Activity

conductor(checkpoint): Phase 1 complete - 3 production call sites migrated to send_result()

Browse Source 
- src/conductor_tech_lead.py:68 (G1, commit bbb3d597): 2-arg call, no callbacks
- src/orchestrator_pm.py:86 (G2, commit 7ea802ab): 3-arg call with enable_tools
- src/multi_agent_conductor.py:591 (G3, commit bdd46299): 8-arg call with 5 callbacks
  (the hardest; per-ticket error handling routes the error to comms +
  pushes a 'response' event with status='error' + marks ticket.status='error')

Verified: uv run rg 'ai_client\.send\(' src/ returns 0 hits in production code
(line 8 of conductor_tech_lead.py is a docstring mention only).

Pending: 7 test files broken by these production migrations need
send_result() mocks instead of send() mocks. These are scheduled in
Phase 2.12-2.18 (added in the plan update bb3b3056).
This commit is contained in:
ed
2026-06-15 16:01:23 -04:00
parent bdd46299b1
commit b7fd4e4f6a
5 changed files with 44 additions and 92 deletions
Download Patch File Download Diff File Expand all files Collapse all files
conductor/tracks/public_api_migration_and_ui_polish_20260615/state.toml
+6 -6
View File
@@ -16,8 +16,8 @@ data_structure_strengthening_20260606 = "planned in this track"
mcp_architecture_refactor_20260606 = "transitively"
[phases]
phase_1 = { status = "pending", checkpointsha = "", name = "Production call site migration" }
phase_2 = { status = "pending", checkpointsha = "", name = "Test file migration" }
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_4 = { status = "pending", checkpointsha = "", name = "Symbol parsing test fix" }
phase_5 = { status = "pending", checkpointsha = "", name = "UI Polish test fixes" }
@@ -26,10 +26,10 @@ phase_7 = { status = "pending", checkpointsha = "", name = "Docs + housekeep" }
[tasks]
# Phase 1
t1_1 = { status = "pending", commit_sha = "", description = "Migrate src/conductor_tech_lead.py:68 to send_result()" }
t1_2 = { status = "pending", commit_sha = "", description = "Migrate src/orchestrator_pm.py:86 to send_result()" }
t1_3 = { status = "pending", commit_sha = "", description = "Migrate src/multi_agent_conductor.py:591 to send_result()" }
t1_4 = { status = "pending", commit_sha = "", description = "Phase 1 checkpoint" }
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" }
# Phase 2
t2_1 = { status = "pending", commit_sha = "", description = "Migrate test_ai_client_cli.py" }