Refactor ExecServer filesystem split between local and remote#15232
Merged
pakrym-oai merged 5 commits intomainfrom Mar 20, 2026
Merged
Refactor ExecServer filesystem split between local and remote#15232pakrym-oai merged 5 commits intomainfrom
pakrym-oai merged 5 commits intomainfrom
Conversation
starr-openai
approved these changes
Mar 19, 2026
…rverfilesystem-and # Conflicts: # codex-rs/exec-server/src/server/file_system_handler.rs
starr-openai
added a commit
that referenced
this pull request
Mar 19, 2026
Match the filesystem structure from #15232 for exec process: expose the trait on Environment, keep LocalProcess as the real implementation, use RemoteProcess as the network proxy, and make ProcessHandler a thin RPC adapter over LocalProcess. Co-authored-by: Codex <noreply@openai.com>
starr-openai
added a commit
that referenced
this pull request
Mar 19, 2026
Stack this on top of PR #15232's filesystem split: expose the process trait on Environment, keep LocalProcess as the real implementation, proxy RemoteProcess over ExecServerClient, and route RPCs through a thin ProcessHandler. Co-authored-by: Codex <noreply@openai.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
For each feature we have: