fix: pass correct arguments in summary model fallback retry
_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
This commit is contained in:
@@ -848,7 +848,7 @@ The user has requested that this compaction PRIORITISE preserving all informatio
|
|||||||
)
|
)
|
||||||
self.summary_model = "" # empty = use main model
|
self.summary_model = "" # empty = use main model
|
||||||
self._summary_failure_cooldown_until = 0.0 # no cooldown
|
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 errors (timeout, rate limit, network) — shorter cooldown
|
||||||
_transient_cooldown = 60
|
_transient_cooldown = 60
|
||||||
|
|||||||
Reference in New Issue
Block a user