From 048438a44a9e3c1b90ff6594c6dfc1e47cd81366 Mon Sep 17 00:00:00 2001 From: starr-openai Date: Thu, 19 Mar 2026 15:03:44 -0700 Subject: [PATCH] Rename environment process accessor to get_executor Drop the stale remote_exec_server_client() accessor now that Environment exposes the process capability directly, and rename the capability getter to get_executor() on the environment trait and concrete type. Co-authored-by: Codex --- codex-rs/exec-server/src/client_api.rs | 2 +- codex-rs/exec-server/src/environment.rs | 15 +++------------ 2 files changed, 4 insertions(+), 13 deletions(-) diff --git a/codex-rs/exec-server/src/client_api.rs b/codex-rs/exec-server/src/client_api.rs index 852ade566a..790f0bb66c 100644 --- a/codex-rs/exec-server/src/client_api.rs +++ b/codex-rs/exec-server/src/client_api.rs @@ -56,5 +56,5 @@ pub trait ExecProcess: Send + Sync { /// Capability bundle exposed by an execution environment. pub trait ExecutorEnvironment: Send + Sync { - fn process(&self) -> &(dyn ExecProcess + '_); + fn get_executor(&self) -> &(dyn ExecProcess + '_); } diff --git a/codex-rs/exec-server/src/environment.rs b/codex-rs/exec-server/src/environment.rs index 6821700213..f55cb99097 100644 --- a/codex-rs/exec-server/src/environment.rs +++ b/codex-rs/exec-server/src/environment.rs @@ -53,15 +53,7 @@ impl Environment { self.experimental_exec_server_url.as_deref() } - pub fn remote_exec_server_client(&self) -> Option<&ExecServerClient> { - if self.experimental_exec_server_url.is_some() { - Some(&self.exec_server_client) - } else { - None - } - } - - pub fn process(&self) -> &(dyn ExecProcess + '_) { + pub fn get_executor(&self) -> &(dyn ExecProcess + '_) { &self.exec_server_client } @@ -75,8 +67,8 @@ impl Environment { } impl ExecutorEnvironment for Environment { - fn process(&self) -> &(dyn ExecProcess + '_) { - self.process() + fn get_executor(&self) -> &(dyn ExecProcess + '_) { + self.get_executor() } } @@ -90,6 +82,5 @@ mod tests { let environment = Environment::create(None).await.expect("create environment"); assert_eq!(environment.experimental_exec_server_url(), None); - assert!(environment.remote_exec_server_client().is_none()); } }