fix: add fallback id to messages if none provided (#3584)

ui/desktop/src/hooks/useMessageStream.ts

@@ -4,6 +4,12 @@ import { getSecretKey } from '../config';
 import { Message, createUserMessage, hasCompletedToolCalls } from '../types/message';
 import { getSessionHistory } from '../api';
 
+let messageIdCounter = 0;
+
+function generateMessageId(): string {
+  return `msg-${Date.now()}-${++messageIdCounter}`;
+}
+
 // Ensure TextDecoder is available in the global scope
 const TextDecoder = globalThis.TextDecoder;
 
@@ -297,6 +303,8 @@ export function useMessageStream({
                     // Create a new message object with the properties preserved or defaulted
                     const newMessage = {
                       ...parsedEvent.message,
+                      // Ensure the message has an ID - if not provided, generate one
+                      id: parsedEvent.message.id || generateMessageId(),
                       // Only set to true if it's undefined (preserve false values)
                       display:
                         parsedEvent.message.display === undefined
@@ -358,7 +366,7 @@ export function useMessageStream({
                     // If this is a token limit error, create a contextLengthExceeded message instead of throwing
                     if (isTokenLimitError) {
                       const contextMessage: Message = {
-                        id: `context-${Date.now()}`,
+                        id: generateMessageId(),
                         role: 'assistant',
                         created: Math.floor(Date.now() / 1000),
                         content: [
@@ -461,7 +469,7 @@ export function useMessageStream({
     async (requestMessages: Message[]) => {
       try {
         mutateLoading(true);
-        mutateWaiting(true);  // Start in waiting state
+        mutateWaiting(true); // Start in waiting state
         mutateStreaming(false);
         setError(undefined);
 
@@ -539,7 +547,17 @@ export function useMessageStream({
       }
     },
     // eslint-disable-next-line react-hooks/exhaustive-deps
-    [api, processMessageStream, mutateLoading, mutateWaiting, mutateStreaming, setError, onResponse, onError, maxSteps]
+    [
+      api,
+      processMessageStream,
+      mutateLoading,
+      mutateWaiting,
+      mutateStreaming,
+      setError,
+      onResponse,
+      onError,
+      maxSteps,
+    ]
   );
 
   // Append a new message and send request
@@ -637,6 +655,7 @@ export function useMessageStream({
 
       // Create a tool response message
       const toolResponseMessage: Message = {
+        id: generateMessageId(),
         role: 'user' as const,
         created: Math.floor(Date.now() / 1000),
         content: [