Skip to content

Preserve side conversations across thread navigation - #52184

Merged
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/ff80566970414ba190ff650d5e413ef63dc65449
Oct 8, 2026
Merged

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

Conversation

@copyberry

@copyberry copyberry Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

Preserve side conversations across thread navigation

Why

Navigating to another thread discarded side conversations, preventing users from returning to them later.

What changed

  • Keep side conversations and their event listeners, channels, and pending requests across thread switches and session resets.
  • Reopen a parent's existing side conversation with /side and submit any supplied question there. Resolve side toggles by the current thread's parent relationship.
  • Limit explicit return-and-discard behavior to the active side conversation, and shut down retained side conversations when exiting.

Testing

Extend regression tests to cover reopening a side conversation after navigating away, preserving its listener and pending approval during a reset, and restricting discard selection to the active side.

## Why

Navigating to another thread discarded side conversations, preventing users from returning to them later.

## What changed

- Keep side conversations and their event listeners, channels, and pending requests across thread switches and session resets.
- Reopen a parent's existing side conversation with `/side` and submit any supplied question there. Resolve side toggles by the current thread's parent relationship.
- Limit explicit return-and-discard behavior to the active side conversation, and shut down retained side conversations when exiting.

## Testing

Extend regression tests to cover reopening a side conversation after navigating away, preserving its listener and pending approval during a reset, and restricting discard selection to the active side.

GitOrigin-RevId: ff80566970414ba190ff650d5e413ef63dc65449
@chatgpt-codex-connector

Copy link
Copy Markdown
Contributor

Note

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/ff80566970414ba190ff650d5e413ef63dc65449 branch from 280dfb5 to 5fe2152 Compare October 8, 2026 17:31
@copyberry
copyberry Bot merged commit 5fe2152 into main Oct 8, 2026
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/ff80566970414ba190ff650d5e413ef63dc65449 branch October 8, 2026 17:31
@danlflh
danlflh deployed to issue-triage October 8, 2026 17:31 — with Image GitHub Actions Active
@danlflh
danlflh deployed to issue-triage October 8, 2026 17:31 — with Image GitHub Actions Active
@danlflh
danlflh deployed to issue-triage October 8, 2026 17:31 — with Image GitHub Actions Active
@github-actions github-actions Bot locked and limited conversation to collaborators Oct 8, 2026
@tsoernes
tsoernes deployed to issue-triage October 8, 2026 17:32 — with Image GitHub Actions Active
@tsoernes
tsoernes deployed to issue-triage October 8, 2026 17:32 — with Image GitHub Actions Active
@tsoernes
tsoernes deployed to issue-triage October 8, 2026 17:32 — with Image GitHub Actions Active
@tsoernes
tsoernes deployed to issue-triage October 8, 2026 17:33 — with Image GitHub Actions Active
@vision0529
vision0529 deployed to issue-triage October 8, 2026 17:38 — with Image GitHub Actions Active
@vision0529
vision0529 deployed to issue-triage October 8, 2026 17:38 — with Image GitHub Actions Active
@vision0529
vision0529 deployed to issue-triage October 8, 2026 17:38 — with Image GitHub Actions Active
@heintruong
heintruong deployed to issue-triage October 8, 2026 17:39 — with Image GitHub Actions Active
@heintruong
heintruong deployed to issue-triage October 8, 2026 17:39 — with Image GitHub Actions Active
@heintruong
heintruong deployed to issue-triage October 8, 2026 17:39 — with Image GitHub Actions Active
@vision0529
vision0529 deployed to issue-triage October 8, 2026 17:39 — with Image GitHub Actions Active
@heintruong
heintruong deployed to issue-triage October 8, 2026 17:40 — with Image GitHub Actions Active

This branch was successfully deployed

1 active deployment
issue-triage — 5fe2152b Deployed Oct 8, 2026 by don-ludemann-aa via Translate non-English issue #18502
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.

9 participants