Add eslint-disable directives for rules-of-hooks

Author: MajorLift

ui/hooks/snaps/useSnapNavigation.ts

@@ -1,22 +1,29 @@
+import { useCallback, useMemo } from 'react';
 import { useNavigate } from 'react-router-dom-v5-compat';
 import { parseMetaMaskUrl } from '@metamask/snaps-utils';
 import { getSnapRoute } from '../../helpers/utils/util';
 
 const useSnapNavigation = () => {
   const navigate = useNavigate();
-  const useSnapNavigate = (url: string) => {
-    let path;
-    const linkData = parseMetaMaskUrl(url);
-    if (linkData.snapId) {
-      path = getSnapRoute(linkData.snapId);
-    } else {
-      path = linkData.path;
-    }
-    navigate(path);
-  };
-  return {
-    useSnapNavigate,
-  };
+  const useSnapNavigate = useCallback(
+    (url: string) => {
+      let path;
+      const linkData = parseMetaMaskUrl(url);
+      if (linkData.snapId) {
+        path = getSnapRoute(linkData.snapId);
+      } else {
+        path = linkData.path;
+      }
+      navigate(path);
+    },
+    [navigate],
+  );
+  return useMemo(
+    () => ({
+      useSnapNavigate,
+    }),
+    [useSnapNavigate],
+  );
 };
 
 export default useSnapNavigation;

ui/pages/confirmations/components/confirm/info/hooks/useNestedTransactionLabels.ts

@@ -15,6 +15,7 @@ export function useNestedTransactionLabels({
     const { data, to } = nestedTransaction;
     // It's safe to call useFourByte here because the length of nestedTransactions
     // remains stable throughout the component's lifecycle.
+    // eslint-disable-next-line react-hooks/rules-of-hooks
     const methodData = useFourByte({ data, to });
     const functionName = methodData?.name;
 

ui/pages/notifications/notification-components/snap/snap-footer-button.tsx

@@ -44,6 +44,8 @@ export const SnapFooterButton = (props: { notification: SnapNotification }) => {
       if (isExternal) {
         setIsOpen(true);
       } else {
+        // This hook is safe to include in this event handler, because its only reactive component is `useNavigate`
+        // eslint-disable-next-line react-hooks/rules-of-hooks
         useSnapNavigate(href);
       }
     },