@vitest-agent/plugin 2.0.16 → 2.2.0

package/index.d.ts

@@ -976,8 +976,16 @@ declare function combineClassifiers(...fns: ReadonlyArray<ClassifyFn>): Classify
  * Async file walker that returns matched absolute paths.
  *
  * Walks `dir` recursively via `node:fs/promises`. Skips `node_modules`, `.git`,
- * and `dist` directories. Matches files against the supplied glob patterns
- * relative to `dir` (e.g. `"src/**\/*.test.ts"`).
+ * and `dist` directories, and does not descend past a nested `package.json`
+ * boundary (any directory other than `dir` itself that has its own
+ * `package.json` is treated as an independent unit — its files belong to a
+ * separate discovery pass, not this one). The package-boundary rule applies
+ * to every supplied pattern, not just an unanchored `**\/` one — an anchored
+ * pattern like `"src/**\/*.test.ts"` will not match a test file under a
+ * nested package.json even though the pattern itself never reaches past
+ * `src/`, because the boundary check runs once per directory, independent of
+ * which pattern is being matched. Matches files against the supplied glob
+ * patterns relative to `dir` (e.g. `"src/**\/*.test.ts"`).
  *
  * Returns an empty array if `dir` does not exist or no files match.
  * @param dir - Absolute path to the directory to walk

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@vitest-agent/plugin",
-	"version": "2.0.16",
+	"version": "2.2.0",
 	"private": false,
 	"description": "Vitest plugin for the vitest-agent ecosystem: owns persistence, classification, baselines, trends, and dispatches rendering to a configurable reporter.",
 	"keywords": [
@@ -42,10 +42,10 @@
 		"@effect/platform-node": "4.0.0-beta.107",
 		"@effect/sql-sqlite-node": "4.0.0-beta.107",
 		"@effected/workspaces": "^0.11.1",
-		"@vitest-agent/cli": "2.0.16",
-		"@vitest-agent/mcp": "2.1.1",
-		"@vitest-agent/reporter": "2.0.16",
-		"@vitest-agent/sdk": "2.0.16",
+		"@vitest-agent/cli": "2.1.0",
+		"@vitest-agent/mcp": "2.1.3",
+		"@vitest-agent/reporter": "2.0.18",
+		"@vitest-agent/sdk": "2.2.0",
 		"effect": "4.0.0-beta.107",
 		"magic-string": "^1.1.0"
 	},

package/plugin.js

@@ -9,7 +9,7 @@ import { injectTags } from "./utils/inject-tags.js";
 import { isBenignViteSourceMapWarning } from "./utils/is-benign-vite-source-map-warning.js";
 import { stripConsoleReporters } from "./utils/strip-console-reporters.js";
 import { execSync } from "node:child_process";
-import { AgentConsoleMode, CiConsoleMode, CoverageLevel, EnvironmentDetector, EnvironmentDetectorLive, HumanConsoleMode, formatFatalError, resolveLogLevel } from "@vitest-agent/sdk";
+import { AgentConsoleMode, CiConsoleMode, CoverageLevel, EnvironmentDetector, EnvironmentDetectorLive, HumanConsoleMode, SRC_DIR, TEST_DIR, formatFatalError, isTestFileName, resolveLogLevel } from "@vitest-agent/sdk";
 import { Effect, Schema } from "effect";
 
 //#region src/plugin.ts
@@ -98,10 +98,9 @@ const aggregatedReporterByVitest = /* @__PURE__ */ new WeakSet();
 *
 * @public
 */
-const CURRENT_PLUGIN_VERSION = "2.0.16";
-const TEST_FILE_SUFFIX_RE = /\.(?:test|spec)\.(?:ts|tsx|js|jsx)$/;
-const TEST_FILE_DIR_RE = /\/(?:src|__test__)\//;
-const isTestFile = (id) => TEST_FILE_SUFFIX_RE.test(id) && TEST_FILE_DIR_RE.test(id);
+const CURRENT_PLUGIN_VERSION = "2.2.0";
+const TEST_FILE_DIR_RE = new RegExp(`/(?:${SRC_DIR}|${TEST_DIR})/`);
+const isTestFile = (id) => isTestFileName(id) && TEST_FILE_DIR_RE.test(id);
 /**
 * Map a detected {@link Environment} to its {@link Executor}. Inline copy
 * of the `ExecutorResolverLive` mapping so the plugin can compute it

package/reporter.js

@@ -7,7 +7,7 @@ import { captureSettings, hashSettings } from "./utils/capture-settings.js";
 import { processFailure } from "./utils/process-failure.js";
 import { routeRenderedOutput } from "./utils/route-rendered-output.js";
 import { stringifyFailureValue } from "./utils/stringify-failure-value.js";
-import { DataReader, DataStore, DetailResolver, EnvironmentDetector, ExecutorResolver, FormatSelector, HistoryTracker, OutputPipelineLive, PathResolutionLive, buildAgentReport, computeTrend, ensureMigrated, formatFatalError, historyKey, isTimeoutError, probeHostMetadataFromEnv, resolveDataPath, resolveLogFile, resolveLogLevel } from "@vitest-agent/sdk";
+import { DataReader, DataStore, DetailResolver, EnvironmentDetector, ExecutorResolver, FormatSelector, HistoryTracker, OutputPipelineLive, PathResolutionLive, buildAgentReport, coerceErrorField, computeTrend, ensureMigrated, formatFatalError, historyKey, isTimeoutError, probeHostMetadataFromEnv, resolveDataPath, resolveLogFile, resolveLogLevel } from "@vitest-agent/sdk";
 import { Effect, Option, PubSub } from "effect";
 import { randomUUID } from "node:crypto";
 import { mkdirSync } from "node:fs";
@@ -36,6 +36,21 @@ import { DefaultVitestAgentReporter } from "@vitest-agent/reporter";
 * @internal
 */
 const dbPathCache = /* @__PURE__ */ new Map();
+/**
+* Safely read a raw Vitest error object's `stacks` array. The property
+* access itself can throw on hostile shapes (live getters — the same
+* ConfigError class of failure `coerceErrorField` guards); frames are
+* dropped rather than crashing the persistence loop.
+*/
+function readErrorStacks(source) {
+	if (source === null || typeof source !== "object") return void 0;
+	try {
+		const value = source.stacks;
+		return Array.isArray(value) ? value : void 0;
+	} catch {
+		return;
+	}
+}
 function resolveDataPathCached(projectDir) {
 	const cached = dbPathCache.get(projectDir);
 	if (cached !== void 0) return cached;
@@ -263,7 +278,9 @@ var AgentReporter = class {
 	*/
 	async onInit(vitest) {
 		this._vitest = vitest;
-		await this.ensureDbPath();
+		try {
+			await this.ensureDbPath();
+		} catch {}
 		await this.initReporters();
 	}
 	/**
@@ -801,7 +818,9 @@ var AgentReporter = class {
 				failCount: fail,
 				skipCount: skip,
 				timeoutCount: timeout,
-				durationMs: totalDuration
+				durationMs: totalDuration,
+				// @vitest-agent/ui, which both populate `collectedModules` on their
+				collectedModules: testModules.length
 			});
 		}
 		const modules = testModules;
@@ -818,11 +837,11 @@ var AgentReporter = class {
 		};
 		const wantsRunEvents = this.wantsRunEvents();
 		let dbPath;
+		let persistDisabled;
 		try {
 			dbPath = await this.ensureDbPath();
 		} catch (err) {
-			process.stderr.write(`vitest-agent: ${formatFatalError(err)}\n`);
-			return;
+			persistDisabled = formatFatalError(err);
 		}
 		const filteredModules = opts.projectFilter ? modules.filter((m) => (m.project.name || "default") === opts.projectFilter) : modules;
 		if (filteredModules.length === 0 && opts.projectFilter) return;
@@ -886,14 +905,33 @@ var AgentReporter = class {
 			});
 			return;
 		}
-		mkdirSync(dirname(dbPath), { recursive: true });
+		if (dbPath !== void 0 && persistDisabled === void 0) try {
+			mkdirSync(dirname(dbPath), { recursive: true });
+		} catch (err) {
+			persistDisabled = formatFatalError(err);
+		}
+		let fallbackReports;
 		try {
-			await ensureMigrated(dbPath, logLevel, logFile);
+			const fallbackProjectGroups = /* @__PURE__ */ new Map();
+			for (const mod of filteredModules) {
+				const key = mod.project.name || "default";
+				const existing = fallbackProjectGroups.get(key);
+				if (existing) existing.push(mod);
+				else fallbackProjectGroups.set(key, [mod]);
+			}
+			const isMultiProjectFallback = fallbackProjectGroups.size > 1 || !!opts.projectFilter;
+			fallbackReports = [];
+			for (const [projectName, projectModules] of fallbackProjectGroups) fallbackReports.push(buildAgentReport(projectModules, errors, reason, { omitPassingTests: opts.omitPassingTests }, isMultiProjectFallback ? projectName : void 0));
 		} catch (err) {
 			process.stderr.write(`vitest-agent: ${formatFatalError(err)}\n`);
 			return;
 		}
-		const program = Effect.gen(function* () {
+		if (dbPath !== void 0 && persistDisabled === void 0) try {
+			await ensureMigrated(dbPath, logLevel, logFile);
+		} catch (err) {
+			persistDisabled = formatFatalError(err);
+		}
+		const persistProgram = Effect.gen(function* () {
 			const store = yield* DataStore;
 			const reader = yield* DataReader;
 			const analyzer = yield* CoverageAnalyzer;
@@ -977,6 +1015,7 @@ var AgentReporter = class {
 					conversationId: null
 				};
 				const hostProbe = probeHostMetadataFromEnv(process.env);
+				const projectReason = reason === "interrupted" ? "interrupted" : baseReport.summary.failed > 0 || baseReport.failedFiles.length > 0 ? "failed" : "passed";
 				const runId = yield* store.writeRun({
 					invocationId,
 					project,
@@ -984,7 +1023,7 @@ var AgentReporter = class {
 					timestamp: baseReport.timestamp,
 					commitSha: process.env.GITHUB_SHA ?? null,
 					branch: process.env.GITHUB_REF_NAME ?? null,
-					reason,
+					reason: projectReason,
 					duration: totalDuration,
 					total: baseReport.summary.total,
 					passed: baseReport.summary.passed,
@@ -1063,7 +1102,17 @@ var AgentReporter = class {
 							const inputs = [];
 							for (let ordinal = 0; ordinal < result.errors.length; ordinal++) {
 								const e = result.errors[ordinal];
-								const { frames, signatureHash } = processFailure(e);
+								const messageText = coerceErrorField(e, "message") ?? "<missing message>";
+								const nameText = coerceErrorField(e, "name");
+								const diffText = coerceErrorField(e, "diff");
+								const stackText = coerceErrorField(e, "stack");
+								const stacksValue = readErrorStacks(e);
+								const { frames, signatureHash } = processFailure({
+									message: messageText,
+									...nameText !== void 0 && { name: nameText },
+									...stackText !== void 0 && { stack: stackText },
+									...stacksValue !== void 0 && { stacks: stacksValue }
+								});
 								if (signatureHash !== null) yield* store.writeFailureSignature({
 									signatureHash,
 									runId,
@@ -1072,10 +1121,10 @@ var AgentReporter = class {
 								inputs.push({
 									testCaseId,
 									scope: "test",
-									message: e.message,
-									...e.name !== void 0 && { name: e.name },
-									...e.diff !== void 0 && { diff: e.diff },
-									...e.stack !== void 0 && { stack: e.stack },
+									message: messageText,
+									...nameText !== void 0 && { name: nameText },
+									...diffText !== void 0 && { diff: diffText },
+									...stackText !== void 0 && { stack: stackText },
 									...signatureHash !== null && { signatureHash },
 									...frames.length > 0 && { frames },
 									ordinal
@@ -1090,7 +1139,16 @@ var AgentReporter = class {
 						const inputs = [];
 						for (let ordinal = 0; ordinal < modErrors.length; ordinal++) {
 							const e = modErrors[ordinal];
-							const { frames, signatureHash } = processFailure(e);
+							const messageText = coerceErrorField(e, "message") ?? "<missing message>";
+							const nameText = coerceErrorField(e, "name");
+							const stackText = coerceErrorField(e, "stack");
+							const stacksValue = readErrorStacks(e);
+							const { frames, signatureHash } = processFailure({
+								message: messageText,
+								...nameText !== void 0 && { name: nameText },
+								...stackText !== void 0 && { stack: stackText },
+								...stacksValue !== void 0 && { stacks: stacksValue }
+							});
 							if (signatureHash !== null) yield* store.writeFailureSignature({
 								signatureHash,
 								runId,
@@ -1099,9 +1157,9 @@ var AgentReporter = class {
 							inputs.push({
 								moduleId,
 								scope: "module",
-								message: e.message,
-								...e.name !== void 0 && { name: e.name },
-								...e.stack !== void 0 && { stack: e.stack },
+								message: messageText,
+								...nameText !== void 0 && { name: nameText },
+								...stackText !== void 0 && { stack: stackText },
 								...signatureHash !== null && { signatureHash },
 								...frames.length > 0 && { frames },
 								ordinal
@@ -1114,7 +1172,16 @@ var AgentReporter = class {
 					const inputs = [];
 					for (let ordinal = 0; ordinal < errors.length; ordinal++) {
 						const e = errors[ordinal];
-						const { frames, signatureHash } = processFailure(e);
+						const messageText = coerceErrorField(e, "message") ?? "<missing message>";
+						const nameText = coerceErrorField(e, "name");
+						const stackText = coerceErrorField(e, "stack");
+						const stacksValue = readErrorStacks(e);
+						const { frames, signatureHash } = processFailure({
+							message: messageText,
+							...nameText !== void 0 && { name: nameText },
+							...stackText !== void 0 && { stack: stackText },
+							...stacksValue !== void 0 && { stacks: stacksValue }
+						});
 						if (signatureHash !== null) yield* store.writeFailureSignature({
 							signatureHash,
 							runId,
@@ -1122,9 +1189,9 @@ var AgentReporter = class {
 						});
 						inputs.push({
 							scope: "unhandled",
-							message: e.message,
-							...e.name !== void 0 && { name: e.name },
-							...e.stack !== void 0 && { stack: e.stack },
+							message: messageText,
+							...nameText !== void 0 && { name: nameText },
+							...stackText !== void 0 && { stack: stackText },
 							...signatureHash !== null && { signatureHash },
 							...frames.length > 0 && { frames },
 							ordinal
@@ -1155,7 +1222,7 @@ var AgentReporter = class {
 						const tcResult = tc.result();
 						if (tcResult?.state === "failed") {
 							const errors = tcResult.errors;
-							errorMap.set(key, errors?.[0]?.message ?? null);
+							errorMap.set(key, coerceErrorField(errors?.[0], "message") ?? null);
 						}
 					}
 				}
@@ -1291,10 +1358,29 @@ var AgentReporter = class {
 				direction: trendSummary.direction,
 				runCount: trendSummary.runCount
 			});
+			const classifications = /* @__PURE__ */ new Map();
+			for (const report of reports) for (const mod of report.failed) for (const test of mod.tests) if (test.classification) classifications.set(test.fullName, test.classification);
 			yield* Effect.logInfo("reports built").pipe(Effect.annotateLogs({
 				count: reports.length,
 				projects: Array.from(projectGroups.keys()).join(", ")
 			}));
+			return {
+				reports,
+				classifications,
+				...trendSummary !== void 0 && { trendSummary }
+			};
+		});
+		let persistFailure;
+		let persistResult;
+		if (persistDisabled === void 0 && dbPath !== void 0) persistResult = await Effect.runPromise(persistProgram.pipe(Effect.annotateLogs("service", "reporter"), Effect.provide(ReporterLive(dbPath, logLevel, logFile)))).catch((err) => {
+			persistFailure = formatFatalError(err);
+		});
+		const renderInputData = persistResult ?? {
+			reports: fallbackReports,
+			classifications: /* @__PURE__ */ new Map()
+		};
+		const renderProgram = Effect.gen(function* () {
+			const { reports, classifications, trendSummary } = renderInputData;
 			const detector = yield* EnvironmentDetector;
 			const executorResolver = yield* ExecutorResolver;
 			const formatSelector = yield* FormatSelector;
@@ -1316,8 +1402,6 @@ var AgentReporter = class {
 				format,
 				detail
 			}));
-			const classifications = /* @__PURE__ */ new Map();
-			for (const report of reports) for (const mod of report.failed) for (const test of mod.tests) if (test.classification) classifications.set(test.fullName, test.classification);
 			const githubSummaryFile = process.env.GITHUB_STEP_SUMMARY;
 			const kit = buildReporterKit({
 				env,
@@ -1350,9 +1434,11 @@ var AgentReporter = class {
 			}));
 			for (const output of allOutputs) routeRenderedOutput(output, { ...githubSummaryFile !== void 0 && { githubSummaryFile } });
 		});
-		await Effect.runPromise(program.pipe(Effect.annotateLogs("service", "reporter"), Effect.provide(ReporterLive(dbPath, logLevel, logFile)))).catch((err) => {
+		await Effect.runPromise(renderProgram.pipe(Effect.annotateLogs("service", "reporter"), Effect.provide(OutputPipelineLive), Effect.provide(NodeServices.layer))).catch((err) => {
 			process.stderr.write(`vitest-agent: ${formatFatalError(err)}\n`);
 		});
+		const persistError = persistDisabled ?? persistFailure;
+		if (persistError !== void 0) process.stderr.write(`vitest-agent: persistence failed — results above were rendered but NOT recorded: ${persistError}\n`);
 	}
 };
 

package/utils/discover-projects.js

@@ -1,5 +1,6 @@
 import { toPosixPath } from "./to-posix-path.js";
 import { DefaultDiscoverStrategy } from "./discover-strategy.js";
+import { NON_DISCOVERABLE_DIRS, SRC_DIR, TEST_DIR } from "@vitest-agent/sdk";
 import { isAbsolute, join, normalize, relative } from "node:path";
 import { readdir, stat } from "node:fs/promises";
 import { findWorkspaceRootSync, getWorkspacePackagesSync } from "@effected/workspaces";
@@ -17,47 +18,60 @@ const _cache = /* @__PURE__ */ new Map();
 * files between `discoverProjects` calls without re-walking test-file globs.
 * Returns an empty string when `dirPath` does not exist — this still produces
 * a stable, comparable signature contribution.
+*
+* Manual per-directory walk (not a single `readdir({ recursive: true })`
+* call) pruning `NON_DISCOVERABLE_DIRS` *before* recursing: a `src/` or
+* `__test__/` dir can itself contain a fixture `node_modules` (e.g. a
+* subprocess-e2e fixture that installs deps), and Node's recursive `readdir`
+* follows symlinked directories, so an unguarded call would walk into it.
 */
 async function computeDirSignature(dirPath) {
+	const parts = [];
+	await walkDirSignature(dirPath, dirPath, parts);
+	parts.sort();
+	return parts.join("|");
+}
+async function walkDirSignature(root, dir, parts) {
 	let entries;
 	try {
-		entries = await readdir(dirPath, {
-			withFileTypes: true,
-			recursive: true
-		});
+		entries = await readdir(dir, { withFileTypes: true });
 	} catch {
-		return "";
+		return;
 	}
-	const parts = [];
 	for (const ent of entries) {
-		if (!ent.isFile()) continue;
-		const parentPath = ent.parentPath ?? dirPath;
-		const fullPath = join(parentPath, ent.name);
-		let mtimeMs;
-		try {
-			mtimeMs = (await stat(fullPath)).mtimeMs;
-		} catch {
-			continue;
+		const fullPath = join(dir, ent.name);
+		if (ent.isDirectory()) {
+			if (NON_DISCOVERABLE_DIRS.has(ent.name)) continue;
+			await walkDirSignature(root, fullPath, parts);
+		} else if (ent.isFile()) {
+			let mtimeMs;
+			try {
+				mtimeMs = (await stat(fullPath)).mtimeMs;
+			} catch {
+				continue;
+			}
+			const relPath = toPosixPath(relative(root, fullPath));
+			parts.push(`${relPath}:${mtimeMs}`);
 		}
-		const relPath = toPosixPath(relative(dirPath, fullPath));
-		parts.push(`${relPath}:${mtimeMs}`);
 	}
-	parts.sort();
-	return parts.join("|");
 }
 /**
-* Computes a cheap whole-workspace directory signature by combining each
-* package's `src/` and `__test__/` signatures (issue #100). Only entries +
-* mtimes are read — no file content — so this stays fast even for large
-* monorepos. A changed signature means a test file was added, removed,
-* moved, or renamed since the cached result was computed.
+* Computes a cheap whole-workspace directory signature from each package's
+* `src/` and `__test__/` directories (issue #100). Only entries + mtimes are
+* read — no file content — so this stays fast for large monorepos. A changed
+* signature means a test file was added, removed, moved, or renamed since
+* the cached result was computed.
+*
+* Only the two anchored directories are fingerprinted. Discovery cannot see a
+* test file anywhere else, so nothing else can change the emitted project list
+* (issue #227).
 */
 async function computeWorkspaceSignature(packages) {
 	const parts = [];
 	for (const pkg of packages) {
-		const srcSig = await computeDirSignature(join(pkg.path, "src"));
-		const testDirSig = await computeDirSignature(join(pkg.path, "__test__"));
-		parts.push(`${pkg.path}::src=${srcSig}::__test__=${testDirSig}`);
+		const srcSig = await computeDirSignature(join(pkg.path, SRC_DIR));
+		const testSig = await computeDirSignature(join(pkg.path, TEST_DIR));
+		parts.push(`${pkg.path}::src=${srcSig}::__test__=${testSig}`);
 	}
 	return parts.join("\n");
 }

package/utils/discover-strategy.js

@@ -1,5 +1,6 @@
 import { findTestFiles } from "./find-test-files.js";
 import { Tag } from "./tag.js";
+import { NON_DISCOVERABLE_DIRS, SRC_DIR, TEST_DIR, TEST_FILE_GLOB_SUFFIX, TEST_HELPER_DIRS } from "@vitest-agent/sdk";
 import { join, sep } from "node:path";
 import { stat } from "node:fs/promises";
 import { configDefaults } from "vitest/config";
@@ -11,18 +12,6 @@ const SETUP_EXTS = [
 	"js",
 	"jsx"
 ];
-const TEST_DIR_HELPER_DIRS = [
-	"utils",
-	"fixtures",
-	"snapshots"
-];
-async function isDir(p) {
-	try {
-		return (await stat(p)).isDirectory();
-	} catch {
-		return false;
-	}
-}
 async function isFile(p) {
 	try {
 		return (await stat(p)).isFile();
@@ -123,17 +112,19 @@ var DefaultDiscoverStrategy = class extends DiscoverStrategy {
 		return ["unit"];
 	}
 	async buildProject(input) {
-		const hasTestDir = await isDir(join(input.path, "__test__"));
-		const srcPrefix = join(input.path, "src");
-		const testPrefix = join(input.path, "__test__");
-		const allFiles = await findTestFiles(input.path, ["src/**/*.{test,spec}.{ts,tsx,js,jsx}", "__test__/**/*.{test,spec}.{ts,tsx,js,jsx}"]);
+		const srcPrefix = join(input.path, SRC_DIR);
+		const allFiles = await findTestFiles(input.path, [`${SRC_DIR}/**/${TEST_FILE_GLOB_SUFFIX}`, `${TEST_DIR}/**/${TEST_FILE_GLOB_SUFFIX}`]);
 		if (allFiles.length === 0) return null;
 		const hasSrcTests = allFiles.some((f) => f.startsWith(`${srcPrefix}${sep}`) || f === srcPrefix);
-		const hasTestDirTests = allFiles.some((f) => f.startsWith(`${testPrefix}${sep}`) || f === testPrefix);
+		const hasTestDirTests = allFiles.some((f) => !(f.startsWith(`${srcPrefix}${sep}`) || f === srcPrefix));
 		const include = [];
-		if (hasSrcTests) include.push(join(input.path, "src/**/*.{test,spec}.{ts,tsx,js,jsx}"));
-		if (hasTestDirTests) include.push(join(input.path, "__test__/**/*.{test,spec}.{ts,tsx,js,jsx}"));
-		const exclude = hasTestDir ? [...configDefaults.exclude, ...TEST_DIR_HELPER_DIRS.map((d) => join(input.path, `__test__/${d}/**`))] : void 0;
+		if (hasSrcTests) include.push(join(input.path, SRC_DIR, "**", TEST_FILE_GLOB_SUFFIX));
+		if (hasTestDirTests) include.push(join(input.path, TEST_DIR, "**", TEST_FILE_GLOB_SUFFIX));
+		const exclude = [
+			...configDefaults.exclude,
+			...[SRC_DIR, TEST_DIR].flatMap((root) => [...NON_DISCOVERABLE_DIRS].map((d) => join(input.path, root, "**", d, "**"))),
+			...hasTestDirTests ? TEST_HELPER_DIRS.map((d) => join(input.path, TEST_DIR, "**", d, "**")) : []
+		];
 		const setupFile = await detectSetupFile(input.path);
 		return {
 			extends: true,
@@ -141,7 +132,7 @@ var DefaultDiscoverStrategy = class extends DiscoverStrategy {
 				name: input.name,
 				environment: "node",
 				include,
-				...exclude ? { exclude } : {},
+				exclude,
 				...setupFile ? { setupFiles: [join(input.path, setupFile)] } : {}
 			}
 		};

package/utils/find-test-files.js

@@ -1,13 +1,9 @@
 import { toPosixPath } from "./to-posix-path.js";
+import { NON_DISCOVERABLE_DIRS } from "@vitest-agent/sdk";
 import { join, relative } from "node:path";
 import { readdir } from "node:fs/promises";
 
 //#region src/utils/find-test-files.ts
-const SKIP_DIRS = /* @__PURE__ */ new Set([
-	"node_modules",
-	".git",
-	"dist"
-]);
 function globToRegex(pattern) {
 	const alts = expandBraces(pattern).map(toRegexFragment);
 	return new RegExp(`^(?:${alts.join("|")})$`);
@@ -57,8 +53,16 @@ function toRegexFragment(glob) {
 * Async file walker that returns matched absolute paths.
 *
 * Walks `dir` recursively via `node:fs/promises`. Skips `node_modules`, `.git`,
-* and `dist` directories. Matches files against the supplied glob patterns
-* relative to `dir` (e.g. `"src/**\/*.test.ts"`).
+* and `dist` directories, and does not descend past a nested `package.json`
+* boundary (any directory other than `dir` itself that has its own
+* `package.json` is treated as an independent unit — its files belong to a
+* separate discovery pass, not this one). The package-boundary rule applies
+* to every supplied pattern, not just an unanchored `**\/` one — an anchored
+* pattern like `"src/**\/*.test.ts"` will not match a test file under a
+* nested package.json even though the pattern itself never reaches past
+* `src/`, because the boundary check runs once per directory, independent of
+* which pattern is being matched. Matches files against the supplied glob
+* patterns relative to `dir` (e.g. `"src/**\/*.test.ts"`).
 *
 * Returns an empty array if `dir` does not exist or no files match.
 * @param dir - Absolute path to the directory to walk
@@ -80,8 +84,9 @@ async function walkDir(root, dir, matchers, results) {
 	} catch {
 		return;
 	}
+	if (dir !== root && entries.some((ent) => ent.isFile() && ent.name === "package.json")) return;
 	for (const ent of entries) {
-		if (SKIP_DIRS.has(ent.name)) continue;
+		if (NON_DISCOVERABLE_DIRS.has(ent.name)) continue;
 		const fullPath = join(dir, ent.name);
 		if (ent.isDirectory()) await walkDir(root, fullPath, matchers, results);
 		else if (ent.isFile()) {

package/utils/stringify-failure-value.js

@@ -28,7 +28,11 @@ const stringifyFailureValue = (value) => {
 	try {
 		return JSON.stringify(value);
 	} catch {
-		return String(value);
+		try {
+			return String(value);
+		} catch {
+			return "<unserializable value>";
+		}
 	}
 };