clamp chat history summary to the column length so completion status is saved

Author: iyernaveenr

server/app/domains/chat/api/history_controller.py

@@ -179,6 +179,10 @@ async def update_chat_history(
         raise HTTPException(status_code=403, detail="You are not allowed to update this chat history")
 
     update_data = data.model_dump(exclude_unset=True)
+    # Clamp summary to the column length so an over-long value cannot fail
+    # the UPDATE and roll back the accompanying status change.
+    if update_data.get("summary"):
+        update_data["summary"] = update_data["summary"][:1024]
     history.update_fields(update_data)
     if "project_name" in update_data:
         _sync_project_display_name(

src/store/chatStore.ts

@@ -627,6 +627,11 @@ async function uploadTaskFiles(
   return results;
 }
 
+/** Max length the server stores for a chat history summary; longer values
+ * are rejected by the database, which would roll back the status update sent
+ * with it. The full text is preserved in the task's end step. */
+const MAX_CHAT_HISTORY_SUMMARY_LENGTH = 1024;
+
 export interface StartTaskOptions {
   preserveTaskId?: boolean;
   skipHistoryCreate?: boolean;
@@ -2367,7 +2372,9 @@ const chatStore = (initial?: Partial<ChatStore>) =>
                 agentMessages.data!.summary_task?.split('|')[0] || '';
               const obj = {
                 project_name: projectName,
-                summary: agentMessages.data!.summary_task?.split('|')[1] || '',
+                summary: (
+                  agentMessages.data!.summary_task?.split('|')[1] || ''
+                ).slice(0, MAX_CHAT_HISTORY_SUMMARY_LENGTH),
                 status: 1,
                 tokens: getTokens(currentTaskId),
               };
@@ -2750,7 +2757,9 @@ const chatStore = (initial?: Partial<ChatStore>) =>
                   tasks[currentTaskId].summaryTask.split('|')[0];
                 const obj = {
                   project_name: projectName,
-                  summary: tasks[currentTaskId].summaryTask.split('|')[1],
+                  summary: (
+                    tasks[currentTaskId].summaryTask.split('|')[1] || ''
+                  ).slice(0, MAX_CHAT_HISTORY_SUMMARY_LENGTH),
                   status: 1,
                   tokens: getTokens(currentTaskId),
                 };
@@ -3618,7 +3627,10 @@ const chatStore = (initial?: Partial<ChatStore>) =>
                 const projectName = parts[0] || '';
                 const obj = {
                   project_name: projectName,
-                  summary: completionSummary,
+                  summary: completionSummary.slice(
+                    0,
+                    MAX_CHAT_HISTORY_SUMMARY_LENGTH
+                  ),
                   status: wasStoppedByUser ? 1 : 2,
                   tokens: getTokens(currentTaskId),
                 };