@react-navigation/routers 7.5.6 → 7.6.1

package/src/BaseRouter.tsx

@@ -6,79 +6,94 @@ import type {
   PartialState,
 } from './types';
 
-/**
- * Base router object that can be used when writing custom routers.
- * This provides few helper methods to handle common actions such as `RESET`.
- */
-export const BaseRouter = {
-  getStateForAction<State extends NavigationState>(
-    state: State,
-    action: CommonNavigationAction
-  ): State | PartialState<State> | null {
-    switch (action.type) {
-      case 'SET_PARAMS':
-      case 'REPLACE_PARAMS': {
-        const index = action.source
-          ? state.routes.findIndex((r) => r.key === action.source)
-          : state.index;
+function getStateForAction<State extends NavigationState>(
+  state: State,
+  action: CommonNavigationAction
+): State | PartialState<State> | null;
+function getStateForAction(
+  state: NavigationState,
+  action: CommonNavigationAction
+) {
+  switch (action.type) {
+    case 'SET_PARAMS':
+    case 'REPLACE_PARAMS': {
+      const index = action.source
+        ? state.routes.findIndex((r) => r.key === action.source)
+        : state.index;
 
-        if (index === -1) {
-          return null;
-        }
+      if (index === -1) {
+        return null;
+      }
 
-        return {
-          ...state,
-          routes: state.routes.map((r, i) =>
-            i === index
-              ? {
-                  ...r,
-                  params:
-                    action.type === 'REPLACE_PARAMS'
-                      ? action.payload.params
-                      : { ...r.params, ...action.payload.params },
-                }
-              : r
-          ),
-        };
+      return {
+        ...state,
+        routes: state.routes.map((r, i) =>
+          i === index
+            ? {
+                ...r,
+                params:
+                  action.type === 'REPLACE_PARAMS'
+                    ? action.payload.params
+                    : { ...r.params, ...action.payload.params },
+              }
+            : r
+        ),
+      };
+    }
+
+    case 'RESET': {
+      const nextState = action.payload;
+
+      if (nextState == null) {
+        return null;
       }
 
-      case 'RESET': {
-        const nextState = action.payload as State | PartialState<State>;
+      const routeNamesSet = new Set(state.routeNames);
 
-        const routeNamesSet = new Set(state.routeNames);
+      if (
+        nextState.routes.length === 0 ||
+        nextState.routes.some(
+          (route: { name: string }) => !routeNamesSet.has(route.name)
+        )
+      ) {
+        return null;
+      }
 
+      if (nextState.stale === false) {
         if (
-          nextState.routes.length === 0 ||
-          nextState.routes.some(
-            (route: { name: string }) => !routeNamesSet.has(route.name)
-          )
+          state.routeNames.length !== nextState.routeNames.length ||
+          nextState.routeNames.some((name) => !routeNamesSet.has(name)) ||
+          !Number.isInteger(nextState.index) ||
+          nextState.index < 0 ||
+          nextState.index >= nextState.routes.length
         ) {
           return null;
         }
 
-        if (nextState.stale === false) {
-          if (
-            state.routeNames.length !== nextState.routeNames.length ||
-            nextState.routeNames.some((name) => !routeNamesSet.has(name))
-          ) {
-            return null;
-          }
-
-          return {
-            ...nextState,
-            routes: nextState.routes.map((route) =>
-              route.key ? route : { ...route, key: `${route.name}-${nanoid()}` }
-            ),
-          };
-        }
-
-        return nextState;
+        return {
+          ...nextState,
+          routes: nextState.routes.map((route) =>
+            'key' in route && route.key
+              ? route
+              : { ...route, key: `${route.name}-${nanoid()}` }
+          ),
+        };
       }
 
-      default:
-        return null;
+      return nextState;
     }
-  },
+
+    default:
+      return null;
+  }
+}
+
+/**
+ * Base router object that can be used when writing custom routers.
+ * This provides few helper methods to handle common actions such as `RESET`.
+ */
+export const BaseRouter = {
+  getStateForAction,
 
   shouldActionChangeFocus(action: CommonNavigationAction) {
     return action.type === 'NAVIGATE' || action.type === 'NAVIGATE_DEPRECATED';

package/src/CommonActions.tsx

@@ -90,6 +90,9 @@ export function navigate(
   }
 ): Action;
 
+/**
+ * @deprecated Use `navigate(name, params, options)` instead.
+ */
 export function navigate(options: {
   name: string;
   params?: object;
@@ -126,6 +129,10 @@ export function navigate(...args: any): Action {
       );
     }
 
+    console.warn(
+      `Passing an object as the argument to 'navigate' is deprecated. Use 'navigate(name, params, options)' instead.`
+    );
+
     return { type: 'NAVIGATE', payload };
   }
 }

package/src/DrawerRouter.tsx

@@ -1,11 +1,11 @@
 import { nanoid } from 'nanoid/non-secure';
 
+import { SwitchRouter } from './SwitchRouter';
 import {
   type TabActionHelpers,
   TabActions,
   type TabActionType,
   type TabNavigationState,
-  TabRouter,
   type TabRouterOptions,
 } from './TabRouter';
 import type {
@@ -14,6 +14,7 @@ import type {
   PartialState,
   Router,
 } from './types';
+
 export type DrawerStatus = 'open' | 'closed';
 
 export type DrawerActionType =
@@ -44,7 +45,7 @@ export type DrawerNavigationState<ParamList extends ParamListBase> = Omit<
    * List of previously visited route keys and drawer open status.
    */
   history: (
-    | { type: 'route'; key: string }
+    | { type: 'route'; key: string; params?: object | undefined }
     | { type: 'drawer'; status: DrawerStatus }
   )[];
 };
@@ -87,10 +88,7 @@ export function DrawerRouter({
   DrawerNavigationState<ParamListBase>,
   DrawerActionType | CommonNavigationAction
 > {
-  const router = TabRouter(rest) as unknown as Router<
-    DrawerNavigationState<ParamListBase>,
-    TabActionType | CommonNavigationAction
-  >;
+  const router = SwitchRouter<'drawer'>(rest);
 
   const isDrawerInHistory = (
     state:
@@ -179,11 +177,16 @@ export function DrawerRouter({
         return partialState;
       }
 
-      let state = router.getRehydratedState(partialState, {
-        routeNames,
-        routeParamList,
-        routeGetIdList,
-      });
+      let state: DrawerNavigationState<ParamListBase> = {
+        ...router.getRehydratedState(partialState, {
+          routeNames,
+          routeParamList,
+          routeGetIdList,
+        }),
+        default: defaultStatus,
+        type: 'drawer',
+        key: `drawer-${nanoid()}`,
+      };
 
       if (isDrawerInHistory(partialState)) {
         // Re-sync the drawer entry in history to correct it if it was wrong
@@ -191,12 +194,7 @@ export function DrawerRouter({
         state = addDrawerToHistory(state);
       }
 
-      return {
-        ...state,
-        default: defaultStatus,
-        type: 'drawer',
-        key: `drawer-${nanoid()}`,
-      };
+      return state;
     },
 
     getStateForRouteFocus(state, key) {
@@ -225,8 +223,12 @@ export function DrawerRouter({
         case 'NAVIGATE_DEPRECATED': {
           const result = router.getStateForAction(state, action, options);
 
-          if (result != null && result.index !== state.index) {
-            return closeDrawer(result as DrawerNavigationState<ParamListBase>);
+          if (
+            result != null &&
+            result.stale === false &&
+            result.index !== state.index
+          ) {
+            return closeDrawer(result);
           }
 
           return result;