@tanstack/router-core 1.131.16 → 1.131.17

package/dist/cjs/router.cjs

@@ -861,10 +861,7 @@ class RouterCore {
       const match = this.getMatch(matchId);
       if (shouldPending && match._nonReactive.pendingTimeout === void 0) {
         const pendingTimeout = setTimeout(() => {
-          try {
-            this.triggerOnReady(innerLoadContext);
-          } catch {
-          }
+          this.triggerOnReady(innerLoadContext);
         }, pendingMs);
         match._nonReactive.pendingTimeout = pendingTimeout;
       }
@@ -887,103 +884,122 @@ class RouterCore {
       return existingMatch._nonReactive.beforeLoadPromise ? existingMatch._nonReactive.beforeLoadPromise.then(then) : then();
     };
     this.executeBeforeLoad = (innerLoadContext, matchId, index, route) => {
-      var _a, _b, _c;
+      var _a;
+      const match = this.getMatch(matchId);
+      match._nonReactive.beforeLoadPromise = utils.createControlledPromise();
+      const prevLoadPromise = match._nonReactive.loadPromise;
+      match._nonReactive.loadPromise = utils.createControlledPromise(() => {
+        prevLoadPromise == null ? void 0 : prevLoadPromise.resolve();
+      });
+      const { paramsError, searchError } = match;
+      if (paramsError) {
+        this.handleSerialError(
+          innerLoadContext,
+          index,
+          paramsError,
+          "PARSE_PARAMS"
+        );
+      }
+      if (searchError) {
+        this.handleSerialError(
+          innerLoadContext,
+          index,
+          searchError,
+          "VALIDATE_SEARCH"
+        );
+      }
+      this.setupPendingTimeout(innerLoadContext, matchId, route);
+      const abortController = new AbortController();
+      const parentMatchId = (_a = innerLoadContext.matches[index - 1]) == null ? void 0 : _a.id;
+      const parentMatch = parentMatchId ? this.getMatch(parentMatchId) : void 0;
+      const parentMatchContext = (parentMatch == null ? void 0 : parentMatch.context) ?? this.options.context ?? void 0;
+      const context = { ...parentMatchContext, ...match.__routeContext };
+      let isPending = false;
+      const pending = () => {
+        if (isPending) return;
+        isPending = true;
+        innerLoadContext.updateMatch(matchId, (prev) => ({
+          ...prev,
+          isFetching: "beforeLoad",
+          fetchCount: prev.fetchCount + 1,
+          abortController,
+          context
+        }));
+      };
       const resolve = () => {
-        innerLoadContext.updateMatch(matchId, (prev) => {
-          var _a2;
-          (_a2 = prev._nonReactive.beforeLoadPromise) == null ? void 0 : _a2.resolve();
-          prev._nonReactive.beforeLoadPromise = void 0;
-          return {
-            ...prev,
-            isFetching: false
-          };
-        });
+        var _a2;
+        (_a2 = match._nonReactive.beforeLoadPromise) == null ? void 0 : _a2.resolve();
+        match._nonReactive.beforeLoadPromise = void 0;
+        innerLoadContext.updateMatch(matchId, (prev) => ({
+          ...prev,
+          isFetching: false
+        }));
       };
-      try {
-        const match = this.getMatch(matchId);
-        match._nonReactive.beforeLoadPromise = utils.createControlledPromise();
-        const prevLoadPromise = match._nonReactive.loadPromise;
-        match._nonReactive.loadPromise = utils.createControlledPromise(() => {
-          prevLoadPromise == null ? void 0 : prevLoadPromise.resolve();
+      if (!route.options.beforeLoad) {
+        store.batch(() => {
+          pending();
+          resolve();
         });
-        const { paramsError, searchError } = this.getMatch(matchId);
-        if (paramsError) {
-          this.handleSerialError(
-            innerLoadContext,
-            index,
-            paramsError,
-            "PARSE_PARAMS"
-          );
+        return;
+      }
+      const { search, params, cause } = match;
+      const preload = this.resolvePreload(innerLoadContext, matchId);
+      const beforeLoadFnContext = {
+        search,
+        abortController,
+        params,
+        preload,
+        context,
+        location: innerLoadContext.location,
+        navigate: (opts) => this.navigate({ ...opts, _fromLocation: innerLoadContext.location }),
+        buildLocation: this.buildLocation,
+        cause: preload ? "preload" : cause,
+        matches: innerLoadContext.matches
+      };
+      const updateContext = (beforeLoadContext2) => {
+        if (beforeLoadContext2 === void 0) {
+          store.batch(() => {
+            pending();
+            resolve();
+          });
+          return;
         }
-        if (searchError) {
+        if (redirect.isRedirect(beforeLoadContext2) || notFound.isNotFound(beforeLoadContext2)) {
+          pending();
           this.handleSerialError(
             innerLoadContext,
             index,
-            searchError,
-            "VALIDATE_SEARCH"
+            beforeLoadContext2,
+            "BEFORE_LOAD"
           );
         }
-        this.setupPendingTimeout(innerLoadContext, matchId, route);
-        const abortController = new AbortController();
-        const parentMatchId = (_a = innerLoadContext.matches[index - 1]) == null ? void 0 : _a.id;
-        const parentMatch = parentMatchId ? this.getMatch(parentMatchId) : void 0;
-        const parentMatchContext = (parentMatch == null ? void 0 : parentMatch.context) ?? this.options.context ?? void 0;
-        innerLoadContext.updateMatch(matchId, (prev) => ({
-          ...prev,
-          isFetching: "beforeLoad",
-          fetchCount: prev.fetchCount + 1,
-          abortController,
-          context: {
-            ...parentMatchContext,
-            ...prev.__routeContext
-          }
-        }));
-        const { search, params, context, cause } = this.getMatch(matchId);
-        const preload = this.resolvePreload(innerLoadContext, matchId);
-        const beforeLoadFnContext = {
-          search,
-          abortController,
-          params,
-          preload,
-          context,
-          location: innerLoadContext.location,
-          navigate: (opts) => this.navigate({ ...opts, _fromLocation: innerLoadContext.location }),
-          buildLocation: this.buildLocation,
-          cause: preload ? "preload" : cause,
-          matches: innerLoadContext.matches
-        };
-        const updateContext = (beforeLoadContext2) => {
-          if (redirect.isRedirect(beforeLoadContext2) || notFound.isNotFound(beforeLoadContext2)) {
-            this.handleSerialError(
-              innerLoadContext,
-              index,
-              beforeLoadContext2,
-              "BEFORE_LOAD"
-            );
-          }
+        store.batch(() => {
+          pending();
           innerLoadContext.updateMatch(matchId, (prev) => ({
             ...prev,
             __beforeLoadContext: beforeLoadContext2,
             context: {
-              ...parentMatchContext,
-              ...prev.__routeContext,
+              ...prev.context,
               ...beforeLoadContext2
-            },
-            abortController
+            }
           }));
-        };
-        const beforeLoadContext = (_c = (_b = route.options).beforeLoad) == null ? void 0 : _c.call(_b, beforeLoadFnContext);
+          resolve();
+        });
+      };
+      let beforeLoadContext;
+      try {
+        beforeLoadContext = route.options.beforeLoad(beforeLoadFnContext);
         if (utils.isPromise(beforeLoadContext)) {
-          return beforeLoadContext.then(updateContext).catch((err) => {
+          pending();
+          return beforeLoadContext.catch((err) => {
             this.handleSerialError(innerLoadContext, index, err, "BEFORE_LOAD");
-          }).then(resolve);
-        } else {
-          updateContext(beforeLoadContext);
+          }).then(updateContext);
         }
       } catch (err) {
+        pending();
         this.handleSerialError(innerLoadContext, index, err, "BEFORE_LOAD");
       }
-      resolve();
+      updateContext(beforeLoadContext);
       return;
     };
     this.handleBeforeLoad = (innerLoadContext, index) => {
@@ -1120,7 +1136,8 @@ class RouterCore {
           }));
         } catch (e) {
           let error = e;
-          await this.potentialPendingMinPromise(matchId);
+          const pendingPromise = this.potentialPendingMinPromise(matchId);
+          if (pendingPromise) await pendingPromise;
           this.handleRedirectAndNotFound(
             innerLoadContext,
             this.getMatch(matchId),