fix(#141): peer verification stability regression

src/components/Room/usePeerVerification.ts

@@ -134,6 +134,7 @@ export const usePeerVerification = ({
   // getDisplayUsername).
   const [scheduledPeerToVerify, setScheduledPeerToVerify] =
     useState<Peer | null>(null)
+
   useEffect(() => {
     if (scheduledPeerToVerify === null || isDirectMessageRoom) return
 
@@ -143,6 +144,8 @@ export const usePeerVerification = ({
   // NOTE: END HACKY WORKAROUND
 
   const verifyPeer = (peer: Peer) => {
+    if (isDirectMessageRoom) return
+
     setScheduledPeerToVerify(peer)
   }
 

src/components/Room/useRoom.ts

@@ -296,18 +296,21 @@ export function useRoom(
           verificationTimer: null,
         }
 
-        setPeerList([...peerList, newPeer])
+        setPeerList(prev => [...prev, newPeer])
         sendTypingStatusChange({ isTyping }, peerId)
         verifyPeer(newPeer)
       } else {
         const oldUsername =
           peerList[peerIndex].customUsername || getPeerName(userId)
         const newUsername = customUsername || getPeerName(userId)
 
-        const newPeerList = [...peerList]
-        const newPeer = { ...newPeerList[peerIndex], userId, customUsername }
-        newPeerList[peerIndex] = newPeer
-        setPeerList(newPeerList)
+        setPeerList(prev => {
+          const newPeerList = [...prev]
+          const newPeer = { ...newPeerList[peerIndex], userId, customUsername }
+          newPeerList[peerIndex] = newPeer
+
+          return newPeerList
+        })
 
         if (oldUsername !== newUsername) {
           showAlert(`${oldUsername} is now ${newUsername}`)
@@ -471,9 +474,12 @@ export function useRoom(
       )
 
       if (doesPeerExist) {
-        const peerListClone = [...peerList]
-        peerListClone.splice(peerIndex, 1)
-        setPeerList(peerListClone)
+        setPeerList(prev => {
+          const peerListClone = [...prev]
+          peerListClone.splice(peerIndex, 1)
+
+          return peerListClone
+        })
       }
     })
   }