@mandujs/core 0.54.32 → 0.55.0-beta.1

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

@@ -16,10 +16,10 @@ import {
   checkPrerenderPollution,
   checkCloneElementWarnings,
   checkDevArtifactsInProd,
-  checkPackageExportGaps,
-  checkNestedInternalCore,
-  checkClientBoundaryManifests,
-} from "../checks";
+  checkPackageExportGaps,
+  checkNestedInternalCore,
+  checkClientBoundaryManifests,
+} from "../checks";
 import { runExtendedDiagnose, buildReport } from "../run";
 
 async function mkTmpRoot(): Promise<string> {
@@ -281,7 +281,7 @@ describe("checkPackageExportGaps", () => {
   }
 
   it("skips gracefully when @mandujs/core is not installed", async () => {
-    await writeFile(rootDir, "src/foo.ts", `import { x } from "@mandujs/core/unknown";`);
+    await writeFile(rootDir, "src/foo.ts", `import { x } from "@mandujs/core/compat/unknown";`);
     const result = await checkPackageExportGaps(rootDir);
     expect(result.ok).toBe(true);
     expect(result.details?.skipped).toBe(true);
@@ -298,7 +298,7 @@ describe("checkPackageExportGaps", () => {
 
   it("flags an import missing from the exports map", async () => {
     await seedCore({ ".": "./src/index.ts", "./client": "./src/client/index.ts" });
-    await writeFile(rootDir, "src/bad.ts", `import { x } from "@mandujs/core/nonexistent";`);
+    await writeFile(rootDir, "src/bad.ts", `import { x } from "@mandujs/core/compat/nonexistent";`);
     const result = await checkPackageExportGaps(rootDir);
     expect(result.ok).toBe(false);
     expect(result.severity).toBe("error");
@@ -307,14 +307,14 @@ describe("checkPackageExportGaps", () => {
 
   it("honors ./* wildcard export as a catch-all", async () => {
     await seedCore({ ".": "./src/index.ts", "./*": "./src/*" });
-    await writeFile(rootDir, "src/a.ts", `import { x } from "@mandujs/core/anything";`);
+    await writeFile(rootDir, "src/a.ts", `import { x } from "@mandujs/core/compat/anything";`);
     const result = await checkPackageExportGaps(rootDir);
     expect(result.ok).toBe(true);
   });
 
   it("recognizes require() specifiers in addition to import", async () => {
     await seedCore({ ".": "./src/index.ts" });
-    await writeFile(rootDir, "src/a.cjs", `const { x } = require("@mandujs/core/ghost");`);
+    await writeFile(rootDir, "src/a.cjs", `const { x } = require("@mandujs/core/compat/ghost");`);
     const result = await checkPackageExportGaps(rootDir);
     expect(result.ok).toBe(false);
     expect(result.severity).toBe("error");
@@ -325,7 +325,7 @@ describe("checkPackageExportGaps", () => {
 // nested_internal_core (#261)
 // ──────────────────────────────────────────────────────────────────
 
-describe("checkNestedInternalCore", () => {
+describe("checkNestedInternalCore", () => {
   let rootDir: string;
   beforeEach(async () => { rootDir = await mkTmpRoot(); });
   afterEach(async () => { await fs.rm(rootDir, { recursive: true, force: true }); });
@@ -388,134 +388,134 @@ describe("checkNestedInternalCore", () => {
     expect(result.ok).toBe(false);
     expect(result.details?.mismatchCount).toBe(2);
   });
-});
-
-// ──────────────────────────────────────────────────────────────────
-// client_boundary_manifests
-// ──────────────────────────────────────────────────────────────────
-
-describe("checkClientBoundaryManifests", () => {
-  let rootDir: string;
-  beforeEach(async () => { rootDir = await mkTmpRoot(); });
-  afterEach(async () => { await fs.rm(rootDir, { recursive: true, force: true }); });
-
-  it("skips gracefully when routes manifest is missing", async () => {
-    const result = await checkClientBoundaryManifests(rootDir);
-    expect(result.ok).toBe(true);
-    expect(result.details?.skipped).toBe(true);
-  });
-
-  it("passes when the routes manifest has no compiler-owned boundaries", async () => {
-    await writeFile(rootDir, ".mandu/routes.manifest.json", JSON.stringify({
-      routes: [{ id: "home", kind: "page" }],
-    }));
-    const result = await checkClientBoundaryManifests(rootDir);
-    expect(result.ok).toBe(true);
-    expect(result.details?.boundaryCount).toBe(0);
-  });
-
-  it("flags duplicate client boundary ids in the routes manifest", async () => {
-    await writeFile(rootDir, ".mandu/routes.manifest.json", JSON.stringify({
-      routes: [
-        {
-          id: "a",
-          boundaries: [{ id: "dup--0", routeId: "a", module: "src/client/A.client.tsx", exportName: "A" }],
-        },
-        {
-          id: "b",
-          boundaries: [{ id: "dup--0", routeId: "b", module: "src/client/B.client.tsx", exportName: "B" }],
-        },
-      ],
-    }));
-    const result = await checkClientBoundaryManifests(rootDir);
-    expect(result.ok).toBe(false);
-    expect(result.severity).toBe("error");
-    expect(result.message).toMatch(/duplicate/);
-  });
-
-  it("flags missing boundary bundle manifest entries", 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" }],
-        },
-      ],
-    }));
-    await writeFile(rootDir, ".mandu/manifest.json", JSON.stringify({
-      version: 1,
-      buildTime: "x",
-      env: "production",
-      bundles: {},
-      boundaries: {},
-      shared: { runtime: "/.mandu/client/_runtime.js", vendor: "/.mandu/client/_react.js" },
-    }));
-    const result = await checkClientBoundaryManifests(rootDir);
-    expect(result.ok).toBe(false);
-    expect(result.severity).toBe("error");
-    expect(result.message).toMatch(/incomplete/);
-    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: [
-        {
-          id: "home",
-          boundaries: [{ id: "home--0", routeId: "home", module: "src/client/Home.client.tsx", exportName: "Home" }],
-        },
-      ],
-    }));
-    await writeFile(rootDir, ".mandu/client/home--0.boundary.js", "export default function Home() {}\n");
-    await writeFile(rootDir, ".mandu/manifest.json", JSON.stringify({
-      version: 1,
-      buildTime: "x",
-      env: "production",
-      bundles: {},
-      boundaries: {
-        "home--0": {
-          route: "home",
-          js: "/.mandu/client/home--0.boundary.js",
-          module: "src/client/Home.client.tsx",
-          exportName: "Home",
-        },
-      },
-      shared: { runtime: "/.mandu/client/_runtime.js", vendor: "/.mandu/client/_react.js" },
-    }));
-    const result = await checkClientBoundaryManifests(rootDir);
-    expect(result.ok).toBe(true);
-    expect(result.details?.boundaryCount).toBe(1);
-  });
-});
-
-// ──────────────────────────────────────────────────────────────────
-// aggregator
-// ──────────────────────────────────────────────────────────────────
+});
+
+// ──────────────────────────────────────────────────────────────────
+// client_boundary_manifests
+// ──────────────────────────────────────────────────────────────────
+
+describe("checkClientBoundaryManifests", () => {
+  let rootDir: string;
+  beforeEach(async () => { rootDir = await mkTmpRoot(); });
+  afterEach(async () => { await fs.rm(rootDir, { recursive: true, force: true }); });
+
+  it("skips gracefully when routes manifest is missing", async () => {
+    const result = await checkClientBoundaryManifests(rootDir);
+    expect(result.ok).toBe(true);
+    expect(result.details?.skipped).toBe(true);
+  });
+
+  it("passes when the routes manifest has no compiler-owned boundaries", async () => {
+    await writeFile(rootDir, ".mandu/routes.manifest.json", JSON.stringify({
+      routes: [{ id: "home", kind: "page" }],
+    }));
+    const result = await checkClientBoundaryManifests(rootDir);
+    expect(result.ok).toBe(true);
+    expect(result.details?.boundaryCount).toBe(0);
+  });
+
+  it("flags duplicate client boundary ids in the routes manifest", async () => {
+    await writeFile(rootDir, ".mandu/routes.manifest.json", JSON.stringify({
+      routes: [
+        {
+          id: "a",
+          boundaries: [{ id: "dup--0", routeId: "a", module: "src/client/A.client.tsx", exportName: "A" }],
+        },
+        {
+          id: "b",
+          boundaries: [{ id: "dup--0", routeId: "b", module: "src/client/B.client.tsx", exportName: "B" }],
+        },
+      ],
+    }));
+    const result = await checkClientBoundaryManifests(rootDir);
+    expect(result.ok).toBe(false);
+    expect(result.severity).toBe("error");
+    expect(result.message).toMatch(/duplicate/);
+  });
+
+  it("flags missing boundary bundle manifest entries", 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" }],
+        },
+      ],
+    }));
+    await writeFile(rootDir, ".mandu/manifest.json", JSON.stringify({
+      version: 1,
+      buildTime: "x",
+      env: "production",
+      bundles: {},
+      boundaries: {},
+      shared: { runtime: "/.mandu/client/_runtime.js", vendor: "/.mandu/client/_react.js" },
+    }));
+    const result = await checkClientBoundaryManifests(rootDir);
+    expect(result.ok).toBe(false);
+    expect(result.severity).toBe("error");
+    expect(result.message).toMatch(/incomplete/);
+    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: [
+        {
+          id: "home",
+          boundaries: [{ id: "home--0", routeId: "home", module: "src/client/Home.client.tsx", exportName: "Home" }],
+        },
+      ],
+    }));
+    await writeFile(rootDir, ".mandu/client/home--0.boundary.js", "export default function Home() {}\n");
+    await writeFile(rootDir, ".mandu/manifest.json", JSON.stringify({
+      version: 1,
+      buildTime: "x",
+      env: "production",
+      bundles: {},
+      boundaries: {
+        "home--0": {
+          route: "home",
+          js: "/.mandu/client/home--0.boundary.js",
+          module: "src/client/Home.client.tsx",
+          exportName: "Home",
+        },
+      },
+      shared: { runtime: "/.mandu/client/_runtime.js", vendor: "/.mandu/client/_react.js" },
+    }));
+    const result = await checkClientBoundaryManifests(rootDir);
+    expect(result.ok).toBe(true);
+    expect(result.details?.boundaryCount).toBe(1);
+  });
+});
+
+// ──────────────────────────────────────────────────────────────────
+// aggregator
+// ──────────────────────────────────────────────────────────────────
 
 describe("runExtendedDiagnose", () => {
   let rootDir: string;
   beforeEach(async () => { rootDir = await mkTmpRoot(); });
   afterEach(async () => { await fs.rm(rootDir, { recursive: true, force: true }); });
 
-  it("runs all 8 extended checks and returns a structured report", async () => {
-    const report = await runExtendedDiagnose(rootDir);
-    // F42/F45 added `client_boundary_manifests` — total is now 8.
-    expect(report.summary.total).toBe(8);
+  it("runs all 8 extended checks and returns a structured report", async () => {
+    const report = await runExtendedDiagnose(rootDir);
+    // F42/F45 added `client_boundary_manifests` — total is now 8.
+    expect(report.summary.total).toBe(8);
     // manifest is missing → at least one error
     expect(report.healthy).toBe(false);
     expect(report.errorCount).toBeGreaterThanOrEqual(1);
@@ -524,11 +524,11 @@ describe("runExtendedDiagnose", () => {
     expect(rules).toContain("prerender_pollution");
     expect(rules).toContain("cloneelement_warnings");
     expect(rules).toContain("dev_artifacts_in_prod");
-    expect(rules).toContain("package_export_gaps");
-    expect(rules).toContain("nested_internal_core");
-    expect(rules).toContain("client_boundary_manifests");
-    expect(rules).toContain("a11y_hints");
-  });
+    expect(rules).toContain("package_export_gaps");
+    expect(rules).toContain("nested_internal_core");
+    expect(rules).toContain("client_boundary_manifests");
+    expect(rules).toContain("a11y_hints");
+  });
 
   it("returns healthy=true when all checks pass (production manifest, no gaps)", async () => {
     await writeFile(rootDir, ".mandu/manifest.json", JSON.stringify({