checkpoint: result_migration_app_controller_20260618 (sub-track 3 of 5)

Browse Source

Sub-track 3 of the result_migration_20260616 umbrella. Migrates 45 sites
in src/app_controller.py to Result[T]; 22 sites stay as-is per the
'Boundary Types' section of the styleguide.

The 4 planning artifacts (spec.md, plan.md, metadata.json, state.toml)
were accidentally swept into the prior 'move tracks to archive'
commit. This empty checkpoint commit records the milestone.

Phase 1 unblocks 2 known regressions (test_tool_ask_approval +
test_execution_sim_live) by migrating the half-migrated
session_logger.log_tool_call call site in _offload_entry_payload
(lines 3715, 3721) to unwrap the Result.

Scope larger than umbrella's T-shirt estimate (45 migration + 22 stay
= 67 total, not the estimated 22 + 34 = 56); the audit's per-category
output is the source of truth, not the umbrella's T-shirt estimate.

Refs: conductor/tracks/result_migration_20260616 (umbrella)

...

This commit is contained in:

ed

2026-06-18 18:53:47 -04:00

parent 93d906fb7b

commit 5c72ad9a92

Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL

Download Patch File Download Diff File Expand all files Collapse all files

Diff Content Not Available