fix: remove eventStore and setter functions from useEffect dependencies

- Remove eventStore from useArticleLoader and useExternalUrlLoader dependencies
- Remove setter functions from dependencies as they shouldn't change
- Only keep naddr/url and previewData/cachedUrlHighlights as dependencies
- This prevents content loaders from re-running when going offline
- Fixes the core issue where loading skeleton appears immediately on offline detection

src/hooks/useArticleLoader.ts

@@ -317,16 +317,6 @@ export function useArticleLoader({
     }
   }, [
     naddr,
-    eventStore,
+    previewData
-    previewData,
-    setSelectedUrl,
-    setReaderContent,
-    setReaderLoading,
-    setIsCollapsed,
-    setHighlights,
-    setHighlightsLoading,
-    setCurrentArticleCoordinate,
-    setCurrentArticleEventId,
-    setCurrentArticle
   ])
 }

src/hooks/useExternalUrlLoader.ts

@@ -167,16 +167,7 @@ export function useExternalUrlLoader({
     }
   }, [
     url,
-    eventStore,
+    cachedUrlHighlights
-    cachedUrlHighlights,
-    setReaderContent,
-    setReaderLoading,
-    setIsCollapsed,
-    setSelectedUrl,
-    setHighlights,
-    setCurrentArticleCoordinate,
-    setCurrentArticleEventId,
-    setHighlightsLoading
   ])
 
   // Keep UI highlights synced with cached store updates without reloading content