@react-router/dev 8.0.1 → 8.1.0

package/CHANGELOG.md

@@ -1,5 +1,19 @@
 # `@react-router/dev`
 
+## v8.1.0
+
+### Patch Changes
+
+- Fix a regression with the new prerendering plugin where the `react-router.config.ts` `buildEnd` hook would run before prerendering was completed ([#15211](https://github.com/remix-run/react-router/pull/15211))
+- Fixed `react-router typegen` crashes under the Bun runtime when Babel default imports are already unwrapped ([#15214](https://github.com/remix-run/react-router/pull/15214))
+- Replace the deprecated `envFile:false` Vite config with `envDir:false` to eliminate a deprecation warning when using vite@8.1.0+ ([#15230](https://github.com/remix-run/react-router/pull/15230))
+- Only add the `"node"` Vite server condition for Framework mode apps that declare a Node server adapter dependency ([#15242](https://github.com/remix-run/react-router/pull/15242))
+  - This prevents non-Node SSR runtimes from resolving Node-specific package exports by default
+- Updated dependencies:
+  - [`react-router@8.1.0`](https://github.com/remix-run/react-router/releases/tag/react-router@8.1.0)
+  - [`@react-router/node@8.1.0`](https://github.com/remix-run/react-router/releases/tag/@react-router/node@8.1.0)
+  - [`@react-router/serve@8.1.0`](https://github.com/remix-run/react-router/releases/tag/@react-router/serve@8.1.0)
+
 ## v8.0.1
 
 ### Patch Changes

package/bin.cjs

@@ -1,13 +1,20 @@
 #!/usr/bin/env node
-void (async () => {
-  let { default: arg } = await import("arg");
+let { parseArgs } = require("node:util");
+
+const commands = new Set(["build", "dev", "reveal", "routes", "typegen"]);
 
+void (async () => {
   // Minimal replication of our actual parsing in `run.ts`. If not already set,
   // default `NODE_ENV` so React loads the proper version in its CJS entry script.
   // We have to do this before importing `run.ts` since that is what imports
   // `react` (indirectly via `react-router`)
-  let args = arg({}, { argv: process.argv.slice(2), permissive: true });
-  if (args._.length === 0 || args._[0] === "dev") {
+  let { positionals } = parseArgs({
+    args: process.argv.slice(2),
+    allowPositionals: true,
+    strict: false,
+  });
+  let command = positionals.find((positional) => commands.has(positional));
+  if (!command || command === "dev") {
     process.env.NODE_ENV = process.env.NODE_ENV ?? "development";
   } else {
     process.env.NODE_ENV = process.env.NODE_ENV ?? "production";

package/dist/{build-ygYOQjbA.js → build-9Do1SAoZ.js}

@@ -1,5 +1,5 @@
 /**
- * @react-router/dev v8.0.1
+ * @react-router/dev v8.1.0
  *
  * Copyright (c) Remix Software Inc.
  *
@@ -8,7 +8,7 @@
  *
  * @license MIT
  */
-import { g as preloadVite, h as getVite } from "./typegen-9e9g0hnu.js";
+import { g as preloadVite, h as getVite } from "./typegen-BdAwfkLl.js";
 //#region vite/build.ts
 async function build(root, viteBuildOptions) {
 	await preloadVite();

package/dist/cli/index.js

@@ -1,6 +1,6 @@
 #!/usr/bin/env node
 /**
- * @react-router/dev v8.0.1
+ * @react-router/dev v8.1.0
  *
  * Copyright (c) Remix Software Inc.
  *
@@ -9,9 +9,9 @@
  *
  * @license MIT
  */
-import { g as preloadVite, h as getVite, l as loadConfig, n as watch, t as run$1 } from "../typegen-9e9g0hnu.js";
+import { g as preloadVite, h as getVite, l as loadConfig, n as watch, t as run$1 } from "../typegen-BdAwfkLl.js";
 import { createRequire } from "node:module";
-import arg from "arg";
+import { parseArgs } from "node:util";
 import semver from "semver";
 import colors from "picocolors";
 import fs, { existsSync } from "node:fs";
@@ -141,7 +141,7 @@ async function routes(rootDirectory, flags = {}) {
 }
 async function build(root, options = {}) {
 	root = resolveRootDirectory(root, options);
-	let { build } = await import("../build-ygYOQjbA.js");
+	let { build } = await import("../build-9Do1SAoZ.js");
 	if (options.profile) await start();
 	try {
 		await build(root, options);
@@ -150,7 +150,7 @@ async function build(root, options = {}) {
 	}
 }
 async function dev(root, options = {}) {
-	let { dev } = await import("../dev-BAYiWwCm.js");
+	let { dev } = await import("../dev-Bnoz8B89.js");
 	if (options.profile) await start();
 	exitHook(() => stop(console.info));
 	root = resolveRootDirectory(root, options);
@@ -373,6 +373,18 @@ ${colors.blueBright("react-router")}
    $ react-router typegen
    $ react-router typegen --watch
 `;
+function getBooleanArg(value) {
+	return typeof value === "boolean" ? value : void 0;
+}
+function getBooleanStringArg(value) {
+	return typeof value === "boolean" || typeof value === "string" ? value : void 0;
+}
+function getNumberArg(value) {
+	return typeof value === "string" ? Number(value) : void 0;
+}
+function getStringArg(value) {
+	return typeof value === "string" ? value : void 0;
+}
 /**
 * Programmatic interface for running the react-router CLI with the given command line
 * arguments.
@@ -385,43 +397,78 @@ async function run(argv = process.argv.slice(2), { isMain = false } = {}) {
 		let nextArg = argv[argv.indexOf(arg) + 1];
 		return !nextArg || nextArg.startsWith("-");
 	};
-	let args = arg({
-		"--force": Boolean,
-		"--help": Boolean,
-		"-h": "--help",
-		"--json": Boolean,
-		"--token": String,
-		"--typescript": Boolean,
-		"--no-typescript": Boolean,
-		"--version": Boolean,
-		"-v": "--version",
-		"--port": Number,
-		"-p": "--port",
-		"--config": String,
-		"-c": "--config",
-		"--assetsInlineLimit": Number,
-		"--clearScreen": Boolean,
-		"--cors": Boolean,
-		"--emptyOutDir": Boolean,
-		"--host": isBooleanFlag("--host") ? Boolean : String,
-		"--logLevel": String,
-		"-l": "--logLevel",
-		"--minify": String,
-		"--mode": String,
-		"-m": "--mode",
-		"--open": isBooleanFlag("--open") ? Boolean : String,
-		"--strictPort": Boolean,
-		"--profile": Boolean,
-		"--sourcemapClient": isBooleanFlag("--sourcemapClient") ? Boolean : String,
-		"--sourcemapServer": isBooleanFlag("--sourcemapServer") ? Boolean : String,
-		"--watch": Boolean
-	}, { argv });
-	let input = args._;
-	let flags = Object.entries(args).reduce((acc, [key, value]) => {
-		key = key.replace(/^--/, "");
-		acc[key] = value;
-		return acc;
-	}, {});
+	let { values, positionals } = parseArgs({
+		args: argv,
+		allowPositionals: true,
+		options: {
+			assetsInlineLimit: { type: "string" },
+			clearScreen: { type: "boolean" },
+			config: {
+				type: "string",
+				short: "c"
+			},
+			cors: { type: "boolean" },
+			emptyOutDir: { type: "boolean" },
+			force: { type: "boolean" },
+			help: {
+				type: "boolean",
+				short: "h"
+			},
+			host: { type: isBooleanFlag("--host") ? "boolean" : "string" },
+			json: { type: "boolean" },
+			logLevel: {
+				type: "string",
+				short: "l"
+			},
+			minify: { type: "string" },
+			mode: {
+				type: "string",
+				short: "m"
+			},
+			"no-typescript": { type: "boolean" },
+			open: { type: isBooleanFlag("--open") ? "boolean" : "string" },
+			port: {
+				type: "string",
+				short: "p"
+			},
+			profile: { type: "boolean" },
+			sourcemapClient: { type: isBooleanFlag("--sourcemapClient") ? "boolean" : "string" },
+			sourcemapServer: { type: isBooleanFlag("--sourcemapServer") ? "boolean" : "string" },
+			strictPort: { type: "boolean" },
+			token: { type: "string" },
+			typescript: { type: "boolean" },
+			version: {
+				type: "boolean",
+				short: "v"
+			},
+			watch: { type: "boolean" }
+		}
+	});
+	let input = positionals;
+	let flags = {
+		assetsInlineLimit: getNumberArg(values.assetsInlineLimit),
+		clearScreen: getBooleanArg(values.clearScreen),
+		config: getStringArg(values.config),
+		cors: getBooleanArg(values.cors),
+		emptyOutDir: getBooleanArg(values.emptyOutDir),
+		force: getBooleanArg(values.force),
+		help: getBooleanArg(values.help),
+		host: getBooleanStringArg(values.host),
+		json: getBooleanArg(values.json),
+		logLevel: getStringArg(values.logLevel),
+		minify: getStringArg(values.minify),
+		mode: getStringArg(values.mode),
+		open: getBooleanStringArg(values.open),
+		port: getNumberArg(values.port),
+		profile: getBooleanArg(values.profile),
+		sourcemapClient: getBooleanStringArg(values.sourcemapClient),
+		sourcemapServer: getBooleanStringArg(values.sourcemapServer),
+		strictPort: getBooleanArg(values.strictPort),
+		token: getStringArg(values.token),
+		typescript: getBooleanArg(values.typescript),
+		version: getBooleanArg(values.version),
+		watch: getBooleanArg(values.watch)
+	};
 	if (flags.help) {
 		console.log(helpText);
 		return;
@@ -431,7 +478,7 @@ async function run(argv = process.argv.slice(2), { isMain = false } = {}) {
 		return;
 	}
 	flags.interactive = flags.interactive ?? isMain;
-	if (args["--no-typescript"]) flags.typescript = false;
+	if (values["no-typescript"]) flags.typescript = false;
 	switch (input[0]) {
 		case "routes":
 			await routes(input[1], flags);

package/dist/config.js

@@ -1,5 +1,5 @@
 /**
- * @react-router/dev v8.0.1
+ * @react-router/dev v8.1.0
  *
  * Copyright (c) Remix Software Inc.
  *

package/dist/{dev-BAYiWwCm.js → dev-Bnoz8B89.js}

@@ -1,5 +1,5 @@
 /**
- * @react-router/dev v8.0.1
+ * @react-router/dev v8.1.0
  *
  * Copyright (c) Remix Software Inc.
  *
@@ -8,7 +8,7 @@
  *
  * @license MIT
  */
-import { g as preloadVite, h as getVite } from "./typegen-9e9g0hnu.js";
+import { g as preloadVite, h as getVite } from "./typegen-BdAwfkLl.js";
 import { n as start, r as stop, t as getSession } from "./cli/index.js";
 import colors from "picocolors";
 //#region vite/dev.ts

package/dist/{routes-kAS9QOpS.js → routes-DGRRTCk8.js}

@@ -1,5 +1,5 @@
 /**
- * @react-router/dev v8.0.1
+ * @react-router/dev v8.1.0
  *
  * Copyright (c) Remix Software Inc.
  *

package/dist/routes.js

@@ -1,5 +1,5 @@
 /**
- * @react-router/dev v8.0.1
+ * @react-router/dev v8.1.0
  *
  * Copyright (c) Remix Software Inc.
  *
@@ -8,5 +8,5 @@
  *
  * @license MIT
  */
-import { a as prefix, i as layout, n as getAppDirectory, o as relative, r as index, s as route } from "./routes-kAS9QOpS.js";
+import { a as prefix, i as layout, n as getAppDirectory, o as relative, r as index, s as route } from "./routes-DGRRTCk8.js";
 export { getAppDirectory, index, layout, prefix, relative, route };

package/dist/{typegen-9e9g0hnu.js → typegen-BdAwfkLl.js}

@@ -1,5 +1,5 @@
 /**
- * @react-router/dev v8.0.1
+ * @react-router/dev v8.1.0
  *
  * Copyright (c) Remix Software Inc.
  *
@@ -8,7 +8,7 @@
  *
  * @license MIT
  */
-import { c as setAppDirectory, l as validateRouteConfig, t as configRoutesToRouteManifest, u as invariant } from "./routes-kAS9QOpS.js";
+import { c as setAppDirectory, l as validateRouteConfig, t as configRoutesToRouteManifest, u as invariant } from "./routes-DGRRTCk8.js";
 import { createRequire } from "node:module";
 import colors from "picocolors";
 import fs from "node:fs";
@@ -95,7 +95,7 @@ async function createContext$1({ root, mode, customLogger }) {
 		optimizeDeps: { noDiscovery: true },
 		css: { postcss: {} },
 		configFile: false,
-		envFile: false,
+		envDir: false,
 		plugins: [],
 		environments: { __config_loader: {
 			consumer: "server",
@@ -543,10 +543,14 @@ var babel_exports = /* @__PURE__ */ __exportAll({
 	generate: () => generate,
 	parse: () => parse$1,
 	t: () => t$1,
-	traverse: () => traverse
+	traverse: () => traverse,
+	unwrapDefault: () => unwrapDefault
 });
-const traverse = _traverse.default;
-const generate = _generate.default;
+function unwrapDefault(value) {
+	return value.default ?? value;
+}
+const traverse = unwrapDefault(_traverse);
+const generate = unwrapDefault(_generate);
 //#endregion
 //#region typegen/params.ts
 function parse(fullpath) {

package/dist/vite.js

@@ -1,5 +1,5 @@
 /**
- * @react-router/dev v8.0.1
+ * @react-router/dev v8.1.0
  *
  * Copyright (c) Remix Software Inc.
  *
@@ -8,14 +8,15 @@
  *
  * @license MIT
  */
-import { a as t, c as createConfigLoader, d as resolveRSCEntryFiles, f as ssrExternals, g as preloadVite, h as getVite, i as parse$1, m as defineOptimizeDepsCompilerOptions, n as watch, o as traverse, p as defineCompilerOptions, r as generate, s as configRouteToBranchRoute, u as resolveEntryFiles } from "./typegen-9e9g0hnu.js";
-import { u as invariant } from "./routes-kAS9QOpS.js";
+import { a as t, c as createConfigLoader, d as resolveRSCEntryFiles, f as ssrExternals, g as preloadVite, h as getVite, i as parse$1, m as defineOptimizeDepsCompilerOptions, n as watch, o as traverse, p as defineCompilerOptions, r as generate, s as configRouteToBranchRoute, u as resolveEntryFiles } from "./typegen-BdAwfkLl.js";
+import { u as invariant } from "./routes-DGRRTCk8.js";
 import { createRequire } from "node:module";
 import colors from "picocolors";
 import { existsSync, readFileSync, readdirSync } from "node:fs";
 import { cp, mkdir, readFile, readdir, rename, rm, writeFile } from "node:fs/promises";
 import * as path$2 from "node:path";
 import path from "node:path";
+import { readPackageJSON } from "pkg-types";
 import { createRequestHandler, matchRoutes, unstable_setDevServerHooks } from "react-router";
 import * as Path from "pathe";
 import path$1, { join } from "pathe";
@@ -1532,23 +1533,11 @@ const reactRouterVitePlugin = () => {
 						async buildApp(builder) {
 							invariant(viteConfig);
 							viteConfig.logger.info("Using Vite Environment API");
-							try {
-								let { reactRouterConfig } = ctx;
-								await cleanBuildDirectory(viteConfig, ctx);
-								await builder.build(builder.environments.client);
-								let serverEnvironments = getServerEnvironmentValues(ctx, builder.environments);
-								await Promise.all(serverEnvironments.map(builder.build));
-								await cleanViteManifests(environments, ctx);
-								let { buildManifest } = ctx;
-								invariant(buildManifest, "Expected build manifest");
-								await reactRouterConfig.buildEnd?.({
-									buildManifest,
-									reactRouterConfig,
-									viteConfig
-								});
-							} finally {
-								await closePluginResources();
-							}
+							await cleanBuildDirectory(viteConfig, ctx);
+							await builder.build(builder.environments.client);
+							let serverEnvironments = getServerEnvironmentValues(ctx, builder.environments);
+							await Promise.all(serverEnvironments.map(builder.build));
+							await cleanViteManifests(environments, ctx);
 						}
 					}
 				};
@@ -1592,7 +1581,7 @@ const reactRouterVitePlugin = () => {
 						hmr: false
 					},
 					configFile: false,
-					envFile: false,
+					envDir: false,
 					plugins: [childCompilerPlugins.filter((plugin) => typeof plugin === "object" && plugin !== null && "name" in plugin && plugin.name !== "react-router" && plugin.name !== "react-router:route-exports" && plugin.name !== "react-router:hmr-updates" && plugin.name !== "react-router:validate-plugin-order").map((plugin) => ({
 						...plugin,
 						configureServer: void 0,
@@ -2202,6 +2191,30 @@ const reactRouterVitePlugin = () => {
 				}
 			}
 		}),
+		{
+			name: "react-router-build-end",
+			sharedDuringBuild: true,
+			config: {
+				order: "post",
+				handler({ builder: { buildApp } = {} }) {
+					return { builder: { async buildApp(builder) {
+						try {
+							await buildApp?.(builder);
+							invariant(viteConfig);
+							let { buildManifest, reactRouterConfig } = ctx;
+							invariant(buildManifest, "Expected build manifest");
+							await reactRouterConfig.buildEnd?.({
+								buildManifest,
+								reactRouterConfig,
+								viteConfig
+							});
+						} finally {
+							await closePluginResources();
+						}
+					} } };
+				}
+			}
+		},
 		validatePluginOrder(),
 		warnOnClientSourceMaps()
 	];
@@ -2545,6 +2558,7 @@ function mergeEnvironmentOptions(base, ...overrides) {
 }
 async function getEnvironmentOptionsResolvers(ctx, viteCommand) {
 	let { serverBuildFile, serverModuleFormat } = ctx.reactRouterConfig;
+	let pkgJson = await readPackageJSON(ctx.rootDirectory).catch(() => ({}));
 	let packageRoot = path$2.dirname(nodeRequire$1.resolve("@react-router/dev/package.json"));
 	let { moduleSyncEnabled } = await import(`file:///${path$2.join(packageRoot, "module-sync-enabled/index.mjs")}`);
 	let vite = getVite();
@@ -2567,7 +2581,11 @@ async function getEnvironmentOptionsResolvers(ctx, viteCommand) {
 		let maybeModuleSyncConditions = [...moduleSyncEnabled ? ["module-sync"] : []];
 		let maybeDevelopmentConditions = viteCommand === "build" ? [] : ["development"];
 		let maybeDefaultServerConditions = vite.defaultServerConditions || [];
-		let defaultExternalConditions = ["node"];
+		let defaultExternalConditions = vite.defaultExternalConditions ?? ["node"];
+		if (!(!pkgJson.dependencies || pkgJson.dependencies["@react-router/node"] || pkgJson.dependencies["@react-router/express"] || pkgJson.dependencies["@react-router/serve"])) {
+			maybeDefaultServerConditions = maybeDefaultServerConditions.filter((c) => c !== "node");
+			defaultExternalConditions = defaultExternalConditions.filter((c) => c !== "node");
+		}
 		let baseConditions = [...maybeDevelopmentConditions, ...maybeModuleSyncConditions];
 		return mergeEnvironmentOptions(getBaseOptions({ viteUserConfig }), {
 			resolve: {

package/package.json

@@ -1,8 +1,15 @@
 {
   "name": "@react-router/dev",
   "type": "module",
-  "version": "8.0.1",
+  "version": "8.1.0",
   "description": "Dev tools and CLI for React Router",
+  "keywords": [
+    "react",
+    "router",
+    "react-router",
+    "vite",
+    "vite-plugin"
+  ],
   "homepage": "https://reactrouter.com",
   "bugs": {
     "url": "https://github.com/remix-run/react-router/issues"
@@ -41,6 +48,17 @@
   "bin": {
     "react-router": "bin.cjs"
   },
+  "compatiblePackages": {
+    "schemaVersion": 1,
+    "rolldown": {
+      "type": "incompatible",
+      "reason": "Uses Vite-specific APIs"
+    },
+    "rollup": {
+      "type": "incompatible",
+      "reason": "Uses Vite-specific APIs"
+    }
+  },
   "wireit": {
     "build": {
       "command": "tsdown",
@@ -70,7 +88,6 @@
     "@babel/traverse": "^7.29.7",
     "@babel/types": "^7.29.7",
     "@remix-run/node-fetch-server": "^0.13.3",
-    "arg": "^5.0.1",
     "babel-dead-code-elimination": "^1.0.12",
     "chokidar": "^5.0.0",
     "dedent": "^1.7.2",
@@ -88,7 +105,7 @@
     "semver": "^7.8.1",
     "tinyglobby": "^0.2.16",
     "valibot": "^1.4.1",
-    "@react-router/node": "8.0.1"
+    "@react-router/node": "8.1.0"
   },
   "devDependencies": {
     "@types/babel__core": "^7.20.5",
@@ -110,8 +127,8 @@
     "vite": "^7.0.0",
     "wireit": "0.14.12",
     "wrangler": "^4.95.0",
-    "react-router": "^8.0.1",
-    "@react-router/serve": "8.0.1"
+    "@react-router/serve": "8.1.0",
+    "react-router": "^8.1.0"
   },
   "peerDependencies": {
     "@vitejs/plugin-rsc": "~0.5.26",
@@ -119,8 +136,8 @@
     "typescript": "^5.1.0 || ^6.0.0",
     "vite": "^7.0.0 || ^8.0.0",
     "wrangler": "^4.0.0",
-    "@react-router/serve": "^8.0.1",
-    "react-router": "^8.0.1"
+    "react-router": "^8.1.0",
+    "@react-router/serve": "^8.1.0"
   },
   "peerDependenciesMeta": {
     "@vitejs/plugin-rsc": {