@genesislcap/vite-builder 15.65.1-alpha-358eaead70.0 → 15.67.0

package/dist/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,wBAAwB,CAAC;yBA2CrC,KAAK,YAAY;AAAvC,wBAyQE"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,wBAAwB,CAAC;yBA0CrC,KAAK,YAAY;AAAvC,wBA0OE"}

package/dist/index.js

@@ -35,94 +35,21 @@ const VITE_DEFAULT_DEP_SCAN_ENTRIES = ['**/*.html', '!**/__tests__/**', '!**/cov
 exports.default = (ctx) => tslib_1.__awaiter(void 0, void 0, void 0, function* () {
     var _a, _b, _c, _d;
     const { config: { app: { rootElement }, env, http, }, dirs: { cwd }, cli: { isDev, isBuild, isAnalyze, options: { open, https }, }, pkg: { description }, resolve: { alias }, } = ctx;
+    const define = (0, build_kit_1.resolveDefineConfig)(env, http, 'vite');
     /**
-     * Everything derived from `process.env`. Returned from the `genesis-env` plugin's `config` hook
-     * rather than baked into the inline config, so it is recomputed whenever Vite resolves config.
+     * The same map, narrowed to keys esbuild will accept as defines (see the `esbuildOptions` note in
+     * `optimizeDeps` below for why the optimizer needs it at all).
+     *
+     * `resolveDefineConfig` derives its keys from `process.env` via pattern match, so a machine with,
+     * say, `GENX-SOMETHING` set would produce a key that is not a valid JS identifier. Vite tolerates
+     * that in its own `define`; esbuild rejects the whole build. Filtering keeps a stray environment
+     * variable from turning into a hard build failure for everyone.
      */
-    let publicPath = (0, build_kit_1.getPublicPath)();
-    const envConfig = () => {
-        const define = (0, build_kit_1.resolveDefineConfig)(env, http, 'vite');
-        /**
-         * The same map, narrowed to keys esbuild will accept as defines (see the `esbuildOptions` note
-         * below for why the optimizer needs it at all).
-         *
-         * `resolveDefineConfig` derives its keys from `process.env` via pattern match, so a machine with,
-         * say, `GENX-SOMETHING` set would produce a key that is not a valid JS identifier. Vite tolerates
-         * that in its own `define`; esbuild rejects the whole build. Filtering keeps a stray environment
-         * variable from turning into a hard build failure for everyone.
-         */
-        const IDENTIFIER = /^[A-Za-z_$][A-Za-z0-9_$]*$/;
-        const depDefine = Object.fromEntries(Object.entries(define).filter(([key]) => IDENTIFIER.test(key)));
-        publicPath = (0, build_kit_1.getPublicPath)();
-        return {
-            define,
-            base: publicPath,
-            optimizeDeps: {
-                /**
-                 * GENC-1581 — the `define` above is applied when Vite TRANSFORMS source, but dependency
-                 * pre-bundling is a separate esbuild pass that does not inherit it. Any env-driven global
-                 * consumed from inside a pre-bundled dependency was therefore left as a bare identifier in
-                 * `node_modules/.vite/deps`, and silently resolved to `undefined` at runtime.
-                 *
-                 * MEASURED before this change: the served `@genesislcap/foundation-utils` chunk contained
-                 * `_DEFAULT_USER = DEFAULT_USER;` with no substitution, and the configured value appeared
-                 * nowhere in the bundle. `foundation-utils` reads these behind `try {} catch {}`, so the
-                 * resulting ReferenceError was swallowed and the variable simply stayed undefined — no
-                 * error, no warning, just a feature that never worked. `DEFAULT_USER`/`DEFAULT_PASSWORD`
-                 * (login pre-fill) is the case that surfaced it; every dependency-resident define was
-                 * equally affected. `API_HOST` masked the problem because `foundation-comms` falls back to
-                 * deriving it from `location`, and the dev proxy makes that fallback correct.
-                 *
-                 * Passing the same map to the optimizer's esbuild keeps source and dependencies consistent.
-                 * Vite hashes `optimizeDeps` into the dep-cache key, so changing a value re-optimizes rather
-                 * than silently serving a stale chunk.
-                 */
-                esbuildOptions: {
-                    define: Object.assign({ 
-                        /**
-                         * Vite seeds its OWN optimizer define with `process.env.NODE_ENV` and then spreads
-                         * `esbuildOptions` straight over it (`vite/dist/node/chunks/config.js`: `define: define$1,
-                         * … …esbuildOptions,` — note `supported` gets an explicit deep-merge just below and
-                         * `define` does not). So a bare `define` here REPLACES that map rather than adding to it,
-                         * and pre-bundled deps lose the substitution.
-                         *
-                         * That is not academic: `@reduxjs/toolkit` resolves through its `module` condition to
-                         * `redux-toolkit.modern.mjs` (57 bare `process.env.NODE_ENV` reads) plus `redux` (22),
-                         * and `foundation-redux/src/store.ts` calls `configureStore` at module scope — so any app
-                         * pulling in `@genesislcap/foundation-redux` would get `ReferenceError: process is not
-                         * defined` out of the dep chunk. Dev only (`vite build` applies `define` through Rollup
-                         * for node_modules too), which is exactly the path this change targets.
-                         *
-                         * Re-seeded before the spread. `depDefine` cannot collide with it: its keys are filtered
-                         * to bare identifiers, and this one is dotted.
-                         */
-                        'process.env.NODE_ENV': JSON.stringify(process.env.NODE_ENV || (isDev ? 'development' : 'production')) }, depDefine),
-                },
-            },
-            server: {
-                proxy: (0, build_kit_1.createDevProxies)(),
-            },
-        };
-    };
-    /**
-     * FUI-2654 — Vite already restarts the dev server in-process when a `.env` file in `envDir` (the
-     * root) changes, but the restart re-resolves the same inline config object, so defines, `base`
-     * and the proxy target captured at startup stayed stale until a manual restart. Plugin `config`
-     * hooks DO re-run on every restart, so reload the env files and hand back fresh values here.
-     */
-    let isFirstConfig = true;
-    const envPlugin = {
-        name: 'genesis-env',
-        config() {
-            // The first resolve runs right after genx's own `setEnvVars`, which already loaded them
-            if (!isFirstConfig)
-                (0, build_kit_1.loadEnvFiles)(ctx);
-            isFirstConfig = false;
-            return envConfig();
-        },
-    };
+    const IDENTIFIER = /^[A-Za-z_$][A-Za-z0-9_$]*$/;
+    const depDefine = Object.fromEntries(Object.entries(define).filter(([key]) => IDENTIFIER.test(key)));
     const handlebars = yield (0, vite_js_1.getHandlebarsModule)();
     const vite = yield (0, vite_js_1.getViteModule)();
+    const publicPath = (0, build_kit_1.getPublicPath)();
     const tsconfigPath = (0, node_path_1.resolve)(cwd, 'tsconfig.json');
     const buildMode = (_c = (_b = (_a = ctx.config.vite) === null || _a === void 0 ? void 0 : _a.checker) === null || _b === void 0 ? void 0 : _b.buildMode) !== null && _c !== void 0 ? _c : false;
     const checkerPlugin = (0, node_fs_1.existsSync)(tsconfigPath)
@@ -140,7 +67,6 @@ exports.default = (ctx) => tslib_1.__awaiter(void 0, void 0, void 0, function* (
     };
     const config = {
         plugins: [
-            envPlugin,
             baseHrefPlugin,
             ...(checkerPlugin ? [checkerPlugin] : []),
             handlebars({
@@ -206,6 +132,7 @@ exports.default = (ctx) => tslib_1.__awaiter(void 0, void 0, void 0, function* (
                 '@genesislcap/pbc-reporting',
             ],
         },
+        define,
         build: {
             /**
              * Left to Vite's production default (esbuild for JS and CSS) unless the app opts out, matching
@@ -239,7 +166,48 @@ exports.default = (ctx) => tslib_1.__awaiter(void 0, void 0, void 0, function* (
              * reference, should we convert the above to a glob.
              */
             include: ['@genesis-community/golden-layout'],
+            /**
+             * GENC-1581 — the `define` above is applied when Vite TRANSFORMS source, but dependency
+             * pre-bundling is a separate esbuild pass that does not inherit it. Any env-driven global
+             * consumed from inside a pre-bundled dependency was therefore left as a bare identifier in
+             * `node_modules/.vite/deps`, and silently resolved to `undefined` at runtime.
+             *
+             * MEASURED before this change: the served `@genesislcap/foundation-utils` chunk contained
+             * `_DEFAULT_USER = DEFAULT_USER;` with no substitution, and the configured value appeared
+             * nowhere in the bundle. `foundation-utils` reads these behind `try {} catch {}`, so the
+             * resulting ReferenceError was swallowed and the variable simply stayed undefined — no
+             * error, no warning, just a feature that never worked. `DEFAULT_USER`/`DEFAULT_PASSWORD`
+             * (login pre-fill) is the case that surfaced it; every dependency-resident define was
+             * equally affected. `API_HOST` masked the problem because `foundation-comms` falls back to
+             * deriving it from `location`, and the dev proxy makes that fallback correct.
+             *
+             * Passing the same map to the optimizer's esbuild keeps source and dependencies consistent.
+             * Vite hashes `optimizeDeps` into the dep-cache key, so changing a value re-optimizes rather
+             * than silently serving a stale chunk.
+             */
+            esbuildOptions: {
+                define: Object.assign({ 
+                    /**
+                     * Vite seeds its OWN optimizer define with `process.env.NODE_ENV` and then spreads
+                     * `esbuildOptions` straight over it (`vite/dist/node/chunks/config.js`: `define: define$1,
+                     * … …esbuildOptions,` — note `supported` gets an explicit deep-merge just below and
+                     * `define` does not). So a bare `define` here REPLACES that map rather than adding to it,
+                     * and pre-bundled deps lose the substitution.
+                     *
+                     * That is not academic: `@reduxjs/toolkit` resolves through its `module` condition to
+                     * `redux-toolkit.modern.mjs` (57 bare `process.env.NODE_ENV` reads) plus `redux` (22),
+                     * and `foundation-redux/src/store.ts` calls `configureStore` at module scope — so any app
+                     * pulling in `@genesislcap/foundation-redux` would get `ReferenceError: process is not
+                     * defined` out of the dep chunk. Dev only (`vite build` applies `define` through Rollup
+                     * for node_modules too), which is exactly the path this change targets.
+                     *
+                     * Re-seeded before the spread. `depDefine` cannot collide with it: its keys are filtered
+                     * to bare identifiers, and this one is dotted.
+                     */
+                    'process.env.NODE_ENV': JSON.stringify(process.env.NODE_ENV || (isDev ? 'development' : 'production')) }, depDefine),
+            },
         },
+        base: publicPath,
     };
     if (!(0, node_fs_1.existsSync)((0, node_path_1.resolve)(cwd, 'index.html'))) {
         consola_1.default.warn('index.html file not found - copying default one.');
@@ -252,6 +220,7 @@ exports.default = (ctx) => tslib_1.__awaiter(void 0, void 0, void 0, function* (
                 open,
                 host: '0.0.0.0',
                 port: (0, build_kit_1.getPort)(ctx.pkg),
+                proxy: (0, build_kit_1.createDevProxies)(),
                 https: https ? (0, build_kit_1.getDevCertOptions)() : undefined,
             } }));
         yield server.listen();

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@genesislcap/vite-builder",
   "description": "Vite builder",
-  "version": "15.65.1-alpha-358eaead70.0",
+  "version": "15.67.0",
   "license": "SEE LICENSE IN license.txt",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
@@ -16,12 +16,12 @@
     "lint:fix": "genx lint -l ox --fix"
   },
   "dependencies": {
-    "@genesislcap/build-kit": "15.65.1-alpha-358eaead70.0",
+    "@genesislcap/build-kit": "15.67.0",
     "consola": "^3.0.2",
     "copyfiles": "^2.4.1",
     "vite": "^7.2.2",
     "vite-plugin-checker": "^0.11.0",
-    "vite-plugin-handlebars": "2.0.0"
+    "vite-plugin-handlebars": "^2.0.3"
   },
   "repository": {
     "type": "git",