fix(nextjs): Update imports to avoid Node.js build warnings on edge runtime - Githubissues

clerk / javascript

Official Javascript repository for Clerk authentication

https://clerk.com

MIT License

1.1k stars 243 forks source link

fix(nextjs): Update imports to avoid Node.js build warnings on edge runtime #3661

Closed LauraBeatris closed 3 months ago

LauraBeatris commented 3 months ago

Description

Fixes https://github.com/clerk/javascript/issues/3660

Uses AsyncLocalStorage available as a global polyfill for the edge runtime, instead of importing from Node.js: https://nextjs.org/docs/pages/api-reference/edge#nextjs-specific-polyfills
Imports logger from nested import

Checklist

[X] npm test runs as expected.
[X] npm run build runs as expected.
[ ] (If applicable) JSDoc comments have been added or updated for any package exports
[ ] (If applicable) Documentation has been updated

Type of change

[X] 🐛 Bug fix
[ ] 🌟 New feature
[ ] 🔨 Breaking change
[ ] 📖 Refactoring / dependency upgrade / documentation
[ ] other:

changeset-bot[bot] commented 3 months ago

🦋 Changeset detected

Latest commit: 00b744726339997f60174a71e621b51cb24b7f4b

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

| Name | Type | | ------------- | ----- | | @clerk/nextjs | Patch |

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR