@jterrazz/test 15.0.0 → 15.1.0

package/dist/checker.js

@@ -218,7 +218,7 @@ function lineAt(text, index) {
 * Every quoted string and simple template literal in a source. Over-collection
 * only ever *hides* a dead-fixture finding, so a greedy regex is safe here.
 */
-const STRING_LITERAL = /'(?<single>(?:[^'\\\n]|\\.)*)'|"(?<double>(?:[^"\\\n]|\\.)*)"|`(?<template>(?:[^`$\\]|\\.)*)`/g;
+const STRING_LITERAL = /'(?<single>(?:[^'\\\n]|\\.)*)'|"(?<double>(?:[^"\\\n]|\\.)*)"|`(?<template>(?:[^`$\\]|\\.)*)`/gu;
 function collectLiterals(text) {
 	const found = /* @__PURE__ */ new Set();
 	for (const match of text.matchAll(STRING_LITERAL)) {
@@ -233,12 +233,12 @@ function collectLiterals(text) {
 * variable makes the reference set incomplete, so the feature is reported at
 * warn level instead of error (a false positive would be worse than a miss).
 */
-const FIXTURE_VERB_ARG = /(?:\.(?:seed|request|fixture)|\btoMatch)\s*\(\s*(?<arg>[^,)]*)/g;
+const FIXTURE_VERB_ARG = /(?:\.(?:seed|request|fixture)|\btoMatch)\s*\(\s*(?<arg>[^,)]*)/gu;
 function hasNonLiteralFixtureArg(text) {
 	for (const match of text.matchAll(FIXTURE_VERB_ARG)) {
 		const arg = (match.groups?.arg ?? "").trim();
 		if (arg.length === 0) continue;
-		if (!(/^'(?:[^'\\\n]|\\.)*'$/.test(arg) || /^"(?:[^"\\\n]|\\.)*"$/.test(arg) || /^`(?:[^`$\\]|\\.)*`$/.test(arg))) return true;
+		if (!(/^'(?:[^'\\\n]|\\.)*'$/u.test(arg) || /^"(?:[^"\\\n]|\\.)*"$/u.test(arg) || /^`(?:[^`$\\]|\\.)*`$/u.test(arg))) return true;
 	}
 	return false;
 }
@@ -252,18 +252,18 @@ function suppressedLines(text, id) {
 	const suppressed = /* @__PURE__ */ new Set();
 	const lines = text.split("\n");
 	const applies = (raw) => {
-		const ids = raw.replace(/--.*$/, "").trim().split(/[\s,]+/);
+		const ids = raw.replace(/--.*$/u, "").trim().split(/[\s,]+/u);
 		return ids.includes(id) || ids.includes("*");
 	};
 	for (const [index, line] of lines.entries()) {
-		let match = /checker-disable-next-line\s+(?<ids>.*)$/i.exec(line);
+		let match = /checker-disable-next-line\s+(?<ids>.*)$/iu.exec(line);
 		if (match !== null && applies(match.groups?.ids ?? "")) {
-			for (let next = index + 1; next < lines.length; next += 1) if (lines[next].trim().length > 0) {
+			for (let next = index + 1; next < lines.length; next += 1) if ((lines[next] ?? "").trim().length > 0) {
 				suppressed.add(next + 1);
 				break;
 			}
 		}
-		match = /checker-disable-line\s+(?<ids>.*)$/i.exec(line);
+		match = /checker-disable-line\s+(?<ids>.*)$/iu.exec(line);
 		if (match !== null && applies(match.groups?.ids ?? "")) suppressed.add(index + 1);
 	}
 	return suppressed;
@@ -314,21 +314,21 @@ function splitTopLevel(body) {
 /** The destructured export names of a `= await specification.<ctor>(` binding. */
 function runnerExportsOf(text) {
 	const names = /* @__PURE__ */ new Set();
-	const decl = new RegExp(String.raw`\{(?<names>[^}]*)\}\s*=\s*await\s+specification\.(?:${CONSTRUCTORS})\s*\(`, "g");
+	const decl = new RegExp(String.raw`\{(?<names>[^}]*)\}\s*=\s*await\s+specification\.(?:${CONSTRUCTORS})\s*\(`, "gu");
 	for (const match of text.matchAll(decl)) for (const raw of (match.groups?.names ?? "").split(",")) {
-		const key = raw.split(":")[0].trim();
+		const key = (raw.split(":")[0] ?? "").trim();
 		if (key.length > 0 && !NON_RUNNER_BINDINGS.has(key)) names.add(key);
 	}
 	return names;
 }
 /** The leading callee identifier of an expression (`postgres()` → `postgres`). */
 function leadingCallee(value) {
-	const match = /^(?<callee>[A-Za-z_$][\w$]*)\s*\(/.exec(value.trim());
+	const match = /^(?<callee>[A-Za-z_$][\w$]*)\s*\(/u.exec(value.trim());
 	return match === null ? match : match.groups?.callee ?? null;
 }
 /** SQL factory count of a `services:` record, or null if non-literal/absent. */
 function sqlDatabaseCount(text) {
-	const keyword = /\bservices\s*:/.exec(text);
+	const keyword = /\bservices\s*:/u.exec(text);
 	if (keyword === null) return null;
 	const braceIndex = text.indexOf("{", keyword.index + keyword[0].length);
 	if (braceIndex === -1 || text.slice(keyword.index + keyword[0].length, braceIndex).trim() !== "") return null;
@@ -348,9 +348,9 @@ function sqlDatabaseCount(text) {
 function modelSpecFile(file) {
 	const text = readSource(file);
 	let dockerAware = false;
-	for (const match of text.matchAll(/specification\.cli\s*\(/g)) {
+	for (const match of text.matchAll(/specification\.cli\s*\(/gu)) {
 		const argsOpen = text.indexOf("(", match.index);
-		if (argsOpen !== -1 && /\bdocker\s*:/.test(balancedParens(text, argsOpen))) dockerAware = true;
+		if (argsOpen !== -1 && /\bdocker\s*:/u.test(balancedParens(text, argsOpen))) dockerAware = true;
 	}
 	return {
 		dockerAware,
@@ -362,17 +362,18 @@ function modelSpecFile(file) {
 /** Resolve a relative import specifier to a `.ts` path (else undefined). */
 function resolveRelativeImport(fromFile, specifier) {
 	if (!specifier.startsWith(".")) return;
-	return resolve(dirname(fromFile), specifier).replace(/\.js$/, ".ts");
+	return resolve(dirname(fromFile), specifier).replace(/\.js$/u, ".ts");
 }
 /** Every `import { a, b as c } from '…'` binding in a file. */
 function namedImports(text) {
 	const bindings = [];
-	for (const match of text.matchAll(/import\s+(?:type\s+)?\{(?<names>[^}]*)\}\s+from\s+['"](?<source>[^'"]+)['"]/g)) {
+	for (const match of text.matchAll(/import\s+(?:type\s+)?\{(?<names>[^}]*)\}\s+from\s+['"](?<source>[^'"]+)['"]/gu)) {
 		const source = match.groups?.source ?? "";
 		for (const raw of (match.groups?.names ?? "").split(",")) {
 			const part = raw.trim();
 			if (part.length === 0) continue;
-			const [name, local] = part.split(/\s+as\s+/).map((token) => token.trim());
+			const [name, local] = part.split(/\s+as\s+/u).map((token) => token.trim());
+			if (name === void 0) continue;
 			bindings.push({
 				local: local ?? name,
 				name,
@@ -405,7 +406,7 @@ function checkDockerRunnerAwaitUsing(rootDir) {
 		}
 		if (runners.size === 0) continue;
 		const suppressed = suppressedLines(readText$1(testFile), "b5");
-		for (const match of text.matchAll(/\b(?<kind>await\s+using|const|let|var)\s+[A-Za-z_$][\w$]*\s*=\s*await\s+(?<runner>[A-Za-z_$][\w$]*)\b[^;]*?\.exec\s*\(/g)) {
+		for (const match of text.matchAll(/\b(?<kind>await\s+using|const|let|var)\s+[A-Za-z_$][\w$]*\s*=\s*await\s+(?<runner>[A-Za-z_$][\w$]*)\b[^;]*?\.exec\s*\(/gu)) {
 			const kind = match.groups?.kind ?? "";
 			const runner = match.groups?.runner ?? "";
 			if (kind.startsWith("await using") || !runners.has(runner)) continue;
@@ -425,18 +426,18 @@ function checkDockerRunnerAwaitUsing(rootDir) {
 /** Local inline runners a test declares itself (`specification.api(…)`). */
 function localRunnerBindings(text) {
 	const locals = /* @__PURE__ */ new Set();
-	const decl = new RegExp(String.raw`\{(?<names>[^}]*)\}\s*=\s*await\s+specification\.(?:${CONSTRUCTORS})\s*\(`, "g");
+	const decl = new RegExp(String.raw`\{(?<names>[^}]*)\}\s*=\s*await\s+specification\.(?:${CONSTRUCTORS})\s*\(`, "gu");
 	for (const match of text.matchAll(decl)) for (const raw of (match.groups?.names ?? "").split(",")) {
 		const part = raw.trim();
-		const local = (part.includes(":") ? part.split(":")[1] : part).trim();
-		if (local.length > 0 && !NON_RUNNER_BINDINGS.has(part.split(":")[0].trim())) locals.add(local);
+		const local = (part.includes(":") ? part.split(":")[1] ?? "" : part).trim();
+		if (local.length > 0 && !NON_RUNNER_BINDINGS.has((part.split(":")[0] ?? "").trim())) locals.add(local);
 	}
 	return locals;
 }
 /** The identifier immediately before a `.seed(`/`.table(` member, if any. */
 function memberObjectIdentifier(text, dotIndex) {
 	const before = text.slice(0, dotIndex);
-	const match = /(?<name>[A-Za-z_$][\w$]*)\s*$/.exec(before);
+	const match = /(?<name>[A-Za-z_$][\w$]*)\s*$/u.exec(before);
 	return match === null ? match : match.groups?.name ?? null;
 }
 function checkDatabaseProperty(rootDir) {
@@ -455,17 +456,18 @@ function checkDatabaseProperty(rootDir) {
 			const model = target === void 0 ? void 0 : enforced.get(target);
 			if (model?.runnerExports.has(binding.name) === true) applicable.add(model);
 		}
-		if (applicable.size !== 1) continue;
-		const count = [...applicable][0].sqlDatabaseCount ?? 0;
+		const [onlyApplicable] = applicable;
+		if (applicable.size !== 1 || onlyApplicable === void 0) continue;
+		const count = onlyApplicable.sqlDatabaseCount ?? 0;
 		const locals = localRunnerBindings(text);
 		const suppressed = suppressedLines(readText$1(testFile), "a7");
 		const rel = relative(rootDir, testFile);
-		for (const match of text.matchAll(/\.(?<verb>seed|table)\s*\(/g)) {
+		for (const match of text.matchAll(/\.(?<verb>seed|table)\s*\(/gu)) {
 			const verb = match.groups?.verb ?? "";
 			const object = memberObjectIdentifier(text, match.index);
 			if (object !== null && locals.has(object)) continue;
 			const args = balancedParens(text, text.indexOf("(", match.index));
-			const hasDatabase = /\bdatabase\s*:/.test(args);
+			const hasDatabase = /\bdatabase\s*:/u.test(args);
 			const line = lineAt(text, match.index);
 			if (suppressed.has(line)) continue;
 			if (count >= 2 && !hasDatabase) violations.push({
@@ -584,7 +586,7 @@ function checkPoolFixtures(rootDir) {
 	return violations;
 }
 /** A `.fixture('…')` call with a plain string argument. */
-const FIXTURE_CALL = /\.fixture\(\s*(?<quote>['"`])(?<path>(?:[^'"`\\]|\\.)*)\k<quote>/g;
+const FIXTURE_CALL = /\.fixture\(\s*(?<quote>['"`])(?<path>(?:[^'"`\\]|\\.)*)\k<quote>/gu;
 /**
 * Every fixture path a spec file names, with the line it names it on. A
 * `<case>.spec.yaml` is read through the runner's own parser; a `.ts` file
@@ -620,7 +622,7 @@ function listReferrers(specsRoot, pool) {
 /** The pool entry a `$FIXTURES/…` path names — its FIRST segment. */
 function poolEntryOf(path) {
 	if (!path.startsWith("$FIXTURES/")) return;
-	const entry = path.slice(10).split("/")[0];
+	const entry = path.slice(10).split("/")[0] ?? "";
 	return entry.length > 0 ? entry : void 0;
 }
 /** Pool entry → the spec directories that reference it, each with its referrers. */
@@ -647,8 +649,9 @@ function misplacedPoolFixtures(rootDir) {
 	for (const pool of findPools(rootDir)) {
 		const usage = poolUsage(dirname(pool), pool);
 		for (const [entry, leaves] of usage) {
-			if (leaves.size !== 1 || !isDir(join(pool, entry))) continue;
-			const [leaf, referrers] = [...leaves][0];
+			const [only] = leaves;
+			if (leaves.size !== 1 || only === void 0 || !isDir(join(pool, entry))) continue;
+			const [leaf, referrers] = only;
 			misplaced.push({
 				entry,
 				leaf,
@@ -869,7 +872,7 @@ function checkBlockScalars(file, lineOf) {
 	}
 	return findings;
 }
-const KEBAB = /^[a-z0-9]+(?:-[a-z0-9]+)*$/;
+const KEBAB = /^[a-z0-9]+(?:-[a-z0-9]+)*$/u;
 /** Words the `.spec.yaml` suffix already says — a case name repeating one says nothing. */
 const REDUNDANT_WORDS = /* @__PURE__ */ new Set([
 	"cli",
@@ -898,7 +901,7 @@ function checkFileName(rel) {
 }
 const MAX_DESCRIPTION = 100;
 /** A first word that is an identifier, not prose — exempt from the lowercase rule. */
-const SYMBOL_WORD = /^[A-Z0-9_]+$/;
+const SYMBOL_WORD = /^[A-Z0-9_]+$/u;
 /**
 * The description IS the vitest test title, and it is held to the title rules:
 * one lowercase line of prose, no trailing period, short enough to read in a
@@ -915,26 +918,26 @@ function checkDescription(document) {
 	}
 	if (text.length > MAX_DESCRIPTION) findings.push(at(`description: is ${text.length} characters — keep it under ${MAX_DESCRIPTION}`));
 	if (text.endsWith(".")) findings.push(at("description: ends with a period — a title is a fragment, not a sentence"));
-	const first = text.split(/\s+/)[0] ?? "";
-	if (/^[A-Z]/.test(text) && !SYMBOL_WORD.test(first)) findings.push(at("description: starts with a capital — a title is lowercase prose"));
+	const first = text.split(/\s+/u)[0] ?? "";
+	if (/^[A-Z]/u.test(text) && !SYMBOL_WORD.test(first)) findings.push(at("description: starts with a capital — a title is lowercase prose"));
 	return findings;
 }
 /** The literals the update writer must have failed to tokenise, and their token. */
 const VOLATILE = [
 	{
-		pattern: /\b(?:127\.0\.0\.1|localhost|0\.0\.0\.0):\d{2,5}\b/,
+		pattern: /\b(?:127\.0\.0\.1|localhost|0\.0\.0\.0):\d{2,5}\b/u,
 		token: "{{url}} (or {{port}})"
 	},
 	{
-		pattern: /(?:^|[\s"'(=])(?:\/private)?\/tmp\//,
+		pattern: /(?:^|[\s"'(=])(?:\/private)?\/tmp\//u,
 		token: "{{workdir}}"
 	},
 	{
-		pattern: /(?:^|[\s"'(=])\/var\/folders\//,
+		pattern: /(?:^|[\s"'(=])\/var\/folders\//u,
 		token: "{{workdir}}"
 	},
 	{
-		pattern: /(?:^|[\s"'(=])\/(?:Users|home)\/[^\s"'/]+\//,
+		pattern: /(?:^|[\s"'(=])\/(?:Users|home)\/[^\s"'/]+\//u,
 		token: "{{workdir}} or {{path}}"
 	}
 ];
@@ -955,8 +958,8 @@ function checkVolatileLiterals(document) {
 	}
 	return findings;
 }
-const ISO8601 = /\d{4}-\d{2}-\d{2}T\d{2}:\d{2}:\d{2}(?:\.\d+)?(?:Z|[+-]\d{2}:\d{2})/;
-const UUID = /[0-9a-fA-F]{8}-[0-9a-fA-F]{4}-[0-9a-fA-F]{4}-[0-9a-fA-F]{4}-[0-9a-fA-F]{12}/;
+const ISO8601 = /\d{4}-\d{2}-\d{2}T\d{2}:\d{2}:\d{2}(?:\.\d+)?(?:Z|[+-]\d{2}:\d{2})/u;
+const UUID = /[0-9a-fA-F]{8}-[0-9a-fA-F]{4}-[0-9a-fA-F]{4}-[0-9a-fA-F]{4}-[0-9a-fA-F]{12}/u;
 /** Everything the document itself FIXED: what it feeds in, and what its fixtures hold. */
 function fixedData(document, dir) {
 	const parts = document.runs.map((run) => run.stdin ?? "");
@@ -1028,7 +1031,7 @@ function objectKeys(source, from) {
 	let depth = 0;
 	let quote = "";
 	for (let index = from; index < source.length; index += 1) {
-		const char = source[index];
+		const char = source[index] ?? "";
 		if (quote !== "") {
 			if (char === quote && source[index - 1] !== "\\") quote = "";
 			continue;
@@ -1049,10 +1052,11 @@ function objectKeys(source, from) {
 			};
 			continue;
 		}
-		if (depth === 1 && /[A-Za-z_$'"]/.test(char)) {
-			const rest = /^(?<quote>['"]?)(?<name>[A-Za-z_$][\w$]*)\k<quote>\s*:/.exec(source.slice(index));
-			if (rest?.groups) {
-				keys.push(rest.groups.name);
+		if (depth === 1 && /[A-Za-z_$'"]/u.test(char)) {
+			const rest = /^(?<quote>['"]?)(?<name>[A-Za-z_$][\w$]*)\k<quote>\s*:/u.exec(source.slice(index));
+			const name = rest?.groups?.name;
+			if (rest !== null && name !== void 0) {
+				keys.push(name);
 				index += rest[0].length - 1;
 			}
 		}
@@ -1083,12 +1087,12 @@ function registeredNames(dir) {
 			const serve = /* @__PURE__ */ new Set();
 			for (const entry of entries) {
 				const text = readText(join(current, entry));
-				for (const call of text.matchAll(/specification\.cli\s*\(/g)) {
+				for (const call of text.matchAll(/specification\.cli\s*\(/gu)) {
 					const open = text.indexOf("{", call.index + call[0].length);
 					if (open === -1) continue;
 					const options = objectKeys(text, open);
 					for (const [key, into] of [["env", env], ["serve", serve]]) {
-						const marker = new RegExp(String.raw`\b${key}\s*:\s*\{`).exec(text.slice(open, options.end));
+						const marker = new RegExp(String.raw`\b${key}\s*:\s*\{`, "u").exec(text.slice(open, options.end));
 						if (marker === null) continue;
 						const at = open + marker.index + marker[0].length - 1;
 						for (const name of objectKeys(text, at).keys) into.add(name);
@@ -1134,7 +1138,7 @@ function checkSpecConventions(text, rel, path) {
 	}
 	const lineAt = (node) => {
 		const range = "range" in node && Array.isArray(node.range) ? node.range : void 0;
-		const key = node.key;
+		const { key } = node;
 		const target = range ?? (isScalar(key) && Array.isArray(key.range) ? key.range : void 0);
 		return target === void 0 ? 1 : file.source.lineAt(target[0]);
 	};
@@ -1204,7 +1208,7 @@ function fixSpecDocument(text, rel) {
 	const reorder = (map, order) => {
 		const sorted = [...pairsOf(map).filter((pair) => order.includes(keyName(pair) ?? ""))].sort((a, b) => order.indexOf(keyName(a)) - order.indexOf(keyName(b)));
 		let cursor = 0;
-		map.items = pairsOf(map).map((pair) => order.includes(keyName(pair) ?? "") ? sorted[cursor++] : pair);
+		map.items = pairsOf(map).map((pair) => (order.includes(keyName(pair) ?? "") ? sorted[cursor++] : pair) ?? pair);
 	};
 	const map = documentMap(file);
 	if (map === null) return null;
@@ -1252,11 +1256,11 @@ function fixSpecFiles(rootDir) {
 * It shares TOKEN_KINDS with the runtime matcher so the channels cannot drift.
 */
 /** A well-formed token: `{{word}}` / `{{word#ref}}`. */
-const VALID_TOKEN = /^[A-Za-z][A-Za-z0-9]*(?:#[\w.-]+)?$/;
+const VALID_TOKEN = /^[A-Za-z][A-Za-z0-9]*(?:#[\w.-]+)?$/u;
 /** Any `{{ … }}` block (no nested braces) — classified by the scanner below. */
-const BRACE_BLOCK = /\{\{(?<inner>[^{}]*)\}\}/g;
+const BRACE_BLOCK = /\{\{(?<inner>[^{}]*)\}\}/gu;
 /** The leading identifier of a brace block, for malformed-ref classification. */
-const LEADING_WORD = /^(?<kind>[A-Za-z][A-Za-z0-9]*)/;
+const LEADING_WORD = /^(?<kind>[A-Za-z][A-Za-z0-9]*)/u;
 const KNOWN = new Set(TOKEN_KINDS);
 /** Directories whose files carry the token grammar (D4). */
 const FIXTURE_DIRS = /* @__PURE__ */ new Set([GROUND_EXPECTED, GROUND_REQUESTS]);
@@ -1283,7 +1287,7 @@ function findUnknownTokens(text) {
 	for (const [index, lineText] of lines.entries()) for (const match of lineText.matchAll(BRACE_BLOCK)) {
 		const inner = match.groups?.inner ?? "";
 		if (VALID_TOKEN.test(inner)) {
-			if (!KNOWN.has(inner.split("#")[0])) violations.push({
+			if (!KNOWN.has(inner.split("#")[0] ?? "")) violations.push({
 				line: index + 1,
 				token: match[0]
 			});
@@ -1303,7 +1307,7 @@ function findKnownTokens(text) {
 	const lines = text.split("\n");
 	for (const [index, lineText] of lines.entries()) for (const match of lineText.matchAll(BRACE_BLOCK)) {
 		const inner = match.groups?.inner ?? "";
-		if (VALID_TOKEN.test(inner) && KNOWN.has(inner.split("#")[0])) found.push({
+		if (VALID_TOKEN.test(inner) && KNOWN.has(inner.split("#")[0] ?? "")) found.push({
 			line: index + 1,
 			token: match[0]
 		});
@@ -1325,8 +1329,8 @@ function firstLine(text) {
 	for (const line of text.split("\n")) if (line.trim().length > 0) return line.trim();
 	return "";
 }
-const REQUEST_LINE = /^(?<method>GET|POST|PUT|PATCH|DELETE|HEAD|OPTIONS) \/\S*/;
-const STATUS_LINE = /^HTTP\/\d(?:\.\d)? \d{3}\b/;
+const REQUEST_LINE = /^(?<method>GET|POST|PUT|PATCH|DELETE|HEAD|OPTIONS) \/\S*/u;
+const STATUS_LINE = /^HTTP\/\d(?:\.\d)? \d{3}\b/u;
 /**
 * Check one `<case>.spec.yaml` — D4b's shape through the SAME parser the runner
 * uses (one grammar, no drift), then D4's token vocabulary over the STREAMS
@@ -1386,10 +1390,7 @@ function checkConventionFiles(rootDir) {
 			}
 			if (entry.name.endsWith(".spec.yaml")) {
 				const text = decodeText(path);
-				if (text !== null) {
-					violations.push(...checkSpecFile(text, rel));
-					violations.push(...checkSpecConventions(text, rel, path));
-				}
+				if (text !== null) violations.push(...checkSpecFile(text, rel), ...checkSpecConventions(text, rel, path));
 				continue;
 			}
 			if (inside === null) continue;