Skip to content

Handle realtime conversation end in the TUI#14903

Merged
aibrahim-oai merged 35 commits intomainfrom
codex/realtime-end-tui
Mar 18, 2026
Merged

Handle realtime conversation end in the TUI#14903
aibrahim-oai merged 35 commits intomainfrom
codex/realtime-end-tui

Conversation

@aibrahim-oai
Copy link
Collaborator

  • close live realtime sessions on errors, ctrl-c, and active meter removal
  • centralize TUI realtime cleanup and avoid duplicate follow-up close info

aibrahim-oai and others added 2 commits March 17, 2026 00:11
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
@aibrahim-oai aibrahim-oai force-pushed the codex/realtime-end-tui branch from 8644bcb to 5c75c37 Compare March 17, 2026 16:34
aibrahim-oai and others added 2 commits March 17, 2026 09:46
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
@aibrahim-oai aibrahim-oai force-pushed the codex/realtime-end-tui branch from 5c75c37 to aae9d44 Compare March 17, 2026 16:51
Co-authored-by: Codex <noreply@openai.com>
@aibrahim-oai aibrahim-oai force-pushed the codex/realtime-end-tui branch from aae9d44 to 9ab7543 Compare March 17, 2026 16:59
Co-authored-by: Codex <noreply@openai.com>
@aibrahim-oai aibrahim-oai force-pushed the codex/realtime-end-tui branch from 9ab7543 to a130fef Compare March 17, 2026 17:34
Co-authored-by: Codex <noreply@openai.com>
@aibrahim-oai aibrahim-oai force-pushed the codex/realtime-end-tui branch from a130fef to c946e24 Compare March 17, 2026 17:50
aibrahim-oai and others added 2 commits March 17, 2026 11:14
Co-own the realtime fanout task so shutdown stops the full session before closed is emitted. Also keep start/connect failures as error-only until started is sent.\n\nCo-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
@aibrahim-oai aibrahim-oai force-pushed the codex/realtime-end-tui branch from c946e24 to 847245d Compare March 17, 2026 18:14
@aibrahim-oai
Copy link
Collaborator Author

@codex review

Co-authored-by: Codex <noreply@openai.com>
Copy link
Contributor

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 847245d9fb

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

aibrahim-oai and others added 12 commits March 17, 2026 20:09
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
- mirror realtime close handling, ctrl-c behavior, and
placeholder-triggered teardown in `tui_app_server`
- keep the app-server TUI tests aligned with the direct TUI lifecycle

Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
aibrahim-oai and others added 2 commits March 17, 2026 22:48
Co-authored-by: Codex <noreply@openai.com>
Base automatically changed from codex/realtime-end-core to main March 17, 2026 22:58
aibrahim-oai and others added 11 commits March 17, 2026 15:59
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
@aibrahim-oai aibrahim-oai merged commit 3ce879c into main Mar 18, 2026
54 of 56 checks passed
@aibrahim-oai aibrahim-oai deleted the codex/realtime-end-tui branch March 18, 2026 04:05
@github-actions github-actions bot locked and limited conversation to collaborators Mar 18, 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