Add eslint-plugin-react-hooks/exhaustive-deps rule to check stale closure dependencies

fixtures/eslint/.eslintrc.json

@@ -9,6 +9,7 @@
   },
   "plugins": ["react-hooks"],
   "rules": {
-    "react-hooks/rules-of-hooks": 2
+    "react-hooks/rules-of-hooks": 2,
+    "react-hooks/exhaustive-deps": 2
   }
 }

fixtures/eslint/index.js

@@ -4,8 +4,26 @@
 // 2. "File > Add Folder to Workspace" this specific folder in VSCode with ESLint extension
 // 3. Changes to the rule source should get picked up without restarting ESLint server
 
-function Foo() {
-  if (condition) {
-    useEffect(() => {});
-  }
+function Comment({comment, commentSource}) {
+  const currentUserID = comment.viewer.id;
+  const environment = RelayEnvironment.forUser(currentUserID);
+  const commentID = nullthrows(comment.id);
+  useEffect(
+    () => {
+      const subscription = SubscriptionCounter.subscribeOnce(
+        `StoreSubscription_${commentID}`,
+        () =>
+          StoreSubscription.subscribe(
+            environment,
+            {
+              comment_id: commentID,
+            },
+            currentUserID,
+            commentSource
+          )
+      );
+      return () => subscription.dispose();
+    },
+    [commentID, commentSource, currentUserID, environment]
+  );
 }