vike 0.4.206 → 0.4.208-commit-8d0a7ba

package/dist/cjs/__internal/index.js

@@ -2,7 +2,7 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.route = route;
 exports.getPagesAndRoutes = getPagesAndRoutes;
-exports.getPageFilesAllSafe = getPageFilesAllSafe;
+exports.getMiddlewares = getMiddlewares;
 const index_js_1 = require("../shared/route/index.js");
 const getPageFiles_js_1 = require("../shared/getPageFiles.js");
 const globalContext_js_1 = require("../node/runtime/globalContext.js");
@@ -27,7 +27,18 @@ async function getPagesAndRoutes() {
         allPageIds
     };
 }
-// TODO: make it cleaner once the internal refactoring about global configs is done.
+// TODO/eventually:
+//  - Remove the need for `isProduction` after Vike's CLI is implemented
+//  - Remove it in favor of https://vike.dev/getGlobalContext
+async function getMiddlewares() {
+    const isProduction = process.env.NODE_ENV === 'production';
+    const { pageConfigs } = await getPageFilesAllSafe(isProduction);
+    const middlewares = pageConfigs[0].configValues.middleware.value.flat(Infinity);
+    return middlewares;
+}
+// TODO/eventually:
+//  - Make it cleaner once the internal refactoring about global configs is done.
+//  - Remove it in favor of https://vike.dev/getGlobalContext
 // Demo usage: https://github.com/vikejs/vike/pull/1823
 async function getPageFilesAllSafe(isProduction) {
     await (0, globalContext_js_1.initGlobalContext_getGlobalContextAsync)(isProduction);

package/dist/cjs/node/plugin/index.js

@@ -62,13 +62,6 @@ function plugin(vikeConfig) {
     ];
     return plugins;
 }
-// Enable `const vike = require('vike/plugin')`.
-//  - This lives at the end of the file to ensure it happens after all assignments to `exports`.
-//  - This is only used for the CJS build; we wrap it in a try-catch for the ESM build.
-try {
-    module.exports = Object.assign(exports.default, exports);
-}
-catch { }
 // Error upon wrong usage
 Object.defineProperty(plugin, 'apply', {
     enumerable: true,
@@ -80,3 +73,9 @@ Object.defineProperty(plugin, 'apply', {
 function assertViteVersion() {
     (0, utils_js_2.assertVersion)('Vite', vite_1.version, '5.1.0');
 }
+// Ensures following works: `const vike = require('vike/plugin')` / `import vike from 'vike/plugin'`
+//  - It needs to live at the end of this file, in order to ensure we do it after all assignments to `exports`.
+try {
+    module.exports = Object.assign(exports.default, exports);
+}
+catch { }

package/dist/cjs/node/plugin/plugins/buildConfig/fixServerAssets.js

@@ -5,6 +5,7 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.fixServerAssets = fixServerAssets;
 exports.fixServerAssets_isEnabled = fixServerAssets_isEnabled;
+exports.fixServerAssets_assertCssCodeSplit = fixServerAssets_assertCssCodeSplit;
 exports.fixServerAssets_assertCssTarget = fixServerAssets_assertCssTarget;
 exports.fixServerAssets_assertCssTarget_populate = fixServerAssets_assertCssTarget_populate;
 const promises_1 = __importDefault(require("fs/promises"));
@@ -157,6 +158,10 @@ function getHash(src) {
     (0, utils_js_1.assert)(hash);
     return hash;
 }
+// https://github.com/vikejs/vike/issues/1993
+function fixServerAssets_assertCssCodeSplit(config) {
+    (0, utils_js_1.assertWarning)(config.build.cssCodeSplit, `${picocolors_1.default.cyan('build.cssCodeSplit')} shouldn't be set to ${picocolors_1.default.cyan('false')} (https://github.com/vikejs/vike/issues/1993)`, { onlyOnce: true });
+}
 const targets = [];
 function fixServerAssets_assertCssTarget_populate(config) {
     const isServerSide = (0, utils_js_1.viteIsSSR)(config);
@@ -176,10 +181,9 @@ async function fixServerAssets_assertCssTarget(config) {
         targetsServer.forEach((targetServer) => {
             const targetCssResolvedServer = resolveCssTarget(targetServer);
             (0, utils_js_1.assertWarning)((0, utils_js_1.isEqualStringList)(targetCssResolvedClient, targetCssResolvedServer), [
-                'The CSS browser target must be the same for both client-side and server-side, but we got:',
+                'The CSS browser target should be the same for both client-side and server-side (https://github.com/vikejs/vike/issues/1815#issuecomment-2507002979) but we got:',
                 `Client-side: ${picocolors_1.default.cyan(JSON.stringify(targetCssResolvedClient))}`,
-                `Server-side: ${picocolors_1.default.cyan(JSON.stringify(targetCssResolvedServer))}`,
-                'See https://github.com/vikejs/vike/issues/1815#issuecomment-2507002979 if you want to know why.'
+                `Server-side: ${picocolors_1.default.cyan(JSON.stringify(targetCssResolvedServer))}`
             ].join('\n'), {
                 showStackTrace: true,
                 onlyOnce: 'different-css-target'

package/dist/cjs/node/plugin/plugins/buildConfig.js

@@ -55,6 +55,7 @@ function buildConfig() {
                             // Required if `ssrEmitAssets: true`, see https://github.com/vitejs/vite/pull/11430#issuecomment-1454800934
                             config.build.cssMinify = 'esbuild';
                             (0, fixServerAssets_js_1.fixServerAssets_assertCssTarget_populate)(config);
+                            (0, fixServerAssets_js_1.fixServerAssets_assertCssCodeSplit)(config);
                         }
                     }
                 }

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

@@ -131,7 +131,9 @@ const configDefinitionsBuiltIn = {
     },
     keepScrollPosition: {
         env: { client: true }
-    }
+    },
+    // TODO/eventually: define it as a global config.
+    middleware: { env: { server: true }, cumulative: true, eager: true }
 };
 exports.configDefinitionsBuiltIn = configDefinitionsBuiltIn;
 const configDefinitionsBuiltInGlobal = {

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

@@ -81,37 +81,35 @@ function processHookReturnValue(hookReturnValue, renderHook) {
     let pageContextPromise = null;
     let pageContextProvidedByRenderHook = null;
     let injectFilter = null;
-    const ret = () => ({ documentHtml, pageContextProvidedByRenderHook, pageContextPromise, injectFilter });
-    if (hookReturnValue === null)
-        return ret();
     if ((0, renderHtml_js_1.isDocumentHtml)(hookReturnValue)) {
         documentHtml = hookReturnValue;
-        return ret();
+        return { documentHtml, pageContextProvidedByRenderHook, pageContextPromise, injectFilter };
     }
     const errPrefix = `The ${renderHook.hookName}() hook defined at ${renderHook.hookFilePath}`;
-    const errSuffix = `a string generated with the ${picocolors_1.default.cyan('escapeInject`<html>...</html>`')} template tag or a string returned by ${picocolors_1.default.cyan('dangerouslySkipEscape()')}, see https://vike.dev/escapeInject`;
+    const errSuffix = `a string generated with ${picocolors_1.default.cyan('escapeInject`<html>...</html>`')} or the value returned by ${picocolors_1.default.cyan('dangerouslySkipEscape()')}, see https://vike.dev/escapeInject`;
     if (typeof hookReturnValue === 'string') {
-        (0, utils_js_1.assertWarning)(false, [errPrefix, 'returned a plain JavaScript string which is dangerous: it should instead return', errSuffix].join(' '), { onlyOnce: true });
+        (0, utils_js_1.assertWarning)(false, [
+            errPrefix,
+            `returned a plain JavaScript string which is ${picocolors_1.default.red(picocolors_1.default.bold('dangerous'))}: it should instead return`,
+            errSuffix
+        ].join(' '), { onlyOnce: true });
         hookReturnValue = (0, renderHtml_js_1.dangerouslySkipEscape)(hookReturnValue);
     }
-    (0, utils_js_1.assertUsage)((0, utils_js_1.isObject)(hookReturnValue), [
-        errPrefix,
-        `should return ${picocolors_1.default.cyan('null')}, the value ${picocolors_1.default.cyan('documentHtml')}, or an object ${picocolors_1.default.cyan('{ documentHtml, pageContext }')} where ${picocolors_1.default.cyan('pageContext')} is ${picocolors_1.default.cyan('undefined')} or an object holding additional pageContext values, and where ${picocolors_1.default.cyan('documentHtml')} is`,
-        errSuffix
-    ].join(' '));
+    const wrongReturnValue = `should return the value ${picocolors_1.default.cyan('documentHtml')} or an object ${picocolors_1.default.cyan('{ documentHtml }')} where ${picocolors_1.default.cyan('documentHtml')} is ${errSuffix}`;
+    (0, utils_js_1.assertUsage)((0, utils_js_1.isObject)(hookReturnValue), `${errPrefix} ${wrongReturnValue}`);
     (0, assertHookReturnedObject_js_1.assertHookReturnedObject)(hookReturnValue, ['documentHtml', 'pageContext', 'injectFilter'], errPrefix);
+    (0, utils_js_1.assertUsage)(hookReturnValue.documentHtml, `${errPrefix} returned an object that is missing the ${picocolors_1.default.code('documentHtml')} property: it ${wrongReturnValue}`);
     if (hookReturnValue.injectFilter) {
         (0, utils_js_1.assertUsage)((0, utils_js_1.isCallable)(hookReturnValue.injectFilter), 'injectFilter should be a function');
         injectFilter = hookReturnValue.injectFilter;
     }
-    (0, utils_js_1.assertUsage)(hookReturnValue.documentHtml, `${errPrefix} returned an object that is missing the ${picocolors_1.default.code('documentHtml')} property.`);
     {
         let val = hookReturnValue.documentHtml;
         const errBegin = `${errPrefix} returned ${picocolors_1.default.cyan('{ documentHtml }')}, but ${picocolors_1.default.cyan('documentHtml')}`;
         if (typeof val === 'string') {
             (0, utils_js_1.assertWarning)(false, [
                 errBegin,
-                `is a plain JavaScript string which is dangerous: ${picocolors_1.default.cyan('documentHtml')} should be`,
+                `is a plain JavaScript string which is ${picocolors_1.default.bold(picocolors_1.default.red('dangerous'))}: ${picocolors_1.default.cyan('documentHtml')} should be`,
                 errSuffix
             ].join(' '), { onlyOnce: true });
             val = (0, renderHtml_js_1.dangerouslySkipEscape)(val);
@@ -132,5 +130,5 @@ function processHookReturnValue(hookReturnValue, renderHook) {
             pageContextProvidedByRenderHook = val;
         }
     }
-    return ret();
+    return { documentHtml, pageContextProvidedByRenderHook, pageContextPromise, injectFilter };
 }

package/dist/cjs/shared/page-configs/assertPlusFileExport.js

@@ -6,34 +6,35 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.assertPlusFileExport = assertPlusFileExport;
 const utils_js_1 = require("../utils.js");
 const picocolors_1 = __importDefault(require("@brillout/picocolors"));
-const EXPORTS_IGNORE = [
+const SIDE_EXPORTS_TOLERATE = [
     // vite-plugin-solid adds `export { $$registrations }`
     '$$registrations',
     // @vitejs/plugin-vue adds `export { _rerender_only }`
     '_rerender_only'
 ];
 // Tolerate `export { frontmatter }` in .mdx files
-const TOLERATE_SIDE_EXPORTS = ['.md', '.mdx'];
+const SIDE_EXPORTS_DO_NOT_CHECK = ['.md', '.mdx'];
 function assertPlusFileExport(fileExports, filePathToShowToUser, configName) {
-    const exportNames = Object.keys(fileExports).filter((exportName) => !EXPORTS_IGNORE.includes(exportName));
+    const exportNames = Object.keys(fileExports);
     const isValid = (exportName) => exportName === 'default' || exportName === configName;
+    // Error upon missing/ambiguous export
     const exportNamesValid = exportNames.filter(isValid);
-    const exportNamesInvalid = exportNames.filter((e) => !isValid(e));
-    if (exportNamesValid.length === 1 && exportNamesInvalid.length === 0) {
-        return;
-    }
     const exportDefault = picocolors_1.default.code('export default');
     const exportNamed = picocolors_1.default.code(`export { ${configName} }`);
-    (0, utils_js_1.assert)(exportNamesValid.length <= 2);
     if (exportNamesValid.length === 0) {
         (0, utils_js_1.assertUsage)(false, `${filePathToShowToUser} should have ${exportNamed} or ${exportDefault}`);
     }
     if (exportNamesValid.length === 2) {
         (0, utils_js_1.assertUsage)(false, `${filePathToShowToUser} is ambiguous: remove ${exportDefault} or ${exportNamed}`);
     }
-    if (!TOLERATE_SIDE_EXPORTS.some((ext) => filePathToShowToUser.endsWith(ext))) {
+    (0, utils_js_1.assert)(exportNamesValid.length === 1);
+    // Warn upon side exports
+    const exportNamesInvalid = exportNames
+        .filter((e) => !isValid(e))
+        .filter((exportName) => !SIDE_EXPORTS_TOLERATE.includes(exportName));
+    if (!SIDE_EXPORTS_DO_NOT_CHECK.some((ext) => filePathToShowToUser.endsWith(ext))) {
         exportNamesInvalid.forEach((exportInvalid) => {
-            (0, utils_js_1.assertWarning)(false, `${filePathToShowToUser} unexpected ${picocolors_1.default.cyan(`export { ${exportInvalid} }`)}`, {
+            (0, utils_js_1.assertWarning)(false, `${filePathToShowToUser} unexpected ${picocolors_1.default.cyan(`export { ${exportInvalid} }`)}, see https://vike.dev/no-side-exports`, {
                 onlyOnce: true
             });
         });

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.206';
+exports.PROJECT_VERSION = '0.4.208-commit-8d0a7ba';

package/dist/cjs/utils/assert.js

@@ -48,7 +48,7 @@ function assert(condition, debugInfo) {
     })();
     const link = picocolors_1.default.blue('https://github.com/vikejs/vike/issues/new');
     let errMsg = [
-        `You stumbled upon a Vike bug. Go to ${link} and copy-paste this error. A maintainer will fix the bug (usually under 24 hours).`,
+        `You stumbled upon a Vike bug. Go to ${link} and copy-paste this error. A maintainer will fix the bug (usually within 24 hours).`,
         debugStr
     ]
         .filter(Boolean)

package/dist/cjs/utils/path-shim.js

@@ -12,3 +12,8 @@ function pathJoin(path1, path2) {
         joined = '/' + joined;
     return joined;
 }
+/* https://github.com/brillout/telefunc/blob/0fd44322acbd07857ae29361ba7c998607f17dd5/telefunc/utils/path-shim.ts#L17-L21
+function isAbsolute(filePath: string) {
+  // ...
+}
+*/

package/dist/esm/__internal/index.d.ts

@@ -1,6 +1,6 @@
 export { route, getPagesAndRoutes };
 export type { PageRoutes, PageFile, PageConfigRuntime as PageConfig };
-export { getPageFilesAllSafe };
+export { getMiddlewares };
 import { route as routeInternal, type PageRoutes } from '../shared/route/index.js';
 import { type PageFile } from '../shared/getPageFiles.js';
 import { PageConfigRuntime } from '../shared/page-configs/PageConfig.js';
@@ -16,12 +16,7 @@ declare function getPagesAndRoutes(): Promise<{
     pageConfigs: PageConfigRuntime[];
     allPageIds: string[];
 }>;
-declare function getPageFilesAllSafe(isProduction: boolean): Promise<{
-    pageFilesAll: PageFile[];
-    allPageIds: string[];
-    pageConfigs: PageConfigRuntime[];
-    pageConfigGlobal: import("../shared/page-configs/PageConfig.js").PageConfigGlobalRuntime;
-}>;
+declare function getMiddlewares(): Promise<unknown[]>;
 declare function route(pageContext: Parameters<typeof routeInternal>[0]): Promise<{
     pageContextAddendum: import("../shared/route/index.js").PageContextFromRoute;
 }>;

package/dist/esm/__internal/index.js

@@ -1,7 +1,7 @@
 // Internals needed by vite-plugin-vercel
 export { route, getPagesAndRoutes };
-// Internals needed by nitedani's experimental implementation of vike-telefunc
-export { getPageFilesAllSafe };
+// Internals needed by experimental implementation of vike-telefunc
+export { getMiddlewares };
 import { route as routeInternal } from '../shared/route/index.js';
 import { getPageFilesAll } from '../shared/getPageFiles.js';
 import { getGlobalContext, initGlobalContext_getGlobalContextAsync } from '../node/runtime/globalContext.js';
@@ -26,7 +26,18 @@ async function getPagesAndRoutes() {
         allPageIds
     };
 }
-// TODO: make it cleaner once the internal refactoring about global configs is done.
+// TODO/eventually:
+//  - Remove the need for `isProduction` after Vike's CLI is implemented
+//  - Remove it in favor of https://vike.dev/getGlobalContext
+async function getMiddlewares() {
+    const isProduction = process.env.NODE_ENV === 'production';
+    const { pageConfigs } = await getPageFilesAllSafe(isProduction);
+    const middlewares = pageConfigs[0].configValues.middleware.value.flat(Infinity);
+    return middlewares;
+}
+// TODO/eventually:
+//  - Make it cleaner once the internal refactoring about global configs is done.
+//  - Remove it in favor of https://vike.dev/getGlobalContext
 // Demo usage: https://github.com/vikejs/vike/pull/1823
 async function getPageFilesAllSafe(isProduction) {
     await initGlobalContext_getGlobalContextAsync(isProduction);

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

@@ -218,6 +218,7 @@ async function renderPageClientSide(renderArgs) {
             isBackwardNavigation,
             isClientSideNavigation,
             isHydration: isFirstRender && !isForErrorPage,
+            // Make it public as `pageContext.previous`? Maybe after https://github.com/vikejs/vike/issues/1268
             _previousPageContext: previousPageContext,
             ...pageContextInitClient
         });

package/dist/esm/node/plugin/index.js

@@ -56,13 +56,6 @@ function plugin(vikeConfig) {
     ];
     return plugins;
 }
-// Enable `const vike = require('vike/plugin')`.
-//  - This lives at the end of the file to ensure it happens after all assignments to `exports`.
-//  - This is only used for the CJS build; we wrap it in a try-catch for the ESM build.
-try {
-    module.exports = Object.assign(exports.default, exports);
-}
-catch { }
 // Error upon wrong usage
 Object.defineProperty(plugin, 'apply', {
     enumerable: true,
@@ -74,3 +67,9 @@ Object.defineProperty(plugin, 'apply', {
 function assertViteVersion() {
     assertVersion('Vite', version, '5.1.0');
 }
+// Ensures following works: `const vike = require('vike/plugin')` / `import vike from 'vike/plugin'`
+//  - It needs to live at the end of this file, in order to ensure we do it after all assignments to `exports`.
+try {
+    module.exports = Object.assign(exports.default, exports);
+}
+catch { }

package/dist/esm/node/plugin/plugins/buildConfig/fixServerAssets.d.ts

@@ -1,5 +1,6 @@
 export { fixServerAssets };
 export { fixServerAssets_isEnabled };
+export { fixServerAssets_assertCssCodeSplit };
 export { fixServerAssets_assertCssTarget };
 export { fixServerAssets_assertCssTarget_populate };
 import { ViteManifest } from '../../../shared/ViteManifest.js';
@@ -13,5 +14,6 @@ import { ResolvedConfig } from 'vite';
 declare function fixServerAssets_isEnabled(): boolean;
 /** https://github.com/vikejs/vike/issues/1339 */
 declare function fixServerAssets(config: ResolvedConfig): Promise<ViteManifest>;
+declare function fixServerAssets_assertCssCodeSplit(config: ResolvedConfig): void;
 declare function fixServerAssets_assertCssTarget_populate(config: ResolvedConfig): void;
 declare function fixServerAssets_assertCssTarget(config: ResolvedConfig): Promise<void>;

package/dist/esm/node/plugin/plugins/buildConfig/fixServerAssets.js

@@ -1,5 +1,6 @@
 export { fixServerAssets };
 export { fixServerAssets_isEnabled };
+export { fixServerAssets_assertCssCodeSplit };
 export { fixServerAssets_assertCssTarget };
 export { fixServerAssets_assertCssTarget_populate };
 import fs from 'fs/promises';
@@ -152,6 +153,10 @@ function getHash(src) {
     assert(hash);
     return hash;
 }
+// https://github.com/vikejs/vike/issues/1993
+function fixServerAssets_assertCssCodeSplit(config) {
+    assertWarning(config.build.cssCodeSplit, `${pc.cyan('build.cssCodeSplit')} shouldn't be set to ${pc.cyan('false')} (https://github.com/vikejs/vike/issues/1993)`, { onlyOnce: true });
+}
 const targets = [];
 function fixServerAssets_assertCssTarget_populate(config) {
     const isServerSide = viteIsSSR(config);
@@ -171,10 +176,9 @@ async function fixServerAssets_assertCssTarget(config) {
         targetsServer.forEach((targetServer) => {
             const targetCssResolvedServer = resolveCssTarget(targetServer);
             assertWarning(isEqualStringList(targetCssResolvedClient, targetCssResolvedServer), [
-                'The CSS browser target must be the same for both client-side and server-side, but we got:',
+                'The CSS browser target should be the same for both client-side and server-side (https://github.com/vikejs/vike/issues/1815#issuecomment-2507002979) but we got:',
                 `Client-side: ${pc.cyan(JSON.stringify(targetCssResolvedClient))}`,
-                `Server-side: ${pc.cyan(JSON.stringify(targetCssResolvedServer))}`,
-                'See https://github.com/vikejs/vike/issues/1815#issuecomment-2507002979 if you want to know why.'
+                `Server-side: ${pc.cyan(JSON.stringify(targetCssResolvedServer))}`
             ].join('\n'), {
                 showStackTrace: true,
                 onlyOnce: 'different-css-target'

package/dist/esm/node/plugin/plugins/buildConfig.js

@@ -11,7 +11,7 @@ import { extractAssetsAddQuery } from '../../shared/extractAssetsQuery.js';
 import { createRequire } from 'module';
 import fs from 'fs/promises';
 import path from 'path';
-import { fixServerAssets, fixServerAssets_assertCssTarget, fixServerAssets_assertCssTarget_populate, fixServerAssets_isEnabled } from './buildConfig/fixServerAssets.js';
+import { fixServerAssets, fixServerAssets_assertCssCodeSplit, fixServerAssets_assertCssTarget, fixServerAssets_assertCssTarget_populate, fixServerAssets_isEnabled } from './buildConfig/fixServerAssets.js';
 import { set_ASSETS_MAP } from './importBuild/index.js';
 import { prependEntriesDir } from '../../shared/prependEntriesDir.js';
 import { getFilePathResolved } from '../shared/getFilePath.js';
@@ -49,6 +49,7 @@ function buildConfig() {
                             // Required if `ssrEmitAssets: true`, see https://github.com/vitejs/vite/pull/11430#issuecomment-1454800934
                             config.build.cssMinify = 'esbuild';
                             fixServerAssets_assertCssTarget_populate(config);
+                            fixServerAssets_assertCssCodeSplit(config);
                         }
                     }
                 }

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

@@ -130,7 +130,9 @@ const configDefinitionsBuiltIn = {
     },
     keepScrollPosition: {
         env: { client: true }
-    }
+    },
+    // TODO/eventually: define it as a global config.
+    middleware: { env: { server: true }, cumulative: true, eager: true }
 };
 const configDefinitionsBuiltInGlobal = {
     onPrerenderStart: {

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

@@ -76,37 +76,35 @@ function processHookReturnValue(hookReturnValue, renderHook) {
     let pageContextPromise = null;
     let pageContextProvidedByRenderHook = null;
     let injectFilter = null;
-    const ret = () => ({ documentHtml, pageContextProvidedByRenderHook, pageContextPromise, injectFilter });
-    if (hookReturnValue === null)
-        return ret();
     if (isDocumentHtml(hookReturnValue)) {
         documentHtml = hookReturnValue;
-        return ret();
+        return { documentHtml, pageContextProvidedByRenderHook, pageContextPromise, injectFilter };
     }
     const errPrefix = `The ${renderHook.hookName}() hook defined at ${renderHook.hookFilePath}`;
-    const errSuffix = `a string generated with the ${pc.cyan('escapeInject`<html>...</html>`')} template tag or a string returned by ${pc.cyan('dangerouslySkipEscape()')}, see https://vike.dev/escapeInject`;
+    const errSuffix = `a string generated with ${pc.cyan('escapeInject`<html>...</html>`')} or the value returned by ${pc.cyan('dangerouslySkipEscape()')}, see https://vike.dev/escapeInject`;
     if (typeof hookReturnValue === 'string') {
-        assertWarning(false, [errPrefix, 'returned a plain JavaScript string which is dangerous: it should instead return', errSuffix].join(' '), { onlyOnce: true });
+        assertWarning(false, [
+            errPrefix,
+            `returned a plain JavaScript string which is ${pc.red(pc.bold('dangerous'))}: it should instead return`,
+            errSuffix
+        ].join(' '), { onlyOnce: true });
         hookReturnValue = dangerouslySkipEscape(hookReturnValue);
     }
-    assertUsage(isObject(hookReturnValue), [
-        errPrefix,
-        `should return ${pc.cyan('null')}, the value ${pc.cyan('documentHtml')}, or an object ${pc.cyan('{ documentHtml, pageContext }')} where ${pc.cyan('pageContext')} is ${pc.cyan('undefined')} or an object holding additional pageContext values, and where ${pc.cyan('documentHtml')} is`,
-        errSuffix
-    ].join(' '));
+    const wrongReturnValue = `should return the value ${pc.cyan('documentHtml')} or an object ${pc.cyan('{ documentHtml }')} where ${pc.cyan('documentHtml')} is ${errSuffix}`;
+    assertUsage(isObject(hookReturnValue), `${errPrefix} ${wrongReturnValue}`);
     assertHookReturnedObject(hookReturnValue, ['documentHtml', 'pageContext', 'injectFilter'], errPrefix);
+    assertUsage(hookReturnValue.documentHtml, `${errPrefix} returned an object that is missing the ${pc.code('documentHtml')} property: it ${wrongReturnValue}`);
     if (hookReturnValue.injectFilter) {
         assertUsage(isCallable(hookReturnValue.injectFilter), 'injectFilter should be a function');
         injectFilter = hookReturnValue.injectFilter;
     }
-    assertUsage(hookReturnValue.documentHtml, `${errPrefix} returned an object that is missing the ${pc.code('documentHtml')} property.`);
     {
         let val = hookReturnValue.documentHtml;
         const errBegin = `${errPrefix} returned ${pc.cyan('{ documentHtml }')}, but ${pc.cyan('documentHtml')}`;
         if (typeof val === 'string') {
             assertWarning(false, [
                 errBegin,
-                `is a plain JavaScript string which is dangerous: ${pc.cyan('documentHtml')} should be`,
+                `is a plain JavaScript string which is ${pc.bold(pc.red('dangerous'))}: ${pc.cyan('documentHtml')} should be`,
                 errSuffix
             ].join(' '), { onlyOnce: true });
             val = dangerouslySkipEscape(val);
@@ -127,5 +125,5 @@ function processHookReturnValue(hookReturnValue, renderHook) {
             pageContextProvidedByRenderHook = val;
         }
     }
-    return ret();
+    return { documentHtml, pageContextProvidedByRenderHook, pageContextPromise, injectFilter };
 }

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

@@ -43,7 +43,7 @@ type HookName = HookNamePage | HookNameGlobal | HookNameOldDesign;
 type HookNamePage = 'onHydrationEnd' | 'onBeforePrerenderStart' | 'onBeforeRender' | 'onPageTransitionStart' | 'onPageTransitionEnd' | 'onRenderHtml' | 'onRenderClient' | 'guard' | 'data';
 type HookNameGlobal = 'onBeforePrerender' | 'onBeforeRoute' | 'onPrerenderStart';
 type HookNameOldDesign = 'render' | 'prerender';
-type ConfigNameBuiltIn = Exclude<keyof Config, keyof ConfigVikeUserProvided | 'onBeforeRoute' | 'onPrerenderStart'> | 'prerender' | 'isClientRuntimeLoaded' | 'onBeforeRenderEnv' | 'dataEnv' | 'hooksTimeout' | 'clientHooks';
+type ConfigNameBuiltIn = Exclude<keyof Config, keyof ConfigVikeUserProvided | 'onBeforeRoute' | 'onPrerenderStart'> | 'prerender' | 'isClientRuntimeLoaded' | 'onBeforeRenderEnv' | 'dataEnv' | 'hooksTimeout' | 'clientHooks' | 'middleware';
 type Config = ConfigBuiltIn & Vike.Config & (VikePackages.ConfigVikeReact | VikePackages.ConfigVikeVue | VikePackages.ConfigVikeSolid | VikePackages.ConfigVikeSvelte | VikePackages.ConfigVikeAngular);
 /** Hook for fetching data.
  *

package/dist/esm/shared/page-configs/assertPlusFileExport.js

@@ -1,34 +1,35 @@
 export { assertPlusFileExport };
 import { assert, assertUsage, assertWarning } from '../utils.js';
 import pc from '@brillout/picocolors';
-const EXPORTS_IGNORE = [
+const SIDE_EXPORTS_TOLERATE = [
     // vite-plugin-solid adds `export { $$registrations }`
     '$$registrations',
     // @vitejs/plugin-vue adds `export { _rerender_only }`
     '_rerender_only'
 ];
 // Tolerate `export { frontmatter }` in .mdx files
-const TOLERATE_SIDE_EXPORTS = ['.md', '.mdx'];
+const SIDE_EXPORTS_DO_NOT_CHECK = ['.md', '.mdx'];
 function assertPlusFileExport(fileExports, filePathToShowToUser, configName) {
-    const exportNames = Object.keys(fileExports).filter((exportName) => !EXPORTS_IGNORE.includes(exportName));
+    const exportNames = Object.keys(fileExports);
     const isValid = (exportName) => exportName === 'default' || exportName === configName;
+    // Error upon missing/ambiguous export
     const exportNamesValid = exportNames.filter(isValid);
-    const exportNamesInvalid = exportNames.filter((e) => !isValid(e));
-    if (exportNamesValid.length === 1 && exportNamesInvalid.length === 0) {
-        return;
-    }
     const exportDefault = pc.code('export default');
     const exportNamed = pc.code(`export { ${configName} }`);
-    assert(exportNamesValid.length <= 2);
     if (exportNamesValid.length === 0) {
         assertUsage(false, `${filePathToShowToUser} should have ${exportNamed} or ${exportDefault}`);
     }
     if (exportNamesValid.length === 2) {
         assertUsage(false, `${filePathToShowToUser} is ambiguous: remove ${exportDefault} or ${exportNamed}`);
     }
-    if (!TOLERATE_SIDE_EXPORTS.some((ext) => filePathToShowToUser.endsWith(ext))) {
+    assert(exportNamesValid.length === 1);
+    // Warn upon side exports
+    const exportNamesInvalid = exportNames
+        .filter((e) => !isValid(e))
+        .filter((exportName) => !SIDE_EXPORTS_TOLERATE.includes(exportName));
+    if (!SIDE_EXPORTS_DO_NOT_CHECK.some((ext) => filePathToShowToUser.endsWith(ext))) {
         exportNamesInvalid.forEach((exportInvalid) => {
-            assertWarning(false, `${filePathToShowToUser} unexpected ${pc.cyan(`export { ${exportInvalid} }`)}`, {
+            assertWarning(false, `${filePathToShowToUser} unexpected ${pc.cyan(`export { ${exportInvalid} }`)}, see https://vike.dev/no-side-exports`, {
                 onlyOnce: true
             });
         });

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

@@ -1 +1 @@
-export declare const PROJECT_VERSION: "0.4.206";
+export declare const PROJECT_VERSION: "0.4.208-commit-8d0a7ba";

package/dist/esm/utils/PROJECT_VERSION.js

@@ -1,2 +1,2 @@
 // Automatically updated by @brillout/release-me
-export const PROJECT_VERSION = '0.4.206';
+export const PROJECT_VERSION = '0.4.208-commit-8d0a7ba';

package/dist/esm/utils/assert.js

@@ -43,7 +43,7 @@ function assert(condition, debugInfo) {
     })();
     const link = pc.blue('https://github.com/vikejs/vike/issues/new');
     let errMsg = [
-        `You stumbled upon a Vike bug. Go to ${link} and copy-paste this error. A maintainer will fix the bug (usually under 24 hours).`,
+        `You stumbled upon a Vike bug. Go to ${link} and copy-paste this error. A maintainer will fix the bug (usually within 24 hours).`,
         debugStr
     ]
         .filter(Boolean)

package/dist/esm/utils/path-shim.js

@@ -10,3 +10,8 @@ function pathJoin(path1, path2) {
         joined = '/' + joined;
     return joined;
 }
+/* https://github.com/brillout/telefunc/blob/0fd44322acbd07857ae29361ba7c998607f17dd5/telefunc/utils/path-shim.ts#L17-L21
+function isAbsolute(filePath: string) {
+  // ...
+}
+*/

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

@@ -1,4 +1,4 @@
 export declare const projectInfo: {
     projectName: "Vike";
-    projectVersion: "0.4.206";
+    projectVersion: "0.4.208-commit-8d0a7ba";
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "vike",
-  "version": "0.4.206",
+  "version": "0.4.208-commit-8d0a7ba",
   "repository": "https://github.com/vikejs/vike",
   "exports": {
     "./server": {
@@ -122,10 +122,10 @@
   },
   "dependencies": {
     "@brillout/import": "^0.2.3",
-    "@brillout/json-serializer": "^0.5.13",
+    "@brillout/json-serializer": "^0.5.15",
     "@brillout/picocolors": "^1.0.15",
     "@brillout/require-shim": "^0.1.2",
-    "@brillout/vite-plugin-server-entry": "^0.5.0",
+    "@brillout/vite-plugin-server-entry": "^0.5.5",
     "acorn": "^8.0.0",
     "cac": "^6.0.0",
     "es-module-lexer": "^1.0.0",
@@ -231,7 +231,7 @@
     "@types/resolve": "^1.20.6",
     "@types/semver": "^7.5.8",
     "@types/source-map-support": "^0.5.10",
-    "react-streaming": "^0.3.43",
+    "react-streaming": "^0.3.44",
     "rimraf": "^5.0.5",
     "typescript": "^5.6.2",
     "vite": "^5.4.0"