vike 0.4.199-commit-33e1230 → 0.4.199-commit-5883046

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.199-commit-33e1230';
+exports.PROJECT_VERSION = '0.4.199-commit-5883046';

package/dist/esm/client/client-routing-runtime/history.d.ts

@@ -1,5 +1,5 @@
-export { enhanceHistoryState, getHistoryState, pushHistory, type ScrollPosition, saveScrollPosition, monkeyPatchHistoryPushState };
-type StateVikeEnhanced = {
+export { getHistoryState, enhanceHistoryState, pushHistoryState, type ScrollPosition, saveScrollPosition };
+type StateEnhanced = {
     timestamp: number;
     scrollPosition: null | ScrollPosition;
     triggeredBy: 'user' | 'vike' | 'browser';
@@ -9,9 +9,7 @@ type ScrollPosition = {
     x: number;
     y: number;
 };
-type StateNotInitialized = null | undefined | Partial<StateVikeEnhanced> | StateVikeEnhanced;
 declare function enhanceHistoryState(): void;
-declare function getHistoryState(): StateNotInitialized;
+declare function getHistoryState(): StateEnhanced;
 declare function saveScrollPosition(): void;
-declare function pushHistory(url: string, overwriteLastHistoryEntry: boolean): void;
-declare function monkeyPatchHistoryPushState(): void;
+declare function pushHistoryState(url: string, overwriteLastHistoryEntry: boolean): void;

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

@@ -1,22 +1,24 @@
-export { enhanceHistoryState, getHistoryState, pushHistory, saveScrollPosition, monkeyPatchHistoryPushState };
+export { getHistoryState, enhanceHistoryState, pushHistoryState, saveScrollPosition };
 import { assert, assertUsage, hasProp, isObject } from './utils.js';
+let initStateEnhanced;
+init();
 // `window.history.state === null` when:
 // - The very first render
 // - Click on `<a href="#some-hash" />`
 // - `location.hash = 'some-hash'`
 function enhanceHistoryState() {
-    const stateNotInitialized = window.history.state;
-    if (isVikeEnhanced(stateNotInitialized))
+    const stateNotEnhanced = getStateNotEnhanced();
+    if (isVikeEnhanced(stateNotEnhanced))
         return;
-    const stateVikeEnhanced = enhance(stateNotInitialized);
+    const stateVikeEnhanced = enhance(stateNotEnhanced);
     replaceHistoryState(stateVikeEnhanced);
 }
-function enhance(stateNotInitialized) {
+function enhance(stateNotEnhanced) {
     const timestamp = getTimestamp();
     const scrollPosition = getScrollPosition();
     const triggeredBy = 'browser';
     let stateVikeEnhanced;
-    if (!stateNotInitialized) {
+    if (!stateNotEnhanced) {
         stateVikeEnhanced = {
             timestamp,
             scrollPosition,
@@ -27,24 +29,32 @@ function enhance(stateNotInitialized) {
     else {
         // State information may be incomplete if `window.history.state` is set by an old Vike version. (E.g. `state.timestamp` was introduced for `pageContext.isBackwardNavigation` in `0.4.19`.)
         stateVikeEnhanced = {
-            timestamp: stateNotInitialized.timestamp ?? timestamp,
-            scrollPosition: stateNotInitialized.scrollPosition ?? scrollPosition,
-            triggeredBy: stateNotInitialized.triggeredBy ?? triggeredBy,
+            timestamp: stateNotEnhanced.timestamp ?? timestamp,
+            scrollPosition: stateNotEnhanced.scrollPosition ?? scrollPosition,
+            triggeredBy: stateNotEnhanced.triggeredBy ?? triggeredBy,
             _isVikeEnhanced: true
         };
     }
     assert(isVikeEnhanced(stateVikeEnhanced));
     return stateVikeEnhanced;
 }
-function getState_alreadyEnhanced() {
-    const state = getHistoryState();
+function getStateEnhanced() {
+    const state = getStateNotEnhanced();
+    // This assert() will most likely eventually cause issues. Let's then:
+    // - Replace the assert() call with enhanceHistoryState()
+    // - Remove the race condition buster `initStateEnhanced` as it won't be needed anymore
     assert(isVikeEnhanced(state));
     return state;
 }
-function getHistoryState() {
+function getStateNotEnhanced() {
     const state = window.history.state;
     return state;
 }
+function getHistoryState() {
+    if (!initStateEnhanced)
+        enhanceHistoryState(); // avoid race condition
+    return getStateEnhanced();
+}
 function getScrollPosition() {
     const scrollPosition = { x: window.scrollX, y: window.scrollY };
     return scrollPosition;
@@ -54,13 +64,13 @@ function getTimestamp() {
 }
 function saveScrollPosition() {
     const scrollPosition = getScrollPosition();
-    const state = getState_alreadyEnhanced();
+    const state = getStateEnhanced();
     replaceHistoryState({ ...state, scrollPosition });
 }
-function pushHistory(url, overwriteLastHistoryEntry) {
+function pushHistoryState(url, overwriteLastHistoryEntry) {
     if (!overwriteLastHistoryEntry) {
         const timestamp = getTimestamp();
-        pushHistoryState({
+        pushState({
             timestamp,
             // I don't remember why I set it to `null`, maybe because setting it now would be too early? (Maybe there is a delay between renderPageClientSide() is finished and the browser updating the scroll position.) Anyways, it seems like autoSaveScrollPosition() is enough.
             scrollPosition: null,
@@ -69,14 +79,14 @@ function pushHistory(url, overwriteLastHistoryEntry) {
         }, url);
     }
     else {
-        replaceHistoryState(getState_alreadyEnhanced(), url);
+        replaceHistoryState(getStateEnhanced(), url);
     }
 }
 function replaceHistoryState(state, url) {
     const url_ = url ?? null; // Passing `undefined` chokes older Edge versions.
     window.history.replaceState(state, '', url_);
 }
-function pushHistoryState(state, url) {
+function pushState(state, url) {
     // Vike should call window.history.pushState() (and not the orignal `pushStateOriginal()`) so that other tools (e.g. user tracking) can listen to Vike's pushState() calls, see https://github.com/vikejs/vike/issues/1582.
     window.history.pushState(state, '', url);
 }
@@ -114,3 +124,8 @@ function assertStateVikeEnhanced(state) {
         assert(hasProp(state.scrollPosition, 'x', 'number') && hasProp(state.scrollPosition, 'y', 'number'));
     }
 }
+function init() {
+    enhanceHistoryState();
+    initStateEnhanced = true;
+    monkeyPatchHistoryPushState();
+}

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

@@ -1,6 +1,5 @@
 export { initClientRouter };
 import { assert } from './utils.js';
-import { enhanceHistoryState, monkeyPatchHistoryPushState } from './history.js';
 import { getRenderCount, renderPageClientSide } from './renderPageClientSide.js';
 import { initOnPopState } from './initOnPopState.js';
 import { initOnLinkClick } from './initOnLinkClick.js';
@@ -29,9 +28,7 @@ async function renderFirstPage() {
 }
 function initHistoryAndScroll() {
     setupNativeScrollRestoration();
-    enhanceHistoryState();
     autoSaveScrollPosition();
-    monkeyPatchHistoryPushState();
     // Handle back-/forward navigation
     initOnPopState();
 }

package/dist/esm/client/client-routing-runtime/initOnPopState.d.ts

@@ -3,25 +3,20 @@ export { updateState };
 export { onPopState };
 declare function initOnPopState(): void;
 type Listener = (arg: {
-    previousState: ReturnType<typeof getState>;
+    previous: ReturnType<typeof getInfo>;
 }) => void | boolean;
 /** Control client-side navigation.
  *
  * https://vike.dev/onPopState
  */
 declare function onPopState(listener: Listener): void;
-declare function getState(): {
+declare function getInfo(): {
     url: `/${string}`;
-    historyState: {
+    state: {
         timestamp: number;
         scrollPosition: null | import("./history.js").ScrollPosition;
         triggeredBy: "user" | "vike" | "browser";
         _isVikeEnhanced: true;
-    } | Partial<{
-        timestamp: number;
-        scrollPosition: null | import("./history.js").ScrollPosition;
-        triggeredBy: "user" | "vike" | "browser";
-        _isVikeEnhanced: true;
-    }> | null | undefined;
+    };
 };
 declare function updateState(): void;

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

@@ -1,11 +1,11 @@
 export { initOnPopState };
 export { updateState };
 export { onPopState };
-import { assert, getCurrentUrl, getGlobalObject } from './utils.js';
+import { getCurrentUrl, getGlobalObject } from './utils.js';
 import { enhanceHistoryState, getHistoryState } from './history.js';
 import { renderPageClientSide } from './renderPageClientSide.js';
 import { setScrollPosition } from './setScrollPosition.js';
-const globalObject = getGlobalObject('initOnPopState.ts', { previousState: getState(), listeners: [] });
+const globalObject = getGlobalObject('initOnPopState.ts', { previous: getInfo(), listeners: [] });
 function initOnPopState() {
     // - The popstate event is trigged upon:
     //   - Back-/forward navigation.
@@ -17,25 +17,20 @@ function initOnPopState() {
     //     - `location.hash = 'some-hash'`
     // - The `event` argument of `window.addEventListener('popstate', (event) => /*...*/)` is useless: the History API doesn't provide the previous state (the popped state), see https://stackoverflow.com/questions/48055323/is-history-state-always-the-same-as-popstate-event-state
     window.addEventListener('popstate', async () => {
-        const { previousState } = globalObject;
-        const currentState = getState();
-        globalObject.previousState = currentState;
-        const scrollTarget = currentState.historyState?.scrollPosition || undefined;
-        const isHashNavigation = removeHash(currentState.url) === removeHash(previousState.url);
-        const isBackwardNavigation = !currentState.historyState?.timestamp || !previousState.historyState?.timestamp
-            ? null
-            : currentState.historyState.timestamp < previousState.historyState.timestamp;
+        const isNewState = window.history.state === null;
+        if (isNewState)
+            enhanceHistoryState();
+        const { previous } = globalObject;
+        const current = getInfo();
+        globalObject.previous = current;
+        const scrollTarget = current.state.scrollPosition || undefined;
+        const isHashNavigation = removeHash(current.url) === removeHash(previous.url);
         // - `history.state === null` when:
         //   - Click on `<a href="#some-hash" />` (note that Vike's `initOnLinkClick()` handler skips hash links)
         //   - `location.hash = 'some-hash'`
         // - `history.state !== null` when `popstate` was triggered by the user clicking on his browser's forward/backward history button.
-        let isHashNavigationNew = isHashNavigation && window.history.state === null;
-        if (window.history.state === null) {
-            assert(isHashNavigation);
-            // The browser already scrolled to `#${hash}` => the current scroll position is the right one => we save it with `enhanceHistoryState()`.
-            enhanceHistoryState();
-            globalObject.previousState = getState();
-        }
+        const isHashNavigationNew = isHashNavigation && isNewState;
+        const isBackwardNavigation = !current.state.timestamp || !previous.state.timestamp ? null : current.state.timestamp < previous.state.timestamp;
         // We have to scroll ourselves because we use `window.history.scrollRestoration = 'manual'`. So far this seems to work. Alternatives in case it doesn't work:
         // - Alternative: we use `window.history.scrollRestoration = 'auto'`
         //   - Problem: I don't think it's possbible to set `window.history.scrollRestoration = 'auto'` only for hash navigation and not for non-hash navigations?
@@ -49,11 +44,14 @@ function initOnPopState() {
             if (!isHashNavigationNew) {
                 setScrollPosition(scrollTarget);
             }
+            else {
+                // The browser already scrolled to `#${hash}` => the current scroll position is the right one => we saved it with `enhanceHistoryState()`.
+            }
             return;
         }
         let abort;
         globalObject.listeners.forEach((listener) => {
-            abort || (abort = listener({ previousState }));
+            abort || (abort = listener({ previous }));
         });
         if (abort)
             return;
@@ -67,15 +65,15 @@ function initOnPopState() {
 function onPopState(listener) {
     globalObject.listeners.push(listener);
 }
-function getState() {
+function getInfo() {
     return {
         url: getCurrentUrl(),
-        historyState: getHistoryState()
+        state: getHistoryState()
     };
 }
 function removeHash(url) {
     return url.split('#')[0];
 }
 function updateState() {
-    globalObject.previousState = getState();
+    globalObject.previous = getInfo();
 }

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

@@ -10,7 +10,7 @@ import { assertInfo, assertWarning, isReact } from './utils.js';
 import { executeOnRenderClientHook } from '../shared/executeOnRenderClientHook.js';
 import { assertHook, getHook } from '../../shared/hooks/getHook.js';
 import { isErrorFetchingStaticAssets, loadUserFilesClientSide } from '../shared/loadUserFilesClientSide.js';
-import { pushHistory } from './history.js';
+import { pushHistoryState } from './history.js';
 import { assertNoInfiniteAbortLoop, getPageContextFromAllRewrites, isAbortError, logAbortErrorHandled } from '../../shared/route/abort.js';
 import { route } from '../../shared/route/index.js';
 import { isClientSideRoutable } from './isClientSideRoutable.js';
@@ -469,7 +469,7 @@ function changeUrl(url, overwriteLastHistoryEntry) {
     if (getCurrentUrl() === url)
         return;
     browserNativeScrollRestoration_disable();
-    pushHistory(url, overwriteLastHistoryEntry);
+    pushHistoryState(url, overwriteLastHistoryEntry);
     updateState();
 }
 function handleErrorFetchingStaticAssets(err, pageContext, isFirstRender) {

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

@@ -1 +1 @@
-export declare const PROJECT_VERSION: "0.4.199-commit-33e1230";
+export declare const PROJECT_VERSION: "0.4.199-commit-5883046";

package/dist/esm/utils/PROJECT_VERSION.js

@@ -1,2 +1,2 @@
 // Automatically updated by @brillout/release-me
-export const PROJECT_VERSION = '0.4.199-commit-33e1230';
+export const PROJECT_VERSION = '0.4.199-commit-5883046';

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

@@ -1,4 +1,4 @@
 export declare const projectInfo: {
     projectName: "Vike";
-    projectVersion: "0.4.199-commit-33e1230";
+    projectVersion: "0.4.199-commit-5883046";
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "vike",
-  "version": "0.4.199-commit-33e1230",
+  "version": "0.4.199-commit-5883046",
   "repository": "https://github.com/vikejs/vike",
   "exports": {
     "./server": {