qunitx-cli 0.5.6 → 0.5.8

package/.dockerignore

@@ -0,0 +1,15 @@
+.claude
+.env
+.github
+.gitignore
+benches
+CHANGELOG.md
+cliff.toml
+docs
+flake.lock
+flake.nix
+Makefile
+scripts
+test
+tmp
+TODO

package/README.md

@@ -133,6 +133,34 @@ qunitx some-test.js --debug
 qunitx some-test.ts
 ```
 
+## Configuration
+
+All CLI flags can also be set in `package.json` under the `qunitx` key, so you don't have to repeat them on every invocation:
+
+```json
+{
+  "qunitx": {
+    "inputs": ["test/**/*-test.js", "test/**/*-test.ts"],
+    "extensions": ["js", "ts"],
+    "output": "tmp",
+    "timeout": 20000,
+    "failFast": false,
+    "port": 1234
+  }
+}
+```
+
+| Key | Default | Description |
+|-----|---------|-------------|
+| `inputs` | `[]` | Glob patterns, file paths, or directories to use as test entry points. Merged with any paths given on the CLI. |
+| `extensions` | `["js", "ts"]` | File extensions tracked for test discovery (directory scans) and watch-mode rebuild triggers. Add `"mjs"`, `"cjs"`, or any other extension your project uses. |
+| `output` | `"tmp"` | Directory where compiled test bundles are written. |
+| `timeout` | `20000` | Maximum milliseconds to wait for the full test suite before timing out. |
+| `failFast` | `false` | Stop the run after the first failing test. |
+| `port` | `1234` | Preferred HTTP server port. qunitx auto-selects a free port if this one is taken. |
+
+CLI flags always override `package.json` values when both are present.
+
 ## CLI Reference
 
 ```
@@ -144,6 +172,7 @@ Options:
   --debug             Print the server URL; pipe browser console to stdout
   --timeout=<ms>      Max ms to wait for the suite to finish  [default: 20000]
   --output=<dir>      Directory for compiled test assets     [default: ./tmp]
+  --extensions=<...>  Comma-separated file extensions to track  [default: js,ts]
   --before=<file>     Script to run (and optionally await) before tests start
   --after=<file>      Script to run (and optionally await) after tests finish
   --port=<n>          HTTP server port (auto-selects a free port if taken)

package/deno.json

@@ -11,6 +11,11 @@
     "recursive-lookup": "npm:recursive-lookup",
     "ws": "npm:ws"
   },
+  "tasks": {
+    "bench": "deno bench --allow-all benches/esbuild.bench.ts benches/server.bench.ts benches/tap.bench.ts benches/e2e.bench.ts",
+    "bench:check": "deno bench --allow-all --json benches/esbuild.bench.ts benches/server.bench.ts benches/tap.bench.ts benches/e2e.bench.ts | deno run --allow-all scripts/check-benchmarks.ts",
+    "bench:update": "deno bench --allow-all --json benches/esbuild.bench.ts benches/server.bench.ts benches/tap.bench.ts benches/e2e.bench.ts | deno run --allow-all scripts/check-benchmarks.ts --save"
+  },
   "lint": {
     "rules": {
       "exclude": ["no-process-global", "no-node-globals", "no-window"]

package/deno.lock

@@ -1,6 +1,7 @@
 {
   "version": "5",
   "specifiers": {
+    "jsr:@std/fmt@*": "1.0.9",
     "npm:@types/js-yaml@*": "4.0.9",
     "npm:@types/picomatch@*": "4.0.2",
     "npm:@types/ws@*": "8.18.1",
@@ -24,10 +25,14 @@
     "npm:qunit@^2.25.0": "2.25.0",
     "npm:qunitx@1": "1.0.0",
     "npm:recursive-lookup@*": "1.1.0",
-    "npm:recursive-lookup@1.1.0": "1.1.0",
     "npm:ws@*": "8.19.0",
     "npm:ws@^8.19.0": "8.19.0"
   },
+  "jsr": {
+    "@std/fmt@1.0.9": {
+      "integrity": "2487343e8899fb2be5d0e3d35013e54477ada198854e52dd05ed0422eddcabe0"
+    }
+  },
   "npm": {
     "@babel/code-frame@7.29.0": {
       "integrity": "sha512-9NhCeYjq9+3uxgdtp20LSiJXJvN0FeCtNGpJxuMFZ1Kv3cWUNb6DOhJwUvcVCzKGR66cw4njwM6hrJLqgOwbcw==",
@@ -1333,7 +1338,6 @@
         "npm:puppeteer@^24.38.0",
         "npm:qunit@^2.25.0",
         "npm:qunitx@1",
-        "npm:recursive-lookup@1.1.0",
         "npm:ws@^8.19.0"
       ]
     }

package/lib/commands/help.js

@@ -23,6 +23,7 @@ ${color('--timeout')} : change default timeout per test case
 ${color('--output')} : folder to distribute built qunitx html and js that a webservers can run[default: tmp]
 ${color('--failFast')} : run the target file or folders with immediate abort if a single test fails
 ${color('--port')} : HTTP server port (auto-selects a free port if the given port is taken)[default: 1234]
+${color('--extensions')} : comma-separated file extensions to track for discovery and watch-mode rebuilds[default: js,ts]
 ${color('--before')} : run a script before the tests(i.e start a new web server before tests)
 ${color('--after')} : run a script after the tests(i.e save test results to a file)
 

package/lib/commands/run/tests-in-browser.js

@@ -63,7 +63,7 @@ export default async function runTestsInBrowser(
 
   // In group mode the COUNTER is shared across all groups and managed by run.js.
   if (!config._groupMode) {
-    config.COUNTER = { testCount: 0, failCount: 0, skipCount: 0, passCount: 0 };
+    config.COUNTER = { testCount: 0, failCount: 0, skipCount: 0, passCount: 0, errorCount: 0 };
   }
   config.lastRanTestFiles = targetTestFilesToFilter || allTestFilePaths;
 

package/lib/commands/run.js

@@ -71,7 +71,7 @@ export default async function run(config) {
     const groups = splitIntoGroups(allFiles, groupCount);
 
     // Shared COUNTER so TAP test numbers are globally sequential across all groups.
-    config.COUNTER = { testCount: 0, failCount: 0, skipCount: 0, passCount: 0 };
+    config.COUNTER = { testCount: 0, failCount: 0, skipCount: 0, passCount: 0, errorCount: 0 };
     config.lastRanTestFiles = allFiles;
 
     const groupConfigs = groups.map((groupFiles, i) => ({

package/lib/setup/default-project-config-values.js

@@ -1,7 +1,8 @@
-/** Default qunitx config values: build output directory, test timeout (ms), fail-fast flag, and HTTP server port. */
+/** Default qunitx config values: build output directory, test timeout (ms), fail-fast flag, HTTP server port, and tracked file extensions. */
 export default {
   output: 'tmp',
   timeout: 20000,
   failFast: false,
   port: 1234,
+  extensions: ['js', 'ts'],
 };

package/lib/setup/file-watcher.js

@@ -6,52 +6,14 @@ import kleur from 'kleur';
  * @returns {object}
  */
 export default function setupFileWatchers(testFileLookupPaths, config, onEventFunc, onFinishFunc) {
-  const extensions = ['js', 'ts'];
+  const extensions = config.extensions || ['js', 'ts'];
   const fileWatchers = testFileLookupPaths.reduce((watcher, watchPath) => {
     return Object.assign(watcher, {
-      [watchPath]: chokidar.watch(watchPath, { ignoreInitial: true }).on('all', (event, path) => {
-        if (extensions.some((extension) => path.endsWith(extension))) {
-          mutateFSTree(config.fsTree, event, path);
-
-          console.log(
-            '#',
-            kleur
-              .magenta()
-              .bold('=================================================================='),
-          );
-          console.log('#', getEventColor(event), path.split(config.projectRoot)[1]);
-          console.log(
-            '#',
-            kleur
-              .magenta()
-              .bold('=================================================================='),
-          );
-
-          if (!global.chokidarBuild) {
-            global.chokidarBuild = true;
-
-            const result = extensions.some((extension) => path.endsWith(extension))
-              ? onEventFunc(event, path)
-              : null;
-
-            if (!(result instanceof Promise)) {
-              global.chokidarBuild = false;
-
-              return result;
-            }
-
-            result
-              .then(() => {
-                onFinishFunc ? onFinishFunc(event, path) : null;
-              })
-              .catch(() => {
-                // TODO: make an index.html to display the error
-                // error type has to be derived from the error!
-              })
-              .finally(() => (global.chokidarBuild = false));
-          }
-        }
-      }),
+      [watchPath]: chokidar
+        .watch(watchPath, { ignoreInitial: true })
+        .on('all', (event, filePath) =>
+          handleWatchEvent(config, extensions, event, filePath, onEventFunc, onFinishFunc),
+        ),
     });
   }, {});
 
@@ -65,15 +27,64 @@ export default function setupFileWatchers(testFileLookupPaths, config, onEventFu
   };
 }
 
-function mutateFSTree(fsTree, event, path) {
+/**
+ * Routes a chokidar event to fsTree mutation and optional rebuild trigger.
+ * `unlinkDir` bypasses the extension filter so deleted directories always clean up fsTree.
+ * @returns {void}
+ */
+export function handleWatchEvent(config, extensions, event, filePath, onEventFunc, onFinishFunc) {
+  const isFileEvent = extensions.some((ext) => filePath.endsWith(`.${ext}`));
+
+  if (!isFileEvent && event !== 'unlinkDir') return;
+
+  mutateFSTree(config.fsTree, event, filePath);
+
+  console.log(
+    '#',
+    kleur.magenta().bold('=================================================================='),
+  );
+  console.log('#', getEventColor(event), filePath.split(config.projectRoot)[1]);
+  console.log(
+    '#',
+    kleur.magenta().bold('=================================================================='),
+  );
+
+  if (!config._building) {
+    config._building = true;
+
+    const result = onEventFunc(event, filePath);
+
+    if (!(result instanceof Promise)) {
+      config._building = false;
+
+      return result;
+    }
+
+    result
+      .then(() => {
+        onFinishFunc ? onFinishFunc(event, filePath) : null;
+      })
+      .catch(() => {
+        // TODO: make an index.html to display the error
+        // error type has to be derived from the error!
+      })
+      .finally(() => (config._building = false));
+  }
+}
+
+/**
+ * Mutates `fsTree` in place based on a chokidar file-system event.
+ * @returns {void}
+ */
+export function mutateFSTree(fsTree, event, path) {
   if (event === 'add') {
     fsTree[path] = null;
   } else if (event === 'unlink') {
     delete fsTree[path];
   } else if (event === 'unlinkDir') {
-    const targetPaths = Object.keys(config.fsTree).filter((treePath) => treePath.startsWith(path));
-
-    targetPaths.forEach((path) => delete config.fsTree[path]);
+    for (const treePath of Object.keys(fsTree)) {
+      if (treePath.startsWith(path)) delete fsTree[treePath];
+    }
   }
 }
 

package/lib/setup/fs-tree.js

@@ -1,15 +1,21 @@
 import fs from 'node:fs/promises';
+import path from 'node:path';
 // @deno-types="npm:@types/picomatch"
 import picomatch from 'picomatch';
-// @deno-types="./recursive-lookup.d.ts"
-import recursiveLookup from 'recursive-lookup';
+
+async function readDirRecursive(dir, filter) {
+  const entries = await fs.readdir(dir, { recursive: true, withFileTypes: true });
+  return entries
+    .filter((e) => e.isFile() && filter(e.name))
+    .map((e) => path.join(e.parentPath, e.name));
+}
 
 /**
  * Resolves an array of file paths, directories, or glob patterns into a flat `{ absolutePath: null }` map.
  * @returns {Promise<object>}
  */
-export default async function buildFSTree(fileAbsolutePaths, _config = {}) {
-  const targetExtensions = ['js', 'ts'];
+export default async function buildFSTree(fileAbsolutePaths, config = {}) {
+  const targetExtensions = config.extensions || ['js', 'ts'];
   const fsTree = {};
 
   await Promise.all(
@@ -20,8 +26,8 @@ export default async function buildFSTree(fileAbsolutePaths, _config = {}) {
 
       try {
         if (glob.isGlob) {
-          const fileNames = await recursiveLookup(glob.base, (path) => {
-            return targetExtensions.some((extension) => path.endsWith(extension));
+          const fileNames = await readDirRecursive(glob.base, (name) => {
+            return targetExtensions.some((extension) => name.endsWith(`.${extension}`));
           });
 
           fileNames.forEach((fileName) => {
@@ -35,8 +41,8 @@ export default async function buildFSTree(fileAbsolutePaths, _config = {}) {
           if (entry.isFile()) {
             fsTree[fileAbsolutePath] = null;
           } else if (entry.isDirectory()) {
-            const fileNames = await recursiveLookup(glob.base, (path) => {
-              return targetExtensions.some((extension) => path.endsWith(extension));
+            const fileNames = await readDirRecursive(glob.base, (name) => {
+              return targetExtensions.some((extension) => name.endsWith(`.${extension}`));
             });
 
             fileNames.forEach((fileName) => {

package/lib/tap/display-test-result.js

@@ -26,7 +26,7 @@ export default function TAPDisplayTestResult(COUNTER, details) {
     );
     details.assertions.reduce((errorCount, assertion, index) => {
       if (!assertion.passed && assertion.todo === false) {
-        COUNTER.errorCount++;
+        COUNTER.errorCount = (COUNTER.errorCount ?? 0) + 1;
         const stack = assertion.stack?.match(/\(.+\)/g);
 
         console.log('  ---');

package/lib/utils/parse-cli-flags.js

@@ -13,7 +13,7 @@ export default function parseCliFlags(projectRoot) {
       } else if (arg.startsWith('--failfast') || arg.startsWith('--failFast')) {
         return Object.assign(result, { failFast: parseBoolean(arg.split('=')[1]) });
       } else if (arg.startsWith('--timeout')) {
-        return Object.assign(result, { timeout: arg.split('=')[1] || 10000 });
+        return Object.assign(result, { timeout: Number(arg.split('=')[1]) || 10000 });
       } else if (arg.startsWith('--output')) {
         return Object.assign(result, { output: arg.split('=')[1] });
       } else if (arg.endsWith('.html')) {
@@ -26,6 +26,13 @@ export default function parseCliFlags(projectRoot) {
         return result;
       } else if (arg.startsWith('--port')) {
         return Object.assign(result, { port: Number(arg.split('=')[1]) });
+      } else if (arg.startsWith('--extensions')) {
+        return Object.assign(result, {
+          extensions: arg
+            .split('=')[1]
+            .split(',')
+            .map((e) => e.trim()),
+        });
       } else if (arg.startsWith('--before')) {
         return Object.assign(result, { before: parseModule(arg.split('=')[1]) });
       } else if (arg.startsWith('--after')) {

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "qunitx-cli",
   "type": "module",
-  "version": "0.5.6",
+  "version": "0.5.8",
   "description": "Browser runner for QUnitx: run your qunitx tests in google-chrome",
   "main": "cli.js",
   "author": "Izel Nakri",
@@ -50,7 +50,6 @@
     "kleur": "^4.1.5",
     "picomatch": "^4.0.3",
     "puppeteer": "^24.38.0",
-    "recursive-lookup": "1.1.0",
     "ws": "^8.19.0"
   },
   "devDependencies": {

package/lib/setup/recursive-lookup.d.ts

@@ -1,7 +0,0 @@
-/** Recursively lists files under `folderPath`, optionally filtered by a predicate. */
-declare function recursiveLookup(
-  folderPath: string,
-  filter?: (name: string) => boolean,
-): Promise<string[]>;
-
-export default recursiveLookup;

package/scripts/lint-docs.js

@@ -1,40 +0,0 @@
-#!/usr/bin/env node
-// TODO: REMOVE THIS FILE once deno fixes the missing-return-type regression.
-//
-// WHY THIS FILE EXISTS:
-// `deno doc --lint` has a regression in deno 2.7.x where JSDoc `@returns` tags
-// are silently ignored for the `missing-return-type` check in JavaScript files.
-// The check requires TypeScript-style return type annotations (`: ReturnType`)
-// which are not valid syntax in `.js` files.
-//
-// All 22 `missing-return-type` errors are false positives caused by this bug.
-// Every function has a correct `@returns` JSDoc tag — deno just doesn't read it.
-//
-// FIX OPTIONS (when ready):
-//   1. Wait for deno to fix the regression (check deno changelog).
-//   2. Convert lib/*.js files to lib/*.ts and add TS return type annotations.
-//
-// This script runs `deno doc --lint` and fails only on `missing-jsdoc` errors
-// (i.e., the real quality check: "is every exported symbol documented?").
-import { spawn } from 'node:child_process';
-
-const proc = spawn('deno', ['doc', '--lint', 'lib/', 'cli.js'], { encoding: 'utf8' });
-let output = '';
-proc.stdout.on('data', (chunk) => (output += chunk));
-proc.stderr.on('data', (chunk) => (output += chunk));
-proc.on('close', () => {
-  // Strip ANSI escape codes so we can match on plain text
-  const plain = output.replace(/\x1b\[[0-9;]*m/g, '');
-
-  // Split into per-error blocks (each block starts with "error[")
-  const blocks = plain.split(/(?=^error\[)/m);
-  const relevant = blocks.filter((b) => !b.startsWith('error[missing-return-type]'));
-  const result = relevant.join('').trim();
-
-  if (result.includes('error[')) {
-    process.stderr.write(result + '\n');
-    process.exit(1);
-  } else if (result) {
-    process.stdout.write(result + '\n');
-  }
-});