@mandujs/core 0.53.3 → 0.54.0

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

@@ -1,93 +1,93 @@
-/**
- * Mandu Diagnose — aggregator.
- *
- * Runs all registered checks in parallel (they are pure I/O, no shared
- * mutable state) and builds a unified `DiagnoseReport`.
- */
-
-import type { DiagnoseCheckResult, DiagnoseReport } from "./types";
-import {
-  checkManifestFreshness,
-  checkPrerenderPollution,
-  checkCloneElementWarnings,
-  checkDevArtifactsInProd,
-  checkPackageExportGaps,
-  checkNestedInternalCore,
-  checkA11yHints,
-} from "./checks";
-
-/**
- * Registered extended checks (Issue #215 + Phase 18.χ).
- *
- * Order matters for display purposes only — result aggregation is
- * order-independent. The a11y smoke is last because it's the most
- * expensive check by an order of magnitude (dynamic import + DOM
- * parse + axe rules) and we prefer failing fast on cheaper structural
- * checks first.
- */
-export const EXTENDED_CHECKS = [
-  { name: "manifest_freshness", run: checkManifestFreshness },
-  { 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: "a11y_hints", run: checkA11yHints },
-] as const;
-
-/**
- * Run every extended check in parallel and return the aggregate report.
- *
- * Legacy checks (kitchen_errors, guard_check, contract_validation,
- * manifest_validation) are NOT run here — they live in the MCP tool
- * surface and have different dependency requirements. Instead, the MCP
- * `mandu.diagnose` composite combines the extended checks with the
- * legacy ones and normalizes both into a single unified report.
- */
-export async function runExtendedDiagnose(rootDir: string): Promise<DiagnoseReport> {
-  const checks = await Promise.all(
-    EXTENDED_CHECKS.map(async ({ run }) => {
-      try {
-        return await run(rootDir);
-      } catch (err) {
-        // Defensive fallback — individual checks shouldn't throw, but if
-        // one does, we surface it as an error rather than crashing the
-        // whole diagnose run.
-        return {
-          ok: false,
-          rule: "diagnose_internal_error",
-          severity: "error" as const,
-          message: `Check threw an error: ${err instanceof Error ? err.message : String(err)}`,
-        };
-      }
-    })
-  );
-
-  return buildReport(checks);
-}
-
-/**
- * Aggregate a pre-computed list of check results into a `DiagnoseReport`.
- * Exported so the MCP composite can pass in both extended + legacy checks
- * at once.
- */
-export function buildReport(checks: DiagnoseCheckResult[]): DiagnoseReport {
-  let errorCount = 0;
-  let warningCount = 0;
-  for (const c of checks) {
-    if (c.ok) continue;
-    if (c.severity === "error") errorCount += 1;
-    else if (c.severity === "warning") warningCount += 1;
-  }
-  return {
-    healthy: errorCount === 0,
-    errorCount,
-    warningCount,
-    checks,
-    summary: {
-      total: checks.length,
-      passed: checks.filter((c) => c.ok).length,
-      failed: checks.filter((c) => !c.ok).length,
-    },
-  };
-}
+/**
+ * Mandu Diagnose — aggregator.
+ *
+ * Runs all registered checks in parallel (they are pure I/O, no shared
+ * mutable state) and builds a unified `DiagnoseReport`.
+ */
+
+import type { DiagnoseCheckResult, DiagnoseReport } from "./types";
+import {
+  checkManifestFreshness,
+  checkPrerenderPollution,
+  checkCloneElementWarnings,
+  checkDevArtifactsInProd,
+  checkPackageExportGaps,
+  checkNestedInternalCore,
+  checkA11yHints,
+} from "./checks";
+
+/**
+ * Registered extended checks (Issue #215 + Phase 18.χ).
+ *
+ * Order matters for display purposes only — result aggregation is
+ * order-independent. The a11y smoke is last because it's the most
+ * expensive check by an order of magnitude (dynamic import + DOM
+ * parse + axe rules) and we prefer failing fast on cheaper structural
+ * checks first.
+ */
+export const EXTENDED_CHECKS = [
+  { name: "manifest_freshness", run: checkManifestFreshness },
+  { 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: "a11y_hints", run: checkA11yHints },
+] as const;
+
+/**
+ * Run every extended check in parallel and return the aggregate report.
+ *
+ * Legacy checks (kitchen_errors, guard_check, contract_validation,
+ * manifest_validation) are NOT run here — they live in the MCP tool
+ * surface and have different dependency requirements. Instead, the MCP
+ * `mandu.diagnose` composite combines the extended checks with the
+ * legacy ones and normalizes both into a single unified report.
+ */
+export async function runExtendedDiagnose(rootDir: string): Promise<DiagnoseReport> {
+  const checks = await Promise.all(
+    EXTENDED_CHECKS.map(async ({ run }) => {
+      try {
+        return await run(rootDir);
+      } catch (err) {
+        // Defensive fallback — individual checks shouldn't throw, but if
+        // one does, we surface it as an error rather than crashing the
+        // whole diagnose run.
+        return {
+          ok: false,
+          rule: "diagnose_internal_error",
+          severity: "error" as const,
+          message: `Check threw an error: ${err instanceof Error ? err.message : String(err)}`,
+        };
+      }
+    })
+  );
+
+  return buildReport(checks);
+}
+
+/**
+ * Aggregate a pre-computed list of check results into a `DiagnoseReport`.
+ * Exported so the MCP composite can pass in both extended + legacy checks
+ * at once.
+ */
+export function buildReport(checks: DiagnoseCheckResult[]): DiagnoseReport {
+  let errorCount = 0;
+  let warningCount = 0;
+  for (const c of checks) {
+    if (c.ok) continue;
+    if (c.severity === "error") errorCount += 1;
+    else if (c.severity === "warning") warningCount += 1;
+  }
+  return {
+    healthy: errorCount === 0,
+    errorCount,
+    warningCount,
+    checks,
+    summary: {
+      total: checks.length,
+      passed: checks.filter((c) => c.ok).length,
+      failed: checks.filter((c) => !c.ok).length,
+    },
+  };
+}

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;
+  };
+}