Ensure task result is always string

Author: ppsreejith

web/src/state/chat/reducer.ts

@@ -231,12 +231,16 @@ const taskToToolCall = (task: Task): ChatCompletionMessageToolCall => ({
 
 /**
  * Converts a Task result to ActionChatMessageContent
- * Assumes task.result is always a string
+ * Ensures task.result is converted to string before adding to content
  */
 const taskResultToContent = (task: Task): ActionChatMessageContent => {
+  const content = typeof task.result === 'string' 
+    ? task.result 
+    : JSON.stringify(task.result)
+    
   return {
     type: 'BLANK',
-    content: task.result as string,
+    content,
     renderInfo: {
       text: '',
       code: undefined,

web/src/state/store.ts

@@ -523,14 +523,29 @@ const migrations = {
     let newState = {...state}
     newState.settings.useTeamMemory = false
     return newState
+  },
+  59: (state: RootState) => {
+    let newState = {...state}
+    // Ensure all tool call responses in chat messages are strings
+    newState.chat.threads.forEach((thread) => {
+      thread.messages.forEach((message) => {
+        if (message.role === 'tool' && message.content && message.content.content !== undefined) {
+          // If content is not already a string, convert it to string
+          if (typeof message.content.content !== 'string') {
+            message.content.content = JSON.stringify(message.content.content)
+          }
+        }
+      })
+    })
+    return newState
   }
 }
 
 const BLACKLIST = ['billing', 'cache', userStateApi.reducerPath, atlasApi.reducerPath]
 
 const persistConfig = {
   key: 'root',
-  version: 57,
+  version: 59,
   storage,
   blacklist: BLACKLIST,
   // @ts-ignore