vike 0.4.240-commit-a80417c → 0.4.240-commit-82bb0c2

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

@@ -3,7 +3,7 @@ export { getRenderCount };
 export { disableClientRouting };
 export { firstRenderStartPromise };
 export { getPageContextClient };
-import { assert, isSameErrorMessage, 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()
@@ -267,14 +267,12 @@ async function renderPageClientSide(renderArgs) {
     }
     async function renderErrorPage(pageContext, args, pageContextAbort) {
         const onError = (err) => {
-            if (!isSameErrorMessage(err, args.err)) {
-                /* When we can't render the error page, we prefer showing a blank page over letting the server-side try because otherwise:
-                   - We risk running into an infinite loop of reloads which would overload the server.
-                   - An infinite reloading page is a even worse UX than a blank page.
-                redirectHard(urlOriginal)
-                */
-                console.error(err);
-            }
+            /* When we can't render the error page, we prefer showing a blank page over letting the server-side try because otherwise:
+               - We risk running into an infinite loop of reloads which would overload the server.
+               - An infinite reloading page is a even worse UX than a blank page.
+            redirectHard(urlOriginal)
+            */
+            logError(err);
         };
         const errorPageId = getErrorPageId(pageContext._pageFilesAll, pageContext._globalContext._pageConfigs);
         if (!errorPageId)
@@ -386,9 +384,7 @@ async function renderPageClientSide(renderArgs) {
                 await handleError({ err });
             }
             else {
-                if (!isSameErrorMessage(err, isErrorPage.err)) {
-                    console.error(err);
-                }
+                logError(err);
             }
         };
         // We use globalObject.onRenderClientPreviousPromise in order to ensure that there is never two concurrent onRenderClient() calls
@@ -519,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, [
@@ -669,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/client/runtime-client-routing/utils.d.ts

@@ -7,7 +7,6 @@ export * from '../../utils/isCallable.js';
 export * from '../../utils/isObject.js';
 export * from '../../utils/isPlainObject.js';
 export * from '../../utils/isReact.js';
-export * from '../../utils/isSameErrorMessage.js';
 export * from '../../utils/objectAssign.js';
 export * from '../../utils/parseUrl.js';
 export * from '../../utils/PromiseType.js';

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

@@ -11,7 +11,6 @@ export * from '../../utils/isCallable.js';
 export * from '../../utils/isObject.js';
 export * from '../../utils/isPlainObject.js';
 export * from '../../utils/isReact.js';
-export * from '../../utils/isSameErrorMessage.js';
 export * from '../../utils/objectAssign.js';
 export * from '../../utils/parseUrl.js';
 export * from '../../utils/PromiseType.js';

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

@@ -1 +1 @@
-export declare const PROJECT_VERSION: "0.4.240-commit-a80417c";
+export declare const PROJECT_VERSION: "0.4.240-commit-82bb0c2";

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-a80417c';
+export const PROJECT_VERSION = '0.4.240-commit-82bb0c2';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "vike",
-  "version": "0.4.240-commit-a80417c",
+  "version": "0.4.240-commit-82bb0c2",
   "repository": "https://github.com/vikejs/vike",
   "exports": {
     "./server": {