From 5c3ca6e6158d97707feb44b379dd1840617623c9 Mon Sep 17 00:00:00 2001 From: Adam Moussa Date: Tue, 30 Jun 2026 16:20:41 -0400 Subject: [PATCH] Revert upstream pnpm switch; keep bun for the UI build The merge auto-adopted upstream's pnpm switch (#1645) in tests/e2e/ global-setup.ts and ui/package.json, but our fork builds the UI with bun (vercel.json + the E2E workflow's setup-bun). That left the Playwright globalSetup running `corepack pnpm install --frozen-lockfile` with no pnpm-lock.yaml, failing E2E at UI build time. Revert global-setup.ts and ui/package.json to the dev (bun) baseline, drop the merge-added ui/pnpm-lock.yaml, and remove the re-added ui/AGENTS.md (our fork had deleted it). --- tests/e2e/global-setup.ts | 4 +- ui/AGENTS.md | 12 - ui/README.md | 2 +- ui/package.json | 4 +- ui/pnpm-lock.yaml | 11946 ------------------------------------ 5 files changed, 4 insertions(+), 11964 deletions(-) delete mode 100644 ui/AGENTS.md delete mode 100644 ui/pnpm-lock.yaml diff --git a/tests/e2e/global-setup.ts b/tests/e2e/global-setup.ts index 17485f1f..bd4ed3d8 100644 --- a/tests/e2e/global-setup.ts +++ b/tests/e2e/global-setup.ts @@ -14,9 +14,9 @@ export default function globalSetup() { if (existsSync(shell) && !process.env.E2E_FORCE_UI_BUILD) return; if (!existsSync(resolve(ui, "node_modules"))) { - execSync("corepack pnpm install --frozen-lockfile", { cwd: ui, stdio: "inherit" }); + execSync("bun install", { cwd: ui, stdio: "inherit" }); } - execSync("corepack pnpm run build", { + execSync("bun run build", { cwd: ui, stdio: "inherit", env: { ...process.env, VITE_DASHBOARD_API_BASE_URL: `http://127.0.0.1:${port}` }, diff --git a/ui/AGENTS.md b/ui/AGENTS.md deleted file mode 100644 index 1903fa68..00000000 --- a/ui/AGENTS.md +++ /dev/null @@ -1,12 +0,0 @@ -# AGENTS.md - -This file applies to all work under `ui/`. - -## Package manager - -- Use **pnpm** for dashboard dependency management and script execution. -- Run UI scripts with `pnpm run