@react-router/dev 8.3.1 → 8.4.0

package/CHANGELOG.md

@@ -1,5 +1,21 @@
 # `@react-router/dev`
 
+## v8.4.0
+
+### Patch Changes
+
+- Fix action submissions behind an HTTPS reverse proxy in `react-router dev` and `vite preview` by using `X-Forwarded-Proto` when constructing `request.url` ([#15466](https://github.com/remix-run/react-router/pull/15466))
+  - No React Router configuration is required
+  - The URL host continues to come from `Host`, not `X-Forwarded-Host`
+  - Configure reverse proxies to overwrite client-supplied `X-Forwarded-Proto` headers
+  - Do not expose dev or preview servers publicly
+- Pass development conditions as CLI flags when relaunching the dev process ([#15435](https://github.com/remix-run/react-router/pull/15435))
+- Fix stack trace locations for split route modules when source maps are enabled in the `reactRouter` Vite plugin ([#15440](https://github.com/remix-run/react-router/pull/15440))
+- Updated dependencies:
+  - [`react-router@8.4.0`](https://github.com/remix-run/react-router/releases/tag/react-router@8.4.0)
+  - [`@react-router/node@8.4.0`](https://github.com/remix-run/react-router/releases/tag/@react-router/node@8.4.0)
+  - [`@react-router/serve@8.4.0`](https://github.com/remix-run/react-router/releases/tag/@react-router/serve@8.4.0)
+
 ## v8.3.1
 
 ### Patch Changes

package/dist/{build-Ou3B3epu.js → build-CsJF5XUO.js}

@@ -1,5 +1,5 @@
 /**
- * @react-router/dev v8.3.1
+ * @react-router/dev v8.4.0
  *
  * Copyright (c) Remix Software Inc.
  *
@@ -8,7 +8,7 @@
  *
  * @license MIT
  */
-import { _ as getVite, v as preloadVite } from "./typegen-DxFMuBQP.js";
+import { _ as getVite, v as preloadVite } from "./typegen-WywajSVn.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.3.1
+ * @react-router/dev v8.4.0
  *
  * Copyright (c) Remix Software Inc.
  *
@@ -9,7 +9,7 @@
  *
  * @license MIT
  */
-import { _ as getVite, l as hasNodeDependency, n as watch, t as run$1, u as loadConfig, v as preloadVite } from "../typegen-DxFMuBQP.js";
+import { _ as getVite, l as hasNodeDependency, n as watch, t as run$1, u as loadConfig, v as preloadVite } from "../typegen-WywajSVn.js";
 import { createRequire } from "node:module";
 import { parseArgs } from "node:util";
 import semver from "semver";
@@ -107,20 +107,34 @@ async function hasReactRouterRscPlugin({ root, viteBuildOptions: { config, logLe
 //#endregion
 //#region restart-with-conditions.ts
 /**
-* Restarts the current Node process, appending new flags to the
-* existing NODE_OPTIONS. SIGINT/SIGTERM are always forwarded to the child.
+* Build the child process invocation used to relaunch the CLI with extra flags.
+*
+* Extra flags are placed on argv immediately after the runtime binary. Putting
+* them in NODE_OPTIONS is not portable: Bun silently ignores `--conditions`
+* there, so the child would restart again and hit the already-restarted guard.
 */
-function restartWithMergedOptions(nodeOptions) {
-	if (process$1.env.REACT_ROUTER_DEV_RESTARTED === "true") throw new Error("restartWithMergedOptions() was called, but the process has already been restarted. This is likely a bug in @react-router/dev.");
-	const mergedOptions = [process$1.env.NODE_OPTIONS, nodeOptions].filter(Boolean).join(" ").trim();
-	console.log(`[restart] Relaunching with NODE_OPTIONS: ${mergedOptions}`);
-	const [cmd, ...args] = process$1.argv;
-	const child = spawn(cmd, args, {
+function resolveRestartSpawn(argv, extraOptions, env) {
+	const extraArgs = extraOptions.split(/\s+/).filter(Boolean);
+	const [command, ...args] = argv;
+	return {
+		command,
+		args: [...extraArgs, ...args],
 		env: {
-			...process$1.env,
-			NODE_OPTIONS: mergedOptions,
+			...env,
 			REACT_ROUTER_DEV_RESTARTED: "true"
-		},
+		}
+	};
+}
+/**
+* Restarts the current process with extra CLI flags after argv[0].
+* SIGINT/SIGTERM are always forwarded to the child.
+*/
+function restartWithMergedOptions(nodeOptions) {
+	if (process$1.env.REACT_ROUTER_DEV_RESTARTED === "true") throw new Error("restartWithMergedOptions() was called, but the process has already been restarted. This is likely a bug in @react-router/dev.");
+	const { command, args, env } = resolveRestartSpawn(process$1.argv, nodeOptions, process$1.env);
+	console.log(`[restart] Relaunching with ${nodeOptions}`);
+	const child = spawn(command, args, {
+		env,
 		stdio: "inherit"
 	});
 	let signalHandlers = ["SIGINT", "SIGTERM"].map((sig) => {
@@ -158,7 +172,7 @@ async function routes(rootDirectory, flags = {}) {
 }
 async function build(root, options = {}) {
 	root = resolveRootDirectory(root, options);
-	let { build } = await import("../build-Ou3B3epu.js");
+	let { build } = await import("../build-CsJF5XUO.js");
 	if (options.profile) await start();
 	try {
 		await build(root, options);
@@ -168,7 +182,7 @@ async function build(root, options = {}) {
 }
 async function dev(root, options = {}) {
 	if (developmentConditionEnabled) {
-		let { dev } = await import("../dev-DFccbHpq.js");
+		let { dev } = await import("../dev-_GaivbeG.js");
 		if (options.profile) await start();
 		exitHook(() => stop(console.info));
 		root = resolveRootDirectory(root, options);

package/dist/config.js

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

package/dist/{dev-DFccbHpq.js → dev-_GaivbeG.js}

@@ -1,5 +1,5 @@
 /**
- * @react-router/dev v8.3.1
+ * @react-router/dev v8.4.0
  *
  * Copyright (c) Remix Software Inc.
  *
@@ -8,7 +8,7 @@
  *
  * @license MIT
  */
-import { _ as getVite, v as preloadVite } from "./typegen-DxFMuBQP.js";
+import { _ as getVite, v as preloadVite } from "./typegen-WywajSVn.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-C-NBxWG3.js → routes-BFdkiVnp.js}

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

package/dist/routes.js

@@ -1,5 +1,5 @@
 /**
- * @react-router/dev v8.3.1
+ * @react-router/dev v8.4.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-C-NBxWG3.js";
+import { a as prefix, i as layout, n as getAppDirectory, o as relative, r as index, s as route } from "./routes-BFdkiVnp.js";
 export { getAppDirectory, index, layout, prefix, relative, route };

package/dist/{typegen-DxFMuBQP.js → typegen-WywajSVn.js}

@@ -1,5 +1,5 @@
 /**
- * @react-router/dev v8.3.1
+ * @react-router/dev v8.4.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-C-NBxWG3.js";
+import { c as setAppDirectory, l as validateRouteConfig, t as configRoutesToRouteManifest, u as invariant } from "./routes-BFdkiVnp.js";
 import { createRequire } from "node:module";
 import colors from "picocolors";
 import fs from "node:fs";

package/dist/vite.js

@@ -1,5 +1,5 @@
 /**
- * @react-router/dev v8.3.1
+ * @react-router/dev v8.4.0
  *
  * Copyright (c) Remix Software Inc.
  *
@@ -8,8 +8,8 @@
  *
  * @license MIT
  */
-import { _ as getVite, a as t, c as createConfigLoader, d as resolveEntryFiles, f as resolveRSCEntryFiles, g as getUserBuildRollupOptions, h as defineOptimizeDepsCompilerOptions, i as parse$1, l as hasNodeDependency, m as defineCompilerOptions, n as watch, o as traverse, p as ssrExternals, r as generate, s as configRouteToBranchRoute, v as preloadVite } from "./typegen-DxFMuBQP.js";
-import { u as invariant } from "./routes-C-NBxWG3.js";
+import { _ as getVite, a as t, c as createConfigLoader, d as resolveEntryFiles, f as resolveRSCEntryFiles, g as getUserBuildRollupOptions, h as defineOptimizeDepsCompilerOptions, i as parse$1, l as hasNodeDependency, m as defineCompilerOptions, n as watch, o as traverse, p as ssrExternals, r as generate, s as configRouteToBranchRoute, v as preloadVite } from "./typegen-WywajSVn.js";
+import { u as invariant } from "./routes-BFdkiVnp.js";
 import { createRequire } from "node:module";
 import colors from "picocolors";
 import { existsSync, readFileSync, readdirSync } from "node:fs";
@@ -37,7 +37,13 @@ import { readFile as readFile$1 } from "fs/promises";
 async function fromNodeRequest(nodeReq, nodeRes) {
 	invariant(nodeReq.originalUrl, "Expected `nodeReq.originalUrl` to be defined");
 	nodeReq.url = nodeReq.originalUrl;
-	return createRequest(nodeReq, nodeRes);
+	return createRequest(nodeReq, nodeRes, { protocol: getForwardedProtocol(nodeReq) });
+}
+function getForwardedProtocol(nodeReq) {
+	let forwardedProto = nodeReq.headers["x-forwarded-proto"];
+	let proto = (Array.isArray(forwardedProto) ? forwardedProto[0] : forwardedProto)?.split(",")[0].trim().toLowerCase();
+	if (proto?.endsWith(":")) proto = proto.slice(0, -1);
+	return proto === "http" || proto === "https" ? `${proto}:` : void 0;
 }
 //#endregion
 //#region vite/resolve-file-url.ts
@@ -614,9 +620,9 @@ const routeChunkExportNames = [
 ];
 const mainChunkName = "main";
 const routeChunkNames = ["main", ...routeChunkExportNames];
-function getRouteChunkCode(code, chunkName, cache, cacheKey) {
-	if (chunkName === mainChunkName) return omitChunkedExports(code, routeChunkExportNames, {}, cache, cacheKey);
-	return getChunkedExport(code, chunkName, {}, cache, cacheKey);
+function getRouteChunkCode(code, chunkName, cache, cacheKey, generateOptions = {}) {
+	if (chunkName === mainChunkName) return omitChunkedExports(code, routeChunkExportNames, generateOptions, cache, cacheKey);
+	return getChunkedExport(code, chunkName, generateOptions, cache, cacheKey);
 }
 const routeChunkQueryStringPrefix = "?route-chunk=";
 const routeChunkQueryStrings = {
@@ -1817,11 +1823,14 @@ const reactRouterVitePlugin = () => {
 				let routeFileName = path$2.basename(routeModuleId);
 				let sourceExports = await getRouteModuleExports(viteChildCompiler, ctx, routeModuleId);
 				let { chunkedExports = [] } = options?.ssr ? {} : await detectRouteChunksIfEnabled(cache, ctx, id, code);
-				return `export { ${sourceExports.filter((exportName) => {
-					let isRouteEntryExport = options?.ssr && SERVER_ONLY_ROUTE_EXPORTS.includes(exportName) || CLIENT_ROUTE_EXPORTS$1.includes(exportName);
-					let isChunkedExport = chunkedExports.includes(exportName);
-					return isRouteEntryExport && !isChunkedExport;
-				}).join(", ")} } from "./${routeFileName}";`;
+				return {
+					code: `export { ${sourceExports.filter((exportName) => {
+						let isRouteEntryExport = options?.ssr && SERVER_ONLY_ROUTE_EXPORTS.includes(exportName) || CLIENT_ROUTE_EXPORTS$1.includes(exportName);
+						let isChunkedExport = chunkedExports.includes(exportName);
+						return isRouteEntryExport && !isChunkedExport;
+					}).join(", ")} } from "./${routeFileName}";`,
+					map: { mappings: "" }
+				};
 			}
 		},
 		{
@@ -2476,7 +2485,10 @@ async function detectRouteChunksIfEnabled(cache, ctx, id, input) {
 }
 async function getRouteChunkIfEnabled(cache, ctx, id, chunkName, input) {
 	if (!ctx.reactRouterConfig.splitRouteModules) return null;
-	return getRouteChunkCode(await resolveRouteFileCode(ctx, input), chunkName, cache, normalizeRelativeFilePath(id, ctx.reactRouterConfig) + (typeof input === "string" ? "" : "?read"));
+	return getRouteChunkCode(await resolveRouteFileCode(ctx, input), chunkName, cache, normalizeRelativeFilePath(id, ctx.reactRouterConfig) + (typeof input === "string" ? "" : "?read"), {
+		sourceMaps: true,
+		sourceFileName: path$2.basename(id.split("?")[0])
+	});
 }
 function validateRouteChunks$1({ ctx, id, valid }) {
 	if (isRootRouteModuleId(ctx, id)) return;

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@react-router/dev",
   "type": "module",
-  "version": "8.3.1",
+  "version": "8.4.0",
   "description": "Dev tools and CLI for React Router",
   "keywords": [
     "react",
@@ -107,7 +107,7 @@
     "semver": "^7.8.1",
     "tinyglobby": "^0.2.16",
     "valibot": "^1.4.1",
-    "@react-router/node": "8.3.1"
+    "@react-router/node": "8.4.0"
   },
   "devDependencies": {
     "@babel/plugin-syntax-jsx": "^7.29.7",
@@ -132,8 +132,8 @@
     "vite": "^7.0.0",
     "wireit": "0.14.12",
     "wrangler": "^4.95.0",
-    "@react-router/serve": "8.3.1",
-    "react-router": "^8.3.1"
+    "react-router": "^8.4.0",
+    "@react-router/serve": "8.4.0"
   },
   "peerDependencies": {
     "@vitejs/plugin-rsc": "~0.5.26",
@@ -141,8 +141,8 @@
     "typescript": "^5.1.0 || ^6.0.0 || ^7.0.0",
     "vite": "^7.0.0 || ^8.0.0",
     "wrangler": "^4.0.0",
-    "@react-router/serve": "^8.3.1",
-    "react-router": "^8.3.1"
+    "@react-router/serve": "^8.4.0",
+    "react-router": "^8.4.0"
   },
   "peerDependenciesMeta": {
     "@vitejs/plugin-rsc": {