E1 — Auth & Accounts#114
Conversation
Wire better-auth socialProviders gated on env vars and surface "Continue with Google/GitHub" buttons on Login + Signup. Providers are skipped cleanly when credentials are absent so local dev and tests don't need real OAuth apps. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
|
Warning You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again! |
|
ECC bundle files are already tracked in this repository. Skipping generation of another bundle PR. |
|
Warning Rate limit exceeded
You’ve run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (6)
✨ Finishing Touches🧪 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. Comment |
|
Copilot review errored out with no actionable feedback. Re-requested review. CI is green (lint/type-check/test/build SUCCESS) and CodeRabbit passed. |
Closes ALO-120.
Summary
E1 was already largely shipped — email verification (ALO-128), password reset (ALO-129), profile (avatar/banner in R2), account settings + GDPR delete (ALO-132), and session management UI (ALO-133) all landed in earlier PRs. This PR closes the last in-scope gap: OAuth providers.
socialProvidersfor Google + GitHub, gated on env vars (GOOGLE_CLIENT_ID/SECRET,GITHUB_CLIENT_ID/SECRET). Absent vars cleanly skip the provider so local dev / tests don't need real OAuth apps.<OAuthButtons />component and surface it on Login + Signup. UsessignIn.social({ provider, callbackURL })so the redirect dance is handled by better-auth.wrangler.toml.socialProvidersis omitted when creds absent and wired correctly when both pairs are present.Done-when checklist
Test plan
npm test— 44 files / 493 tests passnpm run lint— 0 warnings, AI Gateway guard cleannpm run type-check— cleanGOOGLE_*/GITHUB_*secrets, click "Continue with…" buttons, verify session round-trip🤖 Generated with Claude Code