@mandujs/core 0.54.12 → 0.54.14

package/src/bundler/__tests__/cold-start.test.ts

@@ -28,13 +28,13 @@ import { describe, it, expect, beforeEach, afterEach } from "bun:test";
 import { mkdtempSync, writeFileSync, mkdirSync, rmSync } from "fs";
 import { tmpdir } from "os";
 import path from "path";
-import type { RouteSpec, RoutesManifest } from "../../spec/schema";
-import {
-  _testOnly_scanIslandFiles,
-  _testOnly_scanPartialFiles,
-  _testOnly_getHydratedRoutes,
-  _testOnly_getHydrationRoutesMissingClientModule,
-} from "../build";
+import type { RouteSpec, RoutesManifest } from "../../spec/schema";
+import {
+  _testOnly_scanIslandFiles,
+  _testOnly_scanPartialFiles,
+  _testOnly_getHydratedRoutes,
+  _testOnly_getHydrationRoutesMissingClientModule,
+} from "../build";
 import { HMR_PERF } from "../../perf/hmr-markers";
 import {
   _resetCacheForTesting as _resetPerfCache,
@@ -374,7 +374,7 @@ describe("Phase 7.1 R1 Agent C — getHydratedRoutes filter", () => {
   });
 });
 
-describe("Phase 7.1 R1 Agent C — per-island scan skips non-hydrated routes", () => {
+describe("Phase 7.1 R1 Agent C — per-island scan skips non-hydrated routes", () => {
   let project: ReturnType<typeof createProject>;
 
   beforeEach(() => {
@@ -502,55 +502,55 @@ describe("Phase 7.1 R1 Agent C — per-island scan skips non-hydrated routes", (
     ];
     const result = await _testOnly_scanIslandFiles(routes, project.rootDir);
     expect(result).toEqual([]);
-  });
-});
-
-describe("hydration route validation", () => {
-  it("finds pages that request hydration without a client module", () => {
-    const manifest = {
-      version: 1,
-      routes: [
-        {
-          id: "login",
-          pattern: "/login",
-          kind: "page",
-          module: "app/login/page.tsx",
-          componentModule: "app/login/page.tsx",
-          hydration: { strategy: "full", priority: "immediate", preload: false },
-        },
-        pureSsrRoute("about", "/about", "app/about"),
-        pageRoute("dashboard", "/dashboard", "app/dashboard"),
-      ],
-    } as RoutesManifest;
-
-    const missing = _testOnly_getHydrationRoutesMissingClientModule(manifest);
-    expect(missing.map((route) => route.id)).toEqual(["login"]);
-  });
-});
-
-describe("partial bundle scan", () => {
-  let project: ReturnType<typeof createProject>;
-
-  beforeEach(() => {
-    project = createProject();
-  });
-
-  afterEach(() => {
-    try {
-      rmSync(project.rootDir, { recursive: true, force: true });
-    } catch {
-      /* Windows lock tolerance */
-    }
-  });
-
-  it("discovers *.partial.tsx files outside .mandu and node_modules", async () => {
-    project.writeIslandFile("app", "Home.partial.tsx");
-    project.writeIslandFile(".mandu/client", "Stale.partial.tsx");
-    project.writeIslandFile("node_modules/pkg", "Ignored.partial.tsx");
-
-    const result = await _testOnly_scanPartialFiles(project.rootDir);
-
-    expect(result.map((entry) => entry.name)).toEqual(["Home"]);
-    expect(result[0].priority).toBe("visible");
-  });
-});
+  });
+});
+
+describe("hydration route validation", () => {
+  it("finds pages that request hydration without a client module", () => {
+    const manifest = {
+      version: 1,
+      routes: [
+        {
+          id: "login",
+          pattern: "/login",
+          kind: "page",
+          module: "app/login/page.tsx",
+          componentModule: "app/login/page.tsx",
+          hydration: { strategy: "full", priority: "immediate", preload: false },
+        },
+        pureSsrRoute("about", "/about", "app/about"),
+        pageRoute("dashboard", "/dashboard", "app/dashboard"),
+      ],
+    } as RoutesManifest;
+
+    const missing = _testOnly_getHydrationRoutesMissingClientModule(manifest);
+    expect(missing.map((route) => route.id)).toEqual(["login"]);
+  });
+});
+
+describe("partial bundle scan", () => {
+  let project: ReturnType<typeof createProject>;
+
+  beforeEach(() => {
+    project = createProject();
+  });
+
+  afterEach(() => {
+    try {
+      rmSync(project.rootDir, { recursive: true, force: true });
+    } catch {
+      /* Windows lock tolerance */
+    }
+  });
+
+  it("discovers *.partial.tsx files outside .mandu and node_modules", async () => {
+    project.writeIslandFile("app", "Home.partial.tsx");
+    project.writeIslandFile(".mandu/client", "Stale.partial.tsx");
+    project.writeIslandFile("node_modules/pkg", "Ignored.partial.tsx");
+
+    const result = await _testOnly_scanPartialFiles(project.rootDir);
+
+    expect(result.map((entry) => entry.name)).toEqual(["Home"]);
+    expect(result[0].priority).toBe("visible");
+  });
+});

package/src/bundler/__tests__/css.test.ts

@@ -1,20 +1,20 @@
-import { describe, expect, test } from "bun:test";
-import { __private } from "../css";
-
-describe("CSS Tailwind command resolution", () => {
-  test("uses process.execPath when it is Bun", () => {
-    expect(
-      __private.getTailwindCommand(["@tailwindcss/cli"], "C:\\tools\\bun.exe"),
-    ).toEqual(["C:\\tools\\bun.exe", "x", "@tailwindcss/cli"]);
-  });
-
-  test("does not treat standalone mandu.exe as Bun", () => {
-    expect(
-      __private.getTailwindCommand(
-        ["@tailwindcss/cli"],
-        "C:\\Users\\User\\AppData\\Local\\Mandu\\bin\\mandu.exe",
-        () => "C:\\Users\\User\\.bun\\bin\\bun.exe",
-      ),
-    ).toEqual(["C:\\Users\\User\\.bun\\bin\\bun.exe", "x", "@tailwindcss/cli"]);
-  });
-});
+import { describe, expect, test } from "bun:test";
+import { __private } from "../css";
+
+describe("CSS Tailwind command resolution", () => {
+  test("uses process.execPath when it is Bun", () => {
+    expect(
+      __private.getTailwindCommand(["@tailwindcss/cli"], "C:\\tools\\bun.exe"),
+    ).toEqual(["C:\\tools\\bun.exe", "x", "@tailwindcss/cli"]);
+  });
+
+  test("does not treat standalone mandu.exe as Bun", () => {
+    expect(
+      __private.getTailwindCommand(
+        ["@tailwindcss/cli"],
+        "C:\\Users\\User\\AppData\\Local\\Mandu\\bin\\mandu.exe",
+        () => "C:\\Users\\User\\.bun\\bin\\bun.exe",
+      ),
+    ).toEqual(["C:\\Users\\User\\.bun\\bin\\bun.exe", "x", "@tailwindcss/cli"]);
+  });
+});

package/src/bundler/__tests__/fast-refresh.test.ts

@@ -19,9 +19,9 @@
  *   packages/core/src/runtime/fast-refresh-runtime.ts
  */
 
-import { afterAll, beforeAll, beforeEach, describe, expect, test } from "bun:test";
-import { mkdtemp, mkdir, rm, writeFile, readFile } from "fs/promises";
-import path from "path";
+import { afterAll, beforeAll, beforeEach, describe, expect, test } from "bun:test";
+import { mkdtemp, mkdir, rm, writeFile, readFile } from "fs/promises";
+import path from "path";
 
 import {
   appendBoundary,
@@ -40,12 +40,19 @@ import {
   _isRefreshScheduledForTests,
   type ReactRefreshRuntime,
 } from "../../runtime/fast-refresh-runtime";
-import {
-  createManduHot,
-  dispatchReplacement,
-  _resetRegistryForTests,
-} from "../../runtime/hmr-client";
-import { generateFastRefreshPreamble } from "../dev";
+import {
+  createManduHot,
+  dispatchReplacement,
+  _resetRegistryForTests,
+} from "../../runtime/hmr-client";
+import { generateFastRefreshPreamble } from "../dev";
+
+const repoTempRoot = path.resolve(import.meta.dir, "../../../../..", ".tmp-test-artifacts");
+
+async function mkRepoTempDir(prefix: string): Promise<string> {
+  await mkdir(repoTempRoot, { recursive: true });
+  return mkdtemp(path.join(repoTempRoot, prefix));
+}
 
 // ═══════════════════════════════════════════════════════════════════
 // Section A — plugin pure unit tests
@@ -263,10 +270,10 @@ describe("manduHMR — __MANDU_HMR__ global behavior", () => {
 describe.skipIf(process.env.MANDU_SKIP_BUNDLER_TESTS === "1")(
   "Bun.build({ reactFastRefresh: true }) + fastRefreshPlugin()",
   () => {
-    let rootDir: string;
-
-    beforeAll(async () => {
-      rootDir = await mkdtemp(path.join(import.meta.dir, ".tmp-fr-build-"));
+    let rootDir: string;
+
+    beforeAll(async () => {
+      rootDir = await mkRepoTempDir("fr-build-");
       // Three source files — one boundary, one plain, one .island.tsx —
       // give us the minimal matrix for the plugin's include filter.
       await writeFile(
@@ -512,10 +519,10 @@ describe("dispatchReplacement + __MANDU_HMR__ integration", () => {
 describe.skipIf(process.env.MANDU_SKIP_BUNDLER_TESTS === "1")(
   "buildVendorShims emits fast-refresh shims in dev mode",
   () => {
-    let rootDir: string;
-
-    beforeAll(async () => {
-      rootDir = await mkdtemp(path.join(import.meta.dir, ".tmp-fr-vendor-"));
+    let rootDir: string;
+
+    beforeAll(async () => {
+      rootDir = await mkRepoTempDir("fr-vendor-");
       await mkdir(path.join(rootDir, "app"), { recursive: true });
       await writeFile(
         path.join(rootDir, "package.json"),

package/src/bundler/analyzer.ts

@@ -341,26 +341,26 @@ export async function analyzeBundle(
       deps: entry.dependencies ?? [],
     });
   }
-  for (const [islandName, entry] of Object.entries(manifest.islands ?? {})) {
-    // Avoid double-counting: if a per-island chunk shares its route id with
-    // a route-level bundle, we prefer the island entry (finer granularity).
-    const existing = islandSources.findIndex((s) => s.name === islandName);
+  for (const [islandName, entry] of Object.entries(manifest.islands ?? {})) {
+    // Avoid double-counting: if a per-island chunk shares its route id with
+    // a route-level bundle, we prefer the island entry (finer granularity).
+    const existing = islandSources.findIndex((s) => s.name === islandName);
     if (existing !== -1) islandSources.splice(existing, 1);
     islandSources.push({
       name: islandName,
       url: entry.js,
       priority: entry.priority,
-      deps: [],
-    });
-  }
-  for (const [partialName, entry] of Object.entries(manifest.partials ?? {})) {
-    islandSources.push({
-      name: `partial:${partialName}`,
-      url: entry.js,
-      priority: entry.priority,
-      deps: [],
-    });
-  }
+      deps: [],
+    });
+  }
+  for (const [partialName, entry] of Object.entries(manifest.partials ?? {})) {
+    islandSources.push({
+      name: `partial:${partialName}`,
+      url: entry.js,
+      priority: entry.priority,
+      deps: [],
+    });
+  }
 
   const islands: AnalyzeIsland[] = [];
   for (const src of islandSources) {

package/src/bundler/build.test.ts

@@ -1,11 +1,17 @@
-import { afterAll, beforeAll, describe, expect, test } from "bun:test";
-import { mkdtemp, mkdir, readFile, rm, writeFile } from "fs/promises";
-import path from "path";
-import { pathToFileURL } from "url";
-
-// 모든 테스트가 하나의 빌드 결과를 공유 — 병렬 Bun.build 충돌 방지
-let rootDir: string;
-let result: { success: boolean; errors: string[] };
+import { afterAll, beforeAll, describe, expect, test } from "bun:test";
+import { mkdtemp, mkdir, readFile, rm, writeFile } from "fs/promises";
+import path from "path";
+import { pathToFileURL } from "url";
+
+// 모든 테스트가 하나의 빌드 결과를 공유 — 병렬 Bun.build 충돌 방지
+let rootDir: string;
+let result: { success: boolean; errors: string[] };
+const repoTempRoot = path.resolve(import.meta.dir, "../../../..", ".tmp-test-artifacts");
+
+async function mkRepoTempDir(prefix: string): Promise<string> {
+  await mkdir(repoTempRoot, { recursive: true });
+  return mkdtemp(path.join(repoTempRoot, prefix));
+}
 
 async function importBuiltModule(relativePath: string): Promise<Record<string, unknown>> {
   const fileUrl = pathToFileURL(path.join(rootDir, relativePath)).href;
@@ -24,10 +30,13 @@ async function importBuiltModule(relativePath: string): Promise<Record<string, u
  * See `__tests__/build-runner.ts` for the subprocess entrypoint and more
  * background.
  */
-async function runBuildInSubprocess(root: string, mode?: string): Promise<{
-  success: boolean;
-  errors: string[];
-}> {
+async function runBuildInSubprocess(root: string, mode?: string): Promise<{
+  success: boolean;
+  errors: string[];
+  manifest?: {
+    bundles?: Record<string, { js?: string }>;
+  } | null;
+}> {
   const runner = path.join(
     import.meta.dir,
     "__tests__",
@@ -52,25 +61,27 @@ async function runBuildInSubprocess(root: string, mode?: string): Promise<{
     const last = lines[lines.length - 1] ?? "";
     try {
       const parsed = JSON.parse(last);
-      return {
-        success: parsed.success === true,
-        errors: Array.isArray(parsed.errors) ? parsed.errors : [],
-      };
+      return {
+        success: parsed.success === true,
+        errors: Array.isArray(parsed.errors) ? parsed.errors : [],
+        manifest: parsed.manifest ?? null,
+      };
     } catch (e) {
-      return {
-        success: false,
-        errors: [
-          `build-runner output could not be parsed as JSON: ${String(e)}\nLast stdout line: ${last}`,
-        ],
-      };
-    }
-  } catch (err) {
-    return { success: false, errors: [`spawn failed: ${String(err)}`] };
-  }
-}
+      return {
+        success: false,
+        errors: [
+          `build-runner output could not be parsed as JSON: ${String(e)}\nLast stdout line: ${last}`,
+        ],
+        manifest: null,
+      };
+    }
+  } catch (err) {
+    return { success: false, errors: [`spawn failed: ${String(err)}`], manifest: null };
+  }
+}
 
-beforeAll(async () => {
-  rootDir = await mkdtemp(path.join(import.meta.dir, ".tmp-bundler-"));
+beforeAll(async () => {
+  rootDir = await mkRepoTempDir("bundler-");
 
   await mkdir(path.join(rootDir, "app"), { recursive: true });
   await writeFile(
@@ -186,8 +197,8 @@ describe("buildClientBundles vendor shims", () => {
     expect(runtimeSource).toContain("JSON.parse");
   });
 
-  test("does not bundle a server page when stale manifest marks page.tsx as clientModule", async () => {
-    const staleRoot = await mkdtemp(path.join(import.meta.dir, ".tmp-stale-client-module-"));
+  test("does not bundle a server page when stale manifest marks page.tsx as clientModule", async () => {
+    const staleRoot = await mkRepoTempDir("stale-client-module-");
     try {
       await mkdir(path.join(staleRoot, "app"), { recursive: true });
       await mkdir(path.join(staleRoot, "src", "shared", "contracts"), { recursive: true });
@@ -219,8 +230,8 @@ describe("buildClientBundles vendor shims", () => {
     }
   });
 
-  test("rewrites route-component clientModule to the real client import before bundling", async () => {
-    const routeClientRoot = await mkdtemp(path.join(import.meta.dir, ".tmp-route-client-import-"));
+  test("rewrites route-component clientModule to the real client import before bundling", async () => {
+    const routeClientRoot = await mkRepoTempDir("route-client-import-");
     try {
       await mkdir(path.join(routeClientRoot, "app", "login"), { recursive: true });
       await mkdir(path.join(routeClientRoot, "src", "client", "pages", "login"), { recursive: true });
@@ -237,16 +248,16 @@ describe("buildClientBundles vendor shims", () => {
           "}\n",
         "utf-8",
       );
-      await writeFile(
-        path.join(routeClientRoot, "src", "client", "pages", "login", "LoginPage.client.tsx"),
-        '"use client";\n' +
-          'import { useState } from "react";\n' +
-          "export default function LoginPage() {\n" +
-          '  const [email, setEmail] = useState("");\n' +
-          '  return <form><input value={email} onChange={(event) => setEmail(event.currentTarget.value)} /></form>;\n' +
-          "}\n",
-        "utf-8",
-      );
+      await writeFile(
+        path.join(routeClientRoot, "src", "client", "pages", "login", "LoginPage.client.tsx"),
+        '"use client";\n' +
+          'import { useState } from "react";\n' +
+          "export default function LoginPage() {\n" +
+          '  const [email, setEmail] = useState("");\n' +
+          '  return <form><input value={email} onChange={(event) => setEmail(event.currentTarget.value)} /></form>;\n' +
+          "}\n",
+        "utf-8",
+      );
 
       const routeClientResult = await runBuildInSubprocess(routeClientRoot, "server-page-route-client-import");
       expect(routeClientResult.success).toBe(true);
@@ -263,11 +274,88 @@ describe("buildClientBundles vendor shims", () => {
       expect(parseResult.success).toBe(true);
     } finally {
       await rm(routeClientRoot, { recursive: true, force: true });
-    }
-  });
-
-  test("fails when hydration is enabled but no clientModule can be resolved", async () => {
-    const missingRoot = await mkdtemp(path.join(import.meta.dir, ".tmp-hydration-no-client-"));
+    }
+  });
+
+  test("bundles route-level named client exports without requiring a default export", async () => {
+    const routeClientRoot = await mkRepoTempDir("route-named-client-import-");
+    try {
+      await mkdir(path.join(routeClientRoot, "app", "login"), { recursive: true });
+      await mkdir(path.join(routeClientRoot, "src", "client", "pages", "login"), { recursive: true });
+      await writeFile(
+        path.join(routeClientRoot, "package.json"),
+        JSON.stringify({ name: "mandu-route-named-client-import-test", type: "module" }, null, 2),
+        "utf-8",
+      );
+      await writeFile(
+        path.join(routeClientRoot, "app", "login", "page.tsx"),
+        'import { LoginPage } from "@/client/pages/login/LoginPage.client";\n' +
+          "export default function Page() {\n" +
+          "  return <LoginPage />;\n" +
+          "}\n",
+        "utf-8",
+      );
+      await writeFile(
+        path.join(routeClientRoot, "src", "client", "pages", "login", "LoginPage.client.tsx"),
+        '"use client";\n' +
+          'import { useState } from "react";\n' +
+          "export function LoginPage() {\n" +
+          '  const [email, setEmail] = useState("");\n' +
+          '  return <form><input value={email} onChange={(event) => setEmail(event.currentTarget.value)} /></form>;\n' +
+          "}\n",
+        "utf-8",
+      );
+
+      const routeClientResult = await runBuildInSubprocess(routeClientRoot, "server-page-route-named-client-import");
+      if (!routeClientResult.success) {
+        console.error("[named route client] errors:", routeClientResult.errors);
+      }
+      expect(routeClientResult.success).toBe(true);
+      const bundlePath = path.join(routeClientRoot, ".mandu", "client", "login.island.js");
+      expect(await Bun.file(bundlePath).exists()).toBe(true);
+
+      const bundleSource = await readFile(bundlePath, "utf-8");
+      expect(bundleSource).toContain("LoginPage");
+      expect(bundleSource).not.toContain("Client islands cannot use server-side modules");
+      const parseResult = await Bun.build({
+        entrypoints: [bundlePath],
+        target: "browser",
+        external: ["react", "react-dom", "react-dom/client", "react/jsx-dev-runtime"],
+      });
+      expect(parseResult.success).toBe(true);
+    } finally {
+      await rm(routeClientRoot, { recursive: true, force: true });
+    }
+  });
+
+  test("uses Windows-safe asset filenames for locale-prefixed route ids", async () => {
+    const localeRoot = await mkRepoTempDir("i18n-locale-route-id-");
+    try {
+      await mkdir(path.join(localeRoot, "app"), { recursive: true });
+      await writeFile(
+        path.join(localeRoot, "package.json"),
+        JSON.stringify({ name: "mandu-i18n-route-id-test", type: "module" }, null, 2),
+        "utf-8",
+      );
+      await writeFile(
+        path.join(localeRoot, "app", "demo.client.tsx"),
+        "export default function DemoIsland() { return null; }\n",
+        "utf-8",
+      );
+
+      const localeResult = await runBuildInSubprocess(localeRoot, "i18n-locale-route-id");
+      expect(localeResult.success).toBe(true);
+      const bundle = localeResult.manifest?.bundles?.["ko::demo"];
+      expect(bundle?.js).toBe("/.mandu/client/ko_3a__3a_demo.island.js");
+      expect(bundle?.js).not.toContain(":");
+      expect(await Bun.file(path.join(localeRoot, ".mandu", "client", "ko_3a__3a_demo.island.js")).exists()).toBe(true);
+    } finally {
+      await rm(localeRoot, { recursive: true, force: true });
+    }
+  });
+
+  test("fails when hydration is enabled but no clientModule can be resolved", async () => {
+    const missingRoot = await mkRepoTempDir("hydration-no-client-");
     try {
       await mkdir(path.join(missingRoot, "app", "login"), { recursive: true });
       await mkdir(path.join(missingRoot, "src", "client", "widgets", "login-form"), { recursive: true });