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

推荐订阅源

P
Proofpoint News Feed
博客园_首页
WordPress大学
WordPress大学
大猫的无限游戏
大猫的无限游戏
有赞技术团队
有赞技术团队
阮一峰的网络日志
阮一峰的网络日志
Hugging Face - Blog
Hugging Face - Blog
博客园 - 【当耐特】
酷 壳 – CoolShell
酷 壳 – CoolShell
Y
Y Combinator Blog
Vercel News
Vercel News
The GitHub Blog
The GitHub Blog
T
The Blog of Author Tim Ferriss
云风的 BLOG
云风的 BLOG
博客园 - 司徒正美
Engineering at Meta
Engineering at Meta
钛媒体:引领未来商业与生活新知
钛媒体:引领未来商业与生活新知
J
Java Code Geeks
Stack Overflow Blog
Stack Overflow Blog
N
Netflix TechBlog - Medium
Martin Fowler
Martin Fowler
宝玉的分享
宝玉的分享
G
Google Developers Blog
Last Week in AI
Last Week in AI

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: harden smart-quoted argument repair (#86611) · openc...
steipete · 2026-05-29 · via Recent Commits to openclaw:main

@@ -76,6 +76,10 @@ const TOOLCALL_REPAIR_FREEFORM_SUCCESSOR_KEYS: Record<string, string> = {

7676

old_string: "new_string",

7777

oldText: "newText",

7878

};

79+

const TOOLCALL_REPAIR_TOOL_VALUE_SUCCESSOR_KEYS = new Map<

80+

string,

81+

ReadonlyMap<string, readonly string[]>

82+

>([["read", new Map([["path", ["offset", "limit"]]])]]);

7983

const TOOLCALL_REPAIR_JSON_STRING_ESCAPES: Record<string, string> = {

8084

'"': '"',

8185

"\\": "\\",

@@ -229,7 +233,41 @@ function readObjectMemberKeyAfterComma(raw: string, commaIndex: number): string

229233

return key.value;

230234

}

231235232-

function shouldCloseSmartQuotedValueAt(raw: string, quoteIndex: number, valueKey: string): boolean {

236+

function normalizeToolCallRepairToolName(value: string): string | undefined {

237+

const trimmed = value.trim();

238+

if (!/^[a-z0-9_-]{1,128}$/i.test(trimmed)) {

239+

return undefined;

240+

}

241+

return trimmed.toLowerCase();

242+

}

243+244+

function extractToolNameFromLeadingPrefix(prefix: string): string | undefined {

245+

const match = /(?:^|[.\s])(?:functions?|tools?)[._:/-]?([a-z0-9_-]+)/i.exec(prefix);

246+

return match?.[1] ? normalizeToolCallRepairToolName(match[1]) : undefined;

247+

}

248+249+

function isToolSpecificValueSuccessor(params: {

250+

toolName?: string;

251+

valueKey: string;

252+

nextKey: string;

253+

}): boolean {

254+

const toolName = params.toolName;

255+

if (!toolName) {

256+

return false;

257+

}

258+

return (

259+

TOOLCALL_REPAIR_TOOL_VALUE_SUCCESSOR_KEYS.get(toolName)

260+

?.get(params.valueKey)

261+

?.includes(params.nextKey) ?? false

262+

);

263+

}

264+265+

function shouldCloseSmartQuotedValueAt(

266+

raw: string,

267+

quoteIndex: number,

268+

valueKey: string,

269+

toolName?: string,

270+

): boolean {

233271

const nextIndex = skipWhitespace(raw, quoteIndex + 1);

234272

const nextChar = raw[nextIndex];

235273

if (nextIndex >= raw.length || nextChar === "}") {

@@ -244,7 +282,10 @@ function shouldCloseSmartQuotedValueAt(raw: string, quoteIndex: number, valueKey

244282

return false;

245283

}

246284

if (!TOOLCALL_REPAIR_FREEFORM_VALUE_KEYS.has(valueKey)) {

247-

return TOOLCALL_REPAIR_KNOWN_ARG_KEYS.has(nextKey);

285+

return (

286+

TOOLCALL_REPAIR_KNOWN_ARG_KEYS.has(nextKey) ||

287+

isToolSpecificValueSuccessor({ toolName, valueKey, nextKey })

288+

);

248289

}

249290

return TOOLCALL_REPAIR_FREEFORM_SUCCESSOR_KEYS[valueKey] === nextKey;

250291

}

@@ -261,11 +302,12 @@ function readSmartQuotedValue(

261302

raw: string,

262303

startIndex: number,

263304

key: string,

305+

toolName?: string,

264306

): ToolCallRepairJsonValue | undefined {

265307

let value = "";

266308

for (let i = startIndex + 1; i < raw.length; i += 1) {

267309

const char = raw[i];

268-

if (isToolCallRepairSmartQuote(char) && shouldCloseSmartQuotedValueAt(raw, i, key)) {

310+

if (isToolCallRepairSmartQuote(char) && shouldCloseSmartQuotedValueAt(raw, i, key, toolName)) {

269311

return { value: decodeSmartQuotedJsonStringEscapes(value), endIndex: i + 1 };

270312

}

271313

value += char;

@@ -366,13 +408,14 @@ function readObjectValue(

366408

raw: string,

367409

startIndex: number,

368410

key: string,

411+

toolName?: string,

369412

): ToolCallRepairJsonValue | undefined {

370413

const char = raw[startIndex];

371414

if (char === '"') {

372415

return readAsciiQuotedString(raw, startIndex);

373416

}

374417

if (isToolCallRepairSmartQuote(char)) {

375-

return readSmartQuotedValue(raw, startIndex, key);

418+

return readSmartQuotedValue(raw, startIndex, key, toolName);

376419

}

377420

if (key === "edits" && char === "[") {

378421

return readSmartQuotedEditArray(raw, startIndex);

@@ -383,6 +426,7 @@ function readObjectValue(

383426

function parseSmartQuotedToolCallObject(

384427

raw: string,

385428

startIndex: number,

429+

toolName?: string,

386430

): ToolCallRepairParsedObject | undefined {

387431

if (raw[startIndex] !== "{") {

388432

return undefined;

@@ -406,7 +450,7 @@ function parseSmartQuotedToolCallObject(

406450

return undefined;

407451

}

408452409-

const value = readObjectValue(raw, skipWhitespace(raw, index + 1), key.value);

453+

const value = readObjectValue(raw, skipWhitespace(raw, index + 1), key.value, toolName);

410454

if (!value) {

411455

return undefined;

412456

}

@@ -460,7 +504,10 @@ function tryExtractUsableToolCallArgumentsFromJson(

460504

};

461505

}

462506463-

function tryExtractSmartQuotedToolCallArguments(raw: string): ToolCallArgumentRepair | undefined {

507+

function tryExtractSmartQuotedToolCallArguments(

508+

raw: string,

509+

toolNameFromContext?: string,

510+

): ToolCallArgumentRepair | undefined {

464511

if (!/[\u201c\u201d\u201e\u201f]/.test(raw)) {

465512

return undefined;

466513

}

@@ -472,7 +519,11 @@ function tryExtractSmartQuotedToolCallArguments(raw: string): ToolCallArgumentRe

472519

if (!isAllowedToolCallRepairLeadingPrefix(leadingPrefix)) {

473520

return undefined;

474521

}

475-

const parsed = parseSmartQuotedToolCallObject(raw, startIndex);

522+

const parsed = parseSmartQuotedToolCallObject(

523+

raw,

524+

startIndex,

525+

toolNameFromContext ?? extractToolNameFromLeadingPrefix(leadingPrefix),

526+

);

476527

if (!parsed) {

477528

return undefined;

478529

}

@@ -491,7 +542,10 @@ function tryExtractSmartQuotedToolCallArguments(raw: string): ToolCallArgumentRe

491542

};

492543

}

493544494-

function tryExtractUsableToolCallArguments(raw: string): ToolCallArgumentRepair | undefined {

545+

function tryExtractUsableToolCallArguments(

546+

raw: string,

547+

toolNameFromContext?: string,

548+

): ToolCallArgumentRepair | undefined {

495549

if (!raw.trim()) {

496550

return undefined;

497551

}

@@ -506,10 +560,30 @@ function tryExtractUsableToolCallArguments(raw: string): ToolCallArgumentRepair

506560

}

507561508562

return (

509-

tryExtractUsableToolCallArgumentsFromJson(raw) ?? tryExtractSmartQuotedToolCallArguments(raw)

563+

tryExtractUsableToolCallArgumentsFromJson(raw) ??

564+

tryExtractSmartQuotedToolCallArguments(raw, toolNameFromContext)

510565

);

511566

}

512567568+

function readToolCallNameInMessage(message: unknown, contentIndex: number): string | undefined {

569+

if (!message || typeof message !== "object") {

570+

return undefined;

571+

}

572+

const content = (message as { content?: unknown }).content;

573+

if (!Array.isArray(content)) {

574+

return undefined;

575+

}

576+

const block = content[contentIndex];

577+

if (!block || typeof block !== "object") {

578+

return undefined;

579+

}

580+

const typedBlock = block as { type?: unknown; name?: unknown };

581+

if (!isToolCallBlockType(typedBlock.type) || typeof typedBlock.name !== "string") {

582+

return undefined;

583+

}

584+

return normalizeToolCallRepairToolName(typedBlock.name);

585+

}

586+513587

function repairToolCallArgumentsInMessage(

514588

message: unknown,

515589

contentIndex: number,

@@ -635,7 +709,10 @@ function wrapStreamRepairMalformedToolCallArguments(

635709

repairedArgsByIndex.has(event.contentIndex);

636710

if (shouldReevaluateRepair) {

637711

const hadRepairState = repairedArgsByIndex.has(event.contentIndex);

638-

const repair = tryExtractUsableToolCallArguments(nextPartialJson);

712+

const toolName =

713+

readToolCallNameInMessage(event.partial, event.contentIndex) ??

714+

readToolCallNameInMessage(event.message, event.contentIndex);

715+

const repair = tryExtractUsableToolCallArguments(nextPartialJson, toolName);

639716

if (repair) {

640717

if (

641718

!hadRepairState &&