Handle undefined node better

src/core/hooks/useFavoriteNode.test.ts

@@ -87,8 +87,7 @@ describe("useFavoriteNode hook", () => {
       result.current.updateFavorite({ nodeNum: 9999, isFavorite: false });
     });
 
-    expect(mockToast).toHaveBeenCalledWith({
-      title: "Removed node from favorites",
-    });
+    expect(mockUpdateFavorite).not.toHaveBeenCalled();
+    expect(mockToast).not.toHaveBeenCalled();
   });
 });

src/core/hooks/useFavoriteNode.ts

@@ -13,9 +13,10 @@ export function useFavoriteNode() {
 
   const updateFavoriteCB = useCallback(
     ({ nodeNum, isFavorite }: FavoriteNodeOptions) => {
-      updateFavorite(nodeNum, isFavorite);
-
       const node = getNode(nodeNum);
+      if (!node) return;
+
+      updateFavorite(nodeNum, isFavorite);
 
       toast({
         title: `${isFavorite ? "Added" : "Removed"} ${

src/core/hooks/useIgnoreNode.test.ts

@@ -90,8 +90,7 @@ describe("useIgnoreNode hook", () => {
       result.current.updateIgnored({ nodeNum: 9999, isIgnored: false });
     });
 
-    expect(mockToast).toHaveBeenCalledWith({
-      title: "Removed node from ignore list",
-    });
+    expect(mockUpdateIgnore).not.toHaveBeenCalled();
+    expect(mockToast).not.toHaveBeenCalled();
   });
 });

src/core/hooks/useIgnoreNode.ts

@@ -13,9 +13,10 @@ export function useIgnoreNode() {
 
   const updateIgnoredCB = useCallback(
     ({ nodeNum, isIgnored }: IgnoreNodeOptions) => {
-      updateIgnored(nodeNum, isIgnored);
-
       const node = getNode(nodeNum);
+      if (!node) return;
+
+      updateIgnored(nodeNum, isIgnored);
 
       toast({
         title: `${isIgnored ? "Added" : "Removed"} ${