[PR #5994] fix(sso): prevent server instance from leaking to client #14616

Closed
opened 2026-04-13 09:32:36 -05:00 by GiteaMirror · 0 comments
Owner

Original Pull Request: https://github.com/better-auth/better-auth/pull/5994

State: closed
Merged: Yes


When importing the SSO client from the latest beta release, it crashed my app due to missing node dependencies in the browser. I realized that the SSO client is importing the server plugin without a type-only import. This aligns the SSO client plugin with the other client plugins.


Summary by cubic

Prevented server libraries from leaking into the SSO client build by switching the sso import to a type-only import. This avoids browser crashes caused by Node-only dependencies and aligns the SSO client with other client plugins.

Written for commit 74c75f50c1. Summary will update automatically on new commits.

**Original Pull Request:** https://github.com/better-auth/better-auth/pull/5994 **State:** closed **Merged:** Yes --- When importing the SSO client from the latest beta release, it crashed my app due to missing node dependencies in the browser. I realized that the SSO client is importing the server plugin without a type-only import. This aligns the SSO client plugin with the other client plugins. <!-- This is an auto-generated description by cubic. --> --- ## Summary by cubic Prevented server libraries from leaking into the SSO client build by switching the sso import to a type-only import. This avoids browser crashes caused by Node-only dependencies and aligns the SSO client with other client plugins. <sup>Written for commit 74c75f50c102570a974b7a540895ec673086ef21. Summary will update automatically on new commits.</sup> <!-- End of auto-generated description by cubic. -->
GiteaMirror added the pull-request label 2026-04-13 09:32:36 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/better-auth#14616