Skip to content

Add support for UDS in codex --remote#22414

Merged
etraut-openai merged 1 commit into
mainfrom
etraut/remote-uds-and-ws
May 13, 2026
Merged

Add support for UDS in codex --remote#22414
etraut-openai merged 1 commit into
mainfrom
etraut/remote-uds-and-ws

Conversation

@etraut-openai
Copy link
Copy Markdown
Collaborator

@etraut-openai etraut-openai commented May 13, 2026

Why

Added support for UDS connections in codex --remote.

TUI also now connects to local app-server using UDS by default if it is running and set to listen to UDS connection.

What Changed

  • Introduced RemoteAppServerEndpoint with WebSocket and UnixSocket variants.
  • Reused the existing JSON-RPC-over-WebSocket protocol over either a TCP WebSocket stream or a UDS stream.
  • Updated codex --remote to accept ws://host:port, wss://host:port, unix://, and unix://PATH.
  • Kept --remote-auth-token-env restricted to wss:// and loopback ws:// remotes.
  • Added a fast TUI startup probe for the default daemon socket, falling back to the embedded app server when the daemon is absent or unresponsive.

Verification

  • Manually verified that the updated remote flow works.
  • Added coverage for UDS remote round trips, WebSocket auth headers, auth-token transport policy, remote address parsing, and missing-daemon fallback.
  • Ran focused remote test coverage locally.

@etraut-openai etraut-openai changed the title Support UDS and websocket remotes Add support for UDS in codex --remote May 13, 2026
@etraut-openai etraut-openai force-pushed the etraut/remote-uds-and-ws branch 2 times, most recently from dbc508a to d6dc60c Compare May 13, 2026 03:08
Comment thread codex-rs/tui/src/lib.rs Outdated
@etraut-openai etraut-openai force-pushed the etraut/remote-uds-and-ws branch from d6dc60c to f31f1b6 Compare May 13, 2026 03:57
Copy link
Copy Markdown
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: f31f1b6804

ℹ️ 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".

Comment thread codex-rs/app-server-client/Cargo.toml
@etraut-openai etraut-openai merged commit ad57270 into main May 13, 2026
27 checks passed
@etraut-openai etraut-openai deleted the etraut/remote-uds-and-ws branch May 13, 2026 04:17
@github-actions github-actions Bot locked and limited conversation to collaborators May 13, 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.

2 participants