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

推荐订阅源

云风的 BLOG
云风的 BLOG
M
MIT News - Artificial intelligence
Recent Announcements
Recent Announcements
Cyber Security Advisories - MS-ISAC
Cyber Security Advisories - MS-ISAC
Stack Overflow Blog
Stack Overflow Blog
J
Java Code Geeks
Microsoft Azure Blog
Microsoft Azure Blog
罗磊的独立博客
博客园 - 【当耐特】
H
Help Net Security
腾讯CDC
大猫的无限游戏
大猫的无限游戏
GbyAI
GbyAI
Last Week in AI
Last Week in AI
Jina AI
Jina AI
博客园 - 聂微东
Blog — PlanetScale
Blog — PlanetScale
A
About on SuperTechFans
Apple Machine Learning Research
Apple Machine Learning Research
P
Proofpoint News Feed
Y
Y Combinator Blog
C
Check Point Blog
博客园 - 司徒正美
钛媒体:引领未来商业与生活新知
钛媒体:引领未来商业与生活新知

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(providers): harden search tool response schemas · ope...
vincentkoc · 2026-05-16 · via Recent Commits to openclaw:main

@@ -1,6 +1,6 @@

11

import {

22

createProviderHttpError,

3-

readProviderJsonResponse,

3+

readProviderJsonObjectResponse,

44

} from "openclaw/plugin-sdk/provider-http";

55

import type { OpenClawConfig } from "openclaw/plugin-sdk/provider-onboard";

66

import {

@@ -84,6 +84,14 @@ type KimiSearchResult = {

8484

grounded: boolean;

8585

};

868687+

function isRecord(value: unknown): value is Record<string, unknown> {

88+

return typeof value === "object" && value !== null && !Array.isArray(value);

89+

}

90+91+

function throwMalformedKimiResponse(): never {

92+

throw new Error("Kimi API error: malformed JSON response");

93+

}

94+8795

function resolveKimiConfig(searchConfig?: SearchConfigRecord): KimiConfig {

8896

const kimi = searchConfig?.kimi;

8997

return kimi && typeof kimi === "object" && !Array.isArray(kimi) ? (kimi as KimiConfig) : {};

@@ -132,13 +140,30 @@ function extractKimiMessageText(message: KimiMessage | undefined): string | unde

132140

}

133141134142

function extractKimiCitations(data: KimiSearchResponse): string[] {

135-

const citations = (data.search_results ?? [])

136-

.map((entry) => entry.url?.trim())

143+

const searchResults = data.search_results ?? [];

144+

if (!Array.isArray(searchResults)) {

145+

throwMalformedKimiResponse();

146+

}

147+

const citations = searchResults

148+

.map((entry) => (isRecord(entry) && typeof entry.url === "string" ? entry.url.trim() : ""))

137149

.filter((url): url is string => Boolean(url));

138150139-

for (const toolCall of data.choices?.[0]?.message?.tool_calls ?? []) {

140-

const rawArguments = toolCall.function?.arguments;

141-

if (!rawArguments) {

151+

const choices = data.choices ?? [];

152+

if (!Array.isArray(choices)) {

153+

throwMalformedKimiResponse();

154+

}

155+

const firstChoice = choices[0];

156+

const message = firstChoice && isRecord(firstChoice.message) ? firstChoice.message : undefined;

157+

const toolCalls = message?.tool_calls ?? [];

158+

if (!Array.isArray(toolCalls)) {

159+

throwMalformedKimiResponse();

160+

}

161+

for (const toolCall of toolCalls) {

162+

if (!isRecord(toolCall) || !isRecord(toolCall.function)) {

163+

continue;

164+

}

165+

const rawArguments = toolCall.function.arguments;

166+

if (typeof rawArguments !== "string" || !rawArguments) {

142167

continue;

143168

}

144169

try {

@@ -165,11 +190,16 @@ function extractKimiCitations(data: KimiSearchResponse): string[] {

165190

}

166191167192

function hasKimiSearchResults(data: KimiSearchResponse): boolean {

168-

return (data.search_results ?? []).some(

193+

const searchResults = data.search_results ?? [];

194+

if (!Array.isArray(searchResults)) {

195+

throwMalformedKimiResponse();

196+

}

197+

return searchResults.some(

169198

(entry) =>

170-

Boolean(normalizeOptionalString(entry.url)) ||

171-

Boolean(normalizeOptionalString(entry.title)) ||

172-

Boolean(normalizeOptionalString(entry.content)),

199+

isRecord(entry) &&

200+

(Boolean(normalizeOptionalString(entry.url)) ||

201+

Boolean(normalizeOptionalString(entry.title)) ||

202+

Boolean(normalizeOptionalString(entry.content))),

173203

);

174204

}

175205

@@ -219,7 +249,13 @@ async function runKimiSearch(params: {

219249

throw await createProviderHttpError(res, "Kimi API error");

220250

}

221251222-

const data = await readProviderJsonResponse<KimiSearchResponse>(res, "Kimi API error");

252+

const data = (await readProviderJsonObjectResponse(

253+

res,

254+

"Kimi API error",

255+

)) as KimiSearchResponse;

256+

if (!Array.isArray(data.choices)) {

257+

throwMalformedKimiResponse();

258+

}

223259

if (hasKimiSearchResults(data)) {

224260

hasGroundingEvidence = true;

225261

}

@@ -230,14 +266,23 @@ async function runKimiSearch(params: {

230266

hasGroundingEvidence = true;

231267

}

232268

const choice = data.choices?.[0];

269+

if (!isRecord(choice) || !isRecord(choice.message)) {

270+

throwMalformedKimiResponse();

271+

}

233272

const message = choice?.message;

234273

const text = extractKimiMessageText(message);

235274

const toolCalls = message?.tool_calls ?? [];

275+

if (!Array.isArray(toolCalls)) {

276+

throwMalformedKimiResponse();

277+

}

236278237279

if (choice?.finish_reason !== "tool_calls" || toolCalls.length === 0) {

280+

if (!text) {

281+

throwMalformedKimiResponse();

282+

}

238283

return {

239284

done: true,

240-

content: text ?? "No response",

285+

content: text,

241286

citations: [...collectedCitations],

242287

};

243288

}

@@ -269,9 +314,12 @@ async function runKimiSearch(params: {

269314

});

270315

}

271316

if (!pushed) {

317+

if (!text) {

318+

throwMalformedKimiResponse();

319+

}

272320

return {

273321

done: true,

274-

content: text ?? "No response",

322+

content: text,

275323

citations: [...collectedCitations],

276324

};

277325

}