From e2db895d66d27482aeff54228e0c6f4145ad429c Mon Sep 17 00:00:00 2001
From: tapframe <nayifveliya99@gmail.com>
Date: Sat, 21 Jun 2025 20:22:27 +0530
Subject: [PATCH] Enhance ContinueWatchingSection with background refresh and
 loading state management

This update improves the ContinueWatchingSection component by implementing a background refresh mechanism and managing loading states more effectively. The loadContinueWatching function now prevents concurrent refreshes and allows for manual refreshes with a loading indicator. Additionally, the debounce time for background refreshes has been slightly increased, enhancing performance and user experience.
---
 .../home/ContinueWatchingSection.tsx          | 37 ++++++++++++-------
 src/components/metadata/CastSection.tsx       |  3 --
 2 files changed, 24 insertions(+), 16 deletions(-)

diff --git a/src/components/home/ContinueWatchingSection.tsx b/src/components/home/ContinueWatchingSection.tsx
index f0cae1b47..1035d8ad8 100644
--- a/src/components/home/ContinueWatchingSection.tsx
+++ b/src/components/home/ContinueWatchingSection.tsx
@@ -77,10 +77,20 @@ const ContinueWatchingSection = React.forwardRef<ContinueWatchingRef>((props, re
   const appState = useRef(AppState.currentState);
   const refreshTimerRef = useRef<NodeJS.Timeout | null>(null);
 
+  // Use a state to track if a background refresh is in progress
+  const [isRefreshing, setIsRefreshing] = useState(false);
+
   // Modified loadContinueWatching to be more efficient
-  const loadContinueWatching = useCallback(async () => {
-    try {
+  const loadContinueWatching = useCallback(async (isBackgroundRefresh = false) => {
+    // Prevent multiple concurrent refreshes
+    if (isRefreshing) return;
+
+    if (!isBackgroundRefresh) {
       setLoading(true);
+    }
+    setIsRefreshing(true);
+
+    try {
       const allProgress = await storageService.getAllWatchProgress();
       
       if (Object.keys(allProgress).length === 0) {
@@ -193,8 +203,9 @@ const ContinueWatchingSection = React.forwardRef<ContinueWatchingRef>((props, re
       logger.error('Failed to load continue watching items:', error);
     } finally {
       setLoading(false);
+      setIsRefreshing(false);
     }
-  }, []);
+  }, [isRefreshing]);
 
   // Function to handle app state changes
   const handleAppStateChange = useCallback((nextAppState: AppStateStatus) => {
@@ -202,8 +213,8 @@ const ContinueWatchingSection = React.forwardRef<ContinueWatchingRef>((props, re
       appState.current.match(/inactive|background/) &&
       nextAppState === 'active'
     ) {
-      // App has come to the foreground - refresh data
-      loadContinueWatching();
+      // App has come to the foreground - trigger a background refresh
+      loadContinueWatching(true);
     }
     appState.current = nextAppState;
   }, [loadContinueWatching]);
@@ -220,8 +231,9 @@ const ContinueWatchingSection = React.forwardRef<ContinueWatchingRef>((props, re
         clearTimeout(refreshTimerRef.current);
       }
       refreshTimerRef.current = setTimeout(() => {
-        loadContinueWatching();
-      }, 300);
+        // Trigger a background refresh
+        loadContinueWatching(true);
+      }, 500); // Increased debounce time slightly
     };
 
     // Try to set up a custom event listener or use a timer as fallback
@@ -236,7 +248,7 @@ const ContinueWatchingSection = React.forwardRef<ContinueWatchingRef>((props, re
       };
     } else {
       // Fallback: poll for updates every 30 seconds
-      const intervalId = setInterval(loadContinueWatching, 30000);
+      const intervalId = setInterval(() => loadContinueWatching(true), 30000);
       return () => {
         subscription.remove();
         clearInterval(intervalId);
@@ -252,13 +264,12 @@ const ContinueWatchingSection = React.forwardRef<ContinueWatchingRef>((props, re
     loadContinueWatching();
   }, [loadContinueWatching]);
 
-  // Properly expose the refresh method
+  // Expose the refresh function via the ref
   React.useImperativeHandle(ref, () => ({
     refresh: async () => {
-      await loadContinueWatching();
-      // Return whether there are items to help parent determine visibility
-      const hasItems = continueWatchingItems.length > 0;
-      return hasItems;
+      // Allow manual refresh to show loading indicator
+      await loadContinueWatching(false);
+      return true;
     }
   }));
 
diff --git a/src/components/metadata/CastSection.tsx b/src/components/metadata/CastSection.tsx
index 07605cced..305f31e35 100644
--- a/src/components/metadata/CastSection.tsx
+++ b/src/components/metadata/CastSection.tsx
@@ -10,7 +10,6 @@ import {
 import { Image } from 'expo-image';
 import Animated, {
   FadeIn,
-  Layout,
 } from 'react-native-reanimated';
 import { useTheme } from '../../contexts/ThemeContext';
 
@@ -43,7 +42,6 @@ export const CastSection: React.FC<CastSectionProps> = ({
     <Animated.View 
       style={styles.castSection}
       entering={FadeIn.duration(300).delay(150)}
-      layout={Layout}
     >
       <View style={styles.sectionHeader}>
         <Text style={[styles.sectionTitle, { color: currentTheme.colors.highEmphasis }]}>Cast</Text>
@@ -57,7 +55,6 @@ export const CastSection: React.FC<CastSectionProps> = ({
         renderItem={({ item, index }) => (
           <Animated.View 
             entering={FadeIn.duration(300).delay(50 + index * 30)} 
-            layout={Layout}
           >
             <TouchableOpacity 
               style={styles.castCard}