Skip to content

Separate session state from session I/O - #33209

Merged
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/8523bad7e7366e2969fd0cc3849b4247b141427a
Jul 15, 2026
Merged

Separate session state from session I/O#33209
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/8523bad7e7366e2969fd0cc3849b4247b141427a

Conversation

@copyberry

@copyberry copyberry Bot commented Jul 15, 2026

Copy link
Copy Markdown

Separate session state from session I/O

What changed

  • Replace the combined Codex wrapper with an Arc<Session> and a dedicated
    SessionIo handle for submissions, events, status, and loop termination.
  • Make Session::spawn return the session and I/O handle separately, and have
    CodexThread own both.
  • Update thread management, delegated sessions, guardian reviews, and agent
    control to use the appropriate session or I/O interface.

## What changed

- Replace the combined `Codex` wrapper with an `Arc<Session>` and a dedicated
  `SessionIo` handle for submissions, events, status, and loop termination.
- Make `Session::spawn` return the session and I/O handle separately, and have
  `CodexThread` own both.
- Update thread management, delegated sessions, guardian reviews, and agent
  control to use the appropriate session or I/O interface.

GitOrigin-RevId: 8523bad7e7366e2969fd0cc3849b4247b141427a
@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/8523bad7e7366e2969fd0cc3849b4247b141427a branch from ac79f84 to e4711f2 Compare July 15, 2026 02:17
@copyberry
copyberry Bot merged commit e4711f2 into main Jul 15, 2026
14 of 32 checks passed
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/8523bad7e7366e2969fd0cc3849b4247b141427a branch July 15, 2026 02:18
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 15, 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.

3 participants