eslint 8.57.0 → 9.2.0

package/lib/linter/report-translator.js

@@ -11,7 +11,7 @@
 
 const assert = require("assert");
 const ruleFixer = require("./rule-fixer");
-const interpolate = require("./interpolate");
+const { interpolate } = require("./interpolate");
 
 //------------------------------------------------------------------------------
 // Typedefs
@@ -160,7 +160,7 @@ function mergeFixes(fixes, sourceCode) {
 
     const originalText = sourceCode.text;
     const start = fixes[0].range[0];
-    const end = fixes[fixes.length - 1].range[1];
+    const end = fixes.at(-1).range[1];
     let text = "";
     let lastPos = Number.MIN_SAFE_INTEGER;
 
@@ -343,7 +343,7 @@ module.exports = function createReportTranslator(metadata) {
             if (descriptor.message) {
                 throw new TypeError("context.report() called with a message and a messageId. Please only pass one.");
             }
-            if (!messages || !Object.prototype.hasOwnProperty.call(messages, id)) {
+            if (!messages || !Object.hasOwn(messages, id)) {
                 throw new TypeError(`context.report() called with a messageId of '${id}' which is not present in the 'messages' config: ${JSON.stringify(messages, null, 2)}`);
             }
             computedMessage = messages[id];

package/lib/linter/rules.js

@@ -13,18 +13,10 @@
 const builtInRules = require("../rules");
 
 //------------------------------------------------------------------------------
-// Helpers
+// Typedefs
 //------------------------------------------------------------------------------
 
-/**
- * Normalizes a rule module to the new-style API
- * @param {(Function|{create: Function})} rule A rule object, which can either be a function
- * ("old-style") or an object with a `create` method ("new-style")
- * @returns {{create: Function}} A new-style rule.
- */
-function normalizeRule(rule) {
-    return typeof rule === "function" ? Object.assign({ create: rule }, rule) : rule;
-}
+/** @typedef {import("../shared/types").Rule} Rule */
 
 //------------------------------------------------------------------------------
 // Public Interface
@@ -41,18 +33,17 @@ class Rules {
     /**
      * Registers a rule module for rule id in storage.
      * @param {string} ruleId Rule id (file name).
-     * @param {Function} ruleModule Rule handler.
+     * @param {Rule} rule Rule object.
      * @returns {void}
      */
-    define(ruleId, ruleModule) {
-        this._rules[ruleId] = normalizeRule(ruleModule);
+    define(ruleId, rule) {
+        this._rules[ruleId] = rule;
     }
 
     /**
      * Access rule handler by id (file name).
      * @param {string} ruleId Rule id (file name).
-     * @returns {{create: Function, schema: JsonSchema[]}}
-     * A rule. This is normalized to always have the new-style shape with a `create` method.
+     * @returns {Rule} Rule object.
      */
     get(ruleId) {
         if (typeof this._rules[ruleId] === "string") {

package/lib/linter/source-code-fixer.js

@@ -107,7 +107,7 @@ SourceCodeFixer.applyFixes = function(sourceText, messages, shouldFix) {
     }
 
     messages.forEach(problem => {
-        if (Object.prototype.hasOwnProperty.call(problem, "fix")) {
+        if (Object.hasOwn(problem, "fix")) {
             fixes.push(problem);
         } else {
             remainingMessages.push(problem);

package/lib/linter/timing.js

@@ -5,6 +5,8 @@
 
 "use strict";
 
+const { startTime, endTime } = require("../shared/stats");
+
 //------------------------------------------------------------------------------
 // Helpers
 //------------------------------------------------------------------------------
@@ -128,21 +130,27 @@ module.exports = (function() {
      * Time the run
      * @param {any} key key from the data object
      * @param {Function} fn function to be called
+     * @param {boolean} stats if 'stats' is true, return the result and the time difference
      * @returns {Function} function to be executed
      * @private
      */
-    function time(key, fn) {
-        if (typeof data[key] === "undefined") {
-            data[key] = 0;
-        }
+    function time(key, fn, stats) {
 
         return function(...args) {
-            let t = process.hrtime();
+
+            const t = startTime();
             const result = fn(...args);
+            const tdiff = endTime(t);
+
+            if (enabled) {
+                if (typeof data[key] === "undefined") {
+                    data[key] = 0;
+                }
+
+                data[key] += tdiff;
+            }
 
-            t = process.hrtime(t);
-            data[key] += t[0] * 1e3 + t[1] / 1e6;
-            return result;
+            return stats ? { result, tdiff } : result;
         };
     }
 

package/lib/options.js

@@ -38,7 +38,7 @@ const optionator = require("optionator");
  * @property {boolean} [help] Show help
  * @property {boolean} ignore Disable use of ignore files and patterns
  * @property {string} [ignorePath] Specify path of ignore file
- * @property {string[]} [ignorePattern] Pattern of files to ignore (in addition to those in .eslintignore)
+ * @property {string[]} [ignorePattern] Patterns of files to ignore. In eslintrc mode, these are in addition to `.eslintignore`
  * @property {boolean} init Run config initialization wizard
  * @property {boolean} inlineConfig Prevent comments from changing config or rules
  * @property {number} maxWarnings Number of warnings to trigger nonzero exit code
@@ -57,7 +57,10 @@ const optionator = require("optionator");
  * @property {boolean} quiet Report errors only
  * @property {boolean} [version] Output the version number
  * @property {boolean} warnIgnored Show warnings when the file list includes ignored files
+ * @property {boolean} [passOnNoPatterns=false] When set to true, missing patterns cause
+ *      the linting operation to short circuit and not report any failures.
  * @property {string[]} _ Positional filenames or patterns
+ * @property {boolean} [stats] Report additional statistics
  */
 
 //------------------------------------------------------------------------------
@@ -101,6 +104,16 @@ module.exports = function(usingFlatConfig) {
         };
     }
 
+    let inspectConfigFlag;
+
+    if (usingFlatConfig) {
+        inspectConfigFlag = {
+            option: "inspect-config",
+            type: "Boolean",
+            description: "Open the config inspector with the current configuration"
+        };
+    }
+
     let extFlag;
 
     if (!usingFlatConfig) {
@@ -141,6 +154,17 @@ module.exports = function(usingFlatConfig) {
         };
     }
 
+    let statsFlag;
+
+    if (usingFlatConfig) {
+        statsFlag = {
+            option: "stats",
+            type: "Boolean",
+            default: "false",
+            description: "Add statistics to the lint report"
+        };
+    }
+
     let warnIgnoredFlag;
 
     if (usingFlatConfig) {
@@ -171,6 +195,7 @@ module.exports = function(usingFlatConfig) {
                     ? "Use this configuration instead of eslint.config.js, eslint.config.mjs, or eslint.config.cjs"
                     : "Use this configuration, overriding .eslintrc.* config options if present"
             },
+            inspectConfigFlag,
             envFlag,
             extFlag,
             {
@@ -236,7 +261,7 @@ module.exports = function(usingFlatConfig) {
             {
                 option: "ignore-pattern",
                 type: "[String]",
-                description: "Pattern of files to ignore (in addition to those in .eslintignore)",
+                description: `Patterns of files to ignore${usingFlatConfig ? "" : " (in addition to those in .eslintignore)"}`,
                 concatRepeatedArrays: [true, {
                     oneValuePerFlag: true
                 }]
@@ -370,6 +395,12 @@ module.exports = function(usingFlatConfig) {
                 description: "Exit with exit code 2 in case of fatal error"
             },
             warnIgnoredFlag,
+            {
+                option: "pass-on-no-patterns",
+                type: "Boolean",
+                default: false,
+                description: "Exit with exit code 0 in case no file patterns are passed"
+            },
             {
                 option: "debug",
                 type: "Boolean",
@@ -392,7 +423,8 @@ module.exports = function(usingFlatConfig) {
                 option: "print-config",
                 type: "path::String",
                 description: "Print the configuration for the given file"
-            }
+            },
+            statsFlag
         ].filter(value => !!value)
     });
 };

package/lib/rule-tester/index.js

@@ -1,5 +1,7 @@
 "use strict";
 
+const RuleTester = require("./rule-tester");
+
 module.exports = {
-    RuleTester: require("./rule-tester")
+    RuleTester
 };