@vitest-agent/mcp 4.0.2 → 4.0.3

package/idempotency.js

@@ -3,6 +3,13 @@ import { DataReader, DataStore } from "@vitest-agent/engine";
 
 //#region src/idempotency.ts
 /**
+* Narrow a raw tool payload to the action-shaped object every consolidated
+* spec keys on: a non-null object carrying an `action` key. Anything else
+* (including a legacy non-action payload) yields `null`, which every spec
+* maps to "not idempotent".
+*/
+const asActionInput = (input) => input !== null && typeof input === "object" && "action" in input ? input : null;
+/**
 * Registered idempotency specs for mutation tools.
 *
 * `hypothesis validate` is covered (key: `validate:${id}:${outcome}`).
@@ -17,27 +24,17 @@ const idempotencyKeys = [
 	{
 		procedurePath: "hypothesis",
 		deriveKey: (input) => {
-			if (input === null || typeof input !== "object" || !("action" in input)) return null;
-			const i = input;
+			const i = asActionInput(input);
+			if (i === null) return null;
 			if (i.action === "validate" && typeof i.id === "number" && typeof i.outcome === "string") return `validate:${i.id}:${i.outcome}`;
 			return null;
 		}
 	},
-	{
-		procedurePath: "_legacy_hypothesis_validate",
-		deriveKey: (input) => {
-			if (input !== null && typeof input === "object" && "id" in input && "outcome" in input && typeof input.id === "number" && typeof input.outcome === "string") {
-				const i = input;
-				return `${i.id}:${i.outcome}`;
-			}
-			return null;
-		}
-	},
 	{
 		procedurePath: "tdd_task",
 		deriveKey: (input) => {
-			if (input === null || typeof input !== "object" || !("action" in input)) return null;
-			const i = input;
+			const i = asActionInput(input);
+			if (i === null) return null;
 			if (i.action === "start" && typeof i.goal === "string") {
 				if (typeof i.runId === "string") {
 					if (typeof i.sessionId === "number") return `start:sid:${i.sessionId}:run:${i.runId}`;
@@ -53,8 +50,8 @@ const idempotencyKeys = [
 	{
 		procedurePath: "tdd_goal",
 		deriveKey: (input) => {
-			if (input === null || typeof input !== "object" || !("action" in input)) return null;
-			const i = input;
+			const i = asActionInput(input);
+			if (i === null) return null;
 			if (i.action === "create" && typeof i.tddTaskId === "number" && typeof i.goal === "string") return `create:${i.tddTaskId}:${i.goal}`;
 			return null;
 		}
@@ -62,8 +59,8 @@ const idempotencyKeys = [
 	{
 		procedurePath: "tdd_behavior",
 		deriveKey: (input) => {
-			if (input === null || typeof input !== "object" || !("action" in input)) return null;
-			const i = input;
+			const i = asActionInput(input);
+			if (i === null) return null;
 			if (i.action === "create" && typeof i.goalId === "number" && typeof i.behavior === "string") return `create:${i.goalId}:${i.behavior}`;
 			return null;
 		}

package/index.d.ts

@@ -1,5 +1,5 @@
 import { Context, Effect, Layer, Schema, Stdio } from "effect";
-import { DataReader, DataStore, OutputRenderer, ProjectDiscovery, SessionContext } from "@vitest-agent/engine";
+import { DataReader, DataStore, ProjectDiscovery, SessionContext } from "@vitest-agent/engine";
 import { McpSchema, Tool, Toolkit } from "effect/unstable/ai";
 import "@vitest-agent/sdk";
 //#region src/annotations.d.ts
@@ -192,7 +192,7 @@ interface ServerLayerOptions {
  * @param options - the advertised server version
  * @public
  */
-export declare const ServerLayer: (options: ServerLayerOptions) => Layer.Layer<never, never, PlatformServices | McpSession | DataReader | DataStore | ProjectDiscovery | OutputRenderer>;
+export declare const ServerLayer: (options: ServerLayerOptions) => Layer.Layer<never, never, PlatformServices | McpSession | DataReader | DataStore | ProjectDiscovery>;
 //#endregion
 //#region src/tools/ping.d.ts
 /**

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@vitest-agent/mcp",
-	"version": "4.0.2",
+	"version": "4.0.3",
 	"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": [
@@ -45,8 +45,8 @@
 	"dependencies": {
 		"@effect/platform-node": "4.0.0-rc.115",
 		"@effect/sql-sqlite-node": "4.0.0-rc.115",
-		"@vitest-agent/engine": "0.1.2",
-		"@vitest-agent/sdk": "5.0.0",
+		"@vitest-agent/engine": "0.1.3",
+		"@vitest-agent/sdk": "5.0.1",
 		"effect": "4.0.0-rc.115"
 	},
 	"peerDependencies": {

package/tools/run-tests.js

@@ -224,18 +224,15 @@ function findConfigInDir(dir) {
 	}
 	return null;
 }
-function dirHasVitestOrViteConfig(dir) {
-	return findConfigInDir(dir) !== null;
-}
 /**
-* Walk UP from `startDir` looking for the vitest/vite config file, returning
-* its absolute path. Same walk and same git-root bound as
-* `resolveConfigAnchoredRoot`; returns `null` when no config is found in
-* range or when anything about the walk throws.
-*
-* @internal exported for tests
+* The one walk both anchoring helpers share: step UP from `startDir`
+* until a vitest/vite config file is found, returning its absolute path.
+* Bounded at the git root (inclusive — the directory containing `.git`
+* is still examined before the walk stops) and at the filesystem root.
+* Returns `null` when no config is found in range or when anything about
+* the walk throws.
 */
-function resolveAnchoredConfigFile(startDir) {
+function walkUpToConfigFile(startDir) {
 	try {
 		let dir = resolve(startDir);
 		for (;;) {
@@ -251,28 +248,25 @@ function resolveAnchoredConfigFile(startDir) {
 	}
 }
 /**
+* Walk UP from `startDir` looking for the vitest/vite config file, returning
+* its absolute path, or `null` when none is found in range.
+*
+* @internal exported for tests
+*/
+function resolveAnchoredConfigFile(startDir) {
+	return walkUpToConfigFile(startDir);
+}
+/**
 * Walk UP from `startDir` looking for the vitest/vite config Vitest would
-* load anyway, returning the directory that holds it. Bounded at the git
-* root (inclusive — the directory containing `.git` is still examined
-* before the walk stops). Returns `startDir` unchanged when no config is
-* found in range, or when anything about the walk throws. See the
-* issue #259 comment above `validateProjectRoot` for the full rationale.
+* load anyway, returning the directory that holds it. Returns `startDir`
+* unchanged when no config is found in range. See the issue #259 comment
+* above `validateProjectRoot` for the full rationale.
 *
 * @internal exported for tests
 */
 function resolveConfigAnchoredRoot(startDir) {
-	try {
-		let dir = resolve(startDir);
-		for (;;) {
-			if (dirHasVitestOrViteConfig(dir)) return dir;
-			if (existsSync(join(dir, ".git"))) return startDir;
-			const parent = dirname(dir);
-			if (parent === dir) return startDir;
-			dir = parent;
-		}
-	} catch {
-		return startDir;
-	}
+	const found = walkUpToConfigFile(startDir);
+	return found === null ? startDir : dirname(found);
 }
 /**
 * Validate an optional caller-supplied `projectRoot` against `ctxCwd`
@@ -536,6 +530,17 @@ const RunTestsInput = Schema.Struct({
 	})).annotate({ description: "Hook-injected session attribution UUIDs; do not pass manually." })
 });
 /**
+* Build the best-effort runner for a `RunTestsContext`: forks the effect
+* as a fiber on the provided services with its expected failure channel
+* ignored (`Effect.ignore`), so a typed failure never touches the tool
+* result (issue #330). Defects and interrupts are NOT absorbed — a bug in
+* the forked effect still surfaces through the runtime's fiber logging
+* rather than being hidden. Returns the fiber so a test can await its exit.
+*
+* @internal
+*/
+const makeBestEffortFork = (services) => (effect) => Effect.runFork(Effect.provideContext(effect.pipe(Effect.ignore), services));
+/**
 * The run body, promise-shaped because it drives Vitest's promise API and
 * the AsyncLocalStorage stdio capture; every failure is folded into the
 * `{ kind: "error" }` envelope, so it never rejects.
@@ -643,12 +648,12 @@ const runTestsBody = async (input, ctx) => {
 			}));
 		} catch {}
 		const chatId = ctx.currentSessionId.get();
-		if (chatId !== null) ctx.runPromise(Effect.gen(function* () {
+		if (chatId !== null) ctx.runFork(Effect.gen(function* () {
 			yield* (yield* DataStore).associateLatestRunWithSession({
 				chatId,
 				invocationMethod: "mcp"
 			});
-		})).catch(() => void 0);
+		}));
 		let scopedNote = null;
 		if (hasFilter) {
 			const testedFileCount = testModules.length;
@@ -714,7 +719,8 @@ const handleRunTests = (input) => Effect.gen(function* () {
 		cwd: session.cwd,
 		currentSessionId: session.currentSessionId,
 		sessionContext: session.sessionContext,
-		runPromise: (effect) => Effect.runPromise(Effect.provideContext(effect, services))
+		runPromise: (effect) => Effect.runPromise(Effect.provideContext(effect, services)),
+		runFork: makeBestEffortFork(services)
 	};
 	return yield* Semaphore.withPermit(runTestsSemaphore, Effect.promise(() => runTestsBody(input, ctx)));
 });
@@ -735,4 +741,4 @@ const runTestsTool = Tool.make("run_tests", {
 }).annotate(Tool.Title, "Run tests").annotate(Tool.Readonly, false).annotate(Tool.Destructive, false).annotate(Tool.OpenWorld, false).annotate(Tool.Idempotent, false).annotate(RenderText, (encoded) => formatRunTestsMarkdown(encoded));
 
 //#endregion
-export { RunTestsInput, RunTestsResult, coerceErrors, composeTagExpression, formatNoMatchMarkdown, formatReportMarkdown, formatRunTestsMarkdown, handleRunTests, makeCoverageDirOverride, readDiscoveryLastScannedAt, resolveAnchoredConfigFile, resolveConfigAnchoredRoot, resolveGitCommonDir, resolveVitestNodeEntry, runTestsTool, sanitizeTestArgs, validateProjectRoot, vitestLoader, withStdioCaptured };
+export { RunTestsInput, RunTestsResult, coerceErrors, composeTagExpression, formatNoMatchMarkdown, formatReportMarkdown, formatRunTestsMarkdown, handleRunTests, makeBestEffortFork, makeCoverageDirOverride, readDiscoveryLastScannedAt, resolveAnchoredConfigFile, resolveConfigAnchoredRoot, resolveGitCommonDir, resolveVitestNodeEntry, runTestsTool, sanitizeTestArgs, validateProjectRoot, vitestLoader, withStdioCaptured };

package/tools/test.js

@@ -81,10 +81,13 @@ const AttachmentDescriptor = Schema.Struct({
 * per attachment, so a test with many attachments could still flood an
 * agent's context; `maxBytes` (default 0) is the total byte budget for
 * every body in one response. Attachments are walked in order and each
-* body is charged its recorded `byteSize` (falling back to the stored
-* string's length when the row carries none). A body that would take
-* the running total past the budget is dropped along with its
-* `bodyEncoding`; the descriptor half — `contentType`, `path`,
+* body is charged the byte length of the string that will actually be
+* placed in the response — NOT its recorded `byteSize`, which is the
+* decoded size and undercounts a base64 body by a quarter (issue #393),
+* and not `String.length`, which counts UTF-16 code units and undercounts
+* multibyte utf-8. A body
+* that would take the running total past the budget is dropped along
+* with its `bodyEncoding`; the descriptor half — `contentType`, `path`,
 * `byteSize` — always survives.
 */
 const applyBodyBudget = (rows, maxBytes) => {
@@ -94,7 +97,7 @@ const applyBodyBudget = (rows, maxBytes) => {
 		attachments: row.attachments.map((attachment) => {
 			const { body, bodyEncoding, ...descriptor } = attachment;
 			if (body === void 0) return descriptor;
-			const cost = attachment.byteSize ?? body.length;
+			const cost = Buffer.byteLength(body, "utf-8");
 			if (spent + cost > maxBytes) return descriptor;
 			spent += cost;
 			return {

package/version.js

@@ -7,7 +7,7 @@
 *
 * @public
 */
-const CURRENT_MCP_VERSION = "4.0.2";
+const CURRENT_MCP_VERSION = "4.0.3";
 
 //#endregion
 export { CURRENT_MCP_VERSION };