commitizen / commitizen/cz-cli

regression: Pnpm `wrap-ans` require() of ES Module error

Ouverte
#996 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
Langage dominant
JavaScript
Étoiles
17.5k
Forks
566
Merge moyen
8 h 16 min
PR mergées (30 j)
1

Description

### Description
`cz-cli` dependency `wrap-ans` does not import properly, causing error when running `pnpm commit`

**package.json**
```Json
{
"type": "module",
"scripts": {
"commit": "cz",
...
},
...
}
```

**error in cli**
```Bash
require() of ES Module /Users/menglinmaker/Documents/2-Engineering/OpenSource/npm-library-boilerplate/node_modules/wrap-ansi/node_modules/string-width/index.js from /Users/menglinmaker/Documents/2-Engineering/OpenSource/npm-library-boilerplate/node_modules/wrap-ansi/index.js not supported.
Instead change the require of /Users/menglinmaker/Documents/2-Engineering/OpenSource/npm-library-boilerplate/node_modules/wrap-ansi/node_modules/string-width/index.js in /Users/menglinmaker/Documents/2-Engineering/OpenSource/npm-library-boilerplate/node_modules/wrap-ansi/index.js to a dynamic import() which is available in all CommonJS modules.
```

Note: using `"type": "commonjs"` leads to the same result

### Additional details
This issue only occurs on `commitizen@4.2.5` and after.
I found `commitizen@4.2.2` to behave exactly how I liked.

Guide de contribution

Ouvrir le guide de contribution

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.