esm: clarify ERR_REQUIRE_ESM errors

In #39175, better ESM errors were introduced. This commit tweaks the language in the error slightly to make it clear that there are three different options to resolve the error. Refs: #39175 PR-URL: #49521 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Jacob Smith <[email protected]> Reviewed-By: Geoffrey Booth <[email protected]> Reviewed-By: Stephen Belanger <[email protected]>

Author: danielcompton

lib/internal/errors.js

@@ -1542,7 +1542,7 @@ E('ERR_REQUIRE_ESM',
 msg+=`\n${basename} is treated as an ES module file as it is a .js `+
 'file whose nearest parent package.json contains "type": "module" '+
 'which declares all .js files in that package scope as ES modules.'+
-`\nInstead rename ${basename} to end in .cjs, change the requiring `+
+`\nInstead either rename ${basename} to end in .cjs, change the requiring `+
 'code to use dynamic import() which is available in all CommonJS '+
 'modules, or change "type": "module" to "type": "commonjs" in '+
 `${packageJsonPath} to treat all .js files as CommonJS (using .mjs for `+

test/es-module/test-cjs-esm-warn.js

@@ -31,7 +31,7 @@ describe('CJS ↔︎ ESM interop warnings',{concurrency: true }, () =>{
 );
 assert.ok(
 stderr.replaceAll('\r','').includes(
-`Instead rename ${basename} to end in .cjs, change the requiring `+
+`Instead either rename ${basename} to end in .cjs, change the requiring `+
 'code to use dynamic import() which is available in all CommonJS '+
 `modules, or change "type": "module" to "type": "commonjs" in ${pjson} to `+
 'treat all .js files as CommonJS (using .mjs for all ES modules '+