Private
Public Access
plan update for (send_result-to_send)
This commit is contained in:
@@ -49,19 +49,19 @@
|
|||||||
**Files:**
|
**Files:**
|
||||||
- Modify: `src/ai_client.py:1-...` (10 refs throughout the file)
|
- Modify: `src/ai_client.py:1-...` (10 refs throughout the file)
|
||||||
|
|
||||||
### Task 1.1: Rename `send_result` → `send` in `src/ai_client.py`
|
### Task 1.1: Rename `send_result` → `send` in `src/ai_client.py` [5351389]
|
||||||
|
|
||||||
- [ ] **Step 1: Snapshot the pre-rename state**
|
- [x] **Step 1: Snapshot the pre-rename state**
|
||||||
|
|
||||||
Run: `uv run pytest 2>&1 | tail -3`
|
Run: `uv run pytest 2>&1 | tail -3`
|
||||||
Expected: a line like `=== X passed in Y.YYs ===` where X is the current passing count. Record this number mentally as the "before" baseline.
|
Expected: a line like `=== X passed in Y.YYs ===` where X is the current passing count. Record this number mentally as the "before" baseline.
|
||||||
|
|
||||||
- [ ] **Step 2: Identify all 10 references in `src/ai_client.py`**
|
- [x] **Step 2: Identify all 10 references in `src/ai_client.py`**
|
||||||
|
|
||||||
Run: `git grep -n "send_result" -- src/ai_client.py`
|
Run: `git grep -n "send_result" -- src/ai_client.py`
|
||||||
Expected: 10 lines, all in `src/ai_client.py`. Each line shows the line number and the context.
|
Expected: 10 lines, all in `src/ai_client.py`. Each line shows the line number and the context.
|
||||||
|
|
||||||
- [ ] **Step 3: Rename each reference**
|
- [x] **Step 3: Rename each reference**
|
||||||
|
|
||||||
For each of the 10 references:
|
For each of the 10 references:
|
||||||
- `def send_result(` → `def send(`
|
- `def send_result(` → `def send(`
|
||||||
@@ -75,12 +75,12 @@ Use the MCP edit tool. Verify the rename is complete:
|
|||||||
Run: `git grep "send_result" -- src/ai_client.py`
|
Run: `git grep "send_result" -- src/ai_client.py`
|
||||||
Expected: 0 matches (the grep returns nothing).
|
Expected: 0 matches (the grep returns nothing).
|
||||||
|
|
||||||
- [ ] **Step 4: Run the test suite — confirm the "red"**
|
- [x] **Step 4: Run the test suite — confirm the "red"**
|
||||||
|
|
||||||
Run: `uv run pytest 2>&1 | tail -10`
|
Run: `uv run pytest 2>&1 | tail -10`
|
||||||
Expected: many test failures with `AttributeError: module 'src.ai_client' has no attribute 'send_result'` (or `AttributeError: <module> has no attribute 'send_result'` from monkeypatch.setattr). This is the TDD red moment. **Do not panic; this is expected.**
|
Expected: many test failures with `AttributeError: module 'src.ai_client' has no attribute 'send_result'` (or `AttributeError: <module> has no attribute 'send_result'` from monkeypatch.setattr). This is the TDD red moment. **Do not panic; this is expected.**
|
||||||
|
|
||||||
- [ ] **Step 5: Commit the red moment**
|
- [x] **Step 5: Commit the red moment**
|
||||||
|
|
||||||
```bash
|
```bash
|
||||||
git add src/ai_client.py
|
git add src/ai_client.py
|
||||||
@@ -94,7 +94,7 @@ back to green.
|
|||||||
Refs: conductor/tracks/send_result_to_send_20260616/"
|
Refs: conductor/tracks/send_result_to_send_20260616/"
|
||||||
```
|
```
|
||||||
|
|
||||||
- [ ] **Step 6: Attach the git note**
|
- [x] **Step 6: Attach the git note**
|
||||||
|
|
||||||
```bash
|
```bash
|
||||||
git notes add -m "Task 1.1: rename send_result to send in src/ai_client.py
|
git notes add -m "Task 1.1: rename send_result to send in src/ai_client.py
|
||||||
|
|||||||
Reference in New Issue
Block a user