test(cli): log windows warm launch and cli failures

Author: miguel-heygen

.github/workflows/checkfast-measure.yml

@@ -46,7 +46,7 @@ jobs:
         run: bun run build
       - name: Measure cold vs warm, then the warm process tests
         shell: bash
-        run: cd packages/cli && npx vitest run src/warmCheck.process.test.ts --maxWorkers=1 -t "prints what a cold" 2>&1 | tail -150 || true; echo ===DBG; cat warm-dbg.log
+        run: cd packages/cli && npx vitest run src/warmCheck.process.test.ts --maxWorkers=1 2>&1 | tail -150 || true; echo ===DBG; cat warm-dbg.log
       - uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 # v4
         if: always()
         with:

packages/cli/src/capture/captureCompositionFrame.ts

@@ -146,7 +146,7 @@ export function prelaunchCompositionBrowser(
   const abort = new AbortController();
   const options = { browserGpuMode, signal: abort.signal };
   const key = launchKey(html, false, options);
-  const launched = launchCompositionBrowser(html, false, options).catch(() => undefined);
+  const launched = launchCompositionBrowser(html, false, options).catch((e) => { if (process.env.HF_WARM_DBG) process.getBuiltinModule("node:fs").appendFileSync(process.env.HF_WARM_DBG, `${Date.now()} ${process.pid} launch error ${e?.stack ?? e}\n`); return undefined; });
   let taken = false;
   const discard = async () => {
     if (taken) return;

packages/cli/src/cli.ts

@@ -460,6 +460,7 @@ function exitAfterCliFailure(
   error: Error,
 ): never {
   commandFailed = true;
+  if (process.env.HF_WARM_DBG) process.getBuiltinModule("node:fs").appendFileSync(process.env.HF_WARM_DBG, `${Date.now()} ${process.pid} cli failure ${kind} ${error.stack}\n`);
   emitCliErrorEvent(kind, error);
   _flushSync?.();
   process.exit(1);

packages/cli/src/warmCheck.ts

@@ -140,4 +140,4 @@ server.on("error", (error: NodeJS.ErrnoException) => {
 });
 
 listen();
-void prelaunchBrowserForCheck(checkArgs).then(() => __dbg('prelaunched'), (e) => __dbg(`prelaunch failed ${e?.stack ?? e}`));
+void prelaunchBrowserForCheck(checkArgs).then(async (h) => { __dbg('prelaunch handle ' + Boolean(h)); const r = await h?.launched; __dbg('launched ' + Boolean(r) + ' connected=' + r?.browser.connected); }, (e) => __dbg(`prelaunch failed ${e?.stack ?? e}`));