vike 0.4.161 → 0.4.162-commit-8995471

package/dist/esm/node/plugin/plugins/importUserCode/getVirtualFileImportUserCode.js

@@ -1,10 +1,11 @@
 export { getVirtualFileImportUserCode };
-import { assert, assertPosixPath, viteIsSSR_options, isNotNullish, scriptFileExtensions, debugGlob, getOutDirs } from '../../utils.js';
+import { assert, assertPosixPath, viteIsSSR_options, scriptFileExtensions, debugGlob, getOutDirs, isVersionOrAbove, assertWarning } from '../../utils.js';
 import { isVirtualFileIdImportUserCode } from '../../../shared/virtual-files/virtualFileImportUserCode.js';
-import { fileTypes, determineFileType } from '../../../../shared/getPageFiles/fileTypes.js';
+import { version as viteVersion } from 'vite';
+import { fileTypes } from '../../../../shared/getPageFiles/fileTypes.js';
 import path from 'path';
 import { getVirtualFilePageConfigs } from './v1-design/getVirtualFilePageConfigs.js';
-import { generateEagerImport } from './generateEagerImport.js';
+import { isV1Design as isV1Design_ } from './v1-design/getVikeConfig.js';
 async function getVirtualFileImportUserCode(id, options, configVike, config, isDev) {
     const idParsed = isVirtualFileIdImportUserCode(id);
     assert(idParsed);
@@ -21,53 +22,13 @@ async function getCode(config, configVike, isForClientSide, isClientRouting, isP
     assert(isDev === !isBuild);
     let content = '';
     {
-        const globRoots = getGlobRoots(config, configVike);
+        const globRoots = getGlobRoots(config);
         debugGlob('Glob roots: ', globRoots);
         content += await generateGlobImports(globRoots, isBuild, isForClientSide, isClientRouting, configVike, isPrerendering, config, isDev, id);
     }
-    {
-        const extensionsImportPaths = configVike.extensions
-            .map(({ pageConfigsDistFiles }) => pageConfigsDistFiles)
-            .flat()
-            .filter(isNotNullish)
-            .map(({ importPath }) => importPath);
-        content += generateExtensionImports(extensionsImportPaths, isForClientSide, isBuild, isClientRouting, isPrerendering);
-    }
     debugGlob(`Glob imports for ${isForClientSide ? 'client' : 'server'}:\n`, content);
     return content;
 }
-function generateExtensionImports(extensionsImportPaths, isForClientSide, isBuild, isClientRouting, isPrerendering) {
-    let fileContent = '\n\n';
-    extensionsImportPaths
-        .filter((importPath) => {
-        assert(
-        // V1 design
-        importPath.includes('+') ||
-            // V0.4 design
-            importPath.includes('.page.'));
-        return !importPath.includes('+');
-    })
-        .forEach((importPath) => {
-        const fileType = determineFileType(importPath);
-        const { includeImport, includeExportNames } = determineInjection({
-            fileType,
-            isForClientSide,
-            isClientRouting,
-            isPrerendering,
-            isBuild
-        });
-        if (includeImport) {
-            fileContent += addImport(importPath, fileType, false, isBuild);
-        }
-        if (includeExportNames) {
-            fileContent += addImport(importPath, fileType, true, isBuild);
-        }
-        if (!includeImport && !includeExportNames && !isForClientSide) {
-            fileContent += `pageFilesList.push("${importPath}");` + '\n';
-        }
-    });
-    return fileContent;
-}
 function determineInjection({ fileType, isForClientSide, isClientRouting, isPrerendering, isBuild }) {
     if (!isForClientSide) {
         return {
@@ -93,42 +54,6 @@ function determineInjection({ fileType, isForClientSide, isClientRouting, isPrer
         }
     }
 }
-function addImport(importPath, fileType, exportNames, isBuild) {
-    const pageFilesVar = (() => {
-        if (exportNames) {
-            if (isBuild) {
-                return 'pageFilesExportNamesEager';
-            }
-            else {
-                return 'pageFilesExportNamesLazy';
-            }
-        }
-        else {
-            if (fileType === '.page.route') {
-                return 'pageFilesEager';
-            }
-            else {
-                return 'pageFilesLazy';
-            }
-        }
-    })();
-    const query = !exportNames ? '' : '?extractExportNames';
-    let fileContent = '';
-    const mapVar = `${pageFilesVar}['${fileType}']`;
-    fileContent += `${mapVar} = ${mapVar} ?? {};\n`;
-    const value = (() => {
-        if (!pageFilesVar.endsWith('Eager')) {
-            return `() => import('${importPath}${query}')`;
-        }
-        else {
-            const { importName, importStatement } = generateEagerImport(`${importPath}${query}`);
-            fileContent += importStatement + '\n';
-            return importName;
-        }
-    })();
-    fileContent += `${mapVar}['${importPath}'] = ${value};\n`;
-    return fileContent;
-}
 async function generateGlobImports(globRoots, isBuild, isForClientSide, isClientRouting, configVike, isPrerendering, config, isDev, id) {
     let fileContent = `// Generatead by node/plugin/plugins/virtualFiles/index.ts
 
@@ -143,6 +68,8 @@ export const isGeneratedFile = true;
 ${await getVirtualFilePageConfigs(isForClientSide, isDev, id, isClientRouting, config)}
 
 `;
+    // We still use import.meta.glob() when using th V1 design in order to not break the V1 design deprecation warning
+    const isV1Design = await isV1Design_(config, isDev);
     fileTypes
         .filter((fileType) => fileType !== '.css')
         .forEach((fileType) => {
@@ -155,18 +82,18 @@ ${await getVirtualFilePageConfigs(isForClientSide, isDev, id, isClientRouting, c
             isBuild
         });
         if (includeImport) {
-            fileContent += getGlobs(globRoots, isBuild, fileType);
+            fileContent += getGlobs(globRoots, isBuild, fileType, null, isV1Design);
         }
         if (includeExportNames) {
-            fileContent += getGlobs(globRoots, isBuild, fileType, 'extractExportNames');
+            fileContent += getGlobs(globRoots, isBuild, fileType, 'extractExportNames', isV1Design);
         }
     });
     if (configVike.includeAssetsImportedByServer && isForClientSide) {
-        fileContent += getGlobs(globRoots, isBuild, '.page.server', 'extractAssets');
+        fileContent += getGlobs(globRoots, isBuild, '.page.server', 'extractAssets', isV1Design);
     }
     return fileContent;
 }
-function getGlobs(globRoots, isBuild, fileType, query) {
+function getGlobs(globRoots, isBuild, fileType, query, isV1Design) {
     const isEager = isBuild && (query === 'extractExportNames' || fileType === '.page.route');
     let pageFilesVar;
     if (query === 'extractExportNames') {
@@ -206,10 +133,26 @@ function getGlobs(globRoots, isBuild, fileType, query) {
             varNameLocals.push(varNameLocal);
             const globIncludePath = `'${getGlobPath(globRoot.includeDir, fileType)}'`;
             const globExcludePath = globRoot.excludeDir ? `'!${getGlobPath(globRoot.excludeDir, fileType)}'` : null;
-            const globOptions = JSON.stringify({ eager: isEager, as: query });
-            assert(globOptions.startsWith('{"eager":true') || globOptions.startsWith('{"eager":false'));
+            const globOptions = { eager: isEager };
+            if (query) {
+                const isNewViteInterface = isVersionOrAbove(viteVersion, '5.1.0');
+                if (isNewViteInterface &&
+                    // When used for the old design, the new syntax breaks Vike's CI (surprinsigly so). I couldn't reproduce locally (I didn't dig much).
+                    isV1Design) {
+                    globOptions.query = `?${query}`;
+                }
+                else {
+                    globOptions.as = query;
+                    const msg = [
+                        "Update to the new V1 design to get rid of Vite's warning:",
+                        'The glob option "as" has been deprecated in favour of "query".',
+                        'See https://vike.dev/migration/v1-design for how to migrate.'
+                    ].join(' ');
+                    assertWarning(!isNewViteInterface, msg, { onlyOnce: true });
+                }
+            }
             const globPaths = globExcludePath ? `[${globIncludePath}, ${globExcludePath}]` : `[${globIncludePath}]`;
-            const globLine = `const ${varNameLocal} = import.meta.glob(${globPaths}, ${globOptions});`;
+            const globLine = `const ${varNameLocal} = import.meta.glob(${globPaths}, ${JSON.stringify(globOptions)});`;
             return globLine;
         }),
         `const ${varName} = {${varNameLocals.map((varNameLocal) => `...${varNameLocal}`).join(',')}};`,
@@ -217,22 +160,13 @@ function getGlobs(globRoots, isBuild, fileType, query) {
         ''
     ].join('\n');
 }
-function getGlobRoots(config, configVike) {
+function getGlobRoots(config) {
     const globRoots = [
         {
             includeDir: '/',
             excludeDir: path.posix.relative(config.root, getOutDirs(config).outDirRoot)
         }
     ];
-    configVike.extensions
-        .map(({ pageConfigsSrcDir }) => pageConfigsSrcDir)
-        .filter(isNotNullish)
-        .forEach((pageConfigsSrcDir) => {
-        const globRoot = {
-            includeDir: path.posix.relative(config.root, pageConfigsSrcDir)
-        };
-        globRoots.push(globRoot);
-    });
     return globRoots;
 }
 function getGlobPath(globRootDir, fileType) {

package/dist/esm/node/plugin/plugins/importUserCode/index.js

@@ -33,7 +33,7 @@ function importUserCode() {
         },
         handleHotUpdate(ctx) {
             try {
-                return handleHotUpdate(ctx, config, configVike);
+                return handleHotUpdate(ctx, config);
             }
             catch (err) {
                 // Vite swallows errors thrown by handleHotUpdate()
@@ -57,11 +57,11 @@ function importUserCode() {
         },
         configureServer(server) {
             isDev1_onConfigureServer();
-            handleFileAddRemove(server, config, configVike);
+            handleFileAddRemove(server, config);
         }
     };
 }
-function handleFileAddRemove(server, config, configVike) {
+function handleFileAddRemove(server, config) {
     server.watcher.prependListener('add', (f) => listener(f, false));
     server.watcher.prependListener('unlink', (f) => listener(f, true));
     return;
@@ -73,11 +73,11 @@ function handleFileAddRemove(server, config, configVike) {
             virtualModules.forEach((mod) => {
                 server.moduleGraph.invalidateModule(mod);
             });
-            reloadConfig(file, config, configVike, isRemove ? 'removed' : 'created');
+            reloadConfig(file, config, isRemove ? 'removed' : 'created');
         }
     }
 }
-function handleHotUpdate(ctx, config, configVike) {
+function handleHotUpdate(ctx, config) {
     const { file, server } = ctx;
     assertPosixPath(file);
     vikeConfigDependencies.forEach((f) => assertPosixPath(f));
@@ -108,7 +108,7 @@ function handleHotUpdate(ctx, config, configVike) {
         /* Tailwind breaks this assertion, see https://github.com/vikejs/vike/discussions/1330#discussioncomment-7787238
         assert(!isViteModule)
         */
-        reloadConfig(file, config, configVike, 'modified');
+        reloadConfig(file, config, 'modified');
         const virtualModules = getVirtualModules(server);
         return virtualModules;
     }
@@ -116,13 +116,13 @@ function handleHotUpdate(ctx, config, configVike) {
 function isVikeConfigModule(filePathAbsoluteFilesystem) {
     return vikeConfigDependencies.has(filePathAbsoluteFilesystem);
 }
-function reloadConfig(filePath, config, configVike, op) {
+function reloadConfig(filePath, config, op) {
     {
         const filePathToShowToUser = pc.dim(getFilePathRelativeToUserRootDir(filePath, config.root, true));
         const msg = `${op} ${filePathToShowToUser}`;
         logConfigInfo(msg, 'info');
     }
-    reloadVikeConfig(config.root, getOutDirs(config).outDirRoot, configVike.extensions);
+    reloadVikeConfig(config.root, getOutDirs(config).outDirRoot);
 }
 function getVirtualModules(server) {
     const virtualModules = Array.from(server.moduleGraph.urlToModuleMap.keys())

package/dist/esm/node/plugin/plugins/importUserCode/v1-design/getConfigValuesSerialized.js

@@ -60,15 +60,7 @@ function getConfigValueSerialized(value, configName, definedAt) {
         */
         const configValueFilePathToShowToUser = getConfigValueFilePathToShowToUser({ definedAt });
         assert(configValueFilePathToShowToUser);
-        assertUsage(false, [
-            `The code of ${pc.cyan(configName)} cannot live inside ${configValueFilePathToShowToUser},`,
-            'see https://vike.dev/header-file#runtime-code'
-            /* I guess showing this is more confusing than adding value.
-            `(technically speaking: the value of ${pc.cyan(
-              configName
-            )} isn't serializable (${serializationErrMsg}) and it's therefore runtime code that needs to be imported).`
-            //*/
-        ].join(' '));
+        assertUsage(false, `${pc.cyan(configName)} defined by ${configValueFilePathToShowToUser} must be defined over a so-called "pointer import", see https://vike.dev/config#pointer-imports`);
     }
     configValueSerialized = JSON.stringify(configValueSerialized);
     return configValueSerialized;

package/dist/esm/node/plugin/plugins/importUserCode/v1-design/getVikeConfig/configDefinitionsBuiltIn.js

@@ -88,6 +88,9 @@ const configDefinitionsBuiltIn = {
     },
     cacheControl: {
         env: { server: true }
+    },
+    name: {
+        env: { config: true }
     }
 };
 const configDefinitionsBuiltInGlobal = {

package/dist/esm/node/plugin/plugins/importUserCode/v1-design/getVikeConfig/filesystemRouting.d.ts

@@ -13,18 +13,18 @@ export { getLogicalPath };
  * Each config value is assigned with a `locationId` value. That's the source-of-truth for determining inheritance between config values.
  *
  * For Vike extensions, `locationId` is different than the config value's `definedAt`, for example the `onRenderHtml()` hook of `vike-react`:
- *  - `locationId === '/pages'` (the directory of `/pages/+config.h.js` which extends `vike-react`)
+ *  - `locationId === '/pages'` (the directory of `/pages/+config.js` which extends `vike-react`)
  *  - `definedAt.filePathAbsoluteFilesystem === '/home/rom/code/my-vike-app/node_modules/vike-react/dist/renderer/onRenderHtml.js'` (the file where the value is defined)
  *
  *  This is an important distinction because the Vike extension's config should only apply to where it's being extended from, for example:
  *  ```js
- *  // /pages/admin/+config.h.js
+ *  // /pages/admin/+config.js
  *  import vikeVue from 'vike-vue/config'
  *  // Should only apply to /pages/admin/**
  *  export default { extends: [vikeVue] }
  *  ```
  *  ```js
- *  // /pages/marketing/+config.h.js
+ *  // /pages/marketing/+config.js
  *  import vikeReact from 'vike-react/config'
  *  // Should only apply to /pages/marketing/**
  *  export default { extends: [vikeReact] }

package/dist/esm/node/plugin/plugins/importUserCode/v1-design/getVikeConfig/filesystemRouting.js

@@ -15,7 +15,7 @@ import { assert, assertPosixPath, higherFirst } from '../../../../utils.js';
  * The value `locationId` is always a user-land path, because Filesystem Routing/Inheritance only applies to the user-land (Vike never uses Filesystem Routing/Inheritance for `node_modules/**`).
  */
 function getLocationId(
-// We always determine `locationId` from a real user-land file: the `locationId` for Vike extensions is the `locationId` of the the user's `+config.h.js` that extends the Vike extension.
+// We always determine `locationId` from a real user-land file: the `locationId` for Vike extensions is the `locationId` of the the user's `+config.js` that extends the Vike extension.
 filePathRelativeToUserRootDir) {
     assertPosixPath(filePathRelativeToUserRootDir);
     assert(filePathRelativeToUserRootDir.startsWith('/'));

package/dist/esm/node/plugin/plugins/importUserCode/v1-design/getVikeConfig/loadFileAtConfigTime.d.ts

@@ -15,7 +15,7 @@ declare function loadImportedFile(import_: FilePathResolved & {
     fileExportName: string;
 }, userRootDir: string, importedFilesLoaded: ImportedFilesLoaded): Promise<unknown>;
 declare function loadValueFile(interfaceValueFile: InterfaceValueFile, configName: string, userRootDir: string): Promise<void>;
-declare function loadConfigFile(configFilePath: FilePathResolved, userRootDir: string, visited: string[], isConfigOfExtension: boolean): Promise<{
+declare function loadConfigFile(configFilePath: FilePathResolved, userRootDir: string, visited: string[], isExtensionConfig: boolean): Promise<{
     configFile: ConfigFile;
     extendsConfigs: ConfigFile[];
 }>;

package/dist/esm/node/plugin/plugins/importUserCode/v1-design/getVikeConfig/loadFileAtConfigTime.js

@@ -14,7 +14,7 @@ assertIsNotProductionRuntime();
 async function loadImportedFile(import_, userRootDir, importedFilesLoaded) {
     const f = import_.filePathAbsoluteFilesystem;
     if (!importedFilesLoaded[f]) {
-        importedFilesLoaded[f] = transpileAndExecuteFile(import_, false, userRootDir).then((r) => r.fileExports);
+        importedFilesLoaded[f] = transpileAndExecuteFile(import_, userRootDir, false).then((r) => r.fileExports);
     }
     const fileExports = await importedFilesLoaded[f];
     const fileExport = fileExports[import_.fileExportName];
@@ -22,7 +22,7 @@ async function loadImportedFile(import_, userRootDir, importedFilesLoaded) {
 }
 // Load +{configName}.js
 async function loadValueFile(interfaceValueFile, configName, userRootDir) {
-    const { fileExports } = await transpileAndExecuteFile(interfaceValueFile.filePath, false, userRootDir);
+    const { fileExports } = await transpileAndExecuteFile(interfaceValueFile.filePath, userRootDir, false);
     const { filePathToShowToUser } = interfaceValueFile.filePath;
     assertPlusFileExport(fileExports, filePathToShowToUser, configName);
     Object.entries(fileExports).forEach(([exportName, configValue]) => {
@@ -31,10 +31,10 @@ async function loadValueFile(interfaceValueFile, configName, userRootDir) {
     });
 }
 // Load +config.js, including all its extends fake imports
-async function loadConfigFile(configFilePath, userRootDir, visited, isConfigOfExtension) {
+async function loadConfigFile(configFilePath, userRootDir, visited, isExtensionConfig) {
     const { filePathAbsoluteFilesystem } = configFilePath;
     assertNoInfiniteLoop(visited, filePathAbsoluteFilesystem);
-    const { fileExports } = await transpileAndExecuteFile(configFilePath, !isConfigOfExtension, userRootDir, isConfigOfExtension);
+    const { fileExports } = await transpileAndExecuteFile(configFilePath, userRootDir, isExtensionConfig ? 'is-extension-config' : true);
     const { extendsConfigs, extendsFilePaths } = await loadExtendsConfigs(fileExports, configFilePath, userRootDir, [
         ...visited,
         filePathAbsoluteFilesystem

package/dist/esm/node/plugin/plugins/importUserCode/v1-design/getVikeConfig/resolveImportPath.js

@@ -115,7 +115,7 @@ function assertFileEnv(filePathForEnvCheck, configEnv, configName) {
         assertUsage(false, [
             `${filePathForEnvCheck} defines the value of configs living in different environments:`,
             ...[configDifferentEnv, { configName, configEnv }].map((c) => `  - config ${pc.cyan(c.configName)} which value lives in environment ${pc.cyan(JSON.stringify(c.configEnv))}`),
-            'Defining config values in the same file is allowed only if they live in the same environment, see https://vike.dev/header-file'
+            'Defining config values in the same file is allowed only if they live in the same environment, see https://vike.dev/config#pointer-imports'
         ].join('\n'));
     }
 }

package/dist/esm/node/plugin/plugins/importUserCode/v1-design/getVikeConfig/transformFileImports.d.ts

@@ -1,20 +1,8 @@
 export { transformFileImports };
 export { parseImportData };
 export { isImportData };
-export type { FileImport };
 export type { ImportData };
-type FileImport = {
-    importStatementCode: string;
-    importString: string;
-    importLocalName: string;
-};
-declare function transformFileImports(code: string, filePathToShowToUser2: string, skipWarnings?: true): {
-    noTransformation: true;
-} | {
-    noTransformation: false;
-    code: string;
-    fileImportsTransformed: FileImport[];
-};
+declare function transformFileImports(code: string, filePathToShowToUser2: string, pointerImports: 'all' | Record<string, boolean>, skipWarnings?: true): string | null;
 /**
  * Data Structure holding info about import statement:
  *   `import { someExport as someImport } from './some-file'`

package/dist/esm/node/plugin/plugins/importUserCode/v1-design/getVikeConfig/transformFileImports.js

@@ -2,52 +2,55 @@ export { transformFileImports };
 export { parseImportData };
 export { isImportData };
 // Playground: https://github.com/brillout/acorn-playground
-// Import attributes support: https://github.com/acornjs/acorn/issues/983
-//  - Isn't stage 4 yet: https://github.com/tc39/proposal-import-attributes
+// Notes about `with { type: 'pointer' }`
+// - It works well with TypeScript: it doesn't complain upon `with { type: 'unknown-to-typescript' }` and go-to-definition & types are preserved: https://www.typescriptlang.org/docs/handbook/release-notes/typescript-5-3.html#import-attributes
+// - Acorn support for import attributes: https://github.com/acornjs/acorn/issues/983
+//   - Acorn plugin: https://github.com/acornjs/acorn/issues/983
+//   - Isn't stage 4 yet: https://github.com/tc39/proposal-import-attributes
+// - Using a import path suffix such as `import { Layout } from './Layout?real` breaks TypeScript, and TypeScript isn't working on supporting query params: https://github.com/microsoft/TypeScript/issues/10988#issuecomment-867135453
+// - Node.js >=21 supports import attribtues: https://nodejs.org/api/esm.html#import-attributes
+// - Esbuid supports
+//   - Blocker: https://github.com/evanw/esbuild/issues/3646
+//     - Ugly hack to make it work: https://github.com/brillout/esbuild-playground/tree/experiment/import-attribute
+//   - Discussion with esbuild maintainer: https://github.com/evanw/esbuild/issues/3384
+// - Using a magic comment `// @vike-real-import` is probably a bad idea:
+//   - Esbuild removes comments: https://github.com/evanw/esbuild/issues/1439#issuecomment-877656182
+//   - Using source maps to track these magic comments is brittle (source maps can easily break)
 import { parse } from 'acorn';
 import { assert, assertUsage, assertWarning, styleFileRE } from '../../../../utils.js';
 import pc from '@brillout/picocolors';
-function transformFileImports(code, filePathToShowToUser2, 
+// TODO: rename transformFileImports() => transformPointerImports()
+function transformFileImports(code, filePathToShowToUser2, pointerImports, 
 // For ./transformFileImports.spec.ts
 skipWarnings) {
     const spliceOperations = [];
-    const fileImportsTransformed = [];
     // Performance trick
     if (!code.includes('import'))
-        return { noTransformation: true };
+        return null;
     const imports = getImports(code);
     if (imports.length === 0)
-        return { noTransformation: true };
+        return null;
     imports.forEach((node) => {
         if (node.type !== 'ImportDeclaration')
             return;
         const importPath = node.source.value;
         assert(typeof importPath === 'string');
-        // - This doesn't work. To make it work we would need to run esbuild twice: esbuild for TypeScript to JavaScript => transformFileImports() => esbuild for bundling.
-        //   - Or we use an esbuild plugin to apply transformFileImports(). Maybe we can completely skip the need for acorn?
-        // - ?real breaks TypeScript, and TypeScript isn't working on supporting query params: https://github.com/microsoft/TypeScript/issues/10988#issuecomment-867135453
-        // - Import attributes would be the best.
-        //   - But it only works with Node.js >=21: https://nodejs.org/api/esm.html#import-attributes
-        //     - But it's probably ok to tell users "to use real imports you need Node.js 21 or above".
-        //   - It works well with TypeScript: it doesn't complain upon `with { type: 'unknown-to-typescript' }` and go-to-definition & types are preserved: https://www.typescriptlang.org/docs/handbook/release-notes/typescript-5-3.html#import-attributes
-        //   - Esbuid seems to support it: https://esbuild.github.io/plugins/#on-load-arguments:~:text=This%20contains%20a%20map%20of%20the%20import%20attributes%20that
-        //   - acorn supports it over an acorn plugin: https://github.com/acornjs/acorn/issues/983
-        //     - Maybe we can use an esbuild plugin instead of acorn to apply transformFileImports()?
-        // - Using a magic comment `// @vike-real-import` is tricky:
-        //   - Esbuild removes comments: https://github.com/evanw/esbuild/issues/1439#issuecomment-877656182
-        //   - Using source maps to track these magic comments is brittle (source maps can easily break)
-        if (importPath.endsWith('?real')) {
-            const { start, end } = node.source;
-            spliceOperations.push({
-                start,
-                end,
-                replacement: importPath.slice(0, -1 * '?real'.length)
-            });
-            return;
+        if (pointerImports !== 'all') {
+            assert(importPath in pointerImports);
+            const isPointerImport = pointerImports[importPath];
+            assert(isPointerImport === true || isPointerImport === false);
+            if (!isPointerImport)
+                return;
         }
         const { start, end } = node;
         const importStatementCode = code.slice(start, end);
-        // No variable imported
+        /* Pointer import without importing any value => doesn't make sense and doesn't have any effect.
+        ```js
+        // Useless
+        import './some.css'
+        // Useless
+        import './Layout.jsx'
+        ``` */
         if (node.specifiers.length === 0) {
             const isWarning = !styleFileRE.test(importPath);
             let quote = indent(importStatementCode);
@@ -58,8 +61,9 @@ skipWarnings) {
                 quote = pc.bold(pc.red(quote));
             }
             const errMsg = [
-                `As explained in https://vike.dev/header-file the following import in ${filePathToShowToUser2} has no effect:`,
-                quote
+                `The following import in ${filePathToShowToUser2} has no effect:`,
+                quote,
+                'See https://vike.dev/config#pointer-imports'
             ].join('\n');
             if (!skipWarnings) {
                 if (!isWarning) {
@@ -88,11 +92,6 @@ skipWarnings) {
             })();
             const importString = serializeImportData({ importPath, exportName, importStringWasGenerated: true });
             replacement += `const ${importLocalName} = '${importString}';`;
-            fileImportsTransformed.push({
-                importStatementCode,
-                importString,
-                importLocalName
-            });
         });
         spliceOperations.push({
             start,
@@ -101,7 +100,7 @@ skipWarnings) {
         });
     });
     const codeMod = spliceMany(code, spliceOperations);
-    return { code: codeMod, fileImportsTransformed, noTransformation: false };
+    return codeMod;
 }
 function getImports(code) {
     const { body } = parse(code, {

package/dist/esm/node/plugin/plugins/importUserCode/v1-design/getVikeConfig/transpileAndExecuteFile.d.ts

@@ -4,7 +4,7 @@ export { getConfigExecutionErrorIntroMsg };
 export { isTmpFile };
 import 'source-map-support/register.js';
 import type { FilePathResolved } from '../../../../../../shared/page-configs/PageConfig.js';
-declare function transpileAndExecuteFile(filePath: FilePathResolved, transformImports: boolean, userRootDir: string, doNotTranspile?: boolean): Promise<{
+declare function transpileAndExecuteFile(filePath: FilePathResolved, userRootDir: string, isConfigFile: boolean | 'is-extension-config'): Promise<{
     fileExports: Record<string, unknown>;
 }>;
 declare function getConfigBuildErrorFormatted(err: unknown): null | string;