@mandujs/core 0.54.32 → 0.55.0-beta.1

package/src/diagnose/checks.ts

@@ -549,7 +549,7 @@ export async function checkPackageExportGaps(rootDir: string): Promise<DiagnoseC
   }
 
   const files = await collectSourceFiles(rootDir);
-  // Match both `from "@mandujs/core/..."` and `require("@mandujs/core/...")` forms.
+  // Match both `from "@mandujs/core/compat/..."` and `require("@mandujs/core/compat/...")` forms.
   const importRegex = /(?:from|require\()\s*["']([^"']+)["']/g;
   const userSubpaths = new Map<string, string[]>();
 
@@ -767,7 +767,7 @@ async function listMandujsSiblings(rootDir: string): Promise<string[]> {
  * version to the hoisted core. A mismatch is reported as `error`
  * (boot-breaking on the user's machine) with a copy-pastable fix.
  */
-export async function checkNestedInternalCore(rootDir: string): Promise<DiagnoseCheckResult> {
+export async function checkNestedInternalCore(rootDir: string): Promise<DiagnoseCheckResult> {
   const hoistedPath = path.join(rootDir, "node_modules", "@mandujs", "core", "package.json");
   const hoistedVersion = await readPackageVersion(hoistedPath);
 
@@ -828,186 +828,186 @@ export async function checkNestedInternalCore(rootDir: string): Promise<Diagnose
       mismatchCount: mismatches.length,
       mismatches,
     },
-  };
-}
-
-// ────────────────────────────────────────────────────────────────────────
-// 8. client_boundary_manifests (F42/F45)
-// ────────────────────────────────────────────────────────────────────────
-
-interface DiagnoseRouteBoundary {
-  id?: unknown;
-  routeId?: unknown;
-  module?: unknown;
-  exportName?: unknown;
-  source?: unknown;
-}
-
-interface DiagnoseRouteRecord {
-  id?: unknown;
-  boundaries?: unknown;
-}
-
-async function readJsonObject(filePath: string): Promise<Record<string, unknown> | null> {
-  try {
-    const raw = await fs.readFile(filePath, "utf-8");
-    const parsed = JSON.parse(raw) as unknown;
-    return parsed && typeof parsed === "object" && !Array.isArray(parsed)
-      ? parsed as Record<string, unknown>
-      : null;
-  } catch {
-    return null;
-  }
-}
-
-function toRouteRecords(value: unknown): DiagnoseRouteRecord[] {
-  if (!Array.isArray(value)) return [];
-  return value.filter((entry): entry is DiagnoseRouteRecord =>
-    !!entry && typeof entry === "object" && !Array.isArray(entry)
-  );
-}
-
-function toRouteBoundaries(route: DiagnoseRouteRecord): DiagnoseRouteBoundary[] {
-  if (!Array.isArray(route.boundaries)) return [];
-  return route.boundaries.filter((entry): entry is DiagnoseRouteBoundary =>
-    !!entry && typeof entry === "object" && !Array.isArray(entry)
-  );
-}
-
-function bundleAssetExists(rootDir: string, jsPath: unknown): Promise<boolean> {
-  if (typeof jsPath !== "string" || jsPath.length === 0) return Promise.resolve(false);
-  const relativePath = jsPath.startsWith("/")
-    ? jsPath.slice(1)
-    : jsPath;
-  return fs.access(path.join(rootDir, relativePath)).then(
-    () => true,
-    () => false,
-  );
-}
-
-/**
- * F42/F45 check: validate compiler-owned client boundary records.
- *
- * This does not rebuild the app. It inspects generated route and bundle
- * manifests and reports:
- *   - duplicate boundary ids in `.mandu/routes.manifest.json`
- *   - malformed boundary records missing id/module/exportName
- *   - missing boundary bundle entries in `.mandu/manifest.json`
- *   - boundary bundle entries whose JS asset is absent on disk
- */
-export async function checkClientBoundaryManifests(rootDir: string): Promise<DiagnoseCheckResult> {
-  const routesManifestPath = path.join(rootDir, ".mandu", "routes.manifest.json");
-  const routesManifest = await readJsonObject(routesManifestPath);
-
-  if (!routesManifest) {
-    return {
-      ok: true,
-      rule: "client_boundary_manifests",
-      message: "Routes manifest is missing or unreadable — boundary manifest check skipped.",
-      details: { skipped: true, routesManifestPath: path.relative(rootDir, routesManifestPath) },
-    };
-  }
-
-  const boundaries: Array<{ routeId: string; id: string; module: string; exportName: string }> = [];
-  const malformed: Array<{ routeId: string; reason: string }> = [];
-  const seen = new Map<string, string>();
-  const duplicates: Array<{ id: string; firstRouteId: string; duplicateRouteId: string }> = [];
-
-  for (const route of toRouteRecords(routesManifest.routes)) {
-    const routeId = typeof route.id === "string" ? route.id : "(unknown-route)";
-    for (const boundary of toRouteBoundaries(route)) {
-      const id = typeof boundary.id === "string" ? boundary.id : "";
-      const module = typeof boundary.module === "string" ? boundary.module : "";
-      const exportName = typeof boundary.exportName === "string" ? boundary.exportName : "";
-      if (!id || !module || !exportName) {
-        malformed.push({ routeId, reason: "Boundary record must include id, module, and exportName." });
-        continue;
-      }
-
-      const firstRouteId = seen.get(id);
-      if (firstRouteId) {
-        duplicates.push({ id, firstRouteId, duplicateRouteId: routeId });
-      } else {
-        seen.set(id, routeId);
-      }
-      boundaries.push({ routeId, id, module, exportName });
-    }
-  }
-
-  if (malformed.length > 0 || duplicates.length > 0) {
-    return {
-      ok: false,
-      rule: "client_boundary_manifests",
-      severity: "error",
-      message: `Client boundary route manifest has ${malformed.length} malformed record(s) and ${duplicates.length} duplicate id(s).`,
-      suggestion: "Regenerate routes with `mandu generate` or rerun the build; boundary ids must be unique and include id/module/exportName.",
-      details: {
-        boundaryCount: boundaries.length,
-        malformed: malformed.slice(0, 10),
-        duplicates: duplicates.slice(0, 10),
-      },
-    };
-  }
-
-  if (boundaries.length === 0) {
-    return {
-      ok: true,
-      rule: "client_boundary_manifests",
-      message: "No compiler-owned client boundaries recorded in the routes manifest.",
-      details: { boundaryCount: 0 },
-    };
-  }
-
-  const bundleManifestPath = path.join(rootDir, ".mandu", "manifest.json");
-  const bundleManifest = await readJsonObject(bundleManifestPath);
-  if (!bundleManifest) {
-    return {
-      ok: false,
-      rule: "client_boundary_manifests",
-      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) },
-    };
-  }
-
-  const bundleBoundaries = bundleManifest.boundaries && typeof bundleManifest.boundaries === "object" && !Array.isArray(bundleManifest.boundaries)
-    ? bundleManifest.boundaries as Record<string, unknown>
-    : {};
-  const missingEntries: string[] = [];
-  const missingAssets: string[] = [];
-
-  for (const boundary of boundaries) {
-    const entry = bundleBoundaries[boundary.id];
-    if (!entry || typeof entry !== "object" || Array.isArray(entry)) {
-      missingEntries.push(boundary.id);
-      continue;
-    }
-    const jsPath = (entry as { js?: unknown }).js;
-    if (!(await bundleAssetExists(rootDir, jsPath))) {
-      missingAssets.push(boundary.id);
-    }
-  }
-
-  if (missingEntries.length > 0 || missingAssets.length > 0) {
-    return {
-      ok: false,
-      rule: "client_boundary_manifests",
-      severity: "error",
-      message: `Client boundary bundle manifest is incomplete: ${missingEntries.length} missing entr${missingEntries.length === 1 ? "y" : "ies"}, ${missingAssets.length} missing asset(s).`,
-      suggestion: "Run `mandu clean && mandu build`; if the issue persists, inspect `mandu.route.boundaries` with `includeBundle: true`.",
-      details: {
-        boundaryCount: boundaries.length,
-        missingEntries: missingEntries.slice(0, 10),
-        missingAssets: missingAssets.slice(0, 10),
-      },
-    };
-  }
-
-  return {
-    ok: true,
-    rule: "client_boundary_manifests",
-    message: `Client boundary manifests are consistent for ${boundaries.length} boundary record(s).`,
-    details: { boundaryCount: boundaries.length },
-  };
-}
+  };
+}
+
+// ────────────────────────────────────────────────────────────────────────
+// 8. client_boundary_manifests (F42/F45)
+// ────────────────────────────────────────────────────────────────────────
+
+interface DiagnoseRouteBoundary {
+  id?: unknown;
+  routeId?: unknown;
+  module?: unknown;
+  exportName?: unknown;
+  source?: unknown;
+}
+
+interface DiagnoseRouteRecord {
+  id?: unknown;
+  boundaries?: unknown;
+}
+
+async function readJsonObject(filePath: string): Promise<Record<string, unknown> | null> {
+  try {
+    const raw = await fs.readFile(filePath, "utf-8");
+    const parsed = JSON.parse(raw) as unknown;
+    return parsed && typeof parsed === "object" && !Array.isArray(parsed)
+      ? parsed as Record<string, unknown>
+      : null;
+  } catch {
+    return null;
+  }
+}
+
+function toRouteRecords(value: unknown): DiagnoseRouteRecord[] {
+  if (!Array.isArray(value)) return [];
+  return value.filter((entry): entry is DiagnoseRouteRecord =>
+    !!entry && typeof entry === "object" && !Array.isArray(entry)
+  );
+}
+
+function toRouteBoundaries(route: DiagnoseRouteRecord): DiagnoseRouteBoundary[] {
+  if (!Array.isArray(route.boundaries)) return [];
+  return route.boundaries.filter((entry): entry is DiagnoseRouteBoundary =>
+    !!entry && typeof entry === "object" && !Array.isArray(entry)
+  );
+}
+
+function bundleAssetExists(rootDir: string, jsPath: unknown): Promise<boolean> {
+  if (typeof jsPath !== "string" || jsPath.length === 0) return Promise.resolve(false);
+  const relativePath = jsPath.startsWith("/")
+    ? jsPath.slice(1)
+    : jsPath;
+  return fs.access(path.join(rootDir, relativePath)).then(
+    () => true,
+    () => false,
+  );
+}
+
+/**
+ * F42/F45 check: validate compiler-owned client boundary records.
+ *
+ * This does not rebuild the app. It inspects generated route and bundle
+ * manifests and reports:
+ *   - duplicate boundary ids in `.mandu/routes.manifest.json`
+ *   - malformed boundary records missing id/module/exportName
+ *   - missing boundary bundle entries in `.mandu/manifest.json`
+ *   - boundary bundle entries whose JS asset is absent on disk
+ */
+export async function checkClientBoundaryManifests(rootDir: string): Promise<DiagnoseCheckResult> {
+  const routesManifestPath = path.join(rootDir, ".mandu", "routes.manifest.json");
+  const routesManifest = await readJsonObject(routesManifestPath);
+
+  if (!routesManifest) {
+    return {
+      ok: true,
+      rule: "client_boundary_manifests",
+      message: "Routes manifest is missing or unreadable — boundary manifest check skipped.",
+      details: { skipped: true, routesManifestPath: path.relative(rootDir, routesManifestPath) },
+    };
+  }
+
+  const boundaries: Array<{ routeId: string; id: string; module: string; exportName: string }> = [];
+  const malformed: Array<{ routeId: string; reason: string }> = [];
+  const seen = new Map<string, string>();
+  const duplicates: Array<{ id: string; firstRouteId: string; duplicateRouteId: string }> = [];
+
+  for (const route of toRouteRecords(routesManifest.routes)) {
+    const routeId = typeof route.id === "string" ? route.id : "(unknown-route)";
+    for (const boundary of toRouteBoundaries(route)) {
+      const id = typeof boundary.id === "string" ? boundary.id : "";
+      const module = typeof boundary.module === "string" ? boundary.module : "";
+      const exportName = typeof boundary.exportName === "string" ? boundary.exportName : "";
+      if (!id || !module || !exportName) {
+        malformed.push({ routeId, reason: "Boundary record must include id, module, and exportName." });
+        continue;
+      }
+
+      const firstRouteId = seen.get(id);
+      if (firstRouteId) {
+        duplicates.push({ id, firstRouteId, duplicateRouteId: routeId });
+      } else {
+        seen.set(id, routeId);
+      }
+      boundaries.push({ routeId, id, module, exportName });
+    }
+  }
+
+  if (malformed.length > 0 || duplicates.length > 0) {
+    return {
+      ok: false,
+      rule: "client_boundary_manifests",
+      severity: "error",
+      message: `Client boundary route manifest has ${malformed.length} malformed record(s) and ${duplicates.length} duplicate id(s).`,
+      suggestion: "Regenerate routes with `mandu generate` or rerun the build; boundary ids must be unique and include id/module/exportName.",
+      details: {
+        boundaryCount: boundaries.length,
+        malformed: malformed.slice(0, 10),
+        duplicates: duplicates.slice(0, 10),
+      },
+    };
+  }
+
+  if (boundaries.length === 0) {
+    return {
+      ok: true,
+      rule: "client_boundary_manifests",
+      message: "No compiler-owned client boundaries recorded in the routes manifest.",
+      details: { boundaryCount: 0 },
+    };
+  }
+
+  const bundleManifestPath = path.join(rootDir, ".mandu", "manifest.json");
+  const bundleManifest = await readJsonObject(bundleManifestPath);
+  if (!bundleManifest) {
+    return {
+      ok: false,
+      rule: "client_boundary_manifests",
+      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) },
+    };
+  }
+
+  const bundleBoundaries = bundleManifest.boundaries && typeof bundleManifest.boundaries === "object" && !Array.isArray(bundleManifest.boundaries)
+    ? bundleManifest.boundaries as Record<string, unknown>
+    : {};
+  const missingEntries: string[] = [];
+  const missingAssets: string[] = [];
+
+  for (const boundary of boundaries) {
+    const entry = bundleBoundaries[boundary.id];
+    if (!entry || typeof entry !== "object" || Array.isArray(entry)) {
+      missingEntries.push(boundary.id);
+      continue;
+    }
+    const jsPath = (entry as { js?: unknown }).js;
+    if (!(await bundleAssetExists(rootDir, jsPath))) {
+      missingAssets.push(boundary.id);
+    }
+  }
+
+  if (missingEntries.length > 0 || missingAssets.length > 0) {
+    return {
+      ok: false,
+      rule: "client_boundary_manifests",
+      severity: "error",
+      message: `Client boundary bundle manifest is incomplete: ${missingEntries.length} missing entr${missingEntries.length === 1 ? "y" : "ies"}, ${missingAssets.length} missing asset(s).`,
+      suggestion: "Run `mandu clean && mandu build`; if the issue persists, inspect `mandu.route.boundaries` with `includeBundle: true`.",
+      details: {
+        boundaryCount: boundaries.length,
+        missingEntries: missingEntries.slice(0, 10),
+        missingAssets: missingAssets.slice(0, 10),
+      },
+    };
+  }
+
+  return {
+    ok: true,
+    rule: "client_boundary_manifests",
+    message: `Client boundary manifests are consistent for ${boundaries.length} boundary record(s).`,
+    details: { boundaryCount: boundaries.length },
+  };
+}

package/src/diagnose/index.ts

@@ -1,17 +1,17 @@
-/**
- * @mandujs/core/diagnose — structured diagnostic checks.
- *
- * Public surface for Issue #215 diagnose-check expansion.
- */
-
-export type { DiagnoseSeverity, DiagnoseCheckResult, DiagnoseReport } from "./types";
-export {
-  checkManifestFreshness,
-  checkPrerenderPollution,
-  checkCloneElementWarnings,
-  checkDevArtifactsInProd,
-  checkPackageExportGaps,
-  checkNestedInternalCore,
-  checkA11yHints,
-} from "./checks";
-export { EXTENDED_CHECKS, runExtendedDiagnose, buildReport } from "./run";
+/**
+ * @mandujs/core/diagnose — structured diagnostic checks.
+ *
+ * Public surface for Issue #215 diagnose-check expansion.
+ */
+
+export type { DiagnoseSeverity, DiagnoseCheckResult, DiagnoseReport } from "./types";
+export {
+  checkManifestFreshness,
+  checkPrerenderPollution,
+  checkCloneElementWarnings,
+  checkDevArtifactsInProd,
+  checkPackageExportGaps,
+  checkNestedInternalCore,
+  checkA11yHints,
+} from "./checks";
+export { EXTENDED_CHECKS, runExtendedDiagnose, buildReport } from "./run";

package/src/diagnose/run.ts

@@ -11,11 +11,11 @@ import {
   checkPrerenderPollution,
   checkCloneElementWarnings,
   checkDevArtifactsInProd,
-  checkPackageExportGaps,
-  checkNestedInternalCore,
-  checkA11yHints,
-  checkClientBoundaryManifests,
-} from "./checks";
+  checkPackageExportGaps,
+  checkNestedInternalCore,
+  checkA11yHints,
+  checkClientBoundaryManifests,
+} from "./checks";
 
 /**
  * Registered extended checks (Issue #215 + Phase 18.χ).
@@ -31,11 +31,11 @@ export const EXTENDED_CHECKS = [
   { name: "prerender_pollution", run: checkPrerenderPollution },
   { name: "cloneelement_warnings", run: checkCloneElementWarnings },
   { name: "dev_artifacts_in_prod", run: checkDevArtifactsInProd },
-  { name: "package_export_gaps", run: checkPackageExportGaps },
-  { name: "nested_internal_core", run: checkNestedInternalCore },
-  { name: "client_boundary_manifests", run: checkClientBoundaryManifests },
-  { name: "a11y_hints", run: checkA11yHints },
-] as const;
+  { name: "package_export_gaps", run: checkPackageExportGaps },
+  { name: "nested_internal_core", run: checkNestedInternalCore },
+  { name: "client_boundary_manifests", run: checkClientBoundaryManifests },
+  { name: "a11y_hints", run: checkA11yHints },
+] as const;
 
 /**
  * Run every extended check in parallel and return the aggregate report.

package/src/diagnose/types.ts

@@ -1,53 +1,53 @@
-/**
- * Mandu Diagnose — unified check result shape.
- *
- * Every diagnose check returns a `DiagnoseCheckResult`. Severity semantics:
- *
- *   - `error`   : blocks deploy. CI should fail. Example: stale dev-mode
- *                 manifest shipped to prod, package export gap, empty bundles
- *                 with islands declared.
- *   - `warning` : degraded UX or future risk. Does NOT block deploy, but
- *                 surfaces for operator attention. Example: suspicious
- *                 prerendered routes, many cloneElement warnings.
- *   - `info`    : neutral observation, no action required.
- *
- * The `rule` field is a stable machine-readable identifier. The `message`
- * field is human-readable and MAY include route paths or counts. The
- * optional `suggestion` field is a single actionable next step.
- */
-export type DiagnoseSeverity = "error" | "warning" | "info";
-
-export interface DiagnoseCheckResult {
-  /** Overall pass/fail for this check. `true` = healthy. */
-  ok: boolean;
-  /** Stable machine-readable rule id, e.g. `manifest_freshness`. */
-  rule: string;
-  /** Severity when `ok === false`. Omitted when `ok === true`. */
-  severity?: DiagnoseSeverity;
-  /** Human-readable summary. */
-  message: string;
-  /** Single-line actionable next step, e.g. `"Run mandu build"`. */
-  suggestion?: string;
-  /** Structured details (route list, counts, file paths). */
-  details?: Record<string, unknown>;
-}
-
-/**
- * Aggregated report across all checks for a single diagnose run.
- */
-export interface DiagnoseReport {
-  /** `true` when no check has `ok: false` with `severity: 'error'`. */
-  healthy: boolean;
-  /** Count of checks returning `ok: false` with `severity: 'error'`. */
-  errorCount: number;
-  /** Count of checks returning `ok: false` with `severity: 'warning'`. */
-  warningCount: number;
-  /** Individual check results in registration order. */
-  checks: DiagnoseCheckResult[];
-  /** Summary statistics. */
-  summary: {
-    total: number;
-    passed: number;
-    failed: number;
-  };
-}
+/**
+ * Mandu Diagnose — unified check result shape.
+ *
+ * Every diagnose check returns a `DiagnoseCheckResult`. Severity semantics:
+ *
+ *   - `error`   : blocks deploy. CI should fail. Example: stale dev-mode
+ *                 manifest shipped to prod, package export gap, empty bundles
+ *                 with islands declared.
+ *   - `warning` : degraded UX or future risk. Does NOT block deploy, but
+ *                 surfaces for operator attention. Example: suspicious
+ *                 prerendered routes, many cloneElement warnings.
+ *   - `info`    : neutral observation, no action required.
+ *
+ * The `rule` field is a stable machine-readable identifier. The `message`
+ * field is human-readable and MAY include route paths or counts. The
+ * optional `suggestion` field is a single actionable next step.
+ */
+export type DiagnoseSeverity = "error" | "warning" | "info";
+
+export interface DiagnoseCheckResult {
+  /** Overall pass/fail for this check. `true` = healthy. */
+  ok: boolean;
+  /** Stable machine-readable rule id, e.g. `manifest_freshness`. */
+  rule: string;
+  /** Severity when `ok === false`. Omitted when `ok === true`. */
+  severity?: DiagnoseSeverity;
+  /** Human-readable summary. */
+  message: string;
+  /** Single-line actionable next step, e.g. `"Run mandu build"`. */
+  suggestion?: string;
+  /** Structured details (route list, counts, file paths). */
+  details?: Record<string, unknown>;
+}
+
+/**
+ * Aggregated report across all checks for a single diagnose run.
+ */
+export interface DiagnoseReport {
+  /** `true` when no check has `ok: false` with `severity: 'error'`. */
+  healthy: boolean;
+  /** Count of checks returning `ok: false` with `severity: 'error'`. */
+  errorCount: number;
+  /** Count of checks returning `ok: false` with `severity: 'warning'`. */
+  warningCount: number;
+  /** Individual check results in registration order. */
+  checks: DiagnoseCheckResult[];
+  /** Summary statistics. */
+  summary: {
+    total: number;
+    passed: number;
+    failed: number;
+  };
+}