Fix issue where chat history is saved redundantly

[merrygoround-of-life]

This is to fix the following issue:

1. At initialization, AgentActor performs the following tasks:
   a. Fills self._chat_history from app_session_data.
   b. Retrieves a list of conversations up to self._config["chat_history_limit"] from self._chat_history.
   c. Appends the last retrieved conversation list to self._agent_messages (ultimately used for openai chat completion).
2. Subsequently, when the openai client of AgentActor receives finish_reason=stop, it saves chat_history to app_session_data as self._chat_history + self._agent_messages. As a result, the conversation list obtained in b exists on both sides, leading to duplicate conversation storage.

To solve this issue, when saving app_session_data, modify it to exclude conversations up to chat_history_limit and save them to avoid unnecessary duplication.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
llmstack	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Mar 7, 2024 1:07pm

[ajhai]

Thanks for catching this @merrygoround-of-life!