node: imports aren't supported in Node.js v14

[electerious]

The rule.js file uses node: imports which are supported since Node.js v16, but eslint-plugin-unicorn says it supports v14 and up.

The original error message:

Error: Failed to load plugin 'unicorn' declared in '.eslintrc.json': Cannot find module 'node:path'
Require stack:
- eslint-plugin-unicorn@43.0.2_eslint@8.20.0/node_modules/eslint-plugin-unicorn/rules/utils/rule.js
- eslint-plugin-unicorn@43.0.2_eslint@8.20.0/node_modules/eslint-plugin-unicorn/index.js
- @eslint+eslintrc@1.3.0/node_modules/@eslint/eslintrc/dist/eslintrc.cjs

The issue occurs after upgrading from 42.0.0 to 43.0.2.

It's a bit confusing that the tests are passing despite that they're running with v14, too.

[electerious]

My fault. node: is supported since v14.13.1, but my version was slightly below.

[axelboc]

I've just hit this. Seems that Node 14.13.1+ only supports node:* imports from ESM files -- not from CJS files. I found the info here: gregnb/filemanager-webpack-plugin#101 (comment).

I recommend re-opening this issue and either increasing the minimum Node version to v16, reverting to unprefixed paths, or making sure the node: prefixes are removed in the CJS output.

[axelboc]

Okay, got it: they've added support for node:* imports from CJS in Node 14.18.0: https://github.com/nodejs/node/blob/main/doc/changelogs/CHANGELOG_V14.md#2021-09-28-version-14180-fermium-lts-targos. I was confused, because I was seeing the error with Node 14.15, so after 14.13.1.

Bumping the minimum Node version to v14.18.0 should do the trick.

[bmish]

We could mention the Node version support in the rule doc to reduce confusion here: #1874