diff --git a/src/api/providers/__tests__/openai.spec.ts b/src/api/providers/__tests__/openai.spec.ts index a3dcbcc0d5..38b0d9abfc 100644 --- a/src/api/providers/__tests__/openai.spec.ts +++ b/src/api/providers/__tests__/openai.spec.ts @@ -1081,6 +1081,20 @@ describe("OpenAiHandler", () => { expect(model.id).toBe("") expect(model.info).toBeDefined() }) + + it("should set preserveReasoning when openAiR1FormatEnabled is on", () => { + const r1Handler = new OpenAiHandler({ + ...mockOptions, + openAiR1FormatEnabled: true, + }) + const model = r1Handler.getModel() + expect(model.info).toEqual({ ...openAiModelInfoSaneDefaults, preserveReasoning: true }) + }) + + it("should not set preserveReasoning by default", () => { + const model = handler.getModel() + expect(model.info.preserveReasoning).toBeUndefined() + }) }) describe("Azure AI Inference Service", () => { diff --git a/src/api/providers/openai.ts b/src/api/providers/openai.ts index 04b12f233d..02e00bf281 100644 --- a/src/api/providers/openai.ts +++ b/src/api/providers/openai.ts @@ -303,7 +303,15 @@ export class OpenAiHandler extends BaseProvider implements SingleCompletionHandl settings: { ...this.options, reasoningEffort: info.reasoningEffort }, defaultTemperature: 0, }) - return { id, info, ...params } + // Local OpenAI-compatible reasoning models (llama.cpp, LM Studio, Ollama) + // stream reasoning_content, but Roo strips it from the follow-up context + // unless info.preserveReasoning is set. When the user enables the R1 format + // toggle, treat the model as preserving reasoning so the chain is fed back. + return { + id, + info: this.options.openAiR1FormatEnabled ? { ...info, preserveReasoning: true } : info, + ...params, + } } async completePrompt(prompt: string, options?: CompletePromptOptions): Promise {