fix: remove prepareConnection dependency in useEffect

This was causing prepareConnection to get run constantly since its
reference depends on restOptions which changes every call.

The way I see it, if a user is changing their options and wants
prepareconnection to be run for something beyond the initial set
(probably unusual), they can call it themselves on the returned
session object.

Author: 1egoman

packages/react/src/hooks/useSession.ts

@@ -486,12 +486,17 @@ export function useSession(
     // room.prepareConnection(tokenSource, { tokenSourceOptions }),
     await room.prepareConnection(credentials.serverUrl, credentials.participantToken);
   }, [tokenSourceFetch, room]);
-  React.useEffect(() => {
-    prepareConnection().catch((err) => {
-      // FIXME: figure out a better logging solution?
-      console.warn('WARNING: Room.prepareConnection failed:', err);
-    });
-  }, [prepareConnection]);
+  React.useEffect(
+    () => {
+      prepareConnection().catch((err) => {
+        // FIXME: figure out a better logging solution?
+        console.warn('WARNING: Room.prepareConnection failed:', err);
+      });
+    },
+    [
+      /* note: no prepareConnection here, this effect should only ever run once! */
+    ],
+  );
 
   return React.useMemo(
     () => ({