@vitest-agent/mcp 2.1.1 → 2.1.2

package/index.d.ts

@@ -533,6 +533,7 @@ declare const appRouter: import("@trpc/server").TRPCBuiltRouter<{
           readonly failed: number;
           readonly skipped: number;
           readonly duration: number;
+          readonly modules?: number | undefined;
         };
         readonly failed: readonly {
           readonly file: string;

package/index.js

@@ -12,7 +12,7 @@ import { parseSessionEnvExports, recoverSessionContextFromSessionEnv } from "./s
 *
 * @public
 */
-const CURRENT_MCP_VERSION = "2.1.1";
+const CURRENT_MCP_VERSION = "2.1.2";
 
 //#endregion
 export { CURRENT_MCP_VERSION, McpLive, appRouter, buildMcpServer, createCallerFactory, createCurrentSessionIdRef, createSessionContextRef, parseSessionEnvExports, recoverSessionContextFromSessionEnv, startMcpServer };

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@vitest-agent/mcp",
-	"version": "2.1.1",
+	"version": "2.1.2",
 	"private": false,
 	"description": "Model Context Protocol server for vitest-agent. Exposes 53 tools for agent access to test data, TDD lifecycle, and session management.",
 	"keywords": [
@@ -42,7 +42,7 @@
 		"@effect/sql-sqlite-node": "4.0.0-beta.107",
 		"@modelcontextprotocol/sdk": "^1.30.0",
 		"@trpc/server": "^11.18.0",
-		"@vitest-agent/sdk": "2.0.16",
+		"@vitest-agent/sdk": "2.1.0",
 		"effect": "4.0.0-beta.107",
 		"zod": "^4.4.3"
 	},

package/tools/run-tests.js

@@ -1,7 +1,10 @@
 import { publicProcedure } from "../context.js";
-import { AgentReport, DataReader, DataStore, buildAgentReport, buildConsoleLeaks, collectConsoleLeakEntries } from "@vitest-agent/sdk";
+import { AgentReport, DataReader, DataStore, buildAgentReport, buildConsoleLeaks, coerceErrorField, collectConsoleLeakEntries } from "@vitest-agent/sdk";
 import { Effect, Schema, SchemaGetter } from "effect";
 import { AsyncLocalStorage } from "node:async_hooks";
+import { mkdtempSync, rmSync } from "node:fs";
+import { tmpdir } from "node:os";
+import { join } from "node:path";
 import { Writable } from "node:stream";
 
 //#region src/tools/run-tests.ts
@@ -84,6 +87,27 @@ function readDiscoveryLastScannedAt() {
 	const value = globalThis[DISCOVERY_LAST_SCAN_SYMBOL];
 	return typeof value === "string" ? value : void 0;
 }
+/**
+* Per-invocation coverage temp directory. Vitest's v8 provider `rm -rf`s the
+* shared `coverage/` reports directory at run start (`clean: true` default),
+* so two concurrent runs in one checkout destroy each other's `.tmp` files
+* (ENOENT coverage-N.json — issues #159/#191/#194). Each MCP run gets its own
+* directory; coverage persistence is unaffected because the analyzer consumes
+* the in-memory CoverageMap, never the disk artifacts. Consequence: final
+* coverage artifacts (html/lcov) from MCP-driven runs land in the throwaway
+* dir instead of `./coverage` — acceptable because MCP consumers read
+* coverage from SQLite (`CoverageAnalyzer` consumes the in-memory map via
+* `onCoverage`), not from disk artifacts.
+*
+* @internal exported for tests
+*/
+const makeCoverageDirOverride = () => {
+	const dir = mkdtempSync(join(tmpdir(), "vitest-agent-cov-"));
+	return {
+		dir,
+		coverage: { reportsDirectory: dir }
+	};
+};
 const stdoutSinkStorage = new AsyncLocalStorage();
 const stderrSinkStorage = new AsyncLocalStorage();
 let _stdioPatched = false;
@@ -305,10 +329,13 @@ const runTests = publicProcedure.input(Schema.toStandardSchemaV1(Schema.Struct({
 	} });
 	const { createVitest } = await import("vitest/node");
 	let vitest;
+	let covOverride;
 	try {
+		covOverride = makeCoverageDirOverride();
 		vitest = await createVitest("test", {
 			root: ctx.cwd,
 			run: true,
+			coverage: covOverride.coverage,
 			...project ? { project } : {},
 			...resolvedExpression !== null ? { tagsFilter: [resolvedExpression] } : {},
 			...input.passWithNoTests !== void 0 ? { passWithNoTests: input.passWithNoTests } : {}
@@ -334,8 +361,8 @@ const runTests = publicProcedure.input(Schema.toStandardSchemaV1(Schema.Struct({
 				resolvedExpression
 			}
 		};
-		const reason = unhandledErrors.length > 0 || result.testModules.some((m) => m.state() === "failed") ? "failed" : "passed";
-		const baseReport = buildAgentReport(testModules, unhandledErrors, reason, { omitPassingTests: true });
+		const preliminaryReason = unhandledErrors.length > 0 || result.testModules.some((m) => m.state() === "failed") ? "failed" : "passed";
+		const baseReport = buildAgentReport(testModules, unhandledErrors, preliminaryReason, { omitPassingTests: true });
 		const leaks = buildConsoleLeaks(collectConsoleLeakEntries(localVitest.state.getFiles()));
 		const report = leaks !== void 0 ? {
 			...baseReport,
@@ -369,19 +396,34 @@ const runTests = publicProcedure.input(Schema.toStandardSchemaV1(Schema.Struct({
 			discoveryLastScannedAt: readDiscoveryLastScannedAt() ?? null
 		};
 	} catch (err) {
-		if (err instanceof Error && err.message === "VITEST_TIMEOUT") return {
-			kind: "timeout",
-			timeoutSeconds: input.timeout ?? 120
-		};
+		let message;
+		try {
+			if (err instanceof Error && err.message === "VITEST_TIMEOUT") return {
+				kind: "timeout",
+				timeoutSeconds: input.timeout ?? 120
+			};
+			message = err instanceof Error ? err.message : String(err);
+		} catch {
+			message = coerceErrorField(err, "message") ?? "<unserializable error>";
+		}
 		return {
 			kind: "error",
-			message: err instanceof Error ? err.message : String(err)
+			message
 		};
 	} finally {
-		await vitest?.close();
-		nullStream.destroy();
+		try {
+			await vitest?.close();
+		} finally {
+			nullStream.destroy();
+			if (covOverride !== void 0) try {
+				rmSync(covOverride.dir, {
+					recursive: true,
+					force: true
+				});
+			} catch {}
+		}
 	}
 }));
 
 //#endregion
-export { RunTestsAsMarkdown, RunTestsResult, coerceErrors, composeTagExpression, formatNoMatchMarkdown, formatReportMarkdown, formatRunTestsMarkdown, readDiscoveryLastScannedAt, runTests, sanitizeTestArgs, withStdioCaptured };
+export { RunTestsAsMarkdown, RunTestsResult, coerceErrors, composeTagExpression, formatNoMatchMarkdown, formatReportMarkdown, formatRunTestsMarkdown, makeCoverageDirOverride, readDiscoveryLastScannedAt, runTests, sanitizeTestArgs, withStdioCaptured };