Fix lint errors

Various hooks either missing dependencies or with extra ones.

Two remaining errors are from the recapcta code where I can't
work out if the extra dependency is intentional or not.

package.json

@@ -8,7 +8,7 @@
     "build-storybook": "build-storybook",
     "prettier:check": "prettier -c src",
     "prettier:format": "prettier -w src",
-    "lint": "eslint --max-warnings 11 src"
+    "lint": "eslint --max-warnings 2 src"
   },
   "dependencies": {
     "@juggle/resize-observer": "^3.3.1",

src/auth/useInteractiveLogin.js

@@ -7,7 +7,8 @@ export function useInteractiveLogin() {
   const { setClient } = useClient();
   const [state, setState] = useState({ loading: false });
 
-  const auth = useCallback(async (homeserver, username, password) => {
+  const auth = useCallback(
+    async (homeserver, username, password) => {
       const authClient = matrix.createClient(homeserver);
 
       const interactiveAuth = new InteractiveAuth({
@@ -39,7 +40,9 @@ export function useInteractiveLogin() {
       setClient(client, { user_id, access_token, device_id });
 
       return client;
-  }, []);
+    },
+    [setClient]
+  );
 
   return [state, auth];
 }

src/auth/useInteractiveRegistration.js

@@ -89,7 +89,7 @@ export function useInteractiveRegistration() {
 
       return client;
     },
-    []
+    [setClient]
   );
 
   return [state, register];

src/home/useGroupCallRooms.js

@@ -81,7 +81,7 @@ export function useGroupCallRooms(client) {
       client.removeListener("GroupCall.incoming", updateRooms);
       client.removeListener("GroupCall.participants", updateRooms);
     };
-  }, []);
+  }, [client]);
 
   return rooms;
 }

src/room/useLoadGroupCall.js

@@ -86,7 +86,7 @@ export function useLoadGroupCall(client, roomId, viaServers, createIfNotFound) {
       .catch((error) =>
         setState((prevState) => ({ ...prevState, loading: false, error }))
       );
-  }, [client, roomId, state.reloadId]);
+  }, [client, roomId, state.reloadId, createIfNotFound, viaServers]);
 
   return state;
 }

src/room/usePTT.js

@@ -55,7 +55,7 @@ export function usePTT(client, groupCall, userMediaFeeds) {
 
       setState((prevState) => ({ ...prevState, pttButtonHeld: true }));
     }
-  }, []);
+  }, [groupCall, activeSpeakerUserId, isAdmin, talkOverEnabled]);
 
   const stopTalking = useCallback(() => {
     if (!groupCall.isMicrophoneMuted()) {
@@ -63,7 +63,7 @@ export function usePTT(client, groupCall, userMediaFeeds) {
     }
 
     setState((prevState) => ({ ...prevState, pttButtonHeld: false }));
-  }, []);
+  }, [groupCall]);
 
   useEffect(() => {
     function onKeyDown(event) {
@@ -100,7 +100,14 @@ export function usePTT(client, groupCall, userMediaFeeds) {
       window.removeEventListener("keyup", onKeyUp);
       window.removeEventListener("blur", onBlur);
     };
-  }, [activeSpeakerUserId, isAdmin, talkOverEnabled]);
+  }, [
+    groupCall,
+    startTalking,
+    stopTalking,
+    activeSpeakerUserId,
+    isAdmin,
+    talkOverEnabled,
+  ]);
 
   const setTalkOverEnabled = useCallback((talkOverEnabled) => {
     setState((prevState) => ({

src/settings/submit-rageshake.js

@@ -223,7 +223,7 @@ export function useSubmitRageshake() {
         console.error(error);
       }
     },
-    [client]
+    [client, json, sending]
   );
 
   return {
@@ -294,7 +294,7 @@ export function useRageshakeRequestModal(roomId) {
     return () => {
       client.removeListener("event", onEvent);
     };
-  }, [modalState.open, roomId]);
+  }, [modalState.open, roomId, client, modalState]);
 
   return { modalState, modalProps: { ...modalProps, rageshakeRequestId } };
 }