fix: use ESM for cli

[JounQin]

Expected Behavior

wrap-ansi required by cliui by yargs is ESM only now

Current Behavior

$ /Users/JounQin/Workspaces/Alauda/ui/node_modules/.bin/commitlint -e
/Users/JounQin/Workspaces/Alauda/ui/node_modules/cliui/build/index.cjs:293
const wrap = require('wrap-ansi');
             ^

Error [ERR_REQUIRE_ESM]: require() of ES Module /Users/JounQin/Workspaces/Alauda/ui/node_modules/wrap-ansi/index.js from /Users/JounQin/Workspaces/Alauda/ui/node_modules/cliui/build/index.cjs not supported.
Instead change the require of index.js in /Users/JounQin/Workspaces/Alauda/ui/node_modules/cliui/build/index.cjs to a dynamic import() which is available in all CommonJS modules.
    at Object.<anonymous> (/Users/JounQin/Workspaces/Alauda/ui/node_modules/cliui/build/index.cjs:293:14)
    at Object.<anonymous> (/Users/JounQin/Workspaces/Alauda/ui/node_modules/yargs/build/index.cjs:1:60678)
    at Object.<anonymous> (/Users/JounQin/Workspaces/Alauda/ui/node_modules/yargs/index.cjs:5:30)
    at Object.<anonymous> (/Users/JounQin/Workspaces/Alauda/ui/node_modules/@commitlint/cli/lib/cli.js:20:33)
    at Object.<anonymous> (/Users/JounQin/Workspaces/Alauda/ui/node_modules/@commitlint/cli/cli.js:2:1) {
  code: 'ERR_REQUIRE_ESM'
}

Node.js v18.17.1

Affected packages

• cli
• core
• prompt
• config-angular

Possible Solution

Use ESM for cli

related yargs/cliui#139

Steps to Reproduce

N/A

Context

No response

commitlint --version

@commitlint/[email protected]

git --version

2.41.0

node --version

v18.17.1

[escapedcat]

How to repoduce this? You got a repo?

[JounQin]

It seems a package manager issue which resolves wrap-ansi@8 in node_modules/wrap-ansi but it should be node_modules/cliui/node_modules/wrap-ansi actually. Close for now.

[lstoeferle]

Hey @JounQin,
I'm getting the same error in one of my ESM projects using yarn. Do you have any idea how to fix this?

[sankarge]

The following did the trick.

rm -rf node_modules
rm yarn.lock
yarn install

[JounQin]

Hey @JounQin, I'm getting the same error in one of my ESM projects using yarn. Do you have any idea how to fix this?

Sorry, I'm late. I was using patch-package to do the trick.

[escapedcat]

Could this be solved by commitlint with a resolution?

[JounQin]

Could this be solved by commitlint with a resolution?

Maybe as I said, migrate cli to ESM.

[escapedcat]

Maybe as I said, migrate cli to ESM.

Currently no time or motivation. Happy for any help.

[escapedcat]

Could this be solved by commitlint with a resolution?

Happy for a PR here

[JounQin]

OK, I'll raise a PR then.

[JounQin]

@escapedcat Please help to review #3850, I've fixed almost all test cases except a few ones which seems to be related to Terminal colors and locale.

[escapedcat]

Also linking #3251 here