eslint 8.39.0 → 8.47.0

package/lib/rules/max-lines-per-function.js

@@ -73,7 +73,7 @@ module.exports = {
         docs: {
             description: "Enforce a maximum number of lines of code in a function",
             recommended: false,
-            url: "https://eslint.org/docs/rules/max-lines-per-function"
+            url: "https://eslint.org/docs/latest/rules/max-lines-per-function"
         },
 
         schema: [
@@ -85,7 +85,7 @@ module.exports = {
     },
 
     create(context) {
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
         const lines = sourceCode.lines;
 
         const option = context.options[0];

package/lib/rules/max-lines.js

@@ -36,7 +36,7 @@ module.exports = {
         docs: {
             description: "Enforce a maximum number of lines per file",
             recommended: false,
-            url: "https://eslint.org/docs/rules/max-lines"
+            url: "https://eslint.org/docs/latest/rules/max-lines"
         },
 
         schema: [
@@ -87,7 +87,7 @@ module.exports = {
         const skipComments = option && option.skipComments;
         const skipBlankLines = option && option.skipBlankLines;
 
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         /**
          * Returns whether or not a token is a comment node type

package/lib/rules/max-nested-callbacks.js

@@ -17,7 +17,7 @@ module.exports = {
         docs: {
             description: "Enforce a maximum depth that callbacks can be nested",
             recommended: false,
-            url: "https://eslint.org/docs/rules/max-nested-callbacks"
+            url: "https://eslint.org/docs/latest/rules/max-nested-callbacks"
         },
 
         schema: [

package/lib/rules/max-params.js

@@ -24,7 +24,7 @@ module.exports = {
         docs: {
             description: "Enforce a maximum number of parameters in function definitions",
             recommended: false,
-            url: "https://eslint.org/docs/rules/max-params"
+            url: "https://eslint.org/docs/latest/rules/max-params"
         },
 
         schema: [
@@ -57,7 +57,7 @@ module.exports = {
     },
 
     create(context) {
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
         const option = context.options[0];
         let numParams = 3;
 

package/lib/rules/max-statements-per-line.js

@@ -22,7 +22,7 @@ module.exports = {
         docs: {
             description: "Enforce a maximum number of statements allowed per line",
             recommended: false,
-            url: "https://eslint.org/docs/rules/max-statements-per-line"
+            url: "https://eslint.org/docs/latest/rules/max-statements-per-line"
         },
 
         schema: [
@@ -45,7 +45,7 @@ module.exports = {
 
     create(context) {
 
-        const sourceCode = context.getSourceCode(),
+        const sourceCode = context.sourceCode,
             options = context.options[0] || {},
             maxStatementsPerLine = typeof options.max !== "undefined" ? options.max : 1;
 

package/lib/rules/max-statements.js

@@ -24,7 +24,7 @@ module.exports = {
         docs: {
             description: "Enforce a maximum number of statements allowed in function blocks",
             recommended: false,
-            url: "https://eslint.org/docs/rules/max-statements"
+            url: "https://eslint.org/docs/latest/rules/max-statements"
         },
 
         schema: [

package/lib/rules/multiline-comment-style.js

@@ -18,7 +18,7 @@ module.exports = {
         docs: {
             description: "Enforce a particular style for multiline comments",
             recommended: false,
-            url: "https://eslint.org/docs/rules/multiline-comment-style"
+            url: "https://eslint.org/docs/latest/rules/multiline-comment-style"
         },
 
         fixable: "whitespace",
@@ -65,7 +65,7 @@ module.exports = {
     },
 
     create(context) {
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
         const option = context.options[0] || "starred-block";
         const params = context.options[1] || {};
         const checkJSDoc = !!params.checkJSDoc;

package/lib/rules/multiline-ternary.js

@@ -19,7 +19,7 @@ module.exports = {
         docs: {
             description: "Enforce newlines between operands of ternary expressions",
             recommended: false,
-            url: "https://eslint.org/docs/rules/multiline-ternary"
+            url: "https://eslint.org/docs/latest/rules/multiline-ternary"
         },
 
         schema: [
@@ -39,7 +39,7 @@ module.exports = {
     },
 
     create(context) {
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
         const option = context.options[0];
         const multiline = option !== "never";
         const allowSingleLine = option === "always-multiline";

package/lib/rules/new-cap.js

@@ -84,7 +84,7 @@ module.exports = {
         docs: {
             description: "Require constructor names to begin with a capital letter",
             recommended: false,
-            url: "https://eslint.org/docs/rules/new-cap"
+            url: "https://eslint.org/docs/latest/rules/new-cap"
         },
 
         schema: [
@@ -147,7 +147,7 @@ module.exports = {
 
         const listeners = {};
 
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         //--------------------------------------------------------------------------
         // Helpers

package/lib/rules/new-parens.js

@@ -27,7 +27,7 @@ module.exports = {
         docs: {
             description: "Enforce or disallow parentheses when invoking a constructor with no arguments",
             recommended: false,
-            url: "https://eslint.org/docs/rules/new-parens"
+            url: "https://eslint.org/docs/latest/rules/new-parens"
         },
 
         fixable: "code",
@@ -46,7 +46,7 @@ module.exports = {
         const options = context.options;
         const always = options[0] !== "never"; // Default is always
 
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         return {
             NewExpression(node) {

package/lib/rules/newline-after-var.js

@@ -24,7 +24,7 @@ module.exports = {
         docs: {
             description: "Require or disallow an empty line after variable declarations",
             recommended: false,
-            url: "https://eslint.org/docs/rules/newline-after-var"
+            url: "https://eslint.org/docs/latest/rules/newline-after-var"
         },
         schema: [
             {
@@ -43,7 +43,7 @@ module.exports = {
     },
 
     create(context) {
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         // Default `mode` to "always".
         const mode = context.options[0] === "never" ? "never" : "always";
@@ -212,7 +212,6 @@ module.exports = {
                 context.report({
                     node,
                     messageId: "unexpected",
-                    data: { identifier: node.name },
                     fix(fixer) {
                         const linesBetween = sourceCode.getText().slice(lastToken.range[1], nextToken.range[0]).split(astUtils.LINEBREAK_MATCHER);
 
@@ -231,7 +230,6 @@ module.exports = {
                 context.report({
                     node,
                     messageId: "expected",
-                    data: { identifier: node.name },
                     fix(fixer) {
                         if ((noNextLineToken ? getLastCommentLineOfBlock(nextLineNum) : lastToken.loc.end.line) === nextToken.loc.start.line) {
                             return fixer.insertTextBefore(nextToken, "\n\n");

package/lib/rules/newline-before-return.js

@@ -17,7 +17,7 @@ module.exports = {
         docs: {
             description: "Require an empty line before `return` statements",
             recommended: false,
-            url: "https://eslint.org/docs/rules/newline-before-return"
+            url: "https://eslint.org/docs/latest/rules/newline-before-return"
         },
 
         fixable: "whitespace",
@@ -31,7 +31,7 @@ module.exports = {
     },
 
     create(context) {
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         //--------------------------------------------------------------------------
         // Helpers

package/lib/rules/newline-per-chained-call.js

@@ -20,7 +20,7 @@ module.exports = {
         docs: {
             description: "Require a newline after each call in a method chain",
             recommended: false,
-            url: "https://eslint.org/docs/rules/newline-per-chained-call"
+            url: "https://eslint.org/docs/latest/rules/newline-per-chained-call"
         },
 
         fixable: "whitespace",
@@ -47,7 +47,7 @@ module.exports = {
         const options = context.options[0] || {},
             ignoreChainWithDepth = options.ignoreChainWithDepth || 2;
 
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         /**
          * Get the prefix of a given MemberExpression node.

package/lib/rules/no-alert.js

@@ -90,7 +90,7 @@ module.exports = {
         docs: {
             description: "Disallow the use of `alert`, `confirm`, and `prompt`",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-alert"
+            url: "https://eslint.org/docs/latest/rules/no-alert"
         },
 
         schema: [],
@@ -101,7 +101,7 @@ module.exports = {
     },
 
     create(context) {
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         return {
             CallExpression(node) {

package/lib/rules/no-array-constructor.js

@@ -17,7 +17,7 @@ module.exports = {
         docs: {
             description: "Disallow `Array` constructors",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-array-constructor"
+            url: "https://eslint.org/docs/latest/rules/no-array-constructor"
         },
 
         schema: [],

package/lib/rules/no-async-promise-executor.js

@@ -16,7 +16,7 @@ module.exports = {
         docs: {
             description: "Disallow using an async function as a Promise executor",
             recommended: true,
-            url: "https://eslint.org/docs/rules/no-async-promise-executor"
+            url: "https://eslint.org/docs/latest/rules/no-async-promise-executor"
         },
 
         fixable: null,
@@ -30,7 +30,7 @@ module.exports = {
         return {
             "NewExpression[callee.name='Promise'][arguments.0.async=true]"(node) {
                 context.report({
-                    node: context.getSourceCode().getFirstToken(node.arguments[0], token => token.value === "async"),
+                    node: context.sourceCode.getFirstToken(node.arguments[0], token => token.value === "async"),
                     messageId: "async"
                 });
             }

package/lib/rules/no-await-in-loop.js

@@ -61,7 +61,7 @@ module.exports = {
         docs: {
             description: "Disallow `await` inside of loops",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-await-in-loop"
+            url: "https://eslint.org/docs/latest/rules/no-await-in-loop"
         },
 
         schema: [],

package/lib/rules/no-bitwise.js

@@ -28,7 +28,7 @@ module.exports = {
         docs: {
             description: "Disallow bitwise operators",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-bitwise"
+            url: "https://eslint.org/docs/latest/rules/no-bitwise"
         },
 
         schema: [

package/lib/rules/no-buffer-constructor.js

@@ -21,7 +21,7 @@ module.exports = {
         docs: {
             description: "Disallow use of the `Buffer()` constructor",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-buffer-constructor"
+            url: "https://eslint.org/docs/latest/rules/no-buffer-constructor"
         },
 
         schema: [],

package/lib/rules/no-caller.js

@@ -17,7 +17,7 @@ module.exports = {
         docs: {
             description: "Disallow the use of `arguments.caller` or `arguments.callee`",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-caller"
+            url: "https://eslint.org/docs/latest/rules/no-caller"
         },
 
         schema: [],

package/lib/rules/no-case-declarations.js

@@ -16,7 +16,7 @@ module.exports = {
         docs: {
             description: "Disallow lexical declarations in case clauses",
             recommended: true,
-            url: "https://eslint.org/docs/rules/no-case-declarations"
+            url: "https://eslint.org/docs/latest/rules/no-case-declarations"
         },
 
         schema: [],

package/lib/rules/no-catch-shadow.js

@@ -24,7 +24,7 @@ module.exports = {
         docs: {
             description: "Disallow `catch` clause parameters from shadowing variables in the outer scope",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-catch-shadow"
+            url: "https://eslint.org/docs/latest/rules/no-catch-shadow"
         },
 
         replacedBy: ["no-shadow"],
@@ -39,7 +39,7 @@ module.exports = {
 
     create(context) {
 
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         //--------------------------------------------------------------------------
         // Helpers

package/lib/rules/no-class-assign.js

@@ -19,7 +19,7 @@ module.exports = {
         docs: {
             description: "Disallow reassigning class members",
             recommended: true,
-            url: "https://eslint.org/docs/rules/no-class-assign"
+            url: "https://eslint.org/docs/latest/rules/no-class-assign"
         },
 
         schema: [],
@@ -31,7 +31,7 @@ module.exports = {
 
     create(context) {
 
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         /**
          * Finds and reports references that are non initializer and writable.

package/lib/rules/no-compare-neg-zero.js

@@ -16,7 +16,7 @@ module.exports = {
         docs: {
             description: "Disallow comparing against -0",
             recommended: true,
-            url: "https://eslint.org/docs/rules/no-compare-neg-zero"
+            url: "https://eslint.org/docs/latest/rules/no-compare-neg-zero"
         },
 
         fixable: null,

package/lib/rules/no-cond-assign.js

@@ -36,7 +36,7 @@ module.exports = {
         docs: {
             description: "Disallow assignment operators in conditional expressions",
             recommended: true,
-            url: "https://eslint.org/docs/rules/no-cond-assign"
+            url: "https://eslint.org/docs/latest/rules/no-cond-assign"
         },
 
         schema: [
@@ -57,7 +57,7 @@ module.exports = {
 
         const prohibitAssign = (context.options[0] || "except-parens");
 
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         /**
          * Check whether an AST node is the test expression for a conditional statement.

package/lib/rules/no-confusing-arrow.js

@@ -33,7 +33,7 @@ module.exports = {
         docs: {
             description: "Disallow arrow functions where they could be confused with comparisons",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-confusing-arrow"
+            url: "https://eslint.org/docs/latest/rules/no-confusing-arrow"
         },
 
         fixable: "code",
@@ -56,7 +56,7 @@ module.exports = {
         const config = context.options[0] || {};
         const allowParens = config.allowParens || (config.allowParens === void 0);
         const onlyOneSimpleParam = config.onlyOneSimpleParam;
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
 
         /**

package/lib/rules/no-console.js

@@ -23,7 +23,7 @@ module.exports = {
         docs: {
             description: "Disallow the use of `console`",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-console"
+            url: "https://eslint.org/docs/latest/rules/no-console"
         },
 
         schema: [
@@ -51,7 +51,7 @@ module.exports = {
     create(context) {
         const options = context.options[0] || {};
         const allowed = options.allow || [];
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         /**
          * Checks whether the given reference is 'console' or not.

package/lib/rules/no-const-assign.js

@@ -19,7 +19,7 @@ module.exports = {
         docs: {
             description: "Disallow reassigning `const` variables",
             recommended: true,
-            url: "https://eslint.org/docs/rules/no-const-assign"
+            url: "https://eslint.org/docs/latest/rules/no-const-assign"
         },
 
         schema: [],
@@ -31,7 +31,7 @@ module.exports = {
 
     create(context) {
 
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         /**
          * Finds and reports references that are non initializer and writable.

package/lib/rules/no-constant-binary-expression.js

@@ -441,7 +441,7 @@ module.exports = {
         docs: {
             description: "Disallow expressions where the operation doesn't affect the value",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-constant-binary-expression"
+            url: "https://eslint.org/docs/latest/rules/no-constant-binary-expression"
         },
         schema: [],
         messages: {
@@ -453,7 +453,7 @@ module.exports = {
     },
 
     create(context) {
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         return {
             LogicalExpression(node) {

package/lib/rules/no-constant-condition.js

@@ -23,7 +23,7 @@ module.exports = {
         docs: {
             description: "Disallow constant expressions in conditions",
             recommended: true,
-            url: "https://eslint.org/docs/rules/no-constant-condition"
+            url: "https://eslint.org/docs/latest/rules/no-constant-condition"
         },
 
         schema: [
@@ -48,7 +48,7 @@ module.exports = {
         const options = context.options[0] || {},
             checkLoops = options.checkLoops !== false,
             loopSetStack = [];
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         let loopsInCurrentScope = new Set();
 

package/lib/rules/no-constructor-return.js

@@ -17,7 +17,7 @@ module.exports = {
         docs: {
             description: "Disallow returning value from constructor",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-constructor-return"
+            url: "https://eslint.org/docs/latest/rules/no-constructor-return"
         },
 
         schema: {},

package/lib/rules/no-continue.js

@@ -17,7 +17,7 @@ module.exports = {
         docs: {
             description: "Disallow `continue` statements",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-continue"
+            url: "https://eslint.org/docs/latest/rules/no-continue"
         },
 
         schema: [],

package/lib/rules/no-control-regex.js

@@ -14,6 +14,16 @@ const collector = new (class {
     }
 
     onPatternEnter() {
+
+        /*
+         * `RegExpValidator` may parse the pattern twice in one `validatePattern`.
+         * So `this._controlChars` should be cleared here as well.
+         *
+         * For example, the `/(?<a>\x1f)/` regex will parse the pattern twice.
+         * This is based on the content described in Annex B.
+         * If the regex contains a `GroupName` and the `u` flag is not used, `ParseText` will be called twice.
+         * See https://tc39.es/ecma262/2023/multipage/additional-ecmascript-features-for-web-browsers.html#sec-parsepattern-annexb
+         */
         this._controlChars = [];
     }
 
@@ -32,10 +42,13 @@ const collector = new (class {
 
     collectControlChars(regexpStr, flags) {
         const uFlag = typeof flags === "string" && flags.includes("u");
+        const vFlag = typeof flags === "string" && flags.includes("v");
+
+        this._controlChars = [];
+        this._source = regexpStr;
 
         try {
-            this._source = regexpStr;
-            this._validator.validatePattern(regexpStr, void 0, void 0, uFlag); // Call onCharacter hook
+            this._validator.validatePattern(regexpStr, void 0, void 0, { unicode: uFlag, unicodeSets: vFlag }); // Call onCharacter hook
         } catch {
 
             // Ignore syntax errors in RegExp.
@@ -56,7 +69,7 @@ module.exports = {
         docs: {
             description: "Disallow control characters in regular expressions",
             recommended: true,
-            url: "https://eslint.org/docs/rules/no-control-regex"
+            url: "https://eslint.org/docs/latest/rules/no-control-regex"
         },
 
         schema: [],

package/lib/rules/no-debugger.js

@@ -17,7 +17,7 @@ module.exports = {
         docs: {
             description: "Disallow the use of `debugger`",
             recommended: true,
-            url: "https://eslint.org/docs/rules/no-debugger"
+            url: "https://eslint.org/docs/latest/rules/no-debugger"
         },
 
         fixable: null,

package/lib/rules/no-delete-var.js

@@ -17,7 +17,7 @@ module.exports = {
         docs: {
             description: "Disallow deleting variables",
             recommended: true,
-            url: "https://eslint.org/docs/rules/no-delete-var"
+            url: "https://eslint.org/docs/latest/rules/no-delete-var"
         },
 
         schema: [],

package/lib/rules/no-div-regex.js

@@ -17,7 +17,7 @@ module.exports = {
         docs: {
             description: "Disallow equal signs explicitly at the beginning of regular expressions",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-div-regex"
+            url: "https://eslint.org/docs/latest/rules/no-div-regex"
         },
 
         fixable: "code",
@@ -30,7 +30,7 @@ module.exports = {
     },
 
     create(context) {
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         return {
 

package/lib/rules/no-dupe-args.js

@@ -17,7 +17,7 @@ module.exports = {
         docs: {
             description: "Disallow duplicate arguments in `function` definitions",
             recommended: true,
-            url: "https://eslint.org/docs/rules/no-dupe-args"
+            url: "https://eslint.org/docs/latest/rules/no-dupe-args"
         },
 
         schema: [],
@@ -29,7 +29,7 @@ module.exports = {
 
     create(context) {
 
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         //--------------------------------------------------------------------------
         // Helpers

package/lib/rules/no-dupe-class-members.js

@@ -19,7 +19,7 @@ module.exports = {
         docs: {
             description: "Disallow duplicate class members",
             recommended: true,
-            url: "https://eslint.org/docs/rules/no-dupe-class-members"
+            url: "https://eslint.org/docs/latest/rules/no-dupe-class-members"
         },
 
         schema: [],

package/lib/rules/no-dupe-else-if.js

@@ -54,7 +54,7 @@ module.exports = {
         docs: {
             description: "Disallow duplicate conditions in if-else-if chains",
             recommended: true,
-            url: "https://eslint.org/docs/rules/no-dupe-else-if"
+            url: "https://eslint.org/docs/latest/rules/no-dupe-else-if"
         },
 
         schema: [],
@@ -65,7 +65,7 @@ module.exports = {
     },
 
     create(context) {
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         /**
          * Determines whether the two given nodes are considered to be equal. In particular, given that the nodes