From dd82e260ee23860d1a036d11fb3ed3feb6c682ad Mon Sep 17 00:00:00 2001 From: starr-openai Date: Tue, 7 Apr 2026 11:41:08 -0700 Subject: [PATCH] codex: avoid remote sandbox type round-trip Use the locally known sandbox type when constructing the remote unified-exec process in the fallback exec-env path. This removes the extra dependency on the exec-server reported sandbox type that reviewers called out. Co-authored-by: Codex --- codex-rs/core/src/unified_exec/process_manager.rs | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/codex-rs/core/src/unified_exec/process_manager.rs b/codex-rs/core/src/unified_exec/process_manager.rs index 459dac6e34..7e22cba13e 100644 --- a/codex-rs/core/src/unified_exec/process_manager.rs +++ b/codex-rs/core/src/unified_exec/process_manager.rs @@ -613,8 +613,7 @@ impl UnifiedExecProcessManager { }) .await .map_err(|err| UnifiedExecError::create_process(err.to_string()))?; - let sandbox_type = started.sandbox_type; - return UnifiedExecProcess::from_remote_started(started, sandbox_type).await; + return UnifiedExecProcess::from_remote_started(started, env.sandbox).await; } let spawn_result = if tty {