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

conductor(plan): Mark Task 1.3 complete (regression fix for _offload_entry_payload)

Browse Source 
Task 1.3: src/app_controller.py _offload_entry_payload now unwraps the Result
returned by session_logger.log_tool_call. The half-migrated function returned
Result[data=str | None] but the call site did Path(ref_path).name, raising
TypeError on every tool_call event.

Refs: 26e57577
This commit is contained in:
Edward R. Gonzalez
2026-06-18 19:32:52 -04:00
parent 26e5757760
commit e8a4ede534
1 changed files with 2 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
conductor/tracks/result_migration_app_controller_20260618/plan.md
+2
View File
@@ -28,6 +28,8 @@ Focus: register the sub-track, then immediately fix the 2 known regressions (tes
- **COMMIT:** `conductor: register result_migration_app_controller_20260618 in tracks.md`
- **GIT NOTE:** 1-sentence note
- [x] **Task 1.3: Fix `_offload_entry_payload` call site (Regression 1)** [26e57577]
### Task 1.3: Fix `_offload_entry_payload` call site (Regression 1)
- **WHERE:** `src/app_controller.py:3709-3725` (`_offload_entry_payload` method)
- **WHAT:** Unwrap the `Result` returned by `session_logger.log_tool_output` and `session_logger.log_tool_call`. The current code does `Path(ref_path).name` where `ref_path` is a `Result` object — `Path()` expects a string.