@mandujs/core 0.54.10 → 0.54.12

package/src/deploy/inference/context.ts

@@ -78,8 +78,8 @@ export async function buildDeployInferenceContext(
     // still gets the manifest metadata and falls back to defaults.
   }
 
-  const imports = extractImports(source);
-  const dependencyClasses = classifySourceDependencies(source, imports);
+  const imports = extractImports(source);
+  const dependencyClasses = classifySourceDependencies(source, imports);
   // Mandu's manifest patterns use `:param` / `*` (path-pattern style),
   // not bracket-form. Detect both shapes so the heuristic doesn't
   // misclassify dynamic routes as prerenderable. Examples:
@@ -139,82 +139,82 @@ export function extractImports(source: string): string[] {
 }
 
 /** Map import specifiers to coarse dependency classes. */
-export function classifyImports(imports: string[]): ReadonlySet<DependencyClass> {
-  const classes = new Set<DependencyClass>();
-  for (const spec of imports) {
+export function classifyImports(imports: string[]): ReadonlySet<DependencyClass> {
+  const classes = new Set<DependencyClass>();
+  for (const spec of imports) {
     const cls = classifyOne(spec);
     if (cls) classes.add(cls);
   }
   if (classes.size === 0) classes.add("fetch-only");
-  return classes;
-}
-
-/**
- * Classify dependency signals that do not always appear as bare imports.
- *
- * Dogfooding surfaced Mandu API routes importing project-local
- * `src/server/infra/db` helpers and using `db` tagged templates. Those
- * are server-only even when the bare imports look edge-safe.
- */
-export function classifySourceDependencies(
-  source: string,
-  imports: string[] = extractImports(source),
-): ReadonlySet<DependencyClass> {
-  const classes = new Set<DependencyClass>(classifyImports(imports));
-  if (classes.size === 1 && classes.has("fetch-only")) {
-    classes.delete("fetch-only");
-  }
-
-  const allImports = extractAllImportSpecifiers(source);
-  if (allImports.some(isServerInfraImport)) {
-    classes.add("db");
-  }
-
-  if (/\bBun\.SQL\b|\bBun\.sqlite\b/i.test(source)) {
-    classes.add("bun-native");
-  }
-
-  if (
-    /(?:^|[^\w$])db\s*`/.test(source) ||
-    /\bctx\.deps\.db\b/.test(source) ||
-    /\bdb\.(?:query|execute|select|insert|update|delete)\b/.test(source)
-  ) {
-    classes.add("db");
-  }
-
-  if (classes.size === 0) {
-    classes.add("fetch-only");
-  }
-  return classes;
-}
-
-function extractAllImportSpecifiers(source: string): string[] {
-  const out = new Set<string>();
-  const staticImport = /^\s*import\b[^"']*?["']([^"']+)["']/gm;
-  const dynamicImport = /\bimport\(\s*["']([^"']+)["']\s*\)/g;
-  for (const re of [staticImport, dynamicImport]) {
-    let m: RegExpExecArray | null;
-    while ((m = re.exec(source)) !== null) {
-      out.add(m[1]!);
-    }
-  }
-  return [...out].sort();
-}
-
-function isServerInfraImport(specifier: string): boolean {
-  const s = specifier.replace(/\\/g, "/").toLowerCase();
-  return (
-    s === "@/server/infra" ||
-    s.startsWith("@/server/infra/") ||
-    s === "src/server/infra" ||
-    s.startsWith("src/server/infra/") ||
-    s.endsWith("/server/infra") ||
-    s.includes("/server/infra/")
-  );
-}
-
-function classifyOne(spec: string): DependencyClass | null {
-  const s = spec.toLowerCase();
+  return classes;
+}
+
+/**
+ * Classify dependency signals that do not always appear as bare imports.
+ *
+ * Dogfooding surfaced Mandu API routes importing project-local
+ * `src/server/infra/db` helpers and using `db` tagged templates. Those
+ * are server-only even when the bare imports look edge-safe.
+ */
+export function classifySourceDependencies(
+  source: string,
+  imports: string[] = extractImports(source),
+): ReadonlySet<DependencyClass> {
+  const classes = new Set<DependencyClass>(classifyImports(imports));
+  if (classes.size === 1 && classes.has("fetch-only")) {
+    classes.delete("fetch-only");
+  }
+
+  const allImports = extractAllImportSpecifiers(source);
+  if (allImports.some(isServerInfraImport)) {
+    classes.add("db");
+  }
+
+  if (/\bBun\.SQL\b|\bBun\.sqlite\b/i.test(source)) {
+    classes.add("bun-native");
+  }
+
+  if (
+    /(?:^|[^\w$])db\s*`/.test(source) ||
+    /\bctx\.deps\.db\b/.test(source) ||
+    /\bdb\.(?:query|execute|select|insert|update|delete)\b/.test(source)
+  ) {
+    classes.add("db");
+  }
+
+  if (classes.size === 0) {
+    classes.add("fetch-only");
+  }
+  return classes;
+}
+
+function extractAllImportSpecifiers(source: string): string[] {
+  const out = new Set<string>();
+  const staticImport = /^\s*import\b[^"']*?["']([^"']+)["']/gm;
+  const dynamicImport = /\bimport\(\s*["']([^"']+)["']\s*\)/g;
+  for (const re of [staticImport, dynamicImport]) {
+    let m: RegExpExecArray | null;
+    while ((m = re.exec(source)) !== null) {
+      out.add(m[1]!);
+    }
+  }
+  return [...out].sort();
+}
+
+function isServerInfraImport(specifier: string): boolean {
+  const s = specifier.replace(/\\/g, "/").toLowerCase();
+  return (
+    s === "@/server/infra" ||
+    s.startsWith("@/server/infra/") ||
+    s === "src/server/infra" ||
+    s.startsWith("src/server/infra/") ||
+    s.endsWith("/server/infra") ||
+    s.includes("/server/infra/")
+  );
+}
+
+function classifyOne(spec: string): DependencyClass | null {
+  const s = spec.toLowerCase();
   if (
     s === "bun:sqlite" ||
     s === "bun:ffi" ||
@@ -231,14 +231,14 @@ function classifyOne(spec: string): DependencyClass | null {
   if (s === "node:child_process" || s === "child_process" || s === "node:worker_threads" || s === "worker_threads") {
     return "node-child";
   }
-  if (
-    /^(postgres|pg|mysql2?|drizzle-orm(\/.*)?|@prisma\/client|prisma|mongodb|mongoose|@neondatabase\/.+|kysely|sqlite3|better-sqlite3|@planetscale\/.+)$/.test(s)
-  ) {
-    return "db";
-  }
-  if (s === "@mandujs/core/db" || s.startsWith("@mandujs/core/db/")) {
-    return "db";
-  }
+  if (
+    /^(postgres|pg|mysql2?|drizzle-orm(\/.*)?|@prisma\/client|prisma|mongodb|mongoose|@neondatabase\/.+|kysely|sqlite3|better-sqlite3|@planetscale\/.+)$/.test(s)
+  ) {
+    return "db";
+  }
+  if (s === "@mandujs/core/db" || s.startsWith("@mandujs/core/db/")) {
+    return "db";
+  }
   if (/^(@anthropic-ai\/sdk|openai|ai|@ai-sdk\/.+|@google\/generative-ai|cohere-ai)$/.test(s)) {
     return "ai-sdk";
   }

package/src/devtools/client/components/panel/islands-panel.tsx

@@ -183,28 +183,28 @@ export function IslandsPanel({ islands }: IslandsPanelProps): React.ReactElement
     );
   }, [islands]);
 
-  if (islands.length === 0) {
-    return (
-      <div style={styles.container}>
-        <div style={styles.emptyState}>
-          <p>
-            아직 hydration 이벤트가 없습니다.<br />
-            정적 상태는 mandu.runtime.status에서 확인하세요.
-          </p>
-        </div>
-      </div>
-    );
-  }
+  if (islands.length === 0) {
+    return (
+      <div style={styles.container}>
+        <div style={styles.emptyState}>
+          <p>
+            아직 hydration 이벤트가 없습니다.<br />
+            정적 상태는 mandu.runtime.status에서 확인하세요.
+          </p>
+        </div>
+      </div>
+    );
+  }
 
   return (
     <div style={styles.container}>
       {/* Header Stats */}
       <div style={styles.header}>
         <div style={styles.stats}>
-          <div style={styles.stat}>
-            <span>Mounts:</span>
-            <span style={styles.statValue}>{stats.total}</span>
-          </div>
+          <div style={styles.stat}>
+            <span>Mounts:</span>
+            <span style={styles.statValue}>{stats.total}</span>
+          </div>
           <div style={styles.stat}>
             <span>Ready</span>
             <span style={styles.statValue}>{stats.hydrated}</span>

package/src/devtools/client/components/panel/panel-container.tsx

@@ -28,7 +28,7 @@ export interface PanelContainerProps {
 
 export const TABS: TabDefinition[] = [
   { id: 'errors', label: 'Issues', icon: 'ERR', testId: testIds.tabErrors },
-  { id: 'islands', label: 'Client Mounts', icon: 'HYD', testId: testIds.tabIslands },
+  { id: 'islands', label: 'Client Mounts', icon: 'HYD', testId: testIds.tabIslands },
   { id: 'network', label: 'Network', icon: 'NET', testId: testIds.tabNetwork },
   { id: 'guard', label: 'Guard', icon: 'GRD', testId: testIds.tabGuard },
   { id: 'preview', label: 'Changes', icon: 'CHG', testId: testIds.tabPreview },

package/src/error/formatter.ts

@@ -21,20 +21,15 @@ export function formatErrorResponse(error: ManduError, options: FormatOptions =
 
   const response: Record<string, unknown> = {
     errorType: error.errorType,
-    code: error.code,
-    message: error.message,
-    summary: error.summary,
-    cause: error.summary,
-    fix: error.fix,
-    filePath: error.fix.file,
-    solution: error.fix.suggestion,
-  };
-
-  if (error.route) {
-    response.route = error.route;
-    response.routeId = error.route.id;
-    response.routePattern = error.route.pattern;
-  }
+    code: error.code,
+    message: error.message,
+    summary: error.summary,
+    fix: error.fix,
+  };
+
+  if (error.route) {
+    response.route = error.route;
+  }
 
   // 개발 모드에서만 디버그 정보 포함
   if (isDev && error.debug) {
@@ -65,31 +60,27 @@ export function formatErrorForConsole(error: ManduError, options: FormatOptions
   lines.push(`  ${error.message}`);
 
   // 요약
-  if (useColors) {
-    lines.push(`  ${CYAN}→ ${error.summary}${RESET}`);
-  } else {
-    lines.push(`  → ${error.summary}`);
-  }
-  lines.push(`  Cause: ${error.summary}`);
+  if (useColors) {
+    lines.push(`  ${CYAN}→ ${error.summary}${RESET}`);
+  } else {
+    lines.push(`  → ${error.summary}`);
+  }
 
   // 수정 안내
   lines.push("");
   if (useColors) {
     lines.push(`  ${YELLOW}Fix:${RESET} ${error.fix.file}${error.fix.line ? `:${error.fix.line}` : ""}`);
     lines.push(`       ${error.fix.suggestion}`);
-  } else {
-    lines.push(`  Fix: ${error.fix.file}${error.fix.line ? `:${error.fix.line}` : ""}`);
-    lines.push(`       ${error.fix.suggestion}`);
-  }
-  lines.push(`  File: ${error.fix.file}${error.fix.line ? `:${error.fix.line}` : ""}`);
-  lines.push(`  Solution: ${error.fix.suggestion}`);
+  } else {
+    lines.push(`  Fix: ${error.fix.file}${error.fix.line ? `:${error.fix.line}` : ""}`);
+    lines.push(`       ${error.fix.suggestion}`);
+  }
 
   // 라우트 컨텍스트
-  if (error.route) {
-    lines.push("");
-    lines.push(`  Route ID: ${error.route.id}`);
-    lines.push(`  Route: ${error.route.pattern}`);
-  }
+  if (error.route) {
+    lines.push("");
+    lines.push(`  Route: ${error.route.id} (${error.route.pattern})`);
+  }
 
   // 디버그 정보 (개발 모드)
   if (isDev && includeStack && error.debug?.stack) {

package/src/filling/context.ts

@@ -628,23 +628,23 @@ export class ManduContext {
     return this.withCookies(new Response(null, { status: 204 }));
   }
 
-  /** 400 Bad Request, or custom 4xx/5xx error with ctx.error(status, message). */
-  error(message: string, details?: unknown): Response;
-  error(status: number, message: string, details?: unknown): Response;
-  error(
-    statusOrMessage: number | string,
-    messageOrDetails?: string | unknown,
-    maybeDetails?: unknown
-  ): Response {
-    if (typeof statusOrMessage === "number") {
-      const status = Number.isInteger(statusOrMessage) && statusOrMessage >= 400 && statusOrMessage <= 599
-        ? statusOrMessage
-        : 400;
-      const message = typeof messageOrDetails === "string" ? messageOrDetails : "Error";
-      return this.json({ status: "error", message, details: maybeDetails }, status);
-    }
-    return this.json({ status: "error", message: statusOrMessage, details: messageOrDetails }, 400);
-  }
+  /** 400 Bad Request, or custom 4xx/5xx error with ctx.error(status, message). */
+  error(message: string, details?: unknown): Response;
+  error(status: number, message: string, details?: unknown): Response;
+  error(
+    statusOrMessage: number | string,
+    messageOrDetails?: string | unknown,
+    maybeDetails?: unknown
+  ): Response {
+    if (typeof statusOrMessage === "number") {
+      const status = Number.isInteger(statusOrMessage) && statusOrMessage >= 400 && statusOrMessage <= 599
+        ? statusOrMessage
+        : 400;
+      const message = typeof messageOrDetails === "string" ? messageOrDetails : "Error";
+      return this.json({ status: "error", message, details: maybeDetails }, status);
+    }
+    return this.json({ status: "error", message: statusOrMessage, details: messageOrDetails }, 400);
+  }
 
   /** 401 Unauthorized */
   unauthorized(message: string = "Unauthorized"): Response {

package/src/generator/generate.ts

@@ -99,25 +99,25 @@ export interface GeneratedMap {
   frameworkPaths: string[];
 }
 
-async function ensureDir(dirPath: string): Promise<void> {
-  try {
-    await fs.mkdir(dirPath, { recursive: true });
-  } catch  {
-    // ignore if exists
-  }
-}
-
-function touchGenerateStamp(rootDir: string): void {
-  const stampDir = path.join(rootDir, ".mandu");
-  if (!fsSync.existsSync(stampDir)) {
-    fsSync.mkdirSync(stampDir, { recursive: true });
-  }
-  fsSync.writeFileSync(path.join(stampDir, "generate.stamp"), Date.now().toString());
-}
-
-async function getExistingFiles(dir: string): Promise<string[]> {
-  try {
-    const files = await fs.readdir(dir);
+async function ensureDir(dirPath: string): Promise<void> {
+  try {
+    await fs.mkdir(dirPath, { recursive: true });
+  } catch  {
+    // ignore if exists
+  }
+}
+
+function touchGenerateStamp(rootDir: string): void {
+  const stampDir = path.join(rootDir, ".mandu");
+  if (!fsSync.existsSync(stampDir)) {
+    fsSync.mkdirSync(stampDir, { recursive: true });
+  }
+  fsSync.writeFileSync(path.join(stampDir, "generate.stamp"), Date.now().toString());
+}
+
+async function getExistingFiles(dir: string): Promise<string[]> {
+  try {
+    const files = await fs.readdir(dir);
     return files.filter((f) => f.endsWith(".route.ts") || f.endsWith(".route.tsx"));
   } catch {
     return [];
@@ -146,10 +146,10 @@ export async function generateRoutes(
     warnings: [],
   };
 
-  // Suppress watcher during generation to avoid false positives
-  const watcher = getWatcher();
-  watcher?.suppress();
-  touchGenerateStamp(rootDir);
+  // Suppress watcher during generation to avoid false positives
+  const watcher = getWatcher();
+  watcher?.suppress();
+  touchGenerateStamp(rootDir);
 
   const generatedPaths = resolveGeneratedPaths(rootDir);
   const serverRoutesDir = generatedPaths.serverRoutesDir;
@@ -363,10 +363,10 @@ export async function generateRoutes(
   const mapPath = path.join(mapDir, "generated.map.json");
   await Bun.write(mapPath, JSON.stringify(generatedMap, null, 2));
 
-  // Cross-process timestamp: watcher skips warnings if generate finished recently
-  touchGenerateStamp(rootDir);
-  // Resume watcher after the stamp is visible.
-  watcher?.resume();
-
-  return result;
-}
+  // Cross-process timestamp: watcher skips warnings if generate finished recently
+  touchGenerateStamp(rootDir);
+  // Resume watcher after the stamp is visible.
+  watcher?.resume();
+
+  return result;
+}

package/src/generator/index.ts

@@ -1,3 +1,3 @@
-export * from "./generate";
-export * from "./templates";
-export * from "./contract-glue";
+export * from "./generate";
+export * from "./templates";
+export * from "./contract-glue";

package/src/generator/templates.test.ts

@@ -1,65 +1,66 @@
-import { describe, expect, test } from "bun:test";
-import type { RouteSpec } from "../spec/schema";
-import { generatePageComponent } from "./templates";
-
-describe("generatePageComponent", () => {
-  test("refuses to emit a placeholder for hydrating routes without a clientModule", () => {
-    const route: RouteSpec = {
-      id: "login",
-      kind: "page",
-      pattern: "/login",
-      module: "app/login/page.tsx",
-      componentModule: "app/login/page.tsx",
-      hydration: {
-        strategy: "full",
-        priority: "immediate",
-        preload: false,
-      },
-    };
-
-    expect(() => generatePageComponent(route)).toThrow("no clientModule");
-  });
-
-  test("imports the real page module for static routes instead of emitting a placeholder", () => {
-    const route: RouteSpec = {
-      id: "about",
-      kind: "page",
-      pattern: "/about",
-      module: "app/about/page.tsx",
-      componentModule: "app/about/page.tsx",
-    };
-
-    const generated = generatePageComponent(route);
-
-    expect(generated).toContain("Page Module: app/about/page.tsx");
-    expect(generated).toContain('import pageModule from "../../../../app/about/page.tsx"');
-    expect(generated).toContain("React.createElement(pageModule");
-    expect(generated).not.toContain("About Page");
-    expect(generated).not.toContain('React.createElement("p", null, "Route ID: about")');
-  });
-
-  test("renders route-level default-imported client components through the page module", () => {
-    const route: RouteSpec = {
-      id: "login",
-      kind: "page",
-      pattern: "/login",
-      module: "app/login/page.tsx",
-      componentModule: "app/login/page.tsx",
-      clientModule: "app/login/page.tsx",
-      hydration: {
-        strategy: "island",
-        priority: "immediate",
-        preload: false,
-      },
-    };
-
-    const generated = generatePageComponent(route);
-
-    expect(generated).toContain("Client Module: app/login/page.tsx");
-    expect(generated).toContain("Page Module: app/login/page.tsx");
-    expect(generated).toContain('import islandModule from "../../../../app/login/page.tsx"');
-    expect(generated).toContain("islandModule.definition.render");
-    expect(generated).toContain("React.createElement(islandModule");
-    expect(generated).not.toContain("Login Page");
-  });
-});
+import { describe, expect, test } from "bun:test";
+import type { RouteSpec } from "../spec/schema";
+import { generatePageComponent } from "./templates";
+
+describe("generatePageComponent", () => {
+  test("refuses to emit a placeholder for hydrating routes without a clientModule", () => {
+    const route: RouteSpec = {
+      id: "login",
+      kind: "page",
+      pattern: "/login",
+      module: "app/login/page.tsx",
+      componentModule: "app/login/page.tsx",
+      hydration: {
+        strategy: "full",
+        priority: "immediate",
+        preload: false,
+      },
+    };
+
+    expect(() => generatePageComponent(route)).toThrow("no clientModule");
+  });
+
+  test("imports the real page module for static routes instead of emitting a placeholder", () => {
+    const route: RouteSpec = {
+      id: "about",
+      kind: "page",
+      pattern: "/about",
+      module: "app/about/page.tsx",
+      componentModule: "app/about/page.tsx",
+    };
+
+    const generated = generatePageComponent(route);
+
+    expect(generated).toContain("Page Module: app/about/page.tsx");
+    expect(generated).toContain('import pageModule from "../../../../app/about/page.tsx"');
+    expect(generated).toContain("React.createElement(pageModule");
+    expect(generated).not.toContain("About Page");
+    expect(generated).not.toContain('React.createElement("p", null, "Route ID: about")');
+  });
+
+  test("renders route-level default-imported client components through the page module", () => {
+    const route: RouteSpec = {
+      id: "login",
+      kind: "page",
+      pattern: "/login",
+      module: "app/login/page.tsx",
+      componentModule: "app/login/page.tsx",
+      clientModule: "src/client/pages/login/LoginPage.client.tsx",
+      hydration: {
+        strategy: "island",
+        priority: "immediate",
+        preload: false,
+      },
+    };
+
+    const generated = generatePageComponent(route);
+
+    expect(generated).toContain("Client Module: src/client/pages/login/LoginPage.client.tsx");
+    expect(generated).toContain("Page Module: app/login/page.tsx");
+    expect(generated).toContain('import islandModule from "../../../../src/client/pages/login/LoginPage.client.tsx"');
+    expect(generated).toContain('import pageModule from "../../../../app/login/page.tsx"');
+    expect(generated).toContain("islandModule.definition.render");
+    expect(generated).toContain("React.createElement(pageModule");
+    expect(generated).not.toContain("Login Page");
+  });
+});