@tanstack/router-core 0.0.1-beta.176 → 0.0.1-beta.178

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@tanstack/router-core",
   "author": "Tanner Linsley",
-  "version": "0.0.1-beta.176",
+  "version": "0.0.1-beta.178",
   "license": "MIT",
   "repository": "tanstack/router",
   "homepage": "https://tanstack.com/router",

package/src/route.ts

@@ -254,7 +254,6 @@ export type BaseRouteOptions<
   TRouteContext extends RouteContext = RouteContext,
   TAllContext extends AnyContext = AnyContext,
 > = RoutePathOptions<TCustomId, TPath> & {
-  layoutLimit?: string
   getParentRoute: () => TParentRoute
   validateSearch?: SearchSchemaValidator<TSearchSchema>
   loader?: LoaderFn<
@@ -265,7 +264,34 @@ export type BaseRouteOptions<
     NoInfer<TRouteContext>,
     TAllContext
   >
-} & ([TLoader] extends [never]
+} & (keyof PickRequired<RouteContext> extends never
+    ? // This async function is called before a route is loaded.
+      // If an error is thrown here, the route's loader will not be called.
+      // If thrown during a navigation, the navigation will be cancelled and the error will be passed to the `onError` function.
+      // If thrown during a preload event, the error will be logged to the console.
+      {
+        beforeLoad?: BeforeLoadFn<
+          TParentRoute,
+          TAllParams,
+          TSearchSchema,
+          TFullSearchSchema,
+          TParentContext,
+          TAllParentContext,
+          TRouteContext
+        >
+      }
+    : {
+        beforeLoad: BeforeLoadFn<
+          TParentRoute,
+          TAllParams,
+          TSearchSchema,
+          TFullSearchSchema,
+          TParentContext,
+          TAllParentContext,
+          TRouteContext
+        >
+      }) &
+  ([TLoader] extends [never]
     ? {
         loader: 'Loaders must return a type other than never. If you are throwing a redirect() and not returning anything, return a redirect() instead.'
       }
@@ -286,32 +312,12 @@ export type BaseRouteOptions<
         stringifyParams?: never
         parseParams?: never
       }
-  ) &
-  (keyof PickRequired<RouteContext> extends never
-    ? {
-        getContext?: GetContextFn<
-          TParentRoute,
-          TAllParams,
-          TFullSearchSchema,
-          TParentContext,
-          TAllParentContext,
-          TRouteContext
-        >
-      }
-    : {
-        getContext: GetContextFn<
-          TParentRoute,
-          TAllParams,
-          TFullSearchSchema,
-          TParentContext,
-          TAllParentContext,
-          TRouteContext
-        >
-      })
+  )
 
-type GetContextFn<
+type BeforeLoadFn<
   TParentRoute,
   TAllParams,
+  TSearchSchema,
   TFullSearchSchema,
   TParentContext,
   TAllParentContext,
@@ -319,7 +325,10 @@ type GetContextFn<
 > = (
   opts: {
     params: TAllParams
+    routeSearch: TSearchSchema
     search: TFullSearchSchema
+    abortController: AbortController
+    preload: boolean
   } & (TParentRoute extends undefined
     ? {
         context?: TAllParentContext
@@ -329,7 +338,7 @@ type GetContextFn<
         context: TAllParentContext
         parentContext: TParentContext
       }),
-) => TRouteContext
+) => Promise<TRouteContext> | TRouteContext | void
 
 export type UpdatableRouteOptions<
   TLoader,
@@ -378,19 +387,6 @@ export type UpdatableRouteOptions<
   maxAge?: number
   // If set, a match of this route that becomes inactive (or unused) will be garbage collected after this many milliseconds
   gcMaxAge?: number
-  // This async function is called before a route is loaded.
-  // If an error is thrown here, the route's loader will not be called.
-  // If thrown during a navigation, the navigation will be cancelled and the error will be passed to the `onError` function.
-  // If thrown during a preload event, the error will be logged to the console.
-  beforeLoad?: (
-    opts: LoaderContext<
-      TSearchSchema,
-      TFullSearchSchema,
-      TAllParams,
-      NoInfer<TRouteContext>,
-      TAllContext
-    >,
-  ) => Promise<void> | void
   onError?: (err: any) => void
   // These functions are called as route matches are loaded, stick around and leave the active
   // matches
@@ -641,15 +637,7 @@ export class Route<
     TAllParentContext,
     TRouteContext,
     TAllContext
-  > &
-    UpdatableRouteOptions<
-      TLoader,
-      TSearchSchema,
-      TFullSearchSchema,
-      TAllParams,
-      TRouteContext,
-      TAllContext
-    >
+  >
 
   // Set up in this.init()
   parentRoute!: TParentRoute

package/src/router.ts

@@ -445,7 +445,7 @@ export class Router<
 
   #onFocus = () => {
     if (this.options.refetchOnWindowFocus ?? true) {
-      this.reload()
+      this.invalidate()
     }
   }
 
@@ -914,52 +914,24 @@ export class Router<
         }
       })()
 
-      Object.assign(match, {
-        ...searchInfo,
-      })
-
-      const contextInfo = (() => {
-        try {
-          const routeContext =
-            route.options.getContext?.({
-              parentContext: parentMatch?.routeContext ?? {},
-              context: parentMatch?.context ?? this?.options.context ?? {},
-              params: match.params,
-              search: match.search,
-            }) || ({} as any)
-
-          const context = {
-            ...(parentMatch?.context ?? this?.options.context),
-            ...routeContext,
-          } as any
-
-          return {
-            context,
-            routeContext,
-          }
-        } catch (err) {
-          route.options.onError?.(err)
-          throw err
-        }
-      })()
-
-      Object.assign(match, {
-        ...contextInfo,
-      })
+      Object.assign(match, searchInfo)
     })
 
     return matches as any
   }
 
   loadMatches = async (
-    resolvedMatches: AnyRouteMatch[],
+    _resolvedMatches: AnyRouteMatch[],
     opts?: {
       preload?: boolean
       maxAge?: number
     },
   ) => {
+    const getFreshMatches = () =>
+      _resolvedMatches.map((d) => this.getRouteMatch(d.id)!)
+
     if (!opts?.preload) {
-      resolvedMatches.forEach((match) => {
+      getFreshMatches().forEach((match) => {
         // Update each match with its latest route data
         this.setRouteMatch(match.id, (s) => ({
           ...s,
@@ -980,7 +952,8 @@ export class Router<
 
     // Check each match middleware to see if the route can be accessed
     try {
-      for (const [index, match] of resolvedMatches.entries()) {
+      for (const [index, match] of getFreshMatches().entries()) {
+        const parentMatch = getFreshMatches()[index - 1]
         const route = this.getRoute(match.routeId)
 
         const handleError = (err: any, code: string) => {
@@ -1020,10 +993,24 @@ export class Router<
         let didError = false
 
         try {
-          await route.options.beforeLoad?.({
-            ...match,
-            preload: !!opts?.preload,
-          })
+          const routeContext =
+            (await route.options.beforeLoad?.({
+              ...match,
+              preload: !!opts?.preload,
+              parentContext: parentMatch?.routeContext ?? {},
+              context: parentMatch?.context ?? this?.options.context ?? {},
+            })) ?? ({} as any)
+
+          const context = {
+            ...(parentMatch?.context ?? this?.options.context),
+            ...routeContext,
+          } as any
+
+          this.setRouteMatch(match.id, (s) => ({
+            ...s,
+            context,
+            routeContext,
+          }))
         } catch (err) {
           handleError(err, 'BEFORE_LOAD')
           didError = true
@@ -1042,7 +1029,7 @@ export class Router<
       throw err
     }
 
-    const validResolvedMatches = resolvedMatches.slice(0, firstBadMatchIndex)
+    const validResolvedMatches = getFreshMatches().slice(0, firstBadMatchIndex)
     const matchPromises: Promise<any>[] = []
 
     validResolvedMatches.forEach((match, index) => {
@@ -1154,14 +1141,6 @@ export class Router<
     await Promise.all(matchPromises)
   }
 
-  reload = () => {
-    return this.navigate({
-      fromCurrent: true,
-      replace: true,
-      search: true,
-    } as any)
-  }
-
   resolvePath = (from: string, path: string) => {
     return resolvePath(this.basepath!, from, cleanPath(path))
   }
@@ -1854,7 +1833,11 @@ export class Router<
     }
 
     if (opts?.reload ?? true) {
-      return this.reload()
+      return this.navigate({
+        fromCurrent: true,
+        replace: true,
+        search: true,
+      } as any)
     }
   }
 }