commitizen / commitizen/cz-cli

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

Open
#996 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
17.5k
Forks
566
Avg merge
8h 16m
Merged PRs (30d)
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.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.