Skip to content

fix(cache): report prune removal failures accurately - #1285

Open
cedric013 wants to merge 1 commit into
yvgude:mainfrom
cedric013:fix/1281-prune-cache-metrics
Open

fix(cache): report prune removal failures accurately#1285
cedric013 wants to merge 1 commit into
yvgude:mainfrom
cedric013:fix/1281-prune-cache-metrics

Conversation

@cedric013

Copy link
Copy Markdown
Contributor

Summary

  • Track failed BM25/graph cache prune removals explicitly.
  • Count removed entries and freed bytes only after filesystem deletion succeeds.
  • Surface failed removals in cache prune and doctor output.
  • Add regression coverage for failed BM25 directory removal.

Closes #1281

Tests

  • rustfmt --edition 2024 --check src/cli/config_cmd.rs src/doctor/fix.rs tests/index_scoping_scenarios.rs
  • cargo test -q --test index_scoping_scenarios prune_bm25_counts_failed_remove_without_freed_bytes (cancelled after hanging in this workspace; CI will run the full suite)

@cedric013
cedric013 force-pushed the fix/1281-prune-cache-metrics branch from 4f887ce to 0d13b0b Compare July 27, 2026 12:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix cache prune metrics on failed removals

1 participant