vike 0.4.165 → 0.4.166-commit-a5e2596

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

@@ -38,7 +38,7 @@ function plugin(vikeConfig) {
         ...(0, commonConfig_js_1.commonConfig)(),
         (0, index_js_2.importUserCode)(),
         ...(0, index_js_1.devConfig)(),
-        (0, buildConfig_js_1.buildConfig)(),
+        ...(0, buildConfig_js_1.buildConfig)(),
         (0, previewConfig_js_1.previewConfig)(),
         ...(0, autoFullBuild_js_1.autoFullBuild)(),
         (0, packageJsonFile_js_1.packageJsonFile)(),

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

@@ -10,7 +10,7 @@ function baseUrls(configVike) {
     return {
         name: 'vike:baseUrls',
         enforce: 'post',
-        config: (config) => {
+        async config(config) {
             const bases = (0, resolveBase_js_1.resolveBaseFromUserConfig)(config, configVike);
             baseServer = bases.baseServer;
             baseAssets = bases.baseAssets;

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

@@ -49,10 +49,14 @@ async function copyAssets(filesToCopy, config) {
         return;
     (0, utils_js_1.assert)((0, fs_1.existsSync)(assetsDirServer));
     const concurrencyLimit = (0, utils_js_1.pLimit)(10);
-    await Promise.all(filesToCopy.map((file) => concurrencyLimit(() => promises_1.default.cp(path_1.default.posix.join(outDirServer, file), path_1.default.posix.join(outDirClient, file), {
+    await Promise.all(filesToCopy.map((file) => concurrencyLimit(() => 
+    // TODO: move instead of copying
+    promises_1.default.cp(path_1.default.posix.join(outDirServer, file), path_1.default.posix.join(outDirClient, file), {
         recursive: true
     }))));
-    await promises_1.default.rm(assetsDirServer, { recursive: true });
+    /* We cannot do that because, with some edge case Rollup settings (outputing JavaScript chunks and static assets to the same directoy), this removes JavaScript chunks, see https://github.com/vikejs/vike/issues/1154#issuecomment-1975762404
+    await fs.rm(assetsDirServer, { recursive: true })
+    */
 }
 // Add serverManifest resources to clientManifest
 function addServerAssets(clientManifest, serverManifest) {

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

@@ -18,6 +18,7 @@ const path_1 = __importDefault(require("path"));
 const fixServerAssets_js_1 = require("./buildConfig/fixServerAssets.js");
 const index_js_1 = require("./importBuild/index.js");
 const prependEntriesDir_js_1 = require("../../shared/prependEntriesDir.js");
+const getFilePath_js_1 = require("../shared/getFilePath.js");
 // @ts-ignore Shimmed by dist-cjs-fixup.js for CJS build.
 const importMetaUrl = `file://${__filename}`;
 const require_ = (0, module_1.createRequire)(importMetaUrl);
@@ -28,71 +29,81 @@ function buildConfig() {
     let isSsrBuild;
     let outDirs;
     let config;
-    return {
-        name: 'vike:buildConfig',
-        apply: 'build',
-        enforce: 'post',
-        configResolved: {
-            order: 'post',
-            async handler(config_) {
-                config = config_;
-                (0, utils_js_1.assertNodeEnv_build)();
-                assertRollupInput(config);
-                const entries = await getEntries(config);
-                (0, utils_js_1.assert)(Object.keys(entries).length > 0);
-                config.build.rollupOptions.input = (0, utils_js_1.injectRollupInputs)(entries, config);
-                addLogHook();
-                outDirs = (0, utils_js_1.getOutDirs)(config);
-                {
-                    isServerAssetsFixEnabled = (0, fixServerAssets_js_1.fixServerAssets_isEnabled)() && (await (0, getVikeConfig_js_1.isV1Design)(config, false));
-                    if (isServerAssetsFixEnabled) {
-                        // https://github.com/vikejs/vike/issues/1339
-                        config.build.ssrEmitAssets = true;
-                        // Required if `ssrEmitAssets: true`, see https://github.com/vitejs/vite/pull/11430#issuecomment-1454800934
-                        config.build.cssMinify = 'esbuild';
+    return [
+        {
+            name: 'vike:buildConfig:configResolved',
+            apply: 'build',
+            enforce: 'post',
+            configResolved: {
+                order: 'post',
+                async handler(config_) {
+                    config = config_;
+                    (0, utils_js_1.assertNodeEnv_build)();
+                    assertRollupInput(config);
+                    const entries = await getEntries(config);
+                    (0, utils_js_1.assert)(Object.keys(entries).length > 0);
+                    config.build.rollupOptions.input = (0, utils_js_1.injectRollupInputs)(entries, config);
+                    addLogHook();
+                    outDirs = (0, utils_js_1.getOutDirs)(config);
+                    {
+                        isServerAssetsFixEnabled = (0, fixServerAssets_js_1.fixServerAssets_isEnabled)() && (await (0, getVikeConfig_js_1.isV1Design)(config, false));
+                        if (isServerAssetsFixEnabled) {
+                            // https://github.com/vikejs/vike/issues/1339
+                            config.build.ssrEmitAssets = true;
+                            // Required if `ssrEmitAssets: true`, see https://github.com/vitejs/vite/pull/11430#issuecomment-1454800934
+                            config.build.cssMinify = 'esbuild';
+                        }
                     }
                 }
+            },
+            config(config) {
+                (0, utils_js_1.assertNodeEnv_build)();
+                isSsrBuild = (0, utils_js_1.viteIsSSR)(config);
+                return {
+                    build: {
+                        outDir: (0, utils_js_1.resolveOutDir)(config),
+                        manifest: manifestTempFile,
+                        copyPublicDir: !isSsrBuild
+                    }
+                };
+            },
+            buildStart() {
+                (0, utils_js_1.assertNodeEnv_build)();
             }
         },
-        config(config) {
-            (0, utils_js_1.assertNodeEnv_build)();
-            isSsrBuild = (0, utils_js_1.viteIsSSR)(config);
-            return {
-                build: {
-                    outDir: (0, utils_js_1.resolveOutDir)(config),
-                    manifest: manifestTempFile,
-                    copyPublicDir: !isSsrBuild
-                }
-            };
-        },
-        buildStart() {
-            (0, utils_js_1.assertNodeEnv_build)();
-        },
-        writeBundle: {
-            order: 'post',
-            sequential: true,
-            async handler(options, bundle) {
-                if (isSsrBuild) {
-                    // Ideally we'd move dist/_temp_manifest.json to dist/server/client-assets.json instead of dist/assets.json
-                    //  - But we can't because there is no guarentee whether dist/server/ is generated before or after dist/client/ (generating dist/server/ after dist/client/ erases dist/server/client-assets.json)
-                    //  - We'll able to do so once we replace `$ vite build` with `$ vike build`
-                    const assetsJsonFilePath = path_1.default.posix.join(outDirs.outDirRoot, 'assets.json');
-                    const clientManifestFilePath = path_1.default.posix.join(outDirs.outDirClient, manifestTempFile);
-                    const serverManifestFilePath = path_1.default.posix.join(outDirs.outDirServer, manifestTempFile);
-                    if (!isServerAssetsFixEnabled) {
-                        await promises_1.default.copyFile(clientManifestFilePath, assetsJsonFilePath);
-                    }
-                    else {
-                        const clientManifestMod = await (0, fixServerAssets_js_1.fixServerAssets)(config);
-                        await promises_1.default.writeFile(assetsJsonFilePath, JSON.stringify(clientManifestMod, null, 2), 'utf-8');
+        {
+            name: 'vike:buildConfig:writeBundle',
+            apply: 'build',
+            // Make sure other writeBundle() hooks are called after this writeBundle() hook.
+            //  - set_constant_ASSETS_MAP() needs to be called before dist/server/ code is executed.
+            //    - For example, the writeBundle() hook of vite-plugin-vercel needs to be called after this writeBundle() hook, otherwise: https://github.com/vikejs/vike/issues/1527
+            enforce: 'pre',
+            writeBundle: {
+                order: 'pre',
+                sequential: true,
+                async handler(options, bundle) {
+                    if (isSsrBuild) {
+                        // Ideally we'd move dist/_temp_manifest.json to dist/server/client-assets.json instead of dist/assets.json
+                        //  - But we can't because there is no guarentee whether dist/server/ is generated before or after dist/client/ (generating dist/server/ after dist/client/ erases dist/server/client-assets.json)
+                        //  - We'll able to do so once we replace `$ vite build` with `$ vike build`
+                        const assetsJsonFilePath = path_1.default.posix.join(outDirs.outDirRoot, 'assets.json');
+                        const clientManifestFilePath = path_1.default.posix.join(outDirs.outDirClient, manifestTempFile);
+                        const serverManifestFilePath = path_1.default.posix.join(outDirs.outDirServer, manifestTempFile);
+                        if (!isServerAssetsFixEnabled) {
+                            await promises_1.default.copyFile(clientManifestFilePath, assetsJsonFilePath);
+                        }
+                        else {
+                            const clientManifestMod = await (0, fixServerAssets_js_1.fixServerAssets)(config);
+                            await promises_1.default.writeFile(assetsJsonFilePath, JSON.stringify(clientManifestMod, null, 2), 'utf-8');
+                        }
+                        await promises_1.default.rm(clientManifestFilePath);
+                        await promises_1.default.rm(serverManifestFilePath);
+                        await (0, index_js_1.set_constant_ASSETS_MAP)(options, bundle);
                     }
-                    await promises_1.default.rm(clientManifestFilePath);
-                    await promises_1.default.rm(serverManifestFilePath);
-                    await (0, index_js_1.set_constant_ASSETS_MAP)(options, bundle);
                 }
             }
         }
-    };
+    ];
 }
 exports.buildConfig = buildConfig;
 async function getEntries(config) {
@@ -195,18 +206,22 @@ async function getPageFileEntries(config, includeAssetsImportedByServer) {
     return pageFileEntries;
 }
 function getEntryFromClientEntry(clientEntry, config, addExtractAssetsQuery) {
-    if ((0, utils_js_1.isNpmPackageImport)(clientEntry)) {
+    if (!clientEntry.startsWith('/')) {
+        (0, utils_js_1.assertIsNpmPackageImport)(clientEntry);
         const entryTarget = clientEntry;
         const entryName = (0, prependEntriesDir_js_1.prependEntriesDir)(clientEntry);
         return { entryName, entryTarget };
     }
-    const filePath = clientEntry;
-    (0, utils_js_1.assertPosixPath)(filePath);
-    (0, utils_js_1.assert)(filePath.startsWith('/'));
-    let entryTarget = (0, utils_js_1.getFilePathAbsolute)(filePath, config);
+    const filePathAbsoluteUserRootDir = clientEntry;
+    (0, utils_js_1.assert)(filePathAbsoluteUserRootDir.startsWith('/'));
+    const filePath = (0, getFilePath_js_1.getFilePathResolved)({
+        filePathAbsoluteUserRootDir,
+        userRootDir: config.root
+    });
+    let entryTarget = filePath.filePathAbsoluteFilesystem;
     if (addExtractAssetsQuery)
         entryTarget = (0, extractAssetsQuery_js_1.extractAssetsAddQuery)(entryTarget);
-    let entryName = filePath;
+    let entryName = filePathAbsoluteUserRootDir;
     if (addExtractAssetsQuery)
         entryName = (0, extractAssetsQuery_js_1.extractAssetsAddQuery)(entryName);
     entryName = (0, utils_js_1.removeFileExtention)(entryName);

package/dist/cjs/node/plugin/plugins/commonConfig/assertResolveAlias.js

@@ -6,8 +6,21 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.assertResolveAlias = void 0;
 const utils_js_1 = require("../../utils.js");
 const picocolors_1 = __importDefault(require("@brillout/picocolors"));
-// TODO/v1-release: replace assertWarning() with assertUsage()
+// Recommend users to avoid un-distinguishable path aliases.
+// There are a lot of libraries that don't or cannot follow that recommendation, for example:
+// - Nx
+//   - Not sure why, but Nx seems to add a path alias for each monorepo package
+//   - https://github.com/vikejs/vike/discussions/1134
+// - MUI
+//   - https://mui.com/material-ui/guides/minimizing-bundle-size/#how-to-use-custom-bundles
+//   - https://github.com/vikejs/vike/discussions/1549#discussioncomment-8789002
+// - @preact/preset-vite
+//   - Aliases react imports
+// - @vitejs/plugin-vue2
+//   - https://github.com/vikejs/vike/issues/1329
 function assertResolveAlias(config) {
+    // TODO: re-implement warning https://github.com/vikejs/vike/issues/1567
+    return;
     const aliases = getAliases(config);
     const errPrefix = config.configFile || 'Your Vite configuration';
     const errSuffix1 = 'see https://vike.dev/path-aliases#vite';

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

@@ -26,7 +26,7 @@ async function getConfigVikPromise(vikeConfig, config) {
     const fromPluginOptions = (vikeConfig ?? {});
     const fromViteConfig = (config.vike ?? {});
     const configs = [fromPluginOptions, fromViteConfig];
-    const { globalVikeConfig: fromPlusConfigFile } = await (0, getVikeConfig_js_1.getVikeConfig)(config, (0, utils_js_1.isDev2)(config), false);
+    const { globalVikeConfig: fromPlusConfigFile } = await (0, getVikeConfig_js_1.getVikeConfig)(config, (0, utils_js_1.isDev2)(config));
     configs.push(fromPlusConfigFile);
     (0, assertVikeConfig_js_1.assertVikeConfig)(fromPlusConfigFile, ({ prop, errMsg }) => {
         // TODO: add config file path ?

package/dist/cjs/node/plugin/plugins/devConfig/determineOptimizeDeps.js

@@ -7,6 +7,8 @@ const getVikeConfig_js_1 = require("../importUserCode/v1-design/getVikeConfig.js
 const getConfigValueSourcesNotOverriden_js_1 = require("../../shared/getConfigValueSourcesNotOverriden.js");
 const buildConfig_js_1 = require("../buildConfig.js");
 const virtualFileImportUserCode_js_1 = require("../../../shared/virtual-files/virtualFileImportUserCode.js");
+const getFilePath_js_1 = require("../../shared/getFilePath.js");
+const debug = (0, utils_js_1.createDebugger)('vike:optimizeDeps');
 async function determineOptimizeDeps(config, isDev) {
     const { pageConfigs } = await (0, getVikeConfig_js_1.getVikeConfig)(config, isDev);
     const { entries, include } = await getPageDeps(config, pageConfigs, isDev);
@@ -23,7 +25,11 @@ async function determineOptimizeDeps(config, isDev) {
     */
     config.optimizeDeps.include = [...include, ...normalizeInclude(config.optimizeDeps.include)];
     config.optimizeDeps.entries = [...entries, ...normalizeEntries(config.optimizeDeps.entries)];
-    // console.log('config.optimizeDeps', { entries: config.optimizeDeps.entries, include: config.optimizeDeps.include })
+    if (debug.isEnabled)
+        debug('config.optimizeDeps', {
+            'config.optimizeDeps.entries': config.optimizeDeps.entries,
+            'config.optimizeDeps.include': config.optimizeDeps.include
+        });
 }
 exports.determineOptimizeDeps = determineOptimizeDeps;
 async function getPageDeps(config, pageConfigs, isDev) {
@@ -38,7 +44,7 @@ async function getPageDeps(config, pageConfigs, isDev) {
                 const { definedAt, configEnv } = configValueSource;
                 if (!configEnv.client)
                     return;
-                if (definedAt.filePathRelativeToUserRootDir !== null) {
+                if (definedAt.filePathAbsoluteUserRootDir !== null) {
                     const { filePathAbsoluteFilesystem } = definedAt;
                     (0, utils_js_1.assert)(filePathAbsoluteFilesystem);
                     // Surprisingly Vite expects entries to be absolute paths
@@ -47,24 +53,10 @@ async function getPageDeps(config, pageConfigs, isDev) {
                 else {
                     // Adding definedAt.filePathAbsoluteFilesystem doesn't work for npm packages, I guess because of Vite's config.server.fs.allow
                     const { importPathAbsolute } = definedAt;
-                    (0, utils_js_1.assert)(importPathAbsolute);
-                    // We need to differentiate between npm package imports and path aliases.
-                    // There are path aliases that cannot be distinguished from npm package names.
-                    // We recommend users to use the '#' prefix convention for path aliases, see https://vike.dev/path-aliases#vite and assertResolveAlias()
-                    if ((0, utils_js_1.isNpmPackageImport)(importPathAbsolute)) {
-                        // isNpmPackageImport() returns false for a path alias like #root/renderer/onRenderClient
-                        (0, utils_js_1.assert)(!importPathAbsolute.startsWith('#'));
-                        include.push(importPathAbsolute);
-                    }
-                    else {
-                        /* Path aliases, e.g.:
-                         * ```js
-                         * // /renderer/+config.js
-                         * import onRenderClient from '#root/renderer/onRenderClient'
-                         * ```
-                         */
-                        entries.push(importPathAbsolute);
-                    }
+                    (0, utils_js_1.assert)(importPathAbsolute); // Help TS
+                    // Shouldn't be a path alias, as path aliases would need to be added to config.optimizeDeps.entries instead of config.optimizeDeps.include
+                    (0, utils_js_1.assertIsNpmPackageImport)(importPathAbsolute);
+                    include.push(importPathAbsolute);
                 }
             });
         });
@@ -72,9 +64,12 @@ async function getPageDeps(config, pageConfigs, isDev) {
     // V0.4 design
     {
         const pageFiles = await (0, findPageFiles_js_1.findPageFiles)(config, ['.page', '.page.client'], isDev);
-        pageFiles.forEach((filePath) => {
-            const entry = (0, utils_js_1.getFilePathAbsolute)(filePath, config);
-            entries.push(entry);
+        const userRootDir = config.root;
+        pageFiles.forEach((filePathAbsoluteUserRootDir) => {
+            const entry = (0, getFilePath_js_1.getFilePathResolved)({ filePathAbsoluteUserRootDir, userRootDir });
+            const { filePathAbsoluteFilesystem } = entry;
+            (0, utils_js_1.assert)(filePathAbsoluteFilesystem);
+            entries.push(filePathAbsoluteFilesystem);
         });
     }
     entries = (0, utils_js_1.unique)(entries);

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

@@ -4,6 +4,7 @@ exports.applyEnvVar = exports.envVarsPlugin = void 0;
 const vite_1 = require("vite");
 const utils_js_1 = require("../utils.js");
 const rollupSourceMap_js_1 = require("../shared/rollupSourceMap.js");
+const getFilePath_js_1 = require("../shared/getFilePath.js");
 function envVarsPlugin() {
     let envsAll;
     let config;
@@ -45,10 +46,10 @@ function envVarsPlugin() {
                     if (isPrivate && isClientSide) {
                         if (!code.includes(envStatement))
                             return;
-                        const filePathToShowToUser = (0, utils_js_1.getFilePathRelativeToUserRootDir)(id, config.root);
+                        const modulePath = (0, getFilePath_js_1.getModuleFilePath)(id, config);
                         const errMsgAddendum = isBuild ? '' : ' (Vike will prevent your app from building for production)';
                         const keyPublic = `${publicPrefix}${envName}`;
-                        const errMsg = `${envStatement} is used in client-side file ${filePathToShowToUser} 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`;
+                        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);
                         }

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

@@ -120,12 +120,6 @@ function extractAssetsPlugin() {
         {
             name: 'vike:extractAssets-3',
             apply: 'build',
-            async configResolved(config_) {
-                configVike = await (0, getConfigVike_js_1.getConfigVike)(config_);
-                config = config_;
-                vikeConfig = await (0, getVikeConfig_js_1.getVikeConfig)(config, false);
-                isServerAssetsFixEnabled = (0, fixServerAssets_js_1.fixServerAssets_isEnabled)() && (await (0, getVikeConfig_js_1.isV1Design)(config, false));
-            },
             load(id) {
                 if (!(0, utils_js_1.isVirtualFileId)(id))
                     return undefined;
@@ -142,9 +136,15 @@ function extractAssetsPlugin() {
         },
         {
             name: 'vike:extractAssets-4',
-            configResolved(config) {
-                // 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');
+            async configResolved(config_) {
+                configVike = await (0, getConfigVike_js_1.getConfigVike)(config_);
+                config = config_;
+                vikeConfig = await (0, getVikeConfig_js_1.getVikeConfig)(config, false);
+                isServerAssetsFixEnabled = (0, fixServerAssets_js_1.fixServerAssets_isEnabled)() && (await (0, getVikeConfig_js_1.isV1Design)(config, false));
+                if (!isServerAssetsFixEnabled) {
+                    // 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/plugin/plugins/fileEnv.js

@@ -8,6 +8,7 @@ const utils_js_1 = require("../utils.js");
 const extractAssetsPlugin_js_1 = require("./extractAssetsPlugin.js");
 const extractExportNamesPlugin_js_1 = require("./extractExportNamesPlugin.js");
 const picocolors_1 = __importDefault(require("@brillout/picocolors"));
+const getFilePath_js_1 = require("../shared/getFilePath.js");
 function fileEnv() {
     let config;
     let isDev = false;
@@ -44,11 +45,13 @@ function fileEnv() {
                 // resolved is null when import path is erroneous and doesn't actually point to a file
                 if (!resolved)
                     return;
-                const modulePath = resolved.id.split('?')[0];
+                const moduleId = resolved.id;
+                const modulePath = moduleId.split('?')[0];
                 // `.server.js` and `.client.js` should only apply to user files
                 if (modulePath.includes('/node_modules/'))
                     return;
                 // TODO/v1-release: remove
+                // - I don't remember exactly, but I think I've added `TODO/v1-release: remove` because I vaguely remember that we can remove this after we remove the 0.4 design.
                 if (modulePath.endsWith('.css'))
                     return;
                 const isServerSide = options?.ssr;
@@ -60,7 +63,7 @@ function fileEnv() {
                     return;
                 // Show error message
                 let errMsg;
-                let modulePathPretty = (0, utils_js_1.getFilePathRelativeToUserRootDir)(modulePath, config.root);
+                let modulePathPretty = (0, getFilePath_js_1.getModuleFilePath)(moduleId, config);
                 modulePathPretty = modulePathPretty.replaceAll(suffix, picocolors_1.default.bold(suffix));
                 errMsg = `${(0, utils_js_1.capitalizeFirstLetter)(envExpect)}-only module ${modulePathPretty} (https://vike.dev/file-env) imported on the ${envActual}-side`;
                 if (importer &&
@@ -68,7 +71,7 @@ function fileEnv() {
                     !importer.includes('virtual:vike:') &&
                     // I don't know why and who sets importer to '<stdin>' (I guess Vite?)
                     importer !== '<stdin>') {
-                    const importerPath = (0, utils_js_1.getFilePathRelativeToUserRootDir)(importer.split('?')[0], config.root);
+                    const importerPath = (0, getFilePath_js_1.getModuleFilePath)(importer, config);
                     errMsg += ` by ${importerPath}`;
                 }
                 if (isDev) {

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

@@ -14,6 +14,7 @@ const virtualFileImportUserCode_js_1 = require("../../../shared/virtual-files/vi
 const getVikeConfig_js_1 = require("./v1-design/getVikeConfig.js");
 const picocolors_1 = __importDefault(require("@brillout/picocolors"));
 const loggerNotProd_js_1 = require("../../shared/loggerNotProd.js");
+const getFilePath_js_1 = require("../../shared/getFilePath.js");
 function importUserCode() {
     let config;
     let configVike;
@@ -124,8 +125,11 @@ function isVikeConfigModule(filePathAbsoluteFilesystem) {
 }
 function reloadConfig(filePath, config, op) {
     {
-        const filePathToShowToUser = picocolors_1.default.dim((0, utils_js_1.getFilePathRelativeToUserRootDir)(filePath, config.root, true));
-        const msg = `${op} ${filePathToShowToUser}`;
+        const { filePathToShowToUserResolved } = (0, getFilePath_js_1.getFilePathResolved)({
+            filePathAbsoluteFilesystem: filePath,
+            userRootDir: config.root
+        });
+        const msg = `${op} ${picocolors_1.default.dim(filePathToShowToUserResolved)}`;
         (0, loggerNotProd_js_1.logConfigInfo)(msg, 'info');
     }
     (0, getVikeConfig_js_1.reloadVikeConfig)(config.root, (0, utils_js_1.getOutDirs)(config).outDirRoot);

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

@@ -105,7 +105,6 @@ const configDefinitionsBuiltInGlobal = {
     prerender: {
         env: { config: true }
     },
-    extensions: { env: { config: true } },
     disableAutoFullBuild: { env: { config: true } },
     includeAssetsImportedByServer: { env: { config: true } },
     baseAssets: { env: { config: true } },

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

@@ -11,9 +11,9 @@ const child_process_1 = require("child_process");
 const util_1 = require("util");
 const picocolors_1 = __importDefault(require("@brillout/picocolors"));
 const execA = (0, util_1.promisify)(child_process_1.exec);
-const globalObject = (0, utils_js_1.getGlobalObject)('crawlPlusFiles.ts', {
-    gitIsMissing: false
-});
+(0, utils_js_1.assertIsNotProductionRuntime)();
+(0, utils_js_1.assertIsSingleModuleInstance)('crawlPlusFiles.ts');
+let gitIsNotUsable = false;
 async function crawlPlusFiles(userRootDir, outDirAbsoluteFilesystem, isDev) {
     (0, utils_js_1.assertPosixPath)(userRootDir);
     (0, utils_js_1.assertPosixPath)(outDirAbsoluteFilesystem);
@@ -48,15 +48,15 @@ async function crawlPlusFiles(userRootDir, outDirAbsoluteFilesystem, isDev) {
     const plusFiles = files.map((p) => {
         p = (0, utils_js_1.toPosixPath)(p);
         (0, utils_js_1.assert)(!p.startsWith(userRootDir));
-        const filePathRelativeToUserRootDir = path_1.default.posix.join('/', p);
-        return { filePathRelativeToUserRootDir };
+        const filePathAbsoluteUserRootDir = path_1.default.posix.join('/', p);
+        return { filePathAbsoluteUserRootDir };
     });
     return plusFiles;
 }
 exports.crawlPlusFiles = crawlPlusFiles;
 // Same as fastGlob() but using `$ git ls-files`
 async function gitLsFiles(userRootDir, outDirRelativeFromUserRootDir) {
-    if (globalObject.gitIsMissing)
+    if (gitIsNotUsable)
         return null;
     const ignoreAsPatterns = getIgnoreAsPatterns(outDirRelativeFromUserRootDir);
     const ignoreAsFilterFn = getIgnoreAsFilterFn(outDirRelativeFromUserRootDir);
@@ -74,14 +74,14 @@ async function gitLsFiles(userRootDir, outDirRelativeFromUserRootDir) {
         ;
         [files, filesDeleted] = await Promise.all([
             // Main command
-            runCmd(cmd, userRootDir),
+            runCmd1(cmd, userRootDir),
             // Get tracked by deleted files
-            runCmd('git ls-files --deleted', userRootDir)
+            runCmd1('git ls-files --deleted', userRootDir)
         ]);
     }
     catch (err) {
-        if (await isGitMissing(userRootDir)) {
-            globalObject.gitIsMissing = true;
+        if (await isGitNotUsable(userRootDir)) {
+            gitIsNotUsable = true;
             return null;
         }
         throw err;
@@ -125,24 +125,53 @@ function getIgnoreAsFilterFn(outDirRelativeFromUserRootDir) {
         !file.includes('.telefunc.') &&
         (outDirRelativeFromUserRootDir === null || !file.startsWith(`${outDirRelativeFromUserRootDir}/`));
 }
-// Whether Git is installed and whether userRootDir is inside a Git repository
-async function isGitMissing(userRootDir) {
-    let res;
-    try {
-        res = await execA('git rev-parse --is-inside-work-tree', { cwd: userRootDir });
+// Whether Git is installed and whether we can use it
+async function isGitNotUsable(userRootDir) {
+    // Check Git version
+    {
+        const res = await runCmd2('git --version', userRootDir);
+        if ('err' in res)
+            return true;
+        let { stdout, stderr } = res;
+        (0, utils_js_1.assert)(stderr === '');
+        const prefix = 'git version ';
+        (0, utils_js_1.assert)(stdout.startsWith(prefix));
+        const gitVersion = stdout.slice(prefix.length);
+        //  - Works with Git 2.43.1 but also (most certainly) with earlier versions.
+        //    - We didn't bother test which is the earliest verision that works.
+        //  - Git 2.32.0 doesn't seem to work: https://github.com/vikejs/vike/discussions/1549
+        //    - Maybe it's because of StackBlitz: looking at the release notes, Git 2.32.0 should be working.
+        if (!(0, utils_js_1.isVersionOrAbove)(gitVersion, '2.43.1'))
+            return true;
     }
-    catch {
-        return true;
+    // Is userRootDir inside a Git repository?
+    {
+        const res = await runCmd2('git rev-parse --is-inside-work-tree', userRootDir);
+        if ('err' in res)
+            return true;
+        let { stdout, stderr } = res;
+        (0, utils_js_1.assert)(stderr === '');
+        (0, utils_js_1.assert)(stdout === 'true');
+        return false;
     }
-    const { stdout, stderr } = res;
-    (0, utils_js_1.assert)(stderr.toString().trim() === '');
-    (0, utils_js_1.assert)(stdout.toString().trim() === 'true');
-    return false;
 }
-async function runCmd(cmd, cwd) {
-    const res = await execA(cmd, { cwd });
+async function runCmd1(cmd, cwd) {
+    const { stdout } = await execA(cmd, { cwd });
     /* Not always true: https://github.com/vikejs/vike/issues/1440#issuecomment-1892831303
     assert(res.stderr === '')
     */
-    return res.stdout.toString().split('\n').filter(Boolean);
+    return stdout.toString().split('\n').filter(Boolean);
+}
+async function runCmd2(cmd, cwd) {
+    let res;
+    try {
+        res = await execA(cmd, { cwd });
+    }
+    catch (err) {
+        return { err };
+    }
+    let { stdout, stderr } = res;
+    stdout = stdout.toString().trim();
+    stderr = stderr.toString().trim();
+    return { stdout, stderr };
 }

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

@@ -10,10 +10,10 @@ const utils_js_1 = require("../../../../utils.js");
  */
 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.js` that extends the Vike extension.
-filePathRelativeToUserRootDir) {
-    (0, utils_js_1.assertPosixPath)(filePathRelativeToUserRootDir);
-    (0, utils_js_1.assert)(filePathRelativeToUserRootDir.startsWith('/'));
-    const locationId = removeFilename(filePathRelativeToUserRootDir);
+filePathAbsoluteUserRootDir) {
+    (0, utils_js_1.assertPosixPath)(filePathAbsoluteUserRootDir);
+    (0, utils_js_1.assert)(filePathAbsoluteUserRootDir.startsWith('/'));
+    const locationId = removeFilename(filePathAbsoluteUserRootDir);
     assertLocationId(locationId);
     return locationId;
 }
@@ -96,8 +96,8 @@ function removeDirectories(somePath, removeDirs) {
         somePath = '/';
     return somePath;
 }
-function removeFilename(filePathRelativeToUserRootDir) {
-    const filePathParts = filePathRelativeToUserRootDir.split('/');
+function removeFilename(filePathAbsoluteUserRootDir) {
+    const filePathParts = filePathAbsoluteUserRootDir.split('/');
     {
         const filename = filePathParts.slice(-1)[0];
         (0, utils_js_1.assert)(filename.includes('.'));