@ethisyscore/vite-plugin 1.121.0 → 1.123.0

package/dist/index.cjs

@@ -456,7 +456,7 @@ function ethisysContractBPlugin(options = {}) {
       const externalSpecifiers = new Set(CONTRACT_B_IMPORT_MAP_ALLOWLIST);
       return {
         build: {
-          rollupOptions: {
+          rolldownOptions: {
             input: { worker: slash(workerEntryAbs) },
             external: (id) => externalSpecifiers.has(id),
             // CRITICAL: single-file output for the worker bundle. The
@@ -464,7 +464,7 @@ function ethisysContractBPlugin(options = {}) {
             // chunk splitting or dynamic-import shards would defeat it.
             output: {
               format: "es",
-              inlineDynamicImports: true,
+              codeSplitting: false,
               entryFileNames: `${outputPrefix}[name].js`,
               chunkFileNames: `${outputPrefix}[name]-[hash].js`,
               assetFileNames: `${outputPrefix}[name][extname]`
@@ -598,6 +598,48 @@ function ethisysContractBPlugin(options = {}) {
   };
 }
 
+// src/utils/inline-css-sourcemap-log.ts
+var PRE_CSS_POST_PLUGIN = /^(?:@tailwindcss\/vite:|vite:css$)/;
+function isInlineCssSourcemapNoise(log) {
+  return log.code === "SOURCEMAP_BROKEN" && PRE_CSS_POST_PLUGIN.test(log.plugin ?? "") && /\.css\?(?:[^#]*&)?inline(?:[&=#]|$)/.test(log.id ?? "");
+}
+function dropInlineCssSourcemapNoise(next) {
+  const filtered = (level, log, handler) => {
+    if (isInlineCssSourcemapNoise(log)) {
+      return;
+    }
+    if (next) {
+      next(level, log, handler);
+      return;
+    }
+    handler(level, log);
+  };
+  return Object.assign(filtered, { [WRAPPED]: true });
+}
+var WRAPPED = /* @__PURE__ */ Symbol.for("ethisys.vite-plugin.inline-css-sourcemap-noise");
+function wrapResolvedOnLog(config) {
+  const writable = config;
+  writable.build ??= {};
+  writable.build.rolldownOptions ??= {};
+  const current = writable.build.rolldownOptions.onLog;
+  if (current && current[WRAPPED]) {
+    return;
+  }
+  writable.build.rolldownOptions.onLog = dropInlineCssSourcemapNoise(current);
+}
+function inlineCssSourcemapNoisePlugin() {
+  return {
+    name: "ethisys-inline-css-sourcemap-noise",
+    apply: "build",
+    buildStart() {
+      const config = this?.environment?.config;
+      if (config) {
+        wrapResolvedOnLog(config);
+      }
+    }
+  };
+}
+
 // src/utils/strings.ts
 function slash2(p) {
   return p.replace(/\\/g, "/");
@@ -938,6 +980,36 @@ var PLATFORM_REACT_EXTERNALS = [
   "react-dom",
   "@mui/material",
   "@mui/icons-material",
+  // MUI X. Every plugin previously shipped its own copy on top of the host's.
+  //
+  // EXACT STRINGS, and that is load-bearing rather than stylistic. Rolldown matches
+  // a string `external` by equality, verified by building an entry that imports both
+  // `date-fns` and `date-fns/addDays` against `external: ["date-fns"]`: the bare
+  // import stayed external, the subpath was bundled. So a prefix or regex matcher
+  // here would externalise `@mui/x-data-grid/style.css` and
+  // `@mui/x-date-pickers/AdapterDayjs` too, which the host does NOT register, and an
+  // external the host does not serve is an unresolvable bare import that kills the
+  // bundle before any plugin code runs. `startsWith("@mui/x-charts")` additionally
+  // swallows `@mui/x-charts-pro`. Add specifiers one at a time; never widen the match.
+  //
+  // THE CONSUMER-SIDE GATE: exact matching makes this safe but PARTIAL. A plugin that
+  // imports the package root AND a subpath of the same package gets TWO module
+  // instances, host-served and bundled, and React context does not cross that seam.
+  // The concrete break is `LocalizationProvider` from "@mui/x-date-pickers" (host)
+  // with `DatePicker` from "@mui/x-date-pickers/DatePicker" (bundled): the picker
+  // cannot see the provider's context and MUI throws "Can not find the date and time
+  // pickers localization context". Every root-exported symbol must therefore be
+  // imported from the ROOT once its package is externalised.
+  //
+  // `@mui/x-date-pickers/AdapterDayjs` is the deliberate exception and stays a
+  // sub-path import, because the root does not re-export it. That is safe here and it
+  // was checked rather than assumed: `LocalizationProvider` validates the adapter with
+  // `if (!dateAdapter.isMUIAdapter)`, a duck-typed flag, so a bundled adapter instance
+  // handed to the host's provider passes. Contrast `instanceof`, which would not.
+  "@mui/x-data-grid",
+  "@mui/x-charts",
+  "@mui/x-charts-pro",
+  "@mui/x-date-pickers",
   "@emotion/react",
   "@emotion/styled",
   "date-fns",
@@ -1029,6 +1101,34 @@ function rewriteAliasedExternalImports() {
     }
   };
 }
+function assertHostResolvableExternals(external) {
+  return {
+    name: "assert-host-resolvable-externals",
+    renderChunk(code, chunk) {
+      const offences = [];
+      for (const spec of external) {
+        const q = escapeRegExp(spec);
+        const reExport = new RegExp(`\\bexport\\s[^;]{0,200}?from\\s*["']${q}["']`);
+        const dynamic = new RegExp(`\\bimport\\s*\\(\\s*["']${q}["']\\s*\\)`);
+        if (reExport.test(code)) {
+          offences.push(`re-export of "${spec}" (export \u2026 from)`);
+        }
+        if (dynamic.test(code)) {
+          offences.push(`dynamic import of "${spec}"`);
+        }
+      }
+      if (offences.length > 0) {
+        throw new Error(
+          `[build-pages] ${chunk.fileName ?? "bundle"} emits host-provided specifiers in a form the host loader cannot rewrite, so the surface would fail to load at mount:
+` + offences.map((o) => `  - ${o}`).join("\n") + `
+
+Use a STATIC top-level import of the package root instead. A dynamic import of a host-provided package is never needed: the host already loads its namespaces on demand, per bundle. For a re-export, import the symbols and export them explicitly.`
+        );
+      }
+      return null;
+    }
+  };
+}
 function virtualPageEntry(page, config) {
   const VIRTUAL_ID = `\0virtual:surface/${page.id}`;
   const views = page.views ?? {};
@@ -1117,7 +1217,10 @@ function makeDefaultBuild(viteBuild) {
         ctx.surfacePlugin,
         ...ctx.plugins,
         scopePluginCssPlugin(),
-        rewriteAliasedExternalImports()
+        rewriteAliasedExternalImports(),
+        // Last, so it sees exactly the bytes the host will fetch.
+        assertHostResolvableExternals(ctx.external),
+        inlineCssSourcemapNoisePlugin()
       ],
       resolve: {
         dedupe: [...ctx.dedupe],
@@ -1145,13 +1248,13 @@ function makeDefaultBuild(viteBuild) {
         cssMinify: ctx.cssMinify ?? true,
         reportCompressedSize: ctx.reportCompressedSize ?? false,
         sourcemap: ctx.sourcemap,
-        rollupOptions: {
+        rolldownOptions: {
           input: { [ctx.pageId]: ctx.entryId },
           preserveEntrySignatures: "strict",
           external: [...ctx.external],
           output: {
             format: "es",
-            inlineDynamicImports: true,
+            codeSplitting: false,
             sourcemapExcludeSources: ctx.sourcemapExcludeSources ?? true,
             entryFileNames: `${ctx.outputPrefix}/[name].js`,
             chunkFileNames: `${ctx.outputPrefix}/[name]-[hash].js`,
@@ -1348,7 +1451,11 @@ function assertHostOriginRelativePath2(value, label) {
   }
 }
 function ethisysPlatformReactPlugin(options = {}) {
-  return [ethisysPlatformReactPagesPlugin(options), scopePluginCssPlugin()];
+  return [
+    ethisysPlatformReactPagesPlugin(options),
+    scopePluginCssPlugin(),
+    assertHostResolvableExternals([...PLATFORM_REACT_EXTERNALS, "@ethisyscore/host-ui-externals"])
+  ];
 }
 function ethisysPlatformReactPagesPlugin(options = {}) {
   const explicitRoot = options.root;
@@ -1498,7 +1605,7 @@ function ethisysPlatformReactPagesPlugin(options = {}) {
           // throws on the first hook). Vite minifies production builds by
           // default, so this MUST be forced off for every PlatformReact surface.
           minify: false,
-          rollupOptions: {
+          rolldownOptions: {
             // CRITICAL: single-file ESM output per page. The host loads each
             // page via `import(url)` at mount time and reads the declared
             // `exportName` from the resulting module namespace. Chunk-split
@@ -1506,7 +1613,7 @@ function ethisysPlatformReactPagesPlugin(options = {}) {
             // import-map surface to resolve sibling chunks.
             output: {
               format: "es",
-              inlineDynamicImports: true,
+              codeSplitting: false,
               entryFileNames: `${outputPrefix}[name].js`,
               chunkFileNames: `${outputPrefix}[name]-[hash].js`,
               assetFileNames: `${outputPrefix}[name][extname]`
@@ -1559,25 +1666,29 @@ function ethisysPlatformReactPagesPlugin(options = {}) {
       }
       const writable = config;
       writable.build ??= {};
-      writable.build.rollupOptions ??= {};
-      const rollupOptions = writable.build.rollupOptions;
-      const existing = rollupOptions.input;
+      writable.build.rolldownOptions ??= {};
+      const rolldownOptions = writable.build.rolldownOptions;
+      const existing = rolldownOptions.input;
       if (existing && typeof existing === "object" && !Array.isArray(existing)) {
-        rollupOptions.input = { ...existing, ...input };
+        rolldownOptions.input = { ...existing, ...input };
       } else if (typeof existing === "string") {
-        rollupOptions.input = { _entry: existing, ...input };
+        rolldownOptions.input = { _entry: existing, ...input };
       } else if (Array.isArray(existing)) {
         const indexed = {};
         existing.forEach((entry, idx) => {
           indexed[`_entry${idx}`] = entry;
         });
-        rollupOptions.input = { ...indexed, ...input };
+        rolldownOptions.input = { ...indexed, ...input };
       } else {
-        rollupOptions.input = input;
+        rolldownOptions.input = input;
       }
     },
     buildStart() {
       validate();
+      const config = this?.environment?.config;
+      if (isBuild && config) {
+        wrapResolvedOnLog(config);
+      }
     },
     /**
      * Emit the companion `platform-react-pages.json` listing every declared
@@ -1956,7 +2067,7 @@ var defaultBuild = async (page) => {
     build: {
       outDir: page.outDir,
       emptyOutDir: true,
-      rollupOptions: {
+      rolldownOptions: {
         input: path.resolve(page.root, page.entryPath)
       }
     }
@@ -2176,7 +2287,7 @@ function ethisysManifestPlugin(options = {}) {
         }
         return {
           build: {
-            rollupOptions: {
+            rolldownOptions: {
               input
             }
           }
@@ -2294,6 +2405,7 @@ exports.EHX_PLUGIN_SCOPE_ROOTS = EHX_PLUGIN_SCOPE_ROOTS;
 exports.EHX_SCOPED_CSS_LAYERS = EHX_SCOPED_CSS_LAYERS;
 exports.PLATFORM_REACT_DEDUPE = PLATFORM_REACT_DEDUPE;
 exports.PLATFORM_REACT_EXTERNALS = PLATFORM_REACT_EXTERNALS;
+exports.assertHostResolvableExternals = assertHostResolvableExternals;
 exports.buildIframeSandboxPages = buildIframeSandboxPages;
 exports.buildPlatformReactPages = buildPlatformReactPages;
 exports.computePlatformReactPages = computePlatformReactPages;