Skip to content

feat: harden DataStore ownership lock - #830

Merged
enricopiovesan merged 3 commits into
mainfrom
codex/issue-825-datastore-ownership-lock
Jul 28, 2026
Merged

feat: harden DataStore ownership lock#830
enricopiovesan merged 3 commits into
mainfrom
codex/issue-825-datastore-ownership-lock

Conversation

@enricopiovesan

Copy link
Copy Markdown
Collaborator

Summary

  • harden local DataStore lock failures so they never expose host roots
  • document the standard-library lock lifecycle and deterministic unsupported-platform behavior
  • prove cross-process contention, clean owner release, and owner-crash recovery

Governing Spec

  • 518-durable-local-datastore
  • 519-embedder-owned-datastore-integration

Project Item

Definition of Done

  • Stable store_locked details never disclose lock paths or roots.
  • Cross-process contention preserves committed state and reports store_locked.
  • Owner release and process-termination recovery permit a later reopen.
  • Unsupported locking fails with a deterministic typed error.
  • No lease protocol or multi-process write policy was added.

Validation

  • cargo clippy -p traverse-runtime --lib -- -D warnings
  • cargo test -p traverse-runtime data_store --lib
  • cargo test --workspace
  • bash scripts/ci/spec_alignment_check.sh /private/tmp/traverse-pr-825.md
  • cargo fmt --check
  • git diff --check

@enricopiovesan
enricopiovesan enabled auto-merge (squash) July 28, 2026 03:35
@enricopiovesan
enricopiovesan merged commit 5bcc1f5 into main Jul 28, 2026
32 checks passed
@enricopiovesan
enricopiovesan deleted the codex/issue-825-datastore-ownership-lock branch July 28, 2026 03:39
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