@mandujs/mcp 0.37.3 → 0.38.0

package/src/tools/hydration.ts

@@ -18,17 +18,17 @@ export const hydrationToolDefinitions: Tool[] = [
     name: "mandu.build",
     description:
       "Build client bundles for hydration. Compiles client slots (.client.ts) into browser-ready JavaScript bundles.",
-    annotations: {
-      destructiveHint: true,
-      readOnlyHint: false,
-      idempotentHint: true,
-    },
-    inputSchema: {
-      type: "object",
-      properties: {
-        minify: {
-          type: "boolean",
-          description: "Minify the output bundles (default: true in production)",
+    annotations: {
+      destructiveHint: true,
+      readOnlyHint: false,
+      idempotentHint: true,
+    },
+    inputSchema: {
+      type: "object",
+      properties: {
+        minify: {
+          type: "boolean",
+          description: "Minify the output bundles (default: true in production)",
         },
         sourcemap: {
           type: "boolean",
@@ -39,50 +39,53 @@ export const hydrationToolDefinitions: Tool[] = [
           items: { type: "string" },
           description:
             "Only rebuild specific islands by routeId. Skips Runtime/Router/Vendor rebuild for faster incremental updates. Omit to rebuild everything.",
-        },
-      },
-      required: [],
-    },
-  },
+        },
+      },
+      required: [],
+      additionalProperties: false,
+    },
+  },
   {
     name: "mandu.build.status",
     description:
       "Get the current build status, bundle manifest, and statistics for client bundles.",
-    annotations: {
-      readOnlyHint: true,
-    },
-    inputSchema: {
-      type: "object",
-      properties: {},
-      required: [],
-    },
-  },
-  {
-    name: "mandu.island.list",
+    annotations: {
+      readOnlyHint: true,
+    },
+    inputSchema: {
+      type: "object",
+      properties: {},
+      required: [],
+      additionalProperties: false,
+    },
+  },
+  {
+    name: "mandu.island.list",
     description:
       "List all routes that have client-side hydration (islands). Shows hydration strategy and priority for each.",
-    annotations: {
-      readOnlyHint: true,
-    },
-    inputSchema: {
-      type: "object",
-      properties: {},
-      required: [],
-    },
-  },
-  {
-    name: "mandu.hydration.set",
+    annotations: {
+      readOnlyHint: true,
+    },
+    inputSchema: {
+      type: "object",
+      properties: {},
+      required: [],
+      additionalProperties: false,
+    },
+  },
+  {
+    name: "mandu.hydration.set",
     description:
       "Set hydration configuration for a specific route. Updates the route's hydration strategy and priority.",
-    annotations: {
-      readOnlyHint: false,
-    },
-    inputSchema: {
-      type: "object",
-      properties: {
-        routeId: {
-          type: "string",
-          description: "The route ID to configure",
+    annotations: {
+      readOnlyHint: false,
+    },
+    inputSchema: {
+      type: "object",
+      properties: {
+        routeId: {
+          type: "string",
+          description: "The route ID to configure",
         },
         strategy: {
           type: "string",
@@ -100,24 +103,25 @@ export const hydrationToolDefinitions: Tool[] = [
           type: "boolean",
           description: "Whether to preload the bundle with modulepreload",
         },
-      },
-      required: ["routeId"],
-    },
-  },
-  {
+      },
+      required: ["routeId"],
+      additionalProperties: false,
+    },
+  },
+  {
     name: "mandu.hydration.addClientSlot",
     description:
       "Add a client slot file for a route to enable hydration. Creates the .client.ts file and updates the manifest.",
-    annotations: {
-      destructiveHint: false,
-      readOnlyHint: false,
-    },
-    inputSchema: {
-      type: "object",
-      properties: {
-        routeId: {
-          type: "string",
-          description: "The route ID to add client slot for",
+    annotations: {
+      destructiveHint: false,
+      readOnlyHint: false,
+    },
+    inputSchema: {
+      type: "object",
+      properties: {
+        routeId: {
+          type: "string",
+          description: "The route ID to add client slot for",
         },
         strategy: {
           type: "string",
@@ -129,13 +133,14 @@ export const hydrationToolDefinitions: Tool[] = [
           enum: ["immediate", "visible", "idle", "interaction"],
           description: "Hydration priority (default: visible)",
         },
-      },
-      required: ["routeId"],
-    },
-  },
-];
-
-export function hydrationTools(projectRoot: string) {
+      },
+      required: ["routeId"],
+      additionalProperties: false,
+    },
+  },
+];
+
+export function hydrationTools(projectRoot: string) {
   const paths = getProjectPaths(projectRoot);
 
   const handlers: Record<string, (args: Record<string, unknown>) => Promise<unknown>> = {
@@ -223,20 +228,25 @@ export function hydrationTools(projectRoot: string) {
       }
 
       const islands = manifestResult.data.routes
-        .filter((route) => route.kind === "page")
-        .map((route) => {
-          const hydration = getRouteHydration(route);
-          const isIsland = needsHydration(route);
-
-          return {
-            routeId: route.id,
-            pattern: route.pattern,
-            hasClientModule: !!route.clientModule,
-            clientModule: route.clientModule || null,
-            isIsland,
-            hydration: {
-              strategy: hydration.strategy,
-              priority: hydration.priority,
+        .filter((route) => route.kind === "page")
+        .map((route) => {
+          const hydration = getRouteHydration(route);
+          const isIsland = needsHydration(route);
+          const warning = isIsland && !route.clientModule
+            ? `Route has hydration strategy '${hydration.strategy}' but no clientModule; build would emit no route bundle.`
+            : null;
+
+          return {
+            routeId: route.id,
+            pattern: route.pattern,
+            hasClientModule: !!route.clientModule,
+            clientModule: route.clientModule || null,
+            isIsland,
+            status: isIsland ? (route.clientModule ? "ready" : "broken") : "static",
+            warning,
+            hydration: {
+              strategy: hydration.strategy,
+              priority: hydration.priority,
               preload: hydration.preload,
             },
           };
@@ -252,12 +262,19 @@ export function hydrationTools(projectRoot: string) {
         islands: islands.filter((i) => i.isIsland),
         staticPages: islands.filter((i) => !i.isIsland),
       };
-    },
-
-    "mandu.hydration.set": async (args: Record<string, unknown>) => {
-      const { routeId, strategy, priority, preload } = args as {
-        routeId: string;
-        strategy?: SpecHydrationStrategy;
+    },
+
+    "mandu.hydration.set": async (args: Record<string, unknown>) => {
+      const validationError = validateRouteIdArgs(
+        args,
+        "mandu.hydration.set",
+        ["routeId", "strategy", "priority", "preload"],
+      );
+      if (validationError) return { error: validationError };
+
+      const { routeId, strategy, priority, preload } = args as {
+        routeId: string;
+        strategy?: SpecHydrationStrategy;
         priority?: HydrationPriority;
         preload?: boolean;
       };
@@ -311,12 +328,19 @@ export function hydrationTools(projectRoot: string) {
         newHydration,
         message: `Updated hydration config for ${routeId}`,
       };
-    },
-
-    "mandu.hydration.addClientSlot": async (args: Record<string, unknown>) => {
-      const { routeId, strategy = "island", priority = "visible" } = args as {
-        routeId: string;
-        strategy?: SpecHydrationStrategy;
+    },
+
+    "mandu.hydration.addClientSlot": async (args: Record<string, unknown>) => {
+      const validationError = validateRouteIdArgs(
+        args,
+        "mandu.hydration.addClientSlot",
+        ["routeId", "strategy", "priority"],
+      );
+      if (validationError) return { error: validationError };
+
+      const { routeId, strategy = "island", priority = "visible" } = args as {
+        routeId: string;
+        strategy?: SpecHydrationStrategy;
         priority?: HydrationPriority;
       };
 
@@ -396,14 +420,30 @@ export function hydrationTools(projectRoot: string) {
   };
 
   // Backward-compatible aliases (deprecated)
-  handlers["mandu_build"] = handlers["mandu.build"];
-  handlers["mandu_build_status"] = handlers["mandu.build.status"];
-  handlers["mandu_list_islands"] = handlers["mandu.island.list"];
-  handlers["mandu_set_hydration"] = handlers["mandu.hydration.set"];
-  handlers["mandu_add_client_slot"] = handlers["mandu.hydration.addClientSlot"];
-
-  return handlers;
-}
+  handlers["mandu_build"] = handlers["mandu.build"];
+  handlers["mandu_build_status"] = handlers["mandu.build.status"];
+  handlers["mandu_list_islands"] = handlers["mandu.island.list"];
+  handlers["mandu_set_hydration"] = handlers["mandu.hydration.set"];
+  handlers["mandu_hydration_set"] = handlers["mandu.hydration.set"];
+  handlers["mandu_add_client_slot"] = handlers["mandu.hydration.addClientSlot"];
+  handlers["mandu_hydration_add_client_slot"] = handlers["mandu.hydration.addClientSlot"];
+
+  return handlers;
+}
+
+function validateRouteIdArgs(
+  args: Record<string, unknown>,
+  toolName: string,
+  allowedKeys: readonly string[],
+): string | null {
+  if (typeof args.routeId !== "string" || args.routeId.trim().length === 0) {
+    const allowed = new Set(allowedKeys);
+    const unknownKeys = Object.keys(args).filter((key) => !allowed.has(key));
+    const got = unknownKeys.length > 0 ? ` (got unknown key${unknownKeys.length === 1 ? "" : "s"} '${unknownKeys.join("', '")}')` : "";
+    return `${toolName}: missing required parameter 'routeId'${got}`;
+  }
+  return null;
+}
 
 /**
  * Generate a client slot template

package/src/tools/index.ts

@@ -11,8 +11,9 @@ import { mcpToolRegistry } from "../registry/mcp-tool-registry.js";
 import { moduleToPlugins } from "../adapters/tool-adapter.js";
 import { type McpProfile, getProfileCategories } from "../profiles.js";
 
-// 도구 모듈 export
-export { specTools, specToolDefinitions } from "./spec.js";
+// 도구 모듈 export
+export { agentTools, agentToolDefinitions } from "./agent.js";
+export { specTools, specToolDefinitions } from "./spec.js";
 export { generateTools, generateToolDefinitions } from "./generate.js";
 export { transactionTools, transactionToolDefinitions } from "./transaction.js";
 export { historyTools, historyToolDefinitions } from "./history.js";
@@ -85,8 +86,9 @@ export {
   extractContractToolDefinitions,
 } from "./extract-contract.js";
 
-// 도구 모듈 import (등록용)
-import { specTools, specToolDefinitions } from "./spec.js";
+// 도구 모듈 import (등록용)
+import { agentTools, agentToolDefinitions } from "./agent.js";
+import { specTools, specToolDefinitions } from "./spec.js";
 import { generateTools, generateToolDefinitions } from "./generate.js";
 import { transactionTools, transactionToolDefinitions } from "./transaction.js";
 import { historyTools, historyToolDefinitions } from "./history.js";
@@ -189,7 +191,8 @@ export interface ToolModule {
  * 빌트인 도구 모듈 목록
  */
 export const TOOL_MODULES: ToolModule[] = [
-  { category: "spec", definitions: specToolDefinitions, handlers: specTools },
+  { category: "agent", definitions: agentToolDefinitions, handlers: agentTools },
+  { category: "spec", definitions: specToolDefinitions, handlers: specTools },
   { category: "generate", definitions: generateToolDefinitions, handlers: generateTools },
   { category: "transaction", definitions: transactionToolDefinitions, handlers: transactionTools },
   { category: "history", definitions: historyToolDefinitions, handlers: historyTools },