eslint-plugin-n 17.5.1 → 17.6.0

package/README.md

@@ -18,11 +18,11 @@ Additional ESLint rules for Node.js
 npm install --save-dev eslint eslint-plugin-n
 ```
 
-| Version | Supported Node.js | Supported ESLint Version |
-|---------|-------------------|---------------------------|
-| 17.x   | `^18.18.0 \|\| ^20.9.0 \|\| >=21.1.0`  | `>=8.23.0`          |
-| 16.x   | `>=16.0.0`   | `>=7.0.0`          |
-| 15.x   | `>=12.22.0`  | `>=7.0.0`          |
+| Version | Supported Node.js | Supported ESLint Version | Status |
+|---------|-------------------|---------------------------|--------|
+| 17.x   | `^18.18.0 \|\| ^20.9.0 \|\| >=21.1.0`  | `>=8.23.0`          | 🏃‍♂️actively maintained |
+| 16.x   | `>=16.0.0`   | `>=7.0.0`          | ⚠️EOL |
+| 15.x   | `>=12.22.0`  | `>=7.0.0`          | ⚠️EOL |
 
 **Note:** It recommends a use of [the "engines" field of package.json](https://docs.npmjs.com/files/package.json#engines). The "engines" field is used by `n/no-unsupported-features/*` rules.
 

package/lib/rules/no-unsupported-features/node-builtins.js

@@ -37,6 +37,7 @@ module.exports = {
                 type: "object",
                 properties: {
                     version: getConfiguredNodeVersion.schema,
+                    allowExperimental: { type: "boolean" },
                     ignores: {
                         type: "array",
                         items: {

package/lib/util/check-unsupported-builtins.js

@@ -15,51 +15,56 @@ const semverRangeSubset = require("semver/ranges/subset")
  * Parses the options.
  * @param {import('eslint').Rule.RuleContext} context The rule context.
  * @returns {Readonly<{
- *   version: import('semver').Range,
- *   ignores: Set<string>
+ *   version: import('semver').Range;
+ *   ignores: Set<string>;
+ *   allowExperimental: boolean;
  * }>} Parsed value.
  */
 function parseOptions(context) {
     const raw = context.options[0] || {}
     const version = getConfiguredNodeVersion(context)
     const ignores = new Set(raw.ignores || [])
+    const allowExperimental = raw.allowExperimental ?? false
 
-    return Object.freeze({ version, ignores })
+    return Object.freeze({ version, ignores, allowExperimental })
 }
 
 /**
  * Check if it has been supported.
- * @param {import('../unsupported-features/types.js').SupportInfo} info The support info.
- * @param {import('semver').Range} configured The configured version range.
+ * @param {string[] | undefined} featureRange The target features supported range
+ * @param {import('semver').Range} requestedRange The configured version range.
+ * @returns {boolean}
  */
-function isSupported({ supported }, configured) {
-    if (supported == null || supported.length === 0) {
+function isInRange(featureRange, requestedRange) {
+    if (featureRange == null || featureRange.length === 0) {
         return false
     }
 
-    const [latest] = rsort(supported)
+    const [latest] = rsort(featureRange)
     const range = getSemverRange(
-        [...supported.map(version => `^${version}`), `>= ${latest}`].join("||")
+        [...featureRange.map(version => `^${version}`), `>= ${latest}`].join(
+            "||"
+        )
     )
 
     if (range == null) {
         return false
     }
 
-    return semverRangeSubset(configured, range)
+    return semverRangeSubset(requestedRange, range)
 }
 
 /**
  * Get the formatted text of a given supported version.
- * @param {import('../unsupported-features/types.js').SupportInfo} info The support info.
+ * @param {string[] | undefined} versions The support info.
  * @returns {string | undefined}
  */
-function supportedVersionToString({ supported }) {
-    if (supported == null || supported.length === 0) {
+function versionsToString(versions) {
+    if (versions == null) {
         return
     }
 
-    const [latest, ...backported] = rsort(supported)
+    const [latest, ...backported] = rsort(versions)
 
     if (backported.length === 0) {
         return latest
@@ -92,20 +97,54 @@ module.exports.checkUnsupportedBuiltins = function checkUnsupportedBuiltins(
 
     for (const { node, path, info } of references) {
         const name = unprefixNodeColon(path.join("."))
-        const supported = isSupported(info, options.version)
 
-        if (supported === true || options.ignores.has(name)) {
+        if (options.ignores.has(name)) {
             continue
         }
-        const supportedVersion = supportedVersionToString(info)
+
+        if (options.allowExperimental) {
+            if (isInRange(info.experimental, options.version)) {
+                continue
+            }
+
+            const experimentalVersion = versionsToString(info.experimental)
+            if (experimentalVersion) {
+                context.report({
+                    node,
+                    messageId: "not-experimental-till",
+                    data: {
+                        name: path.join("."),
+                        experimental: experimentalVersion,
+                        version: options.version.raw,
+                    },
+                })
+                continue
+            }
+        }
+
+        if (isInRange(info.supported, options.version)) {
+            continue
+        }
+
+        const supportedVersion = versionsToString(info.supported)
+        if (supportedVersion) {
+            context.report({
+                node,
+                messageId: "not-supported-till",
+                data: {
+                    name: path.join("."),
+                    supported: supportedVersion,
+                    version: options.version.raw,
+                },
+            })
+            continue
+        }
+
         context.report({
             node,
-            messageId: supportedVersion
-                ? "not-supported-till"
-                : "not-supported-yet",
+            messageId: "not-supported-yet",
             data: {
                 name: path.join("."),
-                supported: /** @type string */ (supportedVersion),
                 version: options.version.raw,
             },
         })
@@ -113,6 +152,11 @@ module.exports.checkUnsupportedBuiltins = function checkUnsupportedBuiltins(
 }
 
 exports.messages = {
+    "not-experimental-till": [
+        "The '{{name}}' is not an experimental feature",
+        "until Node.js {{experimental}}.",
+        "The configured version range is '{{version}}'.",
+    ].join(" "),
     "not-supported-till": [
         "The '{{name}}' is still an experimental feature",
         "and is not supported until Node.js {{supported}}.",

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "eslint-plugin-n",
-    "version": "17.5.1",
+    "version": "17.6.0",
     "description": "Additional ESLint's rules for Node.js",
     "engines": {
         "node": "^18.18.0 || ^20.9.0 || >=21.1.0"