Crash on node v25

[coderaiser]

c8 tape 'test/**/*.js'
file:///Users/coderaiser/@cloudcmd/fileop/node_modules/yargs/yargs:3
const {applyExtends, cjsPlatformShim, Parser, Yargs, processArgv} = require('./build/index.cjs')
                                                                    ^

ReferenceError: require is not defined in ES module scope, you can use import instead
    at file:///Users/coderaiser/@cloudcmd/fileop/node_modules/yargs/yargs:3:69
    at ModuleJobSync.runSync (node:internal/modules/esm/module_job:534:37)

Fix: #581
Related to yargs/yargs#2509
Possible solution: #582 (comment)

[uhop]

The easiest way out of it is to update dependencies. Specifically go from yargs 17 to 18 — the major version has changed.

[43081j]

Can confirm this has broken a lot of builds. Basically anything that uses node@latest in CI with c8 (which at least shows using latest node did the intended job of catching these things 😄 ).

[sverweij]

As a workaround I've temporarily added an override to our local package.json like so:

  "overrides": {
    "c8": {
      "yargs": "^18.0.0"
    },
  },

... and can confirm that, as @uhop mentions bumping yargs to 18 seems resolve the issue without code changes in c8 (build before override, build after override).

Edit: Was looking to make a PR to bump yargs in c8 itself, but bcoe already has one cooking: #578. So the 'fix' mentioned in this comment is only for the really impatient :-)

[coderaiser]

Possible solution: superc8 with:

• ✅ Fixed TODO (parase-args#L176, check-coverage#L14);
• ✅ ESM-only code;
• ✅ Updated dependencies, including yargs that makes node v25;
• ✅ Applied 🐊Putout default suggestions;
• ✅ When tests failed do not report coverage;
• ✅ Used execa instead of foreground-child;