From d8e79b65cbcaa608c461e52af6c6920f7b56dcef Mon Sep 17 00:00:00 2001 From: Channing Conger Date: Tue, 10 Mar 2026 18:11:21 -0700 Subject: [PATCH] use the consistent defautl_enabled --- codex-rs/app-server-protocol/src/protocol/v2.rs | 6 +----- codex-rs/protocol/src/dynamic_tools.rs | 4 ++-- 2 files changed, 3 insertions(+), 7 deletions(-) diff --git a/codex-rs/app-server-protocol/src/protocol/v2.rs b/codex-rs/app-server-protocol/src/protocol/v2.rs index cfbb496b09..67e98660c2 100644 --- a/codex-rs/app-server-protocol/src/protocol/v2.rs +++ b/codex-rs/app-server-protocol/src/protocol/v2.rs @@ -509,7 +509,7 @@ pub struct DynamicToolSpec { pub name: String, pub description: String, pub input_schema: JsonValue, - #[serde(default = "default_dynamic_tool_inject_into_context")] + #[serde(default = "default_enabled")] pub inject_into_context: bool, } @@ -548,10 +548,6 @@ pub struct ToolProviderUnregisterParams { #[ts(export_to = "v2/")] pub struct ToolProviderUnregisterResponse {} -fn default_dynamic_tool_inject_into_context() -> bool { - true -} - #[derive(Serialize, Deserialize, Debug, Clone, PartialEq, JsonSchema, TS, ExperimentalApi)] #[serde(rename_all = "snake_case")] #[ts(export_to = "v2/")] diff --git a/codex-rs/protocol/src/dynamic_tools.rs b/codex-rs/protocol/src/dynamic_tools.rs index b6a08a36f7..b974f5c992 100644 --- a/codex-rs/protocol/src/dynamic_tools.rs +++ b/codex-rs/protocol/src/dynamic_tools.rs @@ -10,7 +10,7 @@ pub struct DynamicToolSpec { pub name: String, pub description: String, pub input_schema: JsonValue, - #[serde(default = "default_inject_into_context")] + #[serde(default = "default_enabled")] pub inject_into_context: bool, #[serde(default)] pub provider_owned: bool, @@ -45,6 +45,6 @@ pub enum DynamicToolCallOutputContentItem { InputImage { image_url: String }, } -fn default_inject_into_context() -> bool { +fn default_enabled() -> bool { true }