[Feat]: provide compatibility with eslint v9

.github/workflows/node-18+.yml

@@ -26,6 +26,7 @@ jobs:
       matrix:
         node-version: ${{ fromJson(needs.matrix.outputs.latest) }}
         eslint:
+          - 9
           - 8
           - 7
           - 6
@@ -48,6 +49,7 @@ jobs:
             npm install --no-save "eslint@${{ matrix.eslint }}" "@typescript-eslint/parser@5" "babel-eslint@${{ matrix.babel-eslint }}"
         env:
           NPM_CONFIG_LEGACY_PEER_DEPS: true
+          ESLINT_USE_FLAT_CONFIG: false
       - run: npx ls-engines
       - run: npm run unit-test
       - uses: codecov/[email protected]

lib/rules/destructuring-assignment.js

@@ -4,6 +4,7 @@
 
 'use strict';
 
+const astUtil = require('../util/ast');
 const Components = require('../util/Components');
 const docsUrl = require('../util/docsUrl');
 const isAssignmentLHS = require('../util/ast').isAssignmentLHS;
@@ -101,7 +102,7 @@ module.exports = {
     function handleStatelessComponent(node) {
       const params = evalParams(node.params);
 
-      const SFCComponent = components.get(context.getScope(node).block);
+      const SFCComponent = components.get(astUtil.getScope(context, node).block);
       if (!SFCComponent) {
         return;
       }
@@ -119,7 +120,7 @@ module.exports = {
     }
 
     function handleStatelessComponentExit(node) {
-      const SFCComponent = components.get(context.getScope(node).block);
+      const SFCComponent = components.get(astUtil.getScope(context, node).block);
       if (SFCComponent) {
         sfcParams.pop();
       }
@@ -191,7 +192,7 @@ module.exports = {
       'FunctionExpression:exit': handleStatelessComponentExit,
 
       MemberExpression(node) {
-        let scope = context.getScope(node);
+        let scope = astUtil.getScope(context, node);
         let SFCComponent = components.get(scope.block);
         while (!SFCComponent && scope.upper && scope.upper !== scope) {
           SFCComponent = components.get(scope.upper.block);
@@ -209,7 +210,7 @@ module.exports = {
 
       VariableDeclarator(node) {
         const classComponent = utils.getParentComponent(node);
-        const SFCComponent = components.get(context.getScope(node).block);
+        const SFCComponent = components.get(astUtil.getScope(context, node).block);
 
         const destructuring = (node.init && node.id && node.id.type === 'ObjectPattern');
         // let {foo} = props;
@@ -247,7 +248,8 @@ module.exports = {
           && destructureInSignature === 'always'
           && node.init.name === 'props'
         ) {
-          const scopeSetProps = context.getScope().set.get('props');
+          const scope = astUtil.getScope(context, node);
+          const scopeSetProps = scope.set.get('props');
           const propsRefs = scopeSetProps && scopeSetProps.references;
           if (!propsRefs) {
             return;

lib/rules/forbid-prop-types.js

@@ -162,7 +162,7 @@ module.exports = {
           checkProperties(node.properties);
           break;
         case 'Identifier': {
-          const propTypesObject = variableUtil.findVariableByName(context, node.name);
+          const propTypesObject = variableUtil.findVariableByName(context, node);
           if (propTypesObject && propTypesObject.properties) {
             checkProperties(propTypesObject.properties);
           }

lib/rules/jsx-fragments.js

@@ -103,8 +103,8 @@ module.exports = {
       };
     }
 
-    function refersToReactFragment(name) {
-      const variableInit = variableUtil.findVariableByName(context, name);
+    function refersToReactFragment(node, name) {
+      const variableInit = variableUtil.findVariableByName(context, node, name);
       if (!variableInit) {
         return false;
       }
@@ -185,7 +185,7 @@ module.exports = {
           const openingEl = node.openingElement;
           const elName = elementType(openingEl);
 
-          if (fragmentNames.has(elName) || refersToReactFragment(elName)) {
+          if (fragmentNames.has(elName) || refersToReactFragment(node, elName)) {
             if (reportOnReactVersion(node)) {
               return;
             }

lib/rules/jsx-max-depth.js

@@ -149,7 +149,7 @@ module.exports = {
           return;
         }
 
-        const variables = variableUtil.variablesInScope(context);
+        const variables = variableUtil.variablesInScope(context, node);
         const element = findJSXElementOrFragment(variables, node.expression.name, []);
 
         if (element) {

lib/rules/jsx-no-bind.js

@@ -8,6 +8,7 @@
 'use strict';
 
 const propName = require('jsx-ast-utils/propName');
+const astUtil = require('../util/ast');
 const docsUrl = require('../util/docsUrl');
 const jsxUtil = require('../util/jsx');
 const report = require('../util/report');
@@ -123,7 +124,7 @@ module.exports = {
     }
 
     function getBlockStatementAncestors(node) {
-      return context.getAncestors(node).filter(
+      return astUtil.getAncestors(context, node).filter(
         (ancestor) => ancestor.type === 'BlockStatement'
       ).reverse();
     }

lib/rules/jsx-no-constructed-context-values.js

@@ -6,6 +6,7 @@
 
 'use strict';
 
+const astUtil = require('../util/ast');
 const Components = require('../util/Components');
 const docsUrl = require('../util/docsUrl');
 const report = require('../util/report');
@@ -179,7 +180,7 @@ module.exports = {
         }
 
         const valueExpression = valueNode.expression;
-        const invocationScope = context.getScope();
+        const invocationScope = astUtil.getScope(context, node);
 
         // Check if the value prop is a construction
         const constructInfo = isConstruction(valueExpression, invocationScope);

lib/rules/jsx-no-undef.js

@@ -5,6 +5,7 @@
 
 'use strict';
 
+const astUtil = require('../util/ast');
 const docsUrl = require('../util/docsUrl');
 const jsxUtil = require('../util/jsx');
 const report = require('../util/report');
@@ -49,7 +50,7 @@ module.exports = {
      * @returns {void}
      */
     function checkIdentifierInJSX(node) {
-      let scope = context.getScope();
+      let scope = astUtil.getScope(context, node);
       const sourceCode = context.getSourceCode();
       const sourceType = sourceCode.ast.sourceType;
       const scopeUpperBound = !allowGlobals && sourceType === 'module' ? 'module' : 'global';

lib/rules/jsx-sort-default-props.js

@@ -87,11 +87,12 @@ module.exports = {
 
     /**
      * Find a variable by name in the current scope.
-     * @param  {string} name Name of the variable to look for.
+     * @param {ASTNode} node The node to check.
      * @returns {ASTNode|null} Return null if the variable could not be found, ASTNode otherwise.
      */
-    function findVariableByName(name) {
-      const variable = variableUtil.variablesInScope(context).find((item) => item.name === name);
+    function findVariableByName(node) {
+      const name = node.name;
+      const variable = variableUtil.variablesInScope(context, node).find((item) => item.name === name);
 
       if (!variable || !variable.defs[0] || !variable.defs[0].node) {
         return null;
@@ -147,7 +148,7 @@ module.exports = {
       if (node.type === 'ObjectExpression') {
         checkSorted(node.properties);
       } else if (node.type === 'Identifier') {
-        const propTypesObject = findVariableByName(node.name);
+        const propTypesObject = findVariableByName(node);
         if (propTypesObject && propTypesObject.properties) {
           checkSorted(propTypesObject.properties);
         }

lib/rules/no-access-state-in-setstate.js

@@ -5,6 +5,7 @@
 
 'use strict';
 
+const astUtil = require('../util/ast');
 const docsUrl = require('../util/docsUrl');
 const componentUtil = require('../util/componentUtil');
 const report = require('../util/report');
@@ -47,8 +48,9 @@ module.exports = {
       return current.arguments[0] === node;
     }
 
-    function isClassComponent() {
-      return !!(componentUtil.getParentES6Component(context) || componentUtil.getParentES5Component(context));
+    function isClassComponent(node) {
+      return !!(componentUtil.getParentES6Component(context, node)
+        || componentUtil.getParentES5Component(context, node));
     }
 
     // The methods array contains all methods or functions that are using this.state
@@ -58,7 +60,7 @@ module.exports = {
     const vars = [];
     return {
       CallExpression(node) {
-        if (!isClassComponent()) {
+        if (!isClassComponent(node)) {
           return;
         }
         // Appends all the methods that are calling another
@@ -103,7 +105,7 @@ module.exports = {
         if (
           node.property.name === 'state'
           && node.object.type === 'ThisExpression'
-          && isClassComponent()
+          && isClassComponent(node)
         ) {
           let current = node;
           while (current.type !== 'Program') {
@@ -134,7 +136,7 @@ module.exports = {
             if (current.type === 'VariableDeclarator') {
               vars.push({
                 node,
-                scope: context.getScope(),
+                scope: astUtil.getScope(context, node),
                 variableName: current.id.name,
               });
               break;
@@ -155,10 +157,11 @@ module.exports = {
           current.parent.value === current
           || current.parent.object === current
         ) {
+          const scope = astUtil.getScope(context, node);
           while (current.type !== 'Program') {
             if (isFirstArgumentInSetStateCall(current, node)) {
               vars
-                .filter((v) => v.scope === context.getScope() && v.variableName === node.name)
+                .filter((v) => v.scope === scope && v.variableName === node.name)
                 .forEach((v) => {
                   report(context, messages.useCallback, 'useCallback', {
                     node: v.node,
@@ -176,7 +179,7 @@ module.exports = {
           if (property && property.key && property.key.name === 'state' && isDerivedFromThis) {
             vars.push({
               node: property.key,
-              scope: context.getScope(),
+              scope: astUtil.getScope(context, node),
               variableName: property.key.name,
             });
           }

lib/rules/no-array-index-key.js

@@ -28,7 +28,7 @@ function isCreateCloneElement(node, context) {
   }
 
   if (node.type === 'Identifier') {
-    const variable = variableUtil.findVariableByName(context, node.name);
+    const variable = variableUtil.findVariableByName(context, node);
     if (variable && variable.type === 'ImportSpecifier') {
       return variable.parent.source.value === 'react';
     }

lib/rules/no-danger-with-children.js

@@ -31,8 +31,8 @@ module.exports = {
     schema: [], // no options
   },
   create(context) {
-    function findSpreadVariable(name) {
-      return variableUtil.variablesInScope(context).find((item) => item.name === name);
+    function findSpreadVariable(name, node) {
+      return variableUtil.variablesInScope(context, node).find((item) => item.name === name);
     }
     /**
      * Takes a ObjectExpression and returns the value of the prop if it has it
@@ -50,7 +50,7 @@ module.exports = {
           return prop.key.name === propName;
         }
         if (prop.type === 'ExperimentalSpreadProperty' || prop.type === 'SpreadElement') {
-          const variable = findSpreadVariable(prop.argument.name);
+          const variable = findSpreadVariable(prop.argument.name, node);
           if (variable && variable.defs.length && variable.defs[0].node.init) {
             if (seenProps.indexOf(prop.argument.name) > -1) {
               return false;
@@ -73,7 +73,7 @@ module.exports = {
       const attributes = node.openingElement.attributes;
       return attributes.find((attribute) => {
         if (attribute.type === 'JSXSpreadAttribute') {
-          const variable = findSpreadVariable(attribute.argument.name);
+          const variable = findSpreadVariable(attribute.argument.name, node);
           if (variable && variable.defs.length && variable.defs[0].node.init) {
             return findObjectProp(variable.defs[0].node.init, propName, []);
           }
@@ -127,7 +127,7 @@ module.exports = {
           let props = node.arguments[1];
 
           if (props.type === 'Identifier') {
-            const variable = variableUtil.variablesInScope(context).find((item) => item.name === props.name);
+            const variable = variableUtil.variablesInScope(context, node).find((item) => item.name === props.name);
             if (variable && variable.defs.length && variable.defs[0].node.init) {
               props = variable.defs[0].node.init;
             }

lib/rules/no-direct-mutation-state.js

@@ -97,7 +97,7 @@ module.exports = {
       },
 
       AssignmentExpression(node) {
-        const component = components.get(utils.getParentComponent());
+        const component = components.get(utils.getParentComponent(node));
         if (shouldIgnoreComponent(component) || !node.left || !node.left.object) {
           return;
         }
@@ -113,7 +113,7 @@ module.exports = {
       },
 
       UpdateExpression(node) {
-        const component = components.get(utils.getParentComponent());
+        const component = components.get(utils.getParentComponent(node));
         if (shouldIgnoreComponent(component) || node.argument.type !== 'MemberExpression') {
           return;
         }

lib/rules/no-is-mounted.js

@@ -5,6 +5,7 @@
 
 'use strict';
 
+const astUtil = require('../util/ast');
 const docsUrl = require('../util/docsUrl');
 const report = require('../util/report');
 
@@ -40,7 +41,7 @@ module.exports = {
         if (callee.object.type !== 'ThisExpression' || callee.property.name !== 'isMounted') {
           return;
         }
-        const ancestors = context.getAncestors(callee);
+        const ancestors = astUtil.getAncestors(context, callee);
         for (let i = 0, j = ancestors.length; i < j; i++) {
           if (ancestors[i].type === 'Property' || ancestors[i].type === 'MethodDefinition') {
             report(context, messages.noIsMounted, 'noIsMounted', {

lib/rules/no-set-state.js

@@ -67,7 +67,7 @@ module.exports = {
         ) {
           return;
         }
-        const component = components.get(utils.getParentComponent());
+        const component = components.get(utils.getParentComponent(node));
         const setStateUsages = (component && component.setStateUsages) || [];
         setStateUsages.push(callee);
         components.set(node, {

lib/rules/no-string-refs.js

@@ -49,7 +49,7 @@ module.exports = {
      */
     function isRefsUsage(node) {
       return !!(
-        (componentUtil.getParentES6Component(context) || componentUtil.getParentES5Component(context))
+        (componentUtil.getParentES6Component(context, node) || componentUtil.getParentES5Component(context, node))
         && node.object.type === 'ThisExpression'
         && node.property.name === 'refs'
       );

lib/rules/no-this-in-sfc.js

@@ -33,7 +33,7 @@ module.exports = {
   create: Components.detect((context, components, utils) => ({
     MemberExpression(node) {
       if (node.object.type === 'ThisExpression') {
-        const component = components.get(utils.getParentStatelessComponent());
+        const component = components.get(utils.getParentStatelessComponent(node));
         if (!component || (component.node && component.node.parent && component.node.parent.type === 'Property')) {
           return;
         }

lib/rules/no-unstable-nested-components.js

@@ -305,7 +305,7 @@ module.exports = {
      * @returns {Boolean} True if node is inside class component's render block, false if not
      */
     function isInsideRenderMethod(node) {
-      const parentComponent = utils.getParentComponent();
+      const parentComponent = utils.getParentComponent(node);
 
       if (!parentComponent || parentComponent.type !== 'ClassDeclaration') {
         return false;
@@ -333,8 +333,8 @@ module.exports = {
      * @returns {Boolean} True if given node a function component declared inside class component, false if not
      */
     function isFunctionComponentInsideClassComponent(node) {
-      const parentComponent = utils.getParentComponent();
-      const parentStatelessComponent = utils.getParentStatelessComponent();
+      const parentComponent = utils.getParentComponent(node);
+      const parentStatelessComponent = utils.getParentStatelessComponent(node);
 
       return (
         parentComponent