Skip to content

code-mode: define transport-neutral runtime types#29170

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

code-mode: define transport-neutral runtime types#29170
cconger merged 1 commit into
mainfrom
cconger/code-mode-runtime-compact-03a-runtime-types

Conversation

@cconger

@cconger cconger commented Jun 20, 2026

Copy link
Copy Markdown
Contributor

Summary

  • introduce a private session_runtime boundary for cell creation requests, observation modes, lifecycle events, output items, and tool metadata
  • update the cell actor and in-process service to use those transport-neutral types
  • keep cell ID allocation on the owning session side

Motivation

Cell lifecycle vocabulary currently lives inside the cell actor implementation. That makes the service adapter and future session runtime depend on actor-specific types, increasing the size and complexity of the runtime ownership change.

This is the first reviewable slice of the session-runtime stack. It separates the transport-neutral data model without moving lifecycle ownership or changing behavior.

Later slices will move session state behind this boundary, harden terminal and shutdown behavior, and split cell creation from observation.

Behavior

There are no public API or user-visible behavior changes in this PR.

In particular:

  • CodeModeSession::execute and wait are unchanged
  • cell IDs remain allocated by the owning session
  • cell admission, observation, termination, and shutdown behavior are unchanged

@cconger
cconger merged commit 6d993ca into main Jun 21, 2026
31 checks passed
@cconger
cconger deleted the cconger/code-mode-runtime-compact-03a-runtime-types branch June 21, 2026 17:49
@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