vike 0.4.229-commit-2dbbc8d → 0.4.229-commit-845a4e0

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

@@ -58,6 +58,12 @@ const configDefinitionsBuiltIn = {
     data: {
         env: { server: true }
     },
+    onData: {
+        env: { server: true, client: true }
+        /* TODO/now
+        cumulative: true
+        */
+    },
     iKnowThePerformanceRisksOfAsyncRouteFunctions: {
         env: { server: true, client: 'if-client-routing' },
         eager: true

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

@@ -5,7 +5,7 @@ exports.createHttpResponsePageContextJson = createHttpResponsePageContextJson;
 exports.createHttpResponseError = createHttpResponseError;
 exports.createHttpResponseErrorWithoutGlobalContext = createHttpResponseErrorWithoutGlobalContext;
 exports.createHttpResponseRedirect = createHttpResponseRedirect;
-exports.createHttpResponseFavicon404 = createHttpResponseFavicon404;
+exports.createHttpResponse404 = createHttpResponse404;
 exports.createHttpResponseBaseIsMissing = createHttpResponseBaseIsMissing;
 const utils_js_1 = require("../utils.js");
 const error_page_js_1 = require("../../../shared/error-page.js");
@@ -38,8 +38,8 @@ async function createHttpResponsePage(htmlRender, renderHook, pageContext) {
     }
     return createHttpResponse(statusCode, 'text/html;charset=utf-8', headers, htmlRender, earlyHints, renderHook);
 }
-function createHttpResponseFavicon404() {
-    const httpResponse = createHttpResponse(404, 'text/html;charset=utf-8', [], "<p>No favicon.ico found.</p><script>console.log('This HTTP response was generated by Vike.')</script>");
+function createHttpResponse404(errMsg404) {
+    const httpResponse = createHttpResponse(404, 'text/html;charset=utf-8', [], `<p>${errMsg404}.</p><script>console.log('This HTTP response was generated by Vike.')</script>`);
     return httpResponse;
 }
 function createHttpResponseBaseIsMissing(urlOriginal, baseServer) {

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

@@ -22,6 +22,13 @@ async function executeOnBeforeRenderAndDataHooks(pageContext) {
             data: hookResult
         };
         Object.assign(pageContext, pageContextFromHook);
+        // Execute +onData
+        if (!pageContext.isClientSideNavigation) {
+            const onDataHook = (0, getHook_js_1.getHookFromPageContext)(pageContext, 'onData');
+            if (onDataHook) {
+                await (0, executeHook_js_1.executeHook)(() => onDataHook.hookFn(pageContext), dataHook, pageContext);
+            }
+        }
     }
     if (onBeforeRenderHook) {
         const hookResult = await (0, executeHook_js_1.executeHook)(() => onBeforeRenderHook.hookFn(pageContext), onBeforeRenderHook, pageContext);

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

@@ -33,9 +33,9 @@ async function renderPage(pageContextInit) {
     (0, utils_js_1.assert)((0, utils_js_1.hasProp)(pageContextInit, 'urlOriginal', 'string')); // assertUsage() already implemented at assertArguments()
     assertIsUrl(pageContextInit.urlOriginal);
     (0, utils_js_1.onSetupRuntime)();
-    const pageContextInvalidRequest = getPageContextInvalidRequest(pageContextInit);
-    if (pageContextInvalidRequest)
-        return pageContextInvalidRequest;
+    const pageContextSkipRequest = getPageContextSkipRequest(pageContextInit);
+    if (pageContextSkipRequest)
+        return pageContextSkipRequest;
     const httpRequestId = getRequestId();
     const urlOriginalPretty = (0, utils_js_1.getUrlPretty)(pageContextInit.urlOriginal);
     logHttpRequest(urlOriginalPretty, httpRequestId);
@@ -459,13 +459,22 @@ async function checkBaseUrl(pageContextBegin, globalContext) {
     (0, utils_js_1.checkType)(pageContext);
     return pageContext;
 }
-function getPageContextInvalidRequest(pageContextInit) {
+function getPageContextSkipRequest(pageContextInit) {
     const urlPathnameWithBase = (0, utils_js_1.parseUrl)(pageContextInit.urlOriginal, '/').pathname;
     assertIsNotViteRequest(urlPathnameWithBase, pageContextInit.urlOriginal);
-    if (!urlPathnameWithBase.endsWith('/favicon.ico'))
+    let errMsg404;
+    if (urlPathnameWithBase.endsWith('/favicon.ico')) {
+        errMsg404 = 'No favicon.ico found';
+    }
+    if (urlPathnameWithBase.endsWith('.well-known/appspecific/com.chrome.devtools.json')) {
+        // https://chromium.googlesource.com/devtools/devtools-frontend/+/main/docs/ecosystem/automatic_workspace_folders.md
+        // https://www.reddit.com/r/node/comments/1kcr0wh/odd_request_coming_into_my_localhost_server_from/
+        errMsg404 = 'Not spported';
+    }
+    if (!errMsg404)
         return;
     const pageContext = (0, createPageContextServerSide_js_1.createPageContextServerSideWithoutGlobalContext)(pageContextInit);
-    const httpResponse = (0, createHttpResponse_js_1.createHttpResponseFavicon404)();
+    const httpResponse = (0, createHttpResponse_js_1.createHttpResponse404)(errMsg404);
     (0, utils_js_1.objectAssign)(pageContext, { httpResponse });
     (0, utils_js_1.checkType)(pageContext);
     return pageContext;

package/dist/cjs/shared/route/debug.js

@@ -1,13 +1,23 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.debug = debug;
-var _debug;
+exports.setCreateDebugger = setCreateDebugger;
+// Using createDebugger() for isomorphic code without bloating the client-side.
+// On the server-side, this is just a transparent proxy.
+// On the client-side, this is an emtpy shell.
+const getGlobalObject_js_1 = require("../../utils/getGlobalObject.js");
+const globalObject = (0, getGlobalObject_js_1.getGlobalObject)('route/debug.ts', {});
 function debug(...args) {
-    if (!_debug) {
-        // We use this trick instead of `import { createDebugger } from '../../utils/debug` in order to ensure that the `debug` mechanism is only loaded on the server-side
-        _debug = globalThis.__brillout_debug_createDebugger?.('vike:routing');
-    }
-    if (_debug) {
-        _debug(...args);
+    // Client-side => does nothing
+    if (!globalObject.createDebugger)
+        return;
+    // Server-side => just a proxy
+    if (!globalObject.debug) {
+        globalObject.debug = globalObject.createDebugger('vike:routing');
     }
+    globalObject.debug(...args);
+}
+// Called only on the server-side
+function setCreateDebugger(createDebugger) {
+    globalObject.createDebugger = createDebugger;
 }

package/dist/cjs/utils/PROJECT_VERSION.js

@@ -2,4 +2,4 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.PROJECT_VERSION = void 0;
 // Automatically updated by @brillout/release-me
-exports.PROJECT_VERSION = '0.4.229-commit-2dbbc8d';
+exports.PROJECT_VERSION = '0.4.229-commit-845a4e0';

package/dist/cjs/utils/debug.js

@@ -13,11 +13,10 @@ const checkType_js_1 = require("./checkType.js");
 const getTerminWidth_js_1 = require("./getTerminWidth.js");
 const picocolors_1 = __importDefault(require("@brillout/picocolors"));
 const isArray_js_1 = require("./isArray.js");
-// Avoid this to be loaded in the browser. For isomorphic code: instead of `import { createDebugger } from './utils.js'`, use `globalThis.createDebugger()`.
+const isObject_js_1 = require("./isObject.js");
+const debug_js_1 = require("../shared/route/debug.js");
 (0, assert_js_1.assert)(!(0, isBrowser_js_1.isBrowser)());
-globalThis.__brillout_debug_createDebugger = createDebugger;
-// We purposely read process.env.DEBUG early, in order to avoid users from the temptation to set process.env.DEBUG with JavaScript, since reading & writing process.env.DEBUG dynamically leads to inconsistencies: for example https://github.com/vikejs/vike/issues/2239
-const DEBUG = getDEBUG() ?? '';
+(0, debug_js_1.setCreateDebugger)(createDebugger); // for isomorphic code
 const flags = [
     'vike:crawl',
     'vike:error',
@@ -39,6 +38,10 @@ const flags = [
 ];
 const flagsSkipWildcard = ['vike:log'];
 const flagRegex = /\bvike:[a-zA-Z-]+/g;
+// We purposely read process.env.DEBUG early, in order to avoid users from the temptation to set process.env.DEBUG with JavaScript, since reading & writing process.env.DEBUG dynamically leads to inconsistencies such as https://github.com/vikejs/vike/issues/2239
+const DEBUG = getDEBUG() ?? '';
+if (isDebug())
+    Error.stackTraceLimit = Infinity;
 assertFlagsActivated();
 function createDebugger(flag, optionsGlobal) {
     (0, checkType_js_1.checkType)(flag);
@@ -65,9 +68,10 @@ function debug_(flag, options, ...msgs) {
     });
     let logFirst;
     let logsRest;
-    const noNewLine = msgsRest.length <= 1 && [msgFirst, ...msgsRest].every((m) => typeof m === 'string' && !m.includes('\n'));
+    const noNewLine = msgsRest.length <= 1 &&
+        [msgFirst, ...msgsRest].every((m) => (typeof m === 'string' ? !m.includes('\n') : !(0, isObject_js_1.isObject)(m)));
     if (noNewLine) {
-        logFirst = [msgFirst, ...msgsRest].map((m) => String(m).trim());
+        logFirst = [msgFirst, ...msgsRest].map((m) => (typeof m !== 'string' ? m : m.trim()));
         logsRest = [];
     }
     else {
@@ -157,6 +161,10 @@ function getFlagsActivated() {
     const all = DEBUG.includes('vike:*');
     return { flagsActivated, all };
 }
+function isDebug() {
+    const { flagsActivated, all } = getFlagsActivated();
+    return all || flagsActivated.length > 0;
+}
 function getDEBUG() {
     let DEBUG;
     // - `process` can be undefined in edge workers

package/dist/cjs/utils/requireResolve.js

@@ -32,12 +32,12 @@ const debug = (0, debug_js_1.createDebugger)('vike:resolve');
 // - The argument createRequire(argument) seems to be overriden by the `paths` argument require.resolve()
 //   - For example, passing an empty array to `paths` kills the argument passed to `createRequire()`
 //   - Thus, when `paths` is defined, then the context needs to be passed to both createRequire() as well as the `paths` argument of require.resolve()
-function requireResolve_(importPath, importerFilePath, { userRootDir, doNotHandleFileExtension } = {}) {
+function requireResolve_(importPath, importerFilePath, userRootDir, doNotHandleFileExtension = false) {
     (0, path_js_1.assertPosixPath)(importPath);
     const contexts = importerFilePath
         ? [importerFilePath]
         : [userRootDir ? getFakeImporterFile(userRootDir) : importMetaUrl];
-    addExtraContextForNpmPackageImport(contexts, { importPath, userRootDir });
+    addExtraContextForNpmPackageImport(contexts, importPath, userRootDir);
     let importPathResolvedFilePath;
     let failure;
     for (const context of contexts) {
@@ -54,9 +54,8 @@ function requireResolve_(importPath, importerFilePath, { userRootDir, doNotHandl
         }
         catch (err) {
             if (debug.isActivated) {
-                debug('err', err);
-                debug('importPath', importPath);
-                debug('context', context);
+                const stack = new Error().stack;
+                debug('ERROR', { err, importPath, context }, stack);
             }
             failure ?? (failure = { err });
         }
@@ -66,21 +65,23 @@ function requireResolve_(importPath, importerFilePath, { userRootDir, doNotHandl
     if (!importPathResolvedFilePath) {
         (0, assert_js_1.assert)(failure);
         if (debug.isActivated) {
-            debug('FAILURE');
-            debug('importPath', importPath);
-            debug('importerFilePath', importerFilePath);
-            debug('userRootDir', userRootDir);
-            debug('doNotHandleFileExtension', doNotHandleFileExtension);
-            debug('importMetaUrl', importMetaUrl);
-            debug('contexts', contexts);
+            debug('FAILURE', {
+                importPath,
+                importerFilePath,
+                userRootDir,
+                doNotHandleFileExtension,
+                importMetaUrl,
+                contexts
+            });
         }
         return { importPathResolvedFilePath: undefined, err: failure.err, hasFailed: true };
     }
     else {
         if (failure && debug.isActivated) {
-            debug('SUCCESS');
-            debug('importPath', importPath);
-            debug('contexts', contexts);
+            debug('SUCCESS', {
+                importPath,
+                contexts
+            });
         }
         (0, assert_js_1.assert)(importPathResolvedFilePath);
         importPathResolvedFilePath = (0, path_js_1.toPosixPath)(importPathResolvedFilePath);
@@ -88,14 +89,14 @@ function requireResolve_(importPath, importerFilePath, { userRootDir, doNotHandl
     }
 }
 function requireResolveOptional({ importPath, importerFilePath, userRootDir }) {
-    const res = requireResolve_(importPath, importerFilePath, { userRootDir });
+    const res = requireResolve_(importPath, importerFilePath, userRootDir);
     if (res.hasFailed)
         return null;
     return res.importPathResolvedFilePath;
 }
 function requireResolveOptionalDir({ importPath, importerDir, userRootDir }) {
     const importerFilePath = getFakeImporterFile(importerDir);
-    const res = requireResolve_(importPath, importerFilePath, { userRootDir });
+    const res = requireResolve_(importPath, importerFilePath, userRootDir);
     if (res.hasFailed)
         return null;
     return res.importPathResolvedFilePath;
@@ -103,7 +104,7 @@ function requireResolveOptionalDir({ importPath, importerDir, userRootDir }) {
 function requireResolveNpmPackage({ importPathNpmPackage, userRootDir }) {
     (0, parseNpmPackage_js_1.assertIsImportPathNpmPackage)(importPathNpmPackage);
     const importerFilePath = getFakeImporterFile(userRootDir);
-    const res = requireResolve_(importPathNpmPackage, importerFilePath, { userRootDir });
+    const res = requireResolve_(importPathNpmPackage, importerFilePath, userRootDir);
     if (res.hasFailed)
         throw res.err;
     return res.importPathResolvedFilePath;
@@ -117,14 +118,14 @@ function requireResolveVikeDistFile(vikeDistFile) {
     {
         const res = requireResolve_(importPathResolvedFilePath, 
         // No context needed: importPathResolvedFilePath is already resolved and absolute
-        null, { doNotHandleFileExtension: true });
+        null, null, true);
         if (res.hasFailed)
             throw res.err;
         (0, assert_js_1.assert)(res.importPathResolvedFilePath === importPathResolvedFilePath);
     }
     return importPathResolvedFilePath;
 }
-function addExtraContextForNpmPackageImport(contexts, { importPath, userRootDir }) {
+function addExtraContextForNpmPackageImport(contexts, importPath, userRootDir) {
     // We should add extra context only for npm packages, but unfortunately we cannot always disambiguate between npm package imports and path aliases.
     if (!(0, parseNpmPackage_js_1.isImportPathNpmPackageOrPathAlias)(importPath))
         return;

package/dist/esm/client/client-routing-runtime/getPageContextFromHooks.js

@@ -30,7 +30,7 @@ function getPageContextFromHooks_serialized() {
     });
     return pageContextSerialized;
 }
-// TODO/eventually: rename
+// TO-DO/eventually: rename
 async function getPageContextFromHooks_isHydration(pageContext) {
     objectAssign(pageContext, {
         _hasPageContextFromClient: false
@@ -73,6 +73,7 @@ async function getPageContextFromClientHooks(pageContext, isErrorPage) {
     objectAssign(pageContext, {
         _hasPageContextFromClient: false
     });
+    let dataHookExists = false;
     // At this point, we need to call the client-side guard(), data() and onBeforeRender() hooks, if they exist on client
     // env. However if we have fetched pageContext from the server, some of them might have run already on the
     // server-side, so we run only the client-only ones in this case.
@@ -89,7 +90,8 @@ async function getPageContextFromClientHooks(pageContext, isErrorPage) {
             }
         }
         else {
-            assert(hookName === 'data' || hookName === 'onBeforeRender');
+            if (hookName === 'data')
+                dataHookExists = true;
             if (hookClientOnlyExists(hookName, pageContext) || !pageContext._hasPageContextFromServer) {
                 // This won't do anything if no hook has been defined or if the hook's env.client is false.
                 const pageContextFromHook = await executeHookClientSide(hookName, pageContext);
@@ -98,6 +100,14 @@ async function getPageContextFromClientHooks(pageContext, isErrorPage) {
             }
         }
     }
+    // Execute +onData
+    if (dataHookExists) {
+        const hook = getHookFromPageContext(pageContext, 'onData');
+        if (hook) {
+            const pageContextForUserConsumption = preparePageContextForUserConsumptionClientSide(pageContext, true);
+            await executeHook(() => hook.hookFn(pageContextForUserConsumption), hook, pageContext);
+        }
+    }
     const pageContextFromClientHooks = pageContext;
     return pageContextFromClientHooks;
 }

package/dist/esm/client/client-routing-runtime/renderPageClientSide.js

@@ -82,7 +82,7 @@ async function renderPageClientSide(renderArgs) {
         // Route
         if (isFirstRender) {
             const pageContextSerialized = getPageContextFromHooks_serialized();
-            // TODO/eventually: create helper assertPageContextFromHook()
+            // TO-DO/eventually: create helper assertPageContextFromHook()
             assert(!('urlOriginal' in pageContextSerialized));
             objectAssign(pageContext, pageContextSerialized);
             // TODO/pageContext-prefetch: remove or change, because this only makes sense for a pre-rendered page
@@ -127,7 +127,7 @@ async function renderPageClientSide(renderArgs) {
                 // Skip's Vike's rendering; let the user handle the navigation
                 return;
             }
-            // TODO/eventually: create helper assertPageContextFromHook()
+            // TO-DO/eventually: create helper assertPageContextFromHook()
             assert(!('urlOriginal' in pageContextFromRoute));
             objectAssign(pageContext, pageContextFromRoute);
         }
@@ -197,9 +197,24 @@ async function renderPageClientSide(renderArgs) {
             }
             if (isRenderOutdated())
                 return;
-            // TODO/eventually: create helper assertPageContextFromHook()
+            // TO-DO/eventually: create helper assertPageContextFromHook()
             assert(!('urlOriginal' in pageContextFromServerHooks));
             objectAssign(pageContext, pageContextFromServerHooks);
+            // Execute +onData
+            assertHook(pageContext, 'onData');
+            const hook = getHookFromPageContext(pageContext, 'onData');
+            if (hook) {
+                const { hookFn } = hook;
+                try {
+                    await executeHook(() => hookFn(pageContext), hook, pageContext);
+                }
+                catch (err) {
+                    await onError(err);
+                    return;
+                }
+                if (isRenderOutdated())
+                    return;
+            }
             // Get pageContext from client-side hooks
             let pageContextFromClientHooks;
             try {
@@ -365,7 +380,7 @@ async function renderPageClientSide(renderArgs) {
         }
         if (isRenderOutdated())
             return;
-        // TODO/eventually: create helper assertPageContextFromHook()
+        // TO-DO/eventually: create helper assertPageContextFromHook()
         assert(!('urlOriginal' in pageContextFromServerHooks));
         objectAssign(pageContext, pageContextFromServerHooks);
         let pageContextFromClientHooks;

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

@@ -56,6 +56,12 @@ const configDefinitionsBuiltIn = {
     data: {
         env: { server: true }
     },
+    onData: {
+        env: { server: true, client: true }
+        /* TODO/now
+        cumulative: true
+        */
+    },
     iKnowThePerformanceRisksOfAsyncRouteFunctions: {
         env: { server: true, client: 'if-client-routing' },
         eager: true

package/dist/esm/node/runtime/renderPage/createHttpResponse.d.ts

@@ -3,7 +3,7 @@ export { createHttpResponsePageContextJson };
 export { createHttpResponseError };
 export { createHttpResponseErrorWithoutGlobalContext };
 export { createHttpResponseRedirect };
-export { createHttpResponseFavicon404 };
+export { createHttpResponse404 };
 export { createHttpResponseBaseIsMissing };
 export type { HttpResponse };
 import type { GetPageAssets } from './getPageAssets.js';
@@ -30,7 +30,7 @@ declare function createHttpResponsePage(htmlRender: HtmlRender, renderHook: null
     _pageConfigs: PageConfigRuntime[];
     abortStatusCode?: AbortStatusCode;
 }): Promise<HttpResponse>;
-declare function createHttpResponseFavicon404(): HttpResponse;
+declare function createHttpResponse404(errMsg404: string): HttpResponse;
 declare function createHttpResponseBaseIsMissing(urlOriginal: string, baseServer: string): HttpResponse;
 declare function createHttpResponseError(pageContext: {
     _pageFilesAll: PageFile[];

package/dist/esm/node/runtime/renderPage/createHttpResponse.js

@@ -3,7 +3,7 @@ export { createHttpResponsePageContextJson };
 export { createHttpResponseError };
 export { createHttpResponseErrorWithoutGlobalContext };
 export { createHttpResponseRedirect };
-export { createHttpResponseFavicon404 };
+export { createHttpResponse404 };
 export { createHttpResponseBaseIsMissing };
 import { assert, assertWarning, escapeHtml } from '../utils.js';
 import { getErrorPageId, isErrorPage } from '../../../shared/error-page.js';
@@ -36,8 +36,8 @@ async function createHttpResponsePage(htmlRender, renderHook, pageContext) {
     }
     return createHttpResponse(statusCode, 'text/html;charset=utf-8', headers, htmlRender, earlyHints, renderHook);
 }
-function createHttpResponseFavicon404() {
-    const httpResponse = createHttpResponse(404, 'text/html;charset=utf-8', [], "<p>No favicon.ico found.</p><script>console.log('This HTTP response was generated by Vike.')</script>");
+function createHttpResponse404(errMsg404) {
+    const httpResponse = createHttpResponse(404, 'text/html;charset=utf-8', [], `<p>${errMsg404}.</p><script>console.log('This HTTP response was generated by Vike.')</script>`);
     return httpResponse;
 }
 function createHttpResponseBaseIsMissing(urlOriginal, baseServer) {

package/dist/esm/node/runtime/renderPage/executeOnBeforeRenderAndDataHooks.js

@@ -20,6 +20,13 @@ async function executeOnBeforeRenderAndDataHooks(pageContext) {
             data: hookResult
         };
         Object.assign(pageContext, pageContextFromHook);
+        // Execute +onData
+        if (!pageContext.isClientSideNavigation) {
+            const onDataHook = getHookFromPageContext(pageContext, 'onData');
+            if (onDataHook) {
+                await executeHook(() => onDataHook.hookFn(pageContext), dataHook, pageContext);
+            }
+        }
     }
     if (onBeforeRenderHook) {
         const hookResult = await executeHook(() => onBeforeRenderHook.hookFn(pageContext), onBeforeRenderHook, pageContext);

package/dist/esm/node/runtime/renderPage.js

@@ -7,7 +7,7 @@ import { assert, hasProp, objectAssign, isUrl, parseUrl, onSetupRuntime, assertW
 import { assertNoInfiniteAbortLoop, getPageContextFromAllRewrites, isAbortError, logAbortErrorHandled } from '../../shared/route/abort.js';
 import { getGlobalContextServerInternal, initGlobalContext_renderPage } from './globalContext.js';
 import { handlePageContextRequestUrl } from './renderPage/handlePageContextRequestUrl.js';
-import { createHttpResponseFavicon404, createHttpResponseRedirect, createHttpResponsePageContextJson, createHttpResponseError, createHttpResponseErrorWithoutGlobalContext, createHttpResponseBaseIsMissing } from './renderPage/createHttpResponse.js';
+import { createHttpResponse404, createHttpResponseRedirect, createHttpResponsePageContextJson, createHttpResponseError, createHttpResponseErrorWithoutGlobalContext, createHttpResponseBaseIsMissing } from './renderPage/createHttpResponse.js';
 import { logRuntimeError, logRuntimeInfo } from './renderPage/loggerRuntime.js';
 import { isNewError } from './renderPage/isNewError.js';
 import { assertArguments } from './renderPage/assertArguments.js';
@@ -28,9 +28,9 @@ async function renderPage(pageContextInit) {
     assert(hasProp(pageContextInit, 'urlOriginal', 'string')); // assertUsage() already implemented at assertArguments()
     assertIsUrl(pageContextInit.urlOriginal);
     onSetupRuntime();
-    const pageContextInvalidRequest = getPageContextInvalidRequest(pageContextInit);
-    if (pageContextInvalidRequest)
-        return pageContextInvalidRequest;
+    const pageContextSkipRequest = getPageContextSkipRequest(pageContextInit);
+    if (pageContextSkipRequest)
+        return pageContextSkipRequest;
     const httpRequestId = getRequestId();
     const urlOriginalPretty = getUrlPretty(pageContextInit.urlOriginal);
     logHttpRequest(urlOriginalPretty, httpRequestId);
@@ -454,13 +454,22 @@ async function checkBaseUrl(pageContextBegin, globalContext) {
     checkType(pageContext);
     return pageContext;
 }
-function getPageContextInvalidRequest(pageContextInit) {
+function getPageContextSkipRequest(pageContextInit) {
     const urlPathnameWithBase = parseUrl(pageContextInit.urlOriginal, '/').pathname;
     assertIsNotViteRequest(urlPathnameWithBase, pageContextInit.urlOriginal);
-    if (!urlPathnameWithBase.endsWith('/favicon.ico'))
+    let errMsg404;
+    if (urlPathnameWithBase.endsWith('/favicon.ico')) {
+        errMsg404 = 'No favicon.ico found';
+    }
+    if (urlPathnameWithBase.endsWith('.well-known/appspecific/com.chrome.devtools.json')) {
+        // https://chromium.googlesource.com/devtools/devtools-frontend/+/main/docs/ecosystem/automatic_workspace_folders.md
+        // https://www.reddit.com/r/node/comments/1kcr0wh/odd_request_coming_into_my_localhost_server_from/
+        errMsg404 = 'Not spported';
+    }
+    if (!errMsg404)
         return;
     const pageContext = createPageContextServerSideWithoutGlobalContext(pageContextInit);
-    const httpResponse = createHttpResponseFavicon404();
+    const httpResponse = createHttpResponse404(errMsg404);
     objectAssign(pageContext, { httpResponse });
     checkType(pageContext);
     return pageContext;

package/dist/esm/shared/page-configs/Config.d.ts

@@ -44,7 +44,7 @@ import type { HooksTimeoutProvidedByUser } from '../hooks/getHook.js';
 import type { GlobalContext, PageContextClient, PageContextServer } from '../types.js';
 import type { InlineConfig } from 'vite';
 type HookName = HookNamePage | HookNameGlobal | HookNameOldDesign;
-type HookNamePage = 'onHydrationEnd' | 'onBeforePrerenderStart' | 'onBeforeRender' | 'onPageTransitionStart' | 'onPageTransitionEnd' | 'onRenderHtml' | 'onRenderClient' | 'guard' | 'data';
+type HookNamePage = 'onHydrationEnd' | 'onBeforePrerenderStart' | 'onBeforeRender' | 'onPageTransitionStart' | 'onPageTransitionEnd' | 'onRenderHtml' | 'onRenderClient' | 'guard' | 'data' | 'onData';
 type HookNameGlobal = 'onBeforePrerender' | 'onBeforeRoute' | 'onPrerenderStart' | 'onCreatePageContext' | 'onCreateGlobalContext';
 type HookNameOldDesign = 'render' | 'prerender';
 type ConfigNameBuiltIn = Exclude<keyof Config, keyof VikeVitePluginOptions | 'onBeforeRoute' | 'onPrerenderStart' | 'vite' | 'redirects'> | 'prerender' | 'isClientRuntimeLoaded' | 'onBeforeRenderEnv' | 'dataEnv' | 'hooksTimeout' | 'clientHooks' | 'middleware';
@@ -345,6 +345,11 @@ type ConfigBuiltIn = {
      *  https://vike.dev/data
      */
     data?: DataAsync<unknown> | DataSync<unknown> | ImportString | null;
+    /** TODO/now
+     *
+     *  https://vike.dev/onData
+     */
+    onData?: Function;
     /** Determines what pageContext properties are sent to the client-side.
      *
      * https://vike.dev/passToClient

package/dist/esm/shared/route/debug.d.ts

@@ -1,3 +1,6 @@
 export { debug };
-import type { Debug } from '../../utils/debug.js';
+export { setCreateDebugger };
+import type { createDebugger, Debug } from '../../utils/debug.js';
+type CreateDebugger = typeof createDebugger;
 declare function debug(...args: Parameters<Debug>): void;
+declare function setCreateDebugger(createDebugger: CreateDebugger): void;

package/dist/esm/shared/route/debug.js

@@ -1,11 +1,21 @@
 export { debug };
-var _debug;
+export { setCreateDebugger };
+// Using createDebugger() for isomorphic code without bloating the client-side.
+// On the server-side, this is just a transparent proxy.
+// On the client-side, this is an emtpy shell.
+import { getGlobalObject } from '../../utils/getGlobalObject.js';
+const globalObject = getGlobalObject('route/debug.ts', {});
 function debug(...args) {
-    if (!_debug) {
-        // We use this trick instead of `import { createDebugger } from '../../utils/debug` in order to ensure that the `debug` mechanism is only loaded on the server-side
-        _debug = globalThis.__brillout_debug_createDebugger?.('vike:routing');
-    }
-    if (_debug) {
-        _debug(...args);
+    // Client-side => does nothing
+    if (!globalObject.createDebugger)
+        return;
+    // Server-side => just a proxy
+    if (!globalObject.debug) {
+        globalObject.debug = globalObject.createDebugger('vike:routing');
     }
+    globalObject.debug(...args);
+}
+// Called only on the server-side
+function setCreateDebugger(createDebugger) {
+    globalObject.createDebugger = createDebugger;
 }

package/dist/esm/utils/PROJECT_VERSION.d.ts

@@ -1 +1 @@
-export declare const PROJECT_VERSION: "0.4.229-commit-2dbbc8d";
+export declare const PROJECT_VERSION: "0.4.229-commit-845a4e0";

package/dist/esm/utils/PROJECT_VERSION.js

@@ -1,2 +1,2 @@
 // Automatically updated by @brillout/release-me
-export const PROJECT_VERSION = '0.4.229-commit-2dbbc8d';
+export const PROJECT_VERSION = '0.4.229-commit-845a4e0';

package/dist/esm/utils/debug.js

@@ -8,11 +8,10 @@ import { checkType } from './checkType.js';
 import { getTerminalWidth } from './getTerminWidth.js';
 import pc from '@brillout/picocolors';
 import { isArray } from './isArray.js';
-// Avoid this to be loaded in the browser. For isomorphic code: instead of `import { createDebugger } from './utils.js'`, use `globalThis.createDebugger()`.
+import { isObject } from './isObject.js';
+import { setCreateDebugger } from '../shared/route/debug.js';
 assert(!isBrowser());
-globalThis.__brillout_debug_createDebugger = createDebugger;
-// We purposely read process.env.DEBUG early, in order to avoid users from the temptation to set process.env.DEBUG with JavaScript, since reading & writing process.env.DEBUG dynamically leads to inconsistencies: for example https://github.com/vikejs/vike/issues/2239
-const DEBUG = getDEBUG() ?? '';
+setCreateDebugger(createDebugger); // for isomorphic code
 const flags = [
     'vike:crawl',
     'vike:error',
@@ -34,6 +33,10 @@ const flags = [
 ];
 const flagsSkipWildcard = ['vike:log'];
 const flagRegex = /\bvike:[a-zA-Z-]+/g;
+// We purposely read process.env.DEBUG early, in order to avoid users from the temptation to set process.env.DEBUG with JavaScript, since reading & writing process.env.DEBUG dynamically leads to inconsistencies such as https://github.com/vikejs/vike/issues/2239
+const DEBUG = getDEBUG() ?? '';
+if (isDebug())
+    Error.stackTraceLimit = Infinity;
 assertFlagsActivated();
 function createDebugger(flag, optionsGlobal) {
     checkType(flag);
@@ -60,9 +63,10 @@ function debug_(flag, options, ...msgs) {
     });
     let logFirst;
     let logsRest;
-    const noNewLine = msgsRest.length <= 1 && [msgFirst, ...msgsRest].every((m) => typeof m === 'string' && !m.includes('\n'));
+    const noNewLine = msgsRest.length <= 1 &&
+        [msgFirst, ...msgsRest].every((m) => (typeof m === 'string' ? !m.includes('\n') : !isObject(m)));
     if (noNewLine) {
-        logFirst = [msgFirst, ...msgsRest].map((m) => String(m).trim());
+        logFirst = [msgFirst, ...msgsRest].map((m) => (typeof m !== 'string' ? m : m.trim()));
         logsRest = [];
     }
     else {
@@ -152,6 +156,10 @@ function getFlagsActivated() {
     const all = DEBUG.includes('vike:*');
     return { flagsActivated, all };
 }
+function isDebug() {
+    const { flagsActivated, all } = getFlagsActivated();
+    return all || flagsActivated.length > 0;
+}
 function getDEBUG() {
     let DEBUG;
     // - `process` can be undefined in edge workers

package/dist/esm/utils/requireResolve.js

@@ -27,12 +27,12 @@ const debug = createDebugger('vike:resolve');
 // - The argument createRequire(argument) seems to be overriden by the `paths` argument require.resolve()
 //   - For example, passing an empty array to `paths` kills the argument passed to `createRequire()`
 //   - Thus, when `paths` is defined, then the context needs to be passed to both createRequire() as well as the `paths` argument of require.resolve()
-function requireResolve_(importPath, importerFilePath, { userRootDir, doNotHandleFileExtension } = {}) {
+function requireResolve_(importPath, importerFilePath, userRootDir, doNotHandleFileExtension = false) {
     assertPosixPath(importPath);
     const contexts = importerFilePath
         ? [importerFilePath]
         : [userRootDir ? getFakeImporterFile(userRootDir) : importMetaUrl];
-    addExtraContextForNpmPackageImport(contexts, { importPath, userRootDir });
+    addExtraContextForNpmPackageImport(contexts, importPath, userRootDir);
     let importPathResolvedFilePath;
     let failure;
     for (const context of contexts) {
@@ -49,9 +49,8 @@ function requireResolve_(importPath, importerFilePath, { userRootDir, doNotHandl
         }
         catch (err) {
             if (debug.isActivated) {
-                debug('err', err);
-                debug('importPath', importPath);
-                debug('context', context);
+                const stack = new Error().stack;
+                debug('ERROR', { err, importPath, context }, stack);
             }
             failure ?? (failure = { err });
         }
@@ -61,21 +60,23 @@ function requireResolve_(importPath, importerFilePath, { userRootDir, doNotHandl
     if (!importPathResolvedFilePath) {
         assert(failure);
         if (debug.isActivated) {
-            debug('FAILURE');
-            debug('importPath', importPath);
-            debug('importerFilePath', importerFilePath);
-            debug('userRootDir', userRootDir);
-            debug('doNotHandleFileExtension', doNotHandleFileExtension);
-            debug('importMetaUrl', importMetaUrl);
-            debug('contexts', contexts);
+            debug('FAILURE', {
+                importPath,
+                importerFilePath,
+                userRootDir,
+                doNotHandleFileExtension,
+                importMetaUrl,
+                contexts
+            });
         }
         return { importPathResolvedFilePath: undefined, err: failure.err, hasFailed: true };
     }
     else {
         if (failure && debug.isActivated) {
-            debug('SUCCESS');
-            debug('importPath', importPath);
-            debug('contexts', contexts);
+            debug('SUCCESS', {
+                importPath,
+                contexts
+            });
         }
         assert(importPathResolvedFilePath);
         importPathResolvedFilePath = toPosixPath(importPathResolvedFilePath);
@@ -83,14 +84,14 @@ function requireResolve_(importPath, importerFilePath, { userRootDir, doNotHandl
     }
 }
 function requireResolveOptional({ importPath, importerFilePath, userRootDir }) {
-    const res = requireResolve_(importPath, importerFilePath, { userRootDir });
+    const res = requireResolve_(importPath, importerFilePath, userRootDir);
     if (res.hasFailed)
         return null;
     return res.importPathResolvedFilePath;
 }
 function requireResolveOptionalDir({ importPath, importerDir, userRootDir }) {
     const importerFilePath = getFakeImporterFile(importerDir);
-    const res = requireResolve_(importPath, importerFilePath, { userRootDir });
+    const res = requireResolve_(importPath, importerFilePath, userRootDir);
     if (res.hasFailed)
         return null;
     return res.importPathResolvedFilePath;
@@ -98,7 +99,7 @@ function requireResolveOptionalDir({ importPath, importerDir, userRootDir }) {
 function requireResolveNpmPackage({ importPathNpmPackage, userRootDir }) {
     assertIsImportPathNpmPackage(importPathNpmPackage);
     const importerFilePath = getFakeImporterFile(userRootDir);
-    const res = requireResolve_(importPathNpmPackage, importerFilePath, { userRootDir });
+    const res = requireResolve_(importPathNpmPackage, importerFilePath, userRootDir);
     if (res.hasFailed)
         throw res.err;
     return res.importPathResolvedFilePath;
@@ -112,14 +113,14 @@ function requireResolveVikeDistFile(vikeDistFile) {
     {
         const res = requireResolve_(importPathResolvedFilePath, 
         // No context needed: importPathResolvedFilePath is already resolved and absolute
-        null, { doNotHandleFileExtension: true });
+        null, null, true);
         if (res.hasFailed)
             throw res.err;
         assert(res.importPathResolvedFilePath === importPathResolvedFilePath);
     }
     return importPathResolvedFilePath;
 }
-function addExtraContextForNpmPackageImport(contexts, { importPath, userRootDir }) {
+function addExtraContextForNpmPackageImport(contexts, importPath, userRootDir) {
     // We should add extra context only for npm packages, but unfortunately we cannot always disambiguate between npm package imports and path aliases.
     if (!isImportPathNpmPackageOrPathAlias(importPath))
         return;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "vike",
-  "version": "0.4.229-commit-2dbbc8d",
+  "version": "0.4.229-commit-845a4e0",
   "repository": "https://github.com/vikejs/vike",
   "exports": {
     "./server": {