Skip to content

Add minimal exec-server websocket reconnect#23529

Closed
starr-openai wants to merge 7 commits into
mainfrom
starr/exec-reconnect-min-cand-a
Closed

Add minimal exec-server websocket reconnect#23529
starr-openai wants to merge 7 commits into
mainfrom
starr/exec-reconnect-min-cand-a

Conversation

@starr-openai
Copy link
Copy Markdown
Contributor

Summary

  • add minimal websocket-only exec-server client reconnect for later new remote ops
  • cache terminal resume failures and keep stdio one-shot
  • add Docker-backed remote env regression that drops the live exec-server websocket and verifies the next remote filesystem op reconnects

Validation

  • bazel test --bes_backend= --bes_results_url= //codex-rs/exec-server:exec-server-unit-tests
  • bazel build --bes_backend= --bes_results_url= //codex-rs/exec-server:exec-server
  • bazel build --bes_backend= --bes_results_url= //codex-rs/core:core-all-test
  • source scripts/test-remote-env.sh; cd codex-rs; cargo test -p codex-core --test all remote_test_env_reconnects_after_exec_server_websocket_disconnect -- --nocapture

@starr-openai starr-openai requested a review from a team as a code owner May 19, 2026 19:07
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: 27a604b7b8

ℹ️ 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/exec-server/src/client.rs Outdated
Comment thread codex-rs/exec-server/src/client.rs Outdated
Comment thread codex-rs/core/tests/suite/remote_env.rs
@starr-openai
Copy link
Copy Markdown
Contributor Author

Resubmitted as #23562 from a full-ci branch name so rust-ci-full runs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant