fix: regenerate message should not include the assistant's previous response (#2583)

core/src/browser/extensions/engines/OAIEngine.ts

@@ -102,7 +102,7 @@ export abstract class OAIEngine extends AIEngine {
         events.emit(MessageEvent.OnMessageUpdate, message)
       },
       error: async (err: any) => {
-        console.error(`Inference error: ${JSON.stringify(err, null, 2)}`)
+        console.error(`Inference error:`, err)
         if (this.isCancelled || message.content.length) {
           message.status = MessageStatus.Stopped
           events.emit(MessageEvent.OnMessageUpdate, message)

web/hooks/useSendChatMessage.ts

@@ -102,7 +102,9 @@ export default function useSendChatMessage() {
       activeThreadRef.current.assistants[0].model ?? selectedModelRef.current,
       activeThreadRef.current,
       currentMessages
-    ).addSystemMessage(activeThreadRef.current.assistants[0]?.instructions)
+    )
+      .addSystemMessage(activeThreadRef.current.assistants[0]?.instructions)
+      .removeLastAssistantMessage()
 
     const modelId =
       selectedModelRef.current?.id ??

web/utils/messageRequestBuilder.ts

@@ -117,6 +117,19 @@ export class MessageRequestBuilder {
     return this
   }
 
+  removeLastAssistantMessage() {
+    const lastMessageIndex = this.messages.length - 1
+    if (
+      this.messages.length &&
+      this.messages[lastMessageIndex] &&
+      this.messages[lastMessageIndex].role === ChatCompletionRole.Assistant
+    ) {
+      this.messages.pop()
+    }
+
+    return this
+  }
+
   build(): MessageRequest {
     return {
       id: this.msgId,