react-router 7.3.0-pre.0 → 7.3.0

package/dist/development/index.js

@@ -1,5 +1,5 @@
 /**
- * react-router v7.3.0-pre.0
+ * react-router v7.3.0
  *
  * Copyright (c) Remix Software Inc.
  *
@@ -2835,8 +2835,6 @@ function createStaticHandler(routes, opts) {
         "When using middleware in `staticHandler.query()`, any provided `requestContext` must bean instance of `unstable_RouterContextProvider`"
       );
       try {
-        let tailIdx = [...matches].reverse().findIndex((m) => !filterMatchesToLoad || filterMatchesToLoad(m));
-        let lowestLoadingIdx = tailIdx < 0 ? 0 : matches.length - 1 - tailIdx;
         let renderedStaticContext;
         let response = await runMiddlewarePipeline(
           {
@@ -2847,7 +2845,6 @@ function createStaticHandler(routes, opts) {
             // this to the proper type knowing it's not an `AppLoadContext`
             context: requestContext
           },
-          lowestLoadingIdx,
           true,
           async () => {
             let result2 = await queryImpl(
@@ -2967,7 +2964,6 @@ function createStaticHandler(routes, opts) {
           // this to the proper type knowing it's not an `AppLoadContext`
           context: requestContext
         },
-        matches.length - 1,
         true,
         async () => {
           let result2 = await queryImpl(
@@ -3724,55 +3720,39 @@ async function defaultDataStrategyWithMiddleware(args) {
   if (!args.matches.some((m) => m.route.unstable_middleware)) {
     return defaultDataStrategy(args);
   }
-  let lastIndex = args.matches.length - 1;
-  for (let i = lastIndex; i >= 0; i--) {
-    if (args.matches[i].shouldLoad) {
-      lastIndex = i;
-      break;
-    }
-  }
-  let results = await runMiddlewarePipeline(
+  return runMiddlewarePipeline(
     args,
-    lastIndex,
     false,
-    async (keyedResults) => {
-      Object.assign(keyedResults, await defaultDataStrategy(args));
-    },
-    (e, keyedResults) => {
-      Object.assign(keyedResults, {
-        [e.routeId]: { type: "error", result: e.error }
-      });
-    }
+    () => defaultDataStrategy(args),
+    (e) => ({ [e.routeId]: { type: "error", result: e.error } })
   );
-  return results;
 }
-async function runMiddlewarePipeline({
-  request,
-  params,
-  context,
-  matches
-}, lastIndex, propagateResult, handler, errorHandler) {
+async function runMiddlewarePipeline(args, propagateResult, handler, errorHandler) {
+  let { matches, request, params, context } = args;
   let middlewareState = {
-    keyedResults: {},
+    handlerResult: void 0,
     propagateResult
   };
   try {
+    let tuples = matches.flatMap(
+      (m) => m.route.unstable_middleware ? m.route.unstable_middleware.map((fn) => [m.route.id, fn]) : []
+    );
     let result = await callRouteMiddleware(
-      matches.slice(0, lastIndex + 1).flatMap(
-        (m) => m.route.unstable_middleware ? m.route.unstable_middleware.map((fn) => [m.route.id, fn]) : []
-      ),
-      0,
       { request, params, context },
+      tuples,
       middlewareState,
       handler
     );
-    return middlewareState.propagateResult ? result : middlewareState.keyedResults;
+    return middlewareState.propagateResult ? result : middlewareState.handlerResult;
   } catch (e) {
     if (!(e instanceof MiddlewareError)) {
       throw e;
     }
-    let result = await errorHandler(e, middlewareState.keyedResults);
-    return middlewareState.propagateResult ? result : middlewareState.keyedResults;
+    let result = await errorHandler(e);
+    if (propagateResult || !middlewareState.handlerResult) {
+      return result;
+    }
+    return Object.assign(middlewareState.handlerResult, result);
   }
 }
 var MiddlewareError = class {
@@ -3781,7 +3761,7 @@ var MiddlewareError = class {
     this.error = error;
   }
 };
-async function callRouteMiddleware(middlewares, idx, args, middlewareState, handler) {
+async function callRouteMiddleware(args, middlewares, middlewareState, handler, idx = 0) {
   let { request } = args;
   if (request.signal.aborted) {
     if (request.signal.reason) {
@@ -3793,8 +3773,8 @@ async function callRouteMiddleware(middlewares, idx, args, middlewareState, hand
   }
   let tuple = middlewares[idx];
   if (!tuple) {
-    let result = await handler(middlewareState.keyedResults);
-    return result;
+    middlewareState.handlerResult = await handler();
+    return middlewareState.handlerResult;
   }
   let [routeId, middleware] = tuple;
   let nextCalled = false;
@@ -3805,11 +3785,11 @@ async function callRouteMiddleware(middlewares, idx, args, middlewareState, hand
     }
     nextCalled = true;
     let result = await callRouteMiddleware(
-      middlewares,
-      idx + 1,
       args,
+      middlewares,
       middlewareState,
-      handler
+      handler,
+      idx + 1
     );
     if (middlewareState.propagateResult) {
       nextResult = result;
@@ -6172,28 +6152,15 @@ function StreamTransfer({
     )));
   }
 }
-function middlewareErrorHandler(e, keyedResults) {
-  Object.assign(keyedResults, {
-    [e.routeId]: { type: "error", result: e.error }
-  });
-}
 function getSingleFetchDataStrategy(manifest, routeModules, ssr, basename, getRouter) {
   return async (args) => {
     let { request, matches, fetcherKey } = args;
     if (request.method !== "GET") {
       return runMiddlewarePipeline(
         args,
-        matches.findIndex((m) => m.shouldLoad),
         false,
-        async (keyedResults) => {
-          let results = await singleFetchActionStrategy(
-            request,
-            matches,
-            basename
-          );
-          Object.assign(keyedResults, results);
-        },
-        middlewareErrorHandler
+        () => singleFetchActionStrategy(request, matches, basename),
+        (e) => ({ [e.routeId]: { type: "error", result: e.error } })
       );
     }
     if (!ssr) {
@@ -6201,64 +6168,35 @@ function getSingleFetchDataStrategy(manifest, routeModules, ssr, basename, getRo
         (m) => m.shouldLoad && manifest.routes[m.route.id]?.hasLoader && !manifest.routes[m.route.id]?.hasClientLoader
       );
       if (!foundRevalidatingServerLoader) {
-        let tailIdx = [...matches].reverse().findIndex((m) => m.shouldLoad);
-        let lowestLoadingIndex2 = tailIdx < 0 ? 0 : matches.length - 1 - tailIdx;
         return runMiddlewarePipeline(
           args,
-          lowestLoadingIndex2,
           false,
-          async (keyedResults) => {
-            let results = await nonSsrStrategy(
-              manifest,
-              request,
-              matches,
-              basename
-            );
-            Object.assign(keyedResults, results);
-          },
-          middlewareErrorHandler
+          () => nonSsrStrategy(manifest, request, matches, basename),
+          (e) => ({ [e.routeId]: { type: "error", result: e.error } })
         );
       }
     }
     if (fetcherKey) {
       return runMiddlewarePipeline(
         args,
-        matches.findIndex((m) => m.shouldLoad),
         false,
-        async (keyedResults) => {
-          let results = await singleFetchLoaderFetcherStrategy(
-            request,
-            matches,
-            basename
-          );
-          Object.assign(keyedResults, results);
-        },
-        middlewareErrorHandler
+        () => singleFetchLoaderFetcherStrategy(request, matches, basename),
+        (e) => ({ [e.routeId]: { type: "error", result: e.error } })
       );
     }
-    let lowestLoadingIndex = getLowestLoadingIndex(
-      manifest,
-      routeModules,
-      getRouter(),
-      matches
-    );
     return runMiddlewarePipeline(
       args,
-      lowestLoadingIndex,
       false,
-      async (keyedResults) => {
-        let results = await singleFetchLoaderNavigationStrategy(
-          manifest,
-          routeModules,
-          ssr,
-          getRouter(),
-          request,
-          matches,
-          basename
-        );
-        Object.assign(keyedResults, results);
-      },
-      middlewareErrorHandler
+      () => singleFetchLoaderNavigationStrategy(
+        manifest,
+        routeModules,
+        ssr,
+        getRouter(),
+        request,
+        matches,
+        basename
+      ),
+      (e) => ({ [e.routeId]: { type: "error", result: e.error } })
     );
   };
 }
@@ -6311,17 +6249,6 @@ async function nonSsrStrategy(manifest, request, matches, basename) {
 function isOptedOut(manifestRoute, routeModule, match, router) {
   return match.route.id in router.state.loaderData && manifestRoute && manifestRoute.hasLoader && routeModule && routeModule.shouldRevalidate;
 }
-function getLowestLoadingIndex(manifest, routeModules, router, matches) {
-  let tailIdx = [...matches].reverse().findIndex(
-    (m) => m.shouldLoad || !isOptedOut(
-      manifest.routes[m.route.id],
-      routeModules[m.route.id],
-      m,
-      router
-    )
-  );
-  return tailIdx < 0 ? 0 : matches.length - 1 - tailIdx;
-}
 async function singleFetchLoaderNavigationStrategy(manifest, routeModules, ssr, router, request, matches, basename) {
   let routesParams = /* @__PURE__ */ new Set();
   let foundOptOutRoute = false;
@@ -7736,7 +7663,7 @@ function mergeRefs(...refs) {
 var isBrowser = typeof window !== "undefined" && typeof window.document !== "undefined" && typeof window.document.createElement !== "undefined";
 try {
   if (isBrowser) {
-    window.__reactRouterVersion = "7.3.0-pre.0";
+    window.__reactRouterVersion = "7.3.0";
   }
 } catch (e) {
 }

package/dist/development/index.mjs

@@ -1,5 +1,5 @@
 /**
- * react-router v7.3.0-pre.0
+ * react-router v7.3.0
  *
  * Copyright (c) Remix Software Inc.
  *
@@ -123,7 +123,7 @@ import {
   useSearchParams,
   useSubmit,
   useViewTransitionState
-} from "./chunk-W5LWFPOT.mjs";
+} from "./chunk-K6CSEXPM.mjs";
 export {
   Await,
   BrowserRouter,

package/dist/development/lib/types/route-module.d.mts

@@ -99,10 +99,11 @@ type ClientDataFunctionArgs<T extends RouteInfo> = {
     params: T["params"];
     /**
      * When `future.unstable_middleware` is not enabled, this is undefined.
+     *
      * When `future.unstable_middleware` is enabled, this is an instance of
-     * `{@link unstable_RouterContextProvider}` and can be used to access context values
+     * `unstable_RouterContextProvider` and can be used to access context values
      * from your route middlewares.  You may pass in initial context values in your
-     * `{@link unstable_getContext}` function passed tp `{@link HydratedRouter}`
+     * `<HydratedRouter unstable_getContext>` prop
      */
     context: unstable_RouterContextProvider;
 };
@@ -126,15 +127,15 @@ type ServerDataFunctionArgs<T extends RouteInfo> = {
     params: T["params"];
     /**
      * Without `future.unstable_middleware` enabled, this is the context passed in
-     * to your server adapter's `{@link getLoadContext}` function. It's a way to bridge the
+     * to your server adapter's `getLoadContext` function. It's a way to bridge the
      * gap between the adapter's request/response API with your React Router app.
      * It is only applicable if you are using a custom server adapter.
      *
      * With `future.unstable_middleware` enabled, this is an instance of
-     * `{@link unstable_RouterContextProvider}` and can be used for type-safe access to
+     * `unstable_RouterContextProvider` and can be used for type-safe access to
      * context value set in your route middlewares.  If you are using a custom
      * server adapter, you may provide an initial set of context values from your
-     * `{@link getLoadContext}` function.
+     * `getLoadContext` function.
      */
     context: MiddlewareEnabled extends true ? unstable_RouterContextProvider : AppLoadContext;
 };

package/dist/development/lib/types/route-module.d.ts

@@ -99,10 +99,11 @@ type ClientDataFunctionArgs<T extends RouteInfo> = {
     params: T["params"];
     /**
      * When `future.unstable_middleware` is not enabled, this is undefined.
+     *
      * When `future.unstable_middleware` is enabled, this is an instance of
-     * `{@link unstable_RouterContextProvider}` and can be used to access context values
+     * `unstable_RouterContextProvider` and can be used to access context values
      * from your route middlewares.  You may pass in initial context values in your
-     * `{@link unstable_getContext}` function passed tp `{@link HydratedRouter}`
+     * `<HydratedRouter unstable_getContext>` prop
      */
     context: unstable_RouterContextProvider;
 };
@@ -126,15 +127,15 @@ type ServerDataFunctionArgs<T extends RouteInfo> = {
     params: T["params"];
     /**
      * Without `future.unstable_middleware` enabled, this is the context passed in
-     * to your server adapter's `{@link getLoadContext}` function. It's a way to bridge the
+     * to your server adapter's `getLoadContext` function. It's a way to bridge the
      * gap between the adapter's request/response API with your React Router app.
      * It is only applicable if you are using a custom server adapter.
      *
      * With `future.unstable_middleware` enabled, this is an instance of
-     * `{@link unstable_RouterContextProvider}` and can be used for type-safe access to
+     * `unstable_RouterContextProvider` and can be used for type-safe access to
      * context value set in your route middlewares.  If you are using a custom
      * server adapter, you may provide an initial set of context values from your
-     * `{@link getLoadContext}` function.
+     * `getLoadContext` function.
      */
     context: MiddlewareEnabled extends true ? unstable_RouterContextProvider : AppLoadContext;
 };

package/dist/development/lib/types/route-module.js

@@ -1,5 +1,5 @@
 /**
- * react-router v7.3.0-pre.0
+ * react-router v7.3.0
  *
  * Copyright (c) Remix Software Inc.
  *

package/dist/development/lib/types/route-module.mjs

@@ -1,5 +1,5 @@
 /**
- * react-router v7.3.0-pre.0
+ * react-router v7.3.0
  *
  * Copyright (c) Remix Software Inc.
  *

package/dist/production/{chunk-ZITGHOQF.mjs → chunk-RFJZTPXA.mjs}

@@ -1,5 +1,5 @@
 /**
- * react-router v7.3.0-pre.0
+ * react-router v7.3.0
  *
  * Copyright (c) Remix Software Inc.
  *
@@ -2687,8 +2687,6 @@ function createStaticHandler(routes, opts) {
         "When using middleware in `staticHandler.query()`, any provided `requestContext` must bean instance of `unstable_RouterContextProvider`"
       );
       try {
-        let tailIdx = [...matches].reverse().findIndex((m) => !filterMatchesToLoad || filterMatchesToLoad(m));
-        let lowestLoadingIdx = tailIdx < 0 ? 0 : matches.length - 1 - tailIdx;
         let renderedStaticContext;
         let response = await runMiddlewarePipeline(
           {
@@ -2699,7 +2697,6 @@ function createStaticHandler(routes, opts) {
             // this to the proper type knowing it's not an `AppLoadContext`
             context: requestContext
           },
-          lowestLoadingIdx,
           true,
           async () => {
             let result2 = await queryImpl(
@@ -2819,7 +2816,6 @@ function createStaticHandler(routes, opts) {
           // this to the proper type knowing it's not an `AppLoadContext`
           context: requestContext
         },
-        matches.length - 1,
         true,
         async () => {
           let result2 = await queryImpl(
@@ -3576,55 +3572,39 @@ async function defaultDataStrategyWithMiddleware(args) {
   if (!args.matches.some((m) => m.route.unstable_middleware)) {
     return defaultDataStrategy(args);
   }
-  let lastIndex = args.matches.length - 1;
-  for (let i = lastIndex; i >= 0; i--) {
-    if (args.matches[i].shouldLoad) {
-      lastIndex = i;
-      break;
-    }
-  }
-  let results = await runMiddlewarePipeline(
+  return runMiddlewarePipeline(
     args,
-    lastIndex,
     false,
-    async (keyedResults) => {
-      Object.assign(keyedResults, await defaultDataStrategy(args));
-    },
-    (e, keyedResults) => {
-      Object.assign(keyedResults, {
-        [e.routeId]: { type: "error", result: e.error }
-      });
-    }
+    () => defaultDataStrategy(args),
+    (e) => ({ [e.routeId]: { type: "error", result: e.error } })
   );
-  return results;
 }
-async function runMiddlewarePipeline({
-  request,
-  params,
-  context,
-  matches
-}, lastIndex, propagateResult, handler, errorHandler) {
+async function runMiddlewarePipeline(args, propagateResult, handler, errorHandler) {
+  let { matches, request, params, context } = args;
   let middlewareState = {
-    keyedResults: {},
+    handlerResult: void 0,
     propagateResult
   };
   try {
+    let tuples = matches.flatMap(
+      (m) => m.route.unstable_middleware ? m.route.unstable_middleware.map((fn) => [m.route.id, fn]) : []
+    );
     let result = await callRouteMiddleware(
-      matches.slice(0, lastIndex + 1).flatMap(
-        (m) => m.route.unstable_middleware ? m.route.unstable_middleware.map((fn) => [m.route.id, fn]) : []
-      ),
-      0,
       { request, params, context },
+      tuples,
       middlewareState,
       handler
     );
-    return middlewareState.propagateResult ? result : middlewareState.keyedResults;
+    return middlewareState.propagateResult ? result : middlewareState.handlerResult;
   } catch (e) {
     if (!(e instanceof MiddlewareError)) {
       throw e;
     }
-    let result = await errorHandler(e, middlewareState.keyedResults);
-    return middlewareState.propagateResult ? result : middlewareState.keyedResults;
+    let result = await errorHandler(e);
+    if (propagateResult || !middlewareState.handlerResult) {
+      return result;
+    }
+    return Object.assign(middlewareState.handlerResult, result);
   }
 }
 var MiddlewareError = class {
@@ -3633,7 +3613,7 @@ var MiddlewareError = class {
     this.error = error;
   }
 };
-async function callRouteMiddleware(middlewares, idx, args, middlewareState, handler) {
+async function callRouteMiddleware(args, middlewares, middlewareState, handler, idx = 0) {
   let { request } = args;
   if (request.signal.aborted) {
     if (request.signal.reason) {
@@ -3645,8 +3625,8 @@ async function callRouteMiddleware(middlewares, idx, args, middlewareState, hand
   }
   let tuple = middlewares[idx];
   if (!tuple) {
-    let result = await handler(middlewareState.keyedResults);
-    return result;
+    middlewareState.handlerResult = await handler();
+    return middlewareState.handlerResult;
   }
   let [routeId, middleware] = tuple;
   let nextCalled = false;
@@ -3657,11 +3637,11 @@ async function callRouteMiddleware(middlewares, idx, args, middlewareState, hand
     }
     nextCalled = true;
     let result = await callRouteMiddleware(
-      middlewares,
-      idx + 1,
       args,
+      middlewares,
       middlewareState,
-      handler
+      handler,
+      idx + 1
     );
     if (middlewareState.propagateResult) {
       nextResult = result;
@@ -6024,28 +6004,15 @@ function StreamTransfer({
     )));
   }
 }
-function middlewareErrorHandler(e, keyedResults) {
-  Object.assign(keyedResults, {
-    [e.routeId]: { type: "error", result: e.error }
-  });
-}
 function getSingleFetchDataStrategy(manifest, routeModules, ssr, basename, getRouter) {
   return async (args) => {
     let { request, matches, fetcherKey } = args;
     if (request.method !== "GET") {
       return runMiddlewarePipeline(
         args,
-        matches.findIndex((m) => m.shouldLoad),
         false,
-        async (keyedResults) => {
-          let results = await singleFetchActionStrategy(
-            request,
-            matches,
-            basename
-          );
-          Object.assign(keyedResults, results);
-        },
-        middlewareErrorHandler
+        () => singleFetchActionStrategy(request, matches, basename),
+        (e) => ({ [e.routeId]: { type: "error", result: e.error } })
       );
     }
     if (!ssr) {
@@ -6053,64 +6020,35 @@ function getSingleFetchDataStrategy(manifest, routeModules, ssr, basename, getRo
         (m) => m.shouldLoad && manifest.routes[m.route.id]?.hasLoader && !manifest.routes[m.route.id]?.hasClientLoader
       );
       if (!foundRevalidatingServerLoader) {
-        let tailIdx = [...matches].reverse().findIndex((m) => m.shouldLoad);
-        let lowestLoadingIndex2 = tailIdx < 0 ? 0 : matches.length - 1 - tailIdx;
         return runMiddlewarePipeline(
           args,
-          lowestLoadingIndex2,
           false,
-          async (keyedResults) => {
-            let results = await nonSsrStrategy(
-              manifest,
-              request,
-              matches,
-              basename
-            );
-            Object.assign(keyedResults, results);
-          },
-          middlewareErrorHandler
+          () => nonSsrStrategy(manifest, request, matches, basename),
+          (e) => ({ [e.routeId]: { type: "error", result: e.error } })
         );
       }
     }
     if (fetcherKey) {
       return runMiddlewarePipeline(
         args,
-        matches.findIndex((m) => m.shouldLoad),
         false,
-        async (keyedResults) => {
-          let results = await singleFetchLoaderFetcherStrategy(
-            request,
-            matches,
-            basename
-          );
-          Object.assign(keyedResults, results);
-        },
-        middlewareErrorHandler
+        () => singleFetchLoaderFetcherStrategy(request, matches, basename),
+        (e) => ({ [e.routeId]: { type: "error", result: e.error } })
       );
     }
-    let lowestLoadingIndex = getLowestLoadingIndex(
-      manifest,
-      routeModules,
-      getRouter(),
-      matches
-    );
     return runMiddlewarePipeline(
       args,
-      lowestLoadingIndex,
       false,
-      async (keyedResults) => {
-        let results = await singleFetchLoaderNavigationStrategy(
-          manifest,
-          routeModules,
-          ssr,
-          getRouter(),
-          request,
-          matches,
-          basename
-        );
-        Object.assign(keyedResults, results);
-      },
-      middlewareErrorHandler
+      () => singleFetchLoaderNavigationStrategy(
+        manifest,
+        routeModules,
+        ssr,
+        getRouter(),
+        request,
+        matches,
+        basename
+      ),
+      (e) => ({ [e.routeId]: { type: "error", result: e.error } })
     );
   };
 }
@@ -6163,17 +6101,6 @@ async function nonSsrStrategy(manifest, request, matches, basename) {
 function isOptedOut(manifestRoute, routeModule, match, router) {
   return match.route.id in router.state.loaderData && manifestRoute && manifestRoute.hasLoader && routeModule && routeModule.shouldRevalidate;
 }
-function getLowestLoadingIndex(manifest, routeModules, router, matches) {
-  let tailIdx = [...matches].reverse().findIndex(
-    (m) => m.shouldLoad || !isOptedOut(
-      manifest.routes[m.route.id],
-      routeModules[m.route.id],
-      m,
-      router
-    )
-  );
-  return tailIdx < 0 ? 0 : matches.length - 1 - tailIdx;
-}
 async function singleFetchLoaderNavigationStrategy(manifest, routeModules, ssr, router, request, matches, basename) {
   let routesParams = /* @__PURE__ */ new Set();
   let foundOptOutRoute = false;
@@ -7588,7 +7515,7 @@ function mergeRefs(...refs) {
 var isBrowser = typeof window !== "undefined" && typeof window.document !== "undefined" && typeof window.document.createElement !== "undefined";
 try {
   if (isBrowser) {
-    window.__reactRouterVersion = "7.3.0-pre.0";
+    window.__reactRouterVersion = "7.3.0";
   }
 } catch (e) {
 }