vike 0.4.238 → 0.4.239-commit-050a4a3

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

@@ -22,66 +22,72 @@ 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: {
+            handler(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 });
+                            }
                         }
+                        // 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

@@ -35,24 +35,26 @@ 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: {
+                async handler(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);
+                },
             },
         },
         // This plugin appends `?extractAssets` to module IDs
@@ -63,89 +65,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

@@ -17,25 +17,33 @@ function pluginExtractExportNames() {
     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)) {
-                const code = await getExtractExportNamesCode(src, isClientSide, !isDev, id);
-                debug('id ' + id, ['result:\n' + code.code.trim(), 'src:\n' + src.trim()]);
-                return code;
-            }
+        transform: {
+            async handler(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)) {
+                    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' };
+                }
+            },
         },
     };
 }

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

@@ -18,68 +18,78 @@ function pluginFileEnv() {
     let viteDevServer;
     return {
         name: 'vike:pluginFileEnv',
-        load(id, options) {
-            // In build, we use generateBundle() instead of the load() hook. Using load() works for dynamic imports in dev thanks to Vite's lazy transpiling, but it doesn't work in build because Rollup transpiles any dynamically imported module even if it's never actually imported.
-            if (!viteDevServer)
-                return;
-            if (!(0, resolveVikeConfigInternal_js_1.isV1Design)())
-                return;
-            if (skip(id))
-                return;
-            // For `.vue` files: https://github.com/vikejs/vike/issues/1912#issuecomment-2394981475
-            if (id.endsWith('?direct'))
-                id = id.slice(0, -1 * '?direct'.length);
-            const moduleInfo = viteDevServer.moduleGraph.getModuleById(id);
-            (0, utils_js_1.assert)(moduleInfo);
-            const importers = Array.from(moduleInfo.importers)
-                .map((m) => m.id)
-                .filter((id) => id !== null);
-            assertFileEnv(id, (0, isViteServerSide_js_1.isViteServerSide_extraSafe)(config, this.environment, options), importers, 
-            // In dev, we only show a warning because we don't want to disrupt when the user plays with settings such as [ssr](https://vike.dev/ssr).
-            true);
+        load: {
+            handler(id, options) {
+                // In build, we use generateBundle() instead of the load() hook. Using load() works for dynamic imports in dev thanks to Vite's lazy transpiling, but it doesn't work in build because Rollup transpiles any dynamically imported module even if it's never actually imported.
+                if (!viteDevServer)
+                    return;
+                if (!(0, resolveVikeConfigInternal_js_1.isV1Design)())
+                    return;
+                if (skip(id))
+                    return;
+                // For `.vue` files: https://github.com/vikejs/vike/issues/1912#issuecomment-2394981475
+                if (id.endsWith('?direct'))
+                    id = id.slice(0, -1 * '?direct'.length);
+                const moduleInfo = viteDevServer.moduleGraph.getModuleById(id);
+                (0, utils_js_1.assert)(moduleInfo);
+                const importers = Array.from(moduleInfo.importers)
+                    .map((m) => m.id)
+                    .filter((id) => id !== null);
+                assertFileEnv(id, (0, isViteServerSide_js_1.isViteServerSide_extraSafe)(config, this.environment, options), importers, 
+                // In dev, we only show a warning because we don't want to disrupt when the user plays with settings such as [ssr](https://vike.dev/ssr).
+                true);
+            },
         },
         // In production, we have to use transform() to replace modules with a runtime error because generateBundle() doesn't work for dynamic imports. In production, dynamic imports can only be verified at runtime.
-        async transform(code, id, options) {
-            id = (0, normalizeId_js_1.normalizeId)(id);
-            // In dev, only using load() is enough as it also works for dynamic imports (see sibling comment).
-            if (viteDevServer)
-                return;
-            if (skip(id))
-                return;
-            const isServerSide = (0, isViteServerSide_js_1.isViteServerSide_extraSafe)(config, this.environment, options);
-            if (!isWrongEnv(id, isServerSide))
-                return;
-            const { importers } = this.getModuleInfo(id);
-            // Throwing a verbose error doesn't waste client-side KBs as dynamic imports are code split.
-            const errMsg = getErrorMessage(id, isServerSide, importers, false, true);
-            // We have to inject empty exports to avoid Rollup complaining about missing exports, see https://gist.github.com/brillout/5ea45776e65bd65100a52ecd7bfda3ff
-            const { exportNames } = await (0, parseEsModule_js_1.getExportNames)(code);
-            return (0, utils_js_1.rollupSourceMapRemove)([
-                `throw new Error(${JSON.stringify(errMsg)});`,
-                ...exportNames.map((name) => name === 'default' ? 'export default undefined;' : `export const ${name} = undefined;`),
-            ].join('\n'));
-        },
-        generateBundle() {
-            Array.from(this.getModuleIds())
-                .filter((id) => !skip(id))
-                .forEach((moduleId) => {
-                const mod = this.getModuleInfo(moduleId);
-                const { importers } = mod;
-                if (importers.length === 0) {
-                    // Dynamic imports can only be verified at runtime
-                    /* This assertion can fail: https://github.com/vikejs/vike/issues/2227
-                    assert(dynamicImporters.length > 0)
-                    */
+        transform: {
+            async handler(code, id, options) {
+                id = (0, normalizeId_js_1.normalizeId)(id);
+                // In dev, only using load() is enough as it also works for dynamic imports (see sibling comment).
+                if (viteDevServer)
                     return;
-                }
-                assertFileEnv(moduleId, (0, isViteServerSide_js_1.isViteServerSide)(config, this.environment), importers, false);
-            });
+                if (skip(id))
+                    return;
+                const isServerSide = (0, isViteServerSide_js_1.isViteServerSide_extraSafe)(config, this.environment, options);
+                if (!isWrongEnv(id, isServerSide))
+                    return;
+                const { importers } = this.getModuleInfo(id);
+                // Throwing a verbose error doesn't waste client-side KBs as dynamic imports are code split.
+                const errMsg = getErrorMessage(id, isServerSide, importers, false, true);
+                // We have to inject empty exports to avoid Rollup complaining about missing exports, see https://gist.github.com/brillout/5ea45776e65bd65100a52ecd7bfda3ff
+                const { exportNames } = await (0, parseEsModule_js_1.getExportNames)(code);
+                return (0, utils_js_1.rollupSourceMapRemove)([
+                    `throw new Error(${JSON.stringify(errMsg)});`,
+                    ...exportNames.map((name) => name === 'default' ? 'export default undefined;' : `export const ${name} = undefined;`),
+                ].join('\n'));
+            },
+        },
+        generateBundle: {
+            handler() {
+                Array.from(this.getModuleIds())
+                    .filter((id) => !skip(id))
+                    .forEach((moduleId) => {
+                    const mod = this.getModuleInfo(moduleId);
+                    const { importers } = mod;
+                    if (importers.length === 0) {
+                        // Dynamic imports can only be verified at runtime
+                        /* This assertion can fail: https://github.com/vikejs/vike/issues/2227
+                      assert(dynamicImporters.length > 0)
+                      */
+                        return;
+                    }
+                    assertFileEnv(moduleId, (0, isViteServerSide_js_1.isViteServerSide)(config, this.environment), importers, false);
+                });
+            },
         },
-        configResolved(config_) {
-            config = config_;
+        configResolved: {
+            handler(config_) {
+                config = config_;
+            },
         },
-        configureServer(viteDevServer_) {
-            viteDevServer = viteDevServer_;
+        configureServer: {
+            handler(viteDevServer_) {
+                viteDevServer = viteDevServer_;
+            },
         },
     };
     function assertFileEnv(moduleId, isServerSide, importers, onlyWarn) {

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

@@ -25,27 +25,34 @@ function pluginNonRunnableDev() {
     let config;
     return {
         name: 'vike:pluginNonRunnableDev',
-        configureServer(viteDevServer) {
-            (0, utils_js_1.createViteRPC)(viteDevServer, getViteRpcFunctions);
+        configureServer: {
+            handler(viteDevServer) {
+                (0, utils_js_1.createViteRPC)(viteDevServer, getViteRpcFunctions);
+            },
         },
-        configResolved(config_) {
-            config = config_;
+        configResolved: {
+            handler(config_) {
+                config = config_;
+            },
         },
-        transform(code, id) {
-            if (!config._isDev)
-                return;
-            if (id !== distFileIsNonRunnableDev && id !== distFileGlobalContext)
-                return;
-            if ((0, utils_js_1.isRunnableDevEnvironment)(this.environment))
-                return;
-            const { magicString, getMagicStringResult } = (0, getMagicString_js_1.getMagicString)(code, id);
-            if (id === distFileIsNonRunnableDev) {
-                magicString.replaceAll('__VIKE__IS_NON_RUNNABLE_DEV', JSON.stringify(true));
-            }
-            if (id === distFileGlobalContext) {
-                magicString.replaceAll('__VIKE__DYNAMIC_IMPORT', 'import');
-            }
-            return getMagicStringResult();
+        transform: {
+            handler(code, id) {
+                if (!config._isDev)
+                    return;
+                const idWithoutQuery = id.split('?')[0];
+                if (idWithoutQuery !== distFileIsNonRunnableDev && idWithoutQuery !== distFileGlobalContext)
+                    return;
+                if ((0, utils_js_1.isRunnableDevEnvironment)(this.environment))
+                    return;
+                const { magicString, getMagicStringResult } = (0, getMagicString_js_1.getMagicString)(code, id);
+                if (idWithoutQuery === distFileIsNonRunnableDev) {
+                    magicString.replaceAll('__VIKE__IS_NON_RUNNABLE_DEV', JSON.stringify(true));
+                }
+                if (idWithoutQuery === distFileGlobalContext) {
+                    magicString.replaceAll('__VIKE__DYNAMIC_IMPORT', 'import');
+                }
+                return getMagicStringResult();
+            },
         },
     };
 }