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

推荐订阅源

人人都是产品经理
人人都是产品经理
有赞技术团队
有赞技术团队
WordPress大学
WordPress大学
月光博客
月光博客
T
Tailwind CSS Blog
阮一峰的网络日志
阮一峰的网络日志
小众软件
小众软件
钛媒体:引领未来商业与生活新知
钛媒体:引领未来商业与生活新知
Last Week in AI
Last Week in AI
大猫的无限游戏
大猫的无限游戏
S
SegmentFault 最新的问题
罗磊的独立博客
Jina AI
Jina AI
酷 壳 – CoolShell
酷 壳 – CoolShell
宝玉的分享
宝玉的分享
让小产品的独立变现更简单 - ezindie.com
让小产品的独立变现更简单 - ezindie.com
奇客Solidot–传递最新科技情报
奇客Solidot–传递最新科技情报
博客园 - 三生石上(FineUI控件)
量子位
雷峰网
雷峰网
Apple Machine Learning Research
Apple Machine Learning Research
美团技术团队
博客园 - 聂微东
V
V2EX

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): share control runtime state · openclaw/open...
steipete · 2026-04-30 · via Recent Commits to openclaw:main

@@ -0,0 +1,145 @@

1+

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

2+

import { getFreePort } from "../browser/test-port.js";

3+

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

4+5+

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

6+

runtimeConfig: {} as OpenClawConfig,

7+

runtimeSourceConfig: null as OpenClawConfig | null,

8+

ensureBrowserControlAuth: vi.fn(async () => ({ auth: {} })),

9+

resolveBrowserControlAuth: vi.fn(() => ({})),

10+

shouldAutoGenerateBrowserAuth: vi.fn(() => false),

11+

ensureExtensionRelayForProfiles: vi.fn(async () => {}),

12+

stopKnownBrowserProfiles: vi.fn(async () => {}),

13+

isChromeReachable: vi.fn(async () => false),

14+

isChromeCdpReady: vi.fn(async () => false),

15+

}));

16+17+

vi.mock("../config/config.js", async () => {

18+

const actual = await vi.importActual<typeof import("../config/config.js")>("../config/config.js");

19+

return {

20+

...actual,

21+

getRuntimeConfig: () => mocks.runtimeConfig,

22+

getRuntimeConfigSourceSnapshot: () => mocks.runtimeSourceConfig,

23+

loadConfig: () => mocks.runtimeConfig,

24+

};

25+

});

26+27+

vi.mock("../browser/control-auth.js", () => ({

28+

ensureBrowserControlAuth: mocks.ensureBrowserControlAuth,

29+

resolveBrowserControlAuth: mocks.resolveBrowserControlAuth,

30+

shouldAutoGenerateBrowserAuth: mocks.shouldAutoGenerateBrowserAuth,

31+

}));

32+33+

vi.mock("../browser/server-lifecycle.js", () => ({

34+

ensureExtensionRelayForProfiles: mocks.ensureExtensionRelayForProfiles,

35+

stopKnownBrowserProfiles: mocks.stopKnownBrowserProfiles,

36+

}));

37+38+

vi.mock("../browser/chrome.js", () => ({

39+

diagnoseChromeCdp: vi.fn(async () => ({ ok: false })),

40+

formatChromeCdpDiagnostic: vi.fn(() => "not reachable"),

41+

isChromeCdpReady: mocks.isChromeCdpReady,

42+

isChromeReachable: mocks.isChromeReachable,

43+

launchOpenClawChrome: vi.fn(async () => {

44+

throw new Error("launch should not be needed for status");

45+

}),

46+

resolveOpenClawUserDataDir: vi.fn(() => "/tmp/openclaw-browser"),

47+

stopOpenClawChrome: vi.fn(async () => {}),

48+

}));

49+50+

vi.mock("../browser/pw-ai-state.js", () => ({

51+

isPwAiLoaded: vi.fn(() => false),

52+

}));

53+54+

const { startBrowserControlServerFromConfig, stopBrowserControlServer } =

55+

await import("../server.js");

56+

const { stopBrowserControlService } = await import("../control-service.js");

57+

const { browserHandlers } = await import("./browser-request.js");

58+59+

function browserConfig(params: {

60+

gatewayPort: number;

61+

executablePath?: string;

62+

headless?: boolean;

63+

noSandbox?: boolean;

64+

}): OpenClawConfig {

65+

return {

66+

gateway: {

67+

port: params.gatewayPort,

68+

},

69+

browser: {

70+

enabled: true,

71+

defaultProfile: "openclaw",

72+

...(params.executablePath ? { executablePath: params.executablePath } : {}),

73+

...(typeof params.headless === "boolean" ? { headless: params.headless } : {}),

74+

...(typeof params.noSandbox === "boolean" ? { noSandbox: params.noSandbox } : {}),

75+

profiles: {

76+

openclaw: {

77+

cdpPort: params.gatewayPort + 11,

78+

color: "#FF4500",

79+

},

80+

},

81+

},

82+

};

83+

}

84+85+

async function browserRequestStatus(): Promise<unknown> {

86+

const respond = vi.fn();

87+

await browserHandlers["browser.request"]({

88+

params: {

89+

method: "GET",

90+

path: "/",

91+

query: { profile: "openclaw" },

92+

},

93+

respond: respond as never,

94+

context: {

95+

nodeRegistry: {

96+

listConnected: () => [],

97+

},

98+

} as never,

99+

client: null,

100+

req: { type: "req", id: "req-1", method: "browser.request" },

101+

isWebchatConnect: () => false,

102+

});

103+

const call = respond.mock.calls[0];

104+

expect(call?.[0]).toBe(true);

105+

return call?.[1];

106+

}

107+108+

describe("browser.request local control state", () => {

109+

afterEach(async () => {

110+

await stopBrowserControlService();

111+

await stopBrowserControlServer();

112+

mocks.runtimeSourceConfig = null;

113+

vi.clearAllMocks();

114+

});

115+116+

it("uses the same resolved browser config as the HTTP control service", async () => {

117+

const controlPort = await getFreePort();

118+

const gatewayPort = controlPort - 2;

119+120+

mocks.runtimeConfig = browserConfig({

121+

gatewayPort,

122+

executablePath: "/usr/bin/google-chrome",

123+

headless: true,

124+

noSandbox: true,

125+

});

126+

mocks.runtimeSourceConfig = mocks.runtimeConfig;

127+

const httpState = await startBrowserControlServerFromConfig();

128+

expect(httpState?.resolved.executablePath).toBe("/usr/bin/google-chrome");

129+

expect(httpState?.resolved.noSandbox).toBe(true);

130+131+

// The runtime snapshot can lag behind source config after gateway startup;

132+

// browser.request must not fork a second stale control state from it.

133+

mocks.runtimeConfig = browserConfig({

134+

gatewayPort,

135+

headless: false,

136+

noSandbox: false,

137+

});

138+139+

await expect(browserRequestStatus()).resolves.toMatchObject({

140+

executablePath: "/usr/bin/google-chrome",

141+

headless: true,

142+

noSandbox: true,

143+

});

144+

});

145+

});