@vitest-agent/plugin 5.0.4 → 5.1.1

package/index.d.ts

@@ -3,8 +3,8 @@ import { AgentPluginOptions, AgentReporterOptions, ConsoleMode, CoverageBaseline
 import { Context, Effect, Layer, Option } from "effect";
 import { TestProjectInlineConfiguration, TestTagDefinition } from "vitest/config";
 import { ResolvedConfig, VitestPluginContext } from "vitest/node";
-import { SourceMap } from "magic-string";
 import { WorkspacesSyncOptions } from "@effected/workspaces/node-sync";
+import { SourceMap } from "magic-string";
 import { MigrationError } from "effect/sql/Migrator";
 import { SqlError } from "effect/sql/SqlError";
 //#region src/utils/tag.d.ts
@@ -236,6 +236,50 @@ export declare class DefaultDiscoverStrategy extends DiscoverStrategy {
   extend(options: DiscoverStrategyExtendOptions): DiscoverStrategy;
 }
 //#endregion
+//#region src/utils/discover-projects.d.ts
+/**
+ * The resolved output of `discoverProjects` — projects and tag definitions ready for `defineConfig`.
+ * @public
+ */
+interface DiscoverProjectsResult {
+  readonly projects: TestProjectInlineConfiguration[] | undefined;
+  readonly tags: TestTagDefinition[];
+}
+/**
+ * Options for `discoverProjects`.
+ * @public
+ */
+interface DiscoverProjectsOptions {
+  readonly strategy?: DiscoverStrategy;
+  readonly cwd?: string;
+  readonly additionalEntries?: ReadonlyArray<{
+    readonly name: string;
+    readonly path: string;
+  }>;
+  /**
+   * Filesystem port the test-file and signature walks read through. Defaults
+   * to `node:fs`; tests inject a virtual volume instead of a real tmp tree.
+   */
+  readonly fs?: WalkerFileSystem;
+  /**
+   * Sync operations `@effected/workspaces` resolves the workspace root and
+   * package list through. Defaults to its `nodeSyncOps` binding.
+   */
+  readonly syncOps?: WorkspacesSyncOptions;
+  /**
+   * Optional traversal depth override for workspace package enumeration.
+   * Forwarded to `@effected/workspaces`' `getWorkspacePackagesSync`.
+   */
+  readonly maxDepth?: number;
+}
+/**
+ * Scan all workspace packages and additional entries through the active strategy and return projects + tags.
+ * @param options - Optional strategy, working directory, and extra project entries
+ * @returns Resolved projects and tag definitions
+ * @public
+ */
+export declare function discoverProjects(options?: DiscoverProjectsOptions): Promise<DiscoverProjectsResult>;
+//#endregion
 //#region src/utils/inject-tags.d.ts
 /**
  * Return value of the Vite `transform` hook — the rewritten source code and its source map.
@@ -1118,50 +1162,6 @@ export declare class ConfigurationError extends Error {
   readonly name = "ConfigurationError";
 }
 //#endregion
-//#region src/utils/discover-projects.d.ts
-/**
- * The resolved output of `discoverProjects` — projects and tag definitions ready for `defineConfig`.
- * @public
- */
-interface DiscoverProjectsResult {
-  readonly projects: TestProjectInlineConfiguration[] | undefined;
-  readonly tags: TestTagDefinition[];
-}
-/**
- * Options for `discoverProjects`.
- * @public
- */
-interface DiscoverProjectsOptions {
-  readonly strategy?: DiscoverStrategy;
-  readonly cwd?: string;
-  readonly additionalEntries?: ReadonlyArray<{
-    readonly name: string;
-    readonly path: string;
-  }>;
-  /**
-   * Filesystem port the test-file and signature walks read through. Defaults
-   * to `node:fs`; tests inject a virtual volume instead of a real tmp tree.
-   */
-  readonly fs?: WalkerFileSystem;
-  /**
-   * Sync operations `@effected/workspaces` resolves the workspace root and
-   * package list through. Defaults to its `nodeSyncOps` binding.
-   */
-  readonly syncOps?: WorkspacesSyncOptions;
-  /**
-   * Optional traversal depth override for workspace package enumeration.
-   * Forwarded to `@effected/workspaces`' `getWorkspacePackagesSync`.
-   */
-  readonly maxDepth?: number;
-}
-/**
- * Scan all workspace packages and additional entries through the active strategy and return projects + tags.
- * @param options - Optional strategy, working directory, and extra project entries
- * @returns Resolved projects and tag definitions
- * @public
- */
-export declare function discoverProjects(options?: DiscoverProjectsOptions): Promise<DiscoverProjectsResult>;
-//#endregion
 //#region src/utils/classify-helpers.d.ts
 /**
  * Creates a ClassifyFn that maps filename suffix patterns to tag arrays.
@@ -1433,9 +1433,17 @@ interface VitestErrorLike {
  * Returns `null` for the signature when no usable top frame is found
  * (error has no stack, or every frame is in framework code). Frames may
  * still be populated even when the signature is null.
+ *
+ * @param error - The Vitest error to process.
+ * @param options - Optional overrides. `readSource` reads the top frame's
+ *   source file as UTF-8 text for function-boundary detection; it defaults to
+ *   `node:fs`'s `readFileSync`, and a throw is treated as an unreadable file
+ *   (the signature falls back to the raw-line bucket).
  * @public
  */
-export declare const processFailure: (error: VitestErrorLike) => {
+export declare const processFailure: (error: VitestErrorLike, options?: {
+  readonly readSource?: (path: string) => string;
+}) => {
   frames: ReadonlyArray<StackFrameInput>;
   signatureHash: string | null;
 };

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@vitest-agent/plugin",
-	"version": "5.0.4",
+	"version": "5.1.1",
 	"private": false,
 	"description": "Vitest plugin for the vitest-agent ecosystem: owns persistence, classification, baselines, trends, and dispatches rendering to a configurable reporter.",
 	"keywords": [
@@ -46,9 +46,9 @@
 		"@effect/platform-node": "4.0.0-rc.118",
 		"@effected/glob": "^0.9.0",
 		"@effected/workspaces": "^0.30.2",
-		"@vitest-agent/cli": "3.2.4",
-		"@vitest-agent/engine": "0.2.11",
-		"@vitest-agent/mcp": "5.1.2",
+		"@vitest-agent/cli": "3.2.6",
+		"@vitest-agent/engine": "0.3.1",
+		"@vitest-agent/mcp": "5.1.4",
 		"@vitest-agent/reporter": "3.1.3",
 		"@vitest-agent/sdk": "5.1.3",
 		"effect": "4.0.0-rc.118",

package/plugin.js

@@ -357,6 +357,11 @@ const buildPreset = (thresholdsLevel, targetsLevel, perFile) => Object.freeze({
 * `strategy` is provided AND `additionalEntries` is empty. Any `.addProject()`
 * chain or explicit strategy bypasses the cache.
 *
+* Every option — including the `fs` / `syncOps` filesystem ports that
+* `AgentPlugin.discover` never sets — is forwarded unchanged to
+* `discoverProjects`, so tests can drive the builder over a virtual volume.
+* Exported for tests only; not part of the package barrel.
+*
 * @internal
 */
 function makeDiscoverBuilder(options) {
@@ -446,4 +451,4 @@ function makeDiscoverBuilder(options) {
 })(AgentPlugin || (AgentPlugin = {}));
 
 //#endregion
-export { AgentPlugin, CURRENT_PLUGIN_VERSION, resolveConsoleMode };
+export { AgentPlugin, CURRENT_PLUGIN_VERSION, makeDiscoverBuilder, resolveConsoleMode };

package/utils/build-module-info.js

@@ -3,12 +3,16 @@ import { existsSync, readFileSync } from "node:fs";
 import { basename, dirname, relative } from "node:path";
 
 //#region src/utils/build-module-info.ts
+const nodeBuildModuleInfoFs = {
+	exists: (path) => existsSync(path),
+	readFile: (path) => readFileSync(path, "utf8")
+};
 const NOT_FOUND = {
 	packageName: "",
 	packagePath: ""
 };
 const cache = /* @__PURE__ */ new Map();
-const resolvePackageInfo = (filePath) => {
+const resolvePackageInfo = (filePath, fs) => {
 	let dir = dirname(filePath);
 	const visited = [];
 	while (true) {
@@ -19,8 +23,8 @@ const resolvePackageInfo = (filePath) => {
 		}
 		visited.push(dir);
 		const pkgJsonPath = `${dir}/package.json`;
-		if (existsSync(pkgJsonPath)) try {
-			const pkg = JSON.parse(readFileSync(pkgJsonPath, "utf8"));
+		if (fs.exists(pkgJsonPath)) try {
+			const pkg = JSON.parse(fs.readFile(pkgJsonPath));
 			const result = {
 				packageName: typeof pkg.name === "string" ? pkg.name : "",
 				packagePath: dir
@@ -44,11 +48,17 @@ const resolvePackageInfo = (filePath) => {
 *
 * Query strings (Vite virtual module suffixes like `?v=1234`) are stripped
 * before the walk so the cache key is always a clean filesystem path.
+*
+* @param filePath - Module id, optionally carrying a Vite query suffix.
+* @param fs - Filesystem the `package.json` walk reads through. Defaults to
+*   `node:fs`; tests pass a virtual volume. The cache is shared across ports
+*   and keyed only by directory, so a test injecting a volume should use
+*   roots no other caller walks, or call {@link clearBuildModuleInfoCache}.
 */
-const buildModuleInfo = (filePath) => {
+const buildModuleInfo = (filePath, fs = nodeBuildModuleInfoFs) => {
 	const queryIndex = filePath.indexOf("?");
 	const cleanId = queryIndex === -1 ? filePath : filePath.slice(0, queryIndex);
-	const { packageName, packagePath } = resolvePackageInfo(cleanId);
+	const { packageName, packagePath } = resolvePackageInfo(cleanId, fs);
 	return {
 		path: cleanId,
 		relativePath: toPosixPath(relative(process.cwd(), cleanId)),

package/utils/process-failure.js

@@ -22,9 +22,10 @@ const parseFramesFromStackString = (stack) => {
 	}
 	return frames;
 };
-const readSourceSafe = (filePath) => {
+const readSourceNode = (filePath) => readFileSync(filePath, "utf-8");
+const readSourceSafe = (filePath, readSource) => {
 	try {
-		return readFileSync(filePath, "utf-8");
+		return readSource(filePath);
 	} catch {
 		return null;
 	}
@@ -36,9 +37,15 @@ const readSourceSafe = (filePath) => {
 * Returns `null` for the signature when no usable top frame is found
 * (error has no stack, or every frame is in framework code). Frames may
 * still be populated even when the signature is null.
+*
+* @param error - The Vitest error to process.
+* @param options - Optional overrides. `readSource` reads the top frame's
+*   source file as UTF-8 text for function-boundary detection; it defaults to
+*   `node:fs`'s `readFileSync`, and a throw is treated as an unreadable file
+*   (the signature falls back to the raw-line bucket).
 * @public
 */
-const processFailure = (error) => {
+const processFailure = (error, options) => {
 	const rawFrames = error.stacks !== void 0 && error.stacks.length > 0 ? error.stacks.map((f, ordinal) => ({
 		ordinal,
 		method: f.method ?? null,
@@ -52,7 +59,7 @@ const processFailure = (error) => {
 	let topFunctionName = null;
 	if (topFrame !== void 0) {
 		const lineForBoundary = topFrame.line;
-		const source = readSourceSafe(topFrame.filePath);
+		const source = readSourceSafe(topFrame.filePath, options?.readSource ?? readSourceNode);
 		const boundary = source !== null ? findFunctionBoundary(source, lineForBoundary) : null;
 		if (boundary !== null) {
 			topBoundaryLine = boundary.line;

package/version.js

@@ -8,7 +8,7 @@
 *
 * @public
 */
-const CURRENT_PLUGIN_VERSION = "5.0.4";
+const CURRENT_PLUGIN_VERSION = "5.1.1";
 
 //#endregion
 export { CURRENT_PLUGIN_VERSION };