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

推荐订阅源

罗磊的独立博客
G
Google Developers Blog
钛媒体:引领未来商业与生活新知
钛媒体:引领未来商业与生活新知
腾讯CDC
有赞技术团队
有赞技术团队
Vercel News
Vercel News
MongoDB | Blog
MongoDB | Blog
M
MIT News - Artificial intelligence
OSCHINA 社区最新新闻
OSCHINA 社区最新新闻
B
Blog RSS Feed
I
InfoQ
Blog — PlanetScale
Blog — PlanetScale
博客园_首页
The Cloudflare Blog
B
Blog
C
Check Point Blog
Stack Overflow Blog
Stack Overflow Blog
IT之家
IT之家
U
Unit 42
D
Docker
月光博客
月光博客
aimingoo的专栏
aimingoo的专栏
博客园 - Franky
A
About on SuperTechFans

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
refactor(openai): confine legacy codex repair to doctor ·...
steipete · 2026-05-31 · via Recent Commits to openclaw:main
Original file line numberDiff line numberDiff line change

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

11

{

22

"id": "openai",

3-

"legacyPluginIds": ["openai-codex"],

43

"activation": {

54

"onStartup": false

65

},

Original file line numberDiff line numberDiff line change

@@ -106,7 +106,7 @@ describe("OpenAI plugin manifest", () => {

106106

});

107107
108108

it("routes setup through the OpenAI setup runtime", () => {

109-

expect(manifest.legacyPluginIds).toEqual(["openai-codex"]);

109+

expect(manifest.legacyPluginIds).toBeUndefined();

110110

expect(manifest.setup?.providers?.map((provider) => provider.id)).toEqual(["openai"]);

111111

expect(manifest.providerAuthAliases).toBeUndefined();

112112

});

Original file line numberDiff line numberDiff line change

@@ -143,6 +143,7 @@ const shellInlineCommandInterpreters = new Set(["bash", "dash", "ksh", "sh", "zs

143143

const remoteChangedGateEnv = [

144144

"OPENCLAW_CHECK_CHANGED_REMOTE_CHILD=1",

145145

"OPENCLAW_CHANGED_LANES_RAW_SYNC=1",

146+

"CI=1",

146147

];

147148

const shellInlineCommandOptionsWithNextValue = new Set([

148149

"+O",

Original file line numberDiff line numberDiff line change

@@ -167,9 +167,6 @@ function resolveDescription({ manifest, packageJson }) {

167167
168168

const providers = Array.isArray(manifest.providers) ? manifest.providers : [];

169169

if (providers.length > 0) {

170-

if (manifest.providerAuthAliases?.["openai-codex"] === "openai") {

171-

return `Adds ${displayList(providers)} model provider support to OpenClaw, including ChatGPT/Codex OAuth.`;

172-

}

173170

return `Adds ${displayList(providers)} model provider support to OpenClaw.`;

174171

}

175172
Original file line numberDiff line numberDiff line change

@@ -15,8 +15,5 @@ describe("oauth refresh failure hints", () => {

1515

expect(buildOAuthRefreshFailureLoginCommand("openai")).toBe(

1616

"openclaw models auth login --provider openai",

1717

);

18-

expect(buildOAuthRefreshFailureLoginCommand("OpenAI-Codex")).toBe(

19-

"openclaw models auth login --provider openai",

20-

);

2118

});

2219

});

Original file line numberDiff line numberDiff line change

@@ -11,9 +11,6 @@ export type OAuthRefreshFailureReason =

1111
1212

const OAUTH_REFRESH_FAILURE_PROVIDER_RE = /OAuth token refresh failed for ([^:]+):/i;

1313

const SAFE_PROVIDER_ID_RE = /^[a-z0-9][a-z0-9._-]*$/;

14-

const RETIRED_REAUTH_PROVIDER_IDS: Readonly<Record<string, string>> = {

15-

"openai-codex": "openai",

16-

};

1714
1815

function isOAuthRefreshFailureMessage(message: string): boolean {

1916

const lower = message.toLowerCase();

@@ -72,10 +69,7 @@ export function classifyOAuthRefreshFailure(message: string): {

7269
7370

export function buildOAuthRefreshFailureLoginCommand(provider: string | null | undefined): string {

7471

const sanitizedProvider = sanitizeOAuthRefreshFailureProvider(provider);

75-

const reauthProvider = sanitizedProvider

76-

? (RETIRED_REAUTH_PROVIDER_IDS[sanitizedProvider] ?? sanitizedProvider)

77-

: null;

78-

return reauthProvider

79-

? formatCliCommand(`openclaw models auth login --provider ${reauthProvider}`)

72+

return sanitizedProvider

73+

? formatCliCommand(`openclaw models auth login --provider ${sanitizedProvider}`)

8074

: formatCliCommand("openclaw models auth login");

8175

}

Original file line numberDiff line numberDiff line change

@@ -583,36 +583,6 @@ describe("repairSessionFileIfNeeded", () => {

583583

expect(JSON.parse(lines[4])).toEqual(deliveryMirror);

584584

});

585585
586-

it("repairs missing tool results in legacy OpenAI Codex transcripts", async () => {

587-

const { file } = await createTempSessionPath();

588-

const { header, message } = buildSessionHeaderAndMessage();

589-

const toolCallAssistant = {

590-

type: "message",

591-

id: "msg-asst-legacy-process",

592-

parentId: "msg-1",

593-

timestamp: new Date().toISOString(),

594-

message: {

595-

role: "assistant",

596-

provider: "openai-codex",

597-

model: "gpt-5.5",

598-

api: "openai-codex-responses",

599-

content: [{ type: "toolCall", id: "call_process|fc_1", name: "process", arguments: {} }],

600-

stopReason: "toolUse",

601-

},

602-

};

603-

const original = `${JSON.stringify(header)}\n${JSON.stringify(message)}\n${JSON.stringify(toolCallAssistant)}\n`;

604-

await fs.writeFile(file, original, "utf-8");

605-
606-

const result = await repairSessionFileIfNeeded({ sessionFile: file });

607-
608-

expect(result.repaired).toBe(true);

609-

expect(result.insertedToolResults).toBe(1);

610-

const lines = (await fs.readFile(file, "utf-8")).trimEnd().split("\n");

611-

const inserted = JSON.parse(lines[3]);

612-

expect(inserted.message.role).toBe("toolResult");

613-

expect(inserted.message.toolCallId).toBe("call_process|fc_1");

614-

});

615-
616586

it("does not duplicate code-mode tool results that are already persisted", async () => {

617587

const { file } = await createTempSessionPath();

618588

const { header, message } = buildSessionHeaderAndMessage();

Original file line numberDiff line numberDiff line change

@@ -206,17 +206,10 @@ function isCodeModeToolCallRepairCandidate(entry: unknown): entry is SessionMess

206206

provider?: unknown;

207207

stopReason?: unknown;

208208

};

209-

// Persisted transcripts from the retired OpenAI Codex route still need this

210-

// repair so replay sees a complete tool-call/tool-result pair.

211-

const legacyOpenAIProvider = "openai-codex";

212-

const legacyOpenAIResponsesApi = "openai-codex-responses";

213-

const openAIProvider = message.provider === "openai" || message.provider === legacyOpenAIProvider;

214-

const openAIResponsesApi =

215-

message.api === "openai-chatgpt-responses" || message.api === legacyOpenAIResponsesApi;

216209

return (

217210

message.role === "assistant" &&

218-

openAIResponsesApi &&

219-

openAIProvider &&

211+

message.api === "openai-chatgpt-responses" &&

212+

message.provider === "openai" &&

220213

message.stopReason !== "error" &&

221214

message.stopReason !== "aborted"

222215

);

Original file line numberDiff line numberDiff line change

@@ -41,6 +41,9 @@ const LEGACY_CODEX_PROVIDER_ID = "openai-codex";

4141

const CODEX_OAUTH_WARNING_TITLE = "Codex OAuth";

4242

const OPENAI_BASE_URL = "https://api.openai.com/v1";

4343

const LEGACY_CODEX_APIS = new Set(["openai-responses", "openai-completions"]);

44+

const DOCTOR_REAUTH_PROVIDER_ALIASES: Readonly<Record<string, string>> = {

45+

[LEGACY_CODEX_PROVIDER_ID]: OPENAI_PROVIDER_ID,

46+

};

4447
4548

function hasConfiguredCodexOAuthProfile(cfg: OpenClawConfig): boolean {

4649

return Object.values(cfg.auth?.profiles ?? {}).some(

@@ -221,7 +224,10 @@ export function formatOAuthRefreshFailureDoctorLine(params: {

221224

if (!classified) {

222225

return null;

223226

}

224-

const provider = classified.provider ?? params.provider;

227+

const rawProvider = classified.provider ?? params.provider;

228+

const provider = rawProvider

229+

? (DOCTOR_REAUTH_PROVIDER_ALIASES[rawProvider] ?? rawProvider)

230+

: null;

225231

const command = buildOAuthRefreshFailureLoginCommand(provider);

226232

if (classified.reason) {

227233

return `- ${params.profileId}: re-auth required [${formatOAuthRefreshFailureReason(classified.reason)}] — Run \`${command}\`.`;

Original file line numberDiff line numberDiff line change

@@ -143,11 +143,45 @@ describe("doctor session transcript repair", () => {

143143

expect(note).toHaveBeenCalledTimes(1);

144144

const [message, title] = requireFirstMockCall(note, "doctor note") as [string, string];

145145

expect(title).toBe("Session transcripts");

146-

expect(message).toContain("duplicated prompt-rewrite branches");

146+

expect(message).toContain("legacy state");

147147

expect(message).toContain('Run "openclaw doctor --fix"');

148148

expect(countNonEmptyLines(await fs.readFile(filePath, "utf-8"))).toBe(3);

149149

});

150150
151+

it("rewrites legacy OpenAI Codex transcript metadata only during doctor repair", async () => {

152+

const filePath = await writeTranscript([

153+

{ type: "session", version: 3, id: "session-1", timestamp: "2026-04-25T00:00:00Z" },

154+

{

155+

type: "message",

156+

id: "legacy-assistant",

157+

parentId: null,

158+

message: {

159+

role: "assistant",

160+

provider: "openai-codex",

161+

api: "openai-codex-responses",

162+

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

163+

},

164+

},

165+

]);

166+
167+

const preview = await repairBrokenSessionTranscriptFile({ filePath, shouldRepair: false });

168+
169+

expect(preview.broken).toBe(true);

170+

expect(preview.repaired).toBe(false);

171+

expect(preview.legacyOpenAICodexEntries).toBe(1);

172+

expect(await fs.readFile(filePath, "utf-8")).toContain("openai-codex");

173+
174+

const result = await repairBrokenSessionTranscriptFile({ filePath, shouldRepair: true });

175+
176+

expect(result.broken).toBe(true);

177+

expect(result.repaired).toBe(true);

178+

expect(result.legacyOpenAICodexEntries).toBe(1);

179+

const lines = (await fs.readFile(filePath, "utf-8")).trim().split(/\r?\n/);

180+

const assistant = JSON.parse(lines[1]);

181+

expect(assistant.message.provider).toBe("openai");

182+

expect(assistant.message.api).toBe("openai-chatgpt-responses");

183+

});

184+
151185

it("ignores ordinary branch history without internal runtime context", async () => {

152186

const filePath = await writeTranscript([

153187

{ type: "session", version: 3, id: "session-1", timestamp: "2026-04-25T00:00:00Z" },