From ad6d0a2e0ea0bfd1483716c7bc6f1e5975296abb Mon Sep 17 00:00:00 2001 From: Alex Clarke Date: Fri, 17 Jul 2026 17:52:38 -0600 Subject: [PATCH] fix: resolve reasoning effort for the prompt for global defaults as well --- src/config/request_context.rs | 1 + src/config/role.rs | 9 ++++++--- 2 files changed, 7 insertions(+), 3 deletions(-) diff --git a/src/config/request_context.rs b/src/config/request_context.rs index 6409e2d..978ed28 100644 --- a/src/config/request_context.rs +++ b/src/config/request_context.rs @@ -777,6 +777,7 @@ impl RequestContext { let mut role = Role::default(); role.batch_set( &self.model, + app.reasoning_effort.clone(), app.temperature, app.top_p, app.enabled_tools.clone(), diff --git a/src/config/role.rs b/src/config/role.rs index d996ed8..cfb6104 100644 --- a/src/config/role.rs +++ b/src/config/role.rs @@ -255,31 +255,34 @@ impl Role { pub fn sync(&mut self, role_like: &T) { let model = role_like.model(); + let reasoning_effort = role_like.reasoning_effort(); let temperature = role_like.temperature(); let top_p = role_like.top_p(); let enabled_tools = role_like.enabled_tools(); let enabled_mcp_servers = role_like.enabled_mcp_servers(); self.batch_set( model, + reasoning_effort, temperature, top_p, enabled_tools, enabled_mcp_servers, ); - if let Some(v) = role_like.reasoning_effort() { - self.set_reasoning_effort(Some(v)); - } } pub fn batch_set( &mut self, model: &Model, + reasoning_effort: Option, temperature: Option, top_p: Option, enabled_tools: Option>, enabled_mcp_servers: Option>, ) { self.set_model(model.clone()); + if reasoning_effort.is_some() { + self.set_reasoning_effort(reasoning_effort.clone()); + } if temperature.is_some() { self.set_temperature(temperature); }