diff --git a/packages/app/src/components/agent-status-bar.test.ts b/packages/app/src/components/agent-status-bar.test.ts index 3c1d9aafc..4653bf4f8 100644 --- a/packages/app/src/components/agent-status-bar.test.ts +++ b/packages/app/src/components/agent-status-bar.test.ts @@ -112,7 +112,7 @@ describe("resolveAgentModelSelection", () => { expect(selection.activeModelId).toBe("default"); expect(selection.displayModel).toBe("Default (Sonnet 4.6)"); - expect(selection.selectedThinkingId).toBeNull(); - expect(selection.displayThinking).toBe("Default"); + expect(selection.selectedThinkingId).toBe("low"); + expect(selection.displayThinking).toBe("Low"); }); }); diff --git a/packages/app/src/components/agent-status-bar.tsx b/packages/app/src/components/agent-status-bar.tsx index 3e79d1123..f731f3102 100644 --- a/packages/app/src/components/agent-status-bar.tsx +++ b/packages/app/src/components/agent-status-bar.tsx @@ -166,7 +166,11 @@ function ControlledStatusBar({ isModelLoading && (!modelOptions || modelOptions.length === 0) ? "Loading models..." : findOptionLabel(modelOptions, selectedModelId, "Select model"); - const displayThinking = findOptionLabel(thinkingOptions, selectedThinkingOptionId, "Default"); + const displayThinking = findOptionLabel( + thinkingOptions, + selectedThinkingOptionId, + thinkingOptions?.[0]?.label ?? "Unknown", + ); const modeVisuals = selectedModeId ? getModeVisuals(provider, selectedModeId) : undefined; const ModeIconComponent = modeVisuals?.icon ? MODE_ICONS[modeVisuals.icon] : null; diff --git a/packages/app/src/components/agent-status-bar.utils.ts b/packages/app/src/components/agent-status-bar.utils.ts index c68318bc5..f242b00f2 100644 --- a/packages/app/src/components/agent-status-bar.utils.ts +++ b/packages/app/src/components/agent-status-bar.utils.ts @@ -48,15 +48,15 @@ export function resolveAgentModelSelection(input: { selectedModel?.label ?? preferredModelId ?? fallbackModel?.label ?? "Unknown model"; const thinkingOptions = selectedModel?.thinkingOptions ?? null; - const selectedThinkingId = + const resolvedThinkingId = explicitThinkingOptionId && explicitThinkingOptionId !== "default" ? explicitThinkingOptionId : (selectedModel?.defaultThinkingOptionId ?? null); const selectedThinking = - thinkingOptions?.find((option) => option.id === selectedThinkingId) ?? null; - const displayThinking = - selectedThinking?.label ?? - (selectedThinkingId === "default" ? "Model default" : (selectedThinkingId ?? "Default")); + thinkingOptions?.find((option) => option.id === resolvedThinkingId) ?? null; + const effectiveThinking = selectedThinking ?? thinkingOptions?.[0] ?? null; + const selectedThinkingId = effectiveThinking?.id ?? null; + const displayThinking = effectiveThinking?.label ?? selectedThinkingId ?? "Unknown"; return { selectedModel, diff --git a/packages/app/src/hooks/use-agent-form-state.ts b/packages/app/src/hooks/use-agent-form-state.ts index 8d8e70225..01d611712 100644 --- a/packages/app/src/hooks/use-agent-form-state.ts +++ b/packages/app/src/hooks/use-agent-form-state.ts @@ -161,7 +161,7 @@ function resolveThinkingOptionId(args: { return normalizedThinkingOptionId; } - return effectiveModel?.defaultThinkingOptionId ?? ""; + return effectiveModel?.defaultThinkingOptionId ?? thinkingOptions[0]?.id ?? ""; } /**