From 78cdba69c22d90d613eda3fdfbcf550c397bcee6 Mon Sep 17 00:00:00 2001 From: starr-openai Date: Tue, 12 May 2026 16:05:00 -0700 Subject: [PATCH] codex: address PR review feedback (#22200) Co-authored-by: Codex --- codex-rs/core/src/tools/runtimes/unified_exec.rs | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/codex-rs/core/src/tools/runtimes/unified_exec.rs b/codex-rs/core/src/tools/runtimes/unified_exec.rs index fbf7c2a84b..6f0f01b77f 100644 --- a/codex-rs/core/src/tools/runtimes/unified_exec.rs +++ b/codex-rs/core/src/tools/runtimes/unified_exec.rs @@ -280,7 +280,11 @@ impl<'a> ToolRuntime for UnifiedExecRunt if let UnifiedExecShellMode::ZshFork(zsh_fork_config) = &self.shell_mode { let command = build_sandbox_command(&command, &req.cwd, &env, req.additional_permissions.clone()) - .map_err(|_| ToolError::Codex(CodexErr::InvalidRequest("missing command line for PTY".into())))?; + .map_err(|_| { + ToolError::Codex(CodexErr::InvalidRequest( + "missing command line for PTY".into(), + )) + })?; let options = unified_exec_options(attempt.network_denial_cancellation_token.clone()); let mut exec_env = attempt .env_for(command, options, managed_network) @@ -331,7 +335,11 @@ impl<'a> ToolRuntime for UnifiedExecRunt } let command = build_sandbox_command(&command, &req.cwd, &env, req.additional_permissions.clone()) - .map_err(|_| ToolError::Codex(CodexErr::InvalidRequest("missing command line for PTY".into())))?; + .map_err(|_| { + ToolError::Codex(CodexErr::InvalidRequest( + "missing command line for PTY".into(), + )) + })?; let options = unified_exec_options(attempt.network_denial_cancellation_token.clone()); let mut exec_env = attempt .env_for(command, options, managed_network)