fix: introduced custom hook with proper error handling for polling

src/shared/hooks/BackendAPI/useGet.js

@@ -7,6 +7,7 @@ import { v4 as uuid } from 'uuid';
 
 import { useFetch } from 'shared/hooks/BackendAPI/useFetch';
 import { authDataState } from '../../../state/authDataAtom';
+import { usePolling } from './usePolling';
 
 // allow <n> consecutive requests to fail before displaying error
 const ERROR_TOLERANCY = 2;
@@ -96,14 +97,12 @@ const useGetHook = processDataFn =>
       [fetch, authData],
     );
 
-    useEffect(() => {
-      const receivedForbidden = error?.code === 403;
-
-      // POLLING
-      if (!pollingInterval || receivedForbidden || skip) return;
-      const intervalId = setInterval(refetch(true, data), pollingInterval);
-      return _ => clearInterval(intervalId);
-    }, [path, pollingInterval, data, error, skip, refetch]);
+    usePolling(() => refetch(true, data), {
+      pollingInterval,
+      error,
+      skip,
+      path,
+    });
 
     useEffect(() => {
       // INITIAL FETCH on path being set/changed

src/shared/hooks/BackendAPI/usePolling.ts

@@ -0,0 +1,44 @@
+import { useEffect, useCallback, useRef } from 'react';
+
+interface PollingOptions {
+  pollingInterval?: number;
+  error?: any;
+  skip?: boolean;
+  path?: string;
+}
+
+export const usePolling = <T>(
+  refetchFn: () => Promise<T>,
+  options: PollingOptions,
+) => {
+  const { pollingInterval = 0, error, skip = false, path } = options;
+  const intervalIdRef = useRef<NodeJS.Timeout>();
+
+  const cleanup = useCallback(() => {
+    if (intervalIdRef.current) {
+      clearInterval(intervalIdRef.current);
+      intervalIdRef.current = undefined;
+    }
+  }, []);
+
+  const safeRefetch = useCallback(async () => {
+    try {
+      await refetchFn();
+    } catch (error) {
+      cleanup();
+    }
+  }, [refetchFn, cleanup]);
+
+  useEffect(() => {
+    const receivedForbidden = error?.code === 403;
+    if (!pollingInterval || receivedForbidden || skip) {
+      cleanup();
+      return;
+    }
+
+    safeRefetch();
+    intervalIdRef.current = setInterval(safeRefetch, pollingInterval);
+
+    return cleanup;
+  }, [pollingInterval, error, skip, safeRefetch, cleanup, path]);
+};