vike 0.4.229-commit-847f1b6 → 0.4.229-commit-e1ca16d

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-847f1b6';
+exports.PROJECT_VERSION = '0.4.229-commit-e1ca16d';

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

@@ -77,7 +77,7 @@ declare function getPageContextFromHooks_isHydration(pageContext: PageContextSer
         routeParams: Record<string, string>;
     } & import("../../shared/preparePageContextForPublicUsage.js").PageContextPrepareMinimum & import("../../shared/types.js").PageContextInternalClient & Omit<Partial<{
         isPageContext: true;
-        Page: import("../../shared/page-configs/Config.js").Config["Page"];
+        Page: import("../../types/index.js").Config["Page"];
         routeParams: Record<string, string>;
         data: unknown;
         config: import("../../shared/page-configs/Config/PageContextConfig.js").PageContextConfig;
@@ -101,7 +101,7 @@ declare function getPageContextFromHooks_isHydration(pageContext: PageContextSer
         isBaseMissing?: true;
     }> & Pick<{
         isPageContext: true;
-        Page: import("../../shared/page-configs/Config.js").Config["Page"];
+        Page: import("../../types/index.js").Config["Page"];
         routeParams: Record<string, string>;
         data: unknown;
         config: import("../../shared/page-configs/Config/PageContextConfig.js").PageContextConfig;
@@ -148,7 +148,7 @@ declare function getPageContextFromHooks_isHydration(pageContext: PageContextSer
     _hasPageContextFromServer: true;
 } & import("../../shared/preparePageContextForPublicUsage.js").PageContextPrepareMinimum & Omit<Partial<{
     isPageContext: true;
-    Page: import("../../shared/page-configs/Config.js").Config["Page"];
+    Page: import("../../types/index.js").Config["Page"];
     routeParams: Record<string, string>;
     data: unknown;
     config: import("../../shared/page-configs/Config/PageContextConfig.js").PageContextConfig;
@@ -172,7 +172,7 @@ declare function getPageContextFromHooks_isHydration(pageContext: PageContextSer
     isBaseMissing?: true;
 }> & Pick<{
     isPageContext: true;
-    Page: import("../../shared/page-configs/Config.js").Config["Page"];
+    Page: import("../../types/index.js").Config["Page"];
     routeParams: Record<string, string>;
     data: unknown;
     config: import("../../shared/page-configs/Config/PageContextConfig.js").PageContextConfig;
@@ -288,7 +288,7 @@ declare function getPageContextFromClientHooks(pageContext: {
         routeParams: Record<string, string>;
     } & import("../../shared/preparePageContextForPublicUsage.js").PageContextPrepareMinimum & import("../../shared/types.js").PageContextInternalClient & Omit<Partial<{
         isPageContext: true;
-        Page: import("../../shared/page-configs/Config.js").Config["Page"];
+        Page: import("../../types/index.js").Config["Page"];
         routeParams: Record<string, string>;
         data: unknown;
         config: import("../../shared/page-configs/Config/PageContextConfig.js").PageContextConfig;
@@ -312,7 +312,7 @@ declare function getPageContextFromClientHooks(pageContext: {
         isBaseMissing?: true;
     }> & Pick<{
         isPageContext: true;
-        Page: import("../../shared/page-configs/Config.js").Config["Page"];
+        Page: import("../../types/index.js").Config["Page"];
         routeParams: Record<string, string>;
         data: unknown;
         config: import("../../shared/page-configs/Config/PageContextConfig.js").PageContextConfig;
@@ -357,7 +357,7 @@ declare function getPageContextFromClientHooks(pageContext: {
     }) | null;
 } & PageConfigUserFriendlyOld & import("../../shared/preparePageContextForPublicUsage.js").PageContextPrepareMinimum & Omit<Partial<{
     isPageContext: true;
-    Page: import("../../shared/page-configs/Config.js").Config["Page"];
+    Page: import("../../types/index.js").Config["Page"];
     routeParams: Record<string, string>;
     data: unknown;
     config: import("../../shared/page-configs/Config/PageContextConfig.js").PageContextConfig;
@@ -381,7 +381,7 @@ declare function getPageContextFromClientHooks(pageContext: {
     isBaseMissing?: true;
 }> & Pick<{
     isPageContext: true;
-    Page: import("../../shared/page-configs/Config.js").Config["Page"];
+    Page: import("../../types/index.js").Config["Page"];
     routeParams: Record<string, string>;
     data: unknown;
     config: import("../../shared/page-configs/Config/PageContextConfig.js").PageContextConfig;

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

@@ -65,7 +65,7 @@ async function getPageContextFromServerHooks(pageContext, isErrorPage) {
     return { pageContextFromServerHooks };
 }
 async function getPageContextFromClientHooks(pageContext, isErrorPage) {
-    let dataHookExists = false;
+    let dataHookExec = 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.
@@ -82,16 +82,17 @@ async function getPageContextFromClientHooks(pageContext, isErrorPage) {
             }
         }
         else {
-            if (hookName === 'data')
-                dataHookExists = true;
             if (hookClientOnlyExists(hookName, pageContext) || !pageContext._hasPageContextFromServer) {
+                if (hookName === 'data')
+                    dataHookExec = true;
                 // This won't do anything if no hook has been defined or if the hook's env.client is false.
                 await executeDataLikeHook(hookName, pageContext);
             }
         }
     }
     // Execute +onData
-    if (dataHookExists) {
+    const dataHookEnv = getHookEnv('data', pageContext);
+    if ((dataHookExec && dataHookEnv.client) || (pageContext._hasPageContextFromServer && dataHookEnv.server)) {
         await execHookClient('onData', pageContext);
     }
     const pageContextFromClientHooks = pageContext;
@@ -188,23 +189,22 @@ async function hookServerOnlyExists(hookName, pageContext) {
         return hasOnBeforeRenderServerSideOnlyHook;
     }
 }
-/**
- * @param hookName
- * @param pageContext
- * @returns `true` if the given page has a `hookName` hook defined with a client-only env.
- */
 function hookClientOnlyExists(hookName, pageContext) {
+    const hookEnv = getHookEnv(hookName, pageContext);
+    return !!hookEnv.client && !hookEnv.server;
+}
+function getHookEnv(hookName, pageContext) {
     if (pageContext._pageConfigs.length > 0) {
         // V1
         const pageConfig = getPageConfig(pageContext.pageId, pageContext._pageConfigs);
-        const hookEnv = getConfigValueRuntime(pageConfig, `${hookName}Env`)?.value ?? {};
-        assert(isObject(hookEnv));
-        return !!hookEnv.client && !hookEnv.server;
+        // No runtime validation to save client-side KBs
+        const hookEnv = (getConfigValueRuntime(pageConfig, `${hookName}Env`)?.value ?? {});
+        return hookEnv;
     }
     else {
         // TODO/v1-release: remove
         // Client-only onBeforeRender() or data() hooks were never supported for the V0.4 design
-        return false;
+        return { client: false, server: true };
     }
 }
 async function fetchPageContextFromServer(pageContext) {

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

@@ -4,7 +4,7 @@ export { disableClientRouting };
 export { firstRenderStartPromise };
 export { getPageContextClient };
 import { assert, isSameErrorMessage, objectAssign, redirectHard, getGlobalObject, hasProp, augmentType, genPromise, isCallable, catchInfiniteLoop } from './utils.js';
-import { getPageContextFromClientHooks, getPageContextFromServerHooks, getPageContextFromHooks_isHydration, getPageContextFromHooks_serialized, setPageContextInitIsPassedToClient, execHookClient } from './getPageContextFromHooks.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';
 import { assertInfo, assertWarning, isReact } from './utils.js';
@@ -203,16 +203,6 @@ async function renderPageClientSide(renderArgs) {
             // TO-DO/eventually: create helper assertPageContextFromHook()
             assert(!('urlOriginal' in pageContextFromServerHooks));
             objectAssign(pageContext, pageContextFromServerHooks);
-            // Execute +onData
-            try {
-                await execHookClient('onData', pageContext);
-            }
-            catch (err) {
-                await onError(err);
-                return;
-            }
-            if (isRenderOutdated())
-                return;
             // Get pageContext from client-side hooks
             let pageContextFromClientHooks;
             try {

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

@@ -1 +1 @@
-export declare const PROJECT_VERSION: "0.4.229-commit-847f1b6";
+export declare const PROJECT_VERSION: "0.4.229-commit-e1ca16d";

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-847f1b6';
+export const PROJECT_VERSION = '0.4.229-commit-e1ca16d';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "vike",
-  "version": "0.4.229-commit-847f1b6",
+  "version": "0.4.229-commit-e1ca16d",
   "repository": "https://github.com/vikejs/vike",
   "exports": {
     "./server": {