kevinoid / node-project-template / 25987965190

96%

push

github

override yargs for c8 only on Node 26

Forcing yargs@^18 for c8 only works on Node versions which support
[loading ECMAScript modules using
`require()`](https://nodejs.org/api/modules.html#loading-ecmascript-modules-using-require)
(v23, v22.12.0, v20.19.0).  On unsupported versions, it fails with:

    > c8 --reporter=lcov --reporter=text npm run test-unit

    node:internal/modules/cjs/loader:1123
        throw new ERR_REQUIRE_ESM(filename, true);
        ^

    Error [ERR_REQUIRE_ESM]: require() of ES Module /home/runner/work/node-project-template/node-project-template/node_modules/yargs/index.mjs not supported.
    Instead change the require of /home/runner/work/node-project-template/node-project-template/node_modules/yargs/index.mjs to a dynamic import() which is available in all CommonJS modules.
        at Object.<anonymous> (/home/runner/work/node-project-template/node-project-template/node_modules/c8/lib/parse-args.js:5:15)
        at Object.<anonymous> (/home/runner/work/node-project-template/node-project-template/node_modules/c8/bin/c8.js:11:5) {
      code: 'ERR_REQUIRE_ESM'
    }

Fixes: b83e38423d36 ("fix: override for c8 to use yargs@^18 for Node 26")
Signed-off-by: Kevin Locke <kevin@kevinlocke.name>

33 of 38 branches covered (86.84%)

Branch coverage included in aggregate %.

160 of 164 relevant lines covered (97.56%)

18.44 hits per line