don't broadcast null userIds

automerge · Feb 6, 2024 · ca40caa · ca40caa
1 parent 6b19210
commit ca40caa
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/packages/automerge-repo-react-hooks/src/useLocalAwareness.ts b/packages/automerge-repo-react-hooks/src/useLocalAwareness.ts
@@ -47,6 +47,12 @@ export const useLocalAwareness = ({
   }
 
   useEffect(() => {
+    // Don't broadcast if userId isn't set: this avoids bogus broadcasts
+    // during the loading of a userId document.
+    if (!userId) {
+      return
+    }
+
     // Send periodic heartbeats
     const heartbeat = () =>
       void handle.broadcast([userId, localStateRef.current])