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

推荐订阅源

N
Netflix TechBlog - Medium
J
Java Code Geeks
爱范儿
爱范儿
雷峰网
雷峰网
OSCHINA 社区最新新闻
OSCHINA 社区最新新闻
Cyber Security Advisories - MS-ISAC
Cyber Security Advisories - MS-ISAC
博客园 - 三生石上(FineUI控件)
H
Hackread – Cybersecurity News, Data Breaches, AI and More
B
Blog RSS Feed
Google DeepMind News
Google DeepMind News
Jina AI
Jina AI
The GitHub Blog
The GitHub Blog
I
InfoQ
月光博客
月光博客
博客园 - 聂微东
博客园 - Franky
The Cloudflare Blog
阮一峰的网络日志
阮一峰的网络日志
博客园_首页
G
Google Developers Blog
Blog — PlanetScale
Blog — PlanetScale
L
LangChain Blog
罗磊的独立博客
Apple Machine Learning Research
Apple Machine Learning Research

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
test: dedupe codex approval gateway mock calls · openclaw...
steipete · 2026-05-12 · via Recent Commits to openclaw:main

@@ -21,13 +21,17 @@ function requireRecord(value: unknown, label: string): Record<string, unknown> {

21212222

function gatewayRequestPayload(callIndex = 0) {

2323

return requireRecord(

24-

mockCallGatewayTool.mock.calls[callIndex]?.[2],

24+

mockCallGatewayTool.mock.calls.at(callIndex)?.[2],

2525

`gateway request payload ${callIndex + 1}`,

2626

);

2727

}

28282929

function gatewayCallOptions(callIndex = 0) {

30-

return mockCallGatewayTool.mock.calls[callIndex]?.[3];

30+

return mockCallGatewayTool.mock.calls.at(callIndex)?.[3];

31+

}

32+33+

function gatewayCallMethod(callIndex = 0) {

34+

return mockCallGatewayTool.mock.calls.at(callIndex)?.[0];

3135

}

32363337

function findApprovalEvent(

@@ -98,8 +102,8 @@ describe("Codex app-server approval bridge", () => {

98102

"plugin.approval.request",

99103

"plugin.approval.waitDecision",

100104

]);

101-

expect(mockCallGatewayTool.mock.calls[0]?.[0]).toBe("plugin.approval.request");

102-

expect(typeof mockCallGatewayTool.mock.calls[0]?.[1]).toBe("object");

105+

expect(gatewayCallMethod()).toBe("plugin.approval.request");

106+

expect(typeof mockCallGatewayTool.mock.calls.at(0)?.[1]).toBe("object");

103107

const requestPayload = gatewayRequestPayload();

104108

expect(requestPayload.pluginId).toBe("openclaw-codex-app-server");

105109

expect(requestPayload.title).toBe("Codex app-server command approval");

@@ -168,8 +172,7 @@ describe("Codex app-server approval bridge", () => {

168172

});

169173170174

expect(result).toEqual({ decision: "acceptForSession" });

171-

const [, , requestPayload] = mockCallGatewayTool.mock.calls[0] ?? [];

172-

const description = (requestPayload as { description: string }).description;

175+

const description = String(gatewayRequestPayload().description);

173176

expect(description).toContain("Command: npm install");

174177

expect(description).toContain("Additional permissions: network, fileSystem");

175178

expect(description).toContain("High-risk targets: network access, filesystem root");

@@ -207,8 +210,7 @@ describe("Codex app-server approval bridge", () => {

207210

turnId: "turn-1",

208211

});

209212210-

const [, , requestPayload] = mockCallGatewayTool.mock.calls[0] ?? [];

211-

const description = (requestPayload as { description: string }).description;

213+

const description = String(gatewayRequestPayload().description);

212214

expect(description).toContain("[preview truncated or unsafe content omitted]");

213215

expect(description).toContain("Additional permissions: network, fileSystem");

214216

expect(description).toContain("High-risk targets: network access, filesystem root");

@@ -261,8 +263,7 @@ describe("Codex app-server approval bridge", () => {

261263

turnId: "turn-1",

262264

});

263265264-

const [, , requestPayload] = mockCallGatewayTool.mock.calls[0] ?? [];

265-

const description = (requestPayload as { description: string }).description;

266+

const description = String(gatewayRequestPayload().description);

266267

expect(description).toContain(

267268

"printf '&lt;\uff20U123&gt; \uff3btrusted\uff3d\uff08https://evil\uff09 \uff20here'",

268269

);

@@ -372,8 +373,7 @@ describe("Codex app-server approval bridge", () => {

372373

turnId: "turn-1",

373374

});

374375375-

const [, , requestPayload] = mockCallGatewayTool.mock.calls[0] ?? [];

376-

const description = (requestPayload as { description: string }).description;

376+

const description = String(gatewayRequestPayload().description);

377377

expect(description).toContain("[preview truncated or unsafe content omitted]");

378378

const omittedEvent = findApprovalEvent(params, {});

379379

expect(omittedEvent.commandPreviewOmitted).toBe(true);

@@ -610,8 +610,8 @@ describe("Codex app-server approval bridge", () => {

610610

},

611611

scope: "turn",

612612

});

613-

expect(mockCallGatewayTool.mock.calls[0]?.[0]).toBe("plugin.approval.request");

614-

expect(typeof mockCallGatewayTool.mock.calls[0]?.[1]).toBe("object");

613+

expect(gatewayCallMethod()).toBe("plugin.approval.request");

614+

expect(typeof mockCallGatewayTool.mock.calls.at(0)?.[1]).toBe("object");

615615

const requestPayload = gatewayRequestPayload();

616616

expect(requestPayload.title).toBe("Codex app-server permission approval");

617617

expect(requestPayload.toolName).toBe("codex_permission_approval");

@@ -715,8 +715,7 @@ describe("Codex app-server approval bridge", () => {

715715

},

716716

scope: "turn",

717717

});

718-

const [, , requestPayload] = mockCallGatewayTool.mock.calls[0] ?? [];

719-

const description = (requestPayload as { description: string }).description;

718+

const description = String(gatewayRequestPayload().description);

720719

expect(description).toContain("Network enabled: true");

721720

expect(description).toContain("File system read: ~/.ssh/id_rsa; write: /");

722721

expect(description).toContain("entries: read /workspace/project, write /tmp/output (+1 more)");

@@ -747,8 +746,7 @@ describe("Codex app-server approval bridge", () => {

747746

turnId: "turn-1",

748747

});

749748750-

const [, , requestPayload] = mockCallGatewayTool.mock.calls[0] ?? [];

751-

const description = (requestPayload as { description: string }).description;

749+

const description = String(gatewayRequestPayload().description);

752750

expect(description).toContain("File system roots: ~; readPaths: ~/.ssh/id_rsa, ~/project");

753751

expect(description).not.toContain("High-risk targets");

754752

});

@@ -775,8 +773,7 @@ describe("Codex app-server approval bridge", () => {

775773

turnId: "turn-1",

776774

});

777775778-

const [, , requestPayload] = mockCallGatewayTool.mock.calls[0] ?? [];

779-

const description = (requestPayload as { description: string }).description;

776+

const description = String(gatewayRequestPayload().description);

780777

expect(description).toContain("example.com");

781778

expect(description).toContain("safe .example.com");

782779

expect(description).toContain("/tmp/project");