@react-router/dev 8.3.0 → 8.4.0

package/CHANGELOG.md

@@ -1,5 +1,33 @@
 # `@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
+
+- Bump `@remix-run/node-fetch-server` dependency ([#15447](https://github.com/remix-run/react-router/pull/15447))
+- Generate JavaScript entry files at package build time so `react-router reveal --no-typescript` does not require Prettier at runtime ([#15373](https://github.com/remix-run/react-router/pull/15373))
+  - Deprecate the `--no-typescript` flag ahead of its removal in React Router v9
+- Updated dependencies:
+  - [`react-router@8.3.1`](https://github.com/remix-run/react-router/releases/tag/react-router@8.3.1)
+  - [`@react-router/node@8.3.1`](https://github.com/remix-run/react-router/releases/tag/@react-router/node@8.3.1)
+  - [`@react-router/serve@8.3.1`](https://github.com/remix-run/react-router/releases/tag/@react-router/serve@8.3.1)
+
 ## v8.3.0
 
 ### Minor Changes
@@ -12,7 +40,7 @@
 
 ### Unstable Changes
 
-⚠️  _[Unstable features](https://reactrouter.com/community/api-development-strategy#unstable-flags) are not recommended for production use_
+⚠️ _[Unstable features](https://reactrouter.com/community/api-development-strategy#unstable-flags) are not recommended for production use_
 
 - Add `unstable_rsc/client-version` client build version virtual module ([#15318](https://github.com/remix-run/react-router/pull/15318))
 
@@ -35,6 +63,7 @@
     signal: request.signal,
   });
   ```
+
 - Updated dependencies:
   - [`react-router@8.3.0`](https://github.com/remix-run/react-router/releases/tag/react-router@8.3.0)
   - [`@react-router/node@8.3.0`](https://github.com/remix-run/react-router/releases/tag/@react-router/node@8.3.0)

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

@@ -1,5 +1,5 @@
 /**
- * @react-router/dev v8.3.0
+ * @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-C6qQ4fuP.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.0
+ * @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-C6qQ4fuP.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";
@@ -23,10 +23,6 @@ import { readPackageJSON } from "pkg-types";
 import "react-router";
 import developmentConditionEnabled from "#development-condition-enabled";
 import { spawn } from "node:child_process";
-import * as babel from "@babel/core";
-import babelPluginSyntaxJSX from "@babel/plugin-syntax-jsx";
-import babelPresetTypeScript from "@babel/preset-typescript";
-import prettier from "prettier";
 import process$1 from "node:process";
 import packageJson from "@react-router/dev/package.json" with { type: "json" };
 //#region config/format.ts
@@ -72,24 +68,6 @@ function formatRoutesAsJsx(routeManifest) {
 	return output;
 }
 //#endregion
-//#region cli/useJavascript.ts
-async function transpile(tsx, options = {}) {
-	let mjs = babel.transformSync(tsx, {
-		compact: false,
-		cwd: options.cwd,
-		filename: options.filename,
-		plugins: [babelPluginSyntaxJSX],
-		presets: [[babelPresetTypeScript, { jsx: "preserve" }]],
-		retainLines: true
-	});
-	if (!mjs || !mjs.code) throw new Error("Could not parse TypeScript");
-	/**
-	* Babel's `compact` and `retainLines` options are both bad at formatting code.
-	* Use Prettier for nicer formatting.
-	*/
-	return await prettier.format(mjs.code, { parser: "babel" });
-}
-//#endregion
 //#region vite/profiler.ts
 const getSession = () => global.__reactRouter_profile_session;
 const start = async (callback) => {
@@ -129,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) => {
@@ -180,7 +172,7 @@ async function routes(rootDirectory, flags = {}) {
 }
 async function build(root, options = {}) {
 	root = resolveRootDirectory(root, options);
-	let { build } = await import("../build-B8O1OXuF.js");
+	let { build } = await import("../build-CsJF5XUO.js");
 	if (options.profile) await start();
 	try {
 		await build(root, options);
@@ -190,7 +182,7 @@ async function build(root, options = {}) {
 }
 async function dev(root, options = {}) {
 	if (developmentConditionEnabled) {
-		let { dev } = await import("../dev-CznbsRZe.js");
+		let { dev } = await import("../dev-_GaivbeG.js");
 		if (options.profile) await start();
 		exitHook(() => stop(console.info));
 		root = resolveRootDirectory(root, options);
@@ -273,20 +265,13 @@ async function generateEntry(entry, rootDirectory, flags = {}) {
 		await copyFile(defaultEntry, outputFile);
 	} else {
 		let pkgJson = await readPackageJSON(rootDirectory);
-		let defaultEntryClient = path$1.resolve(defaultsDirectory, "entry.client.tsx");
-		let defaultEntryServer = path$1.resolve(defaultsDirectory, hasNodeDependency(pkgJson.dependencies) && !configResult.value.future.unstable_enableNodeReadableStream ? `entry.server.node.tsx` : `entry.server.web.tsx`);
-		let isServerEntry = entry === "entry.server";
-		let contents = isServerEntry ? await createServerEntry(rootDirectory, appDirectory, defaultEntryServer) : await createClientEntry(rootDirectory, appDirectory, defaultEntryClient);
-		let useTypeScript = flags.typescript ?? true;
-		let outputEntry = `${entry}.${useTypeScript ? "tsx" : "jsx"}`;
+		let outputExtension = flags.typescript ?? true ? "tsx" : "jsx";
+		let defaultEntryClient = path$1.resolve(defaultsDirectory, `entry.client.${outputExtension}`);
+		let defaultEntryServer = path$1.resolve(defaultsDirectory, hasNodeDependency(pkgJson.dependencies) && !configResult.value.future.unstable_enableNodeReadableStream ? `entry.server.node.${outputExtension}` : `entry.server.web.${outputExtension}`);
+		let contents = entry === "entry.server" ? await createServerEntry(rootDirectory, appDirectory, defaultEntryServer) : await createClientEntry(rootDirectory, appDirectory, defaultEntryClient);
+		let outputEntry = `${entry}.${outputExtension}`;
 		outputFile = path$1.resolve(appDirectory, outputEntry);
-		if (!useTypeScript) {
-			let javascript = await transpile(contents, {
-				cwd: rootDirectory,
-				filename: isServerEntry ? defaultEntryServer : defaultEntryClient
-			});
-			await writeFile(outputFile, javascript, "utf-8");
-		} else await writeFile(outputFile, contents, "utf-8");
+		await writeFile(outputFile, contents, "utf-8");
 	}
 	console.log(colors.blue(`Entry file ${entry} created at ${path$1.relative(rootDirectory, outputFile)}.`));
 }
@@ -379,7 +364,7 @@ ${colors.blueBright("react-router")}
     --json              Print the routes as JSON
   \`reveal\` Options:
     --config, -c        Use specified Vite config file (string)
-    --no-typescript     Generate plain JavaScript files
+    --no-typescript     Generate plain JavaScript files (deprecated; will be removed in v9)
   \`typegen\` Options:
     --watch             Automatically regenerate types whenever route config (\`routes.ts\`) or route modules change
 
@@ -516,7 +501,10 @@ async function run(argv = process.argv.slice(2), { isMain = false } = {}) {
 		return;
 	}
 	flags.interactive = flags.interactive ?? isMain;
-	if (values["no-typescript"]) flags.typescript = false;
+	if (values["no-typescript"]) {
+		console.warn(colors.yellow("The --no-typescript flag is deprecated and will be removed in React Router v9."));
+		flags.typescript = false;
+	}
 	switch (input[0]) {
 		case "routes":
 			await routes(input[1], flags);

package/dist/config/defaults/entry.client.jsx

@@ -0,0 +1,12 @@
+import { startTransition, StrictMode } from "react";
+import { hydrateRoot } from "react-dom/client";
+import { HydratedRouter } from "react-router/dom";
+
+startTransition(() => {
+  hydrateRoot(
+    document,
+    <StrictMode>
+      <HydratedRouter />
+    </StrictMode>,
+  );
+});

package/dist/config/defaults/entry.server.node.jsx

@@ -0,0 +1,82 @@
+import { PassThrough } from "node:stream";
+
+import { createReadableStreamFromReadable } from "@react-router/node";
+import { ServerRouter } from "react-router";
+import { isbot } from "isbot";
+
+import { renderToPipeableStream } from "react-dom/server";
+
+export const streamTimeout = 5_000;
+
+export default function handleRequest(
+  request,
+  responseStatusCode,
+  responseHeaders,
+  routerContext,
+  loadContext,
+) {
+  // https://httpwg.org/specs/rfc9110.html#HEAD
+  if (request.method.toUpperCase() === "HEAD") {
+    return new Response(null, {
+      status: responseStatusCode,
+      headers: responseHeaders,
+    });
+  }
+
+  return new Promise((resolve, reject) => {
+    let shellRendered = false;
+    let userAgent = request.headers.get("user-agent");
+
+    // Ensure requests from bots and SPA Mode renders wait for all content to load before responding
+    // https://react.dev/reference/react-dom/server/renderToPipeableStream#waiting-for-all-content-to-load-for-crawlers-and-static-generation
+    let readyOption =
+      (userAgent && isbot(userAgent)) || routerContext.isSpaMode
+        ? "onAllReady"
+        : "onShellReady";
+
+    // Abort the rendering stream after the `streamTimeout` so it has time to
+    // flush down the rejected boundaries
+    let timeoutId = setTimeout(() => abort(), streamTimeout + 1000);
+
+    const { pipe, abort } = renderToPipeableStream(
+      <ServerRouter context={routerContext} url={request.url} />,
+      {
+        [readyOption]() {
+          shellRendered = true;
+          const body = new PassThrough({
+            final(callback) {
+              // Clear the timeout to prevent retaining the closure and memory leak
+              clearTimeout(timeoutId);
+              timeoutId = undefined;
+              callback();
+            },
+          });
+          const stream = createReadableStreamFromReadable(body);
+
+          responseHeaders.set("Content-Type", "text/html");
+
+          pipe(body);
+
+          resolve(
+            new Response(stream, {
+              headers: responseHeaders,
+              status: responseStatusCode,
+            }),
+          );
+        },
+        onShellError(error) {
+          reject(error);
+        },
+        onError(error) {
+          responseStatusCode = 500;
+          // Log streaming rendering errors from inside the shell.  Don't log
+          // errors encountered during initial shell rendering since they'll
+          // reject and get logged in handleDocumentRequest.
+          if (shellRendered) {
+            console.error(error);
+          }
+        },
+      },
+    );
+  });
+}

package/dist/config/defaults/entry.server.web.jsx

@@ -0,0 +1,53 @@
+import { ServerRouter } from "react-router";
+import { isbot } from "isbot";
+import { renderToReadableStream } from "react-dom/server";
+
+export const streamTimeout = 5_000;
+
+export default async function handleRequest(
+  request,
+  responseStatusCode,
+  responseHeaders,
+  routerContext,
+  _loadContext,
+) {
+  // https://httpwg.org/specs/rfc9110.html#HEAD
+  if (request.method.toUpperCase() === "HEAD") {
+    return new Response(null, {
+      status: responseStatusCode,
+      headers: responseHeaders,
+    });
+  }
+
+  let shellRendered = false;
+  let userAgent = request.headers.get("user-agent");
+
+  const body = await renderToReadableStream(
+    <ServerRouter context={routerContext} url={request.url} />,
+    {
+      signal: AbortSignal.timeout(streamTimeout + 1000),
+      onError(error) {
+        responseStatusCode = 500;
+        // Log streaming rendering errors from inside the shell. Don't log
+        // errors encountered during initial shell rendering since they'll
+        // reject and get logged in handleDocumentRequest.
+        if (shellRendered) {
+          console.error(error);
+        }
+      },
+    },
+  );
+  shellRendered = true;
+
+  // Ensure requests from bots and SPA Mode renders wait for all content to load before responding
+  // https://react.dev/reference/react-dom/server/renderToPipeableStream#waiting-for-all-content-to-load-for-crawlers-and-static-generation
+  if ((userAgent && isbot(userAgent)) || routerContext.isSpaMode) {
+    await body.allReady;
+  }
+
+  responseHeaders.set("Content-Type", "text/html");
+  return new Response(body, {
+    headers: responseHeaders,
+    status: responseStatusCode,
+  });
+}

package/dist/config.js

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

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

@@ -1,5 +1,5 @@
 /**
- * @react-router/dev v8.3.0
+ * @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-C6qQ4fuP.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-BwOSW5AT.js → routes-BFdkiVnp.js}

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

package/dist/routes.js

@@ -1,5 +1,5 @@
 /**
- * @react-router/dev v8.3.0
+ * @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-BwOSW5AT.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-C6qQ4fuP.js → typegen-WywajSVn.js}

@@ -1,5 +1,5 @@
 /**
- * @react-router/dev v8.3.0
+ * @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-BwOSW5AT.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.0
+ * @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-C6qQ4fuP.js";
-import { u as invariant } from "./routes-BwOSW5AT.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";
@@ -21,9 +21,9 @@ import { createRequestHandler, matchRoutes, unstable_setDevServerHooks } from "r
 import * as Path from "pathe";
 import path$1, { join } from "pathe";
 import pick from "lodash/pick.js";
-import * as babel from "@babel/core";
 import { createHash } from "node:crypto";
 import * as url from "node:url";
+import * as babel from "@babel/core";
 import { createRequest, sendResponse } from "@remix-run/node-fetch-server";
 import { init, parse } from "es-module-lexer";
 import jsesc from "jsesc";
@@ -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;
@@ -2724,7 +2736,7 @@ const nodeRequire = createRequire(import.meta.url);
 function hasDependency({ name, rootDirectory }) {
 	try {
 		return Boolean(nodeRequire.resolve(name, { paths: [rootDirectory] }));
-	} catch (e) {
+	} catch {
 		return false;
 	}
 }

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@react-router/dev",
   "type": "module",
-  "version": "8.3.0",
+  "version": "8.4.0",
   "description": "Dev tools and CLI for React Router",
   "keywords": [
     "react",
@@ -88,11 +88,9 @@
     "@babel/core": "^7.29.7",
     "@babel/generator": "^7.29.7",
     "@babel/parser": "^7.29.7",
-    "@babel/plugin-syntax-jsx": "^7.29.7",
-    "@babel/preset-typescript": "^7.29.7",
     "@babel/traverse": "^7.29.7",
     "@babel/types": "^7.29.7",
-    "@remix-run/node-fetch-server": "^0.13.3",
+    "@remix-run/node-fetch-server": "^0.14.1",
     "babel-dead-code-elimination": "^1.0.12",
     "chokidar": "^5.0.0",
     "dedent": "^1.7.2",
@@ -105,14 +103,15 @@
     "pathe": "^2.0.3",
     "picocolors": "^1.1.1",
     "pkg-types": "^2.3.1",
-    "prettier": "^3.8.3",
     "react-refresh": "^0.18.0",
     "semver": "^7.8.1",
     "tinyglobby": "^0.2.16",
     "valibot": "^1.4.1",
-    "@react-router/node": "8.3.0"
+    "@react-router/node": "8.4.0"
   },
   "devDependencies": {
+    "@babel/plugin-syntax-jsx": "^7.29.7",
+    "@babel/preset-typescript": "^7.29.7",
     "@types/babel__core": "^7.20.5",
     "@types/babel__generator": "^7.27.0",
     "@types/babel__traverse": "^7.28.0",
@@ -127,13 +126,14 @@
     "esbuild-register": "^3.6.0",
     "execa": "9.6.1",
     "fast-glob": "3.3.3",
+    "prettier": "^3.8.3",
     "tsdown": "^0.22.0",
     "typescript": "^6.0.3",
     "vite": "^7.0.0",
     "wireit": "0.14.12",
     "wrangler": "^4.95.0",
-    "react-router": "^8.3.0",
-    "@react-router/serve": "8.3.0"
+    "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": "^8.3.0",
-    "@react-router/serve": "^8.3.0"
+    "@react-router/serve": "^8.4.0",
+    "react-router": "^8.4.0"
   },
   "peerDependenciesMeta": {
     "@vitejs/plugin-rsc": {