Signed-off-by: Matt Bruce <mbrucedogs@gmail.com>

src/firebase/services.ts

@@ -364,7 +364,15 @@ export const disabledSongsService = {
     };
     
     debugLog('Saving disabled song:', disabledSong);
-    await set(disabledSongRef, disabledSong);
+    
+    // Add timeout to prevent hanging
+    const timeoutPromise = new Promise((_, reject) => {
+      setTimeout(() => reject(new Error('Operation timed out')), 10000);
+    });
+    
+    const savePromise = set(disabledSongRef, disabledSong);
+    
+    await Promise.race([savePromise, timeoutPromise]);
     debugLog('Disabled song saved successfully');
   },
 
@@ -372,7 +380,15 @@ export const disabledSongsService = {
   removeDisabledSong: async (controllerName: string, songPath: string) => {
     const songKey = disabledSongsService.generateSongKey(songPath);
     const disabledSongRef = ref(database, `controllers/${controllerName}/disabledSongs/${songKey}`);
-    await remove(disabledSongRef);
+    
+    // Add timeout to prevent hanging
+    const timeoutPromise = new Promise((_, reject) => {
+      setTimeout(() => reject(new Error('Operation timed out')), 10000);
+    });
+    
+    const removePromise = remove(disabledSongRef);
+    
+    await Promise.race([removePromise, timeoutPromise]);
   },
 
   // Check if a song is disabled

src/hooks/useDisabledSongs.ts

@@ -38,21 +38,12 @@ export const useDisabledSongs = () => {
     const unsubscribe = disabledSongsService.subscribeToDisabledSongs(
       controllerName,
       (songs) => {
-        // Only update if the data has actually changed
+        try {
-        setDisabledSongs(prevSongs => {
+          setDisabledSongs(songs);
-          if (JSON.stringify(prevSongs) !== JSON.stringify(songs)) {
+          setDisabledSongPaths(new Set(Object.values(songs).map((song: DisabledSong) => song.path)));
-            return songs;
+        } catch (error) {
-          }
+          console.error('Error updating disabled songs state:', error);
-          return prevSongs;
+        }
-        });
-        
-        setDisabledSongPaths(prevPaths => {
-          const newPaths = new Set(Object.values(songs).map((song: DisabledSong) => song.path));
-          if (JSON.stringify(Array.from(prevPaths)) !== JSON.stringify(Array.from(newPaths))) {
-            return newPaths;
-          }
-          return prevPaths;
-        });
       }
     );