Merge pull request #887: fix(loop): preserve interim content as fallback when retry produces empty response
This commit is contained in:
@@ -203,6 +203,7 @@ class AgentLoop:
|
|||||||
final_content = None
|
final_content = None
|
||||||
tools_used: list[str] = []
|
tools_used: list[str] = []
|
||||||
text_only_retried = False
|
text_only_retried = False
|
||||||
|
interim_content: str | None = None # Fallback if retry produces nothing
|
||||||
|
|
||||||
while iteration < self.max_iterations:
|
while iteration < self.max_iterations:
|
||||||
iteration += 1
|
iteration += 1
|
||||||
@@ -249,12 +250,18 @@ class AgentLoop:
|
|||||||
else:
|
else:
|
||||||
final_content = self._strip_think(response.content)
|
final_content = self._strip_think(response.content)
|
||||||
# Some models send an interim text response before tool calls.
|
# Some models send an interim text response before tool calls.
|
||||||
# Give them one retry; don't forward the text to avoid duplicates.
|
# Give them one retry; save the content as fallback in case
|
||||||
|
# the retry produces nothing useful (e.g. model already answered).
|
||||||
if not tools_used and not text_only_retried and final_content:
|
if not tools_used and not text_only_retried and final_content:
|
||||||
text_only_retried = True
|
text_only_retried = True
|
||||||
|
interim_content = final_content
|
||||||
logger.debug("Interim text response (no tools used yet), retrying: {}", final_content[:80])
|
logger.debug("Interim text response (no tools used yet), retrying: {}", final_content[:80])
|
||||||
final_content = None
|
final_content = None
|
||||||
continue
|
continue
|
||||||
|
# Fall back to interim content if retry produced nothing
|
||||||
|
# and no tools were used (if tools ran, interim was truly interim)
|
||||||
|
if not final_content and interim_content and not tools_used:
|
||||||
|
final_content = interim_content
|
||||||
break
|
break
|
||||||
|
|
||||||
return final_content, tools_used
|
return final_content, tools_used
|
||||||
|
|||||||
Reference in New Issue
Block a user