fix: ipni-check only runs when needed (#83)

* fix: use ipniCheck cid cache to prevent spamming

* fix: ipni-check stores success in localstorage

Author: SgtPooki

src/hooks/use-ipni-check.ts

@@ -1,5 +1,32 @@
 import { useCallback, useEffect, useRef } from 'react'
 
+// Session-scoped cache to prevent repeated IPNI checks per CID within a page session
+// Value indicates the last known result of the IPNI listing check
+const ipniSessionResultByCid: Map<string, 'success' | 'failed'> = new Map()
+
+// LocalStorage helpers for success-only persistence across tabs/sessions
+const LS_SUCCESS_PREFIX = 'ipni-check-success-v1:'
+
+function getLocalStorageSuccess(cid: string): boolean {
+  try {
+    const key = `${LS_SUCCESS_PREFIX}${cid}`
+    return typeof window !== 'undefined' && window.localStorage.getItem(key) === '1'
+  } catch {
+    return false
+  }
+}
+
+function setLocalStorageSuccess(cid: string): void {
+  try {
+    const key = `${LS_SUCCESS_PREFIX}${cid}`
+    if (typeof window !== 'undefined') {
+      window.localStorage.setItem(key, '1')
+    }
+  } catch {
+    // ignore storage write errors (quota/disabled/private mode)
+  }
+}
+
 interface UseIpniCheckOptions {
   cid: string | null
   isActive: boolean
@@ -75,6 +102,8 @@ export const useIpniCheck = ({
 
       if (success) {
         isCheckingRef.current = false
+        ipniSessionResultByCid.set(cid, 'success')
+        setLocalStorageSuccess(cid)
         onSuccessRef.current()
         return
       }
@@ -83,6 +112,7 @@ export const useIpniCheck = ({
       if (attemptRef.current >= maxAttempts) {
         console.debug(`[IpniCheck] Max attempts (${maxAttempts}) reached for CID ${cid}`)
         isCheckingRef.current = false
+        ipniSessionResultByCid.set(cid, 'failed')
         onErrorRef.current?.(new Error(`IPNI check failed after ${maxAttempts} attempts`))
         return
       }
@@ -101,6 +131,30 @@ export const useIpniCheck = ({
   // Start polling when isActive becomes true
   useEffect(() => {
     if (isActive && cid) {
+      // If we've already checked this CID in the current page session, reuse the result and skip polling
+      const prior = ipniSessionResultByCid.get(cid)
+      if (prior === 'success') {
+        console.debug('[IpniCheck] Session cache hit (success) for CID:', cid)
+        if (!getLocalStorageSuccess(cid)) {
+          setLocalStorageSuccess(cid)
+        }
+        onSuccessRef.current()
+        return
+      }
+      if (prior === 'failed') {
+        console.debug('[IpniCheck] Session cache hit (failed) for CID:', cid)
+        onErrorRef.current?.(new Error('IPNI check previously failed in this session'))
+        return
+      }
+
+      // Check cross-tab/session success cache in localStorage
+      if (getLocalStorageSuccess(cid)) {
+        console.debug('[IpniCheck] LocalStorage cache hit (success) for CID:', cid)
+        ipniSessionResultByCid.set(cid, 'success')
+        onSuccessRef.current()
+        return
+      }
+
       console.debug('[IpniCheck] Starting polling for CID:', cid)
       pollWithBackoff()
     }