From 33f6c0f5658fccc0419e8209adbfdb72b4de7546 Mon Sep 17 00:00:00 2001 From: "shuwen.wu" Date: Fri, 25 Sep 2026 16:42:50 +0800 Subject: [PATCH] fix: correct role assignment in chat_history and typo in get_embedding - chat_history.py: messages from other agents should be "user" role not "assistant" to maintain proper conversation alternation for models like LLAMA that require user/assistant/user alternation - openai.py: fix NameError in get_embedding exception handler where 'attempt' was used instead of 'attempts' parameter --- agentverse/llms/openai.py | 2 +- agentverse/memory/chat_history.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/agentverse/llms/openai.py b/agentverse/llms/openai.py index 11b17f02..7b764e75 100644 --- a/agentverse/llms/openai.py +++ b/agentverse/llms/openai.py @@ -511,6 +511,6 @@ def get_embedding(text: str, attempts=3) -> np.array: ).model_dump_json(indent=2) return tuple(embedding) except Exception as e: - attempt += 1 + attempts += 1 logger.error(f"Error {e} when requesting openai models. Retrying") raise diff --git a/agentverse/memory/chat_history.py b/agentverse/memory/chat_history.py index 65d00751..7f427610 100644 --- a/agentverse/memory/chat_history.py +++ b/agentverse/memory/chat_history.py @@ -101,7 +101,7 @@ async def to_messages( continue messages.append( { - "role": "assistant", + "role": "user", "content": f"[{message.sender}]: {message.content}", } )