Convert isDirectCall to recursive for consistency

Author: NickvanDyke

src/rule.js

@@ -75,7 +75,7 @@ export const rule = {
               // Don't analyze HOC prop callbacks -- we don't have control over them to lift state or logic
               !isHOCProp(ref.resolved)),
         )
-        .filter((ref) => isDirectCall(ref))
+        .filter((ref) => isDirectCall(ref.identifier))
         .forEach((ref) => {
           const callExpr = getCallExpr(ref);
 

src/util/react.js

@@ -154,18 +154,17 @@ export const getUseStateNode = (context, ref) => {
 // Also returns false for IIFEs, which technically could cause a false negative.
 // But IIFEs in effects are typically used to call async functions, implying it retrieves external state.
 // So, not a big deal.
-export const isDirectCall = (ref) => {
-  let node = ref.identifier;
-
-  while (
-    node &&
-    node.type !== "ArrowFunctionExpression" &&
-    node.type !== "FunctionExpression"
+export const isDirectCall = (node) => {
+  if (!node) {
+    return false;
+  } else if (
+    node.type === "ArrowFunctionExpression" ||
+    node.type === "FunctionExpression"
   ) {
-    node = node.parent;
+    return isUseEffect(node.parent);
+  } else {
+    return isDirectCall(node.parent);
   }
-
-  return node && isUseEffect(node.parent);
 };
 
 export const findPropUsedToResetAllState = (