lmin/zmo-cli
1
0
Fork 0
Code Issues Pull Requests Actions 6 Packages Projects Releases Wiki Activity

Revert "fix(opencode): preserve signed thinking during anthropic reorder" (#30502)

Browse Source
This commit is contained in:
Aiden Cline
2026-06-02 23:34:05 -05:00
committed by GitHub
parent 42173bca4b
commit a763a14d44
2 changed files with 0 additions and 59 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
packages/opencode/src/provider/transform.ts
View File

@@ -230,18 +230,6 @@ function normalizeMessages(
const parts = msg.content const parts = msg.content
const first = parts.findIndex((part) => part.type === "tool-call") const first = parts.findIndex((part) => part.type === "tool-call")
if (first === -1) return [msg] if (first === -1) return [msg]
// Anthropic signs thinking blocks, so moving them during replay invalidates the request.
if (
parts
.slice(first)
.some(
(part) =>
part.type === "reasoning" &&
(part.providerOptions?.anthropic?.signature != null ||
part.providerOptions?.anthropic?.redactedData != null),
)
)
return [msg]
if (!parts.slice(first).some((part) => part.type !== "tool-call")) return [msg] if (!parts.slice(first).some((part) => part.type !== "tool-call")) return [msg]
return [ return [
{ ...msg, content: parts.filter((part) => part.type !== "tool-call") }, { ...msg, content: parts.filter((part) => part.type !== "tool-call") },

47
packages/opencode/test/provider/transform.test.ts
View File

@@ -1680,25 +1680,6 @@ describe("ProviderTransform.message - anthropic empty content filtering", () =>
]) ])
}) })
test("leaves signed anthropic reasoning after tool calls unchanged", () => {
const msgs = [
{
role: "assistant",
content: [
{ type: "reasoning", text: "First thought", providerOptions: { anthropic: { signature: "sig-1" } } },
{ type: "tool-call", toolCallId: "toolu_1", toolName: "read", input: { filePath: "/root" } },
{ type: "reasoning", text: "Second thought", providerOptions: { anthropic: { signature: "sig-2" } } },
{ type: "tool-call", toolCallId: "toolu_2", toolName: "glob", input: { pattern: "**/*.pdf" } },
],
},
] as any[]
const result = ProviderTransform.message(msgs, anthropicModel, {}) as any[]
expect(result).toHaveLength(1)
expect(result[0].content).toMatchObject(msgs[0].content)
})
test("splits vertex anthropic assistant messages when text trails tool calls", () => { test("splits vertex anthropic assistant messages when text trails tool calls", () => {
const model = { const model = {
...anthropicModel, ...anthropicModel,
@@ -1736,34 +1717,6 @@ describe("ProviderTransform.message - anthropic empty content filtering", () =>
], ],
}) })
}) })
test("leaves redacted vertex anthropic reasoning after tool calls unchanged", () => {
const model = {
...anthropicModel,
providerID: "google-vertex-anthropic",
api: {
id: "claude-sonnet-4@20250514",
url: "https://us-central1-aiplatform.googleapis.com",
npm: "@ai-sdk/google-vertex/anthropic",
},
}
const msgs = [
{
role: "assistant",
content: [
{ type: "tool-call", toolCallId: "toolu_1", toolName: "read", input: { filePath: "/root" } },
{ type: "reasoning", text: "", providerOptions: { anthropic: { redactedData: "redacted-1" } } },
{ type: "tool-call", toolCallId: "toolu_2", toolName: "glob", input: { pattern: "**/*.pdf" } },
],
},
] as any[]
const result = ProviderTransform.message(msgs, model, {}) as any[]
expect(result).toHaveLength(1)
expect(result[0].content).toMatchObject(msgs[0].content)
})
}) })
describe("ProviderTransform.message - strip openai metadata when store=false", () => { describe("ProviderTransform.message - strip openai metadata when store=false", () => {