Skip to content

code-mode: move session ownership into runtime#29285

Merged
cconger merged 1 commit into
mainfrom
cconger/code-mode-runtime-compact-03b-session-runtime
Jun 21, 2026
Merged

code-mode: move session ownership into runtime#29285
cconger merged 1 commit into
mainfrom
cconger/code-mode-runtime-compact-03b-session-runtime

Conversation

@cconger

@cconger cconger commented Jun 21, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Move code-mode cell ownership and shared stored values from CodeModeService into SessionRuntime.
  • Keep the protocol-facing execute/wait behavior behind the existing service adapter.
  • Add runtime-level ownership and isolation coverage.

Why

This establishes a transport-neutral session boundary before later lifecycle and create/observe changes.

Impact

No intended model-facing behavior change. This is an ownership and layering refactor.

Validation

  • Stack-tip validation: just test -p codex-code-mode -p codex-code-mode-protocol (70 passed).
  • Parent branch: cconger/code-mode-runtime-compact-03a-runtime-types.

Base automatically changed from cconger/code-mode-runtime-compact-03a-runtime-types to main June 21, 2026 17:49
@cconger cconger force-pushed the cconger/code-mode-runtime-compact-03b-session-runtime branch from 34b6189 to 5d836cb Compare June 21, 2026 18:00
@cconger cconger marked this pull request as ready for review June 21, 2026 18:01
@cconger cconger merged commit 63f009e into main Jun 21, 2026
31 checks passed
@cconger cconger deleted the cconger/code-mode-runtime-compact-03b-session-runtime branch June 21, 2026 18:18
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 21, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants