@ethisyscore/vite-plugin 1.104.0 → 1.105.0

package/dist/index.d.cts

@@ -1,4 +1,4 @@
-import { Plugin, InlineConfig } from 'vite';
+import { Plugin, ESBuildOptions, InlineConfig } from 'vite';
 
 /**
  * Reference from the plugin manifest to an SDUI resource JSON file.
@@ -440,8 +440,18 @@ interface PlatformReactBuildContext {
     alias: PlatformReactAlias[];
     plugins: Plugin[];
     target: string;
-    minify: boolean;
-    sourcemap: boolean;
+    minify: boolean | "esbuild" | "terser";
+    sourcemap: boolean | "inline" | "hidden";
+    /**
+     * Optional so an existing consumer constructing this context keeps compiling.
+     * This type is exported through the `PlatformReactViteBuild` seam, so a
+     * required field here is a breaking change for anyone who builds one by hand.
+     * The orchestrator always sets them; the defaults below only cover that case.
+     */
+    sourcemapExcludeSources?: boolean;
+    esbuild?: ESBuildOptions | false;
+    cssMinify?: boolean | "esbuild" | "lightningcss";
+    reportCompressedSize?: boolean;
 }
 /** Low-level Vite `build` function. Defaults to a dynamic `import("vite")`. */
 type ViteBuildFn = (config: InlineConfig) => Promise<unknown>;
@@ -474,12 +484,59 @@ interface BuildPlatformReactPagesOptions {
     srcDir?: string;
     /** Alias prefix manifest `src/…` paths are rewritten to. Default `@` (→ `@/…`). */
     aliasPrefix?: string;
-    /** Rollup build target. Default `es2022`. */
+    /**
+     * Rollup build target. Default `esnext`.
+     *
+     * Only worth it alongside minification. Measured across 50 finance surfaces,
+     * `esnext` on unminified output cost 10% MORE gzipped than `es2022`, because
+     * it skips the esbuild pass that also normalises emitted code. Lower the
+     * target too if minification is ever turned off.
+     */
     target?: string;
-    /** Minify output. Default `false` (digest stability + debuggability). */
-    minify?: boolean;
-    /** Emit sourcemaps. Default `true`. */
-    sourcemap?: boolean;
+    /**
+     * Minify output. Default `true`, worth roughly 31% of gzipped transfer.
+     *
+     * DEPLOYMENT ORDER: a minified bundle only mounts on a host whose module
+     * registry walks the import prologue. The earlier rewriter was line-anchored
+     * and matched an import only when it sat on its own line; a minifier puts the
+     * whole prologue on line one, where nothing matched, so `react` and
+     * `@mui/material` reached the browser as unresolvable bare specifiers and the
+     * surface threw at mount. A plugin built with this default must therefore not
+     * be promoted ahead of that host change.
+     */
+    minify?: boolean | "esbuild" | "terser";
+    /**
+     * Sourcemap mode. Default `"hidden"`: the map is emitted but no
+     * `sourceMappingURL` comment is written into the bundle, so a browser never
+     * fetches it and cannot 404 on a map stripped before deploy.
+     */
+    sourcemap?: boolean | "inline" | "hidden";
+    /**
+     * Omit `sourcesContent` from emitted maps. Default `true`. The map keeps the
+     * `mappings` needed to rewrite a stack trace, but carries no source bodies —
+     * which both shrinks it by roughly 80% and removes the source-in-map leak path.
+     */
+    sourcemapExcludeSources?: boolean;
+    /**
+     * esbuild options applied while minifying. The default drops `console` and
+     * `debugger` calls, so diagnostic logging a developer left in a page never
+     * reaches a packaged bundle. Pass `false` to keep them — a plugin that
+     * reports failures through `console.error` must move to the runtime logger
+     * before relying on the default.
+     *
+     * `false` means "apply no esbuild options". It does NOT disable esbuild: Vite
+     * would read that as removing the transform plugin, taking TS and JSX handling
+     * with it.
+     */
+    esbuild?: ESBuildOptions | false;
+    /** Minify CSS before it is inlined into the bundle. Default `"esbuild"`. */
+    cssMinify?: boolean | "esbuild" | "lightningcss";
+    /**
+     * Print the post-build gzip-size table. Default `false`: the numbers are
+     * informational, and producing them compresses every chunk a second time —
+     * once per sub-build, so the cost scales with surface count.
+     */
+    reportCompressedSize?: boolean;
     /** Wipe `outDir` before building. Default `true`. */
     clean?: boolean;
     /** Progress logger. Default `console`. */

package/dist/index.d.ts

@@ -1,4 +1,4 @@
-import { Plugin, InlineConfig } from 'vite';
+import { Plugin, ESBuildOptions, InlineConfig } from 'vite';
 
 /**
  * Reference from the plugin manifest to an SDUI resource JSON file.
@@ -440,8 +440,18 @@ interface PlatformReactBuildContext {
     alias: PlatformReactAlias[];
     plugins: Plugin[];
     target: string;
-    minify: boolean;
-    sourcemap: boolean;
+    minify: boolean | "esbuild" | "terser";
+    sourcemap: boolean | "inline" | "hidden";
+    /**
+     * Optional so an existing consumer constructing this context keeps compiling.
+     * This type is exported through the `PlatformReactViteBuild` seam, so a
+     * required field here is a breaking change for anyone who builds one by hand.
+     * The orchestrator always sets them; the defaults below only cover that case.
+     */
+    sourcemapExcludeSources?: boolean;
+    esbuild?: ESBuildOptions | false;
+    cssMinify?: boolean | "esbuild" | "lightningcss";
+    reportCompressedSize?: boolean;
 }
 /** Low-level Vite `build` function. Defaults to a dynamic `import("vite")`. */
 type ViteBuildFn = (config: InlineConfig) => Promise<unknown>;
@@ -474,12 +484,59 @@ interface BuildPlatformReactPagesOptions {
     srcDir?: string;
     /** Alias prefix manifest `src/…` paths are rewritten to. Default `@` (→ `@/…`). */
     aliasPrefix?: string;
-    /** Rollup build target. Default `es2022`. */
+    /**
+     * Rollup build target. Default `esnext`.
+     *
+     * Only worth it alongside minification. Measured across 50 finance surfaces,
+     * `esnext` on unminified output cost 10% MORE gzipped than `es2022`, because
+     * it skips the esbuild pass that also normalises emitted code. Lower the
+     * target too if minification is ever turned off.
+     */
     target?: string;
-    /** Minify output. Default `false` (digest stability + debuggability). */
-    minify?: boolean;
-    /** Emit sourcemaps. Default `true`. */
-    sourcemap?: boolean;
+    /**
+     * Minify output. Default `true`, worth roughly 31% of gzipped transfer.
+     *
+     * DEPLOYMENT ORDER: a minified bundle only mounts on a host whose module
+     * registry walks the import prologue. The earlier rewriter was line-anchored
+     * and matched an import only when it sat on its own line; a minifier puts the
+     * whole prologue on line one, where nothing matched, so `react` and
+     * `@mui/material` reached the browser as unresolvable bare specifiers and the
+     * surface threw at mount. A plugin built with this default must therefore not
+     * be promoted ahead of that host change.
+     */
+    minify?: boolean | "esbuild" | "terser";
+    /**
+     * Sourcemap mode. Default `"hidden"`: the map is emitted but no
+     * `sourceMappingURL` comment is written into the bundle, so a browser never
+     * fetches it and cannot 404 on a map stripped before deploy.
+     */
+    sourcemap?: boolean | "inline" | "hidden";
+    /**
+     * Omit `sourcesContent` from emitted maps. Default `true`. The map keeps the
+     * `mappings` needed to rewrite a stack trace, but carries no source bodies —
+     * which both shrinks it by roughly 80% and removes the source-in-map leak path.
+     */
+    sourcemapExcludeSources?: boolean;
+    /**
+     * esbuild options applied while minifying. The default drops `console` and
+     * `debugger` calls, so diagnostic logging a developer left in a page never
+     * reaches a packaged bundle. Pass `false` to keep them — a plugin that
+     * reports failures through `console.error` must move to the runtime logger
+     * before relying on the default.
+     *
+     * `false` means "apply no esbuild options". It does NOT disable esbuild: Vite
+     * would read that as removing the transform plugin, taking TS and JSX handling
+     * with it.
+     */
+    esbuild?: ESBuildOptions | false;
+    /** Minify CSS before it is inlined into the bundle. Default `"esbuild"`. */
+    cssMinify?: boolean | "esbuild" | "lightningcss";
+    /**
+     * Print the post-build gzip-size table. Default `false`: the numbers are
+     * informational, and producing them compresses every chunk a second time —
+     * once per sub-build, so the cost scales with surface count.
+     */
+    reportCompressedSize?: boolean;
     /** Wipe `outDir` before building. Default `true`. */
     clean?: boolean;
     /** Progress logger. Default `console`. */

package/dist/index.js

@@ -953,6 +953,19 @@ var DEFAULT_SRC_DIR = "src";
 var DEFAULT_ALIAS_PREFIX = "@";
 var SUMMARY_FILENAME = "platform-react-pages.json";
 var DIGEST_MAP_FILENAME = "platform-react-digest-map.json";
+var DEFAULT_ESBUILD = {
+  // `pure`, not `drop: ["console"]`. A blanket drop removes `console.error` and
+  // `console.warn` too, and the SDK reports real failures through them from code
+  // bundled into every surface: a production URL-resolution failure in
+  // `plugin-ui/src/platform-react/surfaceUrl.ts`, and host-version skew in
+  // `useOrgFormatSync` and `useReportingCurrencySync`. Those would have failed
+  // silently in packaged bundles. Marking the noisy methods side-effect-free
+  // lets tree shaking remove them while diagnostics survive.
+  pure: ["console.log", "console.debug", "console.trace"],
+  drop: ["debugger"],
+  legalComments: "eof",
+  treeShaking: true
+};
 function rewriteAliasedExternalImports() {
   return {
     name: "rewrite-aliased-external-imports",
@@ -1064,12 +1077,23 @@ function makeDefaultBuild(viteBuild) {
         dedupe: [...ctx.dedupe],
         alias: ctx.alias
       },
+      // Top-level, not under `build` — Vite feeds this to esbuild during
+      // minification, which is where `drop` removes console/debugger calls.
+      //
+      // `false` is never forwarded. Vite reads `esbuild: false` as "drop the
+      // esbuild transform plugin entirely", not as "apply no options", so a
+      // plugin opting out of console-stripping would also lose TS/JSX
+      // transformation for any file its own plugins do not handle. An empty
+      // options object keeps the transform and applies nothing.
+      esbuild: ctx.esbuild === false ? {} : ctx.esbuild ?? DEFAULT_ESBUILD,
       build: {
         target: ctx.target,
         outDir: ctx.outDir,
         emptyOutDir: false,
         assetsDir: "",
         minify: ctx.minify,
+        cssMinify: ctx.cssMinify ?? "esbuild",
+        reportCompressedSize: ctx.reportCompressedSize ?? false,
         sourcemap: ctx.sourcemap,
         rollupOptions: {
           input: { [ctx.pageId]: ctx.entryId },
@@ -1078,6 +1102,7 @@ function makeDefaultBuild(viteBuild) {
           output: {
             format: "es",
             inlineDynamicImports: true,
+            sourcemapExcludeSources: ctx.sourcemapExcludeSources ?? true,
             entryFileNames: `${ctx.outputPrefix}/[name].js`,
             chunkFileNames: `${ctx.outputPrefix}/[name]-[hash].js`,
             assetFileNames: `${ctx.outputPrefix}/[name][extname]`
@@ -1174,9 +1199,13 @@ async function buildPlatformReactPages(options = {}) {
       dedupe,
       alias,
       plugins,
-      target: options.target ?? "es2022",
-      minify: options.minify ?? false,
-      sourcemap: options.sourcemap ?? true
+      target: options.target ?? "esnext",
+      minify: options.minify ?? true,
+      sourcemap: options.sourcemap ?? "hidden",
+      sourcemapExcludeSources: options.sourcemapExcludeSources ?? true,
+      esbuild: options.esbuild ?? DEFAULT_ESBUILD,
+      cssMinify: options.cssMinify ?? "esbuild",
+      reportCompressedSize: options.reportCompressedSize ?? false
     });
   }
   if (overlay) {
@@ -1196,9 +1225,13 @@ async function buildPlatformReactPages(options = {}) {
       dedupe,
       alias,
       plugins,
-      target: options.target ?? "es2022",
-      minify: options.minify ?? false,
-      sourcemap: options.sourcemap ?? true
+      target: options.target ?? "esnext",
+      minify: options.minify ?? true,
+      sourcemap: options.sourcemap ?? "hidden",
+      sourcemapExcludeSources: options.sourcemapExcludeSources ?? true,
+      esbuild: options.esbuild ?? DEFAULT_ESBUILD,
+      cssMinify: options.cssMinify ?? "esbuild",
+      reportCompressedSize: options.reportCompressedSize ?? false
     });
   }
   const summaryPages = pages.map((p) => ({
@@ -2192,6 +2225,15 @@ function ethisysManifestPlugin(options = {}) {
     }
   };
 }
+/**
+ * esbuild options applied while minifying.
+ *
+ * `legalComments: "eof"`, not `"none"`: this builder bundles every non-external
+ * dependency into each surface, so stripping `/*!` and `@preserve` banners would
+ * distribute third-party code without the attribution its licence requires.
+ * Collecting them at end of file keeps the notices while leaving the code itself
+ * comment-free.
+ */
 
 export { CONTRACT_B_IMPORT_MAP_ALLOWLIST, CONTRACT_B_RUNTIME_IMPORTS, CONTRACT_B_SEMANTIC_PRIMITIVES, EHX_PLUGIN_CSS_SCOPE, EHX_PLUGIN_ROOT_SCOPE, EHX_PLUGIN_SCOPE_ROOTS, EHX_SCOPED_CSS_LAYERS, PLATFORM_REACT_DEDUPE, PLATFORM_REACT_EXTERNALS, buildIframeSandboxPages, buildPlatformReactPages, computePlatformReactPages, ethisysContractAPlugin, ethisysContractBPlugin, ethisysIframeSandboxPlugin, ethisysManifestPlugin, ethisysPlatformReactPagesPlugin, ethisysPlatformReactPlugin, findUnscopedSelectors, formatUnscopedSelectorError, generatePlatformReactManifest, isScopableCssId, parseExportedNames, parsePlatformReactPages, rewriteAliasedExternalImports, scopePluginCss, scopePluginCssPlugin, validateDeclarativeResource, validateReactiveRule, virtualPageEntry };
 //# sourceMappingURL=index.js.map