fix(agents): add openai-responses family to non-visible t...

Recent Commits to openclaw:main

SebTardif · 2026-05-23 · via Recent Commits to openclaw:main

Original file line number	Diff line number	Diff line change
`@@ -1594,6 +1594,54 @@ describe("runEmbeddedPiAgent incomplete-turn safety", () => {`
`1594`	`1594`	`expect(retryInstruction).toBeNull();`
`1595`	`1595`	`});`
`1596`	`1596`
	`1597`	`+it("retries empty openai-codex-responses turns with non-zero output tokens (#85364)", () => {`
	`1598`	`+const retryInstruction = resolveEmptyResponseRetryInstruction({`
	`1599`	`+provider: "openai-codex",`
	`1600`	`+modelId: "gpt-5.5",`
	`1601`	`+modelApi: "openai-codex-responses",`
	`1602`	`+payloadCount: 0,`
	`1603`	`+aborted: false,`
	`1604`	`+timedOut: false,`
	`1605`	`+attempt: makeAttemptResult({`
	`1606`	`+assistantTexts: [],`
	`1607`	`+lastAssistant: {`
	`1608`	`+role: "assistant",`
	`1609`	`+stopReason: "stop",`
	`1610`	`+provider: "openai-codex",`
	`1611`	`+model: "gpt-5.5",`
	`1612`	`+content: [],`
	`1613`	`+usage: { input: 24794, output: 111, cacheRead: 4608, totalTokens: 29513 },`
	`1614`	`+} as unknown as EmbeddedRunAttemptResult["lastAssistant"],`
	`1615`	`+}),`
	`1616`	`+});`
	`1617`	`+`
	`1618`	`+expect(retryInstruction).toBe(EMPTY_RESPONSE_RETRY_INSTRUCTION);`
	`1619`	`+});`
	`1620`	`+`
	`1621`	`+it("retries empty openai-responses turns without visible text", () => {`
	`1622`	`+const retryInstruction = resolveEmptyResponseRetryInstruction({`
	`1623`	`+provider: "openai",`
	`1624`	`+modelId: "gpt-5.5",`
	`1625`	`+modelApi: "openai-responses",`
	`1626`	`+payloadCount: 0,`
	`1627`	`+aborted: false,`
	`1628`	`+timedOut: false,`
	`1629`	`+attempt: makeAttemptResult({`
	`1630`	`+assistantTexts: [],`
	`1631`	`+lastAssistant: {`
	`1632`	`+role: "assistant",`
	`1633`	`+stopReason: "stop",`
	`1634`	`+provider: "openai",`
	`1635`	`+model: "gpt-5.5",`
	`1636`	`+content: [],`
	`1637`	`+usage: { input: 5000, output: 200, totalTokens: 5200 },`
	`1638`	`+} as unknown as EmbeddedRunAttemptResult["lastAssistant"],`
	`1639`	`+}),`
	`1640`	`+});`
	`1641`	`+`
	`1642`	`+expect(retryInstruction).toBe(EMPTY_RESPONSE_RETRY_INSTRUCTION);`
	`1643`	`+});`
	`1644`	`+`
`1597`	`1645`	`it("retries generic empty OpenAI-compatible turns from custom endpoints", () => {`
`1598`	`1646`	`const retryInstruction = resolveEmptyResponseRetryInstruction({`
`1599`	`1647`	`provider: "llama-cpp-local",`

Original file line number	Diff line number	Diff line change
`@@ -133,6 +133,19 @@ const GEMINI_INCOMPLETE_TURN_MODEL_ID_PATTERN = /^gemini(?:[.-]\|$)/;`
`133`	`133`	// Ollama native `/api/chat` can finish with only thinking/internal blocks when
`134`	`134`	`// constrained, but it should not inherit the stricter planning-only/ack prompts.`
`135`	`135`	`const OLLAMA_INCOMPLETE_TURN_PROVIDER_ID_PATTERN = /^ollama(?:-\|$)/;`
	`136`	`+// Model APIs eligible for the non-visible turn retry guard. OpenAI Responses`
	`137`	`+// family can produce reasoning-only turns where usage.output > 0 but no visible`
	`138`	`+// text is emitted; without the guard these pass through as successful. (#85364)`
	`139`	`+const RETRY_GUARD_MODEL_APIS = new Set([`
	`140`	`+"openai-completions",`
	`141`	`+"anthropic-messages",`
	`142`	`+"bedrock-converse-stream",`
	`143`	`+"openai-responses",`
	`144`	`+"openai-codex-responses",`
	`145`	`+"azure-openai-responses",`
	`146`	`+"openclaw-openai-responses-transport",`
	`147`	`+"openclaw-azure-openai-responses-transport",`
	`148`	`+]);`
`136`	`149`	`const DEFAULT_PLANNING_ONLY_RETRY_LIMIT = 1;`
`137`	`150`	`const STRICT_AGENTIC_PLANNING_ONLY_RETRY_LIMIT = 2;`
`138`	`151`	`// Allow one immediate continuation plus one follow-up continuation before`
`@@ -627,11 +640,7 @@ function shouldApplyNonVisibleTurnRetryGuard(params: {`
`627`	`640`	`if (shouldApplyPlanningOnlyRetryGuard(params)) {`
`628`	`641`	`return true;`
`629`	`642`	`}`
`630`		`-if (`
`631`		`-normalizeLowercaseStringOrEmpty(params.modelApi ?? "") === "openai-completions" \|\|`
`632`		`-normalizeLowercaseStringOrEmpty(params.modelApi ?? "") === "anthropic-messages" \|\|`
`633`		`-normalizeLowercaseStringOrEmpty(params.modelApi ?? "") === "bedrock-converse-stream"`
`634`		`-) {`
	`643`	`+if (RETRY_GUARD_MODEL_APIS.has(normalizeLowercaseStringOrEmpty(params.modelApi ?? ""))) {`
`635`	`644`	`return true;`
`636`	`645`	`}`
`637`	`646`	`// Non-visible final turns are narrower than planning-only turns: there is no`

此内容由惯性聚合(RSS阅读器)自动聚合整理，仅供阅读参考。原文来自 — 版权归原作者所有。

推荐订阅源

Recent Commits to openclaw:main