Skip to content

Commit b0476a0

Browse files
authored
[router][quick fix] Add minimal option for reasoning effort in spec (#12711)
1 parent ffba61a commit b0476a0

File tree

2 files changed

+4
-0
lines changed

2 files changed

+4
-0
lines changed

sgl-router/src/protocols/responses.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -85,6 +85,7 @@ fn default_reasoning_effort() -> Option<ReasoningEffort> {
8585
#[derive(Debug, Clone, Deserialize, Serialize)]
8686
#[serde(rename_all = "snake_case")]
8787
pub enum ReasoningEffort {
88+
Minimal,
8889
Low,
8990
Medium,
9091
High,

sgl-router/src/routers/grpc/harmony/builder.rs

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -277,6 +277,8 @@ impl HarmonyBuilder {
277277
"high" => ReasoningEffort::High,
278278
"medium" => ReasoningEffort::Medium,
279279
"low" => ReasoningEffort::Low,
280+
// Harmony does not support minimal reasoning effort
281+
"minimal" => ReasoningEffort::Low,
280282
_ => ReasoningEffort::Medium,
281283
});
282284

@@ -302,6 +304,7 @@ impl HarmonyBuilder {
302304
ResponsesReasoningEffort::High => ReasoningEffort::High,
303305
ResponsesReasoningEffort::Medium => ReasoningEffort::Medium,
304306
ResponsesReasoningEffort::Low => ReasoningEffort::Low,
307+
ResponsesReasoningEffort::Minimal => ReasoningEffort::Low,
305308
});
306309

307310
self.build_system_message(reasoning_effort, with_custom_tools)

0 commit comments

Comments
 (0)