chore(deps): upgrade TypeScript 7, nanoid 6, jsdom 30, @types/node 26 - #91
Conversation
Code Review SummaryStatus: No Issues Found | Recommendation: Merge Files Reviewed (3 files)
Incremental Review NotesChanges since commit
All changes align with the PR description - verified with Previous Review Summary (commit b75346a)Current summary above is authoritative. Previous snapshots are kept for context only. Previous review (commit b75346a)Status: No Issues Found | Recommendation: Merge Files Reviewed (3 files)
Review NotesThe only source code change is correct and minimal:
This properly accommodates the node-ical 0.27 breaking change where the
All changes align with the PR description - routine dependency upgrades with verified type-check, tests, and build passing. Reviewed by kimi-k2.5 · Input: 69.4K · Output: 1.8K · Cached: 270.8K |
The rest of the original sweep (assistant-ui 0.15, AI SDK, srvx, and the minor/patch batch) landed via #94, and the node-ical VEvent typing fix went with it. What remains here are the four majors. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
b75346a to
e6c0ddc
Compare
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>
Rebased onto main. The bulk of the original sweep — the minor/patch batch, assistant-ui 0.15, the AI SDK bumps, and the
node-ical0.27VEventtyping fix — landed via #94, so this branch is now just the four remaining majors, in one commit.Verified with
pnpm type-check,pnpm test(59 tests),pnpm check, andpnpm build.Upgraded
nanoid()named import.Held back — blocked upstream
better-sqlite3 13 —
@prisma/adapter-better-sqlite3hard-depsbetter-sqlite3@^12.6.0. Bumping ours installs a second copy, so Prisma's connection and the sqlite-vec embeddings sidecar (src/lib/embeddings/db.ts) would each load a different native SQLite build in the same process. This was tested before backing it out: sqlite-vec loads fine under 13.0.2 and the embeddings tests pass — it works, it's just not worth a duplicate native module. Revisit when Prisma bumps its range.Note on srvx
The previous version of this PR held
srvxat 0.11.22 because@tanstack/start-plugin-coreandh3both pin^0.11.x. Dependabot's #94 bumped it to^0.12.5anyway, so main now carries both copies. In practice the two only meet atserver.ts:2, wheresrvx/node'stoNodeHandlerhands a standardRequestto the TanStack-built handler — a Web-standard boundary, and a production server boot serves/loginfine. Flagging it as a known duplicate rather than a blocker.Still needs a human pass
assistant-ui 0.15 is not runtime-verified — it came in via #94 rather than this branch, but nobody has clicked through
/chatin a browser. A 0.x minor is a breaking release for this library. Worth exercising streaming, message editing, branch navigation, and sub-agent rendering.Incidental observation
jsdomis in devDependencies but referenced nowhere outsidepackage.json, and no vitest config selects that environment. Upgraded rather than removed, since that's outside this PR's scope — but it looks like apnpm knipcandidate.🤖 Generated with Claude Code