Private
Public Access
conductor(plan): Mark Task 2.1 complete
This commit is contained in:
@@ -123,14 +123,14 @@ Verify: 10 references in `src/ai_client.py` are renamed; test suite is in the ex
|
||||
- Modify: `src/multi_agent_conductor.py` (2 refs: 1 call + 1 print)
|
||||
- Modify: `src/orchestrator_pm.py` (2 refs: 1 call + 1 print)
|
||||
|
||||
### Task 2.1: Rename in the 5 other src/ files (single batch commit)
|
||||
### Task 2.1: Rename in the 5 other src/ files (single batch commit) [d87d909]
|
||||
|
||||
- [ ] **Step 1: Identify all references in the 5 files**
|
||||
- [x] **Step 1: Identify all references in the 5 files**
|
||||
|
||||
Run: `git grep -n "send_result" -- src/app_controller.py src/conductor_tech_lead.py src/mcp_client.py src/multi_agent_conductor.py src/orchestrator_pm.py`
|
||||
Expected: 10 lines total (2 + 3 + 1 + 2 + 2 = 10).
|
||||
|
||||
- [ ] **Step 2: Rename each reference**
|
||||
- [x] **Step 2: Rename each reference**
|
||||
|
||||
For each of the 10 references:
|
||||
- `ai_client.send_result(...)` → `ai_client.send(...)` (call sites)
|
||||
@@ -144,12 +144,12 @@ Use the MCP edit tool. Special attention:
|
||||
Verify: `git grep "send_result" -- src/app_controller.py src/conductor_tech_lead.py src/mcp_client.py src/multi_agent_conductor.py src/orchestrator_pm.py`
|
||||
Expected: 0 matches.
|
||||
|
||||
- [ ] **Step 3: Run the test suite — confirm partial green**
|
||||
- [x] **Step 3: Run the test suite — confirm partial green**
|
||||
|
||||
Run: `uv run pytest 2>&1 | tail -3`
|
||||
Expected: still many failures, but fewer than Phase 1. The remaining failures are in test files (which still mock `send_result`).
|
||||
|
||||
- [ ] **Step 4: Commit**
|
||||
- [x] **Step 4: Commit**
|
||||
|
||||
```bash
|
||||
git add src/app_controller.py src/conductor_tech_lead.py src/mcp_client.py src/multi_agent_conductor.py src/orchestrator_pm.py
|
||||
@@ -165,7 +165,7 @@ that still reference send_result).
|
||||
Refs: conductor/tracks/send_result_to_send_20260616/"
|
||||
```
|
||||
|
||||
- [ ] **Step 5: Attach the git note**
|
||||
- [x] **Step 5: Attach the git note**
|
||||
|
||||
```bash
|
||||
git notes add -m "Task 2.1: rename in 5 other src/ files (batch)
|
||||
|
||||
Reference in New Issue
Block a user