chore(deps): update dependencies, better-sqlite3 to v13 - #96
Merged
Conversation
Patch/minor bumps: @assistant-ui/react-ai-sdk 1.4.5, @biomejs/biome 2.5.8, @tanstack/react-devtools 0.10.10, globby 16.2.3, mailparser 3.9.15, sonner 2.0.8, zustand 5.0.15. Major: better-sqlite3 12 -> 13 (N-API rewrite, prebuilt binaries shipped in package, prebuild-install dropped) plus @types/better-sqlite3 7 -> 9. The public API is unchanged; verified sqlite-vec 0.1.9 still loads as an extension and KNN queries work against the v13 build. Note: @prisma/adapter-better-sqlite3 7.9.1 hard-depends on better-sqlite3 ^12.6.0, so the tree now carries two native copies — v12 for Prisma, v13 for the embeddings sidecar (src/lib/embeddings/db.ts). Collapses once Prisma widens its range. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Contributor
Code Review SummaryStatus: No Issues Found | Recommendation: Merge Files Reviewed (2 files)
Review NotesThis PR updates multiple dependencies: Minor/Patch Updates:
Major Updates:
The major version bump for All verification steps (type-check, test, build, doctor) have passed as documented in the PR. Reviewed by kimi-k2.5 · Input: 163.8K · Output: 2K · Cached: 0 |
mikevalstar
added a commit
that referenced
this pull request
Aug 16, 2026
Resolves the package.json / pnpm-lock.yaml conflicts. This branch was cut before #94, #91, and #96 landed, so most of its 38 bumps are already in main and several of its targets are now *older* than main (nanoid 5 vs 6, typescript 6 vs 7, jsdom 29 vs 30, better-sqlite3 12 vs 13, biome 2.5.7 vs 2.5.8, zustand 5.0.14 vs 5.0.15, @tanstack/react-devtools 0.10.9 vs 0.10.10, @types/better-sqlite3 7 vs 9). Resolution takes main's package.json as the base and re-applies only the bumps where this branch is genuinely ahead: @ai-sdk/anthropic ^4.0.34 -> ^4.0.36 @ai-sdk/react ^4.0.59 -> ^4.0.61 @assistant-ui/react ^0.15.8 -> ^0.15.12 @assistant-ui/react-markdown ^0.14.9 -> ^0.14.10 @tanstack/router-plugin ^1.168.26 -> ^1.168.28 ai ^7.0.56 -> ^7.0.58 exa-js ^2.16.3 -> ^2.17.0 imapflow ^1.6.5 -> ^1.6.6 lucide-react ^1.30.0 -> ^1.31.0 @types/node ^26.1.2 -> ^26.2.0 tsx ^4.23.10 -> ^4.23.11 Lockfile regenerated from main's rather than hand-merged. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Tracked in
AETH-updmqqig.Minor / patch
@assistant-ui/react-ai-sdk@biomejs/biome(dev)@tanstack/react-devtoolsglobbymailparsersonnerzustandMajor
better-sqlite3@types/better-sqlite3(dev)better-sqlite313.0.0 is an N-API rewrite: prebuilt binaries now ship inside the package,prebuild-installis gone, and Node >=22 is required (we're on 24). The public API is unchanged — only additions (db.explain(),statement.toString()).The one real risk was
sqlite-vec, which loads a native extension into thebetter-sqlite3handle insrc/lib/embeddings/db.ts. Smoke-tested against the v13 build: extension loads (vec_version()→v0.1.9),vec0virtual table creates, and a KNNMATCH ... AND k = 1query returns correctly.Known tradeoff
@prisma/adapter-better-sqlite3@7.9.1hasbetter-sqlite3: ^12.6.0as a hard dependency (not a peer), so the tree now carries two native copies — v12 under the adapter for the Prisma-managed main DB, v13 at the root for the embeddings sidecar. Both build and run fine, but install is heavier. This collapses on its own once Prisma widens the range; worth revertingbetter-sqlite3to^12instead if the duplicate build is unwelcome.Verification
pnpm type-check— cleanpnpm test— 59 passed (8 files)pnpm check— exit 0 (2 pre-existing warnings, unchanged)pnpm build— succeedspnpm debug doctor— DB connects through the Prisma adapter, 2 users read🤖 Generated with Claude Code