@mandujs/core 0.54.18 → 0.54.19

package/src/client/serialize.ts

@@ -1,404 +1,50 @@
-/**
- * Mandu Props Serialization 📦
- * Fresh 스타일 고급 직렬화/역직렬화
- *
- * @see https://fresh.deno.dev/docs/concepts/islands
- *
- * 지원 타입:
- * - 원시형: null, boolean, number, string, bigint, undefined
- * - 특수 객체: Date, URL, RegExp, Map, Set
- * - 순환 참조
- * - 중첩 객체/배열
- */
-
-// ============================================
-// 타입 마커
-// ============================================
-
-const TYPE_MARKERS = {
-  /** undefined */
-  UNDEFINED: "\x00_",
-  /** Date */
-  DATE: "\x00D",
-  /** URL */
-  URL: "\x00U",
-  /** RegExp */
-  REGEXP: "\x00R",
-  /** Map */
-  MAP: "\x00M",
-  /** Set */
-  SET: "\x00S",
-  /** 순환 참조 */
-  REF: "\x00$",
-  /** BigInt */
-  BIGINT: "\x00B",
-  /** Symbol (제한적 지원) */
-  SYMBOL: "\x00Y",
-  /** Error */
-  ERROR: "\x00E",
-} as const;
-
-// ============================================
-// 직렬화
-// ============================================
-
-/**
- * 직렬화 컨텍스트 (순환 참조 추적)
- */
-interface SerializeContext {
-  /** 이미 본 객체 → 인덱스 */
-  seen: Map<object, number>;
-  /** 참조 테이블 */
-  refs: object[];
-}
-
-/**
- * Props 직렬화
- *
- * @example
- * ```typescript
- * const props = {
- *   date: new Date(),
- *   url: new URL('https://example.com'),
- *   items: new Set([1, 2, 3]),
- *   cache: new Map([['key', 'value']]),
- * };
- *
- * const json = serializeProps(props);
- * // 클라이언트로 전송
- * ```
- */
-export function serializeProps(props: Record<string, unknown>): string {
-  const ctx: SerializeContext = { seen: new Map(), refs: [] };
-  return JSON.stringify(serialize(props, ctx));
-}
-
-/**
- * 값 직렬화 (재귀)
- */
-function serialize(value: unknown, ctx: SerializeContext): unknown {
-  // null
-  if (value === null) return null;
-
-  // undefined
-  if (value === undefined) return TYPE_MARKERS.UNDEFINED;
-
-  // 원시형
-  if (typeof value === "boolean" || typeof value === "number") {
-    return value;
-  }
-
-  if (typeof value === "string") {
-    // 타입 마커와 충돌 방지 (첫 문자가 \x00인 경우)
-    if (value.startsWith("\x00")) {
-      return "\x00\x00" + value;
-    }
-    return value;
-  }
-
-  if (typeof value === "bigint") {
-    return TYPE_MARKERS.BIGINT + value.toString();
-  }
-
-  if (typeof value === "symbol") {
-    // Symbol은 description만 보존
-    return TYPE_MARKERS.SYMBOL + (value.description ?? "");
-  }
-
-  // 함수는 직렬화 불가
-  if (typeof value === "function") {
-    console.warn("[Mandu Serialize] Functions cannot be serialized, skipping");
-    return undefined;
-  }
-
-  // 객체 순환 참조 체크
-  if (typeof value === "object") {
-    const existing = ctx.seen.get(value);
-    if (existing !== undefined) {
-      return TYPE_MARKERS.REF + existing;
-    }
-
-    const idx = ctx.refs.length;
-    ctx.seen.set(value, idx);
-    ctx.refs.push(value);
-  }
-
-  // Date
-  if (value instanceof Date) {
-    return TYPE_MARKERS.DATE + value.toISOString();
-  }
-
-  // URL
-  if (value instanceof URL) {
-    return TYPE_MARKERS.URL + value.href;
-  }
-
-  // RegExp
-  if (value instanceof RegExp) {
-    return TYPE_MARKERS.REGEXP + value.toString();
-  }
-
-  // Error
-  if (value instanceof Error) {
-    return [
-      TYPE_MARKERS.ERROR,
-      value.name,
-      value.message,
-      value.stack ?? "",
-    ];
-  }
-
-  // Map
-  if (value instanceof Map) {
-    const entries: [unknown, unknown][] = [];
-    for (const [k, v] of value.entries()) {
-      entries.push([serialize(k, ctx), serialize(v, ctx)]);
-    }
-    return [TYPE_MARKERS.MAP, ...entries];
-  }
-
-  // Set
-  if (value instanceof Set) {
-    const items: unknown[] = [];
-    for (const item of value) {
-      items.push(serialize(item, ctx));
-    }
-    return [TYPE_MARKERS.SET, ...items];
-  }
-
-  // 배열
-  if (Array.isArray(value)) {
-    return value.map((item) => serialize(item, ctx));
-  }
-
-  // 일반 객체
-  const result: Record<string, unknown> = {};
-  for (const [k, v] of Object.entries(value as object)) {
-    const serialized = serialize(v, ctx);
-    if (serialized !== undefined) {
-      result[k] = serialized;
-    }
-  }
-  return result;
-}
-
-// ============================================
-// 역직렬화
-// ============================================
-
-/**
- * 역직렬화 컨텍스트 (순환 참조 복원)
- */
-interface DeserializeContext {
-  refs: unknown[];
-}
-
-/**
- * Props 역직렬화
- *
- * @example
- * ```typescript
- * // 서버에서 받은 JSON
- * const json = '{"date":"\x00D2025-01-28T00:00:00.000Z"}';
- *
- * const props = deserializeProps(json);
- * console.log(props.date instanceof Date); // true
- * ```
- */
-export function deserializeProps(json: string): Record<string, unknown> {
-  const ctx: DeserializeContext = { refs: [] };
-  const parsed = JSON.parse(json);
-  return deserialize(parsed, ctx) as Record<string, unknown>;
-}
-
-/**
- * 값 역직렬화 (재귀)
- */
-function deserialize(value: unknown, ctx: DeserializeContext): unknown {
-  // null
-  if (value === null) return null;
-
-  // 문자열 → 타입 마커 체크
-  if (typeof value === "string") {
-    // undefined
-    if (value === TYPE_MARKERS.UNDEFINED) return undefined;
-
-    // 이스케이프된 문자열 (\x00\x00 → \x00)
-    if (value.startsWith("\x00\x00")) {
-      return value.slice(2);
-    }
-
-    // Date
-    if (value.startsWith(TYPE_MARKERS.DATE)) {
-      return new Date(value.slice(2));
-    }
-
-    // URL
-    if (value.startsWith(TYPE_MARKERS.URL)) {
-      return new URL(value.slice(2));
-    }
-
-    // RegExp
-    if (value.startsWith(TYPE_MARKERS.REGEXP)) {
-      const str = value.slice(2);
-      const match = str.match(/^\/(.*)\/([gimsuy]*)$/);
-      if (match) {
-        return new RegExp(match[1], match[2]);
-      }
-      return str; // 파싱 실패 시 문자열 반환
-    }
-
-    // BigInt
-    if (value.startsWith(TYPE_MARKERS.BIGINT)) {
-      return BigInt(value.slice(2));
-    }
-
-    // Symbol
-    if (value.startsWith(TYPE_MARKERS.SYMBOL)) {
-      return Symbol(value.slice(2));
-    }
-
-    // 순환 참조
-    if (value.startsWith(TYPE_MARKERS.REF)) {
-      const idx = parseInt(value.slice(2), 10);
-      return ctx.refs[idx];
-    }
-
-    return value;
-  }
-
-  // 원시형
-  if (typeof value === "boolean" || typeof value === "number") {
-    return value;
-  }
-
-  // 배열 → 특수 타입 체크
-  if (Array.isArray(value)) {
-    const marker = value[0];
-
-    // Error
-    if (marker === TYPE_MARKERS.ERROR) {
-      const [, name, message, stack] = value as [string, string, string, string];
-      const error = new Error(message);
-      error.name = name;
-      if (stack) error.stack = stack;
-      ctx.refs.push(error);
-      return error;
-    }
-
-    // Map
-    if (marker === TYPE_MARKERS.MAP) {
-      const map = new Map();
-      ctx.refs.push(map);
-      for (let i = 1; i < value.length; i++) {
-        const [k, v] = value[i] as [unknown, unknown];
-        map.set(deserialize(k, ctx), deserialize(v, ctx));
-      }
-      return map;
-    }
-
-    // Set
-    if (marker === TYPE_MARKERS.SET) {
-      const set = new Set();
-      ctx.refs.push(set);
-      for (let i = 1; i < value.length; i++) {
-        set.add(deserialize(value[i], ctx));
-      }
-      return set;
-    }
-
-    // 일반 배열
-    const arr: unknown[] = [];
-    ctx.refs.push(arr);
-    for (const item of value) {
-      arr.push(deserialize(item, ctx));
-    }
-    return arr;
-  }
-
-  // 일반 객체
-  if (typeof value === "object") {
-    const obj: Record<string, unknown> = {};
-    ctx.refs.push(obj);
-    for (const [k, v] of Object.entries(value)) {
-      obj[k] = deserialize(v, ctx);
-    }
-    return obj;
-  }
-
-  return value;
-}
-
-// ============================================
-// 유틸리티
-// ============================================
-
-/**
- * 직렬화 가능 여부 체크
- */
-export function isSerializable(value: unknown): boolean {
-  if (value === null || value === undefined) return true;
-
-  const type = typeof value;
-  if (type === "boolean" || type === "number" || type === "string" || type === "bigint") {
-    return true;
-  }
-
-  if (type === "function" || type === "symbol") {
-    return false;
-  }
-
-  if (value instanceof Date || value instanceof URL || value instanceof RegExp) {
-    return true;
-  }
-
-  if (value instanceof Map || value instanceof Set) {
-    return true;
-  }
-
-  if (Array.isArray(value)) {
-    return value.every(isSerializable);
-  }
-
-  if (type === "object") {
-    return Object.values(value as object).every(isSerializable);
-  }
-
-  return false;
-}
-
-/**
- * SSR에서 클라이언트로 props 전달용 스크립트 생성
- */
-export function generatePropsScript(
-  islandId: string,
-  props: Record<string, unknown>
-): string {
-  const json = serializeProps(props);
-  const escaped = json
-    .replace(/</g, "\\u003c")
-    .replace(/>/g, "\\u003e")
-    .replace(/&/g, "\\u0026");
-
-  return `<script type="application/json" data-mandu-props="${islandId}">${escaped}</script>`;
-}
-
-/**
- * 클라이언트에서 props 스크립트 파싱
- */
-export function parsePropsScript(islandId: string): Record<string, unknown> | null {
-  if (typeof document === "undefined") return null;
-
-  const script = document.querySelector(
-    `script[data-mandu-props="${islandId}"]`
-  ) as HTMLScriptElement | null;
-
-  if (!script?.textContent) return null;
-
-  try {
-    return deserializeProps(script.textContent);
-  } catch (err) {
-    console.error(`[Mandu] Failed to parse props for island ${islandId}:`, err);
-    return null;
-  }
-}
+/**
+ * Mandu props serialization public API.
+ *
+ * Core serialization/deserialization lives in `props-serialization.ts` so the
+ * browser runtime and server render path share exactly one wire format.
+ */
+
+import { deserializeProps, serializeProps } from "./props-serialization";
+
+export {
+  deserializeProps,
+  isSerializable,
+  serializeProps,
+} from "./props-serialization";
+
+/**
+ * SSR에서 클라이언트로 props 전달용 스크립트 생성
+ */
+export function generatePropsScript(
+  islandId: string,
+  props: Record<string, unknown>
+): string {
+  const json = serializeProps(props);
+  const escaped = json
+    .replace(/</g, "\\u003c")
+    .replace(/>/g, "\\u003e")
+    .replace(/&/g, "\\u0026");
+
+  return `<script type="application/json" data-mandu-props="${islandId}">${escaped}</script>`;
+}
+
+/**
+ * 클라이언트에서 props 스크립트 파싱
+ */
+export function parsePropsScript(islandId: string): Record<string, unknown> | null {
+  if (typeof document === "undefined") return null;
+
+  const script = document.querySelector(
+    `script[data-mandu-props="${islandId}"]`
+  ) as HTMLScriptElement | null;
+
+  if (!script?.textContent) return null;
+
+  try {
+    return deserializeProps(script.textContent);
+  } catch (err) {
+    console.error(`[Mandu] Failed to parse props for island ${islandId}:`, err);
+    return null;
+  }
+}

package/src/diagnose/__tests__/checks.test.ts

@@ -457,6 +457,21 @@ describe("checkClientBoundaryManifests", () => {
     expect(result.details?.missingEntries).toEqual(["home--0"]);
   });
 
+  it("treats a missing bundle manifest as an error when routes declare compiler-owned boundaries", async () => {
+    await writeFile(rootDir, ".mandu/routes.manifest.json", JSON.stringify({
+      routes: [
+        {
+          id: "home",
+          boundaries: [{ id: "home--0", routeId: "home", module: "src/client/Home.client.tsx", exportName: "Home" }],
+        },
+      ],
+    }));
+    const result = await checkClientBoundaryManifests(rootDir);
+    expect(result.ok).toBe(false);
+    expect(result.severity).toBe("error");
+    expect(result.message).toMatch(/bundle manifest is missing/);
+  });
+
   it("passes when route and bundle boundary manifests line up", async () => {
     await writeFile(rootDir, ".mandu/routes.manifest.json", JSON.stringify({
       routes: [

package/src/diagnose/checks.ts

@@ -964,7 +964,7 @@ export async function checkClientBoundaryManifests(rootDir: string): Promise<Dia
     return {
       ok: false,
       rule: "client_boundary_manifests",
-      severity: "warning",
+      severity: "error",
       message: `Routes manifest declares ${boundaries.length} client boundary record(s), but bundle manifest is missing.`,
       suggestion: "Run `mandu build` before deploy so boundary bundles are emitted and can be checked.",
       details: { boundaryCount: boundaries.length, bundleManifestPath: path.relative(rootDir, bundleManifestPath) },

package/src/router/client-entry.test.ts

@@ -1,13 +1,14 @@
 import { describe, expect, it } from "bun:test";
 import { mkdir, mkdtemp, rm, writeFile } from "fs/promises";
 import path from "path";
-import {
-  findClientComponentImports,
+import {
+  findClientComponentImports,
   findRouteLevelClientComponentImport,
   findRouteLevelClientComponentImports,
   resolveRouteLevelClientEntryPath,
   validateClientModuleForBrowserBundle,
 } from "./client-entry";
+import { analyzeRouteSource } from "./route-source-analyzer";
 
 const repoTempRoot = path.resolve(import.meta.dir, "../../../..", ".tmp-test-artifacts");
 
@@ -17,10 +18,10 @@ async function mkRepoTempDir(prefix: string): Promise<string> {
 }
 
 describe("findClientComponentImports", () => {
-  it("detects named .client imports for diagnostics", () => {
-    const imports = findClientComponentImports(`
-      import { LoginForm, SubmitButton as Button } from "@/client/widgets/login-form/LoginForm.client";
-      import Header from "./Header.client.tsx";
+  it("detects named .client imports for diagnostics", () => {
+    const imports = findClientComponentImports(`
+      import { LoginForm, SubmitButton as Button } from "@/client/widgets/login-form/LoginForm.client";
+      import Header from "./Header.client.tsx";
     `);
 
     expect(imports).toEqual([
@@ -33,9 +34,34 @@ describe("findClientComponentImports", () => {
         module: "./Header.client.tsx",
         kind: "default",
         names: ["Header"],
-      },
-    ]);
-  });
+      },
+    ]);
+  });
+
+  it("ignores import syntax inside string literals", () => {
+    const imports = findClientComponentImports(`
+      const example = "import Fake from './Fake.client'";
+
+      export default function Page() {
+        return <main>{example}</main>;
+      }
+    `);
+
+    expect(imports).toEqual([]);
+  });
+
+  it("ignores type-only client imports", () => {
+    const imports = findClientComponentImports(`
+      import type { LoginForm } from "@/client/widgets/login-form/LoginForm.client";
+      import { type SubmitButton } from "@/client/widgets/login-form/LoginForm.client";
+
+      export default function Page() {
+        return <main />;
+      }
+    `);
+
+    expect(imports).toEqual([]);
+  });
 
   it("detects a default-imported client component when the page returns only that component", () => {
     const routeClient = findRouteLevelClientComponentImport(`
@@ -125,11 +151,11 @@ describe("findClientComponentImports", () => {
     });
   });
 
-  it("promotes client imports when the page wrapper passes props", () => {
-    const routeClient = findRouteLevelClientComponentImport(`
-      import PledgePage from "@/client/pages/pledges/PledgePage.client";
-
-      export default function Page({ params }) {
+  it("promotes client imports when the page wrapper passes props", () => {
+    const routeClient = findRouteLevelClientComponentImport(`
+      import PledgePage from "@/client/pages/pledges/PledgePage.client";
+
+      export default function Page({ params }) {
         return <PledgePage id={params.id} />;
       }
     `);
@@ -139,7 +165,23 @@ describe("findClientComponentImports", () => {
       localName: "PledgePage",
       exportName: "default",
     });
-  });
+  });
+
+  it("detects client entries rendered by an identifier default export", () => {
+    const routeClient = findRouteLevelClientComponentImport(`
+      import PledgePage from "@/client/pages/pledges/PledgePage.client";
+
+      const Page = ({ params }) => <PledgePage id={params.id} />;
+
+      export default Page;
+    `);
+
+    expect(routeClient).toEqual({
+      module: "@/client/pages/pledges/PledgePage.client",
+      localName: "PledgePage",
+      exportName: "default",
+    });
+  });
 
   it("detects multiple rendered client imports in a server shell", () => {
     const routeClients = findRouteLevelClientComponentImports(`
@@ -188,17 +230,63 @@ describe("findClientComponentImports", () => {
     });
   });
 
-  it("does not promote shared UI primitives to route-level client entries", () => {
-    const routeClient = findRouteLevelClientComponentImport(`
-      import { Button } from "@/client/shared/ui/button";
-
-      export default function Page() {
+  it("does not promote shared UI primitives to route-level client entries", () => {
+    const routeClient = findRouteLevelClientComponentImport(`
+      import { Button } from "@/client/shared/ui/button";
+
+      export default function Page() {
         return <main><Button>Open</Button></main>;
       }
     `);
-
-    expect(routeClient).toBeNull();
-  });
+
+    expect(routeClient).toBeNull();
+  });
+
+  it("classifies default re-exports without treating them as rendered JSX", () => {
+    const analysis = analyzeRouteSource(`
+      export { default } from "./Page";
+    `);
+
+    expect(analysis.defaultExport).toMatchObject({
+      kind: "identifier",
+      localName: "default",
+      source: "./Page",
+      referencesJsx: false,
+      renderedJsxNames: [],
+    });
+  });
+
+  it("diagnoses default export wrapper calls conservatively", () => {
+    const analysis = analyzeRouteSource(`
+      import { memo } from "react";
+
+      const Page = () => <main />;
+
+      export default memo(Page);
+    `);
+
+    expect(analysis.defaultExport.kind).toBe("call");
+    expect(analysis.diagnostics.some((diagnostic) =>
+      diagnostic.code === "MANDU_ROUTE_DEFAULT_EXPORT_WRAPPER"
+    )).toBe(true);
+  });
+
+  it("records namespace imports without promoting member JSX as a route-level client entry", () => {
+    const source = `
+      import * as Widgets from "@/client/widgets/Widget.client";
+
+      export default function Page() {
+        return <Widgets.Panel />;
+      }
+    `;
+    const analysis = analyzeRouteSource(source);
+
+    expect(analysis.imports[0]).toMatchObject({
+      source: "@/client/widgets/Widget.client",
+      namespaceName: "Widgets",
+    });
+    expect(findRouteLevelClientComponentImport(source)).toBeNull();
+  });
 
   it("resolves a route-level client entry by reading a use client target without .client in the path", async () => {
     const rootDir = await mkRepoTempDir("client-entry-");