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

推荐订阅源

D
Docker
小众软件
小众软件
让小产品的独立变现更简单 - ezindie.com
让小产品的独立变现更简单 - ezindie.com
freeCodeCamp Programming Tutorials: Python, JavaScript, Git & More
钛媒体:引领未来商业与生活新知
钛媒体:引领未来商业与生活新知
酷 壳 – CoolShell
酷 壳 – CoolShell
Apple Machine Learning Research
Apple Machine Learning Research
月光博客
月光博客
人人都是产品经理
人人都是产品经理
大猫的无限游戏
大猫的无限游戏
V
V2EX
阮一峰的网络日志
阮一峰的网络日志
奇客Solidot–传递最新科技情报
奇客Solidot–传递最新科技情报
博客园 - Franky
WordPress大学
WordPress大学
有赞技术团队
有赞技术团队
Hugging Face - Blog
Hugging Face - Blog
Jina AI
Jina AI
博客园 - 聂微东
S
SegmentFault 最新的问题
量子位
宝玉的分享
宝玉的分享
OSCHINA 社区最新新闻
OSCHINA 社区最新新闻
博客园_首页

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(diffs): add viewerState, toolbar toggle, shadow root...
liuhao1024 · 2026-06-19 · via Recent Commits to openclaw:main

@@ -55,16 +55,17 @@ const viewerPayload = JSON.stringify({

5555

unsafeCSS: "",

5656

},

5757

langs: ["text"],

58-

oldFile: { fileName: "a.ts", lang: "text", content: "old" },

59-

newFile: { fileName: "a.ts", lang: "text", content: "new" },

58+

oldFile: { name: "a.ts", lang: "text", contents: "old" },

59+

newFile: { name: "a.ts", lang: "text", contents: "new" },

6060

});

616162-

function renderCard(): void {

62+

function renderCard(payloadOverride?: string): void {

63+

const payload = payloadOverride ?? viewerPayload;

6364

document.body.insertAdjacentHTML(

6465

"beforeend",

6566

`<section class="oc-diff-card">

6667

<div data-openclaw-diff-host></div>

67-

<script type="application/json" data-openclaw-diff-payload>${viewerPayload}</script>

68+

<script type="application/json" data-openclaw-diff-payload>${payload}</script>

6869

</section>`,

6970

);

7071

}

@@ -172,3 +173,291 @@ describe("hydrateViewer", () => {

172173

warn.mockRestore();

173174

});

174175

});

176+177+

describe("viewerState initialization", () => {

178+

beforeEach(() => {

179+

document.body.innerHTML = "";

180+

delete document.documentElement.dataset.openclawDiffsError;

181+

delete document.documentElement.dataset.openclawDiffsReady;

182+

delete document.body.dataset.theme;

183+

vi.clearAllMocks();

184+

});

185+186+

it("seeds viewerState from firstPayload options and syncs document theme", async () => {

187+

const customPayload = JSON.stringify({

188+

prerenderedHTML: "<div>diff</div>",

189+

options: {

190+

theme: { light: "pierre-light", dark: "pierre-dark" },

191+

diffStyle: "split",

192+

diffIndicators: "bars",

193+

disableLineNumbers: false,

194+

expandUnchanged: false,

195+

themeType: "light",

196+

backgroundEnabled: false,

197+

overflow: "scroll",

198+

unsafeCSS: "",

199+

},

200+

langs: ["text"],

201+

oldFile: { name: "a.ts", lang: "text", contents: "old" },

202+

newFile: { name: "a.ts", lang: "text", contents: "new" },

203+

});

204+

renderCard(customPayload);

205+

const { hydrateViewer } = await import("./viewer-client.js");

206+207+

await hydrateViewer();

208+209+

expect(document.body.dataset.theme).toBe("light");

210+211+

const opts = fileDiffSetOptionsMock.mock.calls[0]?.[0] as Record<string, unknown>;

212+

expect(opts.diffStyle).toBe("split");

213+

expect(opts.themeType).toBe("light");

214+

expect(opts.overflow).toBe("scroll");

215+

expect(opts.disableBackground).toBe(true);

216+

});

217+218+

it("defaults viewerState to dark/unified/wrap/background when firstPayload uses defaults", async () => {

219+

renderCard();

220+

const { hydrateViewer } = await import("./viewer-client.js");

221+222+

await hydrateViewer();

223+224+

expect(document.body.dataset.theme).toBe("dark");

225+226+

const opts = fileDiffSetOptionsMock.mock.calls[0]?.[0] as Record<string, unknown>;

227+

expect(opts.diffStyle).toBe("unified");

228+

expect(opts.themeType).toBe("dark");

229+

expect(opts.overflow).toBe("wrap");

230+

expect(opts.disableBackground).toBe(false);

231+

});

232+233+

it("preloadHighlighter receives merged language set from all cards", async () => {

234+

const payload1 = JSON.stringify({

235+

prerenderedHTML: "<div>diff1</div>",

236+

options: {

237+

theme: { light: "pierre-light", dark: "pierre-dark" },

238+

diffStyle: "unified",

239+

diffIndicators: "bars",

240+

disableLineNumbers: false,

241+

expandUnchanged: false,

242+

themeType: "dark",

243+

backgroundEnabled: true,

244+

overflow: "wrap",

245+

unsafeCSS: "",

246+

},

247+

langs: ["typescript"],

248+

oldFile: { name: "a.ts", lang: "typescript", contents: "old" },

249+

newFile: { name: "a.ts", lang: "typescript", contents: "new" },

250+

});

251+

const payload2 = JSON.stringify({

252+

prerenderedHTML: "<div>diff2</div>",

253+

options: {

254+

theme: { light: "pierre-light", dark: "pierre-dark" },

255+

diffStyle: "unified",

256+

diffIndicators: "bars",

257+

disableLineNumbers: false,

258+

expandUnchanged: false,

259+

themeType: "dark",

260+

backgroundEnabled: true,

261+

overflow: "wrap",

262+

unsafeCSS: "",

263+

},

264+

langs: ["python"],

265+

oldFile: { name: "b.py", lang: "python", contents: "old" },

266+

newFile: { name: "b.py", lang: "python", contents: "new" },

267+

});

268+

renderCard(payload1);

269+

renderCard(payload2);

270+

const { hydrateViewer } = await import("./viewer-client.js");

271+272+

await hydrateViewer();

273+274+

const preloadArg = (preloadHighlighterMock.mock.calls as unknown[][])[0]?.[0] as { langs: string[]; themes: string[] } | undefined;

275+

expect(preloadArg).toBeDefined();

276+

expect(preloadArg!.langs).toContain("typescript");

277+

expect(preloadArg!.langs).toContain("python");

278+

expect(preloadArg!.themes).toEqual(["pierre-light", "pierre-dark"]);

279+

});

280+

});

281+282+

describe("toolbar button toggles", () => {

283+

beforeEach(() => {

284+

document.body.innerHTML = "";

285+

delete document.body.dataset.theme;

286+

vi.clearAllMocks();

287+

});

288+289+

it("layout toggle switches between unified and split", async () => {

290+

renderCard();

291+

const { hydrateViewer } = await import("./viewer-client.js");

292+

await hydrateViewer();

293+294+

const opts1 = fileDiffSetOptionsMock.mock.calls[0]?.[0] as Record<string, unknown>;

295+

expect(opts1.diffStyle).toBe("unified");

296+297+

const renderHeaderMetadata = opts1.renderHeaderMetadata as () => HTMLElement;

298+

const toolbar = renderHeaderMetadata();

299+

const buttons = toolbar.querySelectorAll("button");

300+301+

buttons[0].click();

302+303+

expect(fileDiffRerenderMock).toHaveBeenCalled();

304+305+

const opts2 = fileDiffSetOptionsMock.mock.calls[fileDiffSetOptionsMock.mock.calls.length - 1]?.[0] as Record<string, unknown>;

306+

expect(opts2.diffStyle).toBe("split");

307+

});

308+309+

it("theme toggle switches between dark and light", async () => {

310+

renderCard();

311+

const { hydrateViewer } = await import("./viewer-client.js");

312+

await hydrateViewer();

313+314+

const opts1 = fileDiffSetOptionsMock.mock.calls[0]?.[0] as Record<string, unknown>;

315+

expect(opts1.themeType).toBe("dark");

316+317+

const renderHeaderMetadata = opts1.renderHeaderMetadata as () => HTMLElement;

318+

const toolbar = renderHeaderMetadata();

319+

const buttons = toolbar.querySelectorAll("button");

320+321+

buttons[3].click();

322+323+

const lastOpts = fileDiffSetOptionsMock.mock.calls[fileDiffSetOptionsMock.mock.calls.length - 1]?.[0] as Record<string, unknown>;

324+

expect(lastOpts.themeType).toBe("light");

325+

expect(document.body.dataset.theme).toBe("light");

326+

});

327+328+

it("wrap toggle switches between wrap and scroll", async () => {

329+

renderCard();

330+

const { hydrateViewer } = await import("./viewer-client.js");

331+

await hydrateViewer();

332+333+

const opts1 = fileDiffSetOptionsMock.mock.calls[0]?.[0] as Record<string, unknown>;

334+

expect(opts1.overflow).toBe("wrap");

335+336+

const renderHeaderMetadata = opts1.renderHeaderMetadata as () => HTMLElement;

337+

const toolbar = renderHeaderMetadata();

338+

const buttons = toolbar.querySelectorAll("button");

339+340+

buttons[1].click();

341+342+

const lastOpts = fileDiffSetOptionsMock.mock.calls[fileDiffSetOptionsMock.mock.calls.length - 1]?.[0] as Record<string, unknown>;

343+

expect(lastOpts.overflow).toBe("scroll");

344+

});

345+346+

it("background toggle inverts disableBackground", async () => {

347+

renderCard();

348+

const { hydrateViewer } = await import("./viewer-client.js");

349+

await hydrateViewer();

350+351+

const opts1 = fileDiffSetOptionsMock.mock.calls[0]?.[0] as Record<string, unknown>;

352+

expect(opts1.disableBackground).toBe(false);

353+354+

const renderHeaderMetadata = opts1.renderHeaderMetadata as () => HTMLElement;

355+

const toolbar = renderHeaderMetadata();

356+

const buttons = toolbar.querySelectorAll("button");

357+358+

buttons[2].click();

359+360+

const lastOpts = fileDiffSetOptionsMock.mock.calls[fileDiffSetOptionsMock.mock.calls.length - 1]?.[0] as Record<string, unknown>;

361+

expect(lastOpts.disableBackground).toBe(true);

362+

});

363+

});

364+365+

describe("ensureShadowRoot", () => {

366+

beforeEach(() => {

367+

document.body.innerHTML = "";

368+

vi.clearAllMocks();

369+

});

370+371+

it("attaches shadow root from template and removes template element", async () => {

372+

renderCard();

373+

const host = document.querySelector<HTMLElement>("[data-openclaw-diff-host]")!;

374+

const template = document.createElement("template");

375+

template.setAttribute("shadowrootmode", "open");

376+

template.innerHTML = "<div>shadow content</div>";

377+

host.append(template);

378+379+

const { hydrateViewer } = await import("./viewer-client.js");

380+

await hydrateViewer();

381+382+

expect(host.shadowRoot).toBeDefined();

383+

expect(host.shadowRoot!.querySelector("div")?.textContent).toBe("shadow content");

384+

expect(host.querySelector("template")).toBeNull();

385+

});

386+387+

it("skips shadow root attachment when no template is present", async () => {

388+

renderCard();

389+

const host = document.querySelector<HTMLElement>("[data-openclaw-diff-host]")!;

390+391+

const { hydrateViewer } = await import("./viewer-client.js");

392+

await hydrateViewer();

393+394+

expect(host.shadowRoot).toBeNull();

395+

expect(fileDiffHydrateMock).toHaveBeenCalled();

396+

});

397+398+

it("skips shadow root when already attached", async () => {

399+

renderCard();

400+

const host = document.querySelector<HTMLElement>("[data-openclaw-diff-host]")!;

401+

host.attachShadow({ mode: "open" });

402+

host.shadowRoot!.innerHTML = "<span>existing</span>";

403+404+

const template = document.createElement("template");

405+

template.setAttribute("shadowrootmode", "open");

406+

template.innerHTML = "<div>new content</div>";

407+

host.append(template);

408+409+

const { hydrateViewer } = await import("./viewer-client.js");

410+

await hydrateViewer();

411+412+

expect(host.shadowRoot!.querySelector("span")?.textContent).toBe("existing");

413+

expect(host.querySelector("template")).not.toBeNull();

414+

});

415+

});

416+417+

describe("getHydrateProps branching", () => {

418+

beforeEach(() => {

419+

document.body.innerHTML = "";

420+

vi.clearAllMocks();

421+

});

422+423+

it("passes fileDiff directly when payload has fileDiff", async () => {

424+

const fileDiffPayload = JSON.stringify({

425+

prerenderedHTML: "<div>diff</div>",

426+

options: {

427+

theme: { light: "pierre-light", dark: "pierre-dark" },

428+

diffStyle: "unified",

429+

diffIndicators: "bars",

430+

disableLineNumbers: false,

431+

expandUnchanged: false,

432+

themeType: "dark",

433+

backgroundEnabled: true,

434+

overflow: "wrap",

435+

unsafeCSS: "",

436+

},

437+

langs: ["text"],

438+

fileDiff: { name: "patch.diff", lang: "text", hunks: [] },

439+

});

440+

renderCard(fileDiffPayload);

441+

const { hydrateViewer } = await import("./viewer-client.js");

442+443+

await hydrateViewer();

444+445+

const hydrateArg = fileDiffHydrateMock.mock.calls[0]?.[0] as Record<string, unknown>;

446+

expect(hydrateArg.fileDiff).toEqual({ name: "patch.diff", lang: "text", hunks: [] });

447+

expect(hydrateArg.oldFile).toBeUndefined();

448+

expect(hydrateArg.newFile).toBeUndefined();

449+

});

450+451+

it("passes oldFile and newFile when payload has them without fileDiff", async () => {

452+

renderCard();

453+

const { hydrateViewer } = await import("./viewer-client.js");

454+455+

await hydrateViewer();

456+457+

const hydrateArg = fileDiffHydrateMock.mock.calls[0]?.[0] as Record<string, unknown>;

458+

expect(hydrateArg.fileDiff).toBeUndefined();

459+

expect(hydrateArg.oldFile).toEqual({ name: "a.ts", lang: "text", contents: "old" });

460+

expect(hydrateArg.newFile).toEqual({ name: "a.ts", lang: "text", contents: "new" });

461+

});

462+463+

});