Upgrade docs Better Auth and Polar to API 2026-10 - #3149
nperez0111 wants to merge 2 commits into
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (3)
Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 9 remain after this review. 📝 WalkthroughWalkthroughThe docs app updates Better Auth and Polar SDK dependencies and initialization. It also changes webhook product and customer field access, user creation options, customer linking, and the database migration command. ChangesBetter Auth and Polar upgrade
Priority: ➖ Normal Estimated code review effort: 3 (Moderate) | ~20 minutes Change: Other Suggested reviewers: Merge Risk: ⚪ Minimal · up to The dependency and webhook upgrade has no identified merge-blocking defect. Complete the planned deployment checks during rollout. Architecture SummaryArchitecture risk: 🔵 Low · up to The change affects 1 system. Changed systems: Architecture concerns Review detailsSystems and components
Before / after behavior
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. A rabbit checks the auth-client door, Comment |
@blocknote/ariakit
@blocknote/code-block
@blocknote/core
@blocknote/diagram-block
@blocknote/mantine
@blocknote/math-block
@blocknote/react
@blocknote/server-util
@blocknote/shadcn
@blocknote/xl-ai
@blocknote/xl-docx-exporter
@blocknote/xl-email-exporter
@blocknote/xl-multi-column
@blocknote/xl-odt-exporter
@blocknote/xl-pdf-exporter
@blocknote/xl-typst-exporter
commit: |
|
Summary
@polar-sh/better-auth2.0.0, and Polar SDK 1.0.1; pin the matchingauthmigration CLI.Related: #2749 (dependency upgrade), #2870 (pay-first checkout).
Validation
git diff --check, mocked auth/billing smoke checks, and offline frozen-lockfile installation passed.subscription.activepayload twice: both requests returned 200 with no duplicate user. This was not a dashboard replay of the original event.publicschema into private local Postgres. Better Authgenerateandmigratereported no migrations needed. All six tables' row counts and data hashes remained unchanged; no duplicate provider account keys were present. This was a schema rehearsal, not a runtime login test against restored records.verification.createdAt/updatedAtcolumns are nullable; neither contained nulls. That drift was not silently altered.Production was accessed read-only for the copy; no production migrations or webhook changes were performed. The copied database, local SQLite data, and temporary Postgres instance were removed. Credentials, database dumps, and local dev proxy settings are not included in this PR. Smoke checks were temporary tooling, not new committed regression tests.
Production rollout (required, not performed by this PR)
(providerId, accountId)keys and review CLI generation against the actual targetPOSTGRES_URL. Fromdocs/, runpnpm exec auth generate --config ./lib/auth.ts --output ./auth-migration.sql, then apply reviewed changes withpnpm exec auth migrate --config ./lib/auth.ts(the tested copy required none). WithoutPOSTGRES_URL, development targets local SQLite instead./api/auth/polar/webhooksendpoint to API 2026-10; queued/redelivered events retain their original contract.Rollback must coordinate the application, database state, and Polar webhook contract; reverting dependencies alone is insufficient.
Summary by CodeRabbit