fix: do not send duplicate messages on regenerating

Author: louis-jan

web-app/src/hooks/useChat.ts

@@ -247,8 +247,7 @@ export const useChat = () => {
           messages,
           currentAssistant?.instructions
         )
-
-        builder.addUserMessage(message)
+        if (troubleshooting) builder.addUserMessage(message)
 
         let isCompleted = false
 

web-app/src/lib/messages.ts

@@ -26,7 +26,7 @@ export class CompletionMessagesBuilder {
               content:
                 msg.role === 'assistant'
                   ? this.normalizeContent(msg.content[0]?.text?.value || '.')
-                  : (msg.content[0]?.text?.value || '.'),
+                  : msg.content[0]?.text?.value || '.',
             }) as ChatCompletionMessageParam
         )
     )
@@ -37,6 +37,10 @@ export class CompletionMessagesBuilder {
    * @param content - The content of the user message.
    */
   addUserMessage(content: string) {
+    // Ensure no consecutive user messages
+    if (this.messages[this.messages.length - 1]?.role === 'user') {
+      this.messages.pop()
+    }
     this.messages.push({
       role: 'user',
       content: content,