Skip to content

fix(sidecar): use canonical mesh document store - #231

Merged
kitplummer merged 1 commit into
mainfrom
fix/1004-canonical-document-store
Aug 6, 2026
Merged

fix(sidecar): use canonical mesh document store#231
kitplummer merged 1 commit into
mainfrom
fix/1004-canonical-document-store

Conversation

@kitplummer

Copy link
Copy Markdown
Collaborator

Summary

  • route sidecar document CRUD through peat-mesh's canonical DocumentStore contract instead of bypassing it through raw AutomergeStore calls
  • retain encrypted-at-rest persistence and the existing change-event lifecycle
  • elect exactly one mDNS dial initiator by endpoint ID so reciprocal discovery cannot replace the connection carrying the initial document sweep

This is the peat-node slice of defenseunicorns/peat#1004. The shared-endpoint prerequisite is defenseunicorns/peat-mesh#375.

Validation

  • cargo fmt --check
  • cargo clippy --workspace --all-targets -- -D warnings
  • cargo test --workspace — 526 passed, 18 ignored
  • cargo test --test node_test
  • cargo test --test cross_peer_encryption_test -- --nocapture
  • ./test/cross-cluster-sync.sh — bidirectional two-cluster convergence and CLI workflow passed; clusters cleaned up
  • physical Samsung SM-X210 Android 16 ↔ ARM64 Linux validation: direct mDNS peer connection, peer count 1, and Automerge convergence in both directions

Relates to defenseunicorns/peat#1004.

@peat-bot peat-bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Peat QA Review (SHA: 890149c)

Linked Issues

No findings.

Validation run locally:

  • cargo test --test node_test -- --nocapture passed: 25 passed.
  • cargo test --test grpc_test -- --nocapture passed: 7 passed.

@kitplummer
kitplummer merged commit 86577dc into main Aug 6, 2026
16 checks passed
@kitplummer
kitplummer deleted the fix/1004-canonical-document-store branch August 6, 2026 15:37
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.

2 participants