fix(auth): pin Better Auth to 1.6.25 - #1062
Conversation
Pin the full Better Auth dependency family to the schema-compatible version and add a manifest regression test for fresh package consumers.\n\nFixes #1 Signed-off-by: 서재원 <cinos81@gmail.com>
This reverts commit 96598ba.
|
Confirming this fix targets a live, repo-wide CI problem — with a reproduction and the concrete mechanism. Root cause, reproduced. The committed lockfile currently pins the family to 1.6.22, so local Why 1.7 breaks auth specifically. better-auth 1.7.0 ships breaking schema changes — most relevantly "Accounts now require Observed impact. On the shared PR preview lane this shows up as a consistent block of Pinning the whole family to an exact, schema-compatible (Reproduced locally; I did not run the auth E2E under 1.7 — that path needs the preview deploy — but the resolution float and the 1.7 schema break are both verified above.) |
Description
Prevents fresh
@sonicjs-cms/coreconsumers from resolving Better Auth 1.7.x before SonicJS has migrated itsauth_accountschema to the new issuer-scoped account identity model.Fixes #1061
Changes
better-auth,@better-auth/core,@better-auth/drizzle-adapter, and@better-auth/telemetryto the repository-tested1.6.25release..worktrees/directory used for isolated contribution work.@better-auth/coreis intentionally direct: a packed-consumer check showed that pinning only the other three packages still allowed peer dependency resolution to install@better-auth/core@1.7.1alongside 1.6.25.This urgent patch does not change the database schema or runtime auth code. A coordinated Better Auth 1.7 issuer migration remains follow-up work.
Testing
Unit Tests
better-auth-dependency-compat.test.tsPackage Consumer Check
packages/corewithnpm pack--package-lock-only1.6.25:better-auth@better-auth/core@better-auth/drizzle-adapter@better-auth/telemetryOther Verification
npm run lint --workspace=@sonicjs-cms/core(passes with existing warnings)npm run type-checknpm run build:coregit diff --checkKnown Repository Baseline Issue
npm ci --ignore-scriptscurrently fails before installation because the checked-in root lockfile already drifts from unrelated workspace manifests (esbuild,eslint, andsharp). The repository CI usesnpm install, which succeeds. This PR deliberately avoids including the resulting 1,500+ lines of unrelated lockfile churn.Screenshots/Videos
Not applicable; no UI changes.
Checklist