fix: allow bailian provider in resolveThinkingProfile

andreacasini · andreacasini · commit f0d0cb18bc22 · 2026-07-19T10:08:19.000Z
The thinking policy resolver only checked for 'vllm' provider, excluding 'bailian' (Alibaba Cloud) even when thinkingFormat was correctly set in compat. This prevented Bailian/Qwen models from using reasoning/thinking features. Adds bailian to isVllmQwenThinkingCompat in both: - src/agents/embedded-agent-runner/model.compat.ts - src/agents/model-selection-shared.ts Fixes #26037
diff --git a/src/agents/embedded-agent-runner/model.compat.ts b/src/agents/embedded-agent-runner/model.compat.ts
@@ -68,7 +68,8 @@ export function resolveMergedConfiguredModelReasoning(params: {
 function isVllmQwenThinkingCompat(params: { provider: string; compat?: unknown }): boolean {
   const thinkingFormat = readCompatThinkingFormat(params.compat);
   return (
-    normalizeProviderId(params.provider) === "vllm" &&
+    (normalizeProviderId(params.provider) === "vllm" ||
+      normalizeProviderId(params.provider) === "bailian") &&
     (thinkingFormat === "qwen" || thinkingFormat === "qwen-chat-template")
   );
 }
diff --git a/src/agents/model-selection-shared.ts b/src/agents/model-selection-shared.ts
@@ -1438,7 +1438,7 @@ function isVllmQwenThinkingCompat(
   compat?: { thinkingFormat?: unknown } | null,
 ): boolean {
   return (
-    providerId === "vllm" &&
+    (providerId === "vllm" || providerId === "bailian") &&
     (compat?.thinkingFormat === "qwen" || compat?.thinkingFormat === "qwen-chat-template")
   );
 }

Original file line number	Diff line number	Diff line change
`@@ -1438,7 +1438,7 @@ function isVllmQwenThinkingCompat(`
`1438`	`1438`	`compat?: { thinkingFormat?: unknown } \| null,`
`1439`	`1439`	`): boolean {`
`1440`	`1440`	`return (`
`1441`		`- providerId === "vllm" &&`
	`1441`	`+ (providerId === "vllm" \|\| providerId === "bailian") &&`
`1442`	`1442`	`(compat?.thinkingFormat === "qwen" \|\| compat?.thinkingFormat === "qwen-chat-template")`
`1443`	`1443`	`);`
`1444`	`1444`	`}`