Fix redirects returned from loader using data

Author: brophdawg11

.changeset/purple-poems-laugh.md

@@ -0,0 +1,5 @@
+---
+"react-router": patch
+---
+
+Fix redirects returned from loaders/actions using `data()`

integration/defer-loader-test.ts

@@ -40,7 +40,9 @@ test.describe("deferred loaders", () => {
               }
             );
           }
-          export default function Redirect() {return null;}
+          export default function Redirect() {
+            return null;
+          }
         `,
 
         "app/routes/direct-promise-access.tsx": js`
@@ -82,17 +84,15 @@ test.describe("deferred loaders", () => {
 
   test.afterAll(async () => appFixture.close());
 
-  test.skip("deferred response can redirect on document request", async ({
+  test("deferred response can redirect on document request", async ({
     page,
   }) => {
     let app = new PlaywrightFixture(appFixture, page);
     await app.goto("/redirect");
     await page.waitForURL(/\?redirected/);
   });
 
-  test.skip("deferred response can redirect on transition", async ({
-    page,
-  }) => {
+  test("deferred response can redirect on transition", async ({ page }) => {
     let app = new PlaywrightFixture(appFixture, page);
     await app.goto("/");
     await app.clickLink("/redirect");

packages/react-router/lib/server-runtime/data.ts

@@ -1,3 +1,5 @@
+import { isDataWithResponseInit } from "../router/router";
+import { isRedirectStatusCode } from "./responses";
 import type {
   ActionFunction,
   ActionFunctionArgs,
@@ -32,6 +34,16 @@ export async function callRouteHandler(
     context: args.context,
   });
 
+  // If they returned a redirect via data(), re-throw it as a Response
+  if (
+    isDataWithResponseInit(result) &&
+    result.init &&
+    result.init.status &&
+    isRedirectStatusCode(result.init.status)
+  ) {
+    throw new Response(null, result.init);
+  }
+
   return result;
 }