Skip to content

feat(zettel): add graph extraction boundary - #22

Closed
adihex wants to merge 1 commit into
split/zettel-graph-toolfrom
split/zettel-graph-extraction
Closed

adihex wants to merge 1 commit into
split/zettel-graph-toolfrom
split/zettel-graph-extraction

Conversation

@adihex

@adihex adihex commented Aug 28, 2026

Copy link
Copy Markdown
Owner

Summary

  • add an injected, model-independent provider boundary for note graph extraction
  • validate and normalize structured model output with Zod, including endpoint integrity
  • persist through replaceNoteGraph only after successful extraction

Tests

  • mise exec -- pnpm exec vitest run apps/zettel/src/notes/graph-extraction.test.ts
  • mise exec -- pnpm exec vitest run apps/zettel/src
  • mise exec -- pnpm build

Stacked replacement for the graph extraction portion of #10. This PR intentionally does not wire note lifecycle or add HTTP/tool/UI/vector behavior.

Co-authored-by: Aditya Balakrishnan <adibalak1925@gmail.com>
@adihex
adihex force-pushed the split/zettel-graph-extraction branch from 9edeb93 to 6130edf Compare August 28, 2026 14:14

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