vike 0.4.238 → 0.4.239-commit-33e55d4

package/dist/cjs/node/vite/plugins/pluginDev.js

@@ -22,54 +22,58 @@ function pluginDev() {
         {
             name: 'vike:pluginDev',
             apply: utils_js_1.applyDev,
-            config() {
-                return {
-                    appType: 'custom',
-                    // TO-DO/next-major-release: remove (AFAICT we only need to use config.optimizeDeps for the old design)
-                    optimizeDeps: {
-                        exclude: [
-                            // We exclude Vike's client runtime to be able to use Vite's import.meta.glob()
-                            'vike/client',
-                            'vike/client/router',
-                            // It seems like client-side/isomorphic imports also need to be excluded, in order to avoid the following:
-                            //   ```
-                            //   Client runtime loaded twice https://vike.dev/client-runtime-duplicated
-                            //   ```
-                            'vike/routing',
-                            'vike/getPageContext',
-                            // We exclude @brillout/json-serializer and @brillout/picocolors to avoid:
-                            // ```
-                            // 9:28:58 AM [vite] ✨ new dependencies optimized: @brillout/json-serializer/parse
-                            // 9:28:58 AM [vite] ✨ optimized dependencies changed. reloading
-                            // ```
-                            '@brillout/json-serializer/parse',
-                            '@brillout/json-serializer/stringify',
-                            '@brillout/picocolors',
-                            // We exclude all packages that depend on any optimizeDeps.exclude entry because, otherwise, the entry cannot be resolved when using pnpm. For example:
-                            // ```
-                            // Failed to resolve import "@brillout/json-serializer/parse" from "../../packages/vike-react-query/dist/renderer/VikeReactQueryWrapper.js". Does the file exist?
-                            // 343|  // ../../node_modules/.pnpm/react-streaming@0.3.16_react-dom@18.2.0_react@18.2.0/node_modules/react-streaming/dist/esm/client/useAsync.js
-                            // 344|  import { parse as parse2 } from "@brillout/json-serializer/parse";
-                            // ```
-                            // The source map is confusing, the import actually lives at node_modules/.vite/deps/vike-react-query_renderer_VikeReactQueryWrapper.js which contains:
-                            // ```js
-                            // // ../../node_modules/.pnpm/react-streaming@0.3.16_react-dom@18.2.0_react@18.2.0/node_modules/react-streaming/dist/esm/client/useAsync.js
-                            // import { parse as parse2 } from "@brillout/json-serializer/parse";
-                            // ```
-                            'react-streaming',
-                        ],
-                    },
-                };
+            config: {
+                handler() {
+                    return {
+                        appType: 'custom',
+                        // TO-DO/next-major-release: remove (AFAICT we only need to use config.optimizeDeps for the old design)
+                        optimizeDeps: {
+                            exclude: [
+                                // We exclude Vike's client runtime to be able to use Vite's import.meta.glob()
+                                'vike/client',
+                                'vike/client/router',
+                                // It seems like client-side/isomorphic imports also need to be excluded, in order to avoid the following:
+                                //   ```
+                                //   Client runtime loaded twice https://vike.dev/client-runtime-duplicated
+                                //   ```
+                                'vike/routing',
+                                'vike/getPageContext',
+                                // We exclude @brillout/json-serializer and @brillout/picocolors to avoid:
+                                // ```
+                                // 9:28:58 AM [vite] ✨ new dependencies optimized: @brillout/json-serializer/parse
+                                // 9:28:58 AM [vite] ✨ optimized dependencies changed. reloading
+                                // ```
+                                '@brillout/json-serializer/parse',
+                                '@brillout/json-serializer/stringify',
+                                '@brillout/picocolors',
+                                // We exclude all packages that depend on any optimizeDeps.exclude entry because, otherwise, the entry cannot be resolved when using pnpm. For example:
+                                // ```
+                                // Failed to resolve import "@brillout/json-serializer/parse" from "../../packages/vike-react-query/dist/renderer/VikeReactQueryWrapper.js". Does the file exist?
+                                // 343|  // ../../node_modules/.pnpm/react-streaming@0.3.16_react-dom@18.2.0_react@18.2.0/node_modules/react-streaming/dist/esm/client/useAsync.js
+                                // 344|  import { parse as parse2 } from "@brillout/json-serializer/parse";
+                                // ```
+                                // The source map is confusing, the import actually lives at node_modules/.vite/deps/vike-react-query_renderer_VikeReactQueryWrapper.js which contains:
+                                // ```js
+                                // // ../../node_modules/.pnpm/react-streaming@0.3.16_react-dom@18.2.0_react@18.2.0/node_modules/react-streaming/dist/esm/client/useAsync.js
+                                // import { parse as parse2 } from "@brillout/json-serializer/parse";
+                                // ```
+                                'react-streaming',
+                            ],
+                        },
+                    };
+                },
             },
-            async configResolved(config_) {
-                config = config_;
-                await (0, determineOptimizeDeps_js_1.determineOptimizeDeps)(config);
-                await (0, determineFsAllowList_js_1.determineFsAllowList)(config);
-                if (!(0, isErrorDebug_js_1.isErrorDebug)()) {
-                    await (0, getHttpRequestAsyncStore_js_1.installHttpRequestAsyncStore)();
-                    (0, loggerVite_js_1.improveViteLogs)(config);
-                }
-                logDockerHint(config.server.host);
+            configResolved: {
+                async handler(config_) {
+                    config = config_;
+                    await (0, determineOptimizeDeps_js_1.determineOptimizeDeps)(config);
+                    await (0, determineFsAllowList_js_1.determineFsAllowList)(config);
+                    if (!(0, isErrorDebug_js_1.isErrorDebug)()) {
+                        await (0, getHttpRequestAsyncStore_js_1.installHttpRequestAsyncStore)();
+                        (0, loggerVite_js_1.improveViteLogs)(config);
+                    }
+                    logDockerHint(config.server.host);
+                },
             },
         },
         {

package/dist/cjs/node/vite/plugins/pluginEnvVars.js

@@ -7,6 +7,7 @@ const getFilePath_js_1 = require("../shared/getFilePath.js");
 const normalizeId_js_1 = require("../shared/normalizeId.js");
 const isViteServerSide_js_1 = require("../shared/isViteServerSide.js");
 const getMagicString_js_1 = require("../shared/getMagicString.js");
+const pluginReplaceConstants_js_1 = require("./pluginReplaceConstants.js");
 // TO-DO/eventually:
 // - Make ({}) work inside +config.js
 //   - For it to work, we'll probably need the user to define the settings (e.g. `envDir`) for loadEnv() inside vike.config.js instead of vite.config.js
@@ -22,66 +23,70 @@ function pluginEnvVars() {
     return {
         name: 'vike:pluginEnvVars',
         enforce: 'post',
-        configResolved(config_) {
-            config = config_;
-            envsAll = (0, vite_1.loadEnv)(config.mode, config.envDir || config.root, '');
-            config.plugins.sort((0, utils_js_1.lowerFirst)((plugin) => (plugin.name === 'vite:define' ? 1 : 0)));
+        configResolved: {
+            handler(config_) {
+                config = config_;
+                envsAll = (0, vite_1.loadEnv)(config.mode, config.envDir || config.root, '');
+                config.plugins.sort((0, utils_js_1.lowerFirst)((plugin) => (plugin.name === 'vite:define' ? 1 : 0)));
+            },
         },
-        transform(code, id, options) {
-            id = (0, normalizeId_js_1.normalizeId)(id);
-            (0, utils_js_1.assertPosixPath)(id);
-            if (id.includes('/node_modules/'))
-                return;
-            (0, utils_js_1.assertPosixPath)(config.root);
-            if (!id.startsWith(config.root))
-                return;
-            if (!code.includes('({}).'))
-                return;
-            const isBuild = config.command === 'build';
-            const isClientSide = !(0, isViteServerSide_js_1.isViteServerSide_extraSafe)(config, this.environment, options);
-            const { magicString, getMagicStringResult } = (0, getMagicString_js_1.getMagicString)(code, id);
-            // Find & check
-            const replacements = Object.entries(envsAll)
-                .filter(([key]) => {
-                // Already handled by Vite
-                const envPrefix = !config.envPrefix ? [] : (0, utils_js_1.isArray)(config.envPrefix) ? config.envPrefix : [config.envPrefix];
-                return !envPrefix.some((prefix) => key.startsWith(prefix));
-            })
-                .map(([envName, envVal]) => {
-                const envStatement = `({}).${envName}`;
-                const envStatementRegExpStr = (0, utils_js_1.escapeRegex)(envStatement) + '\\b';
-                // Security check
-                {
-                    const isPrivate = !envName.startsWith(PUBLIC_ENV_PREFIX) && !PUBLIC_ENV_ALLOWLIST.includes(envName);
-                    if (isPrivate && isClientSide) {
-                        if (!new RegExp(envStatementRegExpStr).test(code))
-                            return;
-                        const modulePath = (0, getFilePath_js_1.getModuleFilePathAbsolute)(id, config);
-                        const errMsgAddendum = isBuild ? '' : ' (Vike will prevent your app from building for production)';
-                        const keyPublic = `${PUBLIC_ENV_PREFIX}${envName}`;
-                        const errMsg = `${envStatement} is used in client-side file ${modulePath} which means that the environment variable ${envName} will be included in client-side bundles and, therefore, ${envName} will be publicly exposed which can be a security leak${errMsgAddendum}. Use ${envStatement} only in server-side files, or rename ${envName} to ${keyPublic}, see https://vike.dev/env`;
-                        if (isBuild) {
-                            (0, utils_js_1.assertUsage)(false, errMsg);
-                        }
-                        else {
-                            // - Only a warning for faster development DX (e.g. when user toggles `ssr: boolean` or `onBeforeRenderIsomorph: boolean`).
-                            // - But only showing a warning can be confusing: https://github.com/vikejs/vike/issues/1641
-                            (0, utils_js_1.assertWarning)(false, errMsg, { onlyOnce: true });
+        transform: {
+            filter: pluginReplaceConstants_js_1.filterRolldown,
+            handler(code, id, options) {
+                id = (0, normalizeId_js_1.normalizeId)(id);
+                (0, utils_js_1.assertPosixPath)(id);
+                (0, utils_js_1.assertPosixPath)(config.root);
+                if (!id.startsWith(config.root))
+                    return; // skip linked dependencies
+                (0, utils_js_1.assert)((0, pluginReplaceConstants_js_1.filterFunction)(id, code));
+                const isBuild = config.command === 'build';
+                const isClientSide = !(0, isViteServerSide_js_1.isViteServerSide_extraSafe)(config, this.environment, options);
+                const { magicString, getMagicStringResult } = (0, getMagicString_js_1.getMagicString)(code, id);
+                // Find & check
+                const replacements = Object.entries(envsAll)
+                    .filter(([key]) => {
+                    // Already handled by Vite
+                    const envPrefix = !config.envPrefix ? [] : (0, utils_js_1.isArray)(config.envPrefix) ? config.envPrefix : [config.envPrefix];
+                    return !envPrefix.some((prefix) => key.startsWith(prefix));
+                })
+                    .map(([envName, envVal]) => {
+                    const envStatement = `({}).${envName}`;
+                    const envStatementRegExpStr = (0, utils_js_1.escapeRegex)(envStatement) + '\\b';
+                    // Security check
+                    {
+                        const isPrivate = !envName.startsWith(PUBLIC_ENV_PREFIX) && !PUBLIC_ENV_ALLOWLIST.includes(envName);
+                        if (isPrivate && isClientSide) {
+                            if (!new RegExp(envStatementRegExpStr).test(code))
+                                return;
+                            const modulePath = (0, getFilePath_js_1.getModuleFilePathAbsolute)(id, config);
+                            const errMsgAddendum = isBuild
+                                ? ''
+                                : ' (Vike will prevent your app from building for production)';
+                            const keyPublic = `${PUBLIC_ENV_PREFIX}${envName}`;
+                            const errMsg = `${envStatement} is used in client-side file ${modulePath} which means that the environment variable ${envName} will be included in client-side bundles and, therefore, ${envName} will be publicly exposed which can be a security leak${errMsgAddendum}. Use ${envStatement} only in server-side files, or rename ${envName} to ${keyPublic}, see https://vike.dev/env`;
+                            if (isBuild) {
+                                (0, utils_js_1.assertUsage)(false, errMsg);
+                            }
+                            else {
+                                // - Only a warning for faster development DX (e.g. when user toggles `ssr: boolean` or `onBeforeRenderIsomorph: boolean`).
+                                // - But only showing a warning can be confusing: https://github.com/vikejs/vike/issues/1641
+                                (0, utils_js_1.assertWarning)(false, errMsg, { onlyOnce: true });
+                            }
                         }
+                        // Double check
+                        (0, utils_js_1.assert)(!(isPrivate && isClientSide) || !isBuild);
                     }
-                    // Double check
-                    (0, utils_js_1.assert)(!(isPrivate && isClientSide) || !isBuild);
-                }
-                return { regExpStr: envStatementRegExpStr, replacement: envVal };
-            })
-                .filter(utils_js_1.isNotNullish);
-            // Apply
-            replacements.forEach(({ regExpStr, replacement }) => {
-                magicString.replaceAll(new RegExp(regExpStr, 'g'), JSON.stringify(replacement));
-            });
-            if (!magicString.hasChanged())
-                return null;
-            return getMagicStringResult();
+                    return { regExpStr: envStatementRegExpStr, replacement: envVal };
+                })
+                    .filter(utils_js_1.isNotNullish);
+                // Apply
+                replacements.forEach(({ regExpStr, replacement }) => {
+                    magicString.replaceAll(new RegExp(regExpStr, 'g'), JSON.stringify(replacement));
+                });
+                if (!magicString.hasChanged())
+                    return null;
+                return getMagicStringResult();
+            },
         },
     };
 }

package/dist/cjs/node/vite/plugins/pluginExtractAssets.js

@@ -1,6 +1,5 @@
 "use strict";
-// Remove this workaround if the other workaround config.build.ssrEmitAssets turns out to be reliable.
-//  - Remove this file then revert this commit: https://github.com/vikejs/vike/commit/805a18974f13420a78fcc30fdd676696e405c3ca
+// TO-DO/next-major-release: remove
 var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
@@ -24,6 +23,12 @@ const rawRE = /(\?|&)raw(?:&|$)/;
 const urlRE = /(\?|&)url(?:&|$)/;
 const EMPTY_MODULE_ID = 'virtual:vike:empty-module';
 const debug = (0, utils_js_1.createDebugger)('vike:pluginExtractAssets');
+const filterRolldown = {
+    id: {
+        include: extractAssetsRE,
+    },
+};
+const filterFunction = (id) => extractAssetsRE.test(id);
 function pluginExtractAssets() {
     let config;
     let vikeConfig;
@@ -35,24 +40,25 @@ function pluginExtractAssets() {
             // In dev, things just work. (Because Vite's module graph erroneously conflates the Vite server-side importees with the client-side importees.)
             apply: 'build',
             enforce: 'post',
-            async transform(src, id, options) {
-                id = (0, normalizeId_js_1.normalizeId)(id);
-                if (!extractAssetsRE.test(id)) {
-                    return;
-                }
-                if (isFixEnabled) {
-                    // I'm guessing isFixEnabled can only be true when mixing both designs: https://github.com/vikejs/vike/issues/1480
-                    (0, assertV1Design_js_1.assertV1Design)(vikeConfig._pageConfigs, true);
-                    (0, utils_js_1.assert)(false);
-                }
-                const includeAssetsImportedByServer = (0, retrievePageAssetsProd_js_1.resolveIncludeAssetsImportedByServer)(vikeConfig.config);
-                (0, utils_js_1.assert)(includeAssetsImportedByServer);
-                (0, utils_js_1.assert)(!(0, isViteServerSide_js_1.isViteServerSide_extraSafe)(config, this.environment, options));
-                const importStatements = await (0, parseEsModule_js_1.getImportStatements)(src);
-                const moduleNames = getImportedModules(importStatements);
-                const code = moduleNames.map((moduleName) => `import '${moduleName}';`).join('\n');
-                debugTransformResult(id, code, importStatements);
-                return (0, utils_js_1.rollupSourceMapRemove)(code);
+            transform: {
+                filter: filterRolldown,
+                async handler(src, id, options) {
+                    id = (0, normalizeId_js_1.normalizeId)(id);
+                    (0, utils_js_1.assert)(filterFunction(id));
+                    if (isFixEnabled) {
+                        // I'm guessing isFixEnabled can only be true when mixing both designs: https://github.com/vikejs/vike/issues/1480
+                        (0, assertV1Design_js_1.assertV1Design)(vikeConfig._pageConfigs, true);
+                        (0, utils_js_1.assert)(false);
+                    }
+                    const includeAssetsImportedByServer = (0, retrievePageAssetsProd_js_1.resolveIncludeAssetsImportedByServer)(vikeConfig.config);
+                    (0, utils_js_1.assert)(includeAssetsImportedByServer);
+                    (0, utils_js_1.assert)(!(0, isViteServerSide_js_1.isViteServerSide_extraSafe)(config, this.environment, options));
+                    const importStatements = await (0, parseEsModule_js_1.getImportStatements)(src);
+                    const moduleNames = getImportedModules(importStatements);
+                    const code = moduleNames.map((moduleName) => `import '${moduleName}';`).join('\n');
+                    debugTransformResult(id, code, importStatements);
+                    return (0, utils_js_1.rollupSourceMapRemove)(code);
+                },
             },
         },
         // This plugin appends `?extractAssets` to module IDs
@@ -63,89 +69,97 @@ function pluginExtractAssets() {
             //  - rollup's `alias` plugin; https://github.com/rollup/plugins/blob/5363f55aa1933b6c650832b08d6a54cb9ea64539/packages/alias/src/index.ts
             //  - Vite's `vite:resolve` plugin; https://github.com/vitejs/vite/blob/d649daba7682791178b711d9a3e44a6b5d00990c/packages/vite/src/node/plugins/resolve.ts#L105
             enforce: 'pre',
-            async resolveId(source, importer, options) {
-                if ((0, isViteServerSide_js_1.isViteServerSide_extraSafe)(config, this.environment, options)) {
-                    // When building for the server, there should never be a `?extractAssets` query
-                    (0, utils_js_1.assert)(!extractAssetsRE.test(source));
-                    (0, utils_js_1.assert)(importer === undefined || !extractAssetsRE.test(importer));
-                    return;
-                }
-                // If there is no `importer` then `module` is an entry.
-                // We don't need to append `?extractAssets` to entries because they already have `?extractAssets` as Vike appends `?extractAssets` to entries by using `import.meta.glob('/**/*.page.server.js', { as: "extractAssets" })` (see `generateImportGlobs.ts`).
-                if (!importer) {
-                    return;
-                }
-                // We only append `?extractAssets` if the parent module has `?extractAssets`
-                if (!extractAssetsRE.test(importer)) {
-                    return;
-                }
-                const includeAssetsImportedByServer = (0, retrievePageAssetsProd_js_1.resolveIncludeAssetsImportedByServer)(vikeConfig.config);
-                (0, utils_js_1.assert)(includeAssetsImportedByServer);
-                let resolution = null;
-                try {
-                    resolution = await this.resolve(source, importer, { skipSelf: true, ...options });
-                }
-                catch { }
-                // Sometimes Rollup fails to resolve. If it fails to resolve, we assume the dependency to be an npm package and we skip it. (I guess Rollup should always be able to resolve local dependencies?)
-                if (!resolution)
-                    return emptyModule(source, importer);
-                const { id: file, external } = resolution;
-                // Nothing is externalized when building for the client-side
-                (0, utils_js_1.assert)(external === false);
-                // We include:
-                //  - CSS(/LESS/SCSS/...) files
-                //  - Asset files (`.svg`, `.pdf`, ...)
-                //  - URL imports (e.g. `import scriptUrl from './script.js?url.js'`)
-                if (utils_js_1.styleFileRE.test(file) || (0, isAsset_js_1.isAsset)(file) || urlRE.test(file)) {
-                    debugOperation('INCLUDED', file, importer);
-                    return resolution;
-                }
-                // We erase `source` if its file doesn't contain JavaScript
-                if (!(0, utils_js_1.isScriptFile)(file)) {
-                    return emptyModule(file, importer);
-                }
-                // If the import path resolves to a file in `node_modules/`, we ignore that file:
-                //  - Direct CSS dependencies are included though, such as `import 'bootstrap/theme/dark.css'`. (Because the above if-branch for CSS files will add the file.)
-                //  - Loading CSS from a library (living in `node_modules/`) in a non-direct way is unconventional; we can safely not support this case. (I'm not aware of any library that does this.)
-                (0, utils_js_1.assertPosixPath)(file);
-                if (file.includes('/node_modules/')) {
-                    return emptyModule(file, importer);
-                }
-                // When a library is symlinked, it lives outside `root`.
-                (0, utils_js_1.assertPosixPath)(config.root);
-                if (!file.startsWith(config.root)) {
-                    return emptyModule(file, importer);
-                }
-                return appendExtractAssetsQuery(file, importer);
+            resolveId: {
+                async handler(source, importer, options) {
+                    if ((0, isViteServerSide_js_1.isViteServerSide_extraSafe)(config, this.environment, options)) {
+                        // When building for the server, there should never be a `?extractAssets` query
+                        (0, utils_js_1.assert)(!extractAssetsRE.test(source));
+                        (0, utils_js_1.assert)(importer === undefined || !extractAssetsRE.test(importer));
+                        return;
+                    }
+                    // If there is no `importer` then `module` is an entry.
+                    // We don't need to append `?extractAssets` to entries because they already have `?extractAssets` as Vike appends `?extractAssets` to entries by using `import.meta.glob('/**/*.page.server.js', { as: "extractAssets" })` (see `generateImportGlobs.ts`).
+                    if (!importer) {
+                        return;
+                    }
+                    // We only append `?extractAssets` if the parent module has `?extractAssets`
+                    if (!extractAssetsRE.test(importer)) {
+                        return;
+                    }
+                    const includeAssetsImportedByServer = (0, retrievePageAssetsProd_js_1.resolveIncludeAssetsImportedByServer)(vikeConfig.config);
+                    (0, utils_js_1.assert)(includeAssetsImportedByServer);
+                    let resolution = null;
+                    try {
+                        resolution = await this.resolve(source, importer, { skipSelf: true, ...options });
+                    }
+                    catch { }
+                    // Sometimes Rollup fails to resolve. If it fails to resolve, we assume the dependency to be an npm package and we skip it. (I guess Rollup should always be able to resolve local dependencies?)
+                    if (!resolution)
+                        return emptyModule(source, importer);
+                    const { id: file, external } = resolution;
+                    // Nothing is externalized when building for the client-side
+                    (0, utils_js_1.assert)(external === false);
+                    // We include:
+                    //  - CSS(/LESS/SCSS/...) files
+                    //  - Asset files (`.svg`, `.pdf`, ...)
+                    //  - URL imports (e.g. `import scriptUrl from './script.js?url.js'`)
+                    if (utils_js_1.styleFileRE.test(file) || (0, isAsset_js_1.isAsset)(file) || urlRE.test(file)) {
+                        debugOperation('INCLUDED', file, importer);
+                        return resolution;
+                    }
+                    // We erase `source` if its file doesn't contain JavaScript
+                    if (!(0, utils_js_1.isScriptFile)(file)) {
+                        return emptyModule(file, importer);
+                    }
+                    // If the import path resolves to a file in `node_modules/`, we ignore that file:
+                    //  - Direct CSS dependencies are included though, such as `import 'bootstrap/theme/dark.css'`. (Because the above if-branch for CSS files will add the file.)
+                    //  - Loading CSS from a library (living in `node_modules/`) in a non-direct way is unconventional; we can safely not support this case. (I'm not aware of any library that does this.)
+                    (0, utils_js_1.assertPosixPath)(file);
+                    if (file.includes('/node_modules/')) {
+                        return emptyModule(file, importer);
+                    }
+                    // When a library is symlinked, it lives outside `root`.
+                    (0, utils_js_1.assertPosixPath)(config.root);
+                    if (!file.startsWith(config.root)) {
+                        return emptyModule(file, importer);
+                    }
+                    return appendExtractAssetsQuery(file, importer);
+                },
             },
         },
         {
             name: 'vike:pluginExtractAssets-3',
             apply: 'build',
-            load(id) {
-                if (!(0, utils_js_1.isVirtualFileId)(id))
-                    return undefined;
-                id = (0, utils_js_1.removeVirtualFileIdPrefix)(id);
-                if (id === EMPTY_MODULE_ID) {
-                    return '// Erased by vike:pluginExtractAssets';
-                }
+            load: {
+                handler(id) {
+                    if (!(0, utils_js_1.isVirtualFileId)(id))
+                        return undefined;
+                    id = (0, utils_js_1.removeVirtualFileIdPrefix)(id);
+                    if (id === EMPTY_MODULE_ID) {
+                        return '// Erased by vike:pluginExtractAssets';
+                    }
+                },
             },
-            config() {
-                if (debug.isActivated) {
-                    return { logLevel: 'silent' };
-                }
+            config: {
+                handler() {
+                    if (debug.isActivated) {
+                        return { logLevel: 'silent' };
+                    }
+                },
             },
         },
         {
             name: 'vike:pluginExtractAssets-4',
-            async configResolved(config_) {
-                config = config_;
-                vikeConfig = await (0, resolveVikeConfigInternal_js_1.getVikeConfigInternal)();
-                isFixEnabled = (0, handleAssetsManifest_js_1.handleAssetsManifest_isFixEnabled)();
-                if (!isFixEnabled) {
-                    // https://github.com/vikejs/vike/issues/1060
-                    (0, utils_js_1.assertUsage)(!config.plugins.find((p) => p.name === 'vite-tsconfig-paths'), 'vite-tsconfig-paths not supported, remove it and use vite.config.js#resolve.alias instead');
-                }
+            configResolved: {
+                async handler(config_) {
+                    config = config_;
+                    vikeConfig = await (0, resolveVikeConfigInternal_js_1.getVikeConfigInternal)();
+                    isFixEnabled = (0, handleAssetsManifest_js_1.handleAssetsManifest_isFixEnabled)();
+                    if (!isFixEnabled) {
+                        // https://github.com/vikejs/vike/issues/1060
+                        (0, utils_js_1.assertUsage)(!config.plugins.find((p) => p.name === 'vite-tsconfig-paths'), 'vite-tsconfig-paths not supported, remove it and use vite.config.js#resolve.alias instead');
+                    }
+                },
             },
         },
     ];

package/dist/cjs/node/vite/plugins/pluginExtractExportNames.js

@@ -11,31 +11,45 @@ const extractExportNamesRE = /(\?|&)extractExportNames(?:&|$)/;
 exports.extractExportNamesRE = extractExportNamesRE;
 const debug = (0, utils_js_1.createDebugger)('vike:pluginExtractExportNames');
 const globalObject = (0, utils_js_1.getGlobalObject)('plugins/pluginExtractExportNames.ts', {});
+const filterRolldown = {
+    id: {
+        include: extractExportNamesRE,
+    },
+};
+const filterFunction = (id) => extractExportNamesRE.test(id);
 function pluginExtractExportNames() {
     let isDev = false;
     let config;
     return {
         name: 'vike:pluginExtractExportNames',
         enforce: 'post',
-        async transform(src, id, options) {
-            id = (0, normalizeId_js_1.normalizeId)(id);
-            const isClientSide = !(0, isViteServerSide_js_1.isViteServerSide_extraSafe)(config, this.environment, options);
-            if (extractExportNamesRE.test(id)) {
+        transform: {
+            filter: filterRolldown,
+            async handler(src, id, options) {
+                id = (0, normalizeId_js_1.normalizeId)(id);
+                const isClientSide = !(0, isViteServerSide_js_1.isViteServerSide_extraSafe)(config, this.environment, options);
+                (0, utils_js_1.assert)(filterFunction(id));
                 const code = await getExtractExportNamesCode(src, isClientSide, !isDev, id);
                 debug('id ' + id, ['result:\n' + code.code.trim(), 'src:\n' + src.trim()]);
                 return code;
-            }
+            },
         },
-        configureServer() {
-            isDev = true;
+        configureServer: {
+            handler() {
+                isDev = true;
+            },
         },
-        configResolved(config_) {
-            config = config_;
+        configResolved: {
+            handler(config_) {
+                config = config_;
+            },
         },
-        config() {
-            if (debug.isActivated) {
-                return { logLevel: 'silent' };
-            }
+        config: {
+            handler() {
+                if (debug.isActivated) {
+                    return { logLevel: 'silent' };
+                }
+            },
         },
     };
 }