@tanstack/router-core 0.0.1-beta.181 → 0.0.1-beta.183

package/package.json

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

package/src/router.ts

@@ -56,6 +56,7 @@ import {
   createMemoryHistory,
   RouterHistory,
 } from './history'
+import warning from 'tiny-warning'
 
 //
 
@@ -204,6 +205,8 @@ export interface RouterState<
   pendingMatchIds: string[]
   matches: RouteMatch<TRouteTree>[]
   pendingMatches: RouteMatch<TRouteTree>[]
+  renderedMatchIds: string[]
+  renderedMatches: RouteMatch<TRouteTree>[]
   location: ParsedLocation<FullSearchSchema<TRouteTree>>
   resolvedLocation: ParsedLocation<FullSearchSchema<TRouteTree>>
   lastUpdated: number
@@ -371,6 +374,21 @@ export class Router<
           })
         }
 
+        if (matchesByIdChanged || matchesChanged || pendingMatchesChanged) {
+          const hasPendingComponent = next.pendingMatches.some((d) => {
+            const route = this.getRoute(d.routeId as any)
+            return !!route?.options.pendingComponent
+          })
+
+          next.renderedMatchIds = hasPendingComponent
+            ? next.pendingMatchIds
+            : next.matchIds
+
+          next.renderedMatches = next.renderedMatchIds.map((id) => {
+            return next.matchesById[id] as any
+          })
+        }
+
         next.isFetching = [...next.matches, ...next.pendingMatches].some(
           (d) => d.isFetching,
         )
@@ -665,24 +683,10 @@ export class Router<
     prevMatchesById: Record<string, RouteMatch<TRouteTree>>,
     nextMatches: AnyRouteMatch[],
   ): Record<string, RouteMatch<TRouteTree>> => {
-    const nextMatchesById: any = {
+    return {
       ...prevMatchesById,
+      ...Object.fromEntries(nextMatches.map((match) => [match.id, match])),
     }
-
-    let hadNew = false
-
-    nextMatches.forEach((match) => {
-      if (!nextMatchesById[match.id]) {
-        hadNew = true
-        nextMatchesById[match.id] = match
-      }
-    })
-
-    if (!hadNew) {
-      return prevMatchesById
-    }
-
-    return nextMatchesById
   }
 
   getRoute = (id: string): Route => {
@@ -892,16 +896,20 @@ export class Router<
               ? route.options.validateSearch.parse
               : route.options.validateSearch
 
-          const routeSearch = validator?.(parentSearchInfo.search) ?? {}
+          let routeSearch = validator?.(parentSearchInfo.search) ?? {}
 
-          const search = {
+          let search = {
             ...parentSearchInfo.search,
             ...routeSearch,
           }
 
+          routeSearch = replaceEqualDeep(match.routeSearch, routeSearch)
+          search = replaceEqualDeep(match.search, search)
+
           return {
-            routeSearch: replaceEqualDeep(match.routeSearch, routeSearch),
-            search: replaceEqualDeep(match.search, search),
+            routeSearch,
+            search,
+            searchDidChange: match.routeSearch !== routeSearch,
           }
         } catch (err: any) {
           match.searchError = new SearchParamError(err.message, {
@@ -1721,7 +1729,6 @@ export class Router<
     location: BuildNextOptions & { replace?: boolean; resetScroll?: boolean },
   ) => {
     const next = this.buildNext(location)
-    const id = '' + Date.now() + Math.random()
 
     if (this.navigateTimeout) clearTimeout(this.navigateTimeout)
 
@@ -1741,10 +1748,7 @@ export class Router<
       next.hash ? `#${next.hash}` : ''
     }`
 
-    this.history[nextAction === 'push' ? 'push' : 'replace'](href, {
-      id,
-      ...next.state,
-    })
+    this.history[nextAction === 'push' ? 'push' : 'replace'](href, next.state)
 
     this.resetNextScroll = location.resetScroll ?? true
 
@@ -1761,7 +1765,7 @@ export class Router<
   ) => {
     this.__store.setState((prev) => {
       if (!prev.matchesById[id]) {
-        console.warn(`No match found with id: ${id}`)
+        return prev
       }
 
       return {
@@ -1862,6 +1866,8 @@ function getInitialRouterState(): RouterState<any> {
     pendingMatchIds: [],
     matches: [],
     pendingMatches: [],
+    renderedMatchIds: [],
+    renderedMatches: [],
     lastUpdated: Date.now(),
   }
 }