Repository navigation
Release 1.3.1: clear dependency security advisories - #72
Merged
Merged
Conversation
pnpm audit reported 54 advisories (3 critical, 19 high). 53 are cleared: - better-auth and plugins 1.7.2 -> 1.7.14, @modelcontextprotocol/sdk -> 1.32.2, solid-js -> 1.9.17, vitest -> 4.1.12, zod 3 -> 4.6.7 - pnpm overrides for transitive fixes (hono, @hono/node-server, fast-uri, qs, brace-expansion, seroval, proxy-addr, body-parser, ip-address, vite, follow-redirects, node-forge, source-map-js, serialize-javascript, js-yaml) The low-severity deepmerge advisory has no fixed release. Better Auth 1.7.3+ dropped the account.issuer column that 1.7.0-1.7.2 required and refuses to boot while it is present, which would have broken every deployment created on NoteMesh 1.3.0. The server now drops the column and its index before migrating. Reproduced against a database written by 1.7.2; tests cover the 1.7.2, backfilled-1.6 and 1.6 shapes. Better Auth also now validates the schema on every API call, so the RPC gate runs auth migrations before getSession, as the other entry points do. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01CgKUWtS6SvWsfLoeX4snXa
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01CgKUWtS6SvWsfLoeX4snXa
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01CgKUWtS6SvWsfLoeX4snXa
wiverson
marked this pull request as ready for review
October 29, 2026 21:24
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.
Summary
Clears the Dependabot /
pnpm auditfindings.pnpm auditwent from 54 advisories (3 critical, 19 high) to 1 (low).Direct bumps: better-auth + plugins 1.7.2 → 1.7.14,
@modelcontextprotocol/sdk→ 1.32.2, solid-js → 1.9.17, vitest → 4.1.12, zod 3 → 4.6.7.Overrides (
pnpm-workspace.yaml, each with its reason): hono, @hono/node-server, fast-uri, qs, brace-expansion, seroval, proxy-addr, body-parser, ip-address, vite, follow-redirects, node-forge, source-map-js, serialize-javascript, js-yaml@4.Remaining:
deepmerge(low, build-time only, via nitropack). There is no fixed release yet.Two fixes the upgrade required
account.issuercolumn. Better Auth 1.7.0–1.7.2 required this column. Later releases removed it and refuse to start while a NOT NULL column they never write is present. I created a database on the currentmain(1.7.2) and booted it on 1.7.14: it threwDatabase schema mismatchat boot. So every deployment set up on 1.3.0 would have stopped working after a redeploy.dropAccountIssuer()replaces the old 1.6 backfill. It drops the column and its unique index before the migrator runs. The value was alwayslocal:credential.auth-upgrade.test.tscovers the 1.7.2 shape, the backfilled 1.6 → 1.3.0 shape and the plain 1.6 shape. Without the fix, 2 of the 3 tests fail./api/rpc/[fn]calledgetSessionbeforerunAuthMigrations, so on a new instance the first request returned 500 instead of 401 (caught bytests/http/security.test.ts). The gate now runs migrations first, as the other entry points do.Testing
pnpm typecheck,pnpm lint,pnpm build: cleanpnpm test: 1100/1100Generated by Claude Code