vike 0.4.167 → 0.4.168-commit-7678a7d

package/dist/cjs/node/plugin/plugins/buildConfig.js

@@ -232,7 +232,15 @@ function getEntryFromPageConfig(pageConfig, isForClientSide) {
     let { pageId } = pageConfig;
     const entryTarget = (0, virtualFilePageConfigValuesAll_js_1.getVirtualFileIdPageConfigValuesAll)(pageId, isForClientSide);
     let entryName = pageId;
+    // Avoid:
+    // ```
+    // dist/client/assets/entries/.Dp9wM6PK.js
+    // dist/server/entries/.mjs
+    // ```
+    if (entryName === '/')
+        entryName = 'root';
     entryName = (0, prependEntriesDir_js_1.prependEntriesDir)(entryName);
+    (0, utils_js_1.assert)(!entryName.endsWith('/'));
     return { entryName, entryTarget };
 }
 function resolve(filePath) {

package/dist/cjs/node/plugin/plugins/distFileNames.js

@@ -116,6 +116,12 @@ function clean(name, removePathSep, fixGlob) {
     }
     name = removeLeadingUnderscoreInFilename(name);
     name = removeUnderscoreDoublets(name);
+    // Avoid:
+    // ```
+    // dist/client/assets/entries/.Dp9wM6PK.js
+    // dist/server/entries/.mjs
+    // ```
+    (0, utils_js_1.assert)(!name.endsWith('/'));
     return name;
 }
 function fixExtractAssetsQuery(name) {

package/dist/cjs/node/plugin/plugins/extractAssetsPlugin.js

@@ -189,7 +189,7 @@ function analyzeImport(importStatement) {
     if (rawRE.test(moduleName)) {
         return { moduleName, skip: true };
     }
-    /* We should not do this because of aliased imports
+    /* We shouldn't do this because of aliased imports
     if (!moduleName.startsWith('.')) {
       return { moduleName, skip: true }
     }

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

@@ -18,9 +18,11 @@ const getFilePath_js_1 = require("../../shared/getFilePath.js");
 function importUserCode() {
     let config;
     let configVike;
+    let isDev_;
     return {
         name: 'vike:importUserCode',
-        config() {
+        config(_, env) {
+            isDev_ = (0, utils_js_1.isDev3)(env);
             return {
                 experimental: {
                     // TODO/v1-release: remove
@@ -48,7 +50,7 @@ function importUserCode() {
             }
         },
         async load(id, options) {
-            const isDev = (0, utils_js_1.isDev1)();
+            const isDev = isDev_ !== null ? isDev_ : (0, utils_js_1.isDev1)();
             if (!(0, utils_js_1.isVirtualFileId)(id))
                 return undefined;
             id = (0, utils_js_1.getVirtualFileId)(id);
@@ -62,7 +64,8 @@ function importUserCode() {
             }
         },
         configureServer(server) {
-            (0, utils_js_1.isDev1_onConfigureServer)();
+            if (isDev_ === null)
+                (0, utils_js_1.isDev1_onConfigureServer)();
             handleFileAddRemove(server, config);
         }
     };

package/dist/cjs/node/plugin/plugins/importUserCode/v1-design/getVikeConfig/crawlPlusFiles.js

@@ -10,6 +10,7 @@ const fast_glob_1 = __importDefault(require("fast-glob"));
 const child_process_1 = require("child_process");
 const util_1 = require("util");
 const picocolors_1 = __importDefault(require("@brillout/picocolors"));
+const transpileAndExecuteFile_js_1 = require("./transpileAndExecuteFile.js");
 const execA = (0, util_1.promisify)(child_process_1.exec);
 (0, utils_js_1.assertIsNotProductionRuntime)();
 (0, utils_js_1.assertIsSingleModuleInstance)('crawlPlusFiles.ts');
@@ -22,7 +23,13 @@ async function crawlPlusFiles(userRootDir, outDirAbsoluteFilesystem, isDev) {
         // config.outDir is outside of config.root => it's going to be ignored anyways
         outDirRelativeFromUserRootDir = null;
     }
-    (0, utils_js_1.assert)(outDirRelativeFromUserRootDir === null || !outDirRelativeFromUserRootDir.startsWith('.'));
+    (0, utils_js_1.assert)(outDirRelativeFromUserRootDir === null ||
+        /* Not true if outDirRelativeFromUserRootDir starts with a hidden directory (i.e. a directory with a name that starts with `.`)
+        !outDirRelativeFromUserRootDir.startsWith('.') &&
+        */
+        (!outDirRelativeFromUserRootDir.startsWith('./') &&
+            //
+            !outDirRelativeFromUserRootDir.startsWith('../')));
     const timeBefore = new Date().getTime();
     let files = [];
     const res = await gitLsFiles(userRootDir, outDirRelativeFromUserRootDir);
@@ -34,6 +41,7 @@ async function crawlPlusFiles(userRootDir, outDirAbsoluteFilesystem, isDev) {
     else {
         files = await fastGlob(userRootDir, outDirRelativeFromUserRootDir);
     }
+    files = files.filter((file) => !(0, transpileAndExecuteFile_js_1.isTemporaryBuildFile)(file));
     {
         const timeAfter = new Date().getTime();
         const timeSpent = timeAfter - timeBefore;

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

@@ -68,7 +68,6 @@ async function loadExtendsConfigs(configFileExports, configFilePath, userRootDir
         const { importPath: importPathAbsolute } = importData;
         const filePathAbsoluteFilesystem = (0, resolveImportPath_js_1.resolveImportPath)(importData, configFilePath);
         (0, resolveImportPath_js_1.assertImportPath)(filePathAbsoluteFilesystem, importData, configFilePath);
-        warnUserLandExtension(importPathAbsolute, configFilePath);
         const filePath = (0, getFilePath_js_1.getFilePathResolved)({ filePathAbsoluteFilesystem, userRootDir, importPathAbsolute });
         extendsConfigFiles.push(filePath);
     });
@@ -81,13 +80,6 @@ async function loadExtendsConfigs(configFileExports, configFilePath, userRootDir
     const extendsFilePaths = extendsConfigFiles.map((f) => f.filePathAbsoluteFilesystem);
     return { extendsConfigs, extendsFilePaths };
 }
-function warnUserLandExtension(importPath, configFilePath) {
-    // We preserve this feature because we may need it for eject
-    (0, utils_js_1.assertWarning)((0, utils_js_1.isNpmPackageImport)(importPath, {
-        // Vike config files don't support path aliases. (If they do one day, then Vike will/should be able to resolve path aliases.)
-        cannotBePathAlias: true
-    }) || importPath.includes('/node_modules/'), `${configFilePath.filePathToShowToUser} uses ${picocolors_1.default.cyan('extends')} to inherit from ${picocolors_1.default.cyan(importPath)} which is a user-land file: this is experimental and may be remove at any time. Reach out to a maintainer if you need this.`, { onlyOnce: true });
-}
 function getExtendsImportData(configFileExports, configFilePath) {
     const { filePathToShowToUser } = configFilePath;
     const configFileExport = (0, getConfigFileExport_js_1.getConfigFileExport)(configFileExports, filePathToShowToUser);

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

@@ -22,13 +22,22 @@ function resolveImport(configValue, importerFilePath, userRootDir, configEnv, co
     const fileExportPathToShowToUser = exportName === 'default' || exportName === configName ? [] : [exportName];
     let filePath;
     if (importPath.startsWith('.')) {
-        // We need to resolve relative paths into absolute paths. Because the import paths are included in virtual files:
+        (0, utils_js_1.assert)(importPath.startsWith('./') || importPath.startsWith('../'));
+        assertImportPath(filePathAbsoluteFilesystem, importData, importerFilePath);
+        filePath = (0, getFilePath_js_1.getFilePathResolved)({ filePathAbsoluteFilesystem, userRootDir });
+        // Imports are included in virtual files, thus the relative path of imports need to resolved.
         // ```
         // [vite] Internal server error: Failed to resolve import "./onPageTransitionHooks" from "virtual:vike:pageConfigValuesAll:client:/pages/index". Does the file exist?
         // ```
-        assertImportPath(filePathAbsoluteFilesystem, importData, importerFilePath);
-        const filePathAbsoluteUserRootDir = resolveImportPath_absoluteUserRootDir(filePathAbsoluteFilesystem, importData, importerFilePath, userRootDir);
-        filePath = (0, getFilePath_js_1.getFilePathResolved)({ filePathAbsoluteUserRootDir, userRootDir });
+        (0, utils_js_1.assertUsage)(filePath.filePathAbsoluteUserRootDir, `${importerFilePath.filePathToShowToUser} imports a relative path ${picocolors_1.default.cyan(importPath)} resolving outside of ${userRootDir} which is forbidden: import from a relative path inside ${userRootDir}, or import from a dependency's package.json#exports entry instead`);
+        // Alternativey, we can try one of the following but last time we tried none of the following worked.
+        // /*
+        // assert(filePathAbsoluteFilesystem.startsWith('/'))
+        // filePath = `/@fs${filePathAbsoluteFilesystem}`
+        // /*/
+        // assert(filePathAbsoluteUserRootDir.startsWith('../'))
+        // filePathAbsoluteUserRootDir = '/' + filePathAbsoluteUserRootDir
+        // //*/
     }
     else {
         // importPath can be:
@@ -54,28 +63,6 @@ function resolveImport(configValue, importerFilePath, userRootDir, configEnv, co
     };
 }
 exports.resolveImport = resolveImport;
-function resolveImportPath_absoluteUserRootDir(filePathAbsoluteFilesystem, importData, configFilePath, userRootDir) {
-    (0, utils_js_1.assertPosixPath)(userRootDir);
-    let filePathAbsoluteUserRootDir;
-    if (filePathAbsoluteFilesystem.startsWith(userRootDir)) {
-        filePathAbsoluteUserRootDir = getVitePathFromAbsolutePath(filePathAbsoluteFilesystem, userRootDir);
-    }
-    else {
-        (0, utils_js_1.assertUsage)(false, `${configFilePath.filePathToShowToUser} imports from a relative path ${picocolors_1.default.cyan(importData.importPath)} outside of ${userRootDir} which is forbidden: import from a relative path inside ${userRootDir}, or import from a dependency's package.json#exports entry instead`);
-        // None of the following works. Seems to be a Vite bug?
-        // /*
-        // assert(filePathAbsoluteFilesystem.startsWith('/'))
-        // filePath = `/@fs${filePathAbsoluteFilesystem}`
-        // /*/
-        // filePathAbsoluteUserRootDir = path.posix.relative(userRootDir, filePathAbsoluteFilesystem)
-        // assert(filePathAbsoluteUserRootDir.startsWith('../'))
-        // filePathAbsoluteUserRootDir = '/' + filePathAbsoluteUserRootDir
-        // //*/
-    }
-    (0, utils_js_1.assertPosixPath)(filePathAbsoluteUserRootDir);
-    (0, utils_js_1.assert)(filePathAbsoluteUserRootDir.startsWith('/'));
-    return filePathAbsoluteUserRootDir;
-}
 function resolveImportPath(importData, importerFilePath) {
     const importerFilePathAbsolute = importerFilePath.filePathAbsoluteFilesystem;
     (0, utils_js_1.assertPosixPath)(importerFilePathAbsolute);
@@ -97,6 +84,7 @@ function assertImportPath(filePathAbsoluteFilesystem, importData, importerFilePa
             : `The import ${picocolors_1.default.cyan(importString)} defined in ${filePathToShowToUser}`;
         const errIntro2 = `${errIntro} couldn't be resolved: does ${importPathString}`;
         if (importPath.startsWith('.')) {
+            (0, utils_js_1.assert)(importPath.startsWith('./') || importPath.startsWith('../'));
             (0, utils_js_1.assertUsage)(false, `${errIntro2} point to an existing file?`);
         }
         else {
@@ -125,12 +113,3 @@ function clearFilesEnvMap() {
     filesEnvMap.clear();
 }
 exports.clearFilesEnvMap = clearFilesEnvMap;
-function getVitePathFromAbsolutePath(filePathAbsoluteFilesystem, root) {
-    (0, utils_js_1.assertPosixPath)(filePathAbsoluteFilesystem);
-    (0, utils_js_1.assertPosixPath)(root);
-    (0, utils_js_1.assert)(filePathAbsoluteFilesystem.startsWith(root));
-    let vitePath = path_1.default.posix.relative(root, filePathAbsoluteFilesystem);
-    (0, utils_js_1.assert)(!vitePath.startsWith('/') && !vitePath.startsWith('.'));
-    vitePath = '/' + vitePath;
-    return vitePath;
-}

package/dist/cjs/node/plugin/plugins/importUserCode/v1-design/getVikeConfig/transpileAndExecuteFile.js

@@ -3,7 +3,7 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.isTmpFile = exports.getConfigExecutionErrorIntroMsg = exports.getConfigBuildErrorFormatted = exports.transpileAndExecuteFile = void 0;
+exports.isTemporaryBuildFile = exports.getConfigExecutionErrorIntroMsg = exports.getConfigBuildErrorFormatted = exports.transpileAndExecuteFile = void 0;
 const esbuild_1 = require("esbuild");
 const fs_1 = __importDefault(require("fs"));
 const path_1 = __importDefault(require("path"));
@@ -82,87 +82,96 @@ async function transpileWithEsbuild(filePath, userRootDir, transformImports) {
         // Esbuild still sometimes removes unused imports because of TypeScript: https://github.com/evanw/esbuild/issues/3034
         treeShaking: false,
         minify: false,
-        metafile: transformImports !== 'all',
-        bundle: transformImports !== 'all'
+        metafile: true,
+        bundle: true
     };
-    let pointerImports;
-    if (transformImports === 'all') {
-        pointerImports = 'all';
-        options.packages = 'external';
-    }
-    else {
-        const pointerImports_ = (pointerImports = {});
-        options.plugins = [
-            // Determine what import should be externalized (and then transformed to a pointer/fake import)
-            {
-                name: 'vike:externalize-heuristic',
-                setup(build) {
-                    // https://github.com/evanw/esbuild/issues/3095#issuecomment-1546916366
+    let pointerImports = {};
+    options.plugins = [
+        // Determine whether an import should be:
+        //  - A pointer import
+        //  - Externalized
+        {
+            name: 'vike-esbuild',
+            setup(build) {
+                // https://github.com/brillout/esbuild-playground
+                build.onResolve({ filter: /.*/ }, async (args) => {
+                    if (args.kind !== 'import-statement')
+                        return;
+                    // Avoid infinite loop: https://github.com/evanw/esbuild/issues/3095#issuecomment-1546916366
                     const useEsbuildResolver = 'useEsbuildResolver';
-                    // https://github.com/brillout/esbuild-playground
-                    build.onResolve({ filter: /.*/ }, async (args) => {
-                        if (args.kind !== 'import-statement')
-                            return;
-                        if (args.pluginData?.[useEsbuildResolver])
-                            return;
-                        const { path, ...opts } = args;
-                        opts.pluginData = { [useEsbuildResolver]: true };
-                        const resolved = await build.resolve(path, opts);
-                        resolved.path = (0, utils_js_1.toPosixPath)(resolved.path);
-                        // vike-{react,vue,solid} follow the convention that their config export resolves to a file named +config.js
-                        //  - This is temporary, see comment below.
-                        const isVikeExtensionConfigImport = resolved.path.endsWith('+config.js');
-                        const isPointerImport = 
+                    if (args.pluginData?.[useEsbuildResolver])
+                        return;
+                    const { path, ...opts } = args;
+                    opts.pluginData = { [useEsbuildResolver]: true };
+                    const resolved = await build.resolve(path, opts);
+                    if (resolved.errors.length > 0) {
+                        /* We could do the following to let Node.js throw the error, but we don't because the error shown by esbuild is prettier: the Node.js error refers to the transpiled [build-f7i251e0iwnw]+config.ts.mjs file which isn't that nice, whereas esbuild refers to the source +config.ts file.
+                        pointerImports[args.path] = false
+                        return { external: true }
+                        */
+                        // Let esbuild throw the error. (It throws a nice & pretty error.)
+                        cleanEsbuildErrors(resolved.errors);
+                        return resolved;
+                    }
+                    (0, utils_js_1.assert)(resolved.path);
+                    resolved.path = (0, utils_js_1.toPosixPath)(resolved.path);
+                    // vike-{react,vue,solid} follow the convention that their config export resolves to a file named +config.js
+                    //  - This is temporary, see comment below.
+                    const isVikeExtensionConfigImport = resolved.path.endsWith('+config.js');
+                    const isPointerImport = transformImports === 'all' ||
                         // .jsx, .vue, .svg, ... => obviously not config code
                         !(0, utils_js_1.isJavaScriptFile)(resolved.path) ||
-                            // Import of a Vike extension config => make it a pointer import because we want to show nice error messages (that can display whether a configas been set by the user or by a Vike extension).
-                            //  - We should have Node.js directly load vike-{react,vue,solid} while enforcing Vike extensions to set 'name' in their +config.js file.
-                            //    - vike@0.4.162 already started soft-requiring Vike extensions to set the name config
-                            isVikeExtensionConfigImport ||
-                            // Cannot be resolved by esbuild => take a leap of faith and make it a pointer import.
-                            //  - For example if esbuild cannot resolve a path alias while Vite can.
-                            //    - When tsconfig.js#compilerOptions.paths is set, then esbuild is able to resolve the path alias.
-                            resolved.errors.length > 0;
-                        pointerImports_[resolved.path] = isPointerImport;
-                        (0, utils_js_1.assertPosixPath)(resolved.path);
-                        const isExternal = isPointerImport ||
-                            // npm package imports that aren't pointer imports (e.g. importing a Vite plugin)
-                            resolved.path.includes('/node_modules/');
-                        if (debug.isActivated)
-                            debug('onResolved()', { args, resolved, isPointerImport, isExternal });
-                        if (isExternal) {
-                            return { external: true, path: resolved.path };
-                        }
-                        else {
-                            return resolved;
-                        }
-                    });
-                }
-            },
-            // Track dependencies
-            {
-                name: 'vike:dependency-tracker',
-                setup(b) {
-                    b.onLoad({ filter: /./ }, (args) => {
-                        // We collect the dependency `args.path` in case the bulid fails (upon build error => error is thrown => no metafile)
-                        let { path } = args;
-                        path = (0, utils_js_1.toPosixPath)(path);
-                        getVikeConfig_js_1.vikeConfigDependencies.add(path);
-                        return undefined;
-                    });
-                    /* To exhaustively collect all dependencies upon build failure, we would also need to use onResolve().
-                     *  - Because onLoad() isn't call if the config dependency can't be resolved.
-                     *  - For example, the following breaks auto-reload (the config is stuck in its error state and the user needs to touch the importer for the config to reload):
-                     *    ```bash
-                     *    mv ./some-config-dependency.js /tmp/ && mv /tmp/some-config-dependency.js .
-                     *    ```
-                     *  - But implementing a fix is complex and isn't worth it.
-                    b.onResolve(...)
-                    */
-                }
+                        // Import of a Vike extension config => make it a pointer import because we want to show nice error messages (that can display whether a configas been set by the user or by a Vike extension).
+                        //  - We should have Node.js directly load vike-{react,vue,solid} while enforcing Vike extensions to set 'name' in their +config.js file.
+                        //    - vike@0.4.162 already started soft-requiring Vike extensions to set the name config
+                        isVikeExtensionConfigImport ||
+                        // Cannot be resolved by esbuild => take a leap of faith and make it a pointer import.
+                        //  - For example if esbuild cannot resolve a path alias while Vite can.
+                        //    - When tsconfig.js#compilerOptions.paths is set, then esbuild is able to resolve the path alias.
+                        resolved.errors.length > 0;
+                    pointerImports[resolved.path] = isPointerImport;
+                    (0, utils_js_1.assertPosixPath)(resolved.path);
+                    const isExternal = isPointerImport ||
+                        // Performance: npm package imports that aren't pointer imports can be externalized. For example, if Vike eventually adds support for setting Vite configs in the vike.config.js file, then the user may import a Vite plugin in his vike.config.js file. (We could as well let esbuild always transpile /node_modules/ code but it would be useless and would unnecessarily slow down transpilation.)
+                        resolved.path.includes('/node_modules/');
+                    if (debug.isActivated)
+                        debug('onResolved()', { args, resolved, isPointerImport, isExternal });
+                    // We need esbuild to resolve path aliases so that we can use:
+                    //   isNpmPackageImport(str, { cannotBePathAlias: true })
+                    //   assertIsNpmPackageImport()
+                    (0, utils_js_1.assertPathIsFilesystemAbsolute)(resolved.path);
+                    if (isExternal) {
+                        return { external: true, path: resolved.path };
+                    }
+                    else {
+                        return resolved;
+                    }
+                });
             }
-        ];
-    }
+        },
+        // Track dependencies
+        {
+            name: 'vike:dependency-tracker',
+            setup(b) {
+                b.onLoad({ filter: /./ }, (args) => {
+                    // We collect the dependency `args.path` in case the bulid fails (upon build error => error is thrown => no metafile)
+                    let { path } = args;
+                    path = (0, utils_js_1.toPosixPath)(path);
+                    getVikeConfig_js_1.vikeConfigDependencies.add(path);
+                    return undefined;
+                });
+                /* To exhaustively collect all dependencies upon build failure, we would also need to use onResolve().
+                 *  - Because onLoad() isn't call if the config dependency can't be resolved.
+                 *  - For example, the following breaks auto-reload (the config is stuck in its error state and the user needs to touch the importer for the config to reload):
+                 *    ```bash
+                 *    mv ./some-config-dependency.js /tmp/ && mv /tmp/some-config-dependency.js .
+                 *    ```
+                 *  - But implementing a fix is complex and isn't worth it.
+                b.onResolve(...)
+                */
+            }
+        }
+    ];
     let result;
     try {
         result = await (0, esbuild_1.build)(options);
@@ -172,15 +181,13 @@ async function transpileWithEsbuild(filePath, userRootDir, transformImports) {
         throw err;
     }
     // Track dependencies
-    if (transformImports !== 'all') {
-        (0, utils_js_1.assert)(result.metafile);
-        Object.keys(result.metafile.inputs).forEach((filePathRelative) => {
-            filePathRelative = (0, utils_js_1.toPosixPath)(filePathRelative);
-            (0, utils_js_1.assertPosixPath)(userRootDir);
-            const filePathAbsoluteFilesystem = path_1.default.posix.join(userRootDir, filePathRelative);
-            getVikeConfig_js_1.vikeConfigDependencies.add(filePathAbsoluteFilesystem);
-        });
-    }
+    (0, utils_js_1.assert)(result.metafile);
+    Object.keys(result.metafile.inputs).forEach((filePathRelative) => {
+        filePathRelative = (0, utils_js_1.toPosixPath)(filePathRelative);
+        (0, utils_js_1.assertPosixPath)(userRootDir);
+        const filePathAbsoluteFilesystem = path_1.default.posix.join(userRootDir, filePathRelative);
+        getVikeConfig_js_1.vikeConfigDependencies.add(filePathAbsoluteFilesystem);
+    });
     const code = result.outputFiles[0].text;
     (0, utils_js_1.assert)(typeof code === 'string');
     return { code, pointerImports };
@@ -189,7 +196,7 @@ async function executeTranspiledFile(filePath, code) {
     const { filePathAbsoluteFilesystem } = filePath;
     // Alternative to using a temporary file: https://github.com/vitejs/vite/pull/13269
     //  - But seems to break source maps, so I don't think it's worth it
-    const filePathTmp = getFilePathTmp(filePathAbsoluteFilesystem);
+    const filePathTmp = getTemporaryBuildFilePath(filePathAbsoluteFilesystem);
     fs_1.default.writeFileSync(filePathTmp, code);
     const clean = () => fs_1.default.unlinkSync(filePathTmp);
     let fileExports = {};
@@ -248,22 +255,21 @@ function getConfigExecutionErrorIntroMsg(err) {
     return errIntroMsg ?? null;
 }
 exports.getConfigExecutionErrorIntroMsg = getConfigExecutionErrorIntroMsg;
-const tmpPrefix = `[build-`;
-function getFilePathTmp(filePathAbsoluteFilesystem) {
+function getTemporaryBuildFilePath(filePathAbsoluteFilesystem) {
     (0, utils_js_1.assertPosixPath)(filePathAbsoluteFilesystem);
     const dirname = path_1.default.posix.dirname(filePathAbsoluteFilesystem);
     const filename = path_1.default.posix.basename(filePathAbsoluteFilesystem);
-    // Syntax with semicolon `[build:${/*...*/}]` doesn't work on Windows: https://github.com/vikejs/vike/issues/800#issuecomment-1517329455
-    const tag = `${tmpPrefix}${(0, utils_js_1.getRandomId)(12)}]`;
-    const filePathTmp = path_1.default.posix.join(dirname, `${tag}${filename}.mjs`);
+    // Syntax with semicolon `build:${/*...*/}` doesn't work on Windows: https://github.com/vikejs/vike/issues/800#issuecomment-1517329455
+    const filePathTmp = path_1.default.posix.join(dirname, `${filename}.build-${(0, utils_js_1.getRandomId)(12)}.mjs`);
+    (0, utils_js_1.assert)(isTemporaryBuildFile(filePathTmp));
     return filePathTmp;
 }
-function isTmpFile(filePath) {
+function isTemporaryBuildFile(filePath) {
     (0, utils_js_1.assertPosixPath)(filePath);
     const fileName = path_1.default.posix.basename(filePath);
-    return fileName.startsWith(tmpPrefix);
+    return /\.build-[a-z0-9]{12}\.mjs$/.test(fileName);
 }
-exports.isTmpFile = isTmpFile;
+exports.isTemporaryBuildFile = isTemporaryBuildFile;
 function isHeaderFile(filePath) {
     (0, utils_js_1.assertPosixPath)(filePath);
     const fileExtensions = getFileExtensions(filePath);
@@ -306,3 +312,23 @@ function getErrIntroMsg(operation, filePath) {
     ].join(' ');
     return msg;
 }
+function cleanEsbuildErrors(errors) {
+    errors.forEach((e) => (e.notes = e.notes.filter((note) => 
+    // Remove note:
+    // ```shell
+    // You can mark the path "#root/renderer/onRenderHtml_typo" as external to exclude it from the bundle, which will remove this error and leave the unresolved path in the bundle.
+    // ```
+    //
+    // From error:
+    // ```shell
+    // ✘ [ERROR] Could not resolve "#root/renderer/onRenderHtml_typo" [plugin vike-esbuild]
+    //
+    //    renderer/+config.h.js:1:29:
+    //      1 │ import { onRenderHtml } from "#root/renderer/onRenderHtml_typo"
+    //        ╵                              ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+    //
+    //  You can mark the path "#root/renderer/onRenderHtml_typo" as external to exclude it from the bundle, which will remove this error and leave the unresolved path in the bundle.
+    //
+    // ```
+    !note.text.includes('as external to exclude it from the bundle'))));
+}

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

@@ -750,7 +750,7 @@ async function findPlusFiles(userRootDir, outDirRoot, isDev) {
 }
 function getConfigName(filePath) {
     (0, utils_js_1.assertPosixPath)(filePath);
-    if ((0, transpileAndExecuteFile_js_1.isTmpFile)(filePath))
+    if ((0, transpileAndExecuteFile_js_1.isTemporaryBuildFile)(filePath))
         return null;
     const fileName = path_1.default.posix.basename(filePath);
     // assertNoUnexpectedPlusSign(filePath, fileName)
@@ -781,7 +781,30 @@ function assertNoUnexpectedPlusSign(filePath: string, fileName: string) {
 }
 */
 function handleUnknownConfig(configName, configNames, filePathToShowToUser) {
-    let errMsg = `${filePathToShowToUser} defines an unknown config ${picocolors_1.default.cyan(configName)}`;
+    {
+        const ui = ['vike-react', 'vike-vue', 'vike-solid'];
+        const knownVikeExntensionConfigs = {
+            description: ui,
+            favicon: ui,
+            Head: ui,
+            Layout: ui,
+            onCreateApp: ['vike-vue'],
+            title: ui,
+            ssr: ui,
+            stream: ui,
+            Wrapper: ui
+        };
+        if (configName in knownVikeExntensionConfigs) {
+            const requiredVikeExtension = knownVikeExntensionConfigs[configName];
+            (0, utils_js_1.assertUsage)(false, [
+                `${filePathToShowToUser} uses the config ${picocolors_1.default.cyan(configName)} (https://vike.dev/${configName})`,
+                `which requires the Vike extension ${requiredVikeExtension.map((e) => picocolors_1.default.bold(e)).join('/')}.`,
+                `Make sure to install the Vike extension,`,
+                `and make sure it applies to ${filePathToShowToUser} as explained at https://vike.dev/extends#inheritance.`
+            ].join(' '));
+        }
+    }
+    let errMsg = `${filePathToShowToUser} sets an unknown config ${picocolors_1.default.cyan(configName)}`;
     let configNameSimilar = null;
     if (configName === 'page') {
         configNameSimilar = 'Page';
@@ -789,17 +812,13 @@ function handleUnknownConfig(configName, configNames, filePathToShowToUser) {
     else {
         configNameSimilar = (0, utils_js_1.getMostSimilar)(configName, configNames);
     }
-    if (configNameSimilar || configName === 'page') {
-        (0, utils_js_1.assert)(configNameSimilar);
+    if (configNameSimilar) {
         (0, utils_js_1.assert)(configNameSimilar !== configName);
-        errMsg += `, did you mean to define ${picocolors_1.default.cyan(configNameSimilar)} instead?`;
+        errMsg += `, did you mean to set ${picocolors_1.default.cyan(configNameSimilar)} instead?`;
         if (configName === 'page') {
             errMsg += ` (The name of the config ${picocolors_1.default.cyan('Page')} starts with a capital letter ${picocolors_1.default.cyan('P')} because it usually defines a UI component: a ubiquitous JavaScript convention is to start the name of UI components with a capital letter.)`;
         }
     }
-    else {
-        errMsg += `, you need to define the config ${picocolors_1.default.cyan(configName)} by using ${picocolors_1.default.cyan('config.meta')} https://vike.dev/meta`;
-    }
     (0, utils_js_1.assertUsage)(false, errMsg);
 }
 function determineRouteFilesystem(locationId, configValueSources) {

package/dist/cjs/node/plugin/shared/getFilePath.js

@@ -90,7 +90,12 @@ function getFilePathAbsoluteUserRootDir({ filePathAbsoluteFilesystem, userRootDi
         (0, utils_js_1.assert)(filePathRelative.startsWith('../'));
         return null;
     }
-    (0, utils_js_1.assert)(!filePathRelative.startsWith('.') && !filePathRelative.startsWith('/'));
+    (0, utils_js_1.assert)(!filePathRelative.startsWith('/') &&
+        /* Not true if filePathRelative starts with a hidden directory  (i.e. a directory with a name that starts with `.`)
+        !filePathRelative.startsWith('.') &&
+        */
+        !filePathRelative.startsWith('./') &&
+        !filePathRelative.startsWith('../'));
     const filePathAbsoluteUserRootDir = `/${filePathRelative}`;
     (0, utils_js_1.assert)(filePathAbsoluteUserRootDir === getFilePathAbsoluteUserRootDir2(filePathAbsoluteFilesystem, userRootDir));
     return filePathAbsoluteUserRootDir;

package/dist/cjs/node/runtime/renderPage/analyzePage.js

@@ -15,7 +15,7 @@ function analyzePage(pageFilesAll, pageConfig, pageId) {
         const clientDependencies = [];
         clientDependencies.push({
             id: (0, virtualFilePageConfigValuesAll_js_1.getVirtualFileIdPageConfigValuesAll)(pageConfig.pageId, true),
-            onlyAssets: false,
+            onlyAssets: isClientSideRenderable ? false : true,
             eagerlyImported: false
         });
         // In production we inject the import of the server virtual module with ?extractAssets inside the client virtual module

package/dist/cjs/node/runtime/renderPage/logErrorHint.js

@@ -21,6 +21,10 @@ const knownErrors = [
         // ```
         errMsg: 'assets.json',
         link: 'https://vike.dev/getGlobalContext'
+    },
+    {
+        errMsg: /Named export.*not found/i,
+        link: 'https://vike.dev/broken-npm-package#named-export-not-found'
     }
 ];
 function logErrorHint(error) {
@@ -290,7 +294,15 @@ function extractFromNodeModulesPath(str) {
     return packageName;
 }
 function includes(str1, str2) {
-    return !!str1 && str1.toLowerCase().includes(str2.toLowerCase());
+    if (!str1)
+        return false;
+    if (str2 instanceof RegExp) {
+        return str2.test(str1.toLowerCase());
+    }
+    if (typeof str2 === 'string') {
+        return str1.toLowerCase().includes(str2.toLowerCase());
+    }
+    return false;
 }
 function includesNodeModules(str) {
     if (!str)

package/dist/cjs/node/shared/prependEntriesDir.js

@@ -7,6 +7,7 @@ function prependEntriesDir(entryName) {
         entryName = entryName.slice(1);
     }
     (0, utils_js_1.assert)(!entryName.startsWith('/'));
+    (0, utils_js_1.assert)(entryName);
     entryName = `entries/${entryName}`;
     return entryName;
 }

package/dist/cjs/utils/assertPathIsFilesystemAbsolute.js

@@ -11,11 +11,11 @@ const filesystemPathHandling_js_1 = require("./filesystemPathHandling.js");
 function assertPathIsFilesystemAbsolute(p) {
     (0, filesystemPathHandling_js_1.assertPosixPath)(p);
     (0, assert_js_1.assert)(!p.startsWith('/@fs/'));
-    if (process.platform === 'win32') {
-        (0, assert_js_1.assert)(path_1.default.win32.isAbsolute(p));
+    if (process.platform !== 'win32') {
+        (0, assert_js_1.assert)(p.startsWith('/'));
     }
     else {
-        (0, assert_js_1.assert)(p.startsWith('/'));
+        (0, assert_js_1.assert)(path_1.default.win32.isAbsolute(p));
     }
 }
 exports.assertPathIsFilesystemAbsolute = assertPathIsFilesystemAbsolute;