eslint 8.39.0 → 8.47.0

package/lib/rules/no-new-native-nonconstructor.js

@@ -23,7 +23,7 @@ module.exports = {
         docs: {
             description: "Disallow `new` operators with global non-constructor functions",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-new-native-nonconstructor"
+            url: "https://eslint.org/docs/latest/rules/no-new-native-nonconstructor"
         },
 
         schema: [],
@@ -35,7 +35,7 @@ module.exports = {
 
     create(context) {
 
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         return {
             "Program:exit"(node) {

package/lib/rules/no-new-object.js

@@ -23,7 +23,7 @@ module.exports = {
         docs: {
             description: "Disallow `Object` constructors",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-new-object"
+            url: "https://eslint.org/docs/latest/rules/no-new-object"
         },
 
         schema: [],
@@ -35,7 +35,7 @@ module.exports = {
 
     create(context) {
 
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         return {
             NewExpression(node) {

package/lib/rules/no-new-require.js

@@ -22,7 +22,7 @@ module.exports = {
         docs: {
             description: "Disallow `new` operators with calls to `require`",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-new-require"
+            url: "https://eslint.org/docs/latest/rules/no-new-require"
         },
 
         schema: [],

package/lib/rules/no-new-symbol.js

@@ -17,7 +17,7 @@ module.exports = {
         docs: {
             description: "Disallow `new` operators with the `Symbol` object",
             recommended: true,
-            url: "https://eslint.org/docs/rules/no-new-symbol"
+            url: "https://eslint.org/docs/latest/rules/no-new-symbol"
         },
 
         schema: [],
@@ -29,7 +29,7 @@ module.exports = {
 
     create(context) {
 
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         return {
             "Program:exit"(node) {

package/lib/rules/no-new-wrappers.js

@@ -5,6 +5,12 @@
 
 "use strict";
 
+//------------------------------------------------------------------------------
+// Requirements
+//------------------------------------------------------------------------------
+
+const { getVariableByName } = require("./utils/ast-utils");
+
 //------------------------------------------------------------------------------
 // Rule Definition
 //------------------------------------------------------------------------------
@@ -17,7 +23,7 @@ module.exports = {
         docs: {
             description: "Disallow `new` operators with the `String`, `Number`, and `Boolean` objects",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-new-wrappers"
+            url: "https://eslint.org/docs/latest/rules/no-new-wrappers"
         },
 
         schema: [],
@@ -28,18 +34,24 @@ module.exports = {
     },
 
     create(context) {
+        const { sourceCode } = context;
 
         return {
 
             NewExpression(node) {
                 const wrapperObjects = ["String", "Number", "Boolean"];
-
-                if (wrapperObjects.includes(node.callee.name)) {
-                    context.report({
-                        node,
-                        messageId: "noConstructor",
-                        data: { fn: node.callee.name }
-                    });
+                const { name } = node.callee;
+
+                if (wrapperObjects.includes(name)) {
+                    const variable = getVariableByName(sourceCode.getScope(node), name);
+
+                    if (variable && variable.identifiers.length === 0) {
+                        context.report({
+                            node,
+                            messageId: "noConstructor",
+                            data: { fn: name }
+                        });
+                    }
                 }
             }
         };

package/lib/rules/no-new.js

@@ -18,7 +18,7 @@ module.exports = {
         docs: {
             description: "Disallow `new` operators outside of assignments or comparisons",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-new"
+            url: "https://eslint.org/docs/latest/rules/no-new"
         },
 
         schema: [],

package/lib/rules/no-nonoctal-decimal-escape.js

@@ -32,7 +32,7 @@ module.exports = {
         docs: {
             description: "Disallow `\\8` and `\\9` escape sequences in string literals",
             recommended: true,
-            url: "https://eslint.org/docs/rules/no-nonoctal-decimal-escape"
+            url: "https://eslint.org/docs/latest/rules/no-nonoctal-decimal-escape"
         },
 
         hasSuggestions: true,
@@ -49,7 +49,7 @@ module.exports = {
     },
 
     create(context) {
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         /**
          * Creates a new Suggestion object.

package/lib/rules/no-obj-calls.js

@@ -45,7 +45,7 @@ module.exports = {
         docs: {
             description: "Disallow calling global object properties as functions",
             recommended: true,
-            url: "https://eslint.org/docs/rules/no-obj-calls"
+            url: "https://eslint.org/docs/latest/rules/no-obj-calls"
         },
 
         schema: [],
@@ -58,7 +58,7 @@ module.exports = {
 
     create(context) {
 
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         return {
             Program(node) {

package/lib/rules/no-octal-escape.js

@@ -17,7 +17,7 @@ module.exports = {
         docs: {
             description: "Disallow octal escape sequences in string literals",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-octal-escape"
+            url: "https://eslint.org/docs/latest/rules/no-octal-escape"
         },
 
         schema: [],

package/lib/rules/no-octal.js

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

package/lib/rules/no-param-reassign.js

@@ -18,7 +18,7 @@ module.exports = {
         docs: {
             description: "Disallow reassigning `function` parameters",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-param-reassign"
+            url: "https://eslint.org/docs/latest/rules/no-param-reassign"
         },
 
         schema: [
@@ -70,7 +70,7 @@ module.exports = {
         const props = context.options[0] && context.options[0].props;
         const ignoredPropertyAssignmentsFor = context.options[0] && context.options[0].ignorePropertyModificationsFor || [];
         const ignoredPropertyAssignmentsForRegex = context.options[0] && context.options[0].ignorePropertyModificationsForRegex || [];
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         /**
          * Checks whether or not the reference modifies properties of its variable.

package/lib/rules/no-path-concat.js

@@ -21,7 +21,7 @@ module.exports = {
         docs: {
             description: "Disallow string concatenation with `__dirname` and `__filename`",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-path-concat"
+            url: "https://eslint.org/docs/latest/rules/no-path-concat"
         },
 
         schema: [],

package/lib/rules/no-plusplus.js

@@ -53,7 +53,7 @@ module.exports = {
         docs: {
             description: "Disallow the unary operators `++` and `--`",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-plusplus"
+            url: "https://eslint.org/docs/latest/rules/no-plusplus"
         },
 
         schema: [

package/lib/rules/no-process-env.js

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

package/lib/rules/no-process-exit.js

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

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

@@ -71,7 +71,7 @@ module.exports = {
         docs: {
             description: "Disallow returning values from Promise executor functions",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-promise-executor-return"
+            url: "https://eslint.org/docs/latest/rules/no-promise-executor-return"
         },
 
         schema: [],
@@ -84,7 +84,7 @@ module.exports = {
     create(context) {
 
         let funcInfo = null;
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         /**
          * Reports the given node.

package/lib/rules/no-proto.js

@@ -23,7 +23,7 @@ module.exports = {
         docs: {
             description: "Disallow the use of the `__proto__` property",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-proto"
+            url: "https://eslint.org/docs/latest/rules/no-proto"
         },
 
         schema: [],

package/lib/rules/no-prototype-builtins.js

@@ -22,7 +22,7 @@ module.exports = {
         docs: {
             description: "Disallow calling some `Object.prototype` methods directly on objects",
             recommended: true,
-            url: "https://eslint.org/docs/rules/no-prototype-builtins"
+            url: "https://eslint.org/docs/latest/rules/no-prototype-builtins"
         },
 
         schema: [],

package/lib/rules/no-redeclare.js

@@ -23,7 +23,7 @@ module.exports = {
         docs: {
             description: "Disallow variable redeclaration",
             recommended: true,
-            url: "https://eslint.org/docs/rules/no-redeclare"
+            url: "https://eslint.org/docs/latest/rules/no-redeclare"
         },
 
         messages: {
@@ -50,7 +50,7 @@ module.exports = {
                 context.options[0].builtinGlobals
             )
         };
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         /**
          * Iterate declarations of a given variable.

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

@@ -41,7 +41,7 @@ module.exports = {
         docs: {
             description: "Disallow multiple spaces in regular expressions",
             recommended: true,
-            url: "https://eslint.org/docs/rules/no-regex-spaces"
+            url: "https://eslint.org/docs/latest/rules/no-regex-spaces"
         },
 
         schema: [],
@@ -54,7 +54,7 @@ module.exports = {
 
     create(context) {
 
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         /**
          * Validate regular expression
@@ -77,7 +77,7 @@ module.exports = {
             let regExpAST;
 
             try {
-                regExpAST = regExpParser.parsePattern(pattern, 0, pattern.length, flags.includes("u"));
+                regExpAST = regExpParser.parsePattern(pattern, 0, pattern.length, { unicode: flags.includes("u"), unicodeSets: flags.includes("v") });
             } catch {
 
                 // Ignore regular expressions with syntax errors
@@ -155,13 +155,28 @@ module.exports = {
             const regExpVar = astUtils.getVariableByName(scope, "RegExp");
             const shadowed = regExpVar && regExpVar.defs.length > 0;
             const patternNode = node.arguments[0];
-            const flagsNode = node.arguments[1];
 
             if (node.callee.type === "Identifier" && node.callee.name === "RegExp" && isString(patternNode) && !shadowed) {
                 const pattern = patternNode.value;
                 const rawPattern = patternNode.raw.slice(1, -1);
                 const rawPatternStartRange = patternNode.range[0] + 1;
-                const flags = isString(flagsNode) ? flagsNode.value : "";
+                let flags;
+
+                if (node.arguments.length < 2) {
+
+                    // It has no flags.
+                    flags = "";
+                } else {
+                    const flagsNode = node.arguments[1];
+
+                    if (isString(flagsNode)) {
+                        flags = flagsNode.value;
+                    } else {
+
+                        // The flags cannot be determined.
+                        return;
+                    }
+                }
 
                 checkRegex(
                     node,

package/lib/rules/no-restricted-exports.js

@@ -23,7 +23,7 @@ module.exports = {
         docs: {
             description: "Disallow specified names in exports",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-restricted-exports"
+            url: "https://eslint.org/docs/latest/rules/no-restricted-exports"
         },
 
         schema: [{
@@ -99,7 +99,7 @@ module.exports = {
 
         const restrictedNames = new Set(context.options[0] && context.options[0].restrictedNamedExports);
         const restrictDefaultExports = context.options[0] && context.options[0].restrictDefaultExports;
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         /**
          * Checks and reports given exported name.

package/lib/rules/no-restricted-globals.js

@@ -16,7 +16,7 @@ module.exports = {
         docs: {
             description: "Disallow specified global variables",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-restricted-globals"
+            url: "https://eslint.org/docs/latest/rules/no-restricted-globals"
         },
 
         schema: {
@@ -50,7 +50,7 @@ module.exports = {
 
     create(context) {
 
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         // If no globals are restricted, we don't need to do anything
         if (context.options.length === 0) {

package/lib/rules/no-restricted-imports.js

@@ -98,7 +98,7 @@ module.exports = {
         docs: {
             description: "Disallow specified modules when loaded by `import`",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-restricted-imports"
+            url: "https://eslint.org/docs/latest/rules/no-restricted-imports"
         },
 
         messages: {
@@ -147,7 +147,7 @@ module.exports = {
     },
 
     create(context) {
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
         const options = Array.isArray(context.options) ? context.options : [];
         const isPathAndPatternsObject =
             typeof options[0] === "object" &&

package/lib/rules/no-restricted-modules.js

@@ -5,6 +5,12 @@
  */
 "use strict";
 
+//------------------------------------------------------------------------------
+// Requirements
+//------------------------------------------------------------------------------
+
+const astUtils = require("./utils/ast-utils");
+
 //------------------------------------------------------------------------------
 // Rule Definition
 //------------------------------------------------------------------------------
@@ -51,7 +57,7 @@ module.exports = {
         docs: {
             description: "Disallow specified modules when loaded by `require`",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-restricted-modules"
+            url: "https://eslint.org/docs/latest/rules/no-restricted-modules"
         },
 
         schema: {
@@ -116,15 +122,6 @@ module.exports = {
             return node && node.type === "Literal" && typeof node.value === "string";
         }
 
-        /**
-         * Function to check if a node is a static string template literal.
-         * @param {ASTNode} node The node to check.
-         * @returns {boolean} If the node is a string template literal.
-         */
-        function isStaticTemplateLiteral(node) {
-            return node && node.type === "TemplateLiteral" && node.expressions.length === 0;
-        }
-
         /**
          * Function to check if a node is a require call.
          * @param {ASTNode} node The node to check.
@@ -144,7 +141,7 @@ module.exports = {
                 return node.value.trim();
             }
 
-            if (isStaticTemplateLiteral(node)) {
+            if (astUtils.isStaticTemplateLiteral(node)) {
                 return node.quasis[0].value.cooked.trim();
             }
 

package/lib/rules/no-restricted-properties.js

@@ -19,7 +19,7 @@ module.exports = {
         docs: {
             description: "Disallow certain properties on certain objects",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-restricted-properties"
+            url: "https://eslint.org/docs/latest/rules/no-restricted-properties"
         },
 
         schema: {

package/lib/rules/no-restricted-syntax.js

@@ -16,7 +16,7 @@ module.exports = {
         docs: {
             description: "Disallow specified syntax",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-restricted-syntax"
+            url: "https://eslint.org/docs/latest/rules/no-restricted-syntax"
         },
 
         schema: {

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

@@ -28,7 +28,7 @@ module.exports = {
         docs: {
             description: "Disallow assignment operators in `return` statements",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-return-assign"
+            url: "https://eslint.org/docs/latest/rules/no-return-assign"
         },
 
         schema: [
@@ -45,7 +45,7 @@ module.exports = {
 
     create(context) {
         const always = (context.options[0] || "except-parens") !== "except-parens";
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         return {
             AssignmentExpression(node) {

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

@@ -1,6 +1,7 @@
 /**
  * @fileoverview Disallows unnecessary `return await`
  * @author Jordan Harband
+ * @deprecated in ESLint v8.46.0
  */
 "use strict";
 
@@ -21,11 +22,15 @@ module.exports = {
 
             recommended: false,
 
-            url: "https://eslint.org/docs/rules/no-return-await"
+            url: "https://eslint.org/docs/latest/rules/no-return-await"
         },
 
         fixable: null,
 
+        deprecated: true,
+
+        replacedBy: [],
+
         schema: [
         ],
 
@@ -44,14 +49,14 @@ module.exports = {
          */
         function reportUnnecessaryAwait(node) {
             context.report({
-                node: context.getSourceCode().getFirstToken(node),
+                node: context.sourceCode.getFirstToken(node),
                 loc: node.loc,
                 messageId: "redundantUseOfAwait",
                 suggest: [
                     {
                         messageId: "removeAwait",
                         fix(fixer) {
-                            const sourceCode = context.getSourceCode();
+                            const sourceCode = context.sourceCode;
                             const [awaitToken, tokenAfterAwait] = sourceCode.getFirstTokens(node, 2);
 
                             const areAwaitAndAwaitedExpressionOnTheSameLine = awaitToken.loc.start.line === tokenAfterAwait.loc.start.line;

package/lib/rules/no-script-url.js

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

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

@@ -132,7 +132,7 @@ module.exports = {
         docs: {
             description: "Disallow assignments where both sides are exactly the same",
             recommended: true,
-            url: "https://eslint.org/docs/rules/no-self-assign"
+            url: "https://eslint.org/docs/latest/rules/no-self-assign"
         },
 
         schema: [
@@ -154,7 +154,7 @@ module.exports = {
     },
 
     create(context) {
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
         const [{ props = true } = {}] = context.options;
 
         /**

package/lib/rules/no-self-compare.js

@@ -18,7 +18,7 @@ module.exports = {
         docs: {
             description: "Disallow comparisons where both sides are exactly the same",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-self-compare"
+            url: "https://eslint.org/docs/latest/rules/no-self-compare"
         },
 
         schema: [],
@@ -29,7 +29,7 @@ module.exports = {
     },
 
     create(context) {
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         /**
          * Determines whether two nodes are composed of the same tokens.

package/lib/rules/no-sequences.js

@@ -31,7 +31,7 @@ module.exports = {
         docs: {
             description: "Disallow comma operators",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-sequences"
+            url: "https://eslint.org/docs/latest/rules/no-sequences"
         },
 
         schema: [{
@@ -51,7 +51,7 @@ module.exports = {
 
     create(context) {
         const options = Object.assign({}, DEFAULT_OPTIONS, context.options[0]);
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         /**
          * Parts of the grammar that are required to have parens.

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

@@ -144,7 +144,7 @@ module.exports = {
         docs: {
             description: "Disallow returning values from setters",
             recommended: true,
-            url: "https://eslint.org/docs/rules/no-setter-return"
+            url: "https://eslint.org/docs/latest/rules/no-setter-return"
         },
 
         schema: [],
@@ -156,7 +156,7 @@ module.exports = {
 
     create(context) {
         let funcInfo = null;
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         /**
          * Creates and pushes to the stack a function info object for the given function node.

package/lib/rules/no-shadow-restricted-names.js

@@ -29,7 +29,7 @@ module.exports = {
         docs: {
             description: "Disallow identifiers from shadowing restricted names",
             recommended: true,
-            url: "https://eslint.org/docs/rules/no-shadow-restricted-names"
+            url: "https://eslint.org/docs/latest/rules/no-shadow-restricted-names"
         },
 
         schema: [],
@@ -43,7 +43,7 @@ module.exports = {
 
 
         const RESTRICTED = new Set(["undefined", "NaN", "Infinity", "arguments", "eval"]);
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         return {
             "VariableDeclaration, :function, CatchClause"(node) {