Updates from code review.

api/routers/chat.py

@@ -113,10 +113,8 @@ async def add_or_remove_reaction(
     metadata = message.metadata or {}
 
     if reaction is None:
-        # Remove the reaction
         metadata.pop('reaction', None)
     else:
-        # Set or update the reaction
         metadata['reaction'] = reaction
 
     honcho.apps.users.sessions.messages.update(

www/app/page.tsx

@@ -163,9 +163,7 @@ export default function Home() {
           }
           return msg;
         });
-      }, false);
-
-      mutateMessages();
+      }, true);
     } catch (error) {
       console.error("Failed to update reaction:", error);
     }

www/utils/api.ts

@@ -217,40 +217,4 @@ export class API {
       throw error;
     }
   }
-
-  async getReaction(
-    conversationId: string,
-    messageId: string,
-  ): Promise<{ reaction: Reaction }> {
-    try {
-      const response = await fetch(
-        `${this.url}/api/reaction/${messageId}?user_id=${this.userId}&conversation_id=${conversationId}`,
-        {
-          method: "GET",
-          headers: {
-            "Content-Type": "application/json",
-          },
-        },
-      );
-
-      if (!response.ok) {
-        throw new Error("Failed to get reaction");
-      }
-
-      const data = await response.json();
-
-      // Validate the reaction
-      if (
-        data.reaction !== null &&
-        !["thumbs_up", "thumbs_down"].includes(data.reaction)
-      ) {
-        throw new Error("Invalid reaction received from server");
-      }
-
-      return data as { reaction: Reaction };
-    } catch (error) {
-      console.error("Error getting reaction:", error);
-      throw error;
-    }
-  }
 }