From 1ff93a23e15e5482429e33f0fb7ca88d2fc4e56a Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Sun, 16 Nov 2025 09:49:46 +0000
Subject: [PATCH 1/2] Initial plan


From c8a21ea75e2e910ff80d66b51803241cdfbf0d8b Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Sun, 16 Nov 2025 09:56:41 +0000
Subject: [PATCH 2/2] fix: add error handling for LLM response stream in
 summarizer

- Check err in response stream iteration to catch LLM API failures
- Fix Parts type to []*genai.Part as per genai API
- Return descriptive error when LLM generation fails

Co-authored-by: raphaelmansuy <1003084+raphaelmansuy@users.noreply.github.com>
---
 adk-code/internal/session/compaction/summarizer.go | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/adk-code/internal/session/compaction/summarizer.go b/adk-code/internal/session/compaction/summarizer.go
index 33043e6..9229f53 100644
--- a/adk-code/internal/session/compaction/summarizer.go
+++ b/adk-code/internal/session/compaction/summarizer.go
@@ -46,8 +46,8 @@ func (ls *LLMSummarizer) Summarize(
 		Contents: []*genai.Content{
 			{
 				Role: "user",
-				Parts: []genai.Part{
-					genai.Text(prompt),
+				Parts: []*genai.Part{
+					{Text: prompt},
 				},
 			},
 		},
@@ -59,7 +59,10 @@ func (ls *LLMSummarizer) Summarize(
 	var usageMetadata *genai.GenerateContentResponseUsageMetadata
 
 	responseStream := ls.llm.GenerateContent(ctx, llmRequest, false)
-	for resp := range responseStream {
+	for resp, err := range responseStream {
+		if err != nil {
+			return nil, fmt.Errorf("LLM generation error: %w", err)
+		}
 		if resp == nil {
 			continue
 		}