julienfroidefond/stripstream-librarian
1
0
Fork 0
Code Issues 4 Pull Requests Actions Packages Projects Releases Wiki Activity

fix(indexer): Book deletion not working due to path mismatch

Browse Source 
The existing HashMap stored local paths but seen HashMap stored
/libraries paths. This caused the deletion logic to never find
matching files. Now both use consistent local path format.
This commit is contained in:
Julien Froidefond
2026-03-06 14:28:57 +01:00
parent b6cd8a895d
commit 762587dcb3
1 changed files with 6 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
apps/indexer/src/main.rs
View File

@@ -562,7 +562,9 @@ async fn scan_library(
})?; })?;
info!("[BDD] Progress update took {:?}", db_start.elapsed()); info!("[BDD] Progress update took {:?}", db_start.elapsed());
seen.insert(abs_path.clone(), true); // Use local path for seen tracking to match existing keys
let seen_key = remap_libraries_path(&abs_path);
seen.insert(seen_key, true);
let meta_start = std::time::Instant::now(); let meta_start = std::time::Instant::now();
let metadata = std::fs::metadata(path) let metadata = std::fs::metadata(path)
@@ -574,7 +576,9 @@ async fn scan_library(
let fingerprint = compute_fingerprint(path, metadata.len(), &mtime)?; let fingerprint = compute_fingerprint(path, metadata.len(), &mtime)?;
info!("[META] Metadata+fingerprint took {:?}", meta_start.elapsed()); info!("[META] Metadata+fingerprint took {:?}", meta_start.elapsed());
if let Some((file_id, book_id, old_fingerprint)) = existing.get(&abs_path).cloned() { // Use local path to lookup in existing (which has local paths as keys)
let lookup_path = remap_libraries_path(&abs_path);
if let Some((file_id, book_id, old_fingerprint)) = existing.get(&lookup_path).cloned() {
// Skip fingerprint check for full rebuilds - always reindex // Skip fingerprint check for full rebuilds - always reindex
if !is_full_rebuild && old_fingerprint == fingerprint { if !is_full_rebuild && old_fingerprint == fingerprint {
info!("[SKIP] File unchanged, skipping: {} (total time: {:?})", file_name, start_time.elapsed()); info!("[SKIP] File unchanged, skipping: {} (total time: {:?})", file_name, start_time.elapsed());