Skip to content

fix(deps): resolve one DuckDB version across Relatr and the graph library - #15

Open
flox1an wants to merge 1 commit into
ContextVM:masterfrom
flox1an:upstream/duckdb-version
Open

flox1an wants to merge 1 commit into
ContextVM:masterfrom
flox1an:upstream/duckdb-version

Conversation

@flox1an

@flox1an flox1an commented Oct 12, 2026 •

Copy link
Copy Markdown

Why

Relatr and nostr-social-duck resolve different DuckDB versions. TypeScript rejects their shared connection, breaking typecheck at the graph integration point.

Change

Use Bun's existing dependency override to resolve one version; update the lockfile and changeset. No graph-policy change.

Verification

  • bun run lint and bun run build: pass.
  • bun test src/tests: 209 pass, 5 skip, 0 fail.
  • DuckDB type error removed. The pre-existing manager.ts:44 SDK authUser error remains.

Checked independently on Bun 1.3.14/macOS with a temporary SERVER_SECRET_KEY for the benchmark. Recommended first merge; other PRs do not require this for runtime behavior.

Related: #16, #17, #18, #19. All branches are independent; combined suite: 221 pass, 5 skip, 0 fail.

nostr-social-duck still depends on @duckdb/node-api ^1.4.4 while relatr
uses ^1.5.2, so both bindings were installed and the connection passed
to the graph library failed typechecking. Override it to one version.
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