vike 0.4.238-commit-d48a597 → 0.4.239-commit-050a4a3

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

@@ -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' };
+                }
+            },
         },
     };
 }