Skip to content

Move child completion routing into the agent controller - #46552

Merged
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/e06871d92c9ca8660de29f50356cf7c37f7e5199
Sep 19, 2026
Merged

copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/e06871d92c9ca8660de29f50356cf7c37f7e5199

Conversation

@copyberry

@copyberry copyberry Bot commented Sep 19, 2026 •

Copy link
Copy Markdown

Move child completion routing into the agent controller

What changed

Have Session capture terminal child state in an AgentTurnOutcome and delegate delivery to LocalAgentControl. Move parent notification and completion activity routing into agent/control/completion.rs, so routing no longer takes the child's live TurnContext.

Preserve best-effort, queue-only parent delivery, completion activity for the initiating agent, and result tracing only after successful delivery.

## What changed

Have `Session` capture terminal child state in an `AgentTurnOutcome` and delegate delivery to `LocalAgentControl`. Move parent notification and completion activity routing into `agent/control/completion.rs`, so routing no longer takes the child's live `TurnContext`.

Preserve best-effort, queue-only parent delivery, completion activity for the initiating agent, and result tracing only after successful delivery.

GitOrigin-RevId: e06871d92c9ca8660de29f50356cf7c37f7e5199
@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/e06871d92c9ca8660de29f50356cf7c37f7e5199 branch from 6b8acd5 to 7a15548 Compare September 19, 2026 01:20
@copyberry
copyberry Bot merged commit 7a15548 into main Sep 19, 2026
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/e06871d92c9ca8660de29f50356cf7c37f7e5199 branch September 19, 2026 01:20
@github-actions github-actions Bot locked and limited conversation to collaborators Sep 19, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant