惯性聚合 高效追踪和阅读你感兴趣的博客、新闻、科技资讯
阅读原文 在惯性聚合中打开

推荐订阅源

让小产品的独立变现更简单 - ezindie.com
让小产品的独立变现更简单 - ezindie.com
罗磊的独立博客
博客园 - 【当耐特】
M
MIT News - Artificial intelligence
月光博客
月光博客
博客园_首页
博客园 - 叶小钗
T
Tailwind CSS Blog
H
Hackread – Cybersecurity News, Data Breaches, AI and More
I
InfoQ
量子位
小众软件
小众软件
爱范儿
爱范儿
The GitHub Blog
The GitHub Blog
IT之家
IT之家
Jina AI
Jina AI
阮一峰的网络日志
阮一峰的网络日志
G
Google Developers Blog
WordPress大学
WordPress大学
人人都是产品经理
人人都是产品经理
钛媒体:引领未来商业与生活新知
钛媒体:引领未来商业与生活新知
J
Java Code Geeks
云风的 BLOG
云风的 BLOG
奇客Solidot–传递最新科技情报
奇客Solidot–传递最新科技情报

Recent Commits to openclaw:main

test: merge chat side-result checks · openclaw/openclaw@ddd2c2a test: merge cron history checks · openclaw/openclaw@f7eb746 test: merge responsive navigation shell checks · openclaw/openclaw@c2e4b47 docs(changelog): add codex oauth fixes · openclaw/openclaw@628e6cd test: merge navigation routing cases · openclaw/openclaw@5d8cecb Tests: mock channel registry bundled fallback · openclaw/openclaw@2b08233 Secrets: avoid broad web search discovery for single plugin config · openclaw/openclaw@a464f59 test: merge config view browser checks · openclaw/openclaw@20cf511 fix(status): align oauth health with runtime · openclaw/openclaw@eed7116 feat: add macOS screen snapshots for monitor preview (#67954) thanks … · openclaw/openclaw@f377db1 fix: report shared auth scopes in hello-ok (#67810) thanks @BunsDev · openclaw/openclaw@0b6c39b Auto-reply: avoid eager bundled route fallback · openclaw/openclaw@3ea1bf4 Tests: narrow session binding contract setup · openclaw/openclaw@54e4e16 fix(macOS): enable undo/redo in webchat composer text input (#34962) · openclaw/openclaw@00951dc Tests: speed up channel setup promotion · openclaw/openclaw@82b529a Docs: refresh agent instructions · openclaw/openclaw@5775fe2 fix(auth): serialize OAuth refresh across agents to fix #26322 (#67876) · openclaw/openclaw@8e79080 test: allow ollama public surface boundary test · openclaw/openclaw@7d4f1a6 Docs: add test performance guardrails · openclaw/openclaw@89706d3 Tests: restore context-engine usage proof · openclaw/openclaw@e4c4f95 Tests: slim context engine runtime coverage · openclaw/openclaw@74c198f ci: retry failed custom checkouts · openclaw/openclaw@0ee5baf test: trim duplicate provider auth onboarding cases · openclaw/openclaw@1ffc02e matrix: fix sessions_spawn --thread subagent session spawning (#67643) · openclaw/openclaw@1ce2596 test: reduce auth choice fixture churn · openclaw/openclaw@857b9cd test: mock health status config boundaries · openclaw/openclaw@9d5ab4a test: mock onboard config io boundary · openclaw/openclaw@299694d test: mock legacy state plugin boundaries · openclaw/openclaw@2713089 test: mock channel install boundaries · openclaw/openclaw@b945248 test: mock doctor preview channel boundaries · openclaw/openclaw@b1a3ad4
fix: preserve valid completions reasoning replay · opencl...
steipete · 2026-05-15 · via Recent Commits to openclaw:main

@@ -5414,17 +5414,8 @@ describe("openai transport stream", () => {

54145414

});

54155415

});

541654165417-

describe("buildOpenAICompletionsParams strips response-only reasoning fields", () => {

5418-

// OpenRouter and other OpenAI-Completions providers echo `reasoning_details`

5419-

// (array) and `reasoning_content` (string) in response choices to expose the

5420-

// model's chain of thought. If those fields leak back into a follow-up

5421-

// request, OpenRouter rejects the call with HTTP 500 ("Internal Server

5422-

// Error"). buildOpenAICompletionsParams must scrub them before the wire.

5423-

// Repro recipe (verified against live OpenRouter):

5424-

// POST /chat/completions with messages[*].reasoning_details: "..." => 500

5425-

// Same body without that key => 200

5426-5427-

const baseModel = {

5417+

describe("buildOpenAICompletionsParams sanitizes reasoning replay fields", () => {

5418+

const openRouterModel = {

54285419

id: "deepseek/deepseek-v4-flash",

54295420

name: "DeepSeek v4 Flash",

54305421

api: "openai-completions",

@@ -5437,6 +5428,19 @@ describe("buildOpenAICompletionsParams strips response-only reasoning fields", (

54375428

maxTokens: 8192,

54385429

} satisfies Model<"openai-completions">;

543954305431+

const openAIModel = {

5432+

id: "gpt-5.4-mini",

5433+

name: "GPT-5.4 Mini",

5434+

api: "openai-completions",

5435+

provider: "openai",

5436+

baseUrl: "https://api.openai.com/v1",

5437+

reasoning: true,

5438+

input: ["text"],

5439+

cost: { input: 0, output: 0, cacheRead: 0, cacheWrite: 0 },

5440+

contextWindow: 200_000,

5441+

maxTokens: 8192,

5442+

} satisfies Model<"openai-completions">;

5443+54405444

function getAssistantMessage(params: { messages: unknown }) {

54415445

expect(Array.isArray(params.messages)).toBe(true);

54425446

const list = params.messages as Array<Record<string, unknown>>;

@@ -5445,13 +5449,80 @@ describe("buildOpenAICompletionsParams strips response-only reasoning fields", (

54455449

return assistant as Record<string, unknown>;

54465450

}

544754515448-

it("removes reasoning_details, reasoning_content, and reasoning from assistant replay", () => {

5452+

function buildReplayParams(model: Model<"openai-completions">, thinkingSignature: string) {

5453+

return buildOpenAICompletionsParams(

5454+

model,

5455+

{

5456+

systemPrompt: "system",

5457+

messages: [

5458+

{ role: "user", content: "hello" },

5459+

{

5460+

role: "assistant",

5461+

provider: model.provider,

5462+

api: model.api,

5463+

model: model.id,

5464+

stopReason: "stop",

5465+

timestamp: 0,

5466+

content: [

5467+

{

5468+

type: "thinking",

5469+

thinking: "Need to answer politely.",

5470+

thinkingSignature,

5471+

},

5472+

{ type: "text", text: "Hello!" },

5473+

],

5474+

},

5475+

{ role: "user", content: "again" },

5476+

],

5477+

tools: [],

5478+

} as never,

5479+

undefined,

5480+

) as { messages: unknown };

5481+

}

5482+5483+

it.each(["reasoning_details", "reasoning_content", "reasoning", "reasoning_text"])(

5484+

"strips %s from stock OpenAI Chat Completions assistant replay",

5485+

(thinkingSignature) => {

5486+

const assistant = getAssistantMessage(buildReplayParams(openAIModel, thinkingSignature));

5487+5488+

expect(assistant).not.toHaveProperty("reasoning_details");

5489+

expect(assistant).not.toHaveProperty("reasoning_content");

5490+

expect(assistant).not.toHaveProperty("reasoning");

5491+

expect(assistant).not.toHaveProperty("reasoning_text");

5492+

},

5493+

);

5494+5495+

it("normalizes OpenRouter string reasoning_details to reasoning", () => {

5496+

const assistant = getAssistantMessage(buildReplayParams(openRouterModel, "reasoning_details"));

5497+5498+

expect(assistant).not.toHaveProperty("reasoning_details");

5499+

expect(assistant.reasoning).toBe("Need to answer politely.");

5500+

});

5501+5502+

it.each(["reasoning", "reasoning_content"])(

5503+

"preserves OpenRouter %s string reasoning replay",

5504+

(thinkingSignature) => {

5505+

const assistant = getAssistantMessage(buildReplayParams(openRouterModel, thinkingSignature));

5506+5507+

expect(assistant[thinkingSignature]).toBe("Need to answer politely.");

5508+

},

5509+

);

5510+5511+

it("normalizes OpenRouter reasoning_text to reasoning", () => {

5512+

const assistant = getAssistantMessage(buildReplayParams(openRouterModel, "reasoning_text"));

5513+5514+

expect(assistant).not.toHaveProperty("reasoning_text");

5515+

expect(assistant.reasoning).toBe("Need to answer politely.");

5516+

});

5517+5518+

it("preserves OpenRouter array reasoning_details from tool-call signatures", () => {

5519+

const reasoningDetail = { type: "reasoning.encrypted", id: "rs_1", data: "ciphertext" };

54495520

const params = buildOpenAICompletionsParams(

5450-

baseModel,

5521+

openRouterModel,

54515522

{

54525523

systemPrompt: "system",

54535524

messages: [

5454-

{ role: "user", content: "你好" },

5525+

{ role: "user", content: "lookup" },

54555526

{

54565527

role: "assistant",

54575528

provider: "openrouter",

@@ -5461,23 +5532,30 @@ describe("buildOpenAICompletionsParams strips response-only reasoning fields", (

54615532

timestamp: 0,

54625533

content: [

54635534

{

5464-

type: "thinking",

5465-

thinking: "User said hi, I should respond politely.",

5466-

thinkingSignature: "considering",

5535+

type: "toolCall",

5536+

id: "call_1",

5537+

name: "lookup",

5538+

arguments: { query: "weather" },

5539+

thoughtSignature: JSON.stringify(reasoningDetail),

54675540

},

5468-

{ type: "text", text: "Hello!" },

54695541

],

54705542

},

5471-

{ role: "user", content: "再来一个" },

5543+

{

5544+

role: "toolResult",

5545+

toolCallId: "call_1",

5546+

toolName: "lookup",

5547+

content: [{ type: "text", text: "sunny" }],

5548+

isError: false,

5549+

timestamp: 1,

5550+

},

5551+

{ role: "user", content: "answer" },

54725552

],

54735553

tools: [],

54745554

} as never,

54755555

undefined,

54765556

) as { messages: unknown };

5477555754785558

const assistant = getAssistantMessage(params);

5479-

expect(assistant).not.toHaveProperty("reasoning_details");

5480-

expect(assistant).not.toHaveProperty("reasoning_content");

5481-

expect(assistant).not.toHaveProperty("reasoning");

5559+

expect(assistant.reasoning_details).toEqual([reasoningDetail]);

54825560

});

54835561

});