Lean child handoffs
Authored by
mfwolffe <wolffemf@dukes.jmu.edu>
- SHA
89dbb361657a7a97ba84fb4bcacbe993eeeada8f- Parents
-
a1cd87d - Tree
8ea86e7
tenseleyflow/loader
/
89dbb36
Browse files
2 changed files
| Status | File | + | - |
|---|---|---|---|
| M |
src/loader/runtime/tool_batches.py
|
3 | 9 |
| M |
tests/test_tool_batches.py
|
3 | 3 |
src/loader/runtime/tool_batches.pymodified@@ -1370,7 +1370,7 @@ class ToolBatchRunner: | ||
| 1370 | 1370 | (resume_target, False), |
| 1371 | 1371 | project_root=self.context.project_root, |
| 1372 | 1372 | messages=session_messages, |
| 1373 | - encourage_initial_version=not has_substantive_file_artifact_progress, | |
| 1373 | + encourage_initial_version=False, | |
| 1374 | 1374 | ) |
| 1375 | 1375 | if compact_resume: |
| 1376 | 1376 | queue_message( |
@@ -1388,7 +1388,7 @@ class ToolBatchRunner: | ||
| 1388 | 1388 | missing_artifact, |
| 1389 | 1389 | project_root=self.context.project_root, |
| 1390 | 1390 | messages=session_messages, |
| 1391 | - encourage_initial_version=True, | |
| 1391 | + encourage_initial_version=False, | |
| 1392 | 1392 | ) |
| 1393 | 1393 | if compact_handoff: |
| 1394 | 1394 | queue_message( |
@@ -1531,12 +1531,6 @@ class ToolBatchRunner: | ||
| 1531 | 1531 | ) |
| 1532 | 1532 | return |
| 1533 | 1533 | |
| 1534 | - has_substantive_file_artifact_progress = ( | |
| 1535 | - _has_confirmed_substantive_file_artifact_progress( | |
| 1536 | - dod, | |
| 1537 | - project_root=self.context.project_root, | |
| 1538 | - ) | |
| 1539 | - ) | |
| 1540 | 1534 | todo_refresh = _todo_refresh_guidance( |
| 1541 | 1535 | dod, |
| 1542 | 1536 | project_root=self.context.project_root, |
@@ -1563,7 +1557,7 @@ class ToolBatchRunner: | ||
| 1563 | 1557 | (resume_target, False), |
| 1564 | 1558 | project_root=self.context.project_root, |
| 1565 | 1559 | messages=session_messages, |
| 1566 | - encourage_initial_version=not has_substantive_file_artifact_progress, | |
| 1560 | + encourage_initial_version=False, | |
| 1567 | 1561 | ) |
| 1568 | 1562 | if compact_resume: |
| 1569 | 1563 | self.context.queue_steering_message( |
tests/test_tool_batches.pymodified@@ -2475,7 +2475,7 @@ async def test_tool_batch_runner_first_chapter_handoff_stays_persistent_until_su | ||
| 2475 | 2475 | f"Prefer one `write(file_path=..., content=...)` call for `{(chapters / '01-introduction.html').resolve(strict=False)}` now." |
| 2476 | 2476 | in message |
| 2477 | 2477 | ) |
| 2478 | - assert "Write a compact but real initial version of that file now" in message | |
| 2478 | + assert "Write a compact but real initial version of that file now" not in message | |
| 2479 | 2479 | assert "Do not reread reference material or spend the next turn on bookkeeping." in message |
| 2480 | 2480 | |
| 2481 | 2481 | |
@@ -2808,7 +2808,7 @@ async def test_tool_batch_runner_softens_first_file_handoff_after_recovery_promp | ||
| 2808 | 2808 | assert ephemeral_messages |
| 2809 | 2809 | message = ephemeral_messages[-1] |
| 2810 | 2810 | assert "Next step: create `01-introduction.html`." in message |
| 2811 | - assert "Write a compact but real initial version of that file now" in message | |
| 2811 | + assert "Write a compact but real initial version of that file now" not in message | |
| 2812 | 2812 | |
| 2813 | 2813 | |
| 2814 | 2814 | @pytest.mark.asyncio |
@@ -3267,7 +3267,7 @@ async def test_tool_batch_runner_mutation_handoff_points_at_next_missing_artifac | ||
| 3267 | 3267 | assert ephemeral_messages == [] |
| 3268 | 3268 | message = persistent_messages[-1] |
| 3269 | 3269 | assert "Next step: create `01-getting-started.html`." in message |
| 3270 | - assert "Write a compact but real initial version of that file now" in message | |
| 3270 | + assert "Write a compact but real initial version of that file now" not in message | |
| 3271 | 3271 | assert "refresh `TodoWrite`" not in message |
| 3272 | 3272 | assert "Do not reread reference material or spend the next turn on bookkeeping." in message |
| 3273 | 3273 | |