eslint-plugin-react-hooks 1.5.0-beta.0 → 1.6.0

package/cjs/eslint-plugin-react-hooks.development.js

@@ -760,17 +760,25 @@ var ExhaustiveDeps = {
       // So no need to check for dependency inclusion.
       var depsIndex = callbackIndex + 1;
       var declaredDependenciesNode = node.parent.arguments[depsIndex];
-      if (!declaredDependenciesNode) {
+      if (!declaredDependenciesNode && !isEffect) {
         // These are only used for optimization.
         if (reactiveHookName === 'useMemo' || reactiveHookName === 'useCallback') {
+          // TODO: Can this have an autofix?
           context.report({
             node: node.parent.callee,
-            message: 'React Hook ' + reactiveHookName + ' doesn\'t serve any purpose ' + 'without a dependency array. To enable ' + 'this optimization, pass an array of values used by the ' + ('inner function as the second argument to ' + reactiveHookName + '.')
+            message: 'React Hook ' + reactiveHookName + ' does nothing when called with ' + 'only one argument. Did you forget to pass an array of ' + 'dependencies?'
           });
         }
         return;
       }
 
+      if (isEffect && node.async) {
+        context.report({
+          node: node,
+          message: 'Effect callbacks are synchronous to prevent race conditions. ' + 'Put the async function inside:\n\n' + 'useEffect(() => {\n' + '  async function fetchData() {\n' + '    // You can await here\n' + '    const response = await MyAPI.getData(someId);\n' + '    // ...\n' + '  }\n' + '  fetchData();\n' + '}, [someId]); // Or [] if effect doesn\'t need props or state\n\n' + 'Learn more about data fetching with Hooks: https://fb.me/react-hooks-data-fetching'
+        });
+      }
+
       // Get the current scope.
       var scope = context.getScope();
 
@@ -1014,6 +1022,12 @@ var ExhaustiveDeps = {
               });
             }
 
+            // Ignore references to the function itself as it's not defined yet.
+            var def = reference.resolved.defs[0];
+            if (def != null && def.node != null && def.node.init === node.parent) {
+              continue;
+            }
+
             // Add the dependency to a map so we can make sure it is referenced
             // again in our dependencies array. Remember whether it's static.
             if (!dependencies.has(dependency)) {
@@ -1096,10 +1110,99 @@ var ExhaustiveDeps = {
         }
         context.report({
           node: dependencyNode.parent.property,
-          message: 'Accessing \'' + dependency + '.current\' during the effect cleanup ' + 'will likely read a different ref value because by this time React ' + 'has already updated the ref. If this ref is managed by React, store ' + ('\'' + dependency + '.current\' in a variable inside ') + 'the effect itself and refer to that variable from the cleanup function.'
+          message: 'The ref value \'' + dependency + '.current\' will likely have ' + 'changed by the time this effect cleanup function runs. If ' + 'this ref points to a node rendered by React, copy ' + ('\'' + dependency + '.current\' to a variable inside the effect, and ') + 'use that variable in the cleanup function.'
+        });
+      });
+
+      // Warn about assigning to variables in the outer scope.
+      // Those are usually bugs.
+      var staleAssignments = new Set();
+      function reportStaleAssignment(writeExpr, key) {
+        if (staleAssignments.has(key)) {
+          return;
+        }
+        staleAssignments.add(key);
+        context.report({
+          node: writeExpr,
+          message: 'Assignments to the \'' + key + '\' variable from inside React Hook ' + (context.getSource(reactiveHook) + ' will be lost after each ') + 'render. To preserve the value over time, store it in a useRef ' + 'Hook and keep the mutable value in the \'.current\' property. ' + 'Otherwise, you can move this variable directly inside ' + (context.getSource(reactiveHook) + '.')
+        });
+      }
+
+      // Remember which deps are optional and report bad usage first.
+      var optionalDependencies = new Set();
+      dependencies.forEach(function (_ref2, key) {
+        var isStatic = _ref2.isStatic,
+            references = _ref2.references;
+
+        if (isStatic) {
+          optionalDependencies.add(key);
+        }
+        references.forEach(function (reference) {
+          if (reference.writeExpr) {
+            reportStaleAssignment(reference.writeExpr, key);
+          }
         });
       });
 
+      if (staleAssignments.size > 0) {
+        // The intent isn't clear so we'll wait until you fix those first.
+        return;
+      }
+
+      if (!declaredDependenciesNode) {
+        // Check if there are any top-level setState() calls.
+        // Those tend to lead to infinite loops.
+        var setStateInsideEffectWithoutDeps = null;
+        dependencies.forEach(function (_ref3, key) {
+          var isStatic = _ref3.isStatic,
+              references = _ref3.references;
+
+          if (setStateInsideEffectWithoutDeps) {
+            return;
+          }
+          references.forEach(function (reference) {
+            if (setStateInsideEffectWithoutDeps) {
+              return;
+            }
+
+            var id = reference.identifier;
+            var isSetState = setStateCallSites.has(id);
+            if (!isSetState) {
+              return;
+            }
+
+            var fnScope = reference.from;
+            while (fnScope.type !== 'function') {
+              fnScope = fnScope.upper;
+            }
+            var isDirectlyInsideEffect = fnScope.block === node;
+            if (isDirectlyInsideEffect) {
+              // TODO: we could potentially ignore early returns.
+              setStateInsideEffectWithoutDeps = key;
+            }
+          });
+        });
+        if (setStateInsideEffectWithoutDeps) {
+          var _collectRecommendatio = collectRecommendations({
+            dependencies: dependencies,
+            declaredDependencies: [],
+            optionalDependencies: optionalDependencies,
+            externalDependencies: new Set(),
+            isEffect: true
+          }),
+              _suggestedDependencies = _collectRecommendatio.suggestedDependencies;
+
+          context.report({
+            node: node.parent.callee,
+            message: 'React Hook ' + reactiveHookName + ' contains a call to \'' + setStateInsideEffectWithoutDeps + '\'. ' + 'Without a list of dependencies, this can lead to an infinite chain of updates. ' + 'To fix this, pass [' + _suggestedDependencies.join(', ') + ('] as a second argument to the ' + reactiveHookName + ' Hook.'),
+            fix: function (fixer) {
+              return fixer.insertTextAfter(node, ', [' + _suggestedDependencies.join(', ') + ']');
+            }
+          });
+        }
+        return;
+      }
+
       var declaredDependencies = [];
       var externalDependencies = new Set();
       if (declaredDependenciesNode.type !== 'ArrayExpression') {
@@ -1108,7 +1211,7 @@ var ExhaustiveDeps = {
         // the user this in an error.
         context.report({
           node: declaredDependenciesNode,
-          message: 'React Hook ' + context.getSource(reactiveHook) + ' has a second ' + "argument which is not an array literal. This means we can't " + "statically verify whether you've passed the correct dependencies."
+          message: 'React Hook ' + context.getSource(reactiveHook) + ' was passed a ' + 'dependency list that is not an array literal. This means we ' + "can't statically verify whether you've passed the correct " + 'dependencies.'
         });
       } else {
         declaredDependenciesNode.elements.forEach(function (declaredDependencyNode) {
@@ -1132,15 +1235,15 @@ var ExhaustiveDeps = {
           } catch (error) {
             if (/Unsupported node type/.test(error.message)) {
               if (declaredDependencyNode.type === 'Literal') {
-                if (typeof declaredDependencyNode.value === 'string') {
+                if (dependencies.has(declaredDependencyNode.value)) {
                   context.report({
                     node: declaredDependencyNode,
-                    message: 'The ' + declaredDependencyNode.raw + ' string literal is not a valid dependency ' + 'because it never changes. Did you mean to ' + ('include ' + declaredDependencyNode.value + ' in the array instead?')
+                    message: 'The ' + declaredDependencyNode.raw + ' literal is not a valid dependency ' + 'because it never changes. ' + ('Did you mean to include ' + declaredDependencyNode.value + ' in the array instead?')
                   });
                 } else {
                   context.report({
                     node: declaredDependencyNode,
-                    message: 'The \'' + declaredDependencyNode.raw + '\' literal is not a valid dependency ' + 'because it never changes. You can safely remove it.'
+                    message: 'The ' + declaredDependencyNode.raw + ' literal is not a valid dependency ' + 'because it never changes. You can safely remove it.'
                   });
                 }
               } else {
@@ -1176,51 +1279,17 @@ var ExhaustiveDeps = {
         });
       }
 
-      // Warn about assigning to variables in the outer scope.
-      // Those are usually bugs.
-      var staleAssignments = new Set();
-      function reportStaleAssignment(writeExpr, key) {
-        if (staleAssignments.has(key)) {
-          return;
-        }
-        staleAssignments.add(key);
-        context.report({
-          node: writeExpr,
-          message: 'Assignments to the \'' + key + '\' variable from inside a React ' + context.getSource(reactiveHook) + ' Hook ' + 'will not persist between re-renders. ' + 'If it\'s only needed by this Hook, move the variable inside it. ' + 'Alternatively, declare a ref with the useRef Hook, ' + 'and keep the mutable value in its \'current\' property.'
-        });
-      }
-
-      // Remember which deps are optional and report bad usage first.
-      var optionalDependencies = new Set();
-      dependencies.forEach(function (_ref2, key) {
-        var isStatic = _ref2.isStatic,
-            references = _ref2.references;
-
-        if (isStatic) {
-          optionalDependencies.add(key);
-        }
-        references.forEach(function (reference) {
-          if (reference.writeExpr) {
-            reportStaleAssignment(reference.writeExpr, key);
-          }
-        });
-      });
-      if (staleAssignments.size > 0) {
-        // The intent isn't clear so we'll wait until you fix those first.
-        return;
-      }
-
-      var _collectRecommendatio = collectRecommendations({
+      var _collectRecommendatio2 = collectRecommendations({
         dependencies: dependencies,
         declaredDependencies: declaredDependencies,
         optionalDependencies: optionalDependencies,
         externalDependencies: externalDependencies,
         isEffect: isEffect
       }),
-          suggestedDependencies = _collectRecommendatio.suggestedDependencies,
-          unnecessaryDependencies = _collectRecommendatio.unnecessaryDependencies,
-          missingDependencies = _collectRecommendatio.missingDependencies,
-          duplicateDependencies = _collectRecommendatio.duplicateDependencies;
+          suggestedDependencies = _collectRecommendatio2.suggestedDependencies,
+          unnecessaryDependencies = _collectRecommendatio2.unnecessaryDependencies,
+          missingDependencies = _collectRecommendatio2.missingDependencies,
+          duplicateDependencies = _collectRecommendatio2.duplicateDependencies;
 
       var problemCount = duplicateDependencies.size + missingDependencies.size + unnecessaryDependencies.size;
 
@@ -1233,17 +1302,20 @@ var ExhaustiveDeps = {
           componentScope: componentScope,
           scope: scope
         });
-        bareFunctions.forEach(function (_ref3) {
-          var fn = _ref3.fn,
-              suggestUseCallback = _ref3.suggestUseCallback;
+        bareFunctions.forEach(function (_ref4) {
+          var fn = _ref4.fn,
+              suggestUseCallback = _ref4.suggestUseCallback;
 
           var message = 'The \'' + fn.name.name + '\' function makes the dependencies of ' + (reactiveHookName + ' Hook (at line ' + declaredDependenciesNode.loc.start.line + ') ') + 'change on every render.';
           if (suggestUseCallback) {
             message += ' To fix this, ' + ('wrap the \'' + fn.name.name + '\' definition into its own useCallback() Hook.');
           } else {
-            message += ' To fix this, move the \'' + fn.name.name + '\' function ' + ('inside the ' + reactiveHookName + ' callback (at line ' + node.loc.start.line + '). ') + ('Alternatively, wrap the \'' + fn.name.name + '\' definition into its own useCallback() Hook.');
+            message += ' Move it inside the ' + reactiveHookName + ' callback. ' + ('Alternatively, wrap the \'' + fn.name.name + '\' definition into its own useCallback() Hook.');
           }
+          // TODO: What if the function needs to change on every render anyway?
+          // Should we suggest removing effect deps as an appropriate fix too?
           context.report({
+            // TODO: Why not report this at the dependency site?
             node: fn.node,
             message: message,
             fix: function (fixer) {
@@ -1253,10 +1325,10 @@ var ExhaustiveDeps = {
                 return [
                 // TODO: also add an import?
                 fixer.insertTextBefore(fn.node.init, 'useCallback('),
-                // TODO: ideally we'd gather deps here but it would
-                // require restructuring the rule code. For now,
-                // this is fine. Note we're intentionally not adding
-                // [] because that changes semantics.
+                // TODO: ideally we'd gather deps here but it would require
+                // restructuring the rule code. This will cause a new lint
+                // error to appear immediately for useCallback. Note we're
+                // not adding [] because would that changes semantics.
                 fixer.insertTextAfter(fn.node.init, ')')];
               }
             }
@@ -1317,10 +1389,14 @@ var ExhaustiveDeps = {
           }
         });
         if (badRef !== null) {
-          extraWarning = ' Mutable values like \'' + badRef + '\' aren\'t valid dependencies ' + "because their mutation doesn't re-render the component.";
+          extraWarning = ' Mutable values like \'' + badRef + '\' aren\'t valid dependencies ' + "because mutating them doesn't re-render the component.";
         } else if (externalDependencies.size > 0) {
           var dep = Array.from(externalDependencies)[0];
-          extraWarning = ' Values like \'' + dep + '\' aren\'t valid dependencies ' + 'because their mutation doesn\'t re-render the component.';
+          // Don't show this warning for things that likely just got moved *inside* the callback
+          // because in that case they're clearly not referring to globals.
+          if (!scope.set.has(dep)) {
+            extraWarning = ' Outer scope values like \'' + dep + '\' aren\'t valid dependencies ' + 'because mutating them doesn\'t re-render the component.';
+          }
         }
       }
 
@@ -1355,7 +1431,7 @@ var ExhaustiveDeps = {
           }
         }
         if (isPropsOnlyUsedInMembers) {
-          extraWarning = ' However, the preferred fix is to destructure the \'props\' ' + ('object outside of the ' + reactiveHookName + ' call and ') + 'refer to specific props directly by their names.';
+          extraWarning = ' However, \'props\' will change when *any* prop changes, so the ' + 'preferred fix is to destructure the \'props\' object outside of ' + ('the ' + reactiveHookName + ' call and refer to those specific props ') + ('inside ' + context.getSource(reactiveHook) + '.');
         }
       }
 
@@ -1397,7 +1473,7 @@ var ExhaustiveDeps = {
           missingCallbackDep = missingDep;
         });
         if (missingCallbackDep !== null) {
-          extraWarning = ' If specifying \'' + missingCallbackDep + '\'' + ' makes the dependencies change too often, ' + 'find the parent component that defines it ' + 'and wrap that definition in useCallback.';
+          extraWarning = ' If \'' + missingCallbackDep + '\' changes too often, ' + 'find the parent component that defines it ' + 'and wrap that definition in useCallback.';
         }
       }
 
@@ -1460,21 +1536,19 @@ var ExhaustiveDeps = {
           }
         });
         if (setStateRecommendation !== null) {
-          var suggestion = void 0;
           switch (setStateRecommendation.form) {
             case 'reducer':
-              suggestion = 'useReducer Hook. This lets you move the calculation ' + 'of next state outside the effect.';
+              extraWarning = ' You can also replace multiple useState variables with useReducer ' + ('if \'' + setStateRecommendation.setter + '\' needs the ') + ('current value of \'' + setStateRecommendation.missingDep + '\'.');
               break;
             case 'inlineReducer':
-              suggestion = 'useReducer Hook. This lets you move the calculation ' + 'of next state outside the effect. You can then ' + ('read \'' + setStateRecommendation.missingDep + '\' from the reducer ') + 'by putting it directly in your component.';
+              extraWarning = ' If \'' + setStateRecommendation.setter + '\' needs the ' + ('current value of \'' + setStateRecommendation.missingDep + '\', ') + 'you can also switch to useReducer instead of useState and ' + ('read \'' + setStateRecommendation.missingDep + '\' in the reducer.');
               break;
             case 'updater':
-              suggestion = setStateRecommendation.setter + '(' + setStateRecommendation.missingDep + ' => ...) form ' + 'which doesn\'t need to depend on the state from outside.';
+              extraWarning = ' You can also do a functional update \'' + setStateRecommendation.setter + '(' + setStateRecommendation.missingDep.substring(0, 1) + ' => ...)\' if you only need \'' + setStateRecommendation.missingDep + '\'' + (' in the \'' + setStateRecommendation.setter + '\' call.');
               break;
             default:
               throw new Error('Unknown case.');
           }
-          extraWarning = ' If \'' + setStateRecommendation.missingDep + '\'' + ' is only necessary for calculating the next state, ' + ('consider refactoring to the ' + suggestion);
         }
       }
 
@@ -1493,12 +1567,12 @@ var ExhaustiveDeps = {
 };
 
 // The meat of the logic.
-function collectRecommendations(_ref4) {
-  var dependencies = _ref4.dependencies,
-      declaredDependencies = _ref4.declaredDependencies,
-      optionalDependencies = _ref4.optionalDependencies,
-      externalDependencies = _ref4.externalDependencies,
-      isEffect = _ref4.isEffect;
+function collectRecommendations(_ref5) {
+  var dependencies = _ref5.dependencies,
+      declaredDependencies = _ref5.declaredDependencies,
+      optionalDependencies = _ref5.optionalDependencies,
+      externalDependencies = _ref5.externalDependencies,
+      isEffect = _ref5.isEffect;
 
   // Our primary data structure.
   // It is a logical representation of property chains:
@@ -1531,8 +1605,8 @@ function collectRecommendations(_ref4) {
 
   // Mark all satisfied nodes.
   // Imagine checkmarks next to each declared dependency.
-  declaredDependencies.forEach(function (_ref5) {
-    var key = _ref5.key;
+  declaredDependencies.forEach(function (_ref6) {
+    var key = _ref6.key;
 
     var node = getOrCreateNodeByPath(depTree, key);
     node.isSatisfiedRecursively = true;
@@ -1648,8 +1722,8 @@ function collectRecommendations(_ref4) {
   var suggestedDependencies = [];
   var unnecessaryDependencies = new Set();
   var duplicateDependencies = new Set();
-  declaredDependencies.forEach(function (_ref6) {
-    var key = _ref6.key;
+  declaredDependencies.forEach(function (_ref7) {
+    var key = _ref7.key;
 
     // Does this declared dep satsify a real need?
     if (satisfyingDependencies.has(key)) {
@@ -1691,14 +1765,14 @@ function collectRecommendations(_ref4) {
 
 // Finds functions declared as dependencies
 // that would invalidate on every render.
-function scanForDeclaredBareFunctions(_ref7) {
-  var declaredDependencies = _ref7.declaredDependencies,
-      declaredDependenciesNode = _ref7.declaredDependenciesNode,
-      componentScope = _ref7.componentScope,
-      scope = _ref7.scope;
-
-  var bareFunctions = declaredDependencies.map(function (_ref8) {
-    var key = _ref8.key;
+function scanForDeclaredBareFunctions(_ref8) {
+  var declaredDependencies = _ref8.declaredDependencies,
+      declaredDependenciesNode = _ref8.declaredDependenciesNode,
+      componentScope = _ref8.componentScope,
+      scope = _ref8.scope;
+
+  var bareFunctions = declaredDependencies.map(function (_ref9) {
+    var key = _ref9.key;
 
     var fnRef = componentScope.set.get(key);
     if (fnRef == null) {

package/cjs/eslint-plugin-react-hooks.production.min.js

@@ -7,41 +7,45 @@
  * LICENSE file in the root directory of this source tree.
  */
 
-'use strict';function Q(a){return"Identifier"===a.type?/^use[A-Z0-9].*$/.test(a.name):"MemberExpression"===a.type&&!a.computed&&Q(a.property)?(a=a.object,"Identifier"===a.type&&"React"===a.name):!1}function U(a){return"Identifier"===a.type?!/^[a-z]/.test(a.name):!1}function aa(a){for(;a;){var b=W(a);if(b&&(U(b)||Q(b)))return!0;a=a.parent}return!1}
-function W(a){if("FunctionDeclaration"===a.type||"FunctionExpression"===a.type&&a.id)return a.id;if("FunctionExpression"===a.type||"ArrowFunctionExpression"===a.type)return"VariableDeclarator"===a.parent.type&&a.parent.init===a?a.parent.id:"AssignmentExpression"===a.parent.type&&a.parent.right===a&&"="===a.parent.operator?a.parent.left:"Property"!==a.parent.type||a.parent.value!==a||a.parent.computed?"AssignmentPattern"!==a.parent.type||a.parent.right!==a||a.parent.computed?void 0:a.parent.left:a.parent.key}
-function ba(a){function b(){return{isRequired:!1,isSatisfiedRecursively:!1,hasRequiredNodesBelow:!1,children:new Map}}function e(a,g){g=g.split(".");var m=!0,l=!1,e=void 0;try{for(var h=g[Symbol.iterator](),k;!(m=(k=h.next()).done);m=!0){var v=k.value,f=a.children.get(v);f||(f=b(),a.children.set(v,f));a=f}}catch(C){l=!0,e=C}finally{try{!m&&h.return&&h.return()}finally{if(l)throw e;}}return a}function h(a,g,l){g=g.split(".");var m=!0,b=!1,h=void 0;try{for(var e=g[Symbol.iterator](),k;!(m=(k=e.next()).done);m=
-!0){var f=a.children.get(k.value);if(!f)break;l(f);a=f}}catch(C){b=!0,h=C}finally{try{!m&&e.return&&e.return()}finally{if(b)throw h;}}}function l(a,b,e,h){a.children.forEach(function(a,m){var k=h(m);a.isSatisfiedRecursively?a.hasRequiredNodesBelow&&e.add(k):a.isRequired?b.add(k):l(a,b,e,function(a){return k+"."+a})})}var v=a.dependencies,n=a.declaredDependencies,p=a.optionalDependencies,k=a.externalDependencies,q=a.isEffect,y=b();v.forEach(function(a,b){e(y,b).isRequired=!0;h(y,b,function(a){a.hasRequiredNodesBelow=
-!0})});n.forEach(function(a){e(y,a.key).isSatisfiedRecursively=!0});p.forEach(function(a){e(y,a).isSatisfiedRecursively=!0});a=new Set;var f=new Set;l(y,a,f,function(a){return a});var w=[],A=new Set,D=new Set;n.forEach(function(a){a=a.key;f.has(a)?-1===w.indexOf(a)?w.push(a):D.add(a):!q||a.endsWith(".current")||k.has(a)?A.add(a):-1===w.indexOf(a)&&w.push(a)});a.forEach(function(a){w.push(a)});return{suggestedDependencies:w,unnecessaryDependencies:A,duplicateDependencies:D,missingDependencies:a}}
-function ca(a){var b=a.declaredDependenciesNode,e=a.componentScope,h=a.scope;return a.declaredDependencies.map(function(a){a=e.set.get(a.key);if(null==a)return null;var b=a.defs[0];return null==b?null:"Variable"===b.type&&"VariableDeclarator"===b.node.type&&null!=b.node.init&&("ArrowFunctionExpression"===b.node.init.type||"FunctionExpression"===b.node.init.type)||"FunctionName"===b.type&&"FunctionDeclaration"===b.node.type?a:null}).filter(Boolean).map(function(a){var e=a.defs[0];a:{for(var l=!1,p=
-0;p<a.references.length;p++){var k=a.references[p];if(k.writeExpr)if(l){a=!0;break a}else{l=!0;continue}for(var q=k.from;q!==h&&null!=q;)q=q.upper;if(q!==h&&!da(b,k.identifier)){a=!0;break a}}a=!1}return{fn:e,suggestUseCallback:a}})}function ea(a){return"MemberExpression"!==a.parent.type||a.parent.object!==a||"current"===a.parent.property.name||a.parent.computed||null!=a.parent.parent&&"CallExpression"===a.parent.parent.type&&a.parent.parent.callee===a.parent?a:ea(a.parent)}
-function Y(a){if("Identifier"===a.type)return a.name;if("MemberExpression"!==a.type||a.computed)throw Error("Unsupported node type: "+a.type);var b=Y(a.object);a=Y(a.property);return b+"."+a}function ha(a){return"MemberExpression"!==a.type||"Identifier"!==a.object.type||"React"!==a.object.name||"Identifier"!==a.property.type||a.computed?a:a.property}
-function ia(a,b){var e=ha(a);if("Identifier"!==e.type)return null;switch(e.name){case "useEffect":case "useLayoutEffect":case "useCallback":case "useMemo":return 0;case "useImperativeHandle":return 1;default:if(e===a&&b&&b.additionalHooks){a=void 0;try{a=Y(e)}catch(h){if(/Unsupported node type/.test(h.message))return 0;throw h;}return b.additionalHooks.test(a)?0:-1}return-1}}
-function Z(a,b){for(var e=[a],h=null;e.length;){h=e.shift();if("Identifier"===h.type&&h.name===b.name&&h.range[0]===b.range[0]&&h.range[1]===b.range[1])return h;if(da(h,b)){a=!0;var l=!1,v=void 0;try{for(var n=Object.entries(h)[Symbol.iterator](),p;!(a=(p=n.next()).done);a=!0){var k=p.value,q=k[1];"parent"!==k[0]&&(ja(q)?(q.parent=h,e.push(q)):Array.isArray(q)&&q.forEach(function(a){ja(a)&&(a.parent=h,e.push(a))}))}}catch(y){l=!0,v=y}finally{try{!a&&n.return&&n.return()}finally{if(l)throw v;}}}}return null}
-function ka(a){for(var b="",e=0;e<a.length;e++)b+=a[e],0===e&&2===a.length?b+=" and ":e===a.length-2&&2<a.length?b+=", and ":e<a.length-1&&(b+=", ");return b}function ja(a){return"object"===typeof a&&null!==a&&!Array.isArray(a)&&"string"===typeof a.type}function da(a,b){return a.range[0]<=b.range[0]&&a.range[1]>=b.range[1]}
-module.exports={rules:{"rules-of-hooks":{create:function(a){var b=[],e=[];return{onCodePathSegmentStart:function(a){return e.push(a)},onCodePathSegmentEnd:function(){return e.pop()},onCodePathStart:function(){return b.push(new Map)},onCodePathEnd:function(e,l){function h(a){var d=h.cache,c=d.get(a.id);if(null===c){if(!q.has(a.id)){q.add(a.id);d=!0;c=!1;var x=void 0;try{for(var u=a.prevSegments[Symbol.iterator](),b;!(d=(b=u.next()).done);d=!0)h(b.value)}catch(M){c=!0,x=M}finally{try{!d&&u.return&&
-u.return()}finally{if(c)throw x;}}}return 0}if(void 0!==c)return c;d.set(a.id,null);if(e.thrownSegments.includes(a))c=0;else if(0===a.prevSegments.length)c=1;else{c=0;u=!0;b=!1;var k=void 0;try{x=a.prevSegments[Symbol.iterator]();for(var f;!(u=(f=x.next()).done);u=!0)c+=h(f.value)}catch(M){b=!0,k=M}finally{try{!u&&x.return&&x.return()}finally{if(b)throw k;}}}a.reachable&&0===c?d.delete(a.id):d.set(a.id,c);return c}function n(a){var d=n.cache,c=d.get(a.id);if(null===c){if(!q.has(a.id)){q.add(a.id);
-d=!0;c=!1;var x=void 0;try{for(var u=a.nextSegments[Symbol.iterator](),b;!(d=(b=u.next()).done);d=!0)n(b.value)}catch(M){c=!0,x=M}finally{try{!d&&u.return&&u.return()}finally{if(c)throw x;}}}return 0}if(void 0!==c)return c;d.set(a.id,null);if(e.thrownSegments.includes(a))c=0;else if(0===a.nextSegments.length)c=1;else{c=0;u=!0;b=!1;var k=void 0;try{x=a.nextSegments[Symbol.iterator]();for(var f;!(u=(f=x.next()).done);u=!0)c+=n(f.value)}catch(M){b=!0,k=M}finally{try{!u&&x.return&&x.return()}finally{if(b)throw k;
-}}}d.set(a.id,c);return c}function p(a){var d=p.cache,c=d.get(a.id);if(null===c)return Infinity;if(void 0!==c)return c;d.set(a.id,null);if(0===a.prevSegments.length)c=1;else{c=Infinity;var x=!0,u=!1,b=void 0;try{for(var e=a.prevSegments[Symbol.iterator](),k;!(x=(k=e.next()).done);x=!0){var f=p(k.value);f<c&&(c=f)}}catch(la){u=!0,b=la}finally{try{!x&&e.return&&e.return()}finally{if(u)throw b;}}c+=1}d.set(a.id,c);return c}var k=b.pop();if(0!==k.size){var q=new Set;h.cache=new Map;n.cache=new Map;p.cache=
-new Map;var y=n(e.initialSegment),f=W(l),w=aa(l),A=f?U(f)||Q(f):!1,D=Infinity,m=!0,g=!1,G=void 0;try{for(var t=e.finalSegments[Symbol.iterator](),J;!(m=(J=t.next()).done);m=!0){var V=J.value;if(V.reachable){var P=p(V);P<D&&(D=P)}}}catch(r){g=!0,G=r}finally{try{!m&&t.return&&t.return()}finally{if(g)throw G;}}m=!0;g=!1;G=void 0;try{for(var z=k[Symbol.iterator](),K;!(m=(K=z.next()).done);m=!0){var C=K.value,E=C[0],R=C[1];if(E.reachable){var H=0===E.nextSegments.length?D<=p(E):D<p(E),S=h(E)*n(E),F=q.has(E.id);
-k=!0;t=!1;J=void 0;try{for(var L=R[Symbol.iterator](),I;!(k=(I=L.next()).done);k=!0){var B=I.value;F&&a.report({node:B,message:'React Hook "'+a.getSource(B)+'" may be executed more than once. Possibly because it is called in a loop. React Hooks must be called in the exact same order in every component render.'});if(A){if(!F&&S!==y){var T='React Hook "'+a.getSource(B)+'" is called conditionally. React Hooks must be called in the exact same order in every component render.'+(H?" Did you accidentally call a React Hook after an early return?":
-"");a.report({node:B,message:T})}}else if(!l.parent||"MethodDefinition"!==l.parent.type&&"ClassProperty"!==l.parent.type||l.parent.value!==l)if(f){var N='React Hook "'+a.getSource(B)+'" is called in '+('function "'+a.getSource(f)+'" ')+"which is neither a React function component or a custom React Hook function.";a.report({node:B,message:N})}else if("Program"!==l.type&&w){var O='React Hook "'+a.getSource(B)+'" cannot be called inside a callback. React Hooks must be called in a React function component or a custom React Hook function.';
-a.report({node:B,message:O})}}}catch(r){t=!0,J=r}finally{try{!k&&L.return&&L.return()}finally{if(t)throw J;}}}}}catch(r){g=!0,G=r}finally{try{!m&&z.return&&z.return()}finally{if(g)throw G;}}}},CallExpression:function(a){if(Q(a.callee)){var h=b[b.length-1],v=e[e.length-1],n=h.get(v);n||(n=[],h.set(v,n));n.push(a.callee)}}}}},"exhaustive-deps":{meta:{fixable:"code",schema:[{type:"object",additionalProperties:!1,properties:{additionalHooks:{type:"string"}}}]},create:function(a){function b(a,b){return function(e){if(b.has(e))return b.get(e);
-var k=a(e);b.set(e,k);return k}}function e(e){function k(a){var c=!0,d=!1,b=void 0;try{for(var f=a.references[Symbol.iterator](),h;!(c=(h=f.next()).done);c=!0){var g=h.value;if(g.resolved&&G.has(g.resolved.scope)){var m=Z(e,g.identifier),l=ea(m),n=Y(l),q;if(q=D&&"Identifier"===l.type&&"MemberExpression"===l.parent.type&&!l.parent.computed&&"Identifier"===l.parent.property.type&&"current"===l.parent.property.name){for(var p=g.from,r=!1;p.block!==e;)"function"===p.type&&(r=null!=p.block.parent&&"ReturnStatement"===
-p.block.parent.type),p=p.upper;q=r}q&&P.set(n,{reference:g,dependencyNode:l});if(z.has(n))z.get(n).references.push(g);else{var t=g.resolved,w=J(t)||V(t);z.set(n,{isStatic:w,references:[g]})}}}}catch(X){d=!0,b=X}finally{try{!c&&f.return&&f.return()}finally{if(d)throw b;}}c=!0;d=!1;b=void 0;try{for(var v=a.childScopes[Symbol.iterator](),y;!(c=(y=v.next()).done);c=!0)k(y.value)}catch(X){d=!0,b=X}finally{try{!c&&v.return&&v.return()}finally{if(d)throw b;}}}function y(a,c,b,e){return 0===a.size?null:(1<
-a.size?"":c+" ")+b+" "+(1<a.size?"dependencies":"dependency")+": "+ka(Array.from(a).sort().map(function(a){return"'"+a+"'"}))+(". Either "+e+" "+(1<a.size?"them":"it")+" or remove the dependency array.")}if(("FunctionExpression"===e.type||"ArrowFunctionExpression"===e.type)&&"CallExpression"===e.parent.type){var f=ia(e.parent.callee,h);if(e.parent.arguments[f]===e){var w=e.parent.callee,A=ha(w).name,D=A.endsWith("Effect"),m=e.parent.arguments[f+1];if(m){var g=a.getScope(),G=new Set,t=null;for(f=g.upper;f;){G.add(f);
-if("function"===f.type)break;f=f.upper}if(f){t=f;var J=b(function(a){if(!Array.isArray(a.defs))return!1;var c=a.defs[0];if(null==c||"VariableDeclarator"!==c.node.type)return!1;var d=c.node.init;if(null==d)return!1;var b=c.node.parent;if(null==b&&(Z(t.block,c.node.id),b=c.node.parent,null==b))return!1;if("const"===b.kind&&"Literal"===d.type&&("string"===typeof d.value||"number"===typeof d.value||null===d.value))return!0;if("CallExpression"!==d.type)return!1;d=d.callee;"MemberExpression"!==d.type||
-"React"!==d.object.name||null==d.property||d.computed||(d=d.property);if("Identifier"!==d.type)return!1;c=c.node.id;d=d.name;if("useRef"===d&&"Identifier"===c.type)return!0;if(("useState"===d||"useReducer"===d)&&"ArrayPattern"===c.type&&2===c.elements.length&&Array.isArray(a.identifiers)){if(c.elements[1]===a.identifiers[0]){if("useState"===d)for(a=a.references,d=0;d<a.length;d++)l.set(a[d].identifier,c.elements[0]);return!0}if(c.elements[0]===a.identifiers[0]&&"useState"===d)for(a=a.references,c=
-0;c<a.length;c++)v.add(a[c].identifier)}return!1},n),V=b(function(a){if(!Array.isArray(a.defs))return!1;a=a.defs[0];if(null==a||null==a.node||null==a.node.id)return!1;var c=a.node,d=t.childScopes;a=null;var b;for(b=0;b<d.length;b++){var e=d[b],f=e.block;if("FunctionDeclaration"===c.type&&f===c||"VariableDeclarator"===c.type&&f.parent===c){a=e;break}}if(null==a)return!1;for(b=0;b<a.through.length;b++)if(c=a.through[b],null!=c.resolved&&G.has(c.resolved.scope)&&!J(c.resolved))return!1;return!0},p),
-P=new Map,z=new Map;k(g);P.forEach(function(d,c){var b=d.dependencyNode;d=d.reference.resolved.references;for(var e=!1,f=0;f<d.length;f++){var g=d[f].identifier.parent;if(null!=g&&"MemberExpression"===g.type&&!g.computed&&"Identifier"===g.property.type&&"current"===g.property.name&&"AssignmentExpression"===g.parent.type&&g.parent.left===g){e=!0;break}}e||a.report({node:b.parent.property,message:"Accessing '"+c+".current' during the effect cleanup will likely read a different ref value because by this time React has already updated the ref. If this ref is managed by React, store "+
-("'"+c+".current' in a variable inside ")+"the effect itself and refer to that variable from the cleanup function."})});var K=[],C=new Set;"ArrayExpression"!==m.type?a.report({node:m,message:"React Hook "+a.getSource(w)+" has a second argument which is not an array literal. This means we can't statically verify whether you've passed the correct dependencies."}):m.elements.forEach(function(d){if(null!==d)if("SpreadElement"===d.type)a.report({node:d,message:"React Hook "+a.getSource(w)+" has a spread element in its dependency array. This means we can't statically verify whether you've passed the correct dependencies."});
-else{var c=void 0;try{c=Y(d)}catch(fa){if(/Unsupported node type/.test(fa.message)){"Literal"===d.type?"string"===typeof d.value?a.report({node:d,message:"The "+d.raw+" string literal is not a valid dependency because it never changes. Did you mean to "+("include "+d.value+" in the array instead?")}):a.report({node:d,message:"The '"+d.raw+"' literal is not a valid dependency because it never changes. You can safely remove it."}):a.report({node:d,message:"React Hook "+a.getSource(w)+" has a complex expression in the dependency array. Extract it to a separate variable so it can be statically checked."});
-return}throw fa;}for(var b=d;"MemberExpression"===b.type;)b=b.object;var e=!t.through.some(function(a){return a.identifier===b});K.push({key:c,node:d});e||C.add(c)}});var E=new Set,R=new Set;z.forEach(function(d,c){var b=d.references;d.isStatic&&R.add(c);b.forEach(function(d){d.writeExpr&&(d=d.writeExpr,E.has(c)||(E.add(c),a.report({node:d,message:"Assignments to the '"+c+"' variable from inside a React "+a.getSource(w)+" Hook will not persist between re-renders. If it's only needed by this Hook, move the variable inside it. Alternatively, declare a ref with the useRef Hook, and keep the mutable value in its 'current' property."})))})});
-if(!(0<E.size)){var H=ba({dependencies:z,declaredDependencies:K,optionalDependencies:R,externalDependencies:C,isEffect:D}),S=H.suggestedDependencies;f=H.unnecessaryDependencies;var F=H.missingDependencies;H=H.duplicateDependencies;if(0===H.size+F.size+f.size)ca({declaredDependencies:K,declaredDependenciesNode:m,componentScope:t,scope:g}).forEach(function(d){var c=d.fn,b=d.suggestUseCallback;d="The '"+c.name.name+"' function makes the dependencies of "+(A+" Hook (at line "+m.loc.start.line+") ")+"change on every render.";
-d=b?d+(" To fix this, "+("wrap the '"+c.name.name+"' definition into its own useCallback() Hook.")):d+(" To fix this, move the '"+c.name.name+"' function "+("inside the "+A+" callback (at line "+e.loc.start.line+"). ")+("Alternatively, wrap the '"+c.name.name+"' definition into its own useCallback() Hook."));a.report({node:c.node,message:d,fix:function(a){if(b&&"Variable"===c.type)return[a.insertTextBefore(c.node.init,"useCallback("),a.insertTextAfter(c.node.init,")")]}})});else{!D&&0<F.size&&(S=
-ba({dependencies:z,declaredDependencies:[],optionalDependencies:R,externalDependencies:C,isEffect:D}).suggestedDependencies);(function(){if(0===K.length)return!0;var a=K.map(function(a){return a.key}),c=a.slice().sort();return a.join(",")===c.join(",")})()&&S.sort();g="";if(0<f.size){var L=null;Array.from(f.keys()).forEach(function(a){null===L&&a.endsWith(".current")&&(L=a)});null!==L?g=" Mutable values like '"+L+"' aren't valid dependencies because their mutation doesn't re-render the component.":
-0<C.size&&(g=" Values like '"+Array.from(C)[0]+"' aren't valid dependencies because their mutation doesn't re-render the component.")}if(!g&&F.has("props")){var I=z.get("props");if(null==I)return;I=I.references;if(!Array.isArray(I))return;for(var B=!0,T=0;T<I.length;T++){var N=Z(t.block,I[T].identifier);if(!N){B=!1;break}N=N.parent;if(null==N){B=!1;break}if("MemberExpression"!==N.type){B=!1;break}}B&&(g=" However, the preferred fix is to destructure the 'props' "+("object outside of the "+A+" call and ")+
-"refer to specific props directly by their names.")}if(!g&&0<F.size){var O=null;F.forEach(function(a){if(!O){var c=t.set.get(a),b=z.get(a);if(b.references[0].resolved===c&&(c=c.defs[0],null!=c&&null!=c.name&&"Parameter"===c.type)){c=!1;for(var d,e=0;e<b.references.length;e++)if(d=b.references[e].identifier,null!=d&&null!=d.parent&&"CallExpression"===d.parent.type&&d.parent.callee===d){c=!0;break}c&&(O=a)}}});null!==O&&(g=" If specifying '"+O+"' makes the dependencies change too often, find the parent component that defines it and wrap that definition in useCallback.")}if(!g&&
-0<F.size){var r=null;F.forEach(function(a){if(null===r)for(var c=z.get(a).references,b,d,e=0;e<c.length;e++){b=c[e].identifier;for(d=b.parent;null!=d&&d!==t.block;){if("CallExpression"===d.type){var f=l.get(d.callee);if(null!=f){f.name===a?r={missingDep:a,setter:d.callee.name,form:"updater"}:v.has(b)?r={missingDep:a,setter:d.callee.name,form:"reducer"}:(b=c[e].resolved,null!=b&&(b=b.defs[0],null!=b&&"Parameter"===b.type&&(r={missingDep:a,setter:d.callee.name,form:"inlineReducer"})));break}}d=d.parent}if(null!==
-r)break}});if(null!==r){g=void 0;switch(r.form){case "reducer":g="useReducer Hook. This lets you move the calculation of next state outside the effect.";break;case "inlineReducer":g="useReducer Hook. This lets you move the calculation of next state outside the effect. You can then "+("read '"+r.missingDep+"' from the reducer ")+"by putting it directly in your component.";break;case "updater":g=r.setter+"("+r.missingDep+" => ...) form which doesn't need to depend on the state from outside.";break;
-default:throw Error("Unknown case.");}g=" If '"+r.missingDep+"' is only necessary for calculating the next state, "+("consider refactoring to the "+g)}}a.report({node:m,message:"React Hook "+a.getSource(w)+" has "+(y(F,"a","missing","include")||y(f,"an","unnecessary","exclude")||y(H,"a","duplicate","omit"))+g,fix:function(a){return a.replaceText(m,"["+S.join(", ")+"]")}})}}}}else"useMemo"!==A&&"useCallback"!==A||a.report({node:e.parent.callee,message:"React Hook "+A+" doesn't serve any purpose without a dependency array. To enable this optimization, pass an array of values used by the "+
-("inner function as the second argument to "+A+".")})}}}var h={additionalHooks:a.options&&a.options[0]&&a.options[0].additionalHooks?new RegExp(a.options[0].additionalHooks):void 0},l=new WeakMap,v=new WeakSet,n=new WeakMap,p=new WeakMap;return{FunctionExpression:e,ArrowFunctionExpression:e}}}}};
+'use strict';function R(a){return"Identifier"===a.type?/^use[A-Z0-9].*$/.test(a.name):"MemberExpression"===a.type&&!a.computed&&R(a.property)?(a=a.object,"Identifier"===a.type&&"React"===a.name):!1}function aa(a){return"Identifier"===a.type?!/^[a-z]/.test(a.name):!1}function ba(a){for(;a;){var b=ca(a);if(b&&(aa(b)||R(b)))return!0;a=a.parent}return!1}
+function ca(a){if("FunctionDeclaration"===a.type||"FunctionExpression"===a.type&&a.id)return a.id;if("FunctionExpression"===a.type||"ArrowFunctionExpression"===a.type)return"VariableDeclarator"===a.parent.type&&a.parent.init===a?a.parent.id:"AssignmentExpression"===a.parent.type&&a.parent.right===a&&"="===a.parent.operator?a.parent.left:"Property"!==a.parent.type||a.parent.value!==a||a.parent.computed?"AssignmentPattern"!==a.parent.type||a.parent.right!==a||a.parent.computed?void 0:a.parent.left:
+a.parent.key}
+function V(a){function b(){return{isRequired:!1,isSatisfiedRecursively:!1,hasRequiredNodesBelow:!1,children:new Map}}function e(a,h){h=h.split(".");var l=!0,c=!1,e=void 0;try{for(var f=h[Symbol.iterator](),p;!(l=(p=f.next()).done);l=!0){var u=p.value,g=a.children.get(u);g||(g=b(),a.children.set(u,g));a=g}}catch(E){c=!0,e=E}finally{try{!l&&f.return&&f.return()}finally{if(c)throw e;}}return a}function f(a,h,c){h=h.split(".");var l=!0,b=!1,f=void 0;try{for(var e=h[Symbol.iterator](),g;!(l=(g=e.next()).done);l=
+!0){var u=a.children.get(g.value);if(!u)break;c(u);a=u}}catch(E){b=!0,f=E}finally{try{!l&&e.return&&e.return()}finally{if(b)throw f;}}}function h(a,c,b,e){a.children.forEach(function(a,l){var f=e(l);a.isSatisfiedRecursively?a.hasRequiredNodesBelow&&b.add(f):a.isRequired?c.add(f):h(a,c,b,function(a){return f+"."+a})})}var u=a.dependencies,q=a.declaredDependencies,t=a.optionalDependencies,c=a.externalDependencies,n=a.isEffect,C=b();u.forEach(function(a,h){e(C,h).isRequired=!0;f(C,h,function(a){a.hasRequiredNodesBelow=
+!0})});q.forEach(function(a){e(C,a.key).isSatisfiedRecursively=!0});t.forEach(function(a){e(C,a).isSatisfiedRecursively=!0});a=new Set;var g=new Set;h(C,a,g,function(a){return a});var v=[],A=new Set,B=new Set;q.forEach(function(a){a=a.key;g.has(a)?-1===v.indexOf(a)?v.push(a):B.add(a):!n||a.endsWith(".current")||c.has(a)?A.add(a):-1===v.indexOf(a)&&v.push(a)});a.forEach(function(a){v.push(a)});return{suggestedDependencies:v,unnecessaryDependencies:A,duplicateDependencies:B,missingDependencies:a}}
+function da(a){var b=a.declaredDependenciesNode,e=a.componentScope,f=a.scope;return a.declaredDependencies.map(function(a){a=e.set.get(a.key);if(null==a)return null;var b=a.defs[0];return null==b?null:"Variable"===b.type&&"VariableDeclarator"===b.node.type&&null!=b.node.init&&("ArrowFunctionExpression"===b.node.init.type||"FunctionExpression"===b.node.init.type)||"FunctionName"===b.type&&"FunctionDeclaration"===b.node.type?a:null}).filter(Boolean).map(function(a){var e=a.defs[0];a:{for(var h=!1,t=
+0;t<a.references.length;t++){var c=a.references[t];if(c.writeExpr)if(h){a=!0;break a}else{h=!0;continue}for(var n=c.from;n!==f&&null!=n;)n=n.upper;if(n!==f&&!ea(b,c.identifier)){a=!0;break a}}a=!1}return{fn:e,suggestUseCallback:a}})}function fa(a){return"MemberExpression"!==a.parent.type||a.parent.object!==a||"current"===a.parent.property.name||a.parent.computed||null!=a.parent.parent&&"CallExpression"===a.parent.parent.type&&a.parent.parent.callee===a.parent?a:fa(a.parent)}
+function X(a){if("Identifier"===a.type)return a.name;if("MemberExpression"!==a.type||a.computed)throw Error("Unsupported node type: "+a.type);var b=X(a.object);a=X(a.property);return b+"."+a}function ja(a){return"MemberExpression"!==a.type||"Identifier"!==a.object.type||"React"!==a.object.name||"Identifier"!==a.property.type||a.computed?a:a.property}
+function ka(a,b){var e=ja(a);if("Identifier"!==e.type)return null;switch(e.name){case "useEffect":case "useLayoutEffect":case "useCallback":case "useMemo":return 0;case "useImperativeHandle":return 1;default:if(e===a&&b&&b.additionalHooks){a=void 0;try{a=X(e)}catch(f){if(/Unsupported node type/.test(f.message))return 0;throw f;}return b.additionalHooks.test(a)?0:-1}return-1}}
+function Y(a,b){for(var e=[a],f=null;e.length;){f=e.shift();if("Identifier"===f.type&&f.name===b.name&&f.range[0]===b.range[0]&&f.range[1]===b.range[1])return f;if(ea(f,b)){a=!0;var h=!1,u=void 0;try{for(var q=Object.entries(f)[Symbol.iterator](),t;!(a=(t=q.next()).done);a=!0){var c=t.value,n=c[1];"parent"!==c[0]&&(la(n)?(n.parent=f,e.push(n)):Array.isArray(n)&&n.forEach(function(a){la(a)&&(a.parent=f,e.push(a))}))}}catch(C){h=!0,u=C}finally{try{!a&&q.return&&q.return()}finally{if(h)throw u;}}}}return null}
+function ma(a){for(var b="",e=0;e<a.length;e++)b+=a[e],0===e&&2===a.length?b+=" and ":e===a.length-2&&2<a.length?b+=", and ":e<a.length-1&&(b+=", ");return b}function la(a){return"object"===typeof a&&null!==a&&!Array.isArray(a)&&"string"===typeof a.type}function ea(a,b){return a.range[0]<=b.range[0]&&a.range[1]>=b.range[1]}
+module.exports={rules:{"rules-of-hooks":{create:function(a){var b=[],e=[];return{onCodePathSegmentStart:function(a){return e.push(a)},onCodePathSegmentEnd:function(){return e.pop()},onCodePathStart:function(){return b.push(new Map)},onCodePathEnd:function(e,h){function f(a){var b=f.cache,c=b.get(a.id);if(null===c){if(!n.has(a.id)){n.add(a.id);b=!0;c=!1;var d=void 0;try{for(var m=a.prevSegments[Symbol.iterator](),r;!(b=(r=m.next()).done);b=!0)f(r.value)}catch(K){c=!0,d=K}finally{try{!b&&m.return&&
+m.return()}finally{if(c)throw d;}}}return 0}if(void 0!==c)return c;b.set(a.id,null);if(e.thrownSegments.includes(a))c=0;else if(0===a.prevSegments.length)c=1;else{c=0;m=!0;r=!1;var H=void 0;try{d=a.prevSegments[Symbol.iterator]();for(var k;!(m=(k=d.next()).done);m=!0)c+=f(k.value)}catch(K){r=!0,H=K}finally{try{!m&&d.return&&d.return()}finally{if(r)throw H;}}}a.reachable&&0===c?b.delete(a.id):b.set(a.id,c);return c}function q(a){var b=q.cache,c=b.get(a.id);if(null===c){if(!n.has(a.id)){n.add(a.id);
+b=!0;c=!1;var d=void 0;try{for(var m=a.nextSegments[Symbol.iterator](),r;!(b=(r=m.next()).done);b=!0)q(r.value)}catch(K){c=!0,d=K}finally{try{!b&&m.return&&m.return()}finally{if(c)throw d;}}}return 0}if(void 0!==c)return c;b.set(a.id,null);if(e.thrownSegments.includes(a))c=0;else if(0===a.nextSegments.length)c=1;else{c=0;m=!0;r=!1;var H=void 0;try{d=a.nextSegments[Symbol.iterator]();for(var f;!(m=(f=d.next()).done);m=!0)c+=q(f.value)}catch(K){r=!0,H=K}finally{try{!m&&d.return&&d.return()}finally{if(r)throw H;
+}}}b.set(a.id,c);return c}function t(a){var c=t.cache,b=c.get(a.id);if(null===b)return Infinity;if(void 0!==b)return b;c.set(a.id,null);if(0===a.prevSegments.length)b=1;else{b=Infinity;var d=!0,m=!1,r=void 0;try{for(var H=a.prevSegments[Symbol.iterator](),e;!(d=(e=H.next()).done);d=!0){var f=t(e.value);f<b&&(b=f)}}catch(na){m=!0,r=na}finally{try{!d&&H.return&&H.return()}finally{if(m)throw r;}}b+=1}c.set(a.id,b);return b}var c=b.pop();if(0!==c.size){var n=new Set;f.cache=new Map;q.cache=new Map;t.cache=
+new Map;var C=q(e.initialSegment),g=ca(h),v=ba(h),A=g?aa(g)||R(g):!1,B=Infinity,l=!0,p=!1,G=void 0;try{for(var w=e.finalSegments[Symbol.iterator](),J;!(l=(J=w.next()).done);l=!0){var W=J.value;if(W.reachable){var Q=t(W);Q<B&&(B=Q)}}}catch(k){p=!0,G=k}finally{try{!l&&w.return&&w.return()}finally{if(p)throw G;}}l=!0;p=!1;G=void 0;try{for(var z=c[Symbol.iterator](),O;!(l=(O=z.next()).done);l=!0){var E=O.value,D=E[0],L=E[1];if(D.reachable){var I=0===D.nextSegments.length?B<=t(D):B<t(D),T=f(D)*q(D),F=
+n.has(D.id);c=!0;w=!1;J=void 0;try{for(var x=L[Symbol.iterator](),M;!(c=(M=x.next()).done);c=!0){var y=M.value;F&&a.report({node:y,message:'React Hook "'+a.getSource(y)+'" may be executed more than once. Possibly because it is called in a loop. React Hooks must be called in the exact same order in every component render.'});if(A){if(!F&&T!==C){var U='React Hook "'+a.getSource(y)+'" is called conditionally. React Hooks must be called in the exact same order in every component render.'+(I?" Did you accidentally call a React Hook after an early return?":
+"");a.report({node:y,message:U})}}else if(!h.parent||"MethodDefinition"!==h.parent.type&&"ClassProperty"!==h.parent.type||h.parent.value!==h)if(g){var N='React Hook "'+a.getSource(y)+'" is called in '+('function "'+a.getSource(g)+'" ')+"which is neither a React function component or a custom React Hook function.";a.report({node:y,message:N})}else if("Program"!==h.type&&v){var P='React Hook "'+a.getSource(y)+'" cannot be called inside a callback. React Hooks must be called in a React function component or a custom React Hook function.';
+a.report({node:y,message:P})}}}catch(k){w=!0,J=k}finally{try{!c&&x.return&&x.return()}finally{if(w)throw J;}}}}}catch(k){p=!0,G=k}finally{try{!l&&z.return&&z.return()}finally{if(p)throw G;}}}},CallExpression:function(a){if(R(a.callee)){var f=b[b.length-1],u=e[e.length-1],q=f.get(u);q||(q=[],f.set(u,q));q.push(a.callee)}}}}},"exhaustive-deps":{meta:{fixable:"code",schema:[{type:"object",additionalProperties:!1,properties:{additionalHooks:{type:"string"}}}]},create:function(a){function b(a,b){return function(c){if(b.has(c))return b.get(c);
+var e=a(c);b.set(c,e);return e}}function e(c){function e(a){var d=!0,r=!1,b=void 0;try{for(var f=a.references[Symbol.iterator](),h;!(d=(h=f.next()).done);d=!0){var g=h.value;if(g.resolved&&G.has(g.resolved.scope)){var l=Y(c,g.identifier),k=fa(l),p=X(k),q;if(q=B&&"Identifier"===k.type&&"MemberExpression"===k.parent.type&&!k.parent.computed&&"Identifier"===k.parent.property.type&&"current"===k.parent.property.name){for(var n=g.from,u=!1;n.block!==c;)"function"===n.type&&(u=null!=n.block.parent&&"ReturnStatement"===
+n.block.parent.type),n=n.upper;q=u}q&&Q.set(p,{reference:g,dependencyNode:k});var t=g.resolved.defs[0];if(null==t||null==t.node||t.node.init!==c.parent)if(z.has(p))z.get(p).references.push(g);else{var v=g.resolved,y=J(v)||W(v);z.set(p,{isStatic:y,references:[g]})}}}}catch(Z){r=!0,b=Z}finally{try{!d&&f.return&&f.return()}finally{if(r)throw b;}}d=!0;r=!1;b=void 0;try{for(var w=a.childScopes[Symbol.iterator](),x;!(d=(x=w.next()).done);d=!0)e(x.value)}catch(Z){r=!0,b=Z}finally{try{!d&&w.return&&w.return()}finally{if(r)throw b;
+}}}function C(a,b,r,c){return 0===a.size?null:(1<a.size?"":b+" ")+r+" "+(1<a.size?"dependencies":"dependency")+": "+ma(Array.from(a).sort().map(function(a){return"'"+a+"'"}))+(". Either "+c+" "+(1<a.size?"them":"it")+" or remove the dependency array.")}if(("FunctionExpression"===c.type||"ArrowFunctionExpression"===c.type)&&"CallExpression"===c.parent.type){var g=ka(c.parent.callee,f);if(c.parent.arguments[g]===c){var v=c.parent.callee,A=ja(v).name,B=A.endsWith("Effect"),l=c.parent.arguments[g+1];
+if(l||B){B&&c.async&&a.report({node:c,message:"Effect callbacks are synchronous to prevent race conditions. Put the async function inside:\n\nuseEffect(() => {\n  async function fetchData() {\n    // You can await here\n    const response = await MyAPI.getData(someId);\n    // ...\n  }\n  fetchData();\n}, [someId]); // Or [] if effect doesn't need props or state\n\nLearn more about data fetching with Hooks: https://fb.me/react-hooks-data-fetching"});var p=a.getScope(),G=new Set,w=null;for(g=p.upper;g;){G.add(g);
+if("function"===g.type)break;g=g.upper}if(g){w=g;var J=b(function(a){if(!Array.isArray(a.defs))return!1;var d=a.defs[0];if(null==d||"VariableDeclarator"!==d.node.type)return!1;var b=d.node.init;if(null==b)return!1;var c=d.node.parent;if(null==c&&(Y(w.block,d.node.id),c=d.node.parent,null==c))return!1;if("const"===c.kind&&"Literal"===b.type&&("string"===typeof b.value||"number"===typeof b.value||null===b.value))return!0;if("CallExpression"!==b.type)return!1;b=b.callee;"MemberExpression"!==b.type||
+"React"!==b.object.name||null==b.property||b.computed||(b=b.property);if("Identifier"!==b.type)return!1;d=d.node.id;b=b.name;if("useRef"===b&&"Identifier"===d.type)return!0;if(("useState"===b||"useReducer"===b)&&"ArrayPattern"===d.type&&2===d.elements.length&&Array.isArray(a.identifiers)){if(d.elements[1]===a.identifiers[0]){if("useState"===b)for(a=a.references,b=0;b<a.length;b++)h.set(a[b].identifier,d.elements[0]);return!0}if(d.elements[0]===a.identifiers[0]&&"useState"===b)for(a=a.references,d=
+0;d<a.length;d++)u.add(a[d].identifier)}return!1},q),W=b(function(a){if(!Array.isArray(a.defs))return!1;a=a.defs[0];if(null==a||null==a.node||null==a.node.id)return!1;var b=a.node,d=w.childScopes;a=null;var c;for(c=0;c<d.length;c++){var e=d[c],f=e.block;if("FunctionDeclaration"===b.type&&f===b||"VariableDeclarator"===b.type&&f.parent===b){a=e;break}}if(null==a)return!1;for(c=0;c<a.through.length;c++)if(b=a.through[c],null!=b.resolved&&G.has(b.resolved.scope)&&!J(b.resolved))return!1;return!0},t),
+Q=new Map,z=new Map;e(p);Q.forEach(function(b,c){var d=b.dependencyNode;b=b.reference.resolved.references;for(var e=!1,f=0;f<b.length;f++){var m=b[f].identifier.parent;if(null!=m&&"MemberExpression"===m.type&&!m.computed&&"Identifier"===m.property.type&&"current"===m.property.name&&"AssignmentExpression"===m.parent.type&&m.parent.left===m){e=!0;break}}e||a.report({node:d.parent.property,message:"The ref value '"+c+".current' will likely have changed by the time this effect cleanup function runs. If this ref points to a node rendered by React, copy "+
+("'"+c+".current' to a variable inside the effect, and ")+"use that variable in the cleanup function."})});var O=new Set,E=new Set;z.forEach(function(b,c){var d=b.references;b.isStatic&&E.add(c);d.forEach(function(b){b.writeExpr&&(b=b.writeExpr,O.has(c)||(O.add(c),a.report({node:b,message:"Assignments to the '"+c+"' variable from inside React Hook "+(a.getSource(v)+" will be lost after each ")+"render. To preserve the value over time, store it in a useRef Hook and keep the mutable value in the '.current' property. Otherwise, you can move this variable directly inside "+
+(a.getSource(v)+".")})))})});if(!(0<O.size))if(l){var D=[],L=new Set;"ArrayExpression"!==l.type?a.report({node:l,message:"React Hook "+a.getSource(v)+" was passed a dependency list that is not an array literal. This means we can't statically verify whether you've passed the correct dependencies."}):l.elements.forEach(function(b){if(null!==b)if("SpreadElement"===b.type)a.report({node:b,message:"React Hook "+a.getSource(v)+" has a spread element in its dependency array. This means we can't statically verify whether you've passed the correct dependencies."});
+else{var c=void 0;try{c=X(b)}catch(ia){if(/Unsupported node type/.test(ia.message)){"Literal"===b.type?z.has(b.value)?a.report({node:b,message:"The "+b.raw+" literal is not a valid dependency because it never changes. "+("Did you mean to include "+b.value+" in the array instead?")}):a.report({node:b,message:"The "+b.raw+" literal is not a valid dependency because it never changes. You can safely remove it."}):a.report({node:b,message:"React Hook "+a.getSource(v)+" has a complex expression in the dependency array. Extract it to a separate variable so it can be statically checked."});
+return}throw ia;}for(var d=b;"MemberExpression"===d.type;)d=d.object;var e=!w.through.some(function(a){return a.identifier===d});D.push({key:c,node:b});e||L.add(c)}});var I=V({dependencies:z,declaredDependencies:D,optionalDependencies:E,externalDependencies:L,isEffect:B}),T=I.suggestedDependencies;g=I.unnecessaryDependencies;var F=I.missingDependencies;I=I.duplicateDependencies;if(0===I.size+F.size+g.size)da({declaredDependencies:D,declaredDependenciesNode:l,componentScope:w,scope:p}).forEach(function(b){var c=
+b.fn,d=b.suggestUseCallback;b="The '"+c.name.name+"' function makes the dependencies of "+(A+" Hook (at line "+l.loc.start.line+") ")+"change on every render.";b=d?b+(" To fix this, "+("wrap the '"+c.name.name+"' definition into its own useCallback() Hook.")):b+(" Move it inside the "+A+" callback. "+("Alternatively, wrap the '"+c.name.name+"' definition into its own useCallback() Hook."));a.report({node:c.node,message:b,fix:function(a){if(d&&"Variable"===c.type)return[a.insertTextBefore(c.node.init,
+"useCallback("),a.insertTextAfter(c.node.init,")")]}})});else{!B&&0<F.size&&(T=V({dependencies:z,declaredDependencies:[],optionalDependencies:E,externalDependencies:L,isEffect:B}).suggestedDependencies);(function(){if(0===D.length)return!0;var a=D.map(function(a){return a.key}),b=a.slice().sort();return a.join(",")===b.join(",")})()&&T.sort();var x="";if(0<g.size){var M=null;Array.from(g.keys()).forEach(function(a){null===M&&a.endsWith(".current")&&(M=a)});if(null!==M)x=" Mutable values like '"+M+
+"' aren't valid dependencies because mutating them doesn't re-render the component.";else if(0<L.size){var y=Array.from(L)[0];p.set.has(y)||(x=" Outer scope values like '"+y+"' aren't valid dependencies because mutating them doesn't re-render the component.")}}if(!x&&F.has("props")){p=z.get("props");if(null==p)return;p=p.references;if(!Array.isArray(p))return;y=!0;for(var U=0;U<p.length;U++){var N=Y(w.block,p[U].identifier);if(!N){y=!1;break}N=N.parent;if(null==N){y=!1;break}if("MemberExpression"!==
+N.type){y=!1;break}}y&&(x=" However, 'props' will change when *any* prop changes, so the preferred fix is to destructure the 'props' object outside of "+("the "+A+" call and refer to those specific props ")+("inside "+a.getSource(v)+"."))}if(!x&&0<F.size){var P=null;F.forEach(function(a){if(!P){var b=w.set.get(a),c=z.get(a);if(c.references[0].resolved===b&&(b=b.defs[0],null!=b&&null!=b.name&&"Parameter"===b.type)){b=!1;for(var d,e=0;e<c.references.length;e++)if(d=c.references[e].identifier,null!=
+d&&null!=d.parent&&"CallExpression"===d.parent.type&&d.parent.callee===d){b=!0;break}b&&(P=a)}}});null!==P&&(x=" If '"+P+"' changes too often, find the parent component that defines it and wrap that definition in useCallback.")}if(!x&&0<F.size){var k=null;F.forEach(function(a){if(null===k)for(var b=z.get(a).references,c,d,e=0;e<b.length;e++){c=b[e].identifier;for(d=c.parent;null!=d&&d!==w.block;){if("CallExpression"===d.type){var f=h.get(d.callee);if(null!=f){f.name===a?k={missingDep:a,setter:d.callee.name,
+form:"updater"}:u.has(c)?k={missingDep:a,setter:d.callee.name,form:"reducer"}:(c=b[e].resolved,null!=c&&(c=c.defs[0],null!=c&&"Parameter"===c.type&&(k={missingDep:a,setter:d.callee.name,form:"inlineReducer"})));break}}d=d.parent}if(null!==k)break}});if(null!==k)switch(k.form){case "reducer":x=" You can also replace multiple useState variables with useReducer "+("if '"+k.setter+"' needs the ")+("current value of '"+k.missingDep+"'.");break;case "inlineReducer":x=" If '"+k.setter+"' needs the "+("current value of '"+
+k.missingDep+"', ")+"you can also switch to useReducer instead of useState and "+("read '"+k.missingDep+"' in the reducer.");break;case "updater":x=" You can also do a functional update '"+k.setter+"("+k.missingDep.substring(0,1)+" => ...)' if you only need '"+k.missingDep+"'"+(" in the '"+k.setter+"' call.");break;default:throw Error("Unknown case.");}}a.report({node:l,message:"React Hook "+a.getSource(v)+" has "+(C(F,"a","missing","include")||C(g,"an","unnecessary","exclude")||C(I,"a","duplicate",
+"omit"))+x,fix:function(a){return a.replaceText(l,"["+T.join(", ")+"]")}})}}else{var S=null;z.forEach(function(a,b){S||a.references.forEach(function(a){if(!S&&h.has(a.identifier)){for(a=a.from;"function"!==a.type;)a=a.upper;a.block===c&&(S=b)}})});if(S){var ha=V({dependencies:z,declaredDependencies:[],optionalDependencies:E,externalDependencies:new Set,isEffect:!0}).suggestedDependencies;a.report({node:c.parent.callee,message:"React Hook "+A+" contains a call to '"+S+"'. Without a list of dependencies, this can lead to an infinite chain of updates. To fix this, pass ["+
+ha.join(", ")+("] as a second argument to the "+A+" Hook."),fix:function(a){return a.insertTextAfter(c,", ["+ha.join(", ")+"]")}})}}}}else"useMemo"!==A&&"useCallback"!==A||a.report({node:c.parent.callee,message:"React Hook "+A+" does nothing when called with only one argument. Did you forget to pass an array of dependencies?"})}}}var f={additionalHooks:a.options&&a.options[0]&&a.options[0].additionalHooks?new RegExp(a.options[0].additionalHooks):void 0},h=new WeakMap,u=new WeakSet,q=new WeakMap,t=
+new WeakMap;return{FunctionExpression:e,ArrowFunctionExpression:e}}}}};

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "eslint-plugin-react-hooks",
   "description": "ESLint rules for React Hooks",
-  "version": "1.5.0-beta.0",
+  "version": "1.6.0",
   "repository": {
     "type": "git",
     "url": "https://github.com/facebook/react.git",