From c03858733d7f13f5d7b1de0c235ca106632f4131 Mon Sep 17 00:00:00 2001 From: sicnuyudidi <23434080+sicnuyudidi@users.noreply.github.com> Date: Thu, 16 Apr 2026 11:51:26 +0800 Subject: [PATCH] fix: pass correct arguments in summary model fallback retry MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit _generate_summary() takes (turns_to_summarize, focus_topic) but the summary model fallback path passed (messages, summary_budget) — where 'messages' is not even in scope, causing a NameError. Fix the recursive call to pass the correct variables so the fallback to the main model actually works when the summary model is unavailable. Fixes: #10721 --- agent/context_compressor.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/agent/context_compressor.py b/agent/context_compressor.py index f959b92a6..f8036851f 100644 --- a/agent/context_compressor.py +++ b/agent/context_compressor.py @@ -848,7 +848,7 @@ The user has requested that this compaction PRIORITISE preserving all informatio ) self.summary_model = "" # empty = use main model self._summary_failure_cooldown_until = 0.0 # no cooldown - return self._generate_summary(turns_to_summarize) # retry immediately + return self._generate_summary(turns_to_summarize, focus_topic=focus_topic) # retry immediately # Transient errors (timeout, rate limit, network) — shorter cooldown _transient_cooldown = 60