test(gateway): fail strict codex subagent timeouts · open...

Recent Commits to openclaw:main

vincentkoc · 2026-06-01 · via Recent Commits to openclaw:main

Original file line number	Diff line number	Diff line change
`@@ -2,11 +2,28 @@ import { describe, expect, it } from "vitest";`
`2`	`2`	`import {`
`3`	`3`	`EXPECTED_CODEX_MODELS_COMMAND_TEXT,`
`4`	`4`	`EXPECTED_CODEX_STATUS_COMMAND_TEXT,`
	`5`	`+isRetryableCodexHarnessLiveError,`
`5`	`6`	`isExpectedCodexModelsCommandText,`
`6`	`7`	`isExpectedCodexStatusCommandText,`
	`8`	`+shouldSkipRetryableCodexHarnessLiveError,`
`7`	`9`	`} from "./gateway-codex-harness.live-helpers.js";`
`8`	`10`
`9`	`11`	`describe("gateway codex harness live helpers", () => {`
	`12`	`+it("does not skip sessions.list timeouts when the Codex subagent probe is enabled", () => {`
	`13`	`+const error = new Error("gateway request timeout for sessions.list");`
	`14`	`+`
	`15`	`+expect(isRetryableCodexHarnessLiveError(error)).toBe(true);`
	`16`	`+expect(shouldSkipRetryableCodexHarnessLiveError(error, { subagentProbe: false })).toBe(true);`
	`17`	`+expect(shouldSkipRetryableCodexHarnessLiveError(error, { subagentProbe: true })).toBe(false);`
	`18`	`+});`
	`19`	`+`
	`20`	`+it("does not classify unrelated live Codex errors as retryable gateway timeouts", () => {`
	`21`	`+const error = new Error("subagent child did not emit lifecycle event");`
	`22`	`+`
	`23`	`+expect(isRetryableCodexHarnessLiveError(error)).toBe(false);`
	`24`	`+expect(shouldSkipRetryableCodexHarnessLiveError(error, { subagentProbe: false })).toBe(false);`
	`25`	`+});`
	`26`	`+`
`10`	`27`	`it("accepts the current codex status prose from the live harness", () => {`
`11`	`28`	`const text =`
`12`	`29`	"OpenClaw is running on `openai/gpt-5.5` with low reasoning/text settings. Context is at `22k/272k` tokens, no compactions, and the current session is `agent:dev:live-codex-harness`.";

Original file line number	Diff line number	Diff line change
`@@ -272,3 +272,17 @@ export function isExpectedCodexModelsCommandText(text: string): boolean {`
`272`	`272`	`isInteractiveTuiSummary`
`273`	`273`	`);`
`274`	`274`	`}`
	`275`	`+`
	`276`	`+export function isRetryableCodexHarnessLiveError(error: unknown): boolean {`
	`277`	`+if (!(error instanceof Error)) {`
	`278`	`+return false;`
	`279`	`+}`
	`280`	`+return error.message.includes("gateway request timeout for sessions.list");`
	`281`	`+}`
	`282`	`+`
	`283`	`+export function shouldSkipRetryableCodexHarnessLiveError(`
	`284`	`+error: unknown,`
	`285`	`+params: { subagentProbe: boolean },`
	`286`	`+): boolean {`
	`287`	`+return isRetryableCodexHarnessLiveError(error) && !params.subagentProbe;`
	`288`	`+}`

Original file line number	Diff line number	Diff line change
`@@ -26,6 +26,7 @@ import {`
`26`	`26`	`EXPECTED_CODEX_STATUS_COMMAND_TEXT,`
`27`	`27`	`isExpectedCodexModelsCommandText,`
`28`	`28`	`isExpectedCodexStatusCommandText,`
	`29`	`+shouldSkipRetryableCodexHarnessLiveError,`
`29`	`30`	`} from "./gateway-codex-harness.live-helpers.js";`
`30`	`31`	`import {`
`31`	`32`	`assertCronJobMatches,`
`@@ -105,13 +106,6 @@ function isCodexAccountTokenError(error: unknown): boolean {`
`105`	`106`	`return error instanceof Error && error.message.includes("Failed to extract accountId from token");`
`106`	`107`	`}`
`107`	`108`
`108`		`-function isRetryableCodexHarnessLiveError(error: unknown): boolean {`
`109`		`-if (!(error instanceof Error)) {`
`110`		`-return false;`
`111`		`-}`
`112`		`-return error.message.includes("gateway request timeout for sessions.list");`
`113`		`-}`
`114`		`-`
`115`	`109`	`function asRecord(value: unknown): Record<string, unknown> \| undefined {`
`116`	`110`	`return value && typeof value === "object" ? (value as Record<string, unknown>) : undefined;`
`117`	`111`	`}`
`@@ -1189,7 +1183,11 @@ describeLive("gateway live (Codex harness)", () => {`
`1189`	`1183`	`console.error(`
`1190`	`1184`	`"SKIP: Codex auth cannot extract accountId from the available token; skipping live Codex harness assertions.",`
`1191`	`1185`	`);`
`1192`		`-} else if (isRetryableCodexHarnessLiveError(error)) {`
	`1186`	`+} else if (`
	`1187`	`+shouldSkipRetryableCodexHarnessLiveError(error, {`
	`1188`	`+subagentProbe: CODEX_HARNESS_SUBAGENT_PROBE,`
	`1189`	`+})`
	`1190`	`+) {`
`1193`	`1191`	`console.error(`
`1194`	`1192`	`SKIP: Codex harness live backend hit a retryable gateway timeout; skipping live Codex harness assertions. ${error instanceof Error ? error.message : String(error)}`,
`1195`	`1193`	`);`

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

推荐订阅源

Recent Commits to openclaw:main