From 4e223ee971f9b9b04b6be3790963d088bb6ea286 Mon Sep 17 00:00:00 2001
From: spencrmartin <spencermartin@squareup.com>
Date: Fri, 14 Nov 2025 13:22:18 -0500
Subject: [PATCH] fix: Remove setChat from Matrix listener dependencies to
 prevent re-renders

- Removed setChat from useEffect dependency array for Matrix message listeners
- setChat was causing constant re-renders and listener teardown/setup
- This was breaking real-time message reception after initial setup
- Matrix listeners now stay stable and don't get recreated on every render
- Should restore real-time bidirectional message sync between users
---
 ui/desktop/src/components/pair.tsx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ui/desktop/src/components/pair.tsx b/ui/desktop/src/components/pair.tsx
index 982240f18a..5a18138bd0 100644
--- a/ui/desktop/src/components/pair.tsx
+++ b/ui/desktop/src/components/pair.tsx
@@ -326,7 +326,7 @@ export default function Pair({
       unsubscribeMessage();
       unsubscribeGooseMessage();
     };
-  }, [isMatrixMode, matrixRoomId, onMessage, onGooseMessage, setChat]);
+  }, [isMatrixMode, matrixRoomId, onMessage, onGooseMessage]); // Removed setChat from dependencies to prevent constant re-renders
 
   const { initialPrompt: recipeInitialPrompt } = useRecipeManager(chat, chat.recipeConfig || null);