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

推荐订阅源

Apple Machine Learning Research
Apple Machine Learning Research
爱范儿
爱范儿
博客园_首页
博客园 - 【当耐特】
V
Visual Studio Blog
博客园 - 叶小钗
月光博客
月光博客
美团技术团队
J
Java Code Geeks
小众软件
小众软件
Y
Y Combinator Blog
博客园 - Franky
Martin Fowler
Martin Fowler
博客园 - 聂微东
Microsoft Azure Blog
Microsoft Azure Blog
IT之家
IT之家
MyScale Blog
MyScale Blog
人人都是产品经理
人人都是产品经理
Microsoft Security Blog
Microsoft Security Blog
OSCHINA 社区最新新闻
OSCHINA 社区最新新闻
阮一峰的网络日志
阮一峰的网络日志
酷 壳 – CoolShell
酷 壳 – CoolShell
钛媒体:引领未来商业与生活新知
钛媒体:引领未来商业与生活新知
云风的 BLOG
云风的 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
feat(browser): extend --labels overlay to full-page and e...
clawsweeper · 2026-06-14 · via Recent Commits to openclaw:main
1+

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

2+

import {

3+

installPwToolsCoreTestHooks,

4+

setPwToolsCoreCurrentPage,

5+

setPwToolsCoreCurrentRefLocator,

6+

} from "./pw-tools-core.test-harness.js";

7+8+

installPwToolsCoreTestHooks();

9+

const mod = await import("./pw-tools-core.js");

10+11+

type EvaluateArg = unknown;

12+13+

function evaluateMockReturning(view: { x: number; y: number; width?: number; height?: number }) {

14+

// Caller reads { x, y, width, height } in one evaluate; default to a normal

15+

// desktop viewport so refs near the top stay in-viewport unless a test puts

16+

// them out of range explicitly.

17+

const result = { width: 1280, height: 720, ...view };

18+

return vi.fn(async (arg: EvaluateArg) => {

19+

if (typeof arg === "function") {

20+

return result;

21+

}

22+

return true;

23+

});

24+

}

25+26+

describe("screenshotWithLabelsViaPlaywright (viewport)", () => {

27+

beforeEach(() => {

28+

vi.clearAllMocks();

29+

});

30+31+

it("calls page.screenshot without fullPage and returns annotations", async () => {

32+

const evaluate = evaluateMockReturning({ x: 0, y: 100 });

33+

const screenshot = vi.fn(async () => Buffer.from("PNG"));

34+

setPwToolsCoreCurrentPage({ evaluate, screenshot, url: () => "https://example.com" });

35+

setPwToolsCoreCurrentRefLocator({

36+

boundingBox: async () => ({ x: 10, y: 200, width: 50, height: 20 }),

37+

});

38+39+

const result = await mod.screenshotWithLabelsViaPlaywright({

40+

cdpUrl: "http://127.0.0.1:18792",

41+

targetId: "T1",

42+

refs: { e1: { role: "button", name: "Submit" } },

43+

type: "png",

44+

});

45+46+

expect(screenshot).toHaveBeenCalledWith(expect.objectContaining({ type: "png" }));

47+

expect(screenshot).not.toHaveBeenCalledWith(expect.objectContaining({ fullPage: true }));

48+49+

expect(result.annotations).toHaveLength(1);

50+

expect(result.annotations[0]).toMatchObject({

51+

ref: "e1",

52+

number: 1,

53+

role: "button",

54+

name: "Submit",

55+

});

56+

// viewport-mode box = doc(box.x + scroll.x, box.y + scroll.y) - scroll = bbox

57+

expect(result.annotations[0]?.box).toEqual({ x: 10, y: 200, width: 50, height: 20 });

58+

expect(result.skipped).toBe(0);

59+

});

60+61+

it("runs the clear script even when screenshot throws", async () => {

62+

const evaluate = evaluateMockReturning({ x: 0, y: 0 });

63+

const screenshot = vi.fn(async () => {

64+

throw new Error("boom");

65+

});

66+

setPwToolsCoreCurrentPage({ evaluate, screenshot });

67+

setPwToolsCoreCurrentRefLocator({

68+

boundingBox: async () => ({ x: 0, y: 0, width: 1, height: 1 }),

69+

});

70+71+

await expect(

72+

mod.screenshotWithLabelsViaPlaywright({

73+

cdpUrl: "http://127.0.0.1:18792",

74+

targetId: "T1",

75+

refs: { e1: { role: "button" } },

76+

}),

77+

).rejects.toThrow(/boom/);

78+79+

// The clear script must have run (string evaluate calls include the overlay attr)

80+

const clearCalls = evaluate.mock.calls.filter(

81+

([arg]) => typeof arg === "string" && arg.includes("data-openclaw-labels"),

82+

);

83+

// inject + clear = at least 2 string evaluations

84+

expect(clearCalls.length).toBeGreaterThanOrEqual(2);

85+

});

86+87+

it("counts off-viewport refs as skipped but still surfaces them in annotations", async () => {

88+

const evaluate = evaluateMockReturning({ x: 0, y: 0, width: 1280, height: 720 });

89+

const screenshot = vi.fn(async () => Buffer.from("PNG"));

90+

setPwToolsCoreCurrentPage({ evaluate, screenshot });

91+

// bbox is far below the viewport (y: 5000): not drawn, but still reported

92+

// so callers keep the position and a non-zero skipped count.

93+

setPwToolsCoreCurrentRefLocator({

94+

boundingBox: async () => ({ x: 0, y: 5000, width: 50, height: 20 }),

95+

});

96+97+

const result = await mod.screenshotWithLabelsViaPlaywright({

98+

cdpUrl: "http://127.0.0.1:18792",

99+

targetId: "T1",

100+

refs: { e1: { role: "button" } },

101+

});

102+103+

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

104+

expect(result.labels).toBe(0);

105+

expect(result.annotations).toHaveLength(1);

106+

expect(result.annotations[0]?.ref).toBe("e1");

107+

});

108+

});

109+110+

describe("screenshotWithLabelsViaPlaywright (fullpage)", () => {

111+

beforeEach(() => vi.clearAllMocks());

112+113+

it("forwards fullPage:true to page.screenshot and uses doc-space annotations", async () => {

114+

const evaluate = evaluateMockReturning({ x: 0, y: 1000 });

115+

const screenshot = vi.fn(async () => Buffer.from("FULL"));

116+

setPwToolsCoreCurrentPage({ evaluate, screenshot });

117+

setPwToolsCoreCurrentRefLocator({

118+

boundingBox: async () => ({ x: 10, y: 200, width: 50, height: 20 }),

119+

});

120+121+

const result = await mod.screenshotWithLabelsViaPlaywright({

122+

cdpUrl: "http://127.0.0.1:18792",

123+

targetId: "T1",

124+

refs: { e1: { role: "button" } },

125+

fullPage: true,

126+

});

127+128+

expect(screenshot).toHaveBeenCalledWith(expect.objectContaining({ fullPage: true }));

129+

// doc-space: scroll y=1000 + bbox y=200 = 1200

130+

expect(result.annotations[0]?.box.y).toBe(1200);

131+

expect(result.annotations[0]?.box.x).toBe(10);

132+

});

133+

});

134+135+

describe("screenshotWithLabelsViaPlaywright (element/ref)", () => {

136+

beforeEach(() => vi.clearAllMocks());

137+138+

it("uses refLocator.screenshot for ref mode and projects relative to element", async () => {

139+

const evaluate = evaluateMockReturning({ x: 0, y: 0 });

140+

// First call resolves the element rect (container), second resolves e1 annotation bbox.

141+

const boundingBox = vi

142+

.fn<() => Promise<{ x: number; y: number; width: number; height: number } | null>>()

143+

.mockResolvedValueOnce({ x: 50, y: 100, width: 200, height: 300 })

144+

.mockResolvedValueOnce({ x: 60, y: 110, width: 30, height: 20 });

145+

const elementScreenshot = vi.fn(async () => Buffer.from("ELEM"));

146+

setPwToolsCoreCurrentPage({ evaluate, screenshot: vi.fn() });

147+

setPwToolsCoreCurrentRefLocator({ boundingBox, screenshot: elementScreenshot });

148+149+

const result = await mod.screenshotWithLabelsViaPlaywright({

150+

cdpUrl: "http://127.0.0.1:18792",

151+

targetId: "T1",

152+

refs: { e1: { role: "button" } },

153+

ref: "container",

154+

});

155+156+

expect(elementScreenshot).toHaveBeenCalledTimes(1);

157+

// Element-relative: doc(60,110) - elementRect(50,100) = (10,10)

158+

expect(result.annotations).toHaveLength(1);

159+

expect(result.annotations[0]?.box).toEqual({ x: 10, y: 10, width: 30, height: 20 });

160+

});

161+162+

it("throws when ref/element cannot be resolved", async () => {

163+

const evaluate = evaluateMockReturning({ x: 0, y: 0 });

164+

setPwToolsCoreCurrentPage({ evaluate, screenshot: vi.fn() });

165+

setPwToolsCoreCurrentRefLocator({

166+

boundingBox: async () => null,

167+

screenshot: vi.fn(),

168+

});

169+170+

await expect(

171+

mod.screenshotWithLabelsViaPlaywright({

172+

cdpUrl: "http://127.0.0.1:18792",

173+

targetId: "T1",

174+

refs: { e1: { role: "button" } },

175+

ref: "missing",

176+

}),

177+

).rejects.toThrow(/element not found/i);

178+

});

179+

});

180+181+

describe("screenshotWithLabelsViaPlaywright (skipped accounting)", () => {

182+

beforeEach(() => vi.clearAllMocks());

183+184+

it("counts refs whose boundingBox is null toward skipped", async () => {

185+

const evaluate = evaluateMockReturning({ x: 0, y: 0 });

186+

const screenshot = vi.fn(async () => Buffer.from("PNG"));

187+

setPwToolsCoreCurrentPage({ evaluate, screenshot });

188+

// Two refs: first returns a box, second returns null (e.g. element detached).

189+

const boundingBox = vi

190+

.fn<() => Promise<{ x: number; y: number; width: number; height: number } | null>>()

191+

.mockResolvedValueOnce({ x: 10, y: 20, width: 30, height: 40 })

192+

.mockResolvedValueOnce(null);

193+

setPwToolsCoreCurrentRefLocator({ boundingBox });

194+195+

const result = await mod.screenshotWithLabelsViaPlaywright({

196+

cdpUrl: "http://127.0.0.1:18792",

197+

targetId: "T1",

198+

refs: { e1: { role: "button" }, e2: { role: "link" } },

199+

});

200+201+

expect(result.annotations).toHaveLength(1);

202+

expect(result.annotations[0]?.ref).toBe("e1");

203+

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

204+

});

205+

});