Guard chatStore setters against no-op updates

chatLocation and chatVisible are persisted through useUserLocalStorage,
so every assignment writes to localStorage and notifies watchers. Guard
each setter against same-value writes so repeated calls (e.g. hideChat
on every /chatgxy route hit, setActiveChatId on every selection change)
don't thrash downstream consumers.

Author: dannon

client/src/stores/chatStore.ts

@@ -16,25 +16,33 @@ export const useChatStore = defineStore("chatStore", () => {
 
     function showChat(chatId?: string | null) {
         if (chatId !== undefined) {
-            activeChatId.value = chatId;
+            setActiveChatId(chatId);
+        }
+        if (!chatVisible.value) {
+            chatVisible.value = true;
         }
-        chatVisible.value = true;
     }
 
     function hideChat() {
-        chatVisible.value = false;
+        if (chatVisible.value) {
+            chatVisible.value = false;
+        }
     }
 
     function toggleChat() {
         chatVisible.value = !chatVisible.value;
     }
 
     function setLocation(loc: ChatLocation) {
-        chatLocation.value = loc;
+        if (chatLocation.value !== loc) {
+            chatLocation.value = loc;
+        }
     }
 
     function setActiveChatId(id: string | null) {
-        activeChatId.value = id;
+        if (activeChatId.value !== id) {
+            activeChatId.value = id;
+        }
     }
 
     return {