logto-io / js

🤓 Logto JS SDKs.
https://docs.logto.io/quick-starts/
MIT License
62 stars 39 forks source link

bug: replace CommonJS 'camelcase-keys' dependency #618

Open David-van-der-Sluijs opened 8 months ago

David-van-der-Sluijs commented 8 months ago

When using this library inside an angular application, the compiler warns about a CommonJS dependency used here:

Warning: node_modules/@logto/js/lib/core/fetch-token.js depends on 'camelcase-keys'. CommonJS or AMD dependencies can cause optimization bailouts.
For more info see: https://angular.io/guide/build#configuring-commonjs-dependencies

I think that is a rather simple library that could be either replaced or the functionality written from scratch. If I have the time, I can look into it further and suggest something / write something.

xiaoyijun commented 8 months ago

Tracked in LOG-8216.

gao-sun commented 6 months ago

it's 2024, we can update our SDKs to ESM-only

charIeszhao commented 2 months ago

How about this one? change-case @gao-sun