feat(memory): chat history memory support

camel/agents/chat_agent.py

@@ -24,35 +24,16 @@
 from camel.agents import BaseAgent
 from camel.configs import BaseConfig, ChatGPTConfig
 from camel.functions import OpenAIFunction
+from camel.memories import BaseMemory, ChatHistoryMemory, MemoryRecord
+from camel.memories.context_creator.default import DefaultContextCreator
 from camel.messages import BaseMessage, FunctionCallingMessage, OpenAIMessage
 from camel.models import BaseModelBackend, ModelFactory
 from camel.responses import ChatAgentResponse
-from camel.terminators import ResponseTerminator, TokenLimitTerminator
-from camel.typing import ModelType, RoleType
+from camel.terminators import ResponseTerminator
+from camel.typing import ModelType, OpenAIBackendRole, RoleType
 from camel.utils import get_model_encoding, openai_api_key_required
 
 
-@dataclass(frozen=True)
-class ChatRecord:
-    r"""Historical records of who made what message.
-
-    Attributes:
-        role_at_backend (str): Role of the message that mirrors OpenAI
-            message role that may be `system` or `user` or `assistant`.
-        message (BaseMessage): Message payload.
-    """
-    role_at_backend: str
-    message: BaseMessage
-
-    def to_openai_message(self):
-        r"""Converts the payload message to OpenAI-compatible format.
-
-        Returns:
-            OpenAIMessage: OpenAI-compatible message
-        """
-        return self.message.to_openai_message(self.role_at_backend)
-
-
 @dataclass(frozen=True)
 class FunctionCallingRecord:
     r"""Historical records of functions called in the conversation.
@@ -86,8 +67,10 @@ class ChatAgent(BaseAgent):
         system_message (BaseMessage): The system message for the chat agent.
         model (ModelType, optional): The LLM model to use for generating
             responses. (default :obj:`ModelType.GPT_3_5_TURBO`)
-        model_config (Any, optional): Configuration options for the LLM model.
-            (default: :obj:`None`)
+        model_config (BaseConfig, optional): Configuration options for the
+            LLM model. (default: :obj:`None`)
+        memory (BaseMemory, optional): The agent memory for managing chat
+            messages. (default: :obj:`ChatHistoryMemory()`)
         message_window_size (int, optional): The maximum number of previous
             messages to include in the context window. If `None`, no windowing
             is performed. (default: :obj:`None`)
@@ -105,7 +88,9 @@ def __init__(
         system_message: BaseMessage,
         model: Optional[ModelType] = None,
         model_config: Optional[BaseConfig] = None,
+        memory: Optional[BaseMemory] = None,
         message_window_size: Optional[int] = None,
+        token_limit: Optional[int] = None,
         output_language: Optional[str] = None,
         function_list: Optional[List[OpenAIFunction]] = None,
         response_terminators: Optional[List[ResponseTerminator]] = None,
@@ -121,7 +106,6 @@ def __init__(
 
         self.model: ModelType = (model if model is not None else
                                  ModelType.GPT_3_5_TURBO)
-        self.message_window_size: Optional[int] = message_window_size
 
         self.func_dict: Dict[str, Callable] = {}
         if function_list is not None:
@@ -131,13 +115,16 @@ def __init__(
 
         self.model_backend: BaseModelBackend = ModelFactory.create(
             self.model, self.model_config.__dict__)
-        self.model_token_limit: int = self.model_backend.token_limit
+        self.model_token_limit = token_limit or self.model_backend.token_limit
+        context_creator = DefaultContextCreator(
+            self.model_backend.token_counter,
+            self.model_token_limit,
+        )
+        self.memory: BaseMemory = memory or ChatHistoryMemory(
+            context_creator, window_size=message_window_size)
 
         self.terminated: bool = False
-        self.token_limit_terminator = TokenLimitTerminator(
-            self.model_token_limit)
         self.response_terminators = response_terminators or []
-        self.stored_messages: List[ChatRecord]
         self.init_messages()
 
     def reset(self):
@@ -149,7 +136,6 @@ def reset(self):
         """
         self.terminated = False
         self.init_messages()
-        self.token_limit_terminator.reset()
         for terminator in self.response_terminators:
             terminator.reset()
 
@@ -182,6 +168,16 @@ def is_function_calling_enabled(self) -> bool:
         """
         return len(self.func_dict) > 0
 
+    def update_memory(self, message: BaseMessage,
+                      role: OpenAIBackendRole) -> None:
+        r"""Updates the agent memory with a new message.
+        Args:
+            message (BaseMessage): The new message to add to the stored
+                messages.
+            role (OpenAIBackendRole): The backend role type.
+        """
+        self.memory.write_record(MemoryRecord(message, role))
+
     def set_output_language(self, output_language: str) -> BaseMessage:
         r"""Sets the output language for the system message. This method
         updates the output language for the system message. The output
@@ -232,24 +228,10 @@ def init_messages(self) -> None:
         r"""Initializes the stored messages list with the initial system
         message.
         """
-        self.stored_messages = [ChatRecord('system', self.system_message)]
-
-    def update_messages(self, role: str,
-                        message: BaseMessage) -> List[ChatRecord]:
-        r"""Updates the stored messages list with a new message.
-
-        Args:
-            role (str): Role of the message at the backend.
-            message (BaseMessage): The new message to add to the stored
-                messages.
-
-        Returns:
-            List[BaseMessage]: The updated stored messages.
-        """
-        if role not in {'system', 'user', 'assistant', 'function'}:
-            raise ValueError(f"Unsupported role {role}")
-        self.stored_messages.append(ChatRecord(role, message))
-        return self.stored_messages
+        system_record = MemoryRecord(self.system_message,
+                                     OpenAIBackendRole.SYSTEM)
+        self.memory.clear()
+        self.memory.write_record(system_record)
 
     def submit_message(self, message: BaseMessage) -> None:
         r"""Submits the externally provided message as if it were an answer of
@@ -260,7 +242,8 @@ def submit_message(self, message: BaseMessage) -> None:
             message (BaseMessage): An external message to be added as an
                 assistant response.
         """
-        self.stored_messages.append(ChatRecord('assistant', message))
+        record = MemoryRecord(message, OpenAIBackendRole.ASSISTANT)
+        self.memory.write_record(record)
 
     @retry(wait=wait_exponential(min=5, max=60), stop=stop_after_attempt(5))
     @openai_api_key_required
@@ -282,23 +265,21 @@ def step(
                 a boolean indicating whether the chat session has terminated,
                 and information about the chat session.
         """
-        messages = self.update_messages('user', input_message)
+        record = MemoryRecord(input_message, OpenAIBackendRole.USER)
+        self.memory.write_record(record)
 
         output_messages: List[BaseMessage]
         info: Dict[str, Any]
         called_funcs: List[FunctionCallingRecord] = []
         while True:
             # Format messages and get the token number
             openai_messages: Optional[List[OpenAIMessage]]
-            num_tokens: int
-            openai_messages, num_tokens = self.preprocess_messages(messages)
 
-            # Terminate when number of tokens exceeds the limit
-            self.terminated, termination_reason = \
-                self.token_limit_terminator.is_terminated(num_tokens)
-            if self.terminated and termination_reason is not None:
-                return self.step_token_exceed(num_tokens, called_funcs,
-                                              termination_reason)
+            try:
+                openai_messages, num_tokens = self.memory.get_context()
+            except RuntimeError as e:
+                return self.step_token_exceed(e.args[1], called_funcs,
+                                              "max_tokens_exceeded")
 
             # Obtain LLM's response and validate it
             response = self.model_backend.run(openai_messages)
@@ -311,16 +292,23 @@ def step(
                 output_messages, finish_reasons, usage_dict, response_id = (
                     self.handle_stream_response(response, num_tokens))
 
-            if self.is_function_calling_enabled(
-            ) and finish_reasons[0] == 'function_call':
+            if (self.is_function_calling_enabled()
+                    and finish_reasons[0] == 'function_call'):
                 # Do function calling
                 func_assistant_msg, func_result_msg, func_record = (
                     self.step_function_call(response))
 
                 # Update the messages
-                messages = self.update_messages('assistant',
-                                                func_assistant_msg)
-                messages = self.update_messages('function', func_result_msg)
+                func_assistant_record = MemoryRecord(
+                    func_assistant_msg,
+                    OpenAIBackendRole.ASSISTANT,
+                )
+                func_result_record = MemoryRecord(
+                    func_result_msg,
+                    OpenAIBackendRole.FUNCTION,
+                )
+                self.memory.write_records(
+                    [func_assistant_record, func_result_record])
                 called_funcs.append(func_record)
             else:
                 # Function calling disabled or chat stopped
@@ -352,35 +340,6 @@ def step(
 
         return ChatAgentResponse(output_messages, self.terminated, info)
 
-    def preprocess_messages(
-            self,
-            messages: List[ChatRecord]) -> Tuple[List[OpenAIMessage], int]:
-        r"""Truncate the list of messages if message window is defined and
-        the current length of message list is beyond the window size. Then
-        convert the list of messages to OpenAI's input format and calculate
-        the number of tokens.
-
-        Args:
-            messages (List[ChatRecord]): The list of structs containing
-                information about previous chat messages.
-
-        Returns:
-            tuple: A tuple containing the truncated list of messages in
-                OpenAI's input format and the number of tokens.
-        """
-
-        if (self.message_window_size
-                is not None) and (len(messages) > self.message_window_size):
-            messages = [ChatRecord('system', self.system_message)
-                        ] + messages[-self.message_window_size:]
-
-        openai_messages: List[OpenAIMessage]
-        openai_messages = [record.to_openai_message() for record in messages]
-        num_tokens = self.model_backend.count_tokens_from_messages(
-            openai_messages)
-
-        return openai_messages, num_tokens
-
     def validate_model_response(self, response: Any) -> None:
         r"""Validate the type of the response returned by the model.
 
@@ -480,7 +439,7 @@ def step_token_exceed(self, num_tokens: int,
             ChatAgentResponse: The struct containing trivial outputs and
                 information about token number and called functions.
         """
-
+        self.terminated = True
         output_messages: List[BaseMessage] = []
 
         info = self.get_info(

camel/agents/critic_agent.py

@@ -18,9 +18,10 @@
 from colorama import Fore
 
 from camel.agents import ChatAgent
+from camel.memories import BaseMemory, MemoryRecord
 from camel.messages import BaseMessage
 from camel.responses import ChatAgentResponse
-from camel.typing import ModelType
+from camel.typing import ModelType, OpenAIBackendRole
 from camel.utils import get_first_int, print_text_animated
 
 
@@ -49,13 +50,14 @@ def __init__(
         system_message: BaseMessage,
         model: ModelType = ModelType.GPT_3_5_TURBO,
         model_config: Optional[Any] = None,
+        memory: Optional[BaseMemory] = None,
         message_window_size: int = 6,
         retry_attempts: int = 2,
         verbose: bool = False,
         logger_color: Any = Fore.MAGENTA,
     ) -> None:
         super().__init__(system_message, model=model,
-                         model_config=model_config,
+                         model_config=model_config, memory=memory,
                          message_window_size=message_window_size)
         self.options_dict: Dict[str, str] = dict()
         self.retry_attempts = retry_attempts
@@ -106,7 +108,9 @@ def get_option(self, input_message: BaseMessage) -> str:
                 raise RuntimeError("Critic step failed.")
 
             critic_msg = critic_response.msg
-            self.update_messages('assistant', critic_msg)
+            critic_record = MemoryRecord(critic_msg,
+                                         OpenAIBackendRole.ASSISTANT)
+            self.memory.write_records([critic_record])
             if self.verbose:
                 print_text_animated(self.logger_color + "\n> Critic response: "
                                     f"\x1b[3m{critic_msg.content}\x1b[0m\n")

camel/memories/__init__.py

@@ -0,0 +1,19 @@
+# =========== Copyright 2023 @ CAMEL-AI.org. All Rights Reserved. ===========
+# Licensed under the Apache License, Version 2.0 (the “License”);
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an “AS IS” BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+# =========== Copyright 2023 @ CAMEL-AI.org. All Rights Reserved. ===========
+
+from .base import BaseMemory
+from .chat_history_memory import ChatHistoryMemory
+from .memory_record import MemoryRecord
+
+__all__ = ['BaseMemory', 'ChatHistoryMemory', 'MemoryRecord']