From 91693aba84e04a95b73d315ae747105944b35d81 Mon Sep 17 00:00:00 2001 From: Mohamed Boudra Date: Sat, 2 May 2026 13:51:36 +0700 Subject: [PATCH] fix(desktop): green up desktop CI on main Two breakages surfaced when CI started running desktop tests: - Root vitest.config.ts replaced default exclude with `["**/.claude/**"]`, losing `**/node_modules/**`. Desktop has no local vitest config, so vitest scanned into `node_modules/zod/src/v4/classic/tests/*.test.ts` and failed on missing peers (`@web-std/file`, `recheck`). Spread `configDefaults.exclude` so node_modules stays excluded. - window-manager dark titlebar color changed from #18181c to #181B1A in 963c7926 (white-flash fix) but the test was not updated. Update the expected color to match the new intended behavior. --- .../desktop/src/window/window-manager.test.ts | 16 ++++++++-------- vitest.config.ts | 4 ++-- 2 files changed, 10 insertions(+), 10 deletions(-) diff --git a/packages/desktop/src/window/window-manager.test.ts b/packages/desktop/src/window/window-manager.test.ts index 269911abc..f095d6789 100644 --- a/packages/desktop/src/window/window-manager.test.ts +++ b/packages/desktop/src/window/window-manager.test.ts @@ -54,7 +54,7 @@ describe("window-manager", () => { it("returns dark title bar overlay colors", () => { expect(getTitleBarOverlayOptions("dark")).toEqual({ - color: "#18181c", + color: "#181B1A", symbolColor: "#e4e4e7", height: 29, }); @@ -66,11 +66,11 @@ describe("window-manager", () => { expect( readWindowControlsOverlayUpdate({ height: 48, - backgroundColor: "#18181c", + backgroundColor: "#181B1A", }), ).toEqual({ height: 48, - backgroundColor: "#18181c", + backgroundColor: "#181B1A", }); }); @@ -107,7 +107,7 @@ describe("window-manager", () => { win: { setTitleBarOverlay }, current: state, update: { - backgroundColor: "#18181c", + backgroundColor: "#181B1A", foregroundColor: "#e4e4e7", }, }); @@ -120,16 +120,16 @@ describe("window-manager", () => { expect(state).toEqual({ height: 48, - backgroundColor: "#18181c", + backgroundColor: "#181B1A", foregroundColor: "#e4e4e7", }); expect(setTitleBarOverlay).toHaveBeenNthCalledWith(1, { - color: "#18181c", + color: "#181B1A", symbolColor: "#e4e4e7", height: 28, }); expect(setTitleBarOverlay).toHaveBeenNthCalledWith(2, { - color: "#18181c", + color: "#181B1A", symbolColor: "#e4e4e7", height: 47, }); @@ -148,7 +148,7 @@ describe("window-manager", () => { frame: false, autoHideMenuBar: true, titleBarOverlay: { - color: "#18181c", + color: "#181B1A", symbolColor: "#e4e4e7", height: 29, }, diff --git a/vitest.config.ts b/vitest.config.ts index a7f329add..df0655a00 100644 --- a/vitest.config.ts +++ b/vitest.config.ts @@ -1,6 +1,6 @@ import fs from "node:fs"; import path from "node:path"; -import { defineConfig } from "vitest/config"; +import { configDefaults, defineConfig } from "vitest/config"; const appDir = path.resolve(__dirname, "packages/app"); const appNodeModules = path.resolve(appDir, "node_modules"); @@ -57,6 +57,6 @@ export default defineConfig({ ], }, test: { - exclude: ["**/.claude/**"], + exclude: [...configDefaults.exclude, "**/.claude/**"], }, });