Skip to content

feat(zettel): add tenant-safe graph store - #18

Closed
adihex wants to merge 1 commit into
mainfrom
split/zettel-graph-store
Closed

adihex wants to merge 1 commit into
mainfrom
split/zettel-graph-store

Conversation

@adihex

@adihex adihex commented Aug 28, 2026

Copy link
Copy Markdown
Owner

Summary

Slice 1 replacing #10.

Dependencies

None.

Behavior

  • adds tenant-scoped entities and note-owned graph relations with safe foreign keys/cascades
  • atomically and idempotently replaces a note graph after enforcing note ownership
  • adds deterministic, tenant-isolated graph traversal with bounded depth and stable deduplication
  • preserves existing note editing behavior and migrates the unsafe legacy graph tables when present

Verification

  • mise exec -- pnpm --filter zettel run test — 5 files, 36 tests passed
  • mise exec -- pnpm build — passed (19 workspace tasks)
  • mise exec -- pnpm exec vp check apps/zettel/src/notes/store.ts apps/zettel/src/notes/store.test.ts — passed, 0 warnings/errors
  • git diff --check — passed

The repo-wide pnpm check remains blocked by 23 pre-existing formatting issues outside this slice.

Amp-Thread-ID: https://ampcode.com/threads/T-01a047e7-d402-72a8-ad22-eb5d3d3df6b1
Co-authored-by: Aditya Balakrishnan <adibalak1925@gmail.com>
@adihex
adihex force-pushed the split/zettel-graph-store branch from e77e2ec to 60a12ae Compare August 28, 2026 14:13

adihex commented Aug 28, 2026

Copy link
Copy Markdown
Owner Author

Landed on main as part of the consolidated, rebased stack ending at e9bd4bd. Verified this PR has no patches absent from origin/main, so closing the now-redundant stacked PR.

@adihex adihex closed this Aug 28, 2026
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