vike 0.4.251 → 0.4.252-commit-1d0a050

package/dist/node/vite/plugins/pluginReplaceConstantsEnvVars.js

@@ -21,19 +21,18 @@ const PUBLIC_ENV_ALLOWLIST = [
 //   - For it to work, we'll probably need the user to define the settings (e.g. `envDir`) for loadEnv() inside vike.config.js instead of vite.config.js
 //   - Or stop using Vite's `mode` implementation and have Vike implement its own `mode` feature? (So that the only dependencies are `$ vike build --mode staging` and `$ MODE=staging vike build`.)
 // === Rolldown filter
-const skipNodeModules = '/node_modules/';
 const skipIrrelevant = 'import.meta.env.';
 const filterRolldown = {
+    /* We don't do that, because vike-react-sentry uses import.meta.env.PUBLIC_ENV__SENTRY_DSN
     id: {
-        exclude: `**${skipNodeModules}**`,
+      exclude: `**${'/node_modules/'}**`,
     },
+    */
     code: {
         include: skipIrrelevant,
     },
 };
-const filterFunction = (id, code) => {
-    if (id.includes(skipNodeModules))
-        return false;
+const filterFunction = (code) => {
     if (!code.includes(skipIrrelevant))
         return false;
     return true;
@@ -62,10 +61,7 @@ function pluginReplaceConstantsEnvVars() {
                 handler(code, id, options) {
                     id = normalizeId(id);
                     assertPosixPath(id);
-                    assertPosixPath(config.root);
-                    if (!id.startsWith(config.root))
-                        return; // skip linked dependencies
-                    assert(filterFunction(id, code));
+                    assert(filterFunction(code));
                     const isBuild = config.command === 'build';
                     const isClientSide = !isViteServerSide_extraSafe(config, this.environment, options);
                     const { magicString, getMagicStringResult } = getMagicString(code, id);

package/dist/node/vite/shared/resolveVikeConfigInternal/transpileAndExecuteFile.js

@@ -9,7 +9,7 @@ import path from 'node:path';
 import crypto from 'node:crypto';
 import pc from '@brillout/picocolors';
 import { import_ } from '@brillout/import';
-import { assert, assertWarning, assertUsage } from '../../../../utils/assert.js';
+import { assert, assertWarning } from '../../../../utils/assert.js';
 import { assertIsNotProductionRuntime } from '../../../../utils/assertSetup.js';
 import { createDebug } from '../../../../utils/debug.js';
 import { genPromise } from '../../../../utils/genPromise.js';
@@ -39,7 +39,12 @@ async function transpileAndExecuteFile(filePath, userRootDir, isExtensionConfig,
     }
     const { promise, resolve } = genPromise();
     esbuildCache.transpileCache[filePathAbsoluteFilesystem] = promise;
-    assertUsage(isPlainScriptFile(filePathAbsoluteFilesystem), `${filePathToShowToUserResolved} has file extension .${fileExtension} but a config file can only be a JavaScript/TypeScript file`);
+    /* We tolerate .tsx so that a file can be both a runtime and config file (`meta.env.config === true && meta.env.server === true`), e.g. https://github.com/brillout/docpress/issues/86
+    assertUsage(
+      isPlainScriptFile(filePathAbsoluteFilesystem),
+      `${filePathToShowToUserResolved} has file extension .${fileExtension} but a config file can only be a JavaScript/TypeScript file`,
+    )
+    */
     const isHeader = isHeaderFile(filePathAbsoluteFilesystem);
     if (isHeader) {
         assertWarning(false, `${pc.cyan('.h.js')} files are deprecated: simply renaming ${filePathToShowToUserResolved} to ${removeHeaderFileExtension(filePathToShowToUserResolved)} is usually enough, although you may occasionally need to use ${pc.cyan("with { type: 'pointer' }")} as explained at https://vike.dev/config#pointer-imports`, { onlyOnce: true });

package/dist/shared-server-client/getPublicProxy.d.ts

@@ -1,10 +1,10 @@
 export { getPublicProxy };
 export type { DangerouslyUseInternals };
-type ObjProxy<Obj> = {
+type ObjProxy<Obj> = Obj & {
     _isProxyObject: true;
     _originalObject: Obj;
 };
-declare function getPublicProxy<Obj extends Record<string | symbol, unknown>>(obj: Obj, objName: 'pageContext' | 'globalContext' | 'prerenderContext' | 'vikeConfig', skipOnInternalProp?: boolean, fallback?: (prop: string | symbol) => unknown): Obj & ObjProxy<Obj> & {
+declare function getPublicProxy<Obj extends Record<string | symbol, unknown>>(obj: Obj, objName: 'pageContext' | 'globalContext' | 'prerenderContext' | 'vikeConfig', skipOnInternalProp?: boolean, fallback?: (prop: string | symbol) => unknown): ObjProxy<Obj> & {
     /** https://vike.dev/warning/internals */
     dangerouslyUseInternals: DangerouslyUseInternals<Obj>;
 };

package/dist/utils/PROJECT_VERSION.d.ts

@@ -1 +1 @@
-export declare const PROJECT_VERSION: "0.4.251";
+export declare const PROJECT_VERSION: "0.4.252-commit-1d0a050";

package/dist/utils/PROJECT_VERSION.js

@@ -1,2 +1,2 @@
 // Automatically updated by @brillout/release-me
-export const PROJECT_VERSION = '0.4.251';
+export const PROJECT_VERSION = '0.4.252-commit-1d0a050';

package/dist/utils/isBrowser.d.ts

@@ -1 +1,2 @@
+/** Test whether the environment is a *real* browser (not a browser simulation such as `jsdom`). */
 export declare function isBrowser(): boolean;

package/dist/utils/isBrowser.js

@@ -1,5 +1,7 @@
+/** Test whether the environment is a *real* browser (not a browser simulation such as `jsdom`). */
 export function isBrowser() {
-    // Using `typeof window !== 'undefined'` alone is not enough because some users use https://www.npmjs.com/package/ssr-window
-    return typeof window !== 'undefined' && typeof window.scrollY === 'number';
-    // Alternatively, test whether environment is a *real* browser: https://github.com/brillout/picocolors/blob/d59a33a0fd52a8a33e4158884069192a89ce0113/picocolors.js#L87-L89
+    // - Using `typeof window !== 'undefined'` alone isn't narrow enough because some users use https://www.npmjs.com/package/ssr-window
+    // - Using `typeof window !== "undefined" && typeof window.scrollY === "number"` still isn't narrow enough because of jsdom
+    // - https://github.com/jsdom/jsdom/issues/1537#issuecomment-1689368267
+    return Object.getOwnPropertyDescriptor(globalThis, 'window')?.get?.toString().includes('[native code]') ?? false;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "vike",
-  "version": "0.4.251",
+  "version": "0.4.252-commit-1d0a050",
   "repository": "https://github.com/vikejs/vike",
   "exports": {
     "./server": {
@@ -153,7 +153,7 @@
     "node": ">=20.19.0"
   },
   "license": "MIT",
-  "description": "The Framework *You* Control - Next.js & Nuxt alternative for unprecedented flexibility and dependability.",
+  "description": "(Replaces Next.js/Nuxt) 🔨 Composable framework to build (advanced) applications with flexibility and stability.",
   "keywords": [
     "vite-plugin",
     "ssr",