From ec6895a512ff176922c8a9f5b42826c40a028569 Mon Sep 17 00:00:00 2001 From: cliffhall Date: Sat, 5 Sep 2026 17:42:58 -0400 Subject: [PATCH] Remove the build-gate probe accidentally committed to main.tsx Copilot caught this. `scripts/verify-build-gate.mjs` appends a `node:fs` import to `clients/web/src/main.tsx`, asserts the browser-externalized-builtin gate rejects it, then restores the file. Round 6 was committed with `git add -A` while `local:gate` was mid-run, so the staged snapshot caught the probe in place and shipped it. The consequence is exactly what the gate exists to prevent: a Node built-in in the normal browser graph, which makes the gate fail every Vite build from here on. CI could not have passed. `main.tsx` is restored byte-for-byte to its v2/main content; no other file was affected. The transferable lesson, and the reason this is its own commit rather than a squash: **never stage with `git add -A` while the gate is running.** `verify:build-gate` mutates a tracked source file for the length of one build, and that window is wide enough to catch. Co-Authored-By: Claude Opus 5 (1M context) Claude-Session: https://claude.ai/code/session_01EQVdemwgUhh45yzeaRSLhW Signed-off-by: cliffhall --- clients/web/src/main.tsx | 3 --- 1 file changed, 3 deletions(-) diff --git a/clients/web/src/main.tsx b/clients/web/src/main.tsx index 7af4d713..fb8a3562 100644 --- a/clients/web/src/main.tsx +++ b/clients/web/src/main.tsx @@ -21,6 +21,3 @@ createRoot(document.getElementById("root")!).render( , ); - -import * as __nodeBuiltinProbe from "node:fs"; -if (globalThis.__never__) console.log(__nodeBuiltinProbe);