@mandujs/mcp 0.38.3 → 0.38.5

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");