mirror of
https://github.com/open-webui/open-webui.git
synced 2026-05-06 19:08:59 -05:00
[PR #21941] [CLOSED] fix: add synthetic tool responses for orphaned tool calls #65209
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/open-webui/open-webui/pull/21941
Author: @maxkuminov
Created: 2/27/2026
Status: ❌ Closed
Base:
dev← Head:fix/orphaned-tool-calls📝 Commits (10+)
fe6783cMerge pull request #19030 from open-webui/devfc05e0aMerge pull request #19405 from open-webui/deve3faec6Merge pull request #19416 from open-webui/dev9899293Merge pull request #19448 from open-webui/dev140605eMerge pull request #19462 from open-webui/dev6f1486fMerge pull request #19466 from open-webui/devd95f533Merge pull request #19729 from open-webui/deva7271530.6.43 (#20093)6adde20Merge pull request #20394 from open-webui/devf9b0534Merge pull request #20522 from open-webui/dev📊 Changes
2 files changed (+123 additions, -0 deletions)
View changed files
➕
backend/open_webui/test/test_convert_output.py(+106 -0)📝
backend/open_webui/utils/misc.py(+17 -0)📄 Description
Pull Request Checklist
devfix:Changelog Entry
Description
When a native function call (e.g., web search) times out or fails without producing a
function_call_output,convert_output_to_messages()emits an assistant message withtool_callsbut no followingtoolrole message. All providers (OpenAI, Azure, Google) reject this as invalid, returning "Provider returned error" for every subsequent message in that chat — permanently breaking the conversation.This is a minimal, targeted fix: after the final
flush_pending()call inconvert_output_to_messages(), check if the last message is an assistant withtool_callsand append synthetictoolrole messages so the conversation remains valid.Fixed
[Tool execution did not return a result]are appended so providers accept the message sequence.Fixes #14577, #12135, #18121, #9435
Additional Information
Files changed:
backend/open_webui/utils/misc.py— 11-line addition afterflush_pending()inconvert_output_to_messages()backend/open_webui/test/test_convert_output.py— 5 new test casesTest plan:
Contributor License Agreement
By submitting this pull request, I confirm that I have read and fully agree to the Contributor License Agreement (CLA), and I am providing my contributions under its terms.
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.