fix(ui): display error on summarization instead of leaving spinning

taciturnaxolotl · andreynering · commit 6bdd233b0310 · 2026-05-06T17:38:29.000-03:00
diff --git a/internal/agent/agent.go b/internal/agent/agent.go
@@ -703,6 +703,12 @@ func (a *sessionAgent) Summarize(ctx context.Context, sessionID string, opts fan
 			deleteErr := a.messages.Delete(ctx, summaryMessage.ID)
 			return deleteErr
 		}
+		// Mark the summary message as finished with an error so the UI
+		// stops spinning.
+		summaryMessage.AddFinish(message.FinishReasonError, "Summarization Error", err.Error())
+		if updateErr := a.messages.Update(ctx, summaryMessage); updateErr != nil {
+			return updateErr
+		}
 		return err
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -703,6 +703,12 @@ func (a *sessionAgent) Summarize(ctx context.Context, sessionID string, opts fan`
`703`	`703`	`deleteErr := a.messages.Delete(ctx, summaryMessage.ID)`
`704`	`704`	`return deleteErr`
`705`	`705`	`}`
	`706`	`+ // Mark the summary message as finished with an error so the UI`
	`707`	`+ // stops spinning.`
	`708`	`+ summaryMessage.AddFinish(message.FinishReasonError, "Summarization Error", err.Error())`
	`709`	`+ if updateErr := a.messages.Update(ctx, summaryMessage); updateErr != nil {`
	`710`	`+ return updateErr`
	`711`	`+ }`
`706`	`712`	`return err`
`707`	`713`	`}`
`708`	`714`