qunitx-cli 0.10.0 → 0.11.0

package/dist/cli.js

@@ -129,7 +129,13 @@ var init_early_chrome = __esm({
     openWatchMode = openFromArgv && watchFromArgv;
     earlyChromeProcRef = null;
     if (!openWatchMode) {
-      process.on("exit", () => earlyChromeProcRef?.kill());
+      process.on("exit", () => {
+        if (!earlyChromeProcRef) return;
+        try {
+          earlyChromeProcRef.kill("SIGKILL");
+        } catch {
+        }
+      });
     }
     perfLog("early-chrome.js: module evaluated");
     earlyBrowserPromise = isRunCommand && browserFromArgv === "chromium" ? findChrome().then((chromePath) => {
@@ -695,10 +701,34 @@ function setupWebServer(config, cachedContent) {
     });
   });
   server.get("/tests.js", (_req, res) => {
+    const bytes = cachedContent.allTestCode?.length ?? null;
+    console.log(
+      `# [HTTPServer] GET /tests.js \u2192 ${bytes !== null ? `${bytes} bytes` : "NOT READY (allTestCode is null)"}`
+    );
+    if (bytes === null) {
+      res.writeHead(503, { "Content-Type": "application/javascript", "Cache-Control": "no-store" });
+      res.end(
+        'console.error("[qunitx] /tests.js requested before bundle was built \u2014 allTestCode is null");'
+      );
+      return;
+    }
+    config._onTestsJsServed?.();
     res.writeHead(200, { "Content-Type": "application/javascript", "Cache-Control": "no-store" });
     res.end(cachedContent.allTestCode);
   });
   server.get("/filtered-tests.js", (_req, res) => {
+    const bytes = cachedContent.filteredTestCode?.length ?? null;
+    console.log(
+      `# [HTTPServer] GET /filtered-tests.js \u2192 ${bytes !== null ? `${bytes} bytes` : "NOT READY (filteredTestCode is null)"}`
+    );
+    if (bytes === null) {
+      res.writeHead(503, { "Content-Type": "application/javascript", "Cache-Control": "no-store" });
+      res.end(
+        'console.error("[qunitx] /filtered-tests.js requested before bundle was built \u2014 filteredTestCode is null");'
+      );
+      return;
+    }
+    config._onTestsJsServed?.();
     res.writeHead(200, { "Content-Type": "application/javascript", "Cache-Control": "no-store" });
     res.end(cachedContent.filteredTestCode);
   });
@@ -1002,7 +1032,9 @@ async function setupBrowser(config, cachedContent, existingBrowser = null) {
     window.IS_PLAYWRIGHT = true;
   });
   page.on("console", async (msg) => {
-    if (!config.debug) return;
+    const type = msg.type();
+    const alwaysShow = type === "warning" || type === "error";
+    if (!alwaysShow && !config.debug) return;
     try {
       const values = await Promise.all(msg.args().map((arg) => arg.jsonValue()));
       console.log(...values);
@@ -1121,23 +1153,31 @@ import esbuild from "esbuild";
 async function buildTestBundle(config, cachedContent) {
   const { projectRoot, output } = config;
   const allTestFilePaths = Object.keys(config.fsTree);
+  if (allTestFilePaths.length === 0) {
+    console.log("# [buildTestBundle] fsTree is empty \u2014 skipping build (no test files found)");
+    return;
+  }
+  const outfile = `${projectRoot}/${output}/tests.js`;
   await Promise.all([
-    esbuild.build({
-      stdin: {
-        contents: allTestFilePaths.map((f) => `import "${f}";`).join(""),
-        resolveDir: process.cwd()
+    buildWithOverlayfsRetry(
+      {
+        stdin: {
+          contents: allTestFilePaths.map((f) => `import "${f}";`).join(""),
+          resolveDir: process.cwd()
+        },
+        bundle: true,
+        logLevel: "error",
+        outfile,
+        keepNames: true,
+        sourcemap: config.debug ? "inline" : config.watch ? "linked" : false,
+        // Signal the runtime that all test modules are registered. The runtime's maybeStart()
+        // waits for both this event and the WebSocket 'open' event before calling QUnit.start().
+        // Dispatching from the bundle (rather than from a script onload attr) is reliable across
+        // all browsers and does not require changes to user test code.
+        footer: { js: 'window.dispatchEvent(new CustomEvent("qunitx:tests-ready"));' }
       },
-      bundle: true,
-      logLevel: "error",
-      outfile: `${projectRoot}/${output}/tests.js`,
-      keepNames: true,
-      sourcemap: config.debug ? "inline" : config.watch ? "linked" : false,
-      // Signal the runtime that all test modules are registered. The runtime's maybeStart()
-      // waits for both this event and the WebSocket 'open' event before calling QUnit.start().
-      // Dispatching from the bundle (rather than from a script onload attr) is reliable across
-      // all browsers and does not require changes to user test code.
-      footer: { js: 'window.dispatchEvent(new CustomEvent("qunitx:tests-ready"));' }
-    }),
+      outfile
+    ),
     Promise.all(
       cachedContent.htmlPathsToRunTests.map(async (htmlPath) => {
         const targetPath = `${config.projectRoot}/${config.output}${htmlPath}`;
@@ -1148,7 +1188,7 @@ async function buildTestBundle(config, cachedContent) {
       })
     )
   ]);
-  cachedContent.allTestCode = await fs8.readFile(`${projectRoot}/${output}/tests.js`);
+  cachedContent.allTestCode = await fs8.readFile(outfile);
 }
 async function runTestsInBrowser(config, cachedContent = {}, connections, targetTestFilesToFilter) {
   const { projectRoot, output } = config;
@@ -1162,6 +1202,9 @@ async function runTestsInBrowser(config, cachedContent = {}, connections, target
     if (!cachedContent.allTestCode) {
       await buildTestBundle(config, cachedContent);
     }
+    if (!cachedContent.allTestCode) {
+      return connections;
+    }
     if (runHasFilter) {
       const outputPath = `${projectRoot}/${output}/filtered-tests.js`;
       await buildFilteredTests(targetTestFilesToFilter, outputPath, config);
@@ -1204,17 +1247,42 @@ async function runTestsInBrowser(config, cachedContent = {}, connections, target
   return connections;
 }
 function buildFilteredTests(filteredTests, outputPath, config) {
-  return esbuild.build({
-    stdin: {
-      contents: filteredTests.map((f) => `import "${f}";`).join(""),
-      resolveDir: process.cwd()
+  return buildWithOverlayfsRetry(
+    {
+      stdin: {
+        contents: filteredTests.map((f) => `import "${f}";`).join(""),
+        resolveDir: process.cwd()
+      },
+      bundle: true,
+      logLevel: "error",
+      outfile: outputPath,
+      sourcemap: config.debug ? "inline" : config.watch ? "linked" : false,
+      footer: { js: 'window.dispatchEvent(new CustomEvent("qunitx:tests-ready"));' }
     },
-    bundle: true,
-    logLevel: "error",
-    outfile: outputPath,
-    sourcemap: config.debug ? "inline" : config.watch ? "linked" : false,
-    footer: { js: 'window.dispatchEvent(new CustomEvent("qunitx:tests-ready"));' }
-  });
+    outputPath
+  );
+}
+async function buildWithOverlayfsRetry(options, outfile) {
+  const RETRY_DELAY_MS = 100;
+  const MAX_RETRIES = 3;
+  const EMPTY_BUNDLE_THRESHOLD = 500;
+  let result = await esbuild.build(options);
+  for (let retry = 1; retry <= MAX_RETRIES; retry++) {
+    const bytes2 = (await fs8.stat(outfile)).size;
+    if (bytes2 >= EMPTY_BUNDLE_THRESHOLD) return result;
+    console.log(
+      `# [buildWithOverlayfsRetry] bundle is ${bytes2} bytes (< ${EMPTY_BUNDLE_THRESHOLD}) on attempt ${retry}/${MAX_RETRIES} \u2014 overlayfs flush race, retrying in ${RETRY_DELAY_MS}ms`
+    );
+    await new Promise((resolve) => setTimeout(resolve, RETRY_DELAY_MS));
+    result = await esbuild.build(options);
+  }
+  const bytes = (await fs8.stat(outfile)).size;
+  if (bytes < EMPTY_BUNDLE_THRESHOLD) {
+    console.log(
+      `# [buildWithOverlayfsRetry] bundle is ${bytes} bytes after ${MAX_RETRIES} retries \u2014 proceeding`
+    );
+  }
+  return result;
 }
 async function runTestInsideHTMLFile(filePath, { page, server, browser }, config) {
   let QUNIT_RESULT;
@@ -1233,6 +1301,10 @@ async function runTestInsideHTMLFile(filePath, { page, server, browser }, config
       clearTimeout(timeoutHandle);
       timeoutHandle = setTimeout(resolveTestRace, config.timeout * 3);
     };
+    config._onTestsJsServed = () => {
+      clearTimeout(timeoutHandle);
+      timeoutHandle = setTimeout(resolveTestRace, config.timeout * 4);
+    };
     config._resetTestTimeout = () => {
       wsConnected = true;
       clearTimeout(timeoutHandle);
@@ -1256,6 +1328,7 @@ async function runTestInsideHTMLFile(filePath, { page, server, browser }, config
   } finally {
     clearTimeout(timeoutHandle);
     config._onWsOpen = null;
+    config._onTestsJsServed = null;
     config._resetTestTimeout = null;
     config._testRunDone = null;
   }
@@ -1341,6 +1414,20 @@ function setupFileWatchers(testFileLookupPaths, config, onEventFunc, onFinishFun
           onFinishFunc
         );
       } catch {
+        await new Promise((resolve) => setTimeout(resolve, 50));
+        try {
+          const s = await stat(fullPath);
+          handleWatchEvent(
+            config,
+            extensions,
+            s.isDirectory() ? "addDir" : "add",
+            fullPath,
+            onEventFunc,
+            onFinishFunc
+          );
+          return;
+        } catch {
+        }
         if (!(config.fsTree && fullPath in config.fsTree)) return;
         handleWatchEvent(config, extensions, "unlink", fullPath, onEventFunc, onFinishFunc);
       }
@@ -1794,7 +1881,7 @@ init_color();
 var package_default = {
   name: "qunitx-cli",
   type: "module",
-  version: "0.10.0",
+  version: "0.11.0",
   description: "Browser runner for QUnitx: run your qunitx tests in google-chrome",
   author: "Izel Nakri",
   license: "MIT",
@@ -1826,6 +1913,7 @@ var package_default = {
     postinstall: "PLAYWRIGHT_SKIP_DOWNLOAD=true deno install --allow-scripts=npm:playwright-core || true",
     test: `node --experimental-strip-types test/setup.ts && FORCE_COLOR=0 node --experimental-strip-types --test --test-concurrency=$(node -p 'require("os").availableParallelism()') test/**/*-test.ts`,
     "test:browser": `node --experimental-strip-types test/setup.ts && FORCE_COLOR=0 node --experimental-strip-types --test --test-concurrency=$(node -p 'require("os").availableParallelism()') test/flags/*-test.ts test/inputs/*-test.ts`,
+    "test:release": "bash scripts/test-release.sh",
     "test:sanity-first": "./cli.ts test/helpers/failing-tests.js test/helpers/failing-tests.ts",
     "test:sanity-second": "./cli.ts test/helpers/passing-tests.js test/helpers/passing-tests.ts"
   },
@@ -1841,15 +1929,15 @@ var package_default = {
     url: "git+https://github.com/izelnakri/qunitx-cli.git"
   },
   dependencies: {
-    esbuild: "^0.27.3",
-    "playwright-core": "^1.58.2",
+    esbuild: "^0.28.0",
+    "playwright-core": "^1.59.1",
     ws: "^8.20.0"
   },
   devDependencies: {
     cors: "^2.8.6",
     express: "^5.2.1",
     "js-yaml": "^4.1.1",
-    prettier: "^3.8.1",
+    prettier: "^3.8.2",
     qunitx: "^1.2.1",
     typescript: "^6.0.2"
   },
@@ -2214,7 +2302,9 @@ async function setupConfig() {
     lastRanTestFiles: null,
     COUNTER: { testCount: 0, failCount: 0, skipCount: 0, passCount: 0, errorCount: 0 },
     _testRunDone: null,
-    _resetTestTimeout: null
+    _resetTestTimeout: null,
+    _onWsOpen: null,
+    _onTestsJsServed: null
   };
   config.htmlPaths = normalizeHTMLPaths(config.projectRoot, config.htmlPaths);
   config.fsTree = await buildFSTree(config.testFileLookupPaths, config);

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "qunitx-cli",
   "type": "module",
-  "version": "0.10.0",
+  "version": "0.11.0",
   "description": "Browser runner for QUnitx: run your qunitx tests in google-chrome",
   "author": "Izel Nakri",
   "license": "MIT",
@@ -33,6 +33,7 @@
     "postinstall": "PLAYWRIGHT_SKIP_DOWNLOAD=true deno install --allow-scripts=npm:playwright-core || true",
     "test": "node --experimental-strip-types test/setup.ts && FORCE_COLOR=0 node --experimental-strip-types --test --test-concurrency=$(node -p 'require(\"os\").availableParallelism()') test/**/*-test.ts",
     "test:browser": "node --experimental-strip-types test/setup.ts && FORCE_COLOR=0 node --experimental-strip-types --test --test-concurrency=$(node -p 'require(\"os\").availableParallelism()') test/flags/*-test.ts test/inputs/*-test.ts",
+    "test:release": "bash scripts/test-release.sh",
     "test:sanity-first": "./cli.ts test/helpers/failing-tests.js test/helpers/failing-tests.ts",
     "test:sanity-second": "./cli.ts test/helpers/passing-tests.js test/helpers/passing-tests.ts"
   },
@@ -48,15 +49,15 @@
     "url": "git+https://github.com/izelnakri/qunitx-cli.git"
   },
   "dependencies": {
-    "esbuild": "^0.27.3",
-    "playwright-core": "^1.58.2",
+    "esbuild": "^0.28.0",
+    "playwright-core": "^1.59.1",
     "ws": "^8.20.0"
   },
   "devDependencies": {
     "cors": "^2.8.6",
     "express": "^5.2.1",
     "js-yaml": "^4.1.1",
-    "prettier": "^3.8.1",
+    "prettier": "^3.8.2",
     "qunitx": "^1.2.1",
     "typescript": "^6.0.2"
   },