Revert "[Fix] destructuring-assignment: Handle destructuring of useContext in SFC"

This reverts commit 523db20.

Author: 102

lib/rules/destructuring-assignment.js

@@ -8,7 +8,6 @@ const Components = require('../util/Components');
 const docsUrl = require('../util/docsUrl');
 const isAssignmentLHS = require('../util/ast').isAssignmentLHS;
 const report = require('../util/report');
-const testReactVersion = require('../util/version').testReactVersion;
 
 const DEFAULT_OPTION = 'always';
 
@@ -95,8 +94,6 @@ module.exports = {
     const destructureInSignature = (context.options[1] && context.options[1].destructureInSignature) || 'ignore';
     const sfcParams = createSFCParams();
 
-    // set to save renamed var of useContext
-    const contextSet = new Set();
     /**
      * @param {ASTNode} node We expect either an ArrowFunctionExpression,
      *   FunctionDeclaration, or FunctionExpression
@@ -131,7 +128,7 @@ module.exports = {
     function handleSFCUsage(node) {
       const propsName = sfcParams.propsName();
       const contextName = sfcParams.contextName();
-      // props.aProp
+      // props.aProp || context.aProp
       const isPropUsed = (
         (propsName && node.object.name === propsName)
           || (contextName && node.object.name === contextName)
@@ -145,21 +142,6 @@ module.exports = {
           },
         });
       }
-
-      // const foo = useContext(aContext);
-      // foo.aProp
-      const isContextUsed = contextSet.has(node.object.name) && !isAssignmentLHS(node);
-      const optional = node.optional
-        // the below is for the old typescript-eslint parser
-        || context.getSourceCode().getText(node).slice(node.object.range[1] - node.range[0], node.object.range[1] - node.range[0] + 1) === '?';
-      if (isContextUsed && configuration === 'always' && !optional) {
-        report(context, messages.useDestructAssignment, 'useDestructAssignment', {
-          node,
-          data: {
-            type: node.object.name,
-          },
-        });
-      }
     }
 
     function isInClassProperty(node) {
@@ -194,9 +176,8 @@ module.exports = {
       }
     }
 
-    const hasHooks = testReactVersion(context, '>= 16.9');
-
     return {
+
       FunctionDeclaration: handleStatelessComponent,
 
       ArrowFunctionExpression: handleStatelessComponent,
@@ -231,31 +212,13 @@ module.exports = {
         const SFCComponent = components.get(context.getScope(node).block);
 
         const destructuring = (node.init && node.id && node.id.type === 'ObjectPattern');
-        const identifier = (node.init && node.id && node.id.type === 'Identifier');
         // let {foo} = props;
-        const destructuringSFC = destructuring && node.init.name === 'props';
-        // let {foo} = useContext(aContext);
-        const destructuringUseContext = hasHooks && destructuring && node.init.callee && node.init.callee.name === 'useContext';
-        // let foo = useContext(aContext);
-        const assignUseContext = hasHooks && identifier && node.init.callee && node.init.callee.name === 'useContext';
+        const destructuringSFC = destructuring && (node.init.name === 'props' || node.init.name === 'context');
         // let {foo} = this.props;
         const destructuringClass = destructuring && node.init.object && node.init.object.type === 'ThisExpression' && (
           node.init.property.name === 'props' || node.init.property.name === 'context' || node.init.property.name === 'state'
         );
 
-        if (SFCComponent && assignUseContext) {
-          contextSet.add(node.id.name);
-        }
-
-        if (SFCComponent && destructuringUseContext && configuration === 'never') {
-          report(context, messages.noDestructAssignment, 'noDestructAssignment', {
-            node,
-            data: {
-              type: node.init.callee.name,
-            },
-          });
-        }
-
         if (SFCComponent && destructuringSFC && configuration === 'never') {
           report(context, messages.noDestructAssignment, 'noDestructAssignment', {
             node,

tests/lib/rules/destructuring-assignment.js

@@ -860,36 +860,6 @@ ruleTester.run('destructuring-assignment', rule, {
         `,
         features: ['ts', 'no-babel'],
       },
-    ] : [],
-    {
-      code: `
-        import { useContext } from 'react';
-
-        const MyComponent = (props) => {
-          const foo = useContext(aContext);
-          return <div>{foo.test}</div>
-        };
-      `,
-      options: ['always'],
-      settings: { react: { version: '16.9.0' } },
-      errors: [
-        { message: 'Must use destructuring foo assignment' },
-      ],
-    },
-    {
-      code: `
-        import { useContext } from 'react';
-
-        const MyComponent = (props) => {
-          const {foo} = useContext(aContext);
-          return <div>{foo}</div>
-        };
-      `,
-      options: ['never'],
-      settings: { react: { version: '16.9.0' } },
-      errors: [
-        { message: 'Must never use destructuring useContext assignment' },
-      ],
-    }
+    ] : []
   )),
 });