@mandujs/mcp 0.38.4 → 0.38.5

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/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/guard.ts

@@ -138,25 +138,7 @@ 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,
@@ -164,25 +146,16 @@ 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>> = {
@@ -548,18 +521,10 @@ 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,