Skip to content

fix(pdf): bind source locators to one snapshot - #117

Merged
McanKul merged 1 commit into
developmentfrom
fix/33-source-snapshot-identity
Oct 4, 2026
Merged

McanKul merged 1 commit into
developmentfrom
fix/33-source-snapshot-identity

Conversation

@McanKul

@McanKul McanKul commented Oct 4, 2026

Copy link
Copy Markdown
Owner

Summary

  • Read the classifier source once through the existing 400 MB hard cap.
  • Hash and parse the exact same in-memory bytes, removing the path reread race.
  • Replace the 64-bit FNV locator identity with a full SHA-256 fingerprint and a strict v2 locator format.
  • Resolve a locator against the same parsed snapshot instead of reopening the source.

This remains a read-only research path. It adds no Tauri command, UI, mutation, save, or release change.

Refs #33

Validation

  • npm run build
  • cargo check --manifest-path src-tauri/Cargo.toml
  • cargo test --manifest-path src-tauri/Cargo.toml --lib source_content -- --nocapture (36 passed)
  • git diff --check

Privacy checklist

  • Files remain local and are never uploaded.
  • The source file is not modified.
  • Malformed and legacy locators fail closed as STALE.

@McanKul
McanKul merged commit 5e2f1e5 into development Oct 4, 2026
2 checks passed
@McanKul
McanKul deleted the fix/33-source-snapshot-identity branch October 4, 2026 11:10
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.

1 participant