Skip to content

feat(zettel): add graph extraction model adapter - #23

Closed
adihex wants to merge 1 commit into
split/zettel-graph-extractionfrom
split/zettel-graph-model-adapter
Closed

adihex wants to merge 1 commit into
split/zettel-graph-extractionfrom
split/zettel-graph-model-adapter

Conversation

@adihex

@adihex adihex commented Aug 28, 2026

Copy link
Copy Markdown
Owner

Adds the concrete Groq structured-output adapter behind the graph extraction provider boundary.

  • reuses existing AI SDK/Groq dependencies
  • keeps tenant and note identifiers out of model prompts
  • validates structured output before the persistence boundary
  • keeps model/provider errors explicit

Stacked on #22. No lifecycle wiring, database, HTTP, UI, vector, or corpus changes.

Validation:

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

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-model-adapter branch from fe8f8ae to 32ecabb 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