@mandujs/mcp 0.38.6 → 0.38.7

package/src/tools/ate.ts

@@ -105,11 +105,11 @@ export const ateToolDefinitions: Tool[] = [
           type: "string",
           description: "Dev server URL (default: http://localhost:3333). Must match the running mandu dev server.",
         },
-        ci: { type: "boolean", description: "CI mode: stricter timeouts, no interactive prompts" },
-        timeoutMs: {
-          type: "number",
-          description: "Hard timeout for the whole Playwright process in milliseconds (default: 600000).",
-        },
+        ci: { type: "boolean", description: "CI mode: stricter timeouts, no interactive prompts" },
+        timeoutMs: {
+          type: "number",
+          description: "Hard timeout for the whole Playwright process in milliseconds (default: 600000).",
+        },
         headless: { type: "boolean", description: "Run browsers headlessly (default: true)" },
         browsers: {
           type: "array",
@@ -264,11 +264,11 @@ export const ateToolDefinitions: Tool[] = [
           enum: ["L0", "L1", "L2", "L3"],
           description: "Assertion depth: L0=smoke, L1=HTTP status, L2=contract schema, L3=full behavioral",
         },
-        ci: { type: "boolean", description: "CI mode: stricter timeouts" },
-        timeoutMs: {
-          type: "number",
-          description: "Hard timeout for the whole Playwright process in milliseconds (default: 600000).",
-        },
+        ci: { type: "boolean", description: "CI mode: stricter timeouts" },
+        timeoutMs: {
+          type: "number",
+          description: "Hard timeout for the whole Playwright process in milliseconds (default: 600000).",
+        },
         useImpactAnalysis: {
           type: "boolean",
           description: "Run impact analysis first and only test changed routes (faster in CI)",
@@ -386,7 +386,7 @@ export function ateTools(projectRoot: string, server?: Server) {
       return await ateRun(input);
     } catch (err) {
       const message = err instanceof Error ? err.message : String(err);
-      const isTimeout = /timeout|timed out|타임아웃/i.test(message);
+      const isTimeout = /timeout|timed out|타임아웃/i.test(message);
       const snap = tracker.snapshot();
       const partial: PartialRunResults = {
         runId: snap.runId ?? `unknown-${Date.now()}`,
@@ -447,21 +447,21 @@ export function ateTools(projectRoot: string, server?: Server) {
         browsers,
         onlyFiles,
         onlyRoutes,
-        grep,
-        timeoutMs,
-        progressToken,
-      } = args as {
+        grep,
+        timeoutMs,
+        progressToken,
+      } = args as {
         repoRoot: string;
         baseURL?: string;
         ci?: boolean;
         headless?: boolean;
         browsers?: ("chromium" | "firefox" | "webkit")[];
         onlyFiles?: string[];
-        onlyRoutes?: string[];
-        grep?: string;
-        timeoutMs?: number;
-        progressToken?: string | number;
-      };
+        onlyRoutes?: string[];
+        grep?: string;
+        timeoutMs?: number;
+        progressToken?: string | number;
+      };
       return await runWithObservability(
         {
           repoRoot,
@@ -469,11 +469,11 @@ export function ateTools(projectRoot: string, server?: Server) {
           ci,
           headless,
           browsers,
-          onlyFiles,
-          onlyRoutes,
-          grep,
-          timeoutMs,
-        },
+          onlyFiles,
+          onlyRoutes,
+          grep,
+          timeoutMs,
+        },
         { progressToken },
       );
     },
@@ -529,10 +529,10 @@ export function ateTools(projectRoot: string, server?: Server) {
     },
     "mandu.ate.auto_pipeline": async (args: Record<string, unknown>) => {
       const {
-        repoRoot, baseURL, oracleLevel, ci, useImpactAnalysis,
-        base, head, autoHeal, tsconfigPath, routeGlobs, buildSalt,
-        timeoutMs,
-      } = args as {
+        repoRoot, baseURL, oracleLevel, ci, useImpactAnalysis,
+        base, head, autoHeal, tsconfigPath, routeGlobs, buildSalt,
+        timeoutMs,
+      } = args as {
         repoRoot: string;
         baseURL?: string;
         oracleLevel?: OracleLevel;
@@ -542,14 +542,14 @@ export function ateTools(projectRoot: string, server?: Server) {
         head?: string;
         autoHeal?: boolean;
         tsconfigPath?: string;
-        routeGlobs?: string[];
-        buildSalt?: string;
-        timeoutMs?: number;
-      };
-      return await runFullPipeline({
-        repoRoot, baseURL, oracleLevel, ci, useImpactAnalysis,
-        base, head, autoHeal, tsconfigPath, routeGlobs, buildSalt, timeoutMs,
-      });
+        routeGlobs?: string[];
+        buildSalt?: string;
+        timeoutMs?: number;
+      };
+      return await runFullPipeline({
+        repoRoot, baseURL, oracleLevel, ci, useImpactAnalysis,
+        base, head, autoHeal, tsconfigPath, routeGlobs, buildSalt, timeoutMs,
+      });
     },
     "mandu.ate.feedback": async (args: Record<string, unknown>) => {
       const { repoRoot, runId, autoApply } = args as {

package/src/tools/brain.ts

@@ -13,19 +13,20 @@
 import type { Tool } from "@modelcontextprotocol/sdk/types.js";
 import type { Server } from "@modelcontextprotocol/sdk/server/index.js";
 import type { ActivityMonitor } from "../activity-monitor.js";
+import { loadManifest, runGuardCheck } from "@mandujs/core";
 import {
-  loadManifest,
-  runGuardCheck,
   analyzeViolations,
   initializeBrain,
   getBrain,
-  startWatcher,
-  stopWatcher,
-  getWatcher,
-  generateJsonStatus,
   initializeArchitectureAnalyzer,
   getArchitectureAnalyzer,
-} from "@mandujs/core";
+} from "@mandujs/core/brain";
+import {
+  generateJsonStatus,
+  getWatcher,
+  startWatcher,
+  stopWatcher,
+} from "@mandujs/core/watcher";
 import { getProjectPaths } from "../utils/project.js";
 
 export const brainToolDefinitions: Tool[] = [
@@ -664,7 +665,7 @@ export function brainTools(projectRoot: string, server?: Server, monitor?: Activ
 
   // #235 followup — brain auth tools (status / login / logout).
   handlers["mandu.brain.status"] = async () => {
-    const core = await import("@mandujs/core");
+    const core = await import("@mandujs/core/brain");
     assertBrainAuthSurface(core);
     const store = core.getCredentialStore();
     const resolution = await core.resolveBrainAdapter({
@@ -744,7 +745,7 @@ export function brainTools(projectRoot: string, server?: Server, monitor?: Activ
     };
 
     if (provider === "openai") {
-      const core = await import("@mandujs/core");
+      const core = await import("@mandujs/core/brain");
       assertBrainAuthSurface(core);
       const auth = new core.ChatGPTAuth();
       const existing = auth.locateAuthFile();
@@ -835,7 +836,7 @@ export function brainTools(projectRoot: string, server?: Server, monitor?: Activ
     }
 
     // Anthropic — Mandu-managed OAuth loopback flow.
-    const core = await import("@mandujs/core");
+    const core = await import("@mandujs/core/brain");
     assertBrainAuthSurface(core);
     try {
       const adapter = new core.AnthropicOAuthAdapter({
@@ -888,7 +889,7 @@ export function brainTools(projectRoot: string, server?: Server, monitor?: Activ
     const { provider = "all" } = args as {
       provider?: "openai" | "anthropic" | "all";
     };
-    const core = await import("@mandujs/core");
+    const core = await import("@mandujs/core/brain");
     assertBrainAuthSurface(core);
     const store = core.getCredentialStore();
     const targets =

package/src/tools/composite.ts

@@ -60,10 +60,10 @@ export const compositeToolDefinitions: Tool[] = [
     },
   },
   {
-    name: "mandu.island.add",
-    description:
-      "Create an island component with correct @mandujs/core/client imports and hydration strategy. " +
-      "Generates a .island.tsx file in app/{route}/ with the wrapComponent() island definition.",
+    name: "mandu.island.add",
+    description:
+      "Create an island component with correct @mandujs/core/client imports and hydration strategy. " +
+      "Generates a .island.tsx file in app/{route}/ with the wrapComponent() island definition.",
     annotations: {
       destructiveHint: true,
       readOnlyHint: false,
@@ -474,10 +474,10 @@ function generateMiddlewareSource(preset: string, options: Record<string, string
   return templates[preset] ?? templates.default;
 }
 
-function generateIslandSource(name: string, strategy: string): string {
-  return `"use client";
-import { wrapComponent } from "@mandujs/core/client";
-import { useState } from "react";
+function generateIslandSource(name: string, strategy: string): string {
+  return `"use client";
+import { wrapComponent } from "@mandujs/core/client";
+import { useState } from "react";
 
 interface ${name}Props {
   [key: string]: unknown;
@@ -493,8 +493,8 @@ function ${name}Inner(props: ${name}Props) {
   );
 }
 
-// Route hydration priority should be set on the page/manifest route.
-// Suggested priority for this island: "${strategy}".
-export default wrapComponent(${name}Inner);
-`;
-}
+// Route hydration priority should be set on the page/manifest route.
+// Suggested priority for this island: "${strategy}".
+export default wrapComponent(${name}Inner);
+`;
+}

package/src/tools/contract.ts

@@ -2,12 +2,12 @@ import type { Tool } from "@modelcontextprotocol/sdk/types.js";
 import {
   loadManifest,
   runContractGuardCheck,
-  generateContractTemplate,
   generateOpenAPIDocument,
   openAPIToJSON,
   type RouteSpec,
   type SpecHttpMethod,
 } from "@mandujs/core";
+import { generateContractTemplate } from "@mandujs/core/generator";
 import { getProjectPaths, writeJsonFile } from "../utils/project.js";
 import path from "path";
 import fs from "fs/promises";

package/src/tools/deploy-plan.ts

@@ -43,9 +43,9 @@ import {
 import {
   generateManifest,
   loadManifest,
-  resolveBrainAdapter,
   type RoutesManifest,
 } from "@mandujs/core";
+import { resolveBrainAdapter } from "@mandujs/core/brain";
 import path from "node:path";
 
 /**

package/src/tools/generate.ts

@@ -1,5 +1,7 @@
 import type { Tool } from "@modelcontextprotocol/sdk/types.js";
-import { loadManifest, generateRoutes, generateManifest, GENERATED_RELATIVE_PATHS, type GeneratedMap, parseResourceSchema, generateResourceArtifacts } from "@mandujs/core";
+import { generateManifest, loadManifest, parseResourceSchema, generateResourceArtifacts } from "@mandujs/core";
+import { generateRoutes, type GeneratedMap } from "@mandujs/core/generator";
+import { GENERATED_RELATIVE_PATHS } from "@mandujs/core/paths";
 import { getProjectPaths, readJsonFile } from "../utils/project.js";
 import path from "path";
 import fs from "fs/promises";

package/src/tools/guard.ts

@@ -7,7 +7,6 @@ import {
   loadManifest,
   runGuardCheck,
   runAutoCorrect,
-  type GeneratedMap,
   // Self-Healing Guard imports
   checkWithHealing,
   healAll,
@@ -19,6 +18,7 @@ import {
   type GuardPreset,
   type Violation,
 } from "@mandujs/core";
+import type { GeneratedMap } from "@mandujs/core/generator";
 import { getProjectPaths, readJsonFile, readConfig } from "../utils/project.js";
 import fs from "fs/promises";
 import path from "path";
@@ -138,7 +138,25 @@ export function guardTools(projectRoot: string) {
     }
   };
 
+  const guardViolationCode = (ruleId: string | undefined, type?: string) =>
+    `MANDU_GUARD_${(ruleId || type || "VIOLATION").toUpperCase().replace(/[^A-Z0-9]+/g, "_")}`;
+
+  const explainViolation = (input: {
+    type?: string;
+    ruleId?: string;
+    fromLayer?: string;
+    toLayer?: string;
+    suggestion?: string;
+  }): string => {
+    const source = input.fromLayer ? ` from ${input.fromLayer}` : "";
+    const target = input.toLayer ? ` to ${input.toLayer}` : "";
+    const rule = input.ruleId ?? input.type ?? "architecture rule";
+    const fix = input.suggestion ? ` Suggested fix: ${input.suggestion}` : "";
+    return `Violation of ${rule}${source}${target}.${fix}`.trim();
+  };
+
   const summarizeArchitectureViolation = (violation: Violation) => ({
+    code: guardViolationCode(violation.ruleName, violation.type),
     ruleId: violation.ruleName,
     type: violation.type,
     file: path.relative(projectRoot, violation.filePath).replace(/\\/g, "/") || violation.filePath,
@@ -146,16 +164,25 @@ export function guardTools(projectRoot: string) {
     column: violation.column,
     message: violation.ruleDescription,
     suggestion: violation.suggestions[0],
+    explanation: explainViolation({
+      type: violation.type,
+      ruleId: violation.ruleName,
+      fromLayer: violation.fromLayer,
+      toLayer: violation.toLayer,
+      suggestion: violation.suggestions[0],
+    }),
     fromLayer: violation.fromLayer,
     toLayer: violation.toLayer,
     importStatement: violation.importStatement,
   });
 
   const summarizeLegacyViolation = (v: Awaited<ReturnType<typeof runGuardCheck>>["violations"][number]) => ({
+    code: guardViolationCode(v.ruleId),
     ruleId: v.ruleId,
     file: v.file,
     message: v.message,
     suggestion: v.suggestion,
+    explanation: explainViolation({ ruleId: v.ruleId, suggestion: v.suggestion }),
   });
 
   const handlers: Record<string, (args: Record<string, unknown>) => Promise<unknown>> = {
@@ -521,10 +548,18 @@ export function guardTools(projectRoot: string) {
         preset: config.preset,
         violations: items.map((item) => ({
           // Violation info
+          code: guardViolationCode(item.violation.ruleName, item.violation.type),
           type: item.violation.type,
           file: item.violation.filePath,
           line: item.violation.line,
           message: item.violation.ruleDescription,
+          explanation: explainViolation({
+            type: item.violation.type,
+            ruleId: item.violation.ruleName,
+            fromLayer: item.violation.fromLayer,
+            toLayer: item.violation.toLayer,
+            suggestion: item.healing.primary.explanation,
+          }),
           fromLayer: item.violation.fromLayer,
           toLayer: item.violation.toLayer,
           importStatement: item.violation.importStatement,

package/src/tools/history.ts

@@ -4,7 +4,7 @@ import {
   pruneHistory,
   getChangeStats,
   readSnapshotById,
-} from "@mandujs/core";
+} from "@mandujs/core/change";
 
 export const historyToolDefinitions: Tool[] = [
   {

package/src/tools/hydration.ts

@@ -1,15 +1,13 @@
 import type { Tool } from "@modelcontextprotocol/sdk/types.js";
 import {
   loadManifest,
-  buildClientBundles,
-  formatSize,
   needsHydration,
   getRouteHydration,
-  type BundleManifest,
   type SpecHydrationStrategy,
   type HydrationPriority,
   type HydrationConfig,
 } from "@mandujs/core";
+import { buildClientBundles, formatSize, type BundleManifest } from "@mandujs/core/bundler";
 import { getProjectPaths, readJsonFile, writeJsonFile } from "../utils/project.js";
 import path from "path";
 

package/src/tools/index.ts

@@ -293,6 +293,7 @@ export function validateBuiltinToolModules(
   const issues: string[] = [];
   const categories = new Set<string>();
   const toolNames = new Map<string, string>();
+  const descriptions = new Map<string, string>();
 
   for (const module of modules) {
     if (categories.has(module.category)) {
@@ -305,6 +306,19 @@ export function validateBuiltinToolModules(
     }
 
     for (const definition of module.definitions) {
+      if (!definition.description?.trim()) {
+        issues.push(`tool definition is missing description: ${definition.name} in ${module.category}`);
+      }
+      const normalizedDescription = definition.description?.trim().replace(/\s+/g, " ").toLowerCase();
+      if (normalizedDescription) {
+        const previousTool = descriptions.get(normalizedDescription);
+        if (previousTool) {
+          issues.push(
+            `duplicate tool description: ${definition.name} and ${previousTool} both describe the same action`
+          );
+        }
+        descriptions.set(normalizedDescription, definition.name);
+      }
       const previousCategory = toolNames.get(definition.name);
       if (previousCategory) {
         issues.push(

package/src/tools/kitchen.ts

@@ -4,10 +4,10 @@
  * Enables any MCP-compatible agent to read client-side errors in real-time.
  */
 
-import type { Tool } from "@modelcontextprotocol/sdk/types.js";
-import { loadManduConfig } from "@mandujs/core";
-import { getDevServerState } from "./project.js";
-import { readRuntimeControl } from "../utils/runtime-control.js";
+import type { Tool } from "@modelcontextprotocol/sdk/types.js";
+import { loadManduConfig } from "@mandujs/core";
+import { getDevServerState } from "./project.js";
+import { readRuntimeControl } from "../utils/runtime-control.js";
 
 export const kitchenToolDefinitions: Tool[] = [
   {
@@ -20,18 +20,18 @@ export const kitchenToolDefinitions: Tool[] = [
     inputSchema: {
       type: "object",
       properties: {
-        clear: {
-          type: "boolean",
-          description: "Clear errors after reading (default: false)",
-        },
-        baseURL: {
-          type: "string",
-          description: "Explicit dev server URL. Overrides runtime-control/config discovery.",
-        },
-        port: {
-          type: "number",
-          description: "Explicit dev server port. Overrides runtime-control/config discovery.",
-        },
+        clear: {
+          type: "boolean",
+          description: "Clear errors after reading (default: false)",
+        },
+        baseURL: {
+          type: "string",
+          description: "Explicit dev server URL. Overrides runtime-control/config discovery.",
+        },
+        port: {
+          type: "number",
+          description: "Explicit dev server port. Overrides runtime-control/config discovery.",
+        },
       },
       required: [],
     },
@@ -56,19 +56,19 @@ export const kitchenToolDefinitions: Tool[] = [
           description:
             "Include the extended diagnose report. Default true. Set false to lower latency when a11y_hints / package_export_gaps are noisy.",
         },
-        includeDiff: {
-          type: "boolean",
-          description:
-            "Include git diff against MANDU_DIFF_BASE (default HEAD). Default true. Set false to skip when git is unavailable.",
-        },
-        baseURL: {
-          type: "string",
-          description: "Explicit dev server URL. Overrides runtime-control/config discovery.",
-        },
-        port: {
-          type: "number",
-          description: "Explicit dev server port. Overrides runtime-control/config discovery.",
-        },
+        includeDiff: {
+          type: "boolean",
+          description:
+            "Include git diff against MANDU_DIFF_BASE (default HEAD). Default true. Set false to skip when git is unavailable.",
+        },
+        baseURL: {
+          type: "string",
+          description: "Explicit dev server URL. Overrides runtime-control/config discovery.",
+        },
+        port: {
+          type: "number",
+          description: "Explicit dev server port. Overrides runtime-control/config discovery.",
+        },
       },
       required: [],
     },
@@ -76,32 +76,32 @@ export const kitchenToolDefinitions: Tool[] = [
 ];
 
 /**
- * Resolve the dev server base URL. Prefers explicit args, then
- * `.mandu/runtime-control.json` from the running dev/start process, then
- * captured stdout, then config/default 3333. Shared by every Kitchen-backed
- * MCP tool so they all agree on where to fetch from.
+ * Resolve the dev server base URL. Prefers explicit args, then
+ * `.mandu/runtime-control.json` from the running dev/start process, then
+ * captured stdout, then config/default 3333. Shared by every Kitchen-backed
+ * MCP tool so they all agree on where to fetch from.
  */
-async function resolveDevServerBaseUrl(
-  projectRoot: string,
-  args: { baseURL?: unknown; port?: unknown } = {},
-): Promise<string> {
-  if (typeof args.baseURL === "string" && args.baseURL.trim()) {
-    return args.baseURL.trim().replace(/\/+$/, "");
-  }
-
-  const explicitPort = normalizePort(args.port);
-  if (explicitPort) {
-    return `http://localhost:${explicitPort}`;
-  }
-
-  const control = await readRuntimeControl(projectRoot);
-  if (control?.baseUrl) {
-    return control.baseUrl.replace(/\/+$/, "");
-  }
-
-  let port: number | undefined;
-
-  const serverState = getDevServerState();
+async function resolveDevServerBaseUrl(
+  projectRoot: string,
+  args: { baseURL?: unknown; port?: unknown } = {},
+): Promise<string> {
+  if (typeof args.baseURL === "string" && args.baseURL.trim()) {
+    return args.baseURL.trim().replace(/\/+$/, "");
+  }
+
+  const explicitPort = normalizePort(args.port);
+  if (explicitPort) {
+    return `http://localhost:${explicitPort}`;
+  }
+
+  const control = await readRuntimeControl(projectRoot);
+  if (control?.baseUrl) {
+    return control.baseUrl.replace(/\/+$/, "");
+  }
+
+  let port: number | undefined;
+
+  const serverState = getDevServerState();
   if (serverState) {
     for (const line of serverState.output) {
       const portMatch = line.match(/https?:\/\/localhost:(\d+)/);
@@ -115,21 +115,21 @@ async function resolveDevServerBaseUrl(
     const config = await loadManduConfig(projectRoot);
     port = config.server?.port ?? 3333;
   }
-
-  return `http://localhost:${port}`;
-}
-
-function normalizePort(value: unknown): number | undefined {
-  const raw = typeof value === "number" ? value : typeof value === "string" ? Number.parseInt(value, 10) : NaN;
-  if (!Number.isInteger(raw) || raw < 1 || raw > 65535) return undefined;
-  return raw;
-}
+
+  return `http://localhost:${port}`;
+}
+
+function normalizePort(value: unknown): number | undefined {
+  const raw = typeof value === "number" ? value : typeof value === "string" ? Number.parseInt(value, 10) : NaN;
+  if (!Number.isInteger(raw) || raw < 1 || raw > 65535) return undefined;
+  return raw;
+}
 
 export function kitchenTools(projectRoot: string) {
   const handlers: Record<string, (args: Record<string, unknown>) => Promise<unknown>> = {
-    "mandu.kitchen.errors": async (args: Record<string, unknown>) => {
-      const { clear = false } = args as { clear?: boolean };
-      const baseUrl = await resolveDevServerBaseUrl(projectRoot, args);
+    "mandu.kitchen.errors": async (args: Record<string, unknown>) => {
+      const { clear = false } = args as { clear?: boolean };
+      const baseUrl = await resolveDevServerBaseUrl(projectRoot, args);
 
       try {
         // Fetch errors from Kitchen API
@@ -185,12 +185,12 @@ export function kitchenTools(projectRoot: string) {
      */
     "mandu.devtools.context": async (args: Record<string, unknown>) => {
       const {
-        includeBundle = true,
-        includeDiagnose = true,
-        includeDiff = true,
-      } = args as { includeBundle?: boolean; includeDiagnose?: boolean; includeDiff?: boolean };
-
-      const baseUrl = await resolveDevServerBaseUrl(projectRoot, args);
+        includeBundle = true,
+        includeDiagnose = true,
+        includeDiff = true,
+      } = args as { includeBundle?: boolean; includeDiagnose?: boolean; includeDiff?: boolean };
+
+      const baseUrl = await resolveDevServerBaseUrl(projectRoot, args);
       const params = new URLSearchParams();
       if (!includeBundle) params.set("bundle", "0");
       if (!includeDiagnose) params.set("diagnose", "0");

package/src/tools/runtime.ts

@@ -10,8 +10,8 @@ import {
   loadManduConfig,
   loadManifest,
   needsHydration,
-  type BundleManifest,
 } from "@mandujs/core";
+import type { BundleManifest } from "@mandujs/core/bundler";
 import { getDevServerState } from "./project.js";
 import { readRuntimeControl } from "../utils/runtime-control.js";
 import path from "path";

package/src/tools/slot-validation.ts

@@ -1,13 +1,13 @@
-import type { Tool } from "@modelcontextprotocol/sdk/types.js";
-import {
-  DEFAULT_SLOT_CONSTRAINTS,
-  API_SLOT_CONSTRAINTS,
-  READONLY_SLOT_CONSTRAINTS,
-} from "@mandujs/core";
-
-export const slotValidationToolDefinitions: Tool[] = [
-  {
-    name: "mandu.slot.constraints",
+import type { Tool } from "@modelcontextprotocol/sdk/types.js";
+import {
+  DEFAULT_SLOT_CONSTRAINTS,
+  API_SLOT_CONSTRAINTS,
+  READONLY_SLOT_CONSTRAINTS,
+} from "@mandujs/core";
+
+export const slotValidationToolDefinitions: Tool[] = [
+  {
+    name: "mandu.slot.constraints",
     description:
       "Get recommended slot constraint presets (default, api, readonly).",
     annotations: {
@@ -27,10 +27,10 @@ export const slotValidationToolDefinitions: Tool[] = [
   },
 ];
 
-export function slotValidationTools(projectRoot: string) {
-  void projectRoot;
-  const handlers: Record<string, (args: Record<string, unknown>) => Promise<unknown>> = {
-    "mandu.slot.constraints": async (args: Record<string, unknown>) => {
+export function slotValidationTools(projectRoot: string) {
+  void projectRoot;
+  const handlers: Record<string, (args: Record<string, unknown>) => Promise<unknown>> = {
+    "mandu.slot.constraints": async (args: Record<string, unknown>) => {
       const { preset } = args as { preset?: "default" | "api" | "readonly" };
 
       const presets = {
@@ -84,10 +84,10 @@ Mandu.filling()
         `.trim(),
       };
     },
-  };
-
-  // Backward-compatible aliases
-  handlers["mandu_get_slot_constraints"] = handlers["mandu.slot.constraints"];
-
+  };
+
+  // Backward-compatible aliases
+  handlers["mandu_get_slot_constraints"] = handlers["mandu.slot.constraints"];
+
   return handlers;
 }