vike 0.4.221 → 0.4.222-commit-207e079

package/dist/cjs/node/api/prepareViteApiCall.js

@@ -49,18 +49,18 @@ const utils_js_1 = require("./utils.js");
 const picocolors_1 = __importDefault(require("@brillout/picocolors"));
 const globalContext_js_1 = require("../runtime/globalContext.js");
 const globalObject = (0, utils_js_1.getGlobalObject)('prepareViteApiCall.ts', {});
-async function prepareViteApiCall(viteConfig, operation) {
+async function prepareViteApiCall(viteConfigFromOptions, operation) {
     clear();
     (0, context_js_1.setContextApiOperation)(operation);
-    return enhanceViteConfig(viteConfig, operation);
+    return enhanceViteConfig(viteConfigFromOptions, operation);
 }
 // For subsequent API calls, e.g. calling prerender() after build()
 function clear() {
     (0, context_js_1.clearContextApiOperation)();
     (0, globalContext_js_1.clearGlobalContext)();
 }
-async function enhanceViteConfig(viteConfig, operation) {
-    const viteInfo = await getInfoFromVite(viteConfig, operation);
+async function enhanceViteConfig(viteConfigFromOptions, operation) {
+    const viteInfo = await getInfoFromVite(viteConfigFromOptions, operation);
     await assertViteRoot2(viteInfo.root, viteInfo.viteConfigEnhanced, operation);
     const vikeConfig = await (0, getVikeConfig_js_1.getVikeConfig2)(viteInfo.root, operation === 'dev', viteInfo.vikeVitePluginOptions);
     const viteConfigEnhanced = addViteSettingsSetByVikeConfig(viteInfo.viteConfigEnhanced, vikeConfig);
@@ -76,6 +76,7 @@ function addViteSettingsSetByVikeConfig(viteConfigEnhanced, vikeConfig) {
     viteConfigs.values.forEach((v) => {
         (0, utils_js_1.assertUsage)((0, utils_js_1.isObject)(v.value), `${v.definedAt} should be an object`);
         viteConfigEnhanced = (0, vite_1.mergeConfig)(viteConfigEnhanced ?? {}, v.value);
+        (0, utils_js_1.assertUsage)(!findVikeVitePlugin(v.value), "Using the +vite setting to add Vike's Vite plugin is forbidden");
     });
     return viteConfigEnhanced;
 }
@@ -85,13 +86,14 @@ async function getViteRoot(operation) {
     (0, utils_js_1.assert)(globalObject.root);
     return globalObject.root;
 }
-async function getInfoFromVite(viteConfig, operation) {
-    const viteConfigFromFile = await loadViteConfigFile(viteConfig, operation);
-    const root = normalizeViteRoot(viteConfigFromFile?.root ?? viteConfig?.root ?? process.cwd());
+async function getInfoFromVite(viteConfigFromOptions, operation) {
+    const viteConfigFromUserViteFile = await loadViteConfigFile(viteConfigFromOptions, operation);
+    const root = normalizeViteRoot(viteConfigFromUserViteFile?.root ?? viteConfigFromOptions?.root ?? process.cwd());
     globalObject.root = root;
     let vikeVitePluginOptions;
-    let viteConfigEnhanced = viteConfig;
-    const found = findVikeVitePlugin([...(viteConfig?.plugins ?? []), ...(viteConfigFromFile?.plugins ?? [])]);
+    let viteConfigEnhanced = viteConfigFromOptions;
+    // If Vike's Vite plugin is found in both viteConfigFromOptions and viteConfigFromUserViteFile then Vike will later throw an error
+    const found = findVikeVitePlugin(viteConfigFromOptions) || findVikeVitePlugin(viteConfigFromUserViteFile);
     if (found) {
         vikeVitePluginOptions = found.vikeVitePluginOptions;
     }
@@ -100,20 +102,20 @@ async function getInfoFromVite(viteConfig, operation) {
         // Using a dynamic import because the script calling the Vike API may not live in the same place as vite.config.js, thus vike/plugin may resolved to two different node_modules/vike directories.
         const { plugin: vikePlugin } = await Promise.resolve().then(() => __importStar(require('../plugin/index.js')));
         viteConfigEnhanced = {
-            ...viteConfig,
-            plugins: [...(viteConfig?.plugins ?? []), vikePlugin()]
+            ...viteConfigFromOptions,
+            plugins: [...(viteConfigFromOptions?.plugins ?? []), vikePlugin()]
         };
-        const res = findVikeVitePlugin(viteConfigEnhanced.plugins);
+        const res = findVikeVitePlugin(viteConfigEnhanced);
         (0, utils_js_1.assert)(res);
         vikeVitePluginOptions = res.vikeVitePluginOptions;
     }
     (0, utils_js_1.assert)(vikeVitePluginOptions);
     return { root, vikeVitePluginOptions, viteConfigEnhanced };
 }
-function findVikeVitePlugin(plugins) {
+function findVikeVitePlugin(viteConfig) {
     let vikeVitePluginOptions;
     let vikeVitePuginFound = false;
-    plugins.forEach((p) => {
+    viteConfig?.plugins?.forEach((p) => {
         if (p && '__vikeVitePluginOptions' in p) {
             vikeVitePuginFound = true;
             const options = p.__vikeVitePluginOptions;
@@ -126,8 +128,8 @@ function findVikeVitePlugin(plugins) {
     return { vikeVitePluginOptions };
 }
 // Copied from https://github.com/vitejs/vite/blob/4f5845a3182fc950eb9cd76d7161698383113b18/packages/vite/src/node/config.ts#L961-L1005
-async function loadViteConfigFile(viteConfig, operation) {
-    const [inlineConfig, command, defaultMode, _defaultNodeEnv, isPreview] = getResolveConfigArgs(viteConfig, operation);
+async function loadViteConfigFile(viteConfigFromOptions, operation) {
+    const [inlineConfig, command, defaultMode, _defaultNodeEnv, isPreview] = getResolveConfigArgs(viteConfigFromOptions, operation);
     let config = inlineConfig;
     let mode = inlineConfig.mode || defaultMode;
     const configEnv = {

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

@@ -35,7 +35,7 @@ const workaroundVite6HmrRegression_js_1 = require("./plugins/workaroundVite6HmrR
 const buildApp_js_1 = require("./plugins/buildApp.js");
 // We don't call this in ./onLoad.ts to avoid a cyclic dependency with utils.ts
 (0, getPageAssets_js_1.setResolveClientEntriesDev)(resolveClientEntriesDev_js_1.resolveClientEntriesDev);
-// Return as `any` to avoid Plugin type mismatches when there are multiple Vite versions installed
+// Return `PluginInterop` instead of `Plugin` to avoid type mismatch upon different Vite versions
 function plugin(vikeVitePluginOptions = {}) {
     const plugins = [
         ...(0, commonConfig_js_1.commonConfig)(vikeVitePluginOptions),

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

@@ -29,7 +29,7 @@ function baseUrls() {
         },
         async configResolved(config) {
             const vikeConfig = await (0, getVikeConfig_js_1.getVikeConfig)(config);
-            const basesResolved2 = (0, resolveBase_js_1.resolveBaseFromResolvedConfig)(vikeConfig.global.config.baseServer ?? null, vikeConfig.global.config.baseAssets ?? null, config);
+            const basesResolved2 = resolveBaseFromResolvedConfig(vikeConfig.global.config.baseServer ?? null, vikeConfig.global.config.baseAssets ?? null, config);
             (0, utils_js_1.assert)(basesResolved2.baseServer === basesResolved.baseServer);
             (0, utils_js_1.assert)(basesResolved2.baseAssets === basesResolved.baseAssets);
             /* In dev, Vite seems buggy around setting vite.config.js#base to an absolute URL (e.g. http://localhost:8080/cdn/)
@@ -40,3 +40,8 @@ function baseUrls() {
         }
     };
 }
+function resolveBaseFromResolvedConfig(baseServer, baseAssets, config) {
+    let baseViteOriginal = config._baseViteOriginal;
+    (0, utils_js_1.assert)(baseViteOriginal === null || typeof baseViteOriginal == 'string');
+    return (0, resolveBase_js_1.resolveBase)(baseViteOriginal, baseServer, baseAssets);
+}

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

@@ -7,7 +7,6 @@ exports.fixServerAssets = fixServerAssets;
 exports.fixServerAssets_isEnabled = fixServerAssets_isEnabled;
 exports.fixServerAssets_assertCssCodeSplit = fixServerAssets_assertCssCodeSplit;
 exports.fixServerAssets_assertCssTarget = fixServerAssets_assertCssTarget;
-exports.fixServerAssets_assertCssTarget_populate = fixServerAssets_assertCssTarget_populate;
 const promises_1 = __importDefault(require("fs/promises"));
 const fs_1 = __importDefault(require("fs"));
 const path_1 = __importDefault(require("path"));
@@ -200,17 +199,15 @@ function fixServerAssets_assertCssCodeSplit(config) {
     (0, utils_js_1.assertWarning)(config.build.cssCodeSplit, `${picocolors_1.default.cyan('build.cssCodeSplit')} shouldn't be set to ${picocolors_1.default.cyan('false')} (https://github.com/vikejs/vike/issues/1993)`, { onlyOnce: true });
 }
 const targets = [];
-function fixServerAssets_assertCssTarget_populate(config) {
-    const isServerSide = (0, viteIsSSR_js_1.viteIsSSR)(config);
-    (0, utils_js_1.assert)(typeof isServerSide === 'boolean');
-    (0, utils_js_1.assert)(config.build.target !== undefined);
-    targets.push({ global: config.build.target, css: config.build.cssTarget, isServerSide });
-}
 async function fixServerAssets_assertCssTarget(config) {
     if (!fixServerAssets_isEnabled())
         return;
     if (!(await (0, getVikeConfig_js_1.isV1Design)(config)))
         return;
+    const isServerSide = (0, viteIsSSR_js_1.viteIsSSR)(config);
+    (0, utils_js_1.assert)(typeof isServerSide === 'boolean');
+    (0, utils_js_1.assert)(config.build.target !== undefined);
+    targets.push({ global: config.build.target, css: config.build.cssTarget, isServerSide });
     const targetsServer = targets.filter((t) => t.isServerSide);
     const targetsClient = targets.filter((t) => !t.isServerSide);
     targetsClient.forEach((targetClient) => {
@@ -218,9 +215,10 @@ async function fixServerAssets_assertCssTarget(config) {
         targetsServer.forEach((targetServer) => {
             const targetCssResolvedServer = resolveCssTarget(targetServer);
             (0, utils_js_1.assertWarning)((0, utils_js_1.isEqualStringList)(targetCssResolvedClient, targetCssResolvedServer), [
-                'The CSS browser target should be the same for both client-side and server-side (https://github.com/vikejs/vike/issues/1815#issuecomment-2507002979) but we got:',
-                `Client-side: ${picocolors_1.default.cyan(JSON.stringify(targetCssResolvedClient))}`,
-                `Server-side: ${picocolors_1.default.cyan(JSON.stringify(targetCssResolvedServer))}`
+                'The CSS browser target should be the same for both client and server, but we got:',
+                `Client: ${picocolors_1.default.cyan(JSON.stringify(targetCssResolvedClient))}`,
+                `Server: ${picocolors_1.default.cyan(JSON.stringify(targetCssResolvedServer))}`,
+                `Different targets lead to CSS duplication, see ${picocolors_1.default.underline('https://github.com/vikejs/vike/issues/1815#issuecomment-2507002979')} for more information.`
             ].join('\n'), {
                 showStackTrace: true,
                 onlyOnce: 'different-css-target'

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

@@ -54,7 +54,6 @@ function buildConfig() {
                             config.build.ssrEmitAssets = true;
                             // Required if `ssrEmitAssets: true`, see https://github.com/vitejs/vite/pull/11430#issuecomment-1454800934
                             config.build.cssMinify = 'esbuild';
-                            (0, fixServerAssets_js_1.fixServerAssets_assertCssTarget_populate)(config);
                             (0, fixServerAssets_js_1.fixServerAssets_assertCssCodeSplit)(config);
                         }
                     }

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

@@ -35,7 +35,10 @@ function commonConfig(vikeVitePluginOptions) {
                         _isDev: isDev,
                         _root: root,
                         _vikeVitePluginOptions: vikeVitePluginOptions,
-                        vike: { config: vikeConfig.global.config },
+                        vike: {
+                            pages: vikeConfig.pages,
+                            config: vikeConfig.global.config
+                        },
                         // TODO/v1-release: remove https://github.com/vikejs/vike/issues/2122
                         configVikePromise: Promise.resolve({
                             prerender: (0, context_js_2.isPrerenderEnabled)(vikeConfig)

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

@@ -15,8 +15,8 @@ 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");
-const getPlusFilesAll_js_1 = require("./v1-design/getVikeConfig/getPlusFilesAll.js");
 const globalContext_js_1 = require("../../../runtime/globalContext.js");
+const crawlPlusFiles_js_1 = require("./v1-design/getVikeConfig/crawlPlusFiles.js");
 function importUserCode() {
     let config;
     let vikeConfig;
@@ -73,50 +73,55 @@ function handleFileAddRemove(server, config) {
     return;
     function listener(file, isRemove) {
         file = (0, vite_1.normalizePath)(file);
-        const isVikeConfig = isVikeConfigModule(file) || isVikeConfigFile(file);
-        if (isVikeConfig) {
-            const virtualModules = getVirtualModules(server);
-            virtualModules.forEach((mod) => {
-                server.moduleGraph.invalidateModule(mod);
-            });
+        if ((0, crawlPlusFiles_js_1.isPlusFile)(file) || isVikeConfigDependency(file, server.moduleGraph)?.modifiesVikeVirtualFiles) {
+            invalidateVikeVirtualFiles(server);
             reloadConfig(file, config, isRemove ? 'removed' : 'created');
         }
     }
 }
+function invalidateVikeVirtualFiles(server) {
+    const vikeVirtualFiles = getVikeVirtualFiles(server);
+    vikeVirtualFiles.forEach((mod) => {
+        server.moduleGraph.invalidateModule(mod);
+    });
+}
 function handleHotUpdate(ctx, config) {
     const { file, server } = ctx;
-    const isVikeConfig = isVikeConfigModule(file);
-    const isViteModule = ctx.modules.length > 0;
-    /* Should we show this?
-    // - Can be useful for server files that aren't processed by Vite.
-    // - Can be annoying for files that obviously aren't processed by Vite.
-    if (!isVikeConfig && !isViteModule) {
-      logViteAny(
-        `${msg} — ${pc.cyan('no HMR')}, see https://vike.dev/on-demand-compiler`,
-        'info',
-        null,
-        true
-      )
-      return
-    }
-    //*/
-    // It can hide an error it shouldn't hide (because the error isn't shown again), but it's ok since users can reload the page and the error will be shown again (Vite transpilation errors are shown again upon a page reload).
-    if (!isVikeConfig && isViteModule) {
-        return;
-    }
+    const isVikeConfig = isVikeConfigDependency(ctx.file, ctx.server.moduleGraph);
     if (isVikeConfig) {
-        /* Tailwind breaks this assertion, see https://github.com/vikejs/vike/discussions/1330#discussioncomment-7787238
-        assert(!isViteModule)
-        */
-        reloadConfig(file, config, 'modified');
-        const virtualModules = getVirtualModules(server);
-        return virtualModules;
+        if (isVikeConfig.modifiesVikeVirtualFiles) {
+            /* Tailwind breaks this assertion, see https://github.com/vikejs/vike/discussions/1330#discussioncomment-7787238
+            const isViteModule = ctx.modules.length > 0
+            assert(!isViteModule)
+            */
+            // Ensure server.ssrLoadModule() loads fresh Vike virtual files (`reloadConfig()` > `updateUserFiles()` > `server.ssrLoadModule()`)
+            invalidateVikeVirtualFiles(server);
+            reloadConfig(file, config, 'modified');
+            // Triggers a full page reload
+            const vikeVirtualFiles = getVikeVirtualFiles(server);
+            return vikeVirtualFiles;
+        }
+        else {
+            // Ensure we invalidate `file` *before* server.ssrLoadModule() in updateUserFiles()
+            // Vite already invalidates it, but possibly *after* handleHotUpdate() and thus after server.ssrLoadModule()
+            ctx.modules.forEach((mod) => server.moduleGraph.invalidateModule(mod));
+            (0, globalContext_js_1.updateUserFiles)();
+        }
     }
 }
-function isVikeConfigModule(filePathAbsoluteFilesystem) {
+function isVikeConfigDependency(filePathAbsoluteFilesystem, moduleGraph) {
+    // Check config-only files, for example all pages/+config.js dependencies. (There aren't part of Vite's module graph.)
     (0, utils_js_1.assertPosixPath)(filePathAbsoluteFilesystem);
     getVikeConfig_js_1.vikeConfigDependencies.forEach((f) => (0, utils_js_1.assertPosixPath)(f));
-    return getVikeConfig_js_1.vikeConfigDependencies.has(filePathAbsoluteFilesystem);
+    if (getVikeConfig_js_1.vikeConfigDependencies.has(filePathAbsoluteFilesystem))
+        return { modifiesVikeVirtualFiles: true };
+    // Check using Vite's module graph, for example all +htmlAttributes dependencies.
+    // Alternatively, simply call updateUserFiles() on every handleHotUpdate() call.
+    const importers = getImporters(filePathAbsoluteFilesystem, moduleGraph);
+    const isPlusValueFileDependency = Array.from(importers).some((importer) => importer.file && (0, crawlPlusFiles_js_1.isPlusFile)(importer.file));
+    if (isPlusValueFileDependency)
+        return { modifiesVikeVirtualFiles: false };
+    return null;
 }
 function reloadConfig(filePath, config, op) {
     {
@@ -127,16 +132,45 @@ function reloadConfig(filePath, config, op) {
     (0, getVikeConfig_js_1.reloadVikeConfig)(config);
     (0, globalContext_js_1.updateUserFiles)();
 }
-function getVirtualModules(server) {
-    const virtualModules = Array.from(server.moduleGraph.urlToModuleMap.keys())
+function getVikeVirtualFiles(server) {
+    const vikeVirtualFiles = Array.from(server.moduleGraph.urlToModuleMap.keys())
         .filter((url) => (0, virtualFilePageConfigValuesAll_js_1.isVirtualFileIdPageConfigValuesAll)(url) || (0, virtualFileImportUserCode_js_1.isVirtualFileIdImportUserCode)(url))
         .map((url) => {
         const mod = server.moduleGraph.urlToModuleMap.get(url);
         (0, utils_js_1.assert)(mod);
         return mod;
     });
-    return virtualModules;
+    return vikeVirtualFiles;
+}
+// Get all transitive importers (including the module itself)
+function getImporters(file, moduleGraph) {
+    const importers = new Set();
+    const mods = moduleGraph.getModulesByFile(file);
+    if (!mods)
+        return importers;
+    for (const mod of mods) {
+        getModuleImporters(mod).forEach((importer) => {
+            if (importer)
+                importers.add(importer);
+        });
+    }
+    return importers;
 }
-function isVikeConfigFile(filePath) {
-    return !!(0, getPlusFilesAll_js_1.getPlusFileValueConfigName)(filePath);
+function getModuleImporters(mod, seen = new Set()) {
+    if (seen.has(mod))
+        return new Set();
+    seen.add(mod);
+    const importers = new Set();
+    if (mod.id)
+        importers.add(mod);
+    // Traverse through the importers (modules that import this module)
+    for (const importer of mod.importers) {
+        if (importer.id)
+            importers.add(importer);
+        getModuleImporters(importer, seen).forEach((importerTransitive) => {
+            if (importerTransitive)
+                importers.add(importerTransitive);
+        });
+    }
+    return importers;
 }

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

@@ -4,6 +4,8 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 };
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.crawlPlusFiles = crawlPlusFiles;
+exports.isPlusFile = isPlusFile;
+exports.getPlusFileValueConfigName = getPlusFileValueConfigName;
 const utils_js_1 = require("../../../../utils.js");
 const path_1 = __importDefault(require("path"));
 const fast_glob_1 = __importDefault(require("fast-glob"));
@@ -12,6 +14,7 @@ const util_1 = require("util");
 const transpileAndExecuteFile_js_1 = require("./transpileAndExecuteFile.js");
 const getEnvVarObject_js_1 = require("../../../../shared/getEnvVarObject.js");
 const execA = (0, util_1.promisify)(child_process_1.exec);
+const debug = (0, utils_js_1.createDebugger)('vike:crawl');
 (0, utils_js_1.assertIsNotProductionRuntime)();
 (0, utils_js_1.assertIsSingleModuleInstance)('crawlPlusFiles.ts');
 let gitIsNotUsable = false;
@@ -35,17 +38,16 @@ async function crawlPlusFiles(userRootDir, outDirAbsoluteFilesystem) {
             //
             !outDirRelativeFromUserRootDir.startsWith('../')));
     // Crawl
-    let files;
-    const res = !isGitCrawlDisabled() && (await gitLsFiles(userRootDir, outDirRelativeFromUserRootDir));
-    if (res &&
-        // Fallback to fast-glob for users that dynamically generate plus files. (Assuming that no plus file is found because of the user's .gitignore list.)
-        res.length > 0) {
-        files = res;
-        // We cannot find files inside symlink directories with `$ git ls-files` => we use fast-glob
-    }
-    else {
-        files = await fastGlob(userRootDir, outDirRelativeFromUserRootDir);
-    }
+    const filesGit = !isGitCrawlDisabled() && (await gitLsFiles(userRootDir, outDirRelativeFromUserRootDir));
+    const filesGitNothingFound = !filesGit || filesGit.length === 0;
+    const filesGlob = (filesGitNothingFound || debug.isActivated) && (await fastGlob(userRootDir, outDirRelativeFromUserRootDir));
+    let files = !filesGitNothingFound
+        ? filesGit
+        : // Fallback to fast-glob for users that dynamically generate plus files. (Assuming that no plus file is found because of the user's .gitignore list.)
+            filesGlob;
+    (0, utils_js_1.assert)(files);
+    if (debug.isActivated)
+        (0, utils_js_1.assert)((0, utils_js_1.deepEqual)(filesGlob, filesGit), "Git and glob results aren't matching.");
     // Filter build files
     files = files.filter((filePath) => !(0, transpileAndExecuteFile_js_1.isTemporaryBuildFile)(filePath));
     // Normalize
@@ -54,6 +56,7 @@ async function crawlPlusFiles(userRootDir, outDirAbsoluteFilesystem) {
         (0, utils_js_1.assertPosixPath)(filePath);
         (0, utils_js_1.assert)(!filePath.startsWith(userRootDir));
         const filePathAbsoluteUserRootDir = path_1.default.posix.join('/', filePath);
+        (0, utils_js_1.assert)(isPlusFile(filePathAbsoluteUserRootDir));
         return { filePathAbsoluteUserRootDir };
     });
     return plusFiles;
@@ -74,7 +77,7 @@ async function gitLsFiles(userRootDir, outDirRelativeFromUserRootDir) {
         preserveUTF8,
         'ls-files',
         // Performance gain seems negligible: https://github.com/vikejs/vike/pull/1688#issuecomment-2166206648
-        ...utils_js_1.scriptFileExtensionList.map((ext) => `"**/+*.${ext}"`),
+        ...utils_js_1.scriptFileExtensionList.map((ext) => `"**/+*.${ext}" "+*.${ext}"`),
         // Performance gain is non-negligible.
         //  - https://github.com/vikejs/vike/pull/1688#issuecomment-2166206648
         //  - When node_modules/ is untracked the performance gain could be significant?
@@ -101,6 +104,12 @@ async function gitLsFiles(userRootDir, outDirRelativeFromUserRootDir) {
         }
         throw err;
     }
+    if (debug.isActivated) {
+        debug('[git] userRootDir:', userRootDir);
+        debug('[git] cmd:', cmd);
+        debug('[git] result:', filesAll);
+        debug('[git] filesDeleted:', filesDeleted);
+    }
     const files = [];
     for (const filePath of filesAll) {
         // + file?
@@ -121,14 +130,21 @@ async function gitLsFiles(userRootDir, outDirRelativeFromUserRootDir) {
 }
 // Same as gitLsFiles() but using fast-glob
 async function fastGlob(userRootDir, outDirRelativeFromUserRootDir) {
-    const files = await (0, fast_glob_1.default)(`**/+*.${utils_js_1.scriptFileExtensions}`, {
+    const pattern = `**/+*.${utils_js_1.scriptFileExtensions}`;
+    const options = {
         ignore: getIgnoreAsPatterns(outDirRelativeFromUserRootDir),
         cwd: userRootDir,
         dot: false
-    });
+    };
+    const files = await (0, fast_glob_1.default)(pattern, options);
     // Make build deterministic, in order to get a stable generated hash for dist/client/assets/entries/entry-client-routing.${hash}.js
     // https://github.com/vikejs/vike/pull/1750
     files.sort();
+    if (debug.isActivated) {
+        debug('[glob] pattern:', pattern);
+        debug('[glob] options:', options);
+        debug('[glob] result:', files);
+    }
     return files;
 }
 // Same as getIgnoreAsFilterFn() but as glob pattern
@@ -215,3 +231,37 @@ function isGitCrawlDisabled() {
     const crawSettings = (0, getEnvVarObject_js_1.getEnvVarObject)('VIKE_CRAWL');
     return crawSettings?.git === false;
 }
+function isPlusFile(filePath) {
+    (0, utils_js_1.assertPosixPath)(filePath);
+    if ((0, transpileAndExecuteFile_js_1.isTemporaryBuildFile)(filePath))
+        return false;
+    const fileName = filePath.split('/').pop();
+    return fileName.startsWith('+');
+}
+function getPlusFileValueConfigName(filePath) {
+    if (!isPlusFile(filePath))
+        return null;
+    const fileName = path_1.default.posix.basename(filePath);
+    // assertNoUnexpectedPlusSign(filePath, fileName)
+    const basename = fileName.split('.')[0];
+    (0, utils_js_1.assert)(basename.startsWith('+'));
+    const configName = basename.slice(1);
+    (0, utils_js_1.assertUsage)(configName !== '', `${filePath} Invalid filename ${fileName}`);
+    return configName;
+}
+/* https://github.com/vikejs/vike/issues/1407
+function assertNoUnexpectedPlusSign(filePath: string, fileName: string) {
+  const dirs = path.posix.dirname(filePath).split('/')
+  dirs.forEach((dir, i) => {
+    const dirPath = dirs.slice(0, i + 1).join('/')
+    assertUsage(
+      !dir.includes('+'),
+      `Character '+' is a reserved character: remove '+' from the directory name ${dirPath}/`
+    )
+  })
+  assertUsage(
+    !fileName.slice(1).includes('+'),
+    `Character '+' is only allowed at the beginning of filenames: make sure ${filePath} doesn't contain any '+' in its filename other than its first letter`
+  )
+}
+*/

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

@@ -8,7 +8,6 @@ exports.getFilesystemRouteDefinedBy = getFilesystemRouteDefinedBy;
 exports.isInherited = isInherited;
 exports.getLocationId = getLocationId;
 exports.sortAfterInheritanceOrder = sortAfterInheritanceOrder;
-exports.isGlobalLocation = isGlobalLocation;
 exports.applyFilesystemRoutingRootEffect = applyFilesystemRoutingRootEffect;
 exports.getLogicalPath = getLogicalPath;
 const picocolors_1 = __importDefault(require("@brillout/picocolors"));
@@ -45,10 +44,6 @@ function getLogicalPath(locationId, ignoredDirs, removeParenthesesDirs) {
     assertIsPath(logicalPath);
     return logicalPath;
 }
-/** Whether configs defined in `locationId` apply in every `locationIds` */
-function isGlobalLocation(locationId, locationIds) {
-    return locationIds.every((locId) => isInherited(locationId, locId) || locationIsRendererDir(locId));
-}
 function sortAfterInheritanceOrder(locationId1, locationId2, locationIdPage) {
     assertLocationId(locationId1);
     assertLocationId(locationId2);

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

@@ -1,27 +1,21 @@
 "use strict";
-var __importDefault = (this && this.__importDefault) || function (mod) {
-    return (mod && mod.__esModule) ? mod : { "default": mod };
-};
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.getPlusFilesAll = getPlusFilesAll;
-exports.getPlusFileValueConfigName = getPlusFileValueConfigName;
 const utils_js_1 = require("../../../../utils.js");
 const configDefinitionsBuiltIn_js_1 = require("./configDefinitionsBuiltIn.js");
 const filesystemRouting_js_1 = require("./filesystemRouting.js");
-const transpileAndExecuteFile_js_1 = require("./transpileAndExecuteFile.js");
 const crawlPlusFiles_js_1 = require("./crawlPlusFiles.js");
 const getConfigFileExport_js_1 = require("./getConfigFileExport.js");
 const loadFileAtConfigTime_js_1 = require("./loadFileAtConfigTime.js");
 const resolvePointerImport_js_1 = require("./resolvePointerImport.js");
 const getFilePath_js_1 = require("../../../../shared/getFilePath.js");
 const assertExtensions_js_1 = require("./assertExtensions.js");
-const node_path_1 = __importDefault(require("node:path"));
 async function getPlusFilesAll(userRootDir, esbuildCache) {
     const plusFiles = await findPlusFiles(userRootDir, null);
     const configFiles = [];
     const valueFiles = [];
     plusFiles.forEach((f) => {
-        if (getPlusFileValueConfigName(f.filePathAbsoluteFilesystem) === 'config') {
+        if ((0, crawlPlusFiles_js_1.getPlusFileValueConfigName)(f.filePathAbsoluteFilesystem) === 'config') {
             configFiles.push(f);
         }
         else {
@@ -66,7 +60,7 @@ async function getPlusFilesAll(userRootDir, esbuildCache) {
         ...valueFiles.map(async (filePath) => {
             const { filePathAbsoluteUserRootDir } = filePath;
             (0, utils_js_1.assert)(filePathAbsoluteUserRootDir);
-            const configName = getPlusFileValueConfigName(filePathAbsoluteUserRootDir);
+            const configName = (0, crawlPlusFiles_js_1.getPlusFileValueConfigName)(filePathAbsoluteUserRootDir);
             (0, utils_js_1.assert)(configName);
             const locationId = (0, filesystemRouting_js_1.getLocationId)(filePathAbsoluteUserRootDir);
             const plusFile = {
@@ -125,35 +119,3 @@ async function findPlusFiles(userRootDir, outDirRoot) {
     const plusFiles = files.map(({ filePathAbsoluteUserRootDir }) => (0, getFilePath_js_1.getFilePathResolved)({ filePathAbsoluteUserRootDir, userRootDir }));
     return plusFiles;
 }
-function getPlusFileValueConfigName(filePath) {
-    (0, utils_js_1.assertPosixPath)(filePath);
-    if ((0, transpileAndExecuteFile_js_1.isTemporaryBuildFile)(filePath))
-        return null;
-    const fileName = node_path_1.default.posix.basename(filePath);
-    // assertNoUnexpectedPlusSign(filePath, fileName)
-    const basename = fileName.split('.')[0];
-    if (!basename.startsWith('+')) {
-        return null;
-    }
-    else {
-        const configName = basename.slice(1);
-        (0, utils_js_1.assertUsage)(configName !== '', `${filePath} Invalid filename ${fileName}`);
-        return configName;
-    }
-}
-/* https://github.com/vikejs/vike/issues/1407
-function assertNoUnexpectedPlusSign(filePath: string, fileName: string) {
-  const dirs = path.posix.dirname(filePath).split('/')
-  dirs.forEach((dir, i) => {
-    const dirPath = dirs.slice(0, i + 1).join('/')
-    assertUsage(
-      !dir.includes('+'),
-      `Character '+' is a reserved character: remove '+' from the directory name ${dirPath}/`
-    )
-  })
-  assertUsage(
-    !fileName.slice(1).includes('+'),
-    `Character '+' is only allowed at the beginning of filenames: make sure ${filePath} doesn't contain any '+' in its filename other than its first letter`
-  )
-}
-*/

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

@@ -56,7 +56,7 @@ async function loadValueFile(interfaceValueFile, configDefinitions, userRootDir,
 async function loadConfigFile(configFilePath, userRootDir, visited, isExtensionConfig, esbuildCache) {
     const { filePathAbsoluteFilesystem } = configFilePath;
     assertNoInfiniteLoop(visited, filePathAbsoluteFilesystem);
-    const { fileExports } = await (0, transpileAndExecuteFile_js_1.transpileAndExecuteFile)(configFilePath, userRootDir, isExtensionConfig ? 'is-extension-config' : true, esbuildCache);
+    const { fileExports } = await (0, transpileAndExecuteFile_js_1.transpileAndExecuteFile)(configFilePath, userRootDir, isExtensionConfig, esbuildCache);
     const { extendsConfigs, extendsFilePaths } = await loadExtendsConfigs(fileExports, configFilePath, userRootDir, [...visited, filePathAbsoluteFilesystem], esbuildCache);
     const configFile = {
         fileExports,

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

@@ -26,8 +26,9 @@ const debug = (0, utils_js_1.createDebugger)('vike:pointer-imports');
 const debugEsbuildResolve = (0, utils_js_1.createDebugger)('vike:esbuild-resolve');
 if (debugEsbuildResolve.isActivated)
     debugEsbuildResolve('esbuild version', esbuild_1.version);
-async function transpileAndExecuteFile(filePath, userRootDir, isConfigFile, esbuildCache) {
+async function transpileAndExecuteFile(filePath, userRootDir, isExtensionConfig, esbuildCache) {
     const { filePathAbsoluteFilesystem, filePathToShowToUserResolved } = filePath;
+    (0, utils_js_1.assert)(filePathAbsoluteFilesystem);
     const fileExtension = getFileExtension(filePathAbsoluteFilesystem);
     if (esbuildCache[filePathAbsoluteFilesystem]) {
         return await esbuildCache[filePathAbsoluteFilesystem];
@@ -40,12 +41,12 @@ async function transpileAndExecuteFile(filePath, userRootDir, isConfigFile, esbu
         (0, utils_js_1.assertWarning)(false, `${picocolors_1.default.cyan('.h.js')} files are deprecated: simply renaming ${filePathToShowToUserResolved} to ${removeHeaderFileExtension(filePathToShowToUserResolved)} is usually enough, although you may occasionally need to use ${picocolors_1.default.cyan("with { type: 'pointer' }")} as explained at https://vike.dev/config#pointer-imports`, { onlyOnce: true });
     }
     let fileExports;
-    if (isConfigFile === 'is-extension-config' && !isHeader && fileExtension.endsWith('js')) {
+    if (isExtensionConfig && !isHeader && fileExtension.endsWith('js')) {
         // This doesn't track dependencies => we should never use this for user land configs
         fileExports = await executeFile(filePathAbsoluteFilesystem, filePath);
     }
     else {
-        const transformImports = isConfigFile && (isHeader ? 'all' : true);
+        const transformImports = isHeader ? 'all' : true;
         const code = await transpileFile(filePath, transformImports, userRootDir);
         fileExports = await executeTranspiledFile(filePath, code);
     }
@@ -54,6 +55,7 @@ async function transpileAndExecuteFile(filePath, userRootDir, isConfigFile, esbu
 }
 async function transpileFile(filePath, transformImports, userRootDir) {
     const { filePathAbsoluteFilesystem, filePathToShowToUserResolved } = filePath;
+    (0, utils_js_1.assert)(filePathAbsoluteFilesystem);
     (0, utils_js_1.assertPosixPath)(filePathAbsoluteFilesystem);
     getVikeConfig_js_1.vikeConfigDependencies.add(filePathAbsoluteFilesystem);
     if (debug.isActivated)
@@ -338,6 +340,7 @@ function isTemporaryBuildFile(filePath) {
     const fileName = path_1.default.posix.basename(filePath);
     return /\.build-[a-z0-9]{12}\.mjs$/.test(fileName);
 }
+// TODO/next-major: remove
 function isHeaderFile(filePath) {
     (0, utils_js_1.assertPosixPath)(filePath);
     const fileExtensions = getFileExtensions(filePath);