vike 0.4.240-commit-098ff81 → 0.4.240-commit-bcd07ae

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

@@ -3,7 +3,7 @@ export { getRenderCount };
 export { disableClientRouting };
 export { firstRenderStartPromise };
 export { getPageContextClient };
-import { assert, objectAssign, redirectHard, getGlobalObject, hasProp, updateType, genPromise, isCallable, catchInfiniteLoop, } from './utils.js';
+import { assert, objectAssign, redirectHard, getGlobalObject, hasProp, updateType, genPromise, isCallable, catchInfiniteLoop, isObject, } from './utils.js';
 import { getPageContextFromClientHooks, getPageContextFromServerHooks, getPageContextFromHooks_isHydration, getPageContextFromHooks_serialized, setPageContextInitIsPassedToClient, } from './getPageContextFromHooks.js';
 import { createPageContextClientSide } from './createPageContextClientSide.js';
 import { addLinkPrefetchHandlers, addLinkPrefetchHandlers_unwatch, addLinkPrefetchHandlers_watch, getPageContextPrefetched, populatePageContextPrefetchCache, } from './prefetch.js';
@@ -241,7 +241,7 @@ async function renderPageClientSide(renderArgs) {
             // We don't swallow 404 errors:
             //  - On the server-side, Vike swallows / doesn't show any 404 error log because it's expected that a user may go to some random non-existent URL. (We don't want to flood the app's error tracking with 404 logs.)
             //  - On the client-side, if the user navigates to a 404 then it means that the UI has a broken link. (It isn't expected that users can go to some random URL using the client-side router, as it would require, for example, the user to manually change the URL of a link by manually manipulating the DOM which highly unlikely.)
-            console.error(err);
+            logError(err);
         }
         else {
             // We swallow throw redirect()/render() called by client-side hooks onBeforeRender()/data()/guard()
@@ -272,7 +272,7 @@ async function renderPageClientSide(renderArgs) {
                - An infinite reloading page is a even worse UX than a blank page.
             redirectHard(urlOriginal)
             */
-            console.error(err);
+            logError(err);
         };
         const errorPageId = getErrorPageId(pageContext._pageFilesAll, pageContext._globalContext._pageConfigs);
         if (!errorPageId)
@@ -384,7 +384,7 @@ async function renderPageClientSide(renderArgs) {
                 await handleError({ err });
             }
             else {
-                console.error(err);
+                logError(err);
             }
         };
         // We use globalObject.onRenderClientPreviousPromise in order to ensure that there is never two concurrent onRenderClient() calls
@@ -515,10 +515,10 @@ function changeUrl(url, overwriteLastHistoryEntry) {
     pushHistoryState(url, overwriteLastHistoryEntry);
 }
 function disableClientRouting(err, log) {
-    assert(isErrorFetchingStaticAssets(err));
     globalObject.clientRoutingIsDisabled = true;
+    assert(isErrorFetchingStaticAssets(err));
     if (log) {
-        // We don't use console.error() to avoid flooding error trackers such as Sentry
+        // We purposely don't use console.error() to avoid flooding error trackers such as Sentry
         console.log(err);
     }
     assertInfo(false, [
@@ -665,3 +665,11 @@ if (import.meta.env.DEV && import.meta.hot)
             });
         }
     });
+function logError(err) {
+    if (isObject(err) &&
+        // Set by vike-react
+        err.isAlreadyLogged) {
+        return;
+    }
+    console.error(err);
+}

package/dist/esm/node/runtime/logErrorServer.d.ts

@@ -0,0 +1,2 @@
+export { logErrorServer };
+declare function logErrorServer(err: unknown): void;

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

@@ -0,0 +1,14 @@
+export { logErrorServer };
+import pc from '@brillout/picocolors';
+import { isCallable, isObject } from './utils.js';
+function logErrorServer(err) {
+    if (isObject(err) &&
+        // Set by react-streaming
+        isCallable(err.prettifyThisError)) {
+        err = err.prettifyThisError(err);
+    }
+    // We ensure we print a string; Cloudflare Workers doesn't seem to properly stringify `Error` objects.
+    // - TO-DO/eventuually: is that still true? Let's eventuually remove it and see if it crashes Cloudflare.
+    const errStr = isObject(err) && 'stack' in err ? String(err.stack) : String(err);
+    console.error(pc.red(errStr));
+}

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

@@ -4,18 +4,16 @@ export { logErrorProd };
 export { onRuntimeError };
 import { isAbortError } from '../../../shared/route/abort.js';
 import { setAlreadyLogged } from './isNewError.js';
-import { isObject, warnIfErrorIsNotObject } from '../utils.js';
-import pc from '@brillout/picocolors';
+import { warnIfErrorIsNotObject } from '../utils.js';
 import { logErrorHint } from './logErrorHint.js';
+import { logErrorServer } from '../logErrorServer.js';
 function logErrorProd(err, _httpRequestId) {
     warnIfErrorIsNotObject(err);
     setAlreadyLogged(err);
     if (isAbortError(err)) {
         return;
     }
-    // We ensure we print a string; Cloudflare Workers doesn't seem to properly stringify `Error` objects.
-    const errStr = isObject(err) && 'stack' in err ? String(err.stack) : String(err);
-    console.error(pc.red(errStr));
+    logErrorServer(err);
     // Needs to be called after logging the error.
     onRuntimeError(err);
 }

package/dist/esm/node/vite/shared/loggerNotProd/log.js

@@ -6,6 +6,7 @@ import { assert, stripAnsi, hasProp, assertIsNotProductionRuntime, PROJECT_VERSI
 import pc from '@brillout/picocolors';
 import { isErrorDebug } from '../../../shared/isErrorDebug.js';
 import { getViteDevServer } from '../../../runtime/globalContext.js';
+import { logErrorServer } from '../../../runtime/logErrorServer.js';
 assertIsNotProductionRuntime();
 function logWithVikeTag(msg, logType, category, showVikeVersion = false) {
     const projectTag = getProjectTag(showVikeVersion);
@@ -40,7 +41,8 @@ function logDirectly(thing, logType) {
         return;
     }
     if (logType === 'error') {
-        console.error(thing);
+        // console.error()
+        logErrorServer(thing);
         return;
     }
     if (logType === 'error-recover') {

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

@@ -1 +1 @@
-export declare const PROJECT_VERSION: "0.4.240-commit-098ff81";
+export declare const PROJECT_VERSION: "0.4.240-commit-bcd07ae";

package/dist/esm/utils/PROJECT_VERSION.js

@@ -1,2 +1,2 @@
 // Automatically updated by @brillout/release-me
-export const PROJECT_VERSION = '0.4.240-commit-098ff81';
+export const PROJECT_VERSION = '0.4.240-commit-bcd07ae';

package/dist/esm/utils/debug.js

@@ -158,7 +158,7 @@ function assertFlagsActivated() {
 function getFlagsActivated() {
     const flagsActivated = DEBUG.match(/\bvike:[a-zA-Z-]+/g) ?? [];
     const isAll = DEBUG.includes('vike:*');
-    const isGlobal = /\bvike\b[^:]/.test(DEBUG);
+    const isGlobal = /\bvike\b([^:]|$)/.test(DEBUG);
     return { flagsActivated, isAll, isGlobal };
 }
 function isDebug() {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "vike",
-  "version": "0.4.240-commit-098ff81",
+  "version": "0.4.240-commit-bcd07ae",
   "repository": "https://github.com/vikejs/vike",
   "exports": {
     "./server": {