Visitar URL original
fix: ensure switching anthropic models mid convo on copilot works wit… · Colin4k1024/opencode@56167e4 · GitHub
Skip to content

Commit 56167e4

Browse files
rekram1-nodeColin4k1024
rekram1-node
authored and
Colin4k1024
committed
fix: ensure switching anthropic models mid convo on copilot works without errors, fix issue with reasoning opaque not being picked up for gemini models (anomalyco#11569)
1 parent fb72cc3 commit 56167e4

4 files changed

Lines changed: 83 additions & 6 deletions

File tree

‎packages/opencode/src/provider/sdk/copilot/chat/convert-to-openai-compatible-chat-messages.ts‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,7 @@ export function convertToOpenAICompatibleChatMessages(prompt: LanguageModelV2Pro
100100
break
101101
}
102102
case "reasoning": {
103-
reasoningText = part.text
103+
if (part.text) reasoningText = part.text
104104
break
105105
}
106106
case "tool-call": {
@@ -122,7 +122,7 @@ export function convertToOpenAICompatibleChatMessages(prompt: LanguageModelV2Pro
122122
role: "assistant",
123123
content: text || null,
124124
tool_calls: toolCalls.length > 0 ? toolCalls : undefined,
125-
reasoning_text: reasoningText,
125+
reasoning_text: reasoningOpaque ? reasoningText : undefined,
126126
reasoning_opaque: reasoningOpaque,
127127
...metadata,
128128
})

‎packages/opencode/src/provider/sdk/copilot/chat/openai-compatible-chat-language-model.ts‎

Lines changed: 17 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -219,7 +219,13 @@ export class OpenAICompatibleChatLanguageModel implements LanguageModelV2 {
219219
// text content:
220220
const text = choice.message.content
221221
if (text != null && text.length > 0) {
222-
content.push({ type: "text", text })
222+
content.push({
223+
type: "text",
224+
text,
225+
providerMetadata: choice.message.reasoning_opaque
226+
? { copilot: { reasoningOpaque: choice.message.reasoning_opaque } }
227+
: undefined,
228+
})
223229
}
224230

225231
// reasoning content (Copilot uses reasoning_text):
@@ -243,6 +249,9 @@ export class OpenAICompatibleChatLanguageModel implements LanguageModelV2 {
243249
toolCallId: toolCall.id ?? generateId(),
244250
toolName: toolCall.function.name,
245251
input: toolCall.function.arguments!,
252+
providerMetadata: choice.message.reasoning_opaque
253+
? { copilot: { reasoningOpaque: choice.message.reasoning_opaque } }
254+
: undefined,
246255
})
247256
}
248257
}
@@ -478,7 +487,11 @@ export class OpenAICompatibleChatLanguageModel implements LanguageModelV2 {
478487
}
479488

480489
if (!isActiveText) {
481-
controller.enqueue({ type: "text-start", id: "txt-0" })
490+
controller.enqueue({
491+
type: "text-start",
492+
id: "txt-0",
493+
providerMetadata: reasoningOpaque ? { copilot: { reasoningOpaque } } : undefined,
494+
})
482495
isActiveText = true
483496
}
484497

@@ -559,6 +572,7 @@ export class OpenAICompatibleChatLanguageModel implements LanguageModelV2 {
559572
toolCallId: toolCall.id ?? generateId(),
560573
toolName: toolCall.function.name,
561574
input: toolCall.function.arguments,
575+
providerMetadata: reasoningOpaque ? { copilot: { reasoningOpaque } } : undefined,
562576
})
563577
toolCall.hasFinished = true
564578
}
@@ -601,6 +615,7 @@ export class OpenAICompatibleChatLanguageModel implements LanguageModelV2 {
601615
toolCallId: toolCall.id ?? generateId(),
602616
toolName: toolCall.function.name,
603617
input: toolCall.function.arguments,
618+
providerMetadata: reasoningOpaque ? { copilot: { reasoningOpaque } } : undefined,
604619
})
605620
toolCall.hasFinished = true
606621
}

‎packages/opencode/test/provider/copilot/convert-to-copilot-messages.test.ts‎

Lines changed: 29 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -354,7 +354,7 @@ describe("tool calls", () => {
354354
})
355355

356356
describe("reasoning (copilot-specific)", () => {
357-
test("should include reasoning_text from reasoning part", () => {
357+
test("should omit reasoning_text without reasoning_opaque", () => {
358358
const result = convertToCopilotMessages([
359359
{
360360
role: "assistant",
@@ -370,7 +370,7 @@ describe("reasoning (copilot-specific)", () => {
370370
role: "assistant",
371371
content: "The answer is 42.",
372372
tool_calls: undefined,
373-
reasoning_text: "Let me think about this...",
373+
reasoning_text: undefined,
374374
reasoning_opaque: undefined,
375375
},
376376
])
@@ -404,6 +404,33 @@ describe("reasoning (copilot-specific)", () => {
404404
])
405405
})
406406

407+
test("should include reasoning_opaque from text part providerOptions", () => {
408+
const result = convertToCopilotMessages([
409+
{
410+
role: "assistant",
411+
content: [
412+
{
413+
type: "text",
414+
text: "Done!",
415+
providerOptions: {
416+
copilot: { reasoningOpaque: "opaque-text-456" },
417+
},
418+
},
419+
],
420+
},
421+
])
422+
423+
expect(result).toEqual([
424+
{
425+
role: "assistant",
426+
content: "Done!",
427+
tool_calls: undefined,
428+
reasoning_text: undefined,
429+
reasoning_opaque: "opaque-text-456",
430+
},
431+
])
432+
})
433+
407434
test("should handle reasoning-only assistant message", () => {
408435
const result = convertToCopilotMessages([
409436
{

‎packages/opencode/test/provider/copilot/copilot-chat-model.test.ts‎

Lines changed: 35 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -65,6 +65,12 @@ const FIXTURES = {
6565
`data: {"choices":[{"finish_reason":"tool_calls","index":0,"delta":{"content":null,"role":"assistant","tool_calls":[{"function":{"arguments":"{\\"code\\":\\"1 + 1\\"}","name":"project_eval"},"id":"call_MHw3RDhmT1J5Z3B6WlhpVjlveTc","index":0,"type":"function"}],"reasoning_opaque":"ytGNWFf2doK38peANDvm7whkLPKrd+Fv6/k34zEPBF6Qwitj4bTZT0FBXleydLb6"}}],"created":1766068644,"id":"oBFEaafzD9DVlOoPkY3l4Qs","usage":{"completion_tokens":12,"prompt_tokens":8677,"prompt_tokens_details":{"cached_tokens":3692},"total_tokens":8768,"reasoning_tokens":79},"model":"gemini-3-pro-preview"}`,
6666
`data: [DONE]`,
6767
],
68+
69+
reasoningOpaqueWithToolCallsNoReasoningText: [
70+
`data: {"choices":[{"index":0,"delta":{"content":null,"role":"assistant","tool_calls":[{"function":{"arguments":"{}","name":"read_file"},"id":"call_reasoning_only","index":0,"type":"function"}],"reasoning_opaque":"opaque-xyz"}}],"created":1769917420,"id":"opaque-only","usage":{"completion_tokens":0,"prompt_tokens":0,"prompt_tokens_details":{"cached_tokens":0},"total_tokens":0,"reasoning_tokens":0},"model":"gemini-3-flash-preview"}`,
71+
`data: {"choices":[{"finish_reason":"tool_calls","index":0,"delta":{"content":null,"role":"assistant","tool_calls":[{"function":{"arguments":"{}","name":"read_file"},"id":"call_reasoning_only_2","index":1,"type":"function"}]}}],"created":1769917420,"id":"opaque-only","usage":{"completion_tokens":12,"prompt_tokens":123,"prompt_tokens_details":{"cached_tokens":0},"total_tokens":135,"reasoning_tokens":0},"model":"gemini-3-flash-preview"}`,
72+
`data: [DONE]`,
73+
],
6874
}
6975

7076
function createMockFetch(chunks: string[]) {
@@ -447,6 +453,35 @@ describe("doStream", () => {
447453
})
448454
})
449455

456+
test("should attach reasoning_opaque to tool calls without reasoning_text", async () => {
457+
const mockFetch = createMockFetch(FIXTURES.reasoningOpaqueWithToolCallsNoReasoningText)
458+
const model = createModel(mockFetch)
459+
460+
const { stream } = await model.doStream({
461+
prompt: TEST_PROMPT,
462+
includeRawChunks: false,
463+
})
464+
465+
const parts = await convertReadableStreamToArray(stream)
466+
const reasoningParts = parts.filter(
467+
(p) => p.type === "reasoning-start" || p.type === "reasoning-delta" || p.type === "reasoning-end",
468+
)
469+
470+
expect(reasoningParts).toHaveLength(0)
471+
472+
const toolCall = parts.find((p) => p.type === "tool-call" && p.toolCallId === "call_reasoning_only")
473+
expect(toolCall).toMatchObject({
474+
type: "tool-call",
475+
toolCallId: "call_reasoning_only",
476+
toolName: "read_file",
477+
providerMetadata: {
478+
copilot: {
479+
reasoningOpaque: "opaque-xyz",
480+
},
481+
},
482+
})
483+
})
484+
450485
test("should include response metadata from first chunk", async () => {
451486
const mockFetch = createMockFetch(FIXTURES.basicText)
452487
const model = createModel(mockFetch)

0 commit comments

Comments
 (0)