-
Notifications
You must be signed in to change notification settings - Fork 83
Description
This issue tracks some of the dependencies that are at least 1 major behind but cannot update b/c they don't have CJS builds (which breaks supported CJS CLIs):
The following deps come (mostly?) fromsindresorhus and have switched to ESM-only builds in a major:
is-wslusing v2, latest is v3widest-lineusing v3, latest is v5string-widthusing v4, latest is v7supports-colorusing v8, latest is 10
As long as we keep supporting CJS we should either replace usage of these deps or wait until node stabilizes requireing ESM from CJS (currently in RC state):
https://nodejs.org/api/modules.html#loading-ecmascript-modules-using-require
One of the requirements for this to work is that the ESM modules should not have TLA, is-wsl doesn't:
https://github.com/sindresorhus/is-wsl/blob/main/index.js
If all of the listed deps don't have TLA in their exports, the feature is GA and supported in our minimum required node version in oclif then we should give it a try and bump everything.
related:
#1234