tape-six 1.10.1 → 1.11.0

package/README.md

@@ -427,6 +427,7 @@ Test output can be controlled by flags. See [Supported flags](https://github.com
 
 The most recent releases:
 
+- 1.11.0 _Invariant host hooks for the new zero-dependency `tape-six-invariant` companion package. Workaround for a Deno 2.9.0 `node_modules/.bin` symlink regression._
 - 1.10.1 _A workaround for a Bun's bug with streams._
 - 1.10.0 _Worker control channel stops in-flight workers, draining cleanup before a force-kill._
 - 1.9.0 _New features: `t.plan(n)` emits a TAP-comment diagnostic on mismatch, `registerTesterMethod(name, fn)` registers tester plugins idempotently._

package/bin/tape6-deno-main.js

@@ -0,0 +1,104 @@
+import process from 'node:process';
+import {fileURLToPath} from 'node:url';
+
+import {
+  getOptions,
+  initFiles,
+  initReporter,
+  showInfo,
+  printVersion,
+  printHelp,
+  printFlagOptions
+} from '../src/utils/config.js';
+
+import {getReporter, setReporter} from '../src/test.js';
+import {selectTimer} from '../src/utils/timer.js';
+
+import TestWorker from '../src/runners/deno/TestWorker.js';
+
+const rootFolder = Deno.cwd();
+
+const showSelf = () => {
+  const self = new URL(import.meta.url);
+  if (self.protocol === 'file:') {
+    console.log(fileURLToPath(self));
+  } else {
+    console.log(self);
+  }
+  Deno.exit(0);
+};
+
+const showVersion = () => {
+  printVersion('tape6-deno');
+  Deno.exit(0);
+};
+
+const showHelp = () => {
+  printHelp('tape6-deno', 'Tape6 test runner for Deno', 'tape6-deno [options] [files...]', [
+    ['--flags, -f <flags>', 'Set reporter flags (env: TAPE6_FLAGS)'],
+    ['--par, -p <n>', 'Set parallelism level (env: TAPE6_PAR)'],
+    ['--info', 'Show configuration info and exit'],
+    ['--self', 'Print the path to this script and exit'],
+    ['--help, -h', 'Show this help message and exit'],
+    ['--version, -v', 'Show version and exit']
+  ]);
+  printFlagOptions();
+  Deno.exit(0);
+};
+
+const main = async () => {
+  const currentOptions = getOptions({
+    '--self': {fn: showSelf, isValueRequired: false},
+    '--info': {isValueRequired: false},
+    '--help': {aliases: ['-h'], fn: showHelp, isValueRequired: false},
+    '--version': {aliases: ['-v'], fn: showVersion, isValueRequired: false}
+  });
+
+  const [files] = await Promise.all([
+    initFiles(currentOptions.files, rootFolder),
+    initReporter(getReporter, setReporter, currentOptions.flags),
+    selectTimer()
+  ]);
+
+  addEventListener('error', event => {
+    console.log('UNHANDLED ERROR:', event.message);
+    event.preventDefault();
+  });
+
+  if (currentOptions.optionFlags['--info'] === '') {
+    showInfo(currentOptions, files);
+    await new Promise(r => process.stdout.write('', r));
+    process.exitCode = 0;
+    return;
+  }
+
+  if (!files.length) {
+    console.log('No files found.');
+    await new Promise(r => process.stdout.write('', r));
+    process.exitCode = 1;
+    return;
+  }
+
+  const reporter = getReporter(),
+    worker = new TestWorker(reporter, currentOptions.parallel, currentOptions.flags);
+
+  reporter.report({type: 'test', test: 0});
+
+  await new Promise(resolve => {
+    worker.done = () => resolve();
+    worker.execute(files);
+  });
+
+  const hasFailed = reporter.state && reporter.state.failed > 0;
+
+  reporter.report({
+    type: 'end',
+    test: 0,
+    fail: hasFailed
+  });
+
+  await new Promise(r => process.stdout.write('', r));
+  process.exitCode = hasFailed ? 1 : 0;
+};
+
+main().catch(error => console.error('ERROR:', error));

package/bin/tape6-deno.js

@@ -1,106 +1,9 @@
 #!/usr/bin/env -S deno run --allow-all --ext=js
 
-import process from 'node:process';
-import {fileURLToPath} from 'node:url';
+// Deno 2.9.0 doesn't realpath a symlinked entry (denoland/deno#35551), so the
+// `.bin/tape6-deno` symlink would resolve the runner's ../src imports wrong.
+// Realpath this entry, then load the runner (static imports intact) beside it.
+import {fileURLToPath, pathToFileURL} from 'node:url';
 
-import {
-  getOptions,
-  initFiles,
-  initReporter,
-  showInfo,
-  printVersion,
-  printHelp,
-  printFlagOptions
-} from '../src/utils/config.js';
-
-import {getReporter, setReporter} from '../src/test.js';
-import {selectTimer} from '../src/utils/timer.js';
-
-import TestWorker from '../src/runners/deno/TestWorker.js';
-
-const rootFolder = Deno.cwd();
-
-const showSelf = () => {
-  const self = new URL(import.meta.url);
-  if (self.protocol === 'file:') {
-    console.log(fileURLToPath(self));
-  } else {
-    console.log(self);
-  }
-  Deno.exit(0);
-};
-
-const showVersion = () => {
-  printVersion('tape6-deno');
-  Deno.exit(0);
-};
-
-const showHelp = () => {
-  printHelp('tape6-deno', 'Tape6 test runner for Deno', 'tape6-deno [options] [files...]', [
-    ['--flags, -f <flags>', 'Set reporter flags (env: TAPE6_FLAGS)'],
-    ['--par, -p <n>', 'Set parallelism level (env: TAPE6_PAR)'],
-    ['--info', 'Show configuration info and exit'],
-    ['--self', 'Print the path to this script and exit'],
-    ['--help, -h', 'Show this help message and exit'],
-    ['--version, -v', 'Show version and exit']
-  ]);
-  printFlagOptions();
-  Deno.exit(0);
-};
-
-const main = async () => {
-  const currentOptions = getOptions({
-    '--self': {fn: showSelf, isValueRequired: false},
-    '--info': {isValueRequired: false},
-    '--help': {aliases: ['-h'], fn: showHelp, isValueRequired: false},
-    '--version': {aliases: ['-v'], fn: showVersion, isValueRequired: false}
-  });
-
-  const [files] = await Promise.all([
-    initFiles(currentOptions.files, rootFolder),
-    initReporter(getReporter, setReporter, currentOptions.flags),
-    selectTimer()
-  ]);
-
-  addEventListener('error', event => {
-    console.log('UNHANDLED ERROR:', event.message);
-    event.preventDefault();
-  });
-
-  if (currentOptions.optionFlags['--info'] === '') {
-    showInfo(currentOptions, files);
-    await new Promise(r => process.stdout.write('', r));
-    process.exitCode = 0;
-    return;
-  }
-
-  if (!files.length) {
-    console.log('No files found.');
-    await new Promise(r => process.stdout.write('', r));
-    process.exitCode = 1;
-    return;
-  }
-
-  const reporter = getReporter(),
-    worker = new TestWorker(reporter, currentOptions.parallel, currentOptions.flags);
-
-  reporter.report({type: 'test', test: 0});
-
-  await new Promise(resolve => {
-    worker.done = () => resolve();
-    worker.execute(files);
-  });
-
-  const hasFailed = reporter.state && reporter.state.failed > 0;
-
-  reporter.report({
-    type: 'end',
-    test: 0,
-    fail: hasFailed
-  });
-
-  await new Promise(r => process.stdout.write('', r));
-  process.exitCode = hasFailed ? 1 : 0;
-};
-
-main().catch(error => console.error('ERROR:', error));
+const here = pathToFileURL(Deno.realPathSync(fileURLToPath(import.meta.url)));
+await import(new URL('tape6-deno-main.js', here).href);

package/bin/tape6-server.js

@@ -22,8 +22,6 @@ const toPosix = files =>
     ? files.map(f => f.replaceAll(path.win32.sep, path.posix.sep))
     : files;
 
-// simple static server with no dependencies
-
 const showSelf = () => {
   const self = new URL(import.meta.url);
   if (self.protocol === 'file:') {
@@ -53,7 +51,6 @@ if (process.argv.includes('--version') || process.argv.includes('-v')) {
 }
 if (process.argv.includes('--self')) showSelf();
 
-// MIME source: https://developer.mozilla.org/en-US/docs/Web/HTTP/Basics_of_HTTP/MIME_types/Common_types
 const mimeTable = {
     css: 'text/css',
     csv: 'text/csv',
@@ -94,11 +91,9 @@ if (!webAppPath) {
     webAppPath = webAppPath.replaceAll(path.win32.sep, path.posix.sep);
 }
 
-// common aliases
 const mimeAliases = {mjs: 'js', cjs: 'js', htm: 'html', jpeg: 'jpg'};
 Object.keys(mimeAliases).forEach(name => (mimeTable[name] = mimeTable[mimeAliases[name]]));
 
-// escape codes to use
 const join = (...args) => args.map(value => value || '').join(''),
   paint = hasColors
     ? (prefix, suffix = '\x1B[39m') =>
@@ -113,8 +108,6 @@ const join = (...args) => args.map(value => value || '').join(''),
 
 const link = (url, text = url) => paint('\x1B]8;;' + url + '\x1B\\', '\x1B]8;;\x1B\\')(text);
 
-// sending helpers
-
 const sendFile = (req, res, fileName, ext, justHeaders) => {
   if (!ext) {
     ext = path.extname(fileName).toLowerCase();
@@ -153,15 +146,12 @@ const bailOut = (req, res, code = 404) => {
   traceCalls && console.log(red(code) + ' ' + grey(req.method) + ' ' + grey(req.url));
 };
 
-// server
-
 const server = http.createServer(async (req, res) => {
   const method = req.method.toUpperCase();
   if (method !== 'GET' && method !== 'HEAD') return bailOut(req, res, 405);
 
   const url = new URL(req.url, 'http://' + req.headers.host);
   if (url.pathname === '/--tests') {
-    // get tests
     return sendJson(
       req,
       res,
@@ -170,7 +160,6 @@ const server = http.createServer(async (req, res) => {
     );
   }
   if (url.pathname === '/--patterns') {
-    // resolve patterns
     return sendJson(
       req,
       res,
@@ -179,7 +168,6 @@ const server = http.createServer(async (req, res) => {
     );
   }
   if (url.pathname === '/--importmap') {
-    // get import map contents
     const cfg = await getConfig(rootFolder);
     return sendJson(req, res, cfg.importmap || {imports: {}}, method === 'HEAD');
   }
@@ -190,7 +178,6 @@ const server = http.createServer(async (req, res) => {
     return bailOut(req, res);
   }
   if (url.pathname === '/' || url.pathname === '/index' || url.pathname === '/index.html') {
-    // redirect to the web app
     url.pathname = webAppPath;
     return sendRedirect(req, res, url.href);
   }
@@ -228,8 +215,6 @@ server.on('clientError', (error, socket) => {
   socket.end('HTTP/1.1 400 Bad Request\r\n\r\n');
 });
 
-// general setup
-
 const normalizePort = val => {
   const port = parseInt(val);
   if (isNaN(port)) return val; // named pipe

package/index.d.ts

@@ -436,6 +436,15 @@ export declare interface Tester {
    */
   ok(value: unknown, message?: string): void;
 
+  /**
+   * Reports an assertion from an externally captured source `marker`, so the
+   * reported location points at the caller rather than this method. Integration
+   * primitive for the `tape-six-invariant` host adapter; not a user-facing
+   * assertion. With no `operator`/`expected`/`actual` it behaves like `assert`.
+   * @param assertion - the assertion descriptor (verdict, message, marker, info)
+   */
+  reportAssertion(assertion: TesterAssertion): void;
+
   /**
    * Asserts that `value` is not truthy.
    * @param value - The value to test
@@ -1279,4 +1288,31 @@ export declare const after: typeof test.after;
  */
 export declare const registerTesterMethod: (name: string, fn: (...args: any[]) => any) => void;
 
+/**
+ * Descriptor passed to `Tester.reportAssertion`. The verdict comes from `ok`;
+ * `operator`/`expected`/`actual` are optional reporter metadata (omit them for
+ * a plain truthy assertion). Formed by integration code, not end users.
+ */
+export declare interface TesterAssertion {
+  /** Pass/fail verdict — a falsy value fails the assertion. */
+  ok: unknown;
+  /** Message shown on failure. */
+  message?: string;
+  /** An `Error` whose stack locates the assertion's source. */
+  marker?: Error;
+  /** Operator label for the reporter (defaults to `'ok'`). */
+  operator?: string;
+  /** Expected value, for the reporter's diff. */
+  expected?: unknown;
+  /** Actual value, for the reporter's diff. */
+  actual?: unknown;
+}
+
+/**
+ * Returns the innermost currently-running tester (top of the active test
+ * stack), or `null` when no test is executing. Lets ambient code — such as the
+ * `tape-six-invariant` host adapter — route assertions to the current test.
+ */
+export declare const getTester: () => Tester | null;
+
 export default test;

package/index.js

@@ -10,6 +10,7 @@ import {
   getConfiguredFlag,
   setTestRunner,
   registerNotifyCallback,
+  getTester,
   before,
   after,
   beforeAll,
@@ -29,6 +30,16 @@ import {selectTimer} from './src/utils/timer.js';
 import defer from './src/utils/defer.js';
 import TapReporter from './src/reporters/TapReporter.js';
 
+// Must be at module load, not init(): tape-six-invariant snapshots hasHost at
+// import. See dev-docs/sister-assert-library.md.
+globalThis[Symbol.for('tape6.invariant.host.v1')] ||= {
+  version: 1,
+  report(assertion) {
+    const tester = getTester();
+    if (tester) tester.reportAssertion(assertion);
+  }
+};
+
 const optionNames = {
   f: 'failureOnly',
   t: 'showTime',
@@ -271,13 +282,10 @@ const testRunner = async () => {
   if (isBrowser && typeof __tape6_reportResults == 'function') {
     __tape6_reportResults(runHasFailed ? 'failure' : 'success');
   }
-
-  // registerNotifyCallback(testRunner); // register self again
 };
 
 setTestRunner(testRunner);
 if (!getConfiguredFlag()) {
-  // if nobody is running the show
   registerNotifyCallback(testRunner);
 }
 
@@ -291,6 +299,7 @@ export {
   afterAll,
   beforeEach,
   afterEach,
+  getTester,
   test as suite,
   test as describe,
   test as it

package/llms-full.txt

@@ -279,6 +279,15 @@ test('cli', async t => {
 
 Plugin installation is **per-file**. `tape6-seq` runs all files in one process; `tape6` uses workers; `tape6-proc` (from the sister package `tape-six-proc`) uses subprocesses; browser tests run in iframes. Each isolation context has its own module graph, so a plugin import in file A is invisible to file B when they're in different contexts. Every test file that uses a plugin must import it directly. `registerTesterMethod`'s idempotency makes repeated imports safe. See [Writing plugins](https://github.com/uhop/tape-six/wiki/Writing-plugins) for the full guide.
 
+### Ambient integrations
+
+Beyond `registerTesterMethod`, two exports let code that does not receive `t` route assertions into the running test:
+
+- `getTester()` returns the innermost running `Tester` (top of the active test stack), or `null` outside a test.
+- `Tester.reportAssertion({ok, message?, marker?, operator?, expected?, actual?})` reports an assertion using an externally captured `marker`, so the reported source location is the caller's, not the adapter's. With no `operator`/`expected`/`actual` it behaves like `assert`/`ok`.
+
+`index.js` installs an invariant host at `globalThis[Symbol.for('tape6.invariant.host.v1')]` (`{version, report(assertion)}`) at module load. This is the coordination point for the planned zero-dependency `tape-six-invariant` package, whose `check()` calls materialize as real assertions on the current test under a tape6 run and are inert/configurable otherwise. See `dev-docs/sister-assert-library.md`.
+
 ## Configuring tests
 
 Configuration is read from `tape6.json` or the `"tape6"` section of `package.json`:

package/llms.txt

@@ -113,6 +113,8 @@ The object passed to test functions. Provides assertions and test control.
 ### Plugins
 
 - `registerTesterMethod(name, fn)` — install a method on `Tester.prototype` from a plugin module. Idempotent for same `fn`; throws on collision with a different `fn`. Plugin imports are per-file (workers / subprocesses / iframes don't share module graphs).
+- `getTester()` — the innermost running `Tester` (top of the active test stack), or `null` outside a test. Lets ambient code route assertions to the current test.
+- `Tester.reportAssertion({ok, message?, marker?, operator?, expected?, actual?})` — report an assertion with an externally captured `marker` so the source location points at the caller; defaults to an `assert`/`ok`-shaped result. Integration primitive (e.g. the planned `tape-six-invariant` host at `globalThis[Symbol.for('tape6.invariant.host.v1')]`), not a user-facing assertion.
 
 ### Hooks
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "tape-six",
-  "version": "1.10.1",
+  "version": "1.11.0",
   "description": "TAP-inspired unit test library for Node, Deno, Bun, and browsers. ES modules, TypeScript, zero dependencies.",
   "type": "module",
   "main": "index.js",
@@ -103,7 +103,7 @@
     }
   },
   "devDependencies": {
-    "@types/node": "^25.9.1",
+    "@types/node": "^26.0.1",
     "typescript": "^6.0.3"
   }
 }

package/src/State.js

@@ -63,7 +63,6 @@ const replacer =
       if (value instanceof Map)
         return {type: 'Map', value: Object.fromEntries(value), [signature]: signature};
 
-      // break circular references
       if (seen.has(value)) return {type: 'Circular', [signature]: signature};
       seen.add(value);
     }
@@ -75,14 +74,10 @@ const serialize = object => {
   try {
     const result = JSON.stringify(object, replacer());
     if (typeof result == 'string') return result;
-  } catch (error) {
-    // squelch
-  }
+  } catch (error) {}
   try {
     return JSON.stringify({type: 'String', value: String(object)});
-  } catch (error) {
-    // squelch
-  }
+  } catch (error) {}
   return JSON.stringify({
     type: 'Problem',
     value: 'cannot convert value to JSON or string',

package/src/Tester.js

@@ -93,8 +93,6 @@ export class Tester {
     });
   }
 
-  // asserts
-
   pass(msg) {
     this.reporter.report({
       name: msg || 'pass',
@@ -133,6 +131,20 @@ export class Tester {
     });
   }
 
+  reportAssertion(assertion) {
+    const {ok, message, marker, operator, expected, actual} = assertion || {};
+    const hasData = expected !== undefined || actual !== undefined;
+    this.reporter.report({
+      name: message || 'invariant',
+      test: this.testNumber,
+      marker: marker instanceof Error ? marker : new Error(),
+      time: this.timer.now(),
+      operator: operator || 'ok',
+      fail: !ok,
+      data: hasData ? {expected, actual} : {expected: true, actual: ok}
+    });
+  }
+
   notOk(value, msg) {
     this.reporter.report({
       name: msg || 'should be falsy',
@@ -445,8 +457,6 @@ export class Tester {
         });
       });
   }
-
-  // missing: T (eval)
 }
 Tester.prototype.any = Tester.prototype._ = any;
 

package/src/reporters/TTYReporter.js

@@ -5,8 +5,6 @@ import {signature} from '../State.js';
 import {normalizeBox, padBox, padBoxLeft, drawBox, stackHorizontally} from '../utils/box.js';
 import {formatNumber, formatTime} from '../utils/formatters.js';
 
-// colors
-
 const join = (...args) => args.filter(value => value).join('');
 const to6 = x => Math.min(5, Math.round((Math.max(0, Math.min(255, x)) / 255) * 6));
 const buildColor = (r, g, b) => 16 + 36 * to6(r) + 6 * to6(g) + to6(b);
@@ -16,8 +14,6 @@ const successStyle = `\x1B[48;5;${buildColor(0, 32, 0)};1;97m`,
   skippedStyle = `\x1B[48;5;${buildColor(0, 0, 64)};1;97m`,
   reset = '\x1B[0m';
 
-// misc
-
 const consoleDict = {
   log: 'log',
   info: 'inf',
@@ -35,8 +31,6 @@ const getType = value => {
   return className ? type + '/' + className : type;
 };
 
-// main
-
 export class TTYReporter extends Reporter {
   constructor({
     output = process.stdout,
@@ -80,7 +74,6 @@ export class TTYReporter extends Reporter {
     this.lines = 0;
     this.testStack = [];
 
-    // colors
     this.red = this.paint('\x1B[31m');
     this.green = this.paint('\x1B[92m');
     this.blue = this.paint('\x1B[94m');
@@ -98,8 +91,6 @@ export class TTYReporter extends Reporter {
     this.stdoutPaint = this.paint('\x1B[90m');
     this.stderrPaint = this.paint('\x1B[31m');
 
-    // watching for console output
-
     this.consoleWasUsed = false;
     this.consoleSkipChecks = false;
 
@@ -416,7 +407,6 @@ export class TTYReporter extends Reporter {
     );
 
     box2[0] = this.brightWhite(box2[0]);
-    // box2[1] = this.brightYellow(box2[1]);
     box2[2] = this.green(box2[2]);
     box2[3] = this.red(box2[3]);
     box2[4] = this.blue(box2[4]);

package/src/runners/deno/TestWorker.js

@@ -18,7 +18,6 @@ export default class TestWorker extends EventServer {
       id = String(++this.counter),
       worker = new Worker(new URL('./worker.js', import.meta.url), {
         type: 'module'
-        // deno: {permissions: 'inherit'}
       });
     this.idToWorker[id] = worker;
     worker.addEventListener('message', event => {

package/src/runners/seq/BypassReporter.js

@@ -1,4 +1,3 @@
-// Faking Reporter
 export class BypassReporter {
   constructor(reporter, reportTo) {
     this.reporter = reporter;

package/src/test.js

@@ -38,9 +38,7 @@ export const getTesters = () => testers;
 export const getTester = () => (testers.length ? testers[testers.length - 1] : null);
 
 const processArgs = (name, options, testFn) => {
-  // normalize arguments
   if (typeof name == 'string') {
-    // nothing
   } else if (typeof options == 'string') {
     [name, options] = [options, name];
   } else if (typeof testFn == 'string') {
@@ -49,12 +47,10 @@ const processArgs = (name, options, testFn) => {
     [name, options, testFn] = [null, name, options];
   }
   if (typeof options == 'object') {
-    // nothing
   } else {
     [options, testFn] = [testFn, options];
   }
 
-  // normalize options
   options = {...options};
   if (name && typeof name == 'string') {
     options.name = name;
@@ -330,8 +326,6 @@ Tester.prototype.asPromise = async function asPromise(name, options, testFn) {
   this.comment('SKIP test: ' + options.name);
 };
 
-// before/after hooks
-
 const addToHook = name => fn => {
   if (testers.length) {
     const tester = testers[testers.length - 1];

package/src/utils/EventServer.js

@@ -29,7 +29,6 @@ export default class EventServer {
     this.retained = {};
     this.readyQueue = [];
 
-    // control plane bookkeeping
     this.liveTasks = new Set();
     this.deadlineTimers = {};
     this.stopRequested = false;
@@ -76,7 +75,6 @@ export default class EventServer {
       }
     }
     if (this.passThroughId === id) {
-      // dump ready events
       for (const events of this.readyQueue) {
         for (const event of events) {
           this.reporter.report(event, true);
@@ -88,12 +86,10 @@ export default class EventServer {
       const events = this.retained[id];
       if (events) {
         if (this.passThroughId === null) {
-          // dump events
           for (const event of events) {
             this.reporter.report(event, true);
           }
         } else {
-          // add to the ready queue
           this.readyQueue.push(events);
         }
         delete this.retained[id];
@@ -149,7 +145,6 @@ export default class EventServer {
     }
   }
   makeTask(fileName) {
-    // TBD in children
     // should return a task id as a string
   }
   destroyTask(id, reason) {

package/src/utils/config.js

@@ -83,14 +83,12 @@ export const resolvePatterns = async (rootFolder, patterns) => {
 export const getConfig = async (rootFolder, traceFn) => {
   let cfg = null;
 
-  // check tape6.json
   try {
     cfg = JSON.parse(await fsp.readFile(path.join(rootFolder, 'tape6.json')));
   } catch (error) {
     traceFn && traceFn('Cannot read tape6.json');
   }
 
-  // check package.json, "tape6" section
   if (!cfg) {
     try {
       const pkg = JSON.parse(await fsp.readFile(path.join(rootFolder, 'package.json')));
@@ -100,7 +98,6 @@ export const getConfig = async (rootFolder, traceFn) => {
     }
   }
 
-  // check well-known files
   if (!cfg) cfg = {tests: ['/tests/test-*.js', '/tests/test-*.mjs'], cli: ['/tests/test-*.cjs']};
 
   return cfg;
@@ -109,7 +106,6 @@ export const getConfig = async (rootFolder, traceFn) => {
 export const resolveTests = async (rootFolder, type, traceFn) => {
   const cfg = await getConfig(rootFolder, traceFn);
 
-  // determine test patterns
   let patterns = [];
   if (cfg[type]) {
     if (Array.isArray(cfg[type])) {
@@ -133,7 +129,6 @@ export const resolveTests = async (rootFolder, type, traceFn) => {
     patterns.push(cfg.tests);
   }
 
-  // resolve patterns
   return resolvePatterns(rootFolder, patterns);
 };
 
@@ -201,8 +196,6 @@ export const getGraceTimeout = () => readTimeoutEnv('TAPE6_GRACE_TIMEOUT', DEFAU
 // it — the default — so a worker deadline fires only when explicitly set.
 export const getWorkerTimeout = () => readTimeoutEnv('TAPE6_WORKER_TIMEOUT', 0);
 
-// parsing options
-
 export const flagNames = Object.fromEntries(
   Object.entries(flagDefs).map(([k, {name}]) => [k, name])
 );
@@ -292,7 +285,6 @@ export const getOptions = extraOptions => {
   const flags = args.flags['--flags'],
     options = {flags: {flags}, parallel: 1, files: args.files, optionFlags: args.flags};
 
-  // set back flags
   if (runtime.name === 'deno') {
     Deno.env.set('TAPE6_FLAGS', flags);
   } else if (runtime.name === 'bun') {

package/src/utils/defer.js

@@ -1,27 +1,13 @@
 let deferImplementation = globalThis.setTimeout;
 
-// initialize the variable
 do {
-  // The selection below doesn't work
-  // if (typeof queueMicrotask == 'function') {
-  //   deferImplementation = globalThis.queueMicrotask;
-  //   console.log('\nSelected: queueMicrotask()\n');
-  //   break;
-  // }
-
-  // The selection below doesn't work in Bun
-  // if (typeof process == 'object' && typeof process?.nextTick == 'function') {
-  //   deferImplementation = process.nextTick;
-  //   break;
-  // }
-
+  // queueMicrotask / process.nextTick were tried and rejected (the latter unreliable on Bun).
   if (typeof setImmediate == 'function') {
     deferImplementation = setImmediate;
     break;
   }
 
   if (typeof window != 'object') break;
-  // below are browser-only implementations
 
   if (typeof window.requestIdleCallback == 'function') {
     deferImplementation = window.requestIdleCallback;

package/src/utils/sanitize.js

@@ -27,14 +27,13 @@ export const sanitize = (value, processed, seen = new Set()) => {
     return value.map(v => sanitize(v, false, seen));
   }
 
-  // simple object
   const result = {};
   for (let [k, v] of Object.entries(value)) {
     if (isProhibited(v, seen)) continue;
     result[k] = sanitize(v, true, seen);
   }
 
-  // was it an error? => copy non-enumerable properties
+  // Error's message/stack are non-enumerable, so the loop above skips them.
   if (value instanceof Error) {
     if (value.name && !result.name) result.name = value.name;
     if (value.message && !result.message) result.message = value.message;

package/src/utils/timer.js

@@ -4,21 +4,16 @@ export const getTimer = () => timer;
 export const setTimer = newTimer => (timer = newTimer);
 
 export const selectTimer = async () => {
-  // set HR timer
   if (typeof performance == 'object' && performance && typeof performance.now == 'function') {
-    // browser or Deno
     setTimer({now: () => performance.now() + performance.timeOrigin});
     return;
   }
   if (typeof process == 'object' && typeof process.exit == 'function') {
-    // Node
     try {
       const {performance} = await import('node:perf_hooks');
       setTimer({now: () => performance.now() + performance.timeOrigin});
       return;
-    } catch (error) {
-      // squelch
-    }
+    } catch (error) {}
   }
   setTimer(Date);
 };

package/src/utils/yamlFormatter.js

@@ -71,7 +71,6 @@ const format = (value, options, level, offset, lines) => {
     return;
   }
 
-  // array
   if (Array.isArray(value)) {
     if (level + 1 > options.maxDepth) {
       lines.push(offset + '[]');
@@ -96,7 +95,6 @@ const format = (value, options, level, offset, lines) => {
     return;
   }
 
-  // regular object
   if (level + 1 > options.maxDepth) {
     lines.push(offset + '{}');
     return;

package/web-app/donut.js

@@ -3,20 +3,6 @@ const TWO_PI = 2 * Math.PI;
 const tmpl = (template, dict) => template.replace(/\$\{([^\}]*)\}/g, (_, name) => dict[name]);
 
 export const makeSegment = (args, options) => {
-  // args is {startAngle, angle, index, className}
-  // optional index points to a data point
-  // optional className is a CSS class
-  // default startAngle=0 (in radians)
-  // default angle=2*PI (in radians)
-
-  // options is {center, innerRadius, radius, gap, precision, document}
-  // default center is {x=0, y=0}
-  // default innerRadius=0
-  // default radius=100
-  // default gap=0 (gap between segments in pixels)
-  // default precision=6 (digits after decimal point)
-  // default document is document
-
   const node = (options.document || document).createElementNS('http://www.w3.org/2000/svg', 'path'),
     center = options.center || {x: 0, y: 0},
     innerRadius = Math.max(options.innerRadius || 0, 0),
@@ -40,14 +26,11 @@ export const makeSegment = (args, options) => {
 
   if (angle >= TWO_PI) {
     data.tr = (2 * radius).toFixed(precision);
-    // generate a circle
     if (innerRadius <= 0) {
-      // a circle
       path = tmpl('M${cx} ${cy}m -${r} 0a${r} ${r} 0 1 0 ${tr} 0a${r} ${r} 0 1 0 -${tr} 0z', data);
     } else {
       data.r0 = innerRadius.toFixed(precision);
       data.tr0 = (2 * innerRadius).toFixed(precision);
-      // a donut
       path = tmpl(
         'M${cx} ${cy}m -${r} 0a${r} ${r} 0 1 0 ${tr} 0a${r} ${r} 0 1 0 -${tr} 0zM${cx} ${cy}m -${r0} 0a${r0} ${r0} 0 1 1 ${tr0} 0a${r0} ${r0} 0 1 1 -${tr0} 0z',
         data
@@ -66,7 +49,6 @@ export const makeSegment = (args, options) => {
     data.x2 = (radius * Math.cos(finish) + cx).toFixed(precision);
     data.y2 = (radius * Math.sin(finish) + cy).toFixed(precision);
     if (innerRadius <= 0) {
-      // a pie slice
       path = tmpl('M${cx} ${cy}L${x1} ${y1}A${r} ${r} 0 ${lg} 1 ${x2} ${y2}L${cx} ${cy}z', data);
     } else {
       start = startAngle + innerGapAngle;
@@ -79,7 +61,6 @@ export const makeSegment = (args, options) => {
       data.y3 = (innerRadius * Math.sin(finish) + cy).toFixed(precision);
       data.x4 = (innerRadius * Math.cos(start) + cx).toFixed(precision);
       data.y4 = (innerRadius * Math.sin(start) + cy).toFixed(precision);
-      // a segment
       path = tmpl(
         'M${x1} ${y1}A${r} ${r} 0 ${lg} 1 ${x2} ${y2}L${x3} ${y3}A${r0} ${r0} 0 ${lg} 0 ${x4} ${y4}L${x1} ${y1}z',
         data
@@ -97,24 +78,6 @@ export const makeSegment = (args, options) => {
 };
 
 export const processPieRun = (data, options) => {
-  // data is [datum, datum...]
-  // datum is {value, className, skip, hide}
-  // value is a positive number
-  // className is an optional CSS class name
-  // skip is a flag (default: false) to skip this segment completely
-  // hide is a flag (default: false) to suppress rendering
-
-  // options is {center, innerRadius, radius, startAngle, minSizeInPx, skipIfLessInPx, emptyClass, precision}
-  // default center is {x=0, y=0}
-  // default innerRadius=0
-  // default radius=100
-  // default startAngle=0 (in radians)
-  // default gap=0 (gap between segments in pixels)
-  // default precision=6 (digits after decimal point)
-  // minSizeInPx is to make non-empty segments at least this big (default: 0).
-  // skipIfLessInPx is a threshold (default: 0), when to skip too small segments.
-  // emptyClass is a CSS class name for an empty run
-
   const radius = Math.max(options.radius || 100, options.innerRadius || 0, 0),
     gap = Math.max(options.gap || 0, 0),
     minSizeInPx = Math.max(options.minSizeInPx || 0, 0),
@@ -128,7 +91,6 @@ export const processPieRun = (data, options) => {
       document: options.document
     };
 
-  // sanitize data
   data.forEach((datum, index) => {
     if (!datum.skip && (isNaN(datum.value) || datum.value === null || datum.value <= 0)) {
       datum.skip = true;
@@ -140,7 +102,6 @@ export const processPieRun = (data, options) => {
 
   let node;
   if (total <= 0) {
-    // empty run
     node = makeSegment(
       {
         index: -1, // to denote that it is not an actionable node
@@ -168,7 +129,6 @@ export const processPieRun = (data, options) => {
     return [node];
   }
 
-  // find too small segments
   const sizes = data.map(datum => {
     let angle = 0;
     if (!datum.skip) {
@@ -179,7 +139,6 @@ export const processPieRun = (data, options) => {
 
   let minAngle, newTotal, changeRatio;
   if (minSizeInPx > 0) {
-    // adjust angles
     minAngle = (minSizeInPx + gap) / radius;
     sizes.forEach((size, index) => {
       const datum = data[index];
@@ -200,7 +159,6 @@ export const processPieRun = (data, options) => {
       }
     });
   } else if (skipIfLessInPx > 0) {
-    // suppress angles
     minAngle = skipIfLessInPx / radius;
     sizes.forEach((size, index) => {
       const datum = data[index];
@@ -217,7 +175,6 @@ export const processPieRun = (data, options) => {
     });
   }
 
-  // generate shape objects
   const shapes = [];
   let startAngle = options.startAngle || 0;
   data.forEach((datum, index) => {

package/web-app/index.js

@@ -106,7 +106,6 @@ window.addEventListener('DOMContentLoaded', () => {
   const donut = document.querySelector('tape6-donut');
   donut.show([{value: 0, className: 'nothing'}], {
     center: {x: 100, y: 100},
-    // gap: 4,
     innerRadius: 40,
     radius: 90,
     startAngle: Math.PI / 2,

package/web-app/tape6-spinner.js

@@ -3,7 +3,6 @@ class Tape6Spinner extends HTMLElement {
     super();
     this.nextState = '';
     this.inTransition = false;
-    // create squares
     let child = null;
     for (let depth = this.getAttribute('depth') || 3; depth > 0; --depth) {
       const node1 = document.createElement('div');
@@ -18,7 +17,6 @@ class Tape6Spinner extends HTMLElement {
     node.className = 'square black';
     child && node.appendChild(child);
     this.appendChild(node);
-    // watch for transitions
     this.addEventListener('transitionend', this);
   }
   handleEvent(event) {