PR #14316: fix: preserve proxy tool call history for Open WebUI sessions

• Repository: NousResearch/hermes-agent
• Author: LeonSGP43
• State: open | merged: False
• Link: https://github.com/NousResearch/hermes-agent/pull/14316

Description (problem / solution / changelog)

Summary

• preserve assistant tool-call turns and tool result messages when gateway proxy mode forwards local history
• keep filtering out session metadata and system history, but stop collapsing proxy history to text-only turns
• update proxy-mode regression coverage to assert tool continuity is preserved

Fixes #14270

Testing

• python3 -m pytest -o addopts='' tests/gateway/test_proxy_mode.py

Changed files

• gateway/run.py (modified, +24/-4)
• tests/gateway/test_proxy_mode.py (modified, +8/-6)

PR #3: fix: preserve tool-turn context in chat/completions (#14270)

• Repository: aideautomation/hermes-agent
• Author: aideautomation
• State: closed | merged: True
• Link: https://github.com/aideautomation/hermes-agent/pull/3

Description (problem / solution / changelog)

결론

Open WebUI 계열 클라이언트에서 도구 호출 맥락이 다음 턴에 소실되는 문제(#14270)를 chat/completions 경로에서 재현 테스트 후 수정했습니다.

원인

• /v1/chat/completions 파서가 user/assistant만 conversation_history에 포함하고 tool role은 버렸습니다.
• 또한 마지막 메시지를 무조건 user_message로 간주해, 일부 클라이언트에서 trailing tool/assistant 턴이 섞일 때 입력 추출이 불안정했습니다.
• assistant tool-call 턴이 content: "" + tool_calls 형태로 전달되면, 이전 턴에 도구를 불렀다는 힌트가 완전히 사라졌습니다.

수정 내용

1. tool role 메시지를 conversation_history에 포함
2. assistant 턴이 empty content + tool_calls인 경우, 툴 이름 기반 marker를 history에 보존
   • 예: [assistant issued tool calls: read_file]
3. user_message 추출을 "마지막 메시지"가 아닌 "마지막 유효 user 턴" 기준으로 변경

테스트

• test_tool_messages_and_assistant_tool_calls_are_preserved
• test_last_user_message_selected_when_trailing_tool_message_exists
• 기존 test_conversation_history_passed 회귀 포함

실행:

• scripts/run_tests.sh tests/gateway/test_api_server.py::TestChatCompletionsEndpoint

결과:

• 17 passed

연계 이슈

• Closes #14270
• #14238, #14210 은 별도 패치로 이어서 진행 예정 (레이스/FD 누수 성격으로 범위가 큼)

Changed files

• agent/error_classifier.py (modified, +24/-13)
• gateway/platforms/api_server.py (modified, +37/-5)
• run_agent.py (modified, +1/-1)
• tests/agent/test_error_classifier.py (modified, +23/-0)
• tests/gateway/test_api_server.py (modified, +69/-0)
• tests/run_agent/test_run_agent.py (modified, +20/-0)
• tests/tools/test_registry.py (modified, +24/-0)
• tests/tools/test_todo_tool.py (modified, +23/-0)
• tools/registry.py (modified, +43/-1)
• tools/todo_tool.py (modified, +16/-2)