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

推荐订阅源

WordPress大学
WordPress大学
The Cloudflare Blog
大猫的无限游戏
大猫的无限游戏
小众软件
小众软件
V
Visual Studio Blog
钛媒体:引领未来商业与生活新知
钛媒体:引领未来商业与生活新知
博客园 - 司徒正美
OSCHINA 社区最新新闻
OSCHINA 社区最新新闻
酷 壳 – CoolShell
酷 壳 – CoolShell
奇客Solidot–传递最新科技情报
奇客Solidot–传递最新科技情报
月光博客
月光博客
The GitHub Blog
The GitHub Blog
L
LangChain Blog
D
DataBreaches.Net
T
The Blog of Author Tim Ferriss
F
Fortinet All Blogs
博客园 - Franky
阮一峰的网络日志
阮一峰的网络日志
GbyAI
GbyAI
Apple Machine Learning Research
Apple Machine Learning Research
宝玉的分享
宝玉的分享
Engineering at Meta
Engineering at Meta
V
V2EX
MyScale Blog
MyScale Blog

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(browser): avoid restart hint for external profiles · ...
steipete · 2026-04-26 · via Recent Commits to openclaw:main

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

11

import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";

22

import "../../test-support/browser-security-runtime.mock.js";

3+

import type { OpenClawConfig } from "../config/config.js";

34

import type { BrowserDispatchResponse } from "./routes/dispatcher.js";

4556

vi.mock("openclaw/plugin-sdk/ssrf-runtime", async () => {

@@ -28,7 +29,7 @@ function okDispatchResponse(): BrowserDispatchResponse {

2829

}

29303031

const mocks = vi.hoisted(() => ({

31-

loadConfig: vi.fn(() => ({

32+

loadConfig: vi.fn<() => OpenClawConfig>(() => ({

3233

gateway: {

3334

auth: {

3435

token: "loopback-token",

@@ -215,6 +216,202 @@ describe("fetchBrowserJson loopback auth", () => {

215216

});

216217

});

217218219+

it("avoids restart-gateway guidance for attachOnly dispatcher timeouts", async () => {

220+

mocks.loadConfig.mockReturnValue({

221+

browser: {

222+

attachOnly: true,

223+

defaultProfile: "manual",

224+

profiles: {

225+

manual: {

226+

cdpUrl: "http://127.0.0.1:9222",

227+

attachOnly: true,

228+

color: "#00AA00",

229+

},

230+

},

231+

},

232+

});

233+

mocks.dispatch.mockRejectedValueOnce(new Error("Chrome CDP handshake timeout"));

234+235+

await expectThrownBrowserFetchError(

236+

() => fetchBrowserJson<{ ok: boolean }>("/tabs?profile=manual"),

237+

{

238+

contains: [

239+

"Chrome CDP handshake timeout",

240+

"browser profile is external to OpenClaw",

241+

"Restarting the OpenClaw gateway will not launch it",

242+

],

243+

omits: ["Restart the OpenClaw gateway", "Do NOT retry the browser tool"],

244+

},

245+

);

246+

});

247+248+

it("avoids restart-gateway guidance for existing-session dispatcher timeouts", async () => {

249+

mocks.loadConfig.mockReturnValue({

250+

browser: {

251+

defaultProfile: "user",

252+

profiles: {

253+

user: {

254+

driver: "existing-session",

255+

attachOnly: true,

256+

color: "#00AA00",

257+

},

258+

},

259+

},

260+

});

261+

mocks.dispatch.mockRejectedValueOnce(new DOMException("operation aborted", "AbortError"));

262+263+

await expectThrownBrowserFetchError(() => fetchBrowserJson<{ ok: boolean }>("/tabs"), {

264+

contains: [

265+

"operation aborted",

266+

"browser profile is external to OpenClaw",

267+

"Restarting the OpenClaw gateway will not launch it",

268+

],

269+

omits: ["Restart the OpenClaw gateway", "Do NOT retry the browser tool"],

270+

});

271+

});

272+273+

it("avoids restart-gateway guidance for remote CDP dispatcher timeouts", async () => {

274+

mocks.loadConfig.mockReturnValue({

275+

browser: {

276+

defaultProfile: "remote",

277+

profiles: {

278+

remote: {

279+

cdpUrl: "https://browserless.example/chrome?token=test",

280+

color: "#00AA00",

281+

},

282+

},

283+

},

284+

});

285+

mocks.dispatch.mockRejectedValueOnce(new Error("timed out"));

286+287+

await expectThrownBrowserFetchError(

288+

() => fetchBrowserJson<{ ok: boolean }>("/tabs?profile=remote"),

289+

{

290+

contains: [

291+

"timed out",

292+

"browser profile is external to OpenClaw",

293+

"Restarting the OpenClaw gateway will not launch it",

294+

],

295+

omits: ["Restart the OpenClaw gateway", "Do NOT retry the browser tool"],

296+

},

297+

);

298+

});

299+300+

it("keeps restart-gateway guidance for managed local dispatcher timeouts", async () => {

301+

mocks.loadConfig.mockReturnValue({

302+

browser: {

303+

defaultProfile: "openclaw",

304+

profiles: {

305+

openclaw: {

306+

cdpPort: 18800,

307+

color: "#FF4500",

308+

},

309+

},

310+

},

311+

});

312+

mocks.dispatch.mockRejectedValueOnce(new Error("Chrome CDP handshake timeout"));

313+314+

await expectThrownBrowserFetchError(

315+

() => fetchBrowserJson<{ ok: boolean }>("/tabs?profile=openclaw"),

316+

{

317+

contains: ["Chrome CDP handshake timeout", "Restart the OpenClaw gateway"],

318+

omits: ["browser profile is external to OpenClaw", "Do NOT retry the browser tool"],

319+

},

320+

);

321+

});

322+323+

it("keeps restart-gateway guidance when dispatcher profile resolution fails", async () => {

324+

mocks.loadConfig.mockImplementation(() => {

325+

throw new Error("config unavailable");

326+

});

327+

mocks.dispatch.mockRejectedValueOnce(new Error("Chrome CDP handshake timeout"));

328+329+

await expectThrownBrowserFetchError(

330+

() => fetchBrowserJson<{ ok: boolean }>("/tabs?profile=manual"),

331+

{

332+

contains: ["Chrome CDP handshake timeout", "Restart the OpenClaw gateway"],

333+

omits: ["browser profile is external to OpenClaw", "Do NOT retry the browser tool"],

334+

},

335+

);

336+

});

337+338+

it("keeps restart-gateway guidance for unknown dispatcher profiles", async () => {

339+

mocks.loadConfig.mockReturnValue({

340+

browser: {

341+

defaultProfile: "openclaw",

342+

profiles: {

343+

openclaw: {

344+

cdpPort: 18800,

345+

color: "#FF4500",

346+

},

347+

},

348+

},

349+

});

350+

mocks.dispatch.mockRejectedValueOnce(new Error("Chrome CDP handshake timeout"));

351+352+

await expectThrownBrowserFetchError(

353+

() => fetchBrowserJson<{ ok: boolean }>("/tabs?profile=missing"),

354+

{

355+

contains: ["Chrome CDP handshake timeout", "Restart the OpenClaw gateway"],

356+

omits: ["browser profile is external to OpenClaw", "Do NOT retry the browser tool"],

357+

},

358+

);

359+

});

360+361+

it("uses the default external profile when dispatcher request omits profile", async () => {

362+

mocks.loadConfig.mockReturnValue({

363+

browser: {

364+

defaultProfile: "manual",

365+

profiles: {

366+

manual: {

367+

cdpUrl: "http://127.0.0.1:9222",

368+

attachOnly: true,

369+

color: "#00AA00",

370+

},

371+

},

372+

},

373+

});

374+

mocks.dispatch.mockRejectedValueOnce(new Error("Chrome CDP handshake timeout"));

375+376+

await expectThrownBrowserFetchError(() => fetchBrowserJson<{ ok: boolean }>("/tabs"), {

377+

contains: [

378+

"Chrome CDP handshake timeout",

379+

"browser profile is external to OpenClaw",

380+

"Restarting the OpenClaw gateway will not launch it",

381+

],

382+

omits: ["Restart the OpenClaw gateway", "Do NOT retry the browser tool"],

383+

});

384+

});

385+386+

it("keeps no-retry hint but not restart guidance for persistent external profile failures", async () => {

387+

mocks.loadConfig.mockReturnValue({

388+

browser: {

389+

attachOnly: true,

390+

defaultProfile: "manual",

391+

profiles: {

392+

manual: {

393+

cdpUrl: "http://127.0.0.1:9222",

394+

attachOnly: true,

395+

color: "#00AA00",

396+

},

397+

},

398+

},

399+

});

400+

mocks.dispatch.mockRejectedValueOnce(new Error("Chrome CDP connection refused"));

401+402+

await expectThrownBrowserFetchError(

403+

() => fetchBrowserJson<{ ok: boolean }>("/tabs?profile=manual"),

404+

{

405+

contains: [

406+

"Chrome CDP connection refused",

407+

"browser profile is external to OpenClaw",

408+

"Do NOT retry the browser tool",

409+

],

410+

omits: ["Restart the OpenClaw gateway"],

411+

},

412+

);

413+

});

414+218415

it("keeps no-retry hint for persistent dispatcher failures", async () => {

219416

mocks.dispatch.mockRejectedValueOnce(new Error("Chrome CDP connection refused"));

220417