Skip to content

Refactor ExecServer filesystem split between local and remote#15232

Merged
pakrym-oai merged 5 commits intomainfrom
pakrym/compare-execserverfilesystem-and
Mar 20, 2026
Merged

Refactor ExecServer filesystem split between local and remote#15232
pakrym-oai merged 5 commits intomainfrom
pakrym/compare-execserverfilesystem-and

Conversation

@pakrym-oai
Copy link
Copy Markdown
Collaborator

@pakrym-oai pakrym-oai commented Mar 19, 2026

For each feature we have:

  1. Trait exposed on environment
  2. Local Implementation of the trait
  3. Remote implementation that uses the client to proxy via network
  4. Handler implementation that handles PRC requests and calls into Local Implementation
…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>
@pakrym-oai pakrym-oai merged commit 403b397 into main Mar 20, 2026
33 checks passed
@pakrym-oai pakrym-oai deleted the pakrym/compare-execserverfilesystem-and branch March 20, 2026 00:08
@github-actions github-actions bot locked and limited conversation to collaborators Mar 20, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

2 participants