build: Add chalk@^4 explicitly

[EltonChou]

chalk V5

The compiled code will use require().
It will throw error if user installed chalk@^5.

[lukasgeiter]

Thanks for your PR. The only reason why I'm even using require() is that I wanted chalk to be an optional dependency.

[EltonChou]

Understand your purpose and it works properly if user installed chalk@^4.

Here is my test context.

Runtime: Node.js v22.12.0

package.json

{
  "name": "gettext-test",
  "packageManager": "[email protected]",
  "dependencies": {
    "chalk": "^5.4.1",
    "gettext-extractor": "^3.8.0"
  }
}

main.mjs (same result when using require() and .cjs)

import { GettextExtractor, JsExtractors } from 'gettext-extractor'

const extractor = new GettextExtractor()

extractor
  .createJsParser([
    JsExtractors.callExpression(['_', 'getText'], {
      arguments: {
        text: 0,
        context: 1,
      },
   })])
  .parseString('')

extractor.printStats()

Error message

node_modules/.store/gettext-extractor-npm-3.8.0-3b82f4373c/package/dist/utils/output.js:11
        return chalk ? chalk.green(value) : value;
                             ^

TypeError: chalk.green is not a function

[lukasgeiter]

Ideally we would change the import code that it works with v5 as well.

If that's not possible, we would have to add it as a dependency in package.json. But then we might as well replace the require() with a regular import, making the code compatible with v5.