Indexing: Fix lost scan metadata and stuck status

- Reset `expected_total_entries` to 0 after `ComputeAllAggregates` in the writer. Without this, micro-scan `InsertEntriesV2` batches emitted spurious `saving_entries` progress events that re-triggered the aggregation overlay permanently (no matching `index-aggregation-complete` followed).
- Move `scan_completed_at` and other meta writes before the reconciler replay in `mod.rs`. The replay can fail (e.g. "database is locked") and cause an early return, which previously skipped persisting `scan_completed_at` — causing a full rescan on every subsequent startup.

Author: vdavid

apps/desktop/src-tauri/src/indexing/mod.rs

@@ -608,6 +608,31 @@ impl IndexManager {
                         },
                     );
 
+                    // Store scan metadata now, before the reconciler replay which
+                    // can fail (e.g. "database is locked") and cause an early return.
+                    // Without this, scan_completed_at is never persisted and the next
+                    // startup triggers a full rescan of the entire volume.
+                    let now = std::time::SystemTime::now()
+                        .duration_since(std::time::UNIX_EPOCH)
+                        .map(|d| d.as_secs().to_string())
+                        .unwrap_or_default();
+                    let _ = writer.send(WriteMessage::UpdateMeta {
+                        key: "scan_completed_at".to_string(),
+                        value: now,
+                    });
+                    let _ = writer.send(WriteMessage::UpdateMeta {
+                        key: "scan_duration_ms".to_string(),
+                        value: summary.duration_ms.to_string(),
+                    });
+                    let _ = writer.send(WriteMessage::UpdateMeta {
+                        key: "total_entries".to_string(),
+                        value: summary.total_entries.to_string(),
+                    });
+                    let _ = writer.send(WriteMessage::UpdateMeta {
+                        key: "volume_path".to_string(),
+                        value: "/".to_string(),
+                    });
+
                     // Open a read connection for path resolution during replay
                     let replay_conn = match IndexStore::open_write_connection(&writer.db_path()) {
                         Ok(c) => c,
@@ -665,28 +690,6 @@ impl IndexManager {
                         *guard = Some(handle);
                     }
 
-                    // Store scan metadata via writer
-                    let now = std::time::SystemTime::now()
-                        .duration_since(std::time::UNIX_EPOCH)
-                        .map(|d| d.as_secs().to_string())
-                        .unwrap_or_default();
-                    let _ = writer.send(WriteMessage::UpdateMeta {
-                        key: "scan_completed_at".to_string(),
-                        value: now,
-                    });
-                    let _ = writer.send(WriteMessage::UpdateMeta {
-                        key: "scan_duration_ms".to_string(),
-                        value: summary.duration_ms.to_string(),
-                    });
-                    let _ = writer.send(WriteMessage::UpdateMeta {
-                        key: "total_entries".to_string(),
-                        value: summary.total_entries.to_string(),
-                    });
-                    let _ = writer.send(WriteMessage::UpdateMeta {
-                        key: "volume_path".to_string(),
-                        value: "/".to_string(),
-                    });
-
                     // Mark micro-scans as superseded (full scan data is authoritative)
                     micro_scans.mark_full_scan_complete().await;
                 }

apps/desktop/src-tauri/src/indexing/writer.rs

@@ -632,8 +632,11 @@ fn process_message(
                     &mut on_progress,
                 )
             };
-            // Maps are consumed; clear to free memory
+            // Maps are consumed; clear to free memory.
+            // Reset expected_total so micro-scan inserts don't emit
+            // spurious saving_entries progress events after the full scan.
             accumulator.clear();
+            expected_total_entries.store(0, Ordering::Relaxed);
             match result {
                 Ok(count) => {
                     log::info!(