as-test 1.1.4 → 1.1.6-patch.1

package/CHANGELOG.md

@@ -1,5 +1,29 @@
 # Change Log
 
+## 2026-05-19 - v1.1.6-patch.1
+
+### Directory-preserving artifact layout
+
+- feat: build artifacts, fuzz artifacts, snapshots, readable logs, coverage logs, and crash records now mirror the source tree under the configured input globs instead of being flattened into a single directory with a `____`-mangled disambiguator suffix. For `assembly/__tests__/nested/array.spec.ts` the artifact is `outDir/<mode>/nested/array.spec.wasm` (previously `outDir/<mode>/array.<mode>.<target>.assembly____tests____nested.wasm`). Sidecar `.js` / `.d.ts` files sit next to the wasm.
+- feat: filename simplified to `<stem>.wasm` — the `.<mode>.<target>` suffix has been dropped since the mode is already a directory level and the target is implied by the mode config.
+- feat: add `resolveGlobBase`, `resolveSpecRelativePath`, and `resolveArtifactPath` in `cli/util.ts` as the shared path helpers used by every code path that writes or looks up a per-spec artifact. Glob bases are computed component-wise (so `assembly/__tests` is not a prefix of `assembly/__tests__/foo.spec.ts`) and the longest matching base wins when multiple configured input patterns overlap. Patterns without a static prefix (e.g. `**/*.spec.ts`) fall back to the file's cwd-relative path.
+- feat: add an up-front collision check in `build()` that throws a clear error naming both source files when two configured inputs would resolve to the same artifact path.
+- fix: `ast test <one-spec>`, `ast run <one-spec>`, and `ast fuzz <one-spec>` no longer drop the disambiguator when only one of two same-basename files is being built. The build side and the runner side now compute the same path from the same configured input set, so the runner no longer reports `bindings artifact not found` after a single-file build.
+- fix: build sites now `mkdir -p` the artifact's parent directory before invoking `asc` — pinned `assemblyscript@0.28.17`'s `-o` flag does not create parents and would otherwise ENOENT for any new nested directory.
+- fix: `persistCrashRecord` now `mkdir -p`'s the entry's parent directory, supporting `/` in entry keys so nested specs and fuzz failures get their own crash files instead of clobbering by basename.
+- fix: replace the hardcoded `/__tests__/` and `/__fuzz__/` markers in snapshot and readable-log path resolution with proper glob-base computation, so projects with custom input layouts (e.g. `tests/specs/**/*.spec.ts`) now nest correctly instead of silently falling back to basename-only paths.
+- fix: clean break on snapshot file layout — the legacy `${base}.snap.json` and `${base}.${disambiguator}.snap.json` fallbacks have been removed. After upgrading, run `--create-snapshots` or `--overwrite-snapshots` once so snapshots are written at their new relative-path locations.
+- chore: delete the duplicate-basename plumbing introduced by the patch immediately preceding this one. Every code path that used to thread `Set<string>` of duplicate basenames now reads paths directly. No back-compat for the old flat-with-suffix scheme — `ast clean` removes any orphan artifacts.
+
+## 2026-05-14 - v1.1.6
+
+- fix: coverage `mode` and `dependencies` filtering now correctly handles AssemblyScript-normalized `~lib/<pkg>/...` paths, which are the actual runtime paths emitted for `node_modules` imports.
+- fix: `ENTRY_FILE` injected by the transform now uses the full relative path instead of the basename, preventing snapshot key collisions between specs with the same filename in different directories; snapshot lookup normalizes the file prefix to maintain backward compatibility with existing `.snap` files.
+- fix: transform visitor and coverage instrumentation now resolve `NodeKind` values at runtime instead of relying on compile-time const enum inlining, so they remain correct across AssemblyScript versions.
+- fix: add a no-op `TupleType` case to the transform visitor so files using tuple types no longer throw during instrumentation.
+- fix: coverage transform no longer wraps `return this` in constructors, preventing AS231 ("A class with a constructor explicitly returning something else than 'this' must be '@final'").
+- fix: coverage transform preserves expression-body arrows instead of converting them to block bodies, preventing TS1140 ("Type argument expected") on typed arrow parameters such as `[1,2,3].map((x: i32) => x + 1)`.
+
 ## 2026-05-14 - v1.1.4
 
 - feat: add `coverage.mode` (`project` or `all`) plus `coverage.dependencies` package allowlisting so dependency coverage can include normal or pnpm-installed packages without raw path globs.

package/assembly/__fuzz__/nested/array.fuzz.ts

@@ -0,0 +1,13 @@
+import { expect, fuzz, FuzzSeed } from "as-test";
+
+fuzz(
+  "nested array.fuzz: index windows stay ordered",
+  (start: i32, end: i32): bool => {
+    expect(start <= end).toBe(true);
+    return end - start <= 64;
+  },
+).generate((seed: FuzzSeed, run: (start: i32, end: i32) => bool): void => {
+  const start = seed.i32({ min: 0, max: 64 });
+  const width = seed.i32({ min: 0, max: 64 });
+  run(start, start + width);
+});

package/bin/commands/build-core.js

@@ -1,10 +1,10 @@
-import { existsSync } from "fs";
+import { existsSync, mkdirSync } from "fs";
 import { glob } from "glob";
 import chalk from "chalk";
 import { spawn } from "child_process";
 import * as path from "path";
 import { createMemoryStream, main as ascMain, } from "assemblyscript/dist/asc.js";
-import { applyMode, getPkgRunner, loadConfig, tokenizeCommand, resolveProjectModule, } from "../util.js";
+import { applyMode, getPkgRunner, loadConfig, resolveArtifactPath, resolveSpecRelativePath, tokenizeCommand, resolveProjectModule, } from "../util.js";
 import { persistCrashRecord } from "../crash-store.js";
 import { BuildWorkerPool } from "../build-worker-pool.js";
 const DEFAULT_CONFIG_PATH = path.join(process.cwd(), "./as-test.config.json");
@@ -36,9 +36,10 @@ export async function build(configPath = DEFAULT_CONFIG_PATH, selectors = [], mo
         ensureDeps(config);
     }
     const pkgRunner = getPkgRunner();
-    const inputPatterns = resolveInputPatterns(config.input, selectors);
+    const sourceInputPatterns = overrides.kind === "fuzz" ? config.fuzz.input : config.input;
+    const inputPatterns = resolveInputPatterns(sourceInputPatterns, selectors);
     const inputFiles = (await glob(inputPatterns)).sort((a, b) => a.localeCompare(b));
-    const duplicateSpecBasenames = resolveDuplicateBasenames(inputFiles);
+    await assertNoArtifactCollisions(sourceInputPatterns);
     const coverageEnabled = resolveCoverageEnabled(config.coverage, featureToggles.coverage);
     const buildEnv = {
         ...mode.env,
@@ -50,7 +51,7 @@ export async function build(configPath = DEFAULT_CONFIG_PATH, selectors = [], mo
         !hasCustomBuildCommand(config)) {
         const pool = getSerialBuildWorkerPool();
         for (const file of inputFiles) {
-            const outFile = `${config.outDir}/${resolveArtifactFileName(file, config.buildOptions.target, modeName, duplicateSpecBasenames)}`;
+            const outFile = path.join(config.outDir, resolveArtifactPath(file, sourceInputPatterns));
             const invocation = getBuildCommand(config, pkgRunner, file, outFile, modeName, featureToggles);
             await pool.buildFileMode({
                 configPath,
@@ -64,7 +65,8 @@ export async function build(configPath = DEFAULT_CONFIG_PATH, selectors = [], mo
         return;
     }
     for (const file of inputFiles) {
-        const outFile = `${config.outDir}/${resolveArtifactFileName(file, config.buildOptions.target, modeName, duplicateSpecBasenames)}`;
+        const outFile = path.join(config.outDir, resolveArtifactPath(file, sourceInputPatterns));
+        mkdirSync(path.dirname(outFile), { recursive: true });
         const invocation = getBuildCommand(config, pkgRunner, file, outFile, modeName, featureToggles);
         try {
             await buildFile(invocation, buildEnv);
@@ -79,6 +81,7 @@ export async function build(configPath = DEFAULT_CONFIG_PATH, selectors = [], mo
                 kind,
                 stage: "build",
                 file,
+                entryKey: resolveSpecRelativePath(file, sourceInputPatterns).replace(/\.ts$/i, ""),
                 mode: modeLabel,
                 cwd: process.cwd(),
                 buildCommand,
@@ -127,8 +130,8 @@ export async function getBuildInvocationPreview(configPath = DEFAULT_CONFIG_PATH
     if (overrides.args?.length) {
         config.buildOptions.args = [...config.buildOptions.args, ...overrides.args];
     }
-    const duplicateSpecBasenames = resolveDuplicateBasenames([file]);
-    const outFile = `${config.outDir}/${resolveArtifactFileName(file, config.buildOptions.target, modeName, duplicateSpecBasenames)}`;
+    const sourceInputPatterns = overrides.kind === "fuzz" ? config.fuzz.input : config.input;
+    const outFile = path.join(config.outDir, resolveArtifactPath(file, sourceInputPatterns));
     return getBuildCommand(config, getPkgRunner(), file, outFile, modeName, featureToggles);
 }
 export async function getBuildReuseInfo(configPath = DEFAULT_CONFIG_PATH, file, modeName, featureToggles = {}, overrides = {}) {
@@ -145,8 +148,8 @@ export async function getBuildReuseInfo(configPath = DEFAULT_CONFIG_PATH, file,
     if (hasCustomBuildCommand(config)) {
         return null;
     }
-    const duplicateSpecBasenames = resolveDuplicateBasenames([file]);
-    const outFile = `${config.outDir}/${resolveArtifactFileName(file, config.buildOptions.target, modeName, duplicateSpecBasenames)}`;
+    const sourceInputPatterns = overrides.kind === "fuzz" ? config.fuzz.input : config.input;
+    const outFile = path.join(config.outDir, resolveArtifactPath(file, sourceInputPatterns));
     const invocation = getBuildCommand(config, getPkgRunner(), file, outFile, modeName, featureToggles);
     const coverageEnabled = resolveCoverageEnabled(config.coverage, featureToggles.coverage);
     const buildEnv = {
@@ -245,46 +248,21 @@ function expandBuildCommand(template, file, outFile, target, modeName) {
         .replace(/<target>/g, target)
         .replace(/<mode>/g, modeName ?? "");
 }
-function resolveArtifactFileName(file, target, modeName, duplicateSpecBasenames = new Set()) {
-    const base = path
-        .basename(file)
-        .replace(/\.spec\.ts$/, "")
-        .replace(/\.ts$/, "");
-    const legacy = !modeName
-        ? `${path.basename(file).replace(".ts", ".wasm")}`
-        : `${base}.${modeName}.${target}.wasm`;
-    if (!duplicateSpecBasenames.has(path.basename(file))) {
-        return legacy;
-    }
-    const disambiguator = resolveDisambiguator(file);
-    if (!disambiguator.length) {
-        return legacy;
-    }
-    const ext = path.extname(legacy);
-    const stem = ext.length ? legacy.slice(0, -ext.length) : legacy;
-    return `${stem}.${disambiguator}${ext}`;
-}
-function resolveDuplicateBasenames(files) {
-    const counts = new Map();
+async function assertNoArtifactCollisions(configured) {
+    const patterns = Array.isArray(configured) ? configured : [configured];
+    const files = await glob(patterns);
+    const seen = new Map();
     for (const file of files) {
-        const base = path.basename(file);
-        counts.set(base, (counts.get(base) ?? 0) + 1);
-    }
-    const duplicates = new Set();
-    for (const [base, count] of counts) {
-        if (count > 1)
-            duplicates.add(base);
-    }
-    return duplicates;
-}
-function resolveDisambiguator(file) {
-    const relDir = path.dirname(path.relative(process.cwd(), file));
-    if (!relDir.length || relDir == ".")
-        return "";
-    return relDir
-        .replace(/[\\/]+/g, "__")
-        .replace(/[^A-Za-z0-9._-]/g, "_")
-        .replace(/^_+|_+$/g, "");
+        const artifact = resolveArtifactPath(file, patterns);
+        const prev = seen.get(artifact);
+        if (prev != null && prev !== file) {
+            throw new Error(`Two input files resolve to the same artifact path "${artifact}":\n` +
+                `  - ${prev}\n` +
+                `  - ${file}\n` +
+                `Rename one of them or narrow the input patterns to disambiguate.`);
+        }
+        seen.set(artifact, file);
+    }
 }
 function resolveInputPatterns(configured, selectors) {
     const configuredInputs = Array.isArray(configured)

package/bin/commands/fuzz-core.js

@@ -3,7 +3,7 @@ import * as path from "path";
 import { pathToFileURL } from "url";
 import { glob } from "glob";
 import { build } from "./build-core.js";
-import { applyMode, loadConfig } from "../util.js";
+import { applyMode, loadConfig, resolveArtifactPath, resolveSpecRelativePath, } from "../util.js";
 import { persistCrashRecord } from "../crash-store.js";
 const DEFAULT_CONFIG_PATH = path.join(process.cwd(), "./as-test.config.json");
 const MAGIC = Buffer.from("WIPC");
@@ -19,14 +19,13 @@ export async function fuzz(configPath = DEFAULT_CONFIG_PATH, selectors = [], mod
     if (!inputFiles.length) {
         throw new Error(`No fuzz files matched: ${selectors.length ? selectors.join(", ") : "configured input patterns"}`);
     }
-    const duplicateBasenames = resolveDuplicateBasenames(inputFiles);
     const results = [];
     for (const file of inputFiles) {
         const buildStartedAt = Date.now();
         await build(configPath, [file], modeName, { coverage: false }, { target: "bindings", args: ["--use", "AS_TEST_FUZZ=1"], kind: "fuzz" }, loadedConfig);
         const buildFinishedAt = Date.now();
         const buildTime = buildFinishedAt - buildStartedAt;
-        results.push(await runFuzzTarget(file, activeConfig.outDir, duplicateBasenames, config, fuzzerSelectors, buildStartedAt, buildFinishedAt, buildTime, modeName));
+        results.push(await runFuzzTarget(file, activeConfig.outDir, config, fuzzerSelectors, buildStartedAt, buildFinishedAt, buildTime, modeName));
     }
     return results;
 }
@@ -70,9 +69,9 @@ function encodeRunsOverrideKind(kind) {
             return 4;
     }
 }
-async function runFuzzTarget(file, outDir, duplicateBasenames, config, fuzzerSelectors, buildStartedAt, buildFinishedAt, buildTime, modeName) {
+async function runFuzzTarget(file, outDir, config, fuzzerSelectors, buildStartedAt, buildFinishedAt, buildTime, modeName) {
     const startedAt = Date.now();
-    const artifact = resolveArtifactFileName(file, duplicateBasenames, modeName);
+    const artifact = resolveArtifactPath(file, config.input);
     const wasmPath = path.resolve(process.cwd(), outDir, artifact);
     const jsPath = resolveBindingsHelperPath(wasmPath);
     const helper = await import(pathToFileURL(jsPath).href + `?t=${Date.now()}`);
@@ -140,7 +139,7 @@ async function runFuzzTarget(file, outDir, duplicateBasenames, config, fuzzerSel
         const crash = persistCrashRecord(config.crashDir, {
             kind: "fuzz",
             file,
-            entryKey: buildFuzzCrashEntryKey(file, modeName ?? "default"),
+            entryKey: buildFuzzCrashEntryKey(file, config.input, modeName ?? "default"),
             mode: modeName ?? "default",
             seed: config.seed,
             error: crashMessage,
@@ -188,7 +187,7 @@ async function runFuzzTarget(file, outDir, duplicateBasenames, config, fuzzerSel
         const crash = persistCrashRecord(config.crashDir, {
             kind: "fuzz",
             file,
-            entryKey: buildFuzzCrashEntryKey(file, modeName ?? "default"),
+            entryKey: buildFuzzCrashEntryKey(file, config.input, modeName ?? "default"),
             mode: modeName ?? "default",
             seed: config.seed,
             error: `${reportParseError ? `invalid fuzz report payload: ${reportParseError}` : `missing fuzz report payload from ${path.basename(file)}`} (${diagnostics})`,
@@ -223,7 +222,7 @@ async function runFuzzTarget(file, outDir, duplicateBasenames, config, fuzzerSel
         const crash = persistCrashRecord(config.crashDir, {
             kind: "fuzz",
             file,
-            entryKey: buildFuzzFailureEntryKey(file, fuzzer.name, modeName ?? "default"),
+            entryKey: buildFuzzFailureEntryKey(file, config.input, fuzzer.name, modeName ?? "default"),
             mode: modeName ?? "default",
             seed: firstFailureSeed,
             reproCommand: buildFuzzReproCommand(file, firstFailureSeed, modeName ?? "default", fuzzer.selector, 1),
@@ -285,11 +284,13 @@ function buildFuzzReproCommand(file, seed, modeName, fuzzer, runs) {
     const runsArg = typeof runs == "number" ? ` --runs ${runs}` : "";
     return `ast fuzz ${file}${modeArg}${fuzzerArg} --seed ${seed}${runsArg}`;
 }
-function buildFuzzFailureEntryKey(file, name, modeName) {
-    return `${path.basename(file).replace(/\.ts$/, "")}.${sanitizeEntryName(modeName)}.${sanitizeEntryName(name)}`;
+function buildFuzzFailureEntryKey(file, inputPatterns, name, modeName) {
+    const stem = resolveSpecRelativePath(file, inputPatterns).replace(/\.ts$/i, "");
+    return `${stem}.${sanitizeEntryName(modeName)}.${sanitizeEntryName(name)}`;
 }
-function buildFuzzCrashEntryKey(file, modeName) {
-    return `${path.basename(file).replace(/\.ts$/, "")}.${sanitizeEntryName(modeName)}`;
+function buildFuzzCrashEntryKey(file, inputPatterns, modeName) {
+    const stem = resolveSpecRelativePath(file, inputPatterns).replace(/\.ts$/i, "");
+    return `${stem}.${sanitizeEntryName(modeName)}`;
 }
 function sanitizeEntryName(name) {
     return (name
@@ -398,47 +399,6 @@ function resolveFuzzInputPatterns(configured, selectors) {
     }
     return [...patterns];
 }
-function resolveArtifactFileName(file, duplicateBasenames, modeName) {
-    const base = path
-        .basename(file)
-        .replace(/\.spec\.ts$/, "")
-        .replace(/\.ts$/, "");
-    const legacy = !modeName
-        ? `${path.basename(file).replace(".ts", ".wasm")}`
-        : `${base}.${modeName}.bindings.wasm`;
-    if (!duplicateBasenames.has(path.basename(file))) {
-        return legacy;
-    }
-    const disambiguator = resolveDisambiguator(file);
-    if (!disambiguator.length) {
-        return legacy;
-    }
-    const ext = path.extname(legacy);
-    const stem = ext.length ? legacy.slice(0, -ext.length) : legacy;
-    return `${stem}.${disambiguator}${ext}`;
-}
-function resolveDuplicateBasenames(files) {
-    const counts = new Map();
-    for (const file of files) {
-        const base = path.basename(file);
-        counts.set(base, (counts.get(base) ?? 0) + 1);
-    }
-    const duplicates = new Set();
-    for (const [base, count] of counts) {
-        if (count > 1)
-            duplicates.add(base);
-    }
-    return duplicates;
-}
-function resolveDisambiguator(file) {
-    const relDir = path.dirname(path.relative(process.cwd(), file));
-    if (!relDir.length || relDir == ".")
-        return "";
-    return relDir
-        .replace(/[\\/]+/g, "__")
-        .replace(/[^A-Za-z0-9._-]/g, "_")
-        .replace(/^_+|_+$/g, "");
-}
 function resolveBindingsHelperPath(wasmPath) {
     const bindingsPath = wasmPath.replace(/\.wasm$/, ".bindings.js");
     if (existsSync(bindingsPath))