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

推荐订阅源

S
SegmentFault 最新的问题
博客园 - 三生石上(FineUI控件)
WordPress大学
WordPress大学
博客园 - 【当耐特】
月光博客
月光博客
Vercel News
Vercel News
D
Docker
I
InfoQ
Apple Machine Learning Research
Apple Machine Learning Research
博客园 - 叶小钗
MongoDB | Blog
MongoDB | Blog
GbyAI
GbyAI
有赞技术团队
有赞技术团队
雷峰网
雷峰网
博客园 - 聂微东
小众软件
小众软件
Y
Y Combinator Blog
腾讯CDC
L
LangChain Blog
The GitHub Blog
The GitHub Blog
宝玉的分享
宝玉的分享
Stack Overflow Blog
Stack Overflow Blog
大猫的无限游戏
大猫的无限游戏
T
The Blog of Author Tim Ferriss

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(codex): harden computer use setup states · openclaw/o...
steipete · 2026-04-28 · via Recent Commits to openclaw:main

@@ -52,6 +52,27 @@ describe("Codex Computer Use setup", () => {

5252

expect(request).not.toHaveBeenCalledWith("plugin/install", expect.anything());

5353

});

545455+

it("reports an installed but disabled Computer Use plugin separately", async () => {

56+

const request = createComputerUseRequest({ installed: true, enabled: false });

57+58+

await expect(

59+

readCodexComputerUseStatus({

60+

pluginConfig: { computerUse: { enabled: true, marketplaceName: "desktop-tools" } },

61+

request,

62+

}),

63+

).resolves.toEqual(

64+

expect.objectContaining({

65+

ready: false,

66+

installed: true,

67+

pluginEnabled: false,

68+

mcpServerAvailable: false,

69+

message:

70+

"Computer Use is installed, but the computer-use plugin is disabled. Run /codex computer-use install or enable computerUse.autoInstall to re-enable it.",

71+

}),

72+

);

73+

expect(request).not.toHaveBeenCalledWith("plugin/install", expect.anything());

74+

});

75+5576

it("does not register marketplace sources during status checks", async () => {

5677

const request = createComputerUseRequest({ installed: true });

5778

@@ -129,6 +150,28 @@ describe("Codex Computer Use setup", () => {

129150

expect(request).toHaveBeenCalledWith("config/mcpServer/reload", undefined);

130151

});

131152153+

it("re-enables an installed but disabled Computer Use plugin during install", async () => {

154+

const request = createComputerUseRequest({ installed: true, enabled: false });

155+156+

await expect(

157+

installCodexComputerUse({

158+

pluginConfig: { computerUse: { marketplaceName: "desktop-tools" } },

159+

request,

160+

}),

161+

).resolves.toEqual(

162+

expect.objectContaining({

163+

ready: true,

164+

installed: true,

165+

pluginEnabled: true,

166+

message: "Computer Use is ready.",

167+

}),

168+

);

169+

expect(request).toHaveBeenCalledWith("plugin/install", {

170+

marketplacePath: "/marketplaces/desktop-tools/.agents/plugins/marketplace.json",

171+

pluginName: "computer-use",

172+

});

173+

});

174+132175

it("fails closed when Computer Use is required but not installed", async () => {

133176

const request = createComputerUseRequest({ installed: false });

134177

@@ -240,6 +283,27 @@ describe("Codex Computer Use setup", () => {

240283

expect(request).not.toHaveBeenCalledWith("plugin/read", expect.anything());

241284

});

242285286+

it("fails closed instead of installing from a remote-only Codex marketplace", async () => {

287+

const request = createRemoteOnlyComputerUseRequest();

288+289+

await expect(

290+

installCodexComputerUse({

291+

pluginConfig: { computerUse: { marketplaceName: "openai-curated" } },

292+

request,

293+

}),

294+

).rejects.toMatchObject({

295+

status: expect.objectContaining({

296+

ready: false,

297+

installed: false,

298+

pluginEnabled: false,

299+

marketplaceName: "openai-curated",

300+

message:

301+

"Computer Use is available in remote Codex marketplace openai-curated, but Codex app-server does not support remote plugin install yet. Configure computerUse.marketplaceSource or computerUse.marketplacePath for a local marketplace, then run /codex computer-use install.",

302+

}),

303+

});

304+

expect(request).not.toHaveBeenCalledWith("plugin/install", expect.anything());

305+

});

306+243307

it("waits for the default Codex marketplace during install", async () => {

244308

vi.useFakeTimers();

245309

const request = createComputerUseRequest({

@@ -291,9 +355,11 @@ describe("Codex Computer Use setup", () => {

291355292356

function createComputerUseRequest(params: {

293357

installed: boolean;

358+

enabled?: boolean;

294359

marketplaceAvailableAfterListCalls?: number;

295360

}): CodexComputerUseRequest {

296361

let installed = params.installed;

362+

let enabled = params.enabled ?? installed;

297363

let pluginListCalls = 0;

298364

return vi.fn(async (method: string, requestParams?: unknown) => {

299365

if (method === "experimentalFeature/enablement/set") {

@@ -317,7 +383,7 @@ function createComputerUseRequest(params: {

317383

name: "desktop-tools",

318384

path: "/marketplaces/desktop-tools/.agents/plugins/marketplace.json",

319385

interface: null,

320-

plugins: [pluginSummary(installed)],

386+

plugins: [pluginSummary(installed, "desktop-tools", enabled)],

321387

},

322388

]

323389

: [],

@@ -335,7 +401,7 @@ function createComputerUseRequest(params: {

335401

plugin: {

336402

marketplaceName: "desktop-tools",

337403

marketplacePath: "/marketplaces/desktop-tools/.agents/plugins/marketplace.json",

338-

summary: pluginSummary(installed),

404+

summary: pluginSummary(installed, "desktop-tools", enabled),

339405

description: "Control desktop apps.",

340406

skills: [],

341407

apps: [],

@@ -345,36 +411,78 @@ function createComputerUseRequest(params: {

345411

}

346412

if (method === "plugin/install") {

347413

installed = true;

414+

enabled = true;

348415

return { authPolicy: "ON_INSTALL", appsNeedingAuth: [] };

349416

}

350417

if (method === "config/mcpServer/reload") {

351418

return undefined;

352419

}

353420

if (method === "mcpServerStatus/list") {

354421

return {

355-

data: installed

356-

? [

357-

{

358-

name: "computer-use",

359-

tools: {

360-

list_apps: {

361-

name: "list_apps",

362-

inputSchema: { type: "object" },

422+

data:

423+

installed && enabled

424+

? [

425+

{

426+

name: "computer-use",

427+

tools: {

428+

list_apps: {

429+

name: "list_apps",

430+

inputSchema: { type: "object" },

431+

},

363432

},

433+

resources: [],

434+

resourceTemplates: [],

435+

authStatus: "unsupported",

364436

},

365-

resources: [],

366-

resourceTemplates: [],

367-

authStatus: "unsupported",

368-

},

369-

]

370-

: [],

437+

]

438+

: [],

371439

nextCursor: null,

372440

};

373441

}

374442

throw new Error(`unexpected request ${method}`);

375443

}) as CodexComputerUseRequest;

376444

}

377445446+

function createRemoteOnlyComputerUseRequest(): CodexComputerUseRequest {

447+

return vi.fn(async (method: string, requestParams?: unknown) => {

448+

if (method === "experimentalFeature/enablement/set") {

449+

return { enablement: { plugins: true } };

450+

}

451+

if (method === "plugin/list") {

452+

return {

453+

marketplaces: [

454+

{

455+

name: "openai-curated",

456+

path: null,

457+

interface: null,

458+

plugins: [pluginSummary(false, "openai-curated", false, "remote")],

459+

},

460+

],

461+

marketplaceLoadErrors: [],

462+

featuredPluginIds: [],

463+

};

464+

}

465+

if (method === "plugin/read") {

466+

expect(requestParams).toEqual({

467+

remoteMarketplaceName: "openai-curated",

468+

pluginName: "computer-use",

469+

});

470+

return {

471+

plugin: {

472+

marketplaceName: "openai-curated",

473+

marketplacePath: null,

474+

summary: pluginSummary(false, "openai-curated", false, "remote"),

475+

description: "Control desktop apps.",

476+

skills: [],

477+

apps: [],

478+

mcpServers: ["computer-use"],

479+

},

480+

};

481+

}

482+

throw new Error(`unexpected request ${method}`);

483+

}) as CodexComputerUseRequest;

484+

}

485+378486

function createAmbiguousComputerUseRequest(): CodexComputerUseRequest {

379487

return vi.fn(async (method: string) => {

380488

if (method === "plugin/list") {

@@ -488,13 +596,21 @@ function marketplaceEntry(marketplaceName: string, installed: boolean) {

488596

};

489597

}

490598491-

function pluginSummary(installed: boolean, marketplaceName = "desktop-tools") {

599+

function pluginSummary(

600+

installed: boolean,

601+

marketplaceName = "desktop-tools",

602+

enabled = installed,

603+

source: "local" | "remote" = "local",

604+

) {

492605

return {

493606

id: `computer-use@${marketplaceName}`,

494607

name: "computer-use",

495-

source: { type: "local", path: `/marketplaces/${marketplaceName}/plugins/computer-use` },

608+

source:

609+

source === "local"

610+

? { type: "local", path: `/marketplaces/${marketplaceName}/plugins/computer-use` }

611+

: { type: "remote" },

496612

installed,

497-

enabled: installed,

613+

enabled,

498614

installPolicy: "AVAILABLE",

499615

authPolicy: "ON_INSTALL",

500616

interface: null,