eslint 8.38.0 → 8.40.0

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

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

@@ -51,7 +51,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: {

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

@@ -21,7 +21,7 @@ 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,
@@ -44,14 +44,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) {

package/lib/rules/no-shadow.js

@@ -32,7 +32,7 @@ module.exports = {
         docs: {
             description: "Disallow variable declarations from shadowing variables declared in the outer scope",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-shadow"
+            url: "https://eslint.org/docs/latest/rules/no-shadow"
         },
 
         schema: [
@@ -67,7 +67,7 @@ module.exports = {
             allow: (context.options[0] && context.options[0].allow) || [],
             ignoreOnInitialization: context.options[0] && context.options[0].ignoreOnInitialization
         };
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         /**
          * Checks whether or not a given location is inside of the range of a given node.

package/lib/rules/no-spaced-func.js

@@ -18,7 +18,7 @@ module.exports = {
         docs: {
             description: "Disallow spacing between function identifiers and their applications (deprecated)",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-spaced-func"
+            url: "https://eslint.org/docs/latest/rules/no-spaced-func"
         },
 
         deprecated: true,
@@ -35,7 +35,7 @@ module.exports = {
 
     create(context) {
 
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         /**
          * Check if open space is present in a function name

package/lib/rules/no-sparse-arrays.js

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

package/lib/rules/no-sync.js

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

package/lib/rules/no-tabs.js

@@ -24,7 +24,7 @@ module.exports = {
         docs: {
             description: "Disallow all tabs",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-tabs"
+            url: "https://eslint.org/docs/latest/rules/no-tabs"
         },
         schema: [{
             type: "object",
@@ -43,7 +43,7 @@ module.exports = {
     },
 
     create(context) {
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
         const allowIndentationTabs = context.options && context.options[0] && context.options[0].allowIndentationTabs;
 
         return {

package/lib/rules/no-template-curly-in-string.js

@@ -16,7 +16,7 @@ module.exports = {
         docs: {
             description: "Disallow template literal placeholder syntax in regular strings",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-template-curly-in-string"
+            url: "https://eslint.org/docs/latest/rules/no-template-curly-in-string"
         },
 
         schema: [],

package/lib/rules/no-ternary.js

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

package/lib/rules/no-this-before-super.js

@@ -42,7 +42,7 @@ module.exports = {
         docs: {
             description: "Disallow `this`/`super` before calling `super()` in constructors",
             recommended: true,
-            url: "https://eslint.org/docs/rules/no-this-before-super"
+            url: "https://eslint.org/docs/latest/rules/no-this-before-super"
         },
 
         schema: [],

package/lib/rules/no-throw-literal.js

@@ -19,7 +19,7 @@ module.exports = {
         docs: {
             description: "Disallow throwing literals as exceptions",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-throw-literal"
+            url: "https://eslint.org/docs/latest/rules/no-throw-literal"
         },
 
         schema: [],

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

@@ -22,7 +22,7 @@ module.exports = {
         docs: {
             description: "Disallow trailing whitespace at the end of lines",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-trailing-spaces"
+            url: "https://eslint.org/docs/latest/rules/no-trailing-spaces"
         },
 
         fixable: "whitespace",
@@ -50,7 +50,7 @@ module.exports = {
     },
 
     create(context) {
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         const BLANK_CLASS = "[ \t\u00a0\u2000-\u200b\u3000]",
             SKIP_BLANK = `^${BLANK_CLASS}*$`,

package/lib/rules/no-undef-init.js

@@ -19,7 +19,7 @@ module.exports = {
         docs: {
             description: "Disallow initializing variables to `undefined`",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-undef-init"
+            url: "https://eslint.org/docs/latest/rules/no-undef-init"
         },
 
         schema: [],
@@ -32,7 +32,7 @@ module.exports = {
 
     create(context) {
 
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         return {
 

package/lib/rules/no-undef.js

@@ -31,7 +31,7 @@ module.exports = {
         docs: {
             description: "Disallow the use of undeclared variables unless mentioned in `/*global */` comments",
             recommended: true,
-            url: "https://eslint.org/docs/rules/no-undef"
+            url: "https://eslint.org/docs/latest/rules/no-undef"
         },
 
         schema: [
@@ -54,7 +54,7 @@ module.exports = {
     create(context) {
         const options = context.options[0];
         const considerTypeOf = options && options.typeof === true || false;
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         return {
             "Program:exit"(node) {

package/lib/rules/no-undefined.js

@@ -16,7 +16,7 @@ module.exports = {
         docs: {
             description: "Disallow the use of `undefined` as an identifier",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-undefined"
+            url: "https://eslint.org/docs/latest/rules/no-undefined"
         },
 
         schema: [],
@@ -28,7 +28,7 @@ module.exports = {
 
     create(context) {
 
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         /**
          * Report an invalid "undefined" identifier node.

package/lib/rules/no-underscore-dangle.js

@@ -17,7 +17,7 @@ module.exports = {
         docs: {
             description: "Disallow dangling underscores in identifiers",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-underscore-dangle"
+            url: "https://eslint.org/docs/latest/rules/no-underscore-dangle"
         },
 
         schema: [
@@ -84,7 +84,7 @@ module.exports = {
         const allowFunctionParams = typeof options.allowFunctionParams !== "undefined" ? options.allowFunctionParams : true;
         const allowInArrayDestructuring = typeof options.allowInArrayDestructuring !== "undefined" ? options.allowInArrayDestructuring : true;
         const allowInObjectDestructuring = typeof options.allowInObjectDestructuring !== "undefined" ? options.allowInObjectDestructuring : true;
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         //-------------------------------------------------------------------------
         // Helpers

package/lib/rules/no-unexpected-multiline.js

@@ -22,7 +22,7 @@ module.exports = {
         docs: {
             description: "Disallow confusing multiline expressions",
             recommended: true,
-            url: "https://eslint.org/docs/rules/no-unexpected-multiline"
+            url: "https://eslint.org/docs/latest/rules/no-unexpected-multiline"
         },
 
         schema: [],
@@ -38,7 +38,7 @@ module.exports = {
 
         const REGEX_FLAG_MATCHER = /^[gimsuy]+$/u;
 
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         /**
          * Check to see if there is a newline between the node and the following open bracket

package/lib/rules/no-unmodified-loop-condition.js

@@ -164,7 +164,7 @@ module.exports = {
         docs: {
             description: "Disallow unmodified loop conditions",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-unmodified-loop-condition"
+            url: "https://eslint.org/docs/latest/rules/no-unmodified-loop-condition"
         },
 
         schema: [],
@@ -175,7 +175,7 @@ module.exports = {
     },
 
     create(context) {
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
         let groupMap = null;
 
         /**

package/lib/rules/no-unneeded-ternary.js

@@ -31,7 +31,7 @@ module.exports = {
         docs: {
             description: "Disallow ternary operators when simpler alternatives exist",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-unneeded-ternary"
+            url: "https://eslint.org/docs/latest/rules/no-unneeded-ternary"
         },
 
         schema: [
@@ -58,7 +58,7 @@ module.exports = {
     create(context) {
         const options = context.options[0] || {};
         const defaultAssignment = options.defaultAssignment !== false;
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
 
         /**
          * Test if the node is a boolean literal

package/lib/rules/no-unreachable-loop.js

@@ -61,7 +61,7 @@ module.exports = {
         docs: {
             description: "Disallow loops with a body that allows only one iteration",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-unreachable-loop"
+            url: "https://eslint.org/docs/latest/rules/no-unreachable-loop"
         },
 
         schema: [{

package/lib/rules/no-unreachable.js

@@ -113,7 +113,7 @@ module.exports = {
         docs: {
             description: "Disallow unreachable code after `return`, `throw`, `continue`, and `break` statements",
             recommended: true,
-            url: "https://eslint.org/docs/rules/no-unreachable"
+            url: "https://eslint.org/docs/latest/rules/no-unreachable"
         },
 
         schema: [],
@@ -130,7 +130,7 @@ module.exports = {
         let constructorInfo = null;
 
         /** @type {ConsecutiveRange} */
-        const range = new ConsecutiveRange(context.getSourceCode());
+        const range = new ConsecutiveRange(context.sourceCode);
 
         /**
          * Reports a given node if it's unreachable.

package/lib/rules/no-unsafe-finally.js

@@ -26,7 +26,7 @@ module.exports = {
         docs: {
             description: "Disallow control flow statements in `finally` blocks",
             recommended: true,
-            url: "https://eslint.org/docs/rules/no-unsafe-finally"
+            url: "https://eslint.org/docs/latest/rules/no-unsafe-finally"
         },
 
         schema: [],

package/lib/rules/no-unsafe-negation.js

@@ -54,7 +54,7 @@ module.exports = {
         docs: {
             description: "Disallow negating the left operand of relational operators",
             recommended: true,
-            url: "https://eslint.org/docs/rules/no-unsafe-negation"
+            url: "https://eslint.org/docs/latest/rules/no-unsafe-negation"
         },
 
         hasSuggestions: true,
@@ -82,7 +82,7 @@ module.exports = {
     },
 
     create(context) {
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
         const options = context.options[0] || {};
         const enforceForOrderingRelations = options.enforceForOrderingRelations === true;
 

package/lib/rules/no-unsafe-optional-chaining.js

@@ -26,7 +26,7 @@ module.exports = {
         docs: {
             description: "Disallow use of optional chaining in contexts where the `undefined` value is not allowed",
             recommended: true,
-            url: "https://eslint.org/docs/rules/no-unsafe-optional-chaining"
+            url: "https://eslint.org/docs/latest/rules/no-unsafe-optional-chaining"
         },
         schema: [{
             type: "object",

package/lib/rules/no-unused-expressions.js

@@ -32,7 +32,7 @@ module.exports = {
         docs: {
             description: "Disallow unused expressions",
             recommended: false,
-            url: "https://eslint.org/docs/rules/no-unused-expressions"
+            url: "https://eslint.org/docs/latest/rules/no-unused-expressions"
         },
 
         schema: [
@@ -70,8 +70,7 @@ module.exports = {
             allowShortCircuit = config.allowShortCircuit || false,
             allowTernary = config.allowTernary || false,
             allowTaggedTemplates = config.allowTaggedTemplates || false,
-            enforceForJSX = config.enforceForJSX || false,
-            sourceCode = context.getSourceCode();
+            enforceForJSX = config.enforceForJSX || false;
 
         /**
          * Has AST suggesting a directive.
@@ -110,12 +109,11 @@ module.exports = {
         /**
          * Detect if a Node is a directive.
          * @param {ASTNode} node any node
-         * @param {ASTNode[]} ancestors the given node's ancestors
          * @returns {boolean} whether the given node is considered a directive in its current position
          */
-        function isDirective(node, ancestors) {
-            const parent = ancestors[ancestors.length - 1],
-                grandparent = ancestors[ancestors.length - 2];
+        function isDirective(node) {
+            const parent = node.parent,
+                grandparent = parent.parent;
 
             /**
              * https://tc39.es/ecma262/#directive-prologue
@@ -181,7 +179,7 @@ module.exports = {
 
         return {
             ExpressionStatement(node) {
-                if (Checker.isDisallowed(node.expression) && !isDirective(node, sourceCode.getAncestors(node))) {
+                if (Checker.isDisallowed(node.expression) && !isDirective(node)) {
                     context.report({ node, messageId: "unusedExpression" });
                 }
             }