From 83b4682b0e21704072ee06ed268c1172f331d366 Mon Sep 17 00:00:00 2001
From: Geert Rademakes <g26107@eon.com>
Date: Fri, 8 Aug 2025 08:47:39 +0200
Subject: [PATCH] chore(frontend): remove live song-list refresh during S3 sync
 per request; refresh now only on manual actions/page reload

---
 packages/frontend/src/App.tsx                          | 10 ++--------
 .../frontend/src/components/BackgroundJobProgress.tsx  |  8 --------
 2 files changed, 2 insertions(+), 16 deletions(-)

diff --git a/packages/frontend/src/App.tsx b/packages/frontend/src/App.tsx
index 445affd..4d5c0e0 100644
--- a/packages/frontend/src/App.tsx
+++ b/packages/frontend/src/App.tsx
@@ -128,8 +128,7 @@ const RekordboxReader: React.FC = () => {
     totalDuration,
     loadNextPage,
     searchSongs,
-    searchQuery,
-    refresh
+    searchQuery
   } = usePaginatedSongs({ pageSize: 100, playlistName: currentPlaylist });
 
   // Export library to XML
@@ -661,12 +660,7 @@ const RekordboxReader: React.FC = () => {
       />
 
       {/* Background Job Progress */}
-      <BackgroundJobProgress
-        onTickWhileS3Sync={() => {
-          // Light refresh to pick up newly set hasS3File flags promptly
-          refresh();
-        }}
-      />
+      <BackgroundJobProgress />
     </Box>
   );
 };
diff --git a/packages/frontend/src/components/BackgroundJobProgress.tsx b/packages/frontend/src/components/BackgroundJobProgress.tsx
index 0f794be..d7b9787 100644
--- a/packages/frontend/src/components/BackgroundJobProgress.tsx
+++ b/packages/frontend/src/components/BackgroundJobProgress.tsx
@@ -45,15 +45,12 @@ interface BackgroundJobProgressProps {
   jobId?: string;
   onJobComplete?: (result: any) => void;
   onJobError?: (error: string) => void;
-  // Called on each polling tick while an S3 sync job is running
-  onTickWhileS3Sync?: () => void;
 }
 
 export const BackgroundJobProgress: React.FC<BackgroundJobProgressProps> = ({
   jobId,
   onJobComplete,
   onJobError,
-  onTickWhileS3Sync,
 }) => {
   const [jobs, setJobs] = useState<JobProgress[]>([]);
   const [loading, setLoading] = useState(false);
@@ -113,11 +110,6 @@ export const BackgroundJobProgress: React.FC<BackgroundJobProgressProps> = ({
       if (jobId) {
         updateJobProgress(jobId);
       }
-      
-      // Notify consumer while an S3 sync is running (to refresh song list, etc.)
-      if (onTickWhileS3Sync && activeJobs.some(j => j.type === 's3-sync')) {
-        onTickWhileS3Sync();
-      }
     }, 2000); // Poll every 2 seconds for less frequent updates
   };