[Feat]: provide compatibility with eslint v9

jsx-eslint · Apr 7, 2024 · cbe8e69 · cbe8e69
1 parent e4ecbcf
commit cbe8e69
Show file tree

Hide file tree

Showing 39 changed files with 158 additions and 129 deletions.
diff --git a/lib/rules/destructuring-assignment.js b/lib/rules/destructuring-assignment.js
@@ -101,7 +101,7 @@ module.exports = {
     function handleStatelessComponent(node) {
       const params = evalParams(node.params);
 
-      const SFCComponent = components.get(context.getScope(node).block);
+      const SFCComponent = components.get((context.sourceCode || context).getScope(node).block);
       if (!SFCComponent) {
         return;
       }
@@ -119,7 +119,7 @@ module.exports = {
     }
 
     function handleStatelessComponentExit(node) {
-      const SFCComponent = components.get(context.getScope(node).block);
+      const SFCComponent = components.get((context.sourceCode || context).getScope(node).block);
       if (SFCComponent) {
         sfcParams.pop();
       }
@@ -191,7 +191,7 @@ module.exports = {
       'FunctionExpression:exit': handleStatelessComponentExit,
 
       MemberExpression(node) {
-        let scope = context.getScope(node);
+        let scope = (context.sourceCode || context).getScope(node);
         let SFCComponent = components.get(scope.block);
         while (!SFCComponent && scope.upper && scope.upper !== scope) {
           SFCComponent = components.get(scope.upper.block);
@@ -209,7 +209,7 @@ module.exports = {
 
       VariableDeclarator(node) {
         const classComponent = utils.getParentComponent(node);
-        const SFCComponent = components.get(context.getScope(node).block);
+        const SFCComponent = components.get((context.sourceCode || context).getScope(node).block);
 
         const destructuring = (node.init && node.id && node.id.type === 'ObjectPattern');
         // let {foo} = props;
@@ -247,7 +247,8 @@ module.exports = {
           && destructureInSignature === 'always'
           && node.init.name === 'props'
         ) {
-          const scopeSetProps = context.getScope().set.get('props');
+          const scope = (context.sourceCode || context).getScope(node);
+          const scopeSetProps = scope.set.get('props');
           const propsRefs = scopeSetProps && scopeSetProps.references;
           if (!propsRefs) {
             return;

diff --git a/lib/rules/forbid-prop-types.js b/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);
           }

diff --git a/lib/rules/jsx-fragments.js b/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;
             }

diff --git a/lib/rules/jsx-max-depth.js b/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) {

diff --git a/lib/rules/jsx-no-constructed-context-values.js b/lib/rules/jsx-no-constructed-context-values.js
@@ -179,7 +179,7 @@ module.exports = {
         }
 
         const valueExpression = valueNode.expression;
-        const invocationScope = context.getScope();
+        const invocationScope = (context.sourceCode || context).getScope(node);
 
         // Check if the value prop is a construction
         const constructInfo = isConstruction(valueExpression, invocationScope);

diff --git a/lib/rules/jsx-no-undef.js b/lib/rules/jsx-no-undef.js
@@ -49,7 +49,7 @@ module.exports = {
      * @returns {void}
      */
     function checkIdentifierInJSX(node) {
-      let scope = context.getScope();
+      let scope = (context.sourceCode || context).getScope(node);
       const sourceCode = context.getSourceCode();
       const sourceType = sourceCode.ast.sourceType;
       const scopeUpperBound = !allowGlobals && sourceType === 'module' ? 'module' : 'global';

diff --git a/lib/rules/jsx-sort-default-props.js b/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);
         }

diff --git a/lib/rules/no-access-state-in-setstate.js b/lib/rules/no-access-state-in-setstate.js
@@ -47,8 +47,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 +59,7 @@ module.exports = {
     const vars = [];
     return {
       CallExpression(node) {
-        if (!isClassComponent()) {
+        if (!isClassComponent(node)) {
           return;
         }
         // Appends all the methods that are calling another
@@ -103,7 +104,7 @@ module.exports = {
         if (
           node.property.name === 'state'
           && node.object.type === 'ThisExpression'
-          && isClassComponent()
+          && isClassComponent(node)
         ) {
           let current = node;
           while (current.type !== 'Program') {
@@ -134,7 +135,7 @@ module.exports = {
             if (current.type === 'VariableDeclarator') {
               vars.push({
                 node,
-                scope: context.getScope(),
+                scope: (context.sourceCode || context).getScope(node),
                 variableName: current.id.name,
               });
               break;
@@ -155,10 +156,11 @@ module.exports = {
           current.parent.value === current
           || current.parent.object === current
         ) {
+          const scope = (context.sourceCode || context).getScope(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 +178,7 @@ module.exports = {
           if (property && property.key && property.key.name === 'state' && isDerivedFromThis) {
             vars.push({
               node: property.key,
-              scope: context.getScope(),
+              scope: (context.sourceCode || context).getScope(node),
               variableName: property.key.name,
             });
           }

diff --git a/lib/rules/no-array-index-key.js b/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';
     }

diff --git a/lib/rules/no-danger-with-children.js b/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;
             }

diff --git a/lib/rules/no-direct-mutation-state.js b/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;
         }

diff --git a/lib/rules/no-set-state.js b/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, {

diff --git a/lib/rules/no-string-refs.js b/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'
       );

diff --git a/lib/rules/no-this-in-sfc.js b/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;
         }

diff --git a/lib/rules/no-unstable-nested-components.js b/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

diff --git a/lib/rules/no-unused-state.js b/lib/rules/no-unused-state.js
@@ -107,7 +107,7 @@ module.exports = {
         'componentDidUpdate',
       ];
 
-      let scope = context.getScope();
+      let scope = (context.sourceCode || context).getScope(node);
       while (scope) {
         const parent = scope.block && scope.block.parent;
         if (
@@ -368,7 +368,8 @@ module.exports = {
           return;
         }
 
-        const childScope = context.getScope().childScopes.find((x) => x.block === node.value);
+        const nodeScope = (context.sourceCode || context).getScope(node);
+        const childScope = nodeScope.childScopes.find((x) => x.block === node.value);
         if (!childScope) {
           return;
         }

diff --git a/lib/rules/prefer-exact-props.js b/lib/rules/prefer-exact-props.js
@@ -147,8 +147,7 @@ module.exports = {
         } else if (isNonExactPropWrapperFunction(right)) {
           reportPropTypesError(node);
         } else if (right.type === 'Identifier') {
-          const identifier = right.name;
-          const propsDefinition = variableUtil.findVariableByName(context, identifier);
+          const propsDefinition = variableUtil.findVariableByName(context, right);
           if (isNonEmptyObjectExpression(propsDefinition)) {
             reportPropTypesError(node);
           } else if (isNonExactPropWrapperFunction(propsDefinition)) {

diff --git a/lib/rules/prefer-stateless-function.js b/lib/rules/prefer-stateless-function.js
@@ -341,7 +341,7 @@ module.exports = {
       // Mark `render` that do not return some JSX
       ReturnStatement(node) {
         let blockNode;
-        let scope = context.getScope();
+        let scope = (context.sourceCode || context).getScope(node);
         while (scope) {
           blockNode = scope.block && scope.block.parent;
           if (blockNode && (blockNode.type === 'MethodDefinition' || blockNode.type === 'Property')) {

diff --git a/lib/rules/react-in-jsx-scope.js b/lib/rules/react-in-jsx-scope.js
@@ -36,7 +36,7 @@ module.exports = {
     const pragma = pragmaUtil.getFromContext(context);
 
     function checkIfReactIsInScope(node) {
-      const variables = variableUtil.variablesInScope(context);
+      const variables = variableUtil.variablesInScope(context, node);
       if (variableUtil.findVariable(variables, pragma)) {
         return;
       }

diff --git a/lib/rules/require-optimization.js b/lib/rules/require-optimization.js
@@ -153,11 +153,12 @@ module.exports = {
 
     /**
      * Checks if we are declaring function in class
+     * @param {ASTNode} node The AST node being checked.
      * @returns {Boolean} True if we are declaring function in class, false if not.
      */
-    function isFunctionInClass() {
+    function isFunctionInClass(node) {
       let blockNode;
-      let scope = context.getScope();
+      let scope = (context.sourceCode || context).getScope(node);
       while (scope) {
         blockNode = scope.block;
         if (blockNode && blockNode.type === 'ClassDeclaration') {
@@ -172,7 +173,7 @@ module.exports = {
     return {
       ArrowFunctionExpression(node) {
         // Skip if the function is declared in the class
-        if (isFunctionInClass()) {
+        if (isFunctionInClass(node)) {
           return;
         }
         // Stateless Functional Components cannot be optimized (yet)
@@ -192,7 +193,7 @@ module.exports = {
 
       FunctionDeclaration(node) {
         // Skip if the function is declared in the class
-        if (isFunctionInClass()) {
+        if (isFunctionInClass(node)) {
           return;
         }
         // Stateless Functional Components cannot be optimized (yet)
@@ -201,7 +202,7 @@ module.exports = {
 
       FunctionExpression(node) {
         // Skip if the function is declared in the class
-        if (isFunctionInClass()) {
+        if (isFunctionInClass(node)) {
           return;
         }
         // Stateless Functional Components cannot be optimized (yet)

diff --git a/lib/rules/require-render-return.js b/lib/rules/require-render-return.js
@@ -60,7 +60,7 @@ module.exports = {
 
     return {
       ReturnStatement(node) {
-        const ancestors = context.getAncestors(node).reverse();
+        const ancestors = (context.sourceCode || context).getAncestors(node).reverse();
         let depth = 0;
         ancestors.forEach((ancestor) => {
           if (/Function(Expression|Declaration)$/.test(ancestor.type)) {