fix(cloud): 🐛 correct retry count tracking for partial failures (#67)

Previously, during retry of failed pages (status PARTIAL_FAILED), the
completed count was based on page number from replayed events, which
could lead to duplicate counting or incorrect progress. The failed
count was not decremented when a retried page succeeded.

This commit ensures:
- Completed count increments and failed count decrements for each
  successful retried page.
- When all pages are completed after a retry (failed count reaches 0),
  any remaining non-completed pages are marked as completed.
- Both CloudPreview and List components handle retry scenarios
  consistently.

Author: jorben

src/renderer/pages/CloudPreview.tsx

@@ -250,13 +250,21 @@ const CloudPreview: React.FC = () => {
             }
             return [...prev, { page, status: 2, status_name: 'COMPLETED', markdown, width_mm: 210, height_mm: 297 }];
           });
-          setTask(prev => prev ? {
-            ...prev,
-            pages_completed: (prev.pages_completed || 0) + 1,
-          } : null);
+          setTask(prev => {
+            if (!prev) return null;
+            const updates: Partial<typeof prev> = {
+              pages_completed: (prev.pages_completed || 0) + 1,
+            };
+            // During retry (PARTIAL_FAILED), decrement failed count
+            if (prev.status === 8) {
+              updates.pages_failed = Math.max(0, (prev.pages_failed || 0) - 1);
+            }
+            return { ...prev, ...updates };
+          });
           break;
         }
-        case 'page_started': {
+        case 'page_started':
+        case 'page_retry_started': {
           const { page } = event.data as any;
           setPages(prev => {
             const idx = prev.findIndex(p => p.page === page);
@@ -294,6 +302,10 @@ const CloudPreview: React.FC = () => {
             pages_completed: data.pages_completed,
             pages_failed: data.pages_failed,
           } : null);
+          // When all pages completed (e.g. after retry), mark remaining pages as completed
+          if (data.pages_failed === 0) {
+            setPages(prev => prev.map(p => p.status !== 2 ? { ...p, status: 2, status_name: 'COMPLETED' } : p));
+          }
           break;
         }
         case 'error': {

src/renderer/pages/List.tsx

@@ -346,11 +346,18 @@ const List: React.FC = () => {
           case 'page_completed': {
             const pageNumber = (data as any).page;
             const totalPages = (data as any).total_pages || task.pages || 1;
-            // Use page number directly to avoid duplicate counting from replayed events
-            // page is 1-based, so completed_count = page number when pages complete in order
-            const completed = Math.max(task.completed_count || 0, pageNumber || 0);
-            task.completed_count = completed;
-            task.progress = Math.round((completed / totalPages) * 100);
+
+            if (task.status === 8) {
+              // Retry scenario (PARTIAL_FAILED): increment completed, decrement failed
+              task.completed_count = (task.completed_count || 0) + 1;
+              task.failed_count = Math.max(0, (task.failed_count || 0) - 1);
+            } else {
+              // Normal processing: use page number as progress indicator
+              // page is 1-based, so completed_count = page number when pages complete in order
+              task.completed_count = Math.max(task.completed_count || 0, pageNumber || 0);
+            }
+
+            task.progress = Math.round(((task.completed_count || 0) / totalPages) * 100);
             task.status = 3; // PROCESSING
             break;
           }