as-test 1.5.1 → 1.5.2

package/CHANGELOG.md

@@ -1,5 +1,14 @@
 # Change Log
 
+## 2026-06-01 - v1.5.2
+
+### Selectors resolve folders, files, and globs consistently
+
+- feat: positional selectors for `ast test`/`ast run`/`ast build` now resolve through a single shared resolver (`cli/selectors.ts:resolveSpecFiles`), replacing three drifting private copies of `resolveInputPatterns` (in `build-core`, `run-core`, and `index`). Three input shapes are supported:
+  - **Bare folders/files/globs** (no leading `./`) resolve against the configured input root(s) — the static prefix of each `input` glob, e.g. `assembly/__tests__` — searched recursively, and fall back to the cwd only if nothing matched there: `ast test rfc/` → `<root>/**/rfc/**/*.spec.ts`; `ast test foo` → `<root>/**/foo.spec.ts`; `ast test 'rfc/*.spec.ts'` → `<root>/**/rfc/*.spec.ts` (the user's glob appended verbatim). A bare path shorthand like `nested/array` is tried as a cwd path first, then anchored to the test folder.
+  - **`./`-prefixed** selectors (and absolute / `~` paths) are cwd-relative only; on a miss we emit a `did you mean "rfc/*.spec.ts"` hint pointing at the test-folder form when that would have matched.
+- feat: a bare selector that matches under more than one configured input root is flagged with a `WARN` (it still runs everything that matched), and a selector that matches nothing emits a `WARN` naming where it looked. Warnings are deduped by text across the orchestrator + per-file build/run passes (`emitSelectorWarnings`), so each prints once per invocation. Folder selectors (`rfc/`) and `,`-joined bare names (`a,b`) are recognized; selectors with an internal path separator (e.g. the orchestrator's own `assembly/__tests__/foo.spec.ts`) are still treated as direct cwd paths, preserving existing per-file dispatch.
+
 ## 2026-05-30 - v1.5.1
 
 ### An early-exiting runtime now fails instead of warning

package/bin/commands/build-core.js

@@ -22,6 +22,7 @@ import {
 } from "../util.js";
 import { persistCrashRecord } from "../crash-store.js";
 import { BuildWorkerPool } from "../build-worker-pool.js";
+import { resolveSpecFiles, emitSelectorWarnings } from "../selectors.js";
 const DEFAULT_CONFIG_PATH = path.join(process.cwd(), "./as-test.config.json");
 export const buildRecorderStorage = new AsyncLocalStorage();
 export class BuildFailureError extends Error {
@@ -67,14 +68,9 @@ export async function build(
   const pkgRunner = getPkgRunner();
   const sourceInputPatterns =
     overrides.kind === "fuzz" ? config.fuzz.input : config.input;
-  const inputPatterns = resolveInputPatterns(sourceInputPatterns, selectors);
-  const includePatterns = inputPatterns.filter((p) => !p.startsWith("!"));
-  const ignorePatterns = inputPatterns
-    .filter((p) => p.startsWith("!"))
-    .map((p) => p.slice(1));
-  const inputFiles = (
-    await glob(includePatterns, { ignore: ignorePatterns })
-  ).sort((a, b) => a.localeCompare(b));
+  const { files: inputFiles, warnings: selectorWarnings } =
+    await resolveSpecFiles(sourceInputPatterns, selectors);
+  emitSelectorWarnings(selectorWarnings);
   await assertNoArtifactCollisions(sourceInputPatterns);
   warnOnUnknownModeReferences(inputFiles, loadedConfig.modes ?? {});
   const coverageEnabled = resolveCoverageEnabled(
@@ -540,61 +536,6 @@ async function assertNoArtifactCollisions(configured) {
     seen.set(artifact, file);
   }
 }
-function resolveInputPatterns(configured, selectors) {
-  const configuredInputs = Array.isArray(configured)
-    ? configured
-    : [configured];
-  if (!selectors.length) return configuredInputs;
-  const patterns = new Set();
-  for (const selector of expandSelectors(selectors)) {
-    if (!selector) continue;
-    if (isBareSuiteSelector(selector)) {
-      const base = stripSuiteSuffix(selector);
-      for (const configuredInput of configuredInputs) {
-        patterns.add(
-          path.join(path.dirname(configuredInput), `${base}.spec.ts`),
-        );
-      }
-      continue;
-    }
-    patterns.add(selector);
-  }
-  return [...patterns];
-}
-function expandSelectors(selectors) {
-  const expanded = [];
-  for (const selector of selectors) {
-    if (!selector) continue;
-    if (!shouldSplitSelector(selector)) {
-      expanded.push(selector);
-      continue;
-    }
-    for (const token of selector.split(",")) {
-      const trimmed = token.trim();
-      if (!trimmed.length) continue;
-      expanded.push(trimmed);
-    }
-  }
-  return expanded;
-}
-function shouldSplitSelector(selector) {
-  return (
-    selector.includes(",") &&
-    !selector.includes("/") &&
-    !selector.includes("\\") &&
-    !/[*?[\]{}]/.test(selector)
-  );
-}
-function isBareSuiteSelector(selector) {
-  return (
-    !selector.includes("/") &&
-    !selector.includes("\\") &&
-    !/[*?[\]{}]/.test(selector)
-  );
-}
-function stripSuiteSuffix(selector) {
-  return selector.replace(/\.spec\.ts$/, "").replace(/\.ts$/, "");
-}
 function ensureDeps(config) {
   if (config.buildOptions.target == "wasi") {
     if (!resolveWasiShim()) {

package/bin/commands/run-core.js

@@ -1,6 +1,5 @@
 import chalk from "chalk";
 import { spawn } from "child_process";
-import { glob } from "glob";
 import { minimatch } from "minimatch";
 import { Channel, MessageType } from "../wipc.js";
 import {
@@ -21,6 +20,7 @@ import { PassThrough } from "stream";
 import { buildWebRunnerSource } from "./web-runner-source.js";
 import { PersistentWebSessionHost } from "./web-session.js";
 import { build } from "./build-core.js";
+import { resolveSpecFiles, emitSelectorWarnings } from "../selectors.js";
 import { createReporter as createDefaultReporter } from "../reporters/default.js";
 import { createTapReporter } from "../reporters/tap.js";
 import { persistCrashRecord } from "../crash-store.js";
@@ -733,14 +733,9 @@ export async function run(
       }
     }
   }
-  const inputPatterns = resolveInputPatterns(config.input, selectors);
-  const includePatterns = inputPatterns.filter((p) => !p.startsWith("!"));
-  const ignorePatterns = inputPatterns
-    .filter((p) => p.startsWith("!"))
-    .map((p) => p.slice(1));
-  const inputFiles = (
-    await glob(includePatterns, { ignore: ignorePatterns })
-  ).sort((a, b) => a.localeCompare(b));
+  const { files: inputFiles, warnings: selectorWarnings } =
+    await resolveSpecFiles(config.input, selectors);
+  emitSelectorWarnings(selectorWarnings);
   const snapshotEnabled = flags.snapshot !== false;
   const createSnapshots = Boolean(flags.createSnapshots);
   const overwriteSnapshots = Boolean(flags.overwriteSnapshots);
@@ -1364,61 +1359,6 @@ function runtimeNameFromCommand(command) {
   const token = command.trim().split(/\s+/)[0];
   return token && token.length ? token : "runtime";
 }
-function resolveInputPatterns(configured, selectors) {
-  const configuredInputs = Array.isArray(configured)
-    ? configured
-    : [configured];
-  if (!selectors.length) return configuredInputs;
-  const patterns = new Set();
-  for (const selector of expandSelectors(selectors)) {
-    if (!selector) continue;
-    if (isBareSuiteSelector(selector)) {
-      const base = stripSuiteSuffix(selector);
-      for (const configuredInput of configuredInputs) {
-        patterns.add(
-          path.join(path.dirname(configuredInput), `${base}.spec.ts`),
-        );
-      }
-      continue;
-    }
-    patterns.add(selector);
-  }
-  return [...patterns];
-}
-function expandSelectors(selectors) {
-  const expanded = [];
-  for (const selector of selectors) {
-    if (!selector) continue;
-    if (!shouldSplitSelector(selector)) {
-      expanded.push(selector);
-      continue;
-    }
-    for (const token of selector.split(",")) {
-      const trimmed = token.trim();
-      if (!trimmed.length) continue;
-      expanded.push(trimmed);
-    }
-  }
-  return expanded;
-}
-function shouldSplitSelector(selector) {
-  return (
-    selector.includes(",") &&
-    !selector.includes("/") &&
-    !selector.includes("\\") &&
-    !/[*?[\]{}]/.test(selector)
-  );
-}
-function isBareSuiteSelector(selector) {
-  return (
-    !selector.includes("/") &&
-    !selector.includes("\\") &&
-    !/[*?[\]{}]/.test(selector)
-  );
-}
-function stripSuiteSuffix(selector) {
-  return selector.replace(/\.spec\.ts$/, "").replace(/\.ts$/, "");
-}
 function normalizeReport(raw) {
   if (Array.isArray(raw)) {
     return {

package/bin/index.js

@@ -41,6 +41,7 @@ import { BuildWorkerPool } from "./build-worker-pool.js";
 import { PersistentWebSessionHost } from "./commands/web-session.js";
 import { buildRecorderStorage } from "./commands/build-core.js";
 import { DependencyGraph } from "./dependency-graph.js";
+import { resolveSpecFiles, emitSelectorWarnings } from "./selectors.js";
 const _args = process.argv.slice(2);
 const flags = [];
 const args = [];
@@ -3855,9 +3856,9 @@ async function resolveSelectedFiles(configPath, selectors, warn = true) {
   const resolvedConfigPath =
     configPath ?? path.join(process.cwd(), "./as-test.config.json");
   const config = loadConfig(resolvedConfigPath, warn);
-  const patterns = resolveInputPatterns(config.input, selectors);
-  const matches = await glob(patterns);
-  const specs = matches.filter((file) => file.endsWith(".spec.ts"));
+  const { files, warnings } = await resolveSpecFiles(config.input, selectors);
+  if (warn) emitSelectorWarnings(warnings);
+  const specs = files.filter((file) => file.endsWith(".spec.ts"));
   return [...new Set(specs)].sort((a, b) => a.localeCompare(b));
 }
 async function resolveSelectedFuzzFiles(
@@ -3996,27 +3997,6 @@ function levenshteinDistance(left, right) {
   }
   return matrix[left.length][right.length];
 }
-function resolveInputPatterns(configured, selectors) {
-  const configuredInputs = Array.isArray(configured)
-    ? configured
-    : [configured];
-  if (!selectors.length) return configuredInputs;
-  const patterns = new Set();
-  for (const selector of expandSelectors(selectors)) {
-    if (!selector) continue;
-    if (isBareSuiteSelector(selector)) {
-      const base = stripSuiteSuffix(selector);
-      for (const configuredInput of configuredInputs) {
-        patterns.add(
-          path.join(path.dirname(configuredInput), `${base}.spec.ts`),
-        );
-      }
-      continue;
-    }
-    patterns.add(selector);
-  }
-  return [...patterns];
-}
 function resolveFuzzPatterns(configured, selectors) {
   const configuredInputs = Array.isArray(configured)
     ? configured

package/bin/selectors.js

@@ -0,0 +1,208 @@
+import { glob } from "glob";
+import chalk from "chalk";
+import * as path from "path";
+// Selector resolution runs in several places per command (the orchestrator,
+// then build/run cores per file); dedupe by text so a warning prints once per
+// process regardless of how many resolvers see the same selector.
+const reportedSelectorWarnings = new Set();
+export function emitSelectorWarnings(warnings) {
+  for (const warning of warnings) {
+    if (reportedSelectorWarnings.has(warning)) continue;
+    reportedSelectorWarnings.add(warning);
+    process.stderr.write(`${chalk.yellow.bold("WARN")} ${warning}\n`);
+  }
+}
+const GLOB_MAGIC = /[*?[\]{}]/;
+function hasGlobMagic(selector) {
+  return GLOB_MAGIC.test(selector);
+}
+function endsWithSlash(selector) {
+  return /[\\/]$/.test(selector);
+}
+function stripTrailingSlash(selector) {
+  return selector.replace(/[\\/]+$/, "");
+}
+function stripSuiteSuffix(selector) {
+  return selector.replace(/\.spec\.ts$/, "").replace(/\.ts$/, "");
+}
+function isCwdRelative(selector) {
+  return (
+    selector.startsWith("./") ||
+    selector.startsWith("../") ||
+    selector.startsWith(".\\") ||
+    selector.startsWith("..\\") ||
+    selector.startsWith("/") ||
+    selector.startsWith("~") ||
+    path.isAbsolute(selector)
+  );
+}
+// A selector with a path separator that is not merely a single trailing slash
+// (e.g. `assembly/__tests__/foo.spec.ts`, passed verbatim by the orchestrator)
+// is treated as a direct cwd-relative path rather than a test-folder alias.
+function hasInternalSlash(selector) {
+  return /[\\/]/.test(stripTrailingSlash(selector));
+}
+// The longest leading run of path segments containing no glob magic — the
+// static "test folder" of an input pattern (`assembly/__tests__/**/*.spec.ts`
+// -> `assembly/__tests__`).
+function globBase(pattern) {
+  const segments = pattern.split("/");
+  const base = [];
+  for (const segment of segments) {
+    if (hasGlobMagic(segment)) break;
+    base.push(segment);
+  }
+  return base.join("/") || ".";
+}
+function uniqueInputRoots(configuredInputs) {
+  const roots = new Set();
+  for (const pattern of configuredInputs) {
+    if (pattern.startsWith("!")) continue;
+    roots.add(globBase(pattern));
+  }
+  return [...roots];
+}
+// Turn a cwd-relative selector into the spec glob(s) it stands for.
+function cwdPatterns(selector) {
+  if (endsWithSlash(selector)) {
+    return [`${stripTrailingSlash(selector)}/**/*.spec.ts`];
+  }
+  if (/\.ts$/.test(selector)) return [selector];
+  return [`${stripSuiteSuffix(selector)}.spec.ts`];
+}
+// Turn a bare selector into the spec glob(s) it stands for, anchored to a
+// configured input root and searched recursively beneath it. A selector that
+// already carries glob magic (`rfc/*.spec.ts`, `*.spec.ts`) is appended
+// verbatim so the user's pattern controls the match; a plain folder/name has
+// the spec suffix supplied.
+function barePatterns(root, selector) {
+  if (hasGlobMagic(selector)) {
+    return [`${root}/**/${selector}`];
+  }
+  if (endsWithSlash(selector)) {
+    return [`${root}/**/${stripTrailingSlash(selector)}/**/*.spec.ts`];
+  }
+  return [`${root}/**/${stripSuiteSuffix(selector)}.spec.ts`];
+}
+// Split comma-joined bare selectors (`a,b,c`) while leaving paths and globs
+// (which can legitimately contain commas, e.g. `{a,b}`) intact.
+function expandSelectors(selectors) {
+  const expanded = [];
+  for (const selector of selectors) {
+    if (!selector) continue;
+    if (
+      selector.includes(",") &&
+      !hasInternalSlash(selector) &&
+      !endsWithSlash(selector) &&
+      !hasGlobMagic(selector)
+    ) {
+      for (const token of selector.split(",")) {
+        const trimmed = token.trim();
+        if (trimmed.length) expanded.push(trimmed);
+      }
+      continue;
+    }
+    expanded.push(selector);
+  }
+  return expanded;
+}
+async function globFiles(patterns) {
+  return glob(patterns);
+}
+async function resolveSelector(selector, inputRoots) {
+  const warnings = [];
+  const isGlob = hasGlobMagic(selector);
+  // Explicit cwd-relative selector (`./`, `../`, absolute, `~`) — resolve from
+  // the cwd only. A glob is matched verbatim; a plain path gets the spec suffix.
+  if (isCwdRelative(selector)) {
+    const files = await globFiles(isGlob ? [selector] : cwdPatterns(selector));
+    if (!files.length) {
+      const bare = selector.replace(/^\.[\\/]/, "");
+      let suggestion = null;
+      for (const root of inputRoots) {
+        const inRoot = await globFiles(barePatterns(root, bare));
+        if (inRoot.length) {
+          suggestion = bare;
+          break;
+        }
+      }
+      warnings.push(
+        suggestion
+          ? `"${selector}" not found relative to the current directory — did you mean "${suggestion}" (searches the configured test folder)?`
+          : `"${selector}" not found relative to the current directory`,
+      );
+    }
+    return { files, warnings };
+  }
+  // A plain path with an internal separator (e.g. the orchestrator's own
+  // `assembly/__tests__/foo.spec.ts`) resolves from the cwd verbatim. Globs
+  // skip this and fall through to test-folder anchoring below.
+  if (!isGlob && hasInternalSlash(selector)) {
+    const direct = await globFiles(cwdPatterns(selector));
+    if (direct.length) return { files: direct, warnings };
+    // Fall through to test-folder resolution for user shorthands like
+    // `nested/array` that aren't a real cwd path.
+  }
+  // Bare name/folder or relative glob — configured input root(s) first.
+  const perRoot = [];
+  for (const root of inputRoots) {
+    const files = await globFiles(barePatterns(root, selector));
+    if (files.length) perRoot.push({ root, files });
+  }
+  if (perRoot.length) {
+    if (perRoot.length > 1) {
+      warnings.push(
+        `selector "${selector}" matched specs under ${perRoot.length} input roots (${perRoot
+          .map((entry) => entry.root)
+          .join(", ")}) — running all of them`,
+      );
+    }
+    return { files: perRoot.flatMap((entry) => entry.files), warnings };
+  }
+  // Fall back to the cwd before giving up.
+  const cwdFiles = await globFiles(
+    isGlob ? [selector] : cwdPatterns(`./${selector}`),
+  );
+  if (cwdFiles.length) {
+    return { files: cwdFiles, warnings };
+  }
+  warnings.push(
+    inputRoots.length
+      ? `no spec files matched "${selector}" in ${inputRoots.join(", ")} or the current directory`
+      : `no spec files matched "${selector}"`,
+  );
+  return { files: [], warnings };
+}
+// Resolve configured input patterns + positional selectors into the concrete
+// set of spec files to act on, along with any human-readable warnings. With no
+// selectors this is just the configured globs (honoring `!`-negations); with
+// selectors the per-selector rules above apply and config negations are
+// intentionally bypassed so an explicit pick always wins.
+export async function resolveSpecFiles(configured, selectors) {
+  const configuredInputs = Array.isArray(configured)
+    ? configured
+    : [configured];
+  if (!selectors.length) {
+    const include = configuredInputs.filter((p) => !p.startsWith("!"));
+    const ignore = configuredInputs
+      .filter((p) => p.startsWith("!"))
+      .map((p) => p.slice(1));
+    const files = (await glob(include, { ignore })).sort((a, b) =>
+      a.localeCompare(b),
+    );
+    return { files, warnings: [] };
+  }
+  const inputRoots = uniqueInputRoots(configuredInputs);
+  const files = new Set();
+  const warnings = [];
+  for (const selector of expandSelectors(selectors)) {
+    if (!selector) continue;
+    const resolved = await resolveSelector(selector, inputRoots);
+    for (const file of resolved.files) files.add(file);
+    warnings.push(...resolved.warnings);
+  }
+  return {
+    files: [...files].sort((a, b) => a.localeCompare(b)),
+    warnings,
+  };
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "as-test",
-  "version": "1.5.1",
+  "version": "1.5.2",
   "author": "Jairus Tanaka",
   "repository": {
     "type": "git",