@vitest-agent/mcp 4.3.1 → 5.0.0

package/index.js

@@ -2,7 +2,6 @@ import { RenderText } from "./annotations.js";
 import { withIdempotency } from "./idempotency.js";
 import { McpSession, createCurrentSessionIdRef, createSessionContextRef, sessionContextFromEnv } from "./session.js";
 import { PromptsLayer } from "./prompts/layer.js";
-import { registerStrictToolkit } from "./register-toolkit.js";
 import { AcceptanceMetricsResult } from "./tools/acceptance-metrics.js";
 import { CacheHealthResult } from "./tools/cache-health.js";
 import { CommitChangesInput, CommitChangesResult } from "./tools/commit-changes.js";
@@ -32,5 +31,6 @@ import { WrapupPromptInput, WrapupPromptResult } from "./tools/wrapup-prompt.js"
 import { Kit, ToolsLayer, toolHandlers } from "./toolkit.js";
 import { SERVER_INSTRUCTIONS, ServerLayer } from "./server.js";
 import { CURRENT_MCP_VERSION } from "./version.js";
+import { ToolRefusal } from "@effected/mcp";
 
-export { AcceptanceMetricsResult, CURRENT_MCP_VERSION, CacheHealthResult, CommitChangesInput, CommitChangesResult, ConfigureInput, ConfigureResult, FailureSignatureGetInput, FailureSignatureGetResult, FileCoverageInput, FileCoverageResult, HelpResult, InventoryInput, InventoryResult, Kit, McpSession, NoteParams, NoteResult, PhaseTransitionInput, PhaseTransitionResult, PingResult, PromptsLayer, RegisterAgentInput, RegisterAgentResult, RenderText, RunTestsInput, RunTestsResult, SERVER_INSTRUCTIONS, ServerLayer, SettingsListResult, TddArtifactListInput, TddArtifactListResult, TddProgressPushInput, TddProgressPushResult, TestCoverageInput, TestCoverageResult, TestErrorsInput, TestErrorsResult, TestHistoryInput, TestHistoryResult, TestInput, TestOverviewInput, TestOverviewResult, TestResult, TestStatusInput, TestStatusResult, TestTrendsInput, TestTrendsResult, ToolsLayer, TriageBriefInput, TriageBriefResult, TurnSearchInput, TurnSearchResult, WrapupPromptInput, WrapupPromptResult, createCurrentSessionIdRef, createSessionContextRef, registerStrictToolkit, sessionContextFromEnv, toolHandlers, withIdempotency };
+export { AcceptanceMetricsResult, CURRENT_MCP_VERSION, CacheHealthResult, CommitChangesInput, CommitChangesResult, ConfigureInput, ConfigureResult, FailureSignatureGetInput, FailureSignatureGetResult, FileCoverageInput, FileCoverageResult, HelpResult, InventoryInput, InventoryResult, Kit, McpSession, NoteParams, NoteResult, PhaseTransitionInput, PhaseTransitionResult, PingResult, PromptsLayer, RegisterAgentInput, RegisterAgentResult, RenderText, RunTestsInput, RunTestsResult, SERVER_INSTRUCTIONS, ServerLayer, SettingsListResult, TddArtifactListInput, TddArtifactListResult, TddProgressPushInput, TddProgressPushResult, TestCoverageInput, TestCoverageResult, TestErrorsInput, TestErrorsResult, TestHistoryInput, TestHistoryResult, TestInput, TestOverviewInput, TestOverviewResult, TestResult, TestStatusInput, TestStatusResult, TestTrendsInput, TestTrendsResult, ToolRefusal, ToolsLayer, TriageBriefInput, TriageBriefResult, TurnSearchInput, TurnSearchResult, WrapupPromptInput, WrapupPromptResult, createCurrentSessionIdRef, createSessionContextRef, sessionContextFromEnv, toolHandlers, withIdempotency };

package/main.d.ts

@@ -1,31 +1,30 @@
+import { Distribution } from "@effected/engine";
 //#region src/main.d.ts
 /**
- * The assembled vitest-agent MCP server program over stdio. Owns the process.
+ * Options for {@link main}.
  *
- * This module deliberately carries NO static imports of the server graph:
- * the `unhandledRejection` and `uncaughtException` guards are registered
- * before `NodeRuntime`, the engine platform and `ServerLayer` are ever
- * evaluated, so a throw during module evaluation is still reported on stderr
- * rather than crashing silently. Adding a static import here would defeat
- * that. `bin.ts` is the published bin shim; this module is also published as
- * the `./main` subpath so the carrier (`@vitest-agent/plugin`) can ship its
- * own `vitest-agent-mcp` bin over it.
- *
- * Every `process` read lives here, not in the server layer or the tools.
- *
- * @packageDocumentation
+ * @public
  */
+export interface MainOptions {
+  /**
+   * The package whose bin launched the server — the carrier
+   * (`@vitest-agent/plugin`) passes its own name and version. Omitted for a
+   * direct `@vitest-agent/mcp` install. Surfaced as `ping`'s `distribution`.
+   */
+  readonly distribution?: Distribution | undefined;
+}
 /**
  * Run the vitest-agent MCP server over stdio. Owns the process: registers
- * the crash guards, resolves the project directory and `data.db` path,
+ * the crash guards through `McpGuard.run`, resolves the project directory and `data.db` path,
  * recovers the host session context, and launches the server layer under
- * `NodeRuntime.runMain`.
+ * `NodeRuntime.runMain` with `McpStdio.launch` / `McpStdio.teardown`.
  *
  * Not re-exported from `index.ts` — a library consumer's import graph must
  * not pull in the process-owning module.
  *
+ * @param options - the launching distribution, when a carrier shipped the bin
  * @public
  */
-export declare const main: () => Promise<void>;
+export declare const main: (options?: MainOptions) => Promise<void>;
 //#endregion
 //# sourceMappingURL=main.d.ts.map

package/main.js

@@ -1,105 +1,72 @@
-import { shouldExitOnUncaughtException } from "./utils/crash-guards.js";
+import { McpGuard } from "@effected/mcp/guard";
 
 //#region src/main.ts
 /**
-* The assembled vitest-agent MCP server program over stdio. Owns the process.
-*
-* This module deliberately carries NO static imports of the server graph:
-* the `unhandledRejection` and `uncaughtException` guards are registered
-* before `NodeRuntime`, the engine platform and `ServerLayer` are ever
-* evaluated, so a throw during module evaluation is still reported on stderr
-* rather than crashing silently. Adding a static import here would defeat
-* that. `bin.ts` is the published bin shim; this module is also published as
-* the `./main` subpath so the carrier (`@vitest-agent/plugin`) can ship its
-* own `vitest-agent-mcp` bin over it.
-*
-* Every `process` read lives here, not in the server layer or the tools.
-*
-* @packageDocumentation
-*/
-/**
-* Whether the stdio transport is live for this process — set once the
-* whole server layer graph (the stdio protocol included) has been built.
-* Read by the `uncaughtException` guard via `shouldExitOnUncaughtException`;
-* see that function's doc comment for the judgment call this flag backs
-* (issue #191).
-*/
-let transportConnected = false;
-/**
-* Formatter for the crash guards. Starts as a dependency-free fallback so a
-* throw during module evaluation of the server graph is still described,
-* and is swapped for the SDK's `safeFormatFatalError` once that import
-* has resolved.
-*/
-let formatFatal = (error) => {
-	try {
-		return error instanceof Error ? error.stack ?? error.message : String(error);
-	} catch {
-		return "<unformattable error value>";
-	}
-};
-/**
-* Test-only crash injection, gated by an env var so it can never fire in a
-* normal install. Exists so `__test__/bin-crash-resilience.e2e.test.ts` can
-* exercise the guards against a *real* child process. Fires exactly once,
-* on the next event-loop turn after the transport is known to be connected,
-* so ordering relative to `transportConnected` is deterministic regardless
-* of client-side handshake timing.
-*/
-const scheduleTestCrashInjection = () => {
-	const kind = process.env.VITEST_AGENT_MCP_TEST_INJECT_CRASH;
-	if (kind !== "unhandledRejection" && kind !== "uncaughtException") return;
-	setImmediate(() => {
-		if (kind === "unhandledRejection") Promise.reject(/* @__PURE__ */ new Error("[test-injected] unhandledRejection"));
-		else throw new Error("[test-injected] uncaughtException");
-	});
-};
-/**
 * Optional first positional argument: an initial Claude Code chat UUID (the
 * host's `chatId`) to seed the MCP server's session association. Claude Code
-* substitutes unknown `${...}` variables to literal text in some surfaces, so
-* a literal substitution is treated as absent rather than seeding garbage.
+* passes unknown `${...}` variables through as literal text in some surfaces,
+* so an unsubstituted placeholder is treated as absent rather than seeding
+* garbage.
 *
 * @param argv - the raw `process.argv`
-* @returns the trimmed seed, or `null` when absent, empty, or a literal `${...}`
+* @param isUnsubstituted - `LaunchContext.isUnsubstituted`, passed in because
+*   this module may not statically import the server graph
+* @returns the trimmed seed, or `null` when absent, empty, or a placeholder
 */
-const resolveInitialSessionId = (argv) => {
-	const first = argv[2];
-	if (first === void 0) return null;
-	const trimmed = first.trim();
-	if (trimmed.length === 0) return null;
-	if (trimmed.startsWith("${") && trimmed.endsWith("}")) return null;
+const resolveInitialSessionId = (argv, isUnsubstituted) => {
+	const trimmed = argv[2]?.trim();
+	if (trimmed === void 0 || trimmed.length === 0 || isUnsubstituted(trimmed)) return null;
 	return trimmed;
 };
+const INJECT_AT = ["load", "connected"];
+const INJECT_KIND = ["uncaughtException", "unhandledRejection"];
+/**
+* Parse the test-only `VITEST_AGENT_MCP_TEST_INJECT_CRASH` value into the
+* guard's `injectCrash`. Accepts `<at>:<kind>` (`at` is `load` or
+* `connected`, `kind` is `uncaughtException` or `unhandledRejection`), and a
+* bare `<kind>` as the legacy spelling of `connected:<kind>`. Anything else,
+* or no value, is `undefined`: no injection.
+*/
+const parseInjectCrash = (value) => {
+	if (value === void 0) return void 0;
+	const [first, second, ...rest] = value.split(":");
+	if (rest.length > 0) return void 0;
+	const at = second === void 0 ? "connected" : first;
+	const kind = second === void 0 ? first : second;
+	const validAt = INJECT_AT.find((candidate) => candidate === at);
+	const validKind = INJECT_KIND.find((candidate) => candidate === kind);
+	return validAt === void 0 || validKind === void 0 ? void 0 : {
+		at: validAt,
+		kind: validKind
+	};
+};
 /**
 * Run the vitest-agent MCP server over stdio. Owns the process: registers
-* the crash guards, resolves the project directory and `data.db` path,
+* the crash guards through `McpGuard.run`, resolves the project directory and `data.db` path,
 * recovers the host session context, and launches the server layer under
-* `NodeRuntime.runMain`.
+* `NodeRuntime.runMain` with `McpStdio.launch` / `McpStdio.teardown`.
 *
 * Not re-exported from `index.ts` — a library consumer's import graph must
 * not pull in the process-owning module.
 *
+* @param options - the launching distribution, when a carrier shipped the bin
 * @public
 */
-const main = async () => {
-	process.on("unhandledRejection", (reason) => {
-		process.stderr.write(`vitest-agent-mcp: unhandledRejection: ${formatFatal(reason)}\n`);
-	});
-	process.on("uncaughtException", (err, origin) => {
-		process.stderr.write(`vitest-agent-mcp: uncaughtException (${origin}): ${formatFatal(err)}\n`);
-		if (shouldExitOnUncaughtException(transportConnected)) {
-			process.exitCode = 1;
-			process.exit(1);
-		}
-	});
-	try {
+const main = (options = {}) => McpGuard.run({
+	label: "vitest-agent-mcp",
+	host: process,
+	policy: {
+		onUncaught: "exitBeforeConnect",
+		onRejection: "log"
+	},
+	injectCrash: parseInjectCrash(process.env.VITEST_AGENT_MCP_TEST_INJECT_CRASH),
+	load: async () => {
 		const { safeFormatFatalError } = await import("./utils/safe-format-fatal-error.js");
-		formatFatal = safeFormatFatalError;
 		const NodeRuntime = await import("@effect/platform-node/NodeRuntime");
 		const NodeServices = await import("@effect/platform-node/NodeServices");
 		const NodeStdio = await import("@effect/platform-node/NodeStdio");
-		const { Cause, Effect, Exit, Layer, Logger, Runtime } = await import("effect");
+		const { Effect, Layer, Option } = await import("effect");
+		const { CurrentDistribution, LaunchContext } = await import("@effected/engine");
 		const { PathResolutionLive, PlatformLive, recoverSessionContextFromSessionEnv, resolveDataPath, resolveLogFile, resolveLogLevel, resolveProjectDir } = await import("@vitest-agent/engine");
 		const { McpSession, sessionContextFromEnv } = await import("./session.js");
 		const { ServerLayer } = await import("./server.js");
@@ -109,7 +76,7 @@ const main = async () => {
 			env,
 			cwd: process.cwd()
 		});
-		const initialSessionId = resolveInitialSessionId(process.argv);
+		const initialSessionId = resolveInitialSessionId(process.argv, LaunchContext.isUnsubstituted);
 		const dbPath = await Effect.runPromise(resolveDataPath(projectDir).pipe(Effect.provide(PathResolutionLive(projectDir)), Effect.provide(NodeServices.layer)));
 		const recovered = sessionContextFromEnv(env);
 		const homeDir = env.HOME ?? env.USERPROFILE ?? "";
@@ -122,23 +89,18 @@ const main = async () => {
 				homeDir
 			})
 		});
-		const Main = ServerLayer({ version: CURRENT_MCP_VERSION }).pipe(Layer.provide(Session), Layer.provide(PlatformLive({
-			dbPath,
-			env,
-			logLevel: resolveLogLevel(env),
-			logFile: resolveLogFile(env)
-		})), Layer.provide(NodeStdio.layer), Layer.provide(Layer.succeed(Logger.LogToStderr, true)));
-		const Connected = Layer.effectDiscard(Effect.sync(() => {
-			transportConnected = true;
-			scheduleTestCrashInjection();
-		})).pipe(Layer.provide(Main));
-		const program = Layer.launch(Connected).pipe(Effect.provideService(Logger.LogToStderr, true));
-		NodeRuntime.runMain(program, { teardown: (exit, onExit) => Exit.isSuccess(exit) || Cause.hasInterruptsOnly(exit.cause) ? onExit(0) : Runtime.defaultTeardown(exit, onExit) });
-	} catch (err) {
-		process.stderr.write(`vitest-agent-mcp: startup failed: ${formatFatal(err)}\n`);
-		process.exit(1);
+		return {
+			layer: ServerLayer({ version: CURRENT_MCP_VERSION }).pipe(Layer.provide(Session), Layer.provide(PlatformLive({
+				dbPath,
+				env,
+				logLevel: resolveLogLevel(env),
+				logFile: resolveLogFile(env)
+			})), Layer.provide(Layer.succeed(CurrentDistribution, Option.fromNullishOr(options.distribution))), Layer.provide(NodeStdio.layer)),
+			runMain: NodeRuntime.runMain,
+			format: safeFormatFatalError
+		};
 	}
-};
+});
 
 //#endregion
 export { main };

package/package.json

@@ -1,8 +1,8 @@
 {
 	"name": "@vitest-agent/mcp",
-	"version": "4.3.1",
+	"version": "5.0.0",
 	"private": false,
-	"description": "Model Context Protocol server for vitest-agent. Exposes 53 tools for agent access to test data, TDD lifecycle, and session management.",
+	"description": "Model Context Protocol server for vitest-agent. Exposes 30 tools for agent access to test data, TDD lifecycle, and session management.",
 	"keywords": [
 		"vitest",
 		"agent",
@@ -45,7 +45,9 @@
 	"dependencies": {
 		"@effect/platform-node": "4.0.0-rc.117",
 		"@effect/sql-sqlite-node": "4.0.0-rc.117",
-		"@vitest-agent/engine": "0.2.5",
+		"@effected/engine": "^0.1.0",
+		"@effected/mcp": "^0.2.0",
+		"@vitest-agent/engine": "0.2.7",
 		"@vitest-agent/sdk": "5.1.2",
 		"effect": "4.0.0-rc.117"
 	},

package/server.js

@@ -1,8 +1,7 @@
 import { PromptsLayer } from "./prompts/layer.js";
-import { registerStrictToolkit } from "./register-toolkit.js";
 import { Kit, ToolsLayer } from "./toolkit.js";
-import { Layer, Logger } from "effect";
-import { McpProtocol, McpServer } from "effect/unstable/ai";
+import { McpStdio, McpToolkit } from "@effected/mcp";
+import { Layer } from "effect";
 
 //#region src/server.ts
 /** One-line human summary agents see on `serverInfo.description`. */
@@ -20,42 +19,31 @@ const SERVER_INSTRUCTIONS = [
 	"vitest-agent serves the test landscape of one Vitest project to LLM coding agents: run results, failure detail and history, coverage, flakiness, and the TDD lifecycle (tasks, goals, behaviors, phase transitions, hypotheses, notes).",
 	"Call the help tool first for the full tool reference; triage_brief orients on the current landscape; run_tests executes Vitest and persists the run so every query tool reads it back.",
 	"Every input is strict at every object level: an unknown key is rejected with a message naming the unrecognized key(s) and the accepted params, so fix the call rather than retrying it.",
-	"Every successful result carries the typed object in structuredContent, and content[0].text holds the same object as JSON; read structuredContent when you need fields.",
+	"Every successful result carries the typed object in structuredContent; read structuredContent for every field.",
 	"An expected domain error is NOT an isError result: it is a normal result whose structuredContent has ok: false and an error object. tdd_goal and tdd_behavior put _tag, the failing ids and a remediation naming the tool to call next in that object; register_agent puts a code and a message in it, plus existingAgentId or expectedPrefix when the code calls for one.",
-	"An isError result means the call itself failed: invalid params (bare text in content[0].text, no structuredContent) or an unexpected crash inside the tool (structuredContent is an UnexpectedToolError envelope with ok: false and a remediation). Read the message and fix the call."
+	"An isError result means the call itself failed and carries no structuredContent. Its text either names what to fix (invalid params, or a refused call such as an unknown id, often ending 'Try <tool>.' with the tool to call next) or is a generic internal-error message (the server logged the detail). Fix the call, or report the internal failure rather than retrying it unchanged."
 ].join("\n");
 /**
-* The server layer: every tool registered under the strict contract, plus
-* the six framing prompts, over `McpServer.layerStdio`.
+* The server layer: every tool registered through `McpToolkit.layer` (strict
+* by default, every unknown key named in one `InvalidParams`) plus the six
+* framing prompts, over `McpStdio.layer`.
 *
-* `protocols` order is load-bearing (rc.116 `unstable/ai/internal/mcpRuntime.ts`):
-* a request carrying `_meta["io.modelcontextprotocol/protocolVersion"]`
-* routes to that adapter, an `initialize` matches the STATEFUL adapters
-* only, and anything else with no session falls to `protocols[0]`. The
-* stateless `2026-07-28` adapter (SEP-2575: no handshake, no session,
-* `server/discover` instead of `initialize`) is listed first, then the two
-* newest stateful ones — every shipping client (Claude Code's default stdio
-* session, Copilot, Cursor, the Inspector) still opens with `initialize`,
-* which a server offering ONLY `2026-07-28` answers with
-* `METHOD_NOT_FOUND`. At most one stateless adapter is allowed; a second
-* fails the layer with `Cause.IllegalArgumentError`, which is why the
-* error channel is `orDie`d: `protocols` is a static literal, so a failure
-* there is an implementer-time defect.
+* `McpStdio.layer` serves `McpStdio.protocols` — the stateless `2026-07-28`
+* adapter first, then `2025-11-25` and `2025-06-18` — provides
+* `LogToStderr` (stdout is the JSON-RPC wire) to everything it provides, and
+* answers a malformed stdin line itself instead of wedging core's decoder.
+* It is provided, not merged, so the output stays `never`; a test registers
+* extra tools beside it through the shared module-level `McpServer` registry.
 *
 * @param options - the advertised server version
 * @public
 */
-const ServerLayer = (options) => Layer.mergeAll(registerStrictToolkit(Kit).pipe(Layer.provide(ToolsLayer)), PromptsLayer).pipe(Layer.provide(McpServer.layerStdio({
+const ServerLayer = (options) => Layer.mergeAll(McpToolkit.layer(Kit).pipe(Layer.provide(ToolsLayer)), PromptsLayer).pipe(Layer.provide(McpStdio.layer({
 	name: "vitest-agent",
 	version: options.version,
 	description: DESCRIPTION,
-	instructions: SERVER_INSTRUCTIONS,
-	protocols: [
-		McpProtocol.v2026_07_28,
-		McpProtocol.v2025_11_25,
-		McpProtocol.v2025_06_18
-	]
-})), Layer.provide(Layer.succeed(Logger.LogToStderr, true)), Layer.orDie);
+	instructions: SERVER_INSTRUCTIONS
+})));
 
 //#endregion
 export { SERVER_INSTRUCTIONS, ServerLayer };

package/toolkit.js

@@ -29,6 +29,7 @@ import { handleTestTrends, testTrendsTool } from "./tools/trends.js";
 import { handleTriageBrief, triageBriefTool } from "./tools/triage-brief.js";
 import { handleTurnSearch, turnSearchTool } from "./tools/turn-search.js";
 import { handleWrapupPrompt, wrapupPromptTool } from "./tools/wrapup-prompt.js";
+import { McpToolkit } from "@effected/mcp";
 import { Toolkit } from "effect/unstable/ai";
 
 //#region src/toolkit.ts
@@ -39,7 +40,10 @@ import { Toolkit } from "effect/unstable/ai";
 */
 const Kit = Toolkit.make(pingTool, helpTool, testStatusTool, testOverviewTool, testCoverageTool, testHistoryTool, testTrendsTool, testErrorsTool, fileCoverageTool, settingsListTool, cacheHealthTool, configureTool, commitChangesTool, turnSearchTool, failureSignatureGetTool, acceptanceMetricsTool, triageBriefTool, wrapupPromptTool, inventoryTool, testTool, registerAgentTool, noteTool, hypothesisTool, tddTaskTool, tddPhaseTransitionRequestTool, tddGoalTool, tddBehaviorTool, tddArtifactListTool, tddProgressPushTool, runTestsTool);
 /**
-* The handler for each tool in {@link Kit}, keyed by tool name.
+* The handler for each tool in {@link Kit}, keyed by tool name. The seven
+* action-keyed tools are `Tool.dynamic` (a union root cannot be a `Tool.make`
+* parameters schema), so their handlers take the raw payload and decode it
+* through `McpToolkit.unionHandler`.
 *
 * @public
 */
@@ -62,15 +66,15 @@ const toolHandlers = {
 	acceptance_metrics: handleAcceptanceMetrics,
 	triage_brief: handleTriageBrief,
 	wrapup_prompt: handleWrapupPrompt,
-	inventory: handleInventory,
-	test: handleTest,
+	inventory: McpToolkit.unionHandler(inventoryTool, handleInventory),
+	test: McpToolkit.unionHandler(testTool, handleTest),
 	register_agent: handleRegisterAgent,
-	note: handleNote,
-	hypothesis: withIdempotency("hypothesis", handleHypothesis),
-	tdd_task: withIdempotency("tdd_task", handleTddTask),
+	note: McpToolkit.unionHandler(noteTool, handleNote),
+	hypothesis: McpToolkit.unionHandler(hypothesisTool, withIdempotency("hypothesis", handleHypothesis)),
+	tdd_task: McpToolkit.unionHandler(tddTaskTool, withIdempotency("tdd_task", handleTddTask)),
 	tdd_phase_transition_request: handlePhaseTransitionRequest,
-	tdd_goal: withIdempotency("tdd_goal", handleTddGoal),
-	tdd_behavior: withIdempotency("tdd_behavior", handleTddBehavior),
+	tdd_goal: McpToolkit.unionHandler(tddGoalTool, withIdempotency("tdd_goal", handleTddGoal)),
+	tdd_behavior: McpToolkit.unionHandler(tddBehaviorTool, withIdempotency("tdd_behavior", handleTddBehavior)),
 	tdd_artifact_list: handleTddArtifactList,
 	tdd_progress_push: handleTddProgressPush,
 	run_tests: handleRunTests

package/tools/_tdd-error-envelope.js

@@ -2,15 +2,6 @@ import { Effect } from "effect";
 import { BehaviorNotFoundError, GoalNotFoundError, IllegalStatusTransitionError, TddTaskAlreadyEndedError, TddTaskNotFoundError } from "@vitest-agent/sdk";
 
 //#region src/tools/_tdd-error-envelope.ts
-/**
-* Shared error-to-success-shape envelope conversion for goal / behavior CRUD
-* tools. Maps each tagged TDD error to an ok:false response matching the
-* tdd_phase_transition_request accept/deny envelope shape, so the agent
-* sees a normal tool response instead of a transport error.
-*
-* `Effect.catchTags` returns `Effect.succeed(envelope)` so the success
-* channel carries the union `Success | ErrorEnvelope`.
-*/
 const goalNotFound = (e) => ({
 	ok: false,
 	error: {
@@ -20,7 +11,7 @@ const goalNotFound = (e) => ({
 		remediation: {
 			suggestedTool: "tdd_goal",
 			suggestedArgs: { action: "list" },
-			humanHint: `No tdd_session_goals row with id=${e.id}. Call tdd_goal({ action: "list", tddTaskId }) to find the correct goal id.`
+			hint: `No tdd_session_goals row with id=${e.id}. Call tdd_goal({ action: "list", tddTaskId }) to find the correct goal id.`
 		}
 	}
 });
@@ -33,7 +24,7 @@ const behaviorNotFound = (e) => ({
 		remediation: {
 			suggestedTool: "tdd_behavior",
 			suggestedArgs: { action: "list_by_goal" },
-			humanHint: `No tdd_session_behaviors row with id=${e.id}. Call tdd_behavior({ action: "list_by_goal", goalId }) or tdd_behavior({ action: "list_by_tdd_task", tddTaskId }) to find the correct behavior id.`
+			hint: `No tdd_session_behaviors row with id=${e.id}. Call tdd_behavior({ action: "list_by_goal", goalId }) or tdd_behavior({ action: "list_by_tdd_task", tddTaskId }) to find the correct behavior id.`
 		}
 	}
 });
@@ -46,7 +37,7 @@ const tddTaskNotFound = (e) => ({
 		remediation: {
 			suggestedTool: "tdd_task",
 			suggestedArgs: { action: "start" },
-			humanHint: `No tdd_tasks row with id=${e.id}. Call tdd_task({ action: "start" }) to open a TDD task before creating goals or behaviors.`
+			hint: `No tdd_tasks row with id=${e.id}. Call tdd_task({ action: "start" }) to open a TDD task before creating goals or behaviors.`
 		}
 	}
 });
@@ -60,7 +51,7 @@ const tddTaskAlreadyEnded = (e) => ({
 		remediation: {
 			suggestedTool: "tdd_task",
 			suggestedArgs: { action: "start" },
-			humanHint: `tdd_tasks row id=${e.id} is already ended (outcome=${e.outcome}). Open a new TDD task if you need to add more goals or behaviors.`
+			hint: `tdd_tasks row id=${e.id} is already ended (outcome=${e.outcome}). Open a new TDD task if you need to add more goals or behaviors.`
 		}
 	}
 });
@@ -80,7 +71,7 @@ const illegalStatusTransition = (e) => ({
 				id: e.id,
 				status: "abandoned"
 			},
-			humanHint: `Cannot transition ${e.entity} id=${e.id} from ${e.from} to ${e.to}. Use status:'abandoned' to drop work; do not delete unless the entity was created by mistake.`
+			hint: `Cannot transition ${e.entity} id=${e.id} from ${e.from} to ${e.to}. Use status:'abandoned' to drop work; do not delete unless the entity was created by mistake.`
 		}
 	}
 });

package/tools/cache-health.js

@@ -1,3 +1,4 @@
+import { ToolOutputSchema } from "@effected/mcp";
 import { Effect, Option, Schema } from "effect";
 import { DataReader } from "@vitest-agent/engine";
 import { Tool } from "effect/unstable/ai";
@@ -22,7 +23,7 @@ const ManifestAbsent = Schema.Struct({ manifestPresent: Schema.Literal(false).an
 *
 * @public
 */
-const CacheHealthResult = Schema.Union([ManifestPresent, ManifestAbsent]).annotate({
+const CacheHealthResult = ToolOutputSchema.objectRooted(Schema.Union([ManifestPresent, ManifestAbsent])).annotate({
 	identifier: "CacheHealthResult",
 	title: "cache_health result",
 	description: "Cache health snapshot. Discriminate on `manifestPresent` to see whether the manifest exists."

package/tools/configure.js

@@ -1,3 +1,4 @@
+import { ToolOutputSchema } from "@effected/mcp";
 import { Effect, Option, Schema } from "effect";
 import { DataReader } from "@vitest-agent/engine";
 import { Tool } from "effect/unstable/ai";
@@ -35,7 +36,7 @@ const SettingsAbsent = Schema.Struct({
 *
 * @public
 */
-const ConfigureResult = Schema.Union([SettingsFound, SettingsAbsent]).annotate({
+const ConfigureResult = ToolOutputSchema.objectRooted(Schema.Union([SettingsFound, SettingsAbsent])).annotate({
 	identifier: "ConfigureResult",
 	title: "configure result",
 	description: "Captured Vitest settings for a run, or an absence record when the lookup found nothing."

package/tools/coverage.js

@@ -1,3 +1,4 @@
+import { ToolOutputSchema } from "@effected/mcp";
 import { Effect, Option, Schema } from "effect";
 import { DataReader } from "@vitest-agent/engine";
 import { Tool } from "effect/unstable/ai";
@@ -18,7 +19,7 @@ const CoverageAbsent = Schema.Struct({
 *
 * @public
 */
-const TestCoverageResult = Schema.Union([CoverageAvailable, CoverageAbsent]).annotate({
+const TestCoverageResult = ToolOutputSchema.objectRooted(Schema.Union([CoverageAvailable, CoverageAbsent])).annotate({
 	identifier: "TestCoverageResult",
 	title: "test_coverage result",
 	description: "Per-project coverage report. Discriminate on `dataAvailable` for cold-start handling."

package/tools/failure-signature-get.js

@@ -1,3 +1,4 @@
+import { ToolOutputSchema } from "@effected/mcp";
 import { Effect, Option, Schema } from "effect";
 import { DataReader } from "@vitest-agent/engine";
 import { Tool } from "effect/unstable/ai";
@@ -29,7 +30,7 @@ const SignatureMissing = Schema.Struct({
 *
 * @public
 */
-const FailureSignatureGetResult = Schema.Union([SignatureFound, SignatureMissing]).annotate({
+const FailureSignatureGetResult = ToolOutputSchema.objectRooted(Schema.Union([SignatureFound, SignatureMissing])).annotate({
 	identifier: "FailureSignatureGetResult",
 	title: "failure_signature_get result",
 	description: "Discriminate on `found`. Found rows carry first/last-seen timestamps and recent occurrences."

package/tools/file-coverage.js

@@ -1,3 +1,4 @@
+import { ToolOutputSchema } from "@effected/mcp";
 import { Effect, Option, Schema } from "effect";
 import { DataReader } from "@vitest-agent/engine";
 import { Tool } from "effect/unstable/ai";
@@ -34,11 +35,11 @@ const FileCoverageAbsent = Schema.Struct({
 *
 * @public
 */
-const FileCoverageResult = Schema.Union([
+const FileCoverageResult = ToolOutputSchema.objectRooted(Schema.Union([
 	FileCoverageMatched,
 	FileCoverageNoMatch,
 	FileCoverageAbsent
-]).annotate({
+])).annotate({
 	identifier: "FileCoverageResult",
 	title: "file_coverage result",
 	description: "Per-file coverage with related tests. Discriminate on `dataAvailable` then on `matched`."