Skip to content

Suppress hydration-only reader reindexes - #132

Merged
Shengyu Fu (shengyfu) merged 2 commits into
mainfrom
shengyfu-hydration-content-identities
Sep 3, 2026
Merged

Suppress hydration-only reader reindexes#132
Shengyu Fu (shengyfu) merged 2 commits into
mainfrom
shengyfu-hydration-content-identities

Conversation

@shengyfu

@shengyfu Shengyu Fu (shengyfu) commented Sep 3, 2026

Copy link
Copy Markdown
Member

Summary

  • persist a domain-versioned BLAKE3-128 identity for the exact decoded bytes used to build each text file's postings, as optional compact c evidence in filestamps.json
  • suppress only forced reader-backed events whose freshly decoded identity still matches trusted evidence, while preserving byte/version verification, cache invalidation, stamp refresh, filename membership repair, and all live overlay/tombstone behavior
  • propagate identities through full, delta, stale, live, and checkpointed background builds, and make direct, staged, rollback, and empty-index publication generation-safe
  • preserve the public FileStamp { mtime, size } API/equality and legacy JSON compatibility; missing or malformed identities fail open

Stack

#130 is merged. This PR now targets main and remains the upper member of stack #133. It leaves #130's exact raw-byte + overlay-ID duplicate suppression unchanged.

Validation

  • mutation control: disabling content-ID matching makes the reader-backed suppression test fail with a new live overlay and reindex: modified
  • cargo fmt --all -- --check
  • cargo clippy --workspace --all-targets -- -D warnings
  • cargo test --workspace
  • cargo build --release --workspace

Fixes #131

Base automatically changed from shengyfu-deduplicate-watcher-saves to main September 3, 2026 16:24
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@shengyfu
Shengyu Fu (shengyfu) force-pushed the shengyfu-hydration-content-identities branch from 951349f to 3c38233 Compare September 3, 2026 16:30
@shengyfu
Shengyu Fu (shengyfu) merged commit c46efb5 into main Sep 3, 2026
9 checks passed
@shengyfu
Shengyu Fu (shengyfu) deleted the shengyfu-hydration-content-identities branch September 3, 2026 16:42
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.

Avoid watcher reindex churn from virtualized file hydration

1 participant