lite-agent 0.4.0__tar.gz → 0.4.1__tar.gz

{lite_agent-0.4.0 → lite_agent-0.4.1}/CHANGELOG.md

@@ -1,3 +1,11 @@
+## v0.4.1
+
+[v0.4.0...v0.4.1](https://github.com/Jannchie/lite-agent/compare/v0.4.0...v0.4.1)
+
+### :art: Refactors
+
+- **runner**: replace if-elif with match-case for chunk handling - By [Jannchie](mailto:jannchie@gmail.com) in [e9a8464](https://github.com/Jannchie/lite-agent/commit/e9a8464)
+
 ## v0.4.0
 
 [v0.3.0...v0.4.0](https://github.com/Jannchie/lite-agent/compare/v0.3.0...v0.4.0)

{lite_agent-0.4.0 → lite_agent-0.4.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: lite-agent
-Version: 0.4.0
+Version: 0.4.1
 Summary: A lightweight, extensible framework for building AI agent.
 Author-email: Jianqi Pan <jannchie@gmail.com>
 License: MIT

{lite_agent-0.4.0 → lite_agent-0.4.1}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "lite-agent"
-version = "0.4.0"
+version = "0.4.1"
 description = "A lightweight, extensible framework for building AI agent."
 readme = "README.md"
 authors = [{ name = "Jianqi Pan", email = "jannchie@gmail.com" }]

{lite_agent-0.4.0 → lite_agent-0.4.1}/src/lite_agent/agent.py

@@ -174,9 +174,11 @@ class Agent:
         if self.completion_condition == "call":
             instructions = WAIT_FOR_USER_INSTRUCTIONS_TEMPLATE.render(extra_instructions=None) + "\n\n" + instructions
         return [
-            system_message_to_llm_dict(NewSystemMessage(
-                content=f"You are {self.name}. {instructions}",
-            )),
+            system_message_to_llm_dict(
+                NewSystemMessage(
+                    content=f"You are {self.name}. {instructions}",
+                ),
+            ),
             *converted_messages,
         ]
 

{lite_agent-0.4.0 → lite_agent-0.4.1}/src/lite_agent/runner.py

@@ -30,6 +30,7 @@ from lite_agent.types import (
     UserMessageContent,
     UserTextContent,
 )
+from lite_agent.types.events import AssistantMessageEvent
 
 DEFAULT_INCLUDES: tuple[AgentChunkType, ...] = (
     "completion_raw",
@@ -56,38 +57,31 @@ class Runner:
 
     def _start_assistant_message(self, content: str = "", meta: AssistantMessageMeta | None = None) -> None:
         """Start a new assistant message."""
-        if meta is None:
-            meta = AssistantMessageMeta()
-
-        # Always add text content, even if empty (we can update it later)
-        assistant_content_items: list[AssistantMessageContent] = [AssistantTextContent(text=content)]
         self._current_assistant_message = NewAssistantMessage(
-            content=assistant_content_items,
-            meta=meta,
+            content=[AssistantTextContent(text=content)],
+            meta=meta or AssistantMessageMeta(),
         )
 
-    def _add_to_current_assistant_message(self, content_item: AssistantTextContent | AssistantToolCall | AssistantToolCallResult) -> None:
-        """Add content to the current assistant message."""
+    def _ensure_current_assistant_message(self) -> NewAssistantMessage:
+        """Ensure current assistant message exists and return it."""
         if self._current_assistant_message is None:
             self._start_assistant_message()
+        return self._current_assistant_message  # type: ignore[return-value]
 
-        if self._current_assistant_message is not None:
-            self._current_assistant_message.content.append(content_item)
+    def _add_to_current_assistant_message(self, content_item: AssistantTextContent | AssistantToolCall | AssistantToolCallResult) -> None:
+        """Add content to the current assistant message."""
+        self._ensure_current_assistant_message().content.append(content_item)
 
     def _add_text_content_to_current_assistant_message(self, delta: str) -> None:
         """Add text delta to the current assistant message's text content."""
-        if self._current_assistant_message is None:
-            self._start_assistant_message()
-
-        if self._current_assistant_message is not None:
-            # Find the first text content item and append the delta
-            for content_item in self._current_assistant_message.content:
-                if content_item.type == "text":
-                    content_item.text += delta
-                    return
-            # If no text content found, add new text content
-            new_content = AssistantTextContent(text=delta)
-            self._current_assistant_message.content.append(new_content)
+        message = self._ensure_current_assistant_message()
+        # Find the first text content item and append the delta
+        for content_item in message.content:
+            if content_item.type == "text":
+                content_item.text += delta
+                return
+        # If no text content found, add new text content
+        message.content.append(AssistantTextContent(text=delta))
 
     def _finalize_assistant_message(self) -> None:
         """Finalize the current assistant message and add it to messages."""
@@ -131,7 +125,7 @@ class Runner:
             for i, tool_call in enumerate(transfer_calls):
                 if i == 0:
                     # Execute the first transfer
-                    await self._handle_agent_transfer(tool_call, includes)
+                    await self._handle_agent_transfer(tool_call)
                 else:
                     # Add response for additional transfer calls without executing them
                     self._add_tool_call_result(
@@ -146,7 +140,7 @@ class Runner:
             for i, tool_call in enumerate(return_parent_calls):
                 if i == 0:
                     # Execute the first transfer
-                    await self._handle_parent_transfer(tool_call, includes)
+                    await self._handle_parent_transfer(tool_call)
                 else:
                     # Add response for additional transfer calls without executing them
                     self._add_tool_call_result(
@@ -184,17 +178,16 @@ class Runner:
     ) -> AsyncGenerator[AgentChunk, None]:
         """Run the agent and return a RunResponse object that can be asynchronously iterated for each chunk."""
         includes = self._normalize_includes(includes)
-        if isinstance(user_input, str):
-            user_message = NewUserMessage(content=[UserTextContent(text=user_input)])
-            self.messages.append(user_message)
-        elif isinstance(user_input, (list, tuple)):
-            # Handle sequence of messages
-            for message in user_input:
-                self.append_message(message)
-        else:
-            # Handle single message (BaseModel, TypedDict, or dict)
-            # Type assertion needed due to the complex union type
-            self.append_message(user_input)  # type: ignore[arg-type]
+        match user_input:
+            case str():
+                self.messages.append(NewUserMessage(content=[UserTextContent(text=user_input)]))
+            case list() | tuple():
+                # Handle sequence of messages
+                for message in user_input:
+                    self.append_message(message)
+            case _:
+                # Handle single message (BaseModel, TypedDict, or dict)
+                self.append_message(user_input)  # type: ignore[arg-type]
         return self._run(max_steps, includes, self._normalize_record_path(record_to), context=context)
 
     async def _run(self, max_steps: int, includes: Sequence[AgentChunkType], record_to: Path | None = None, context: Any | None = None) -> AsyncGenerator[AgentChunk, None]:  # noqa: ANN401
@@ -229,62 +222,79 @@ class Runner:
                     msg = f"Unknown API type: {self.api}"
                     raise ValueError(msg)
             async for chunk in resp:
-                if chunk.type in includes:
-                    yield chunk
-                if chunk.type == "assistant_message":
-                    # Start or update assistant message in new format
-                    meta = AssistantMessageMeta(
-                        sent_at=chunk.message.meta.sent_at,
-                        latency_ms=getattr(chunk.message.meta, "latency_ms", None),
-                        total_time_ms=getattr(chunk.message.meta, "output_time_ms", None),
-                    )
-                    # If we already have a current assistant message, just update its metadata
-                    if self._current_assistant_message is not None:
-                        self._current_assistant_message.meta = meta
-                    else:
-                        # Extract text content from the new message format
-                        text_content = ""
-                        if chunk.message.content:
-                            for item in chunk.message.content:
-                                if hasattr(item, "type") and item.type == "text":
-                                    text_content = item.text
-                                    break
-                        self._start_assistant_message(text_content, meta)
-                if chunk.type == "content_delta":
-                    # Accumulate text content to current assistant message
-                    self._add_text_content_to_current_assistant_message(chunk.delta)
-                if chunk.type == "function_call":
-                    # Add tool call to current assistant message
-                    # Keep arguments as string for compatibility with funcall library
-                    tool_call = AssistantToolCall(
-                        call_id=chunk.call_id,
-                        name=chunk.name,
-                        arguments=chunk.arguments or "{}",
-                    )
-                    self._add_to_current_assistant_message(tool_call)
-                if chunk.type == "usage":
-                    # Update the last assistant message with usage data and output_time_ms
-                    usage_time = datetime.now(timezone.utc)
-                    for i in range(len(self.messages) - 1, -1, -1):
-                        current_message = self.messages[i]
-                        if isinstance(current_message, NewAssistantMessage):
-                            # Update usage information
-                            if current_message.meta.usage is None:
-                                current_message.meta.usage = MessageUsage()
-                            current_message.meta.usage.input_tokens = chunk.usage.input_tokens
-                            current_message.meta.usage.output_tokens = chunk.usage.output_tokens
-                            current_message.meta.usage.total_tokens = (chunk.usage.input_tokens or 0) + (chunk.usage.output_tokens or 0)
-
-                            # Calculate output_time_ms if latency_ms is available
-                            if current_message.meta.latency_ms is not None:
-                                # We need to calculate from first output to usage time
-                                # We'll calculate: usage_time - (sent_at - latency_ms)
-                                # This gives us the time from first output to usage completion
-                                # sent_at is when the message was completed, so sent_at - latency_ms approximates first output time
-                                first_output_time_approx = current_message.meta.sent_at - timedelta(milliseconds=current_message.meta.latency_ms)
-                                output_time_ms = int((usage_time - first_output_time_approx).total_seconds() * 1000)
-                                current_message.meta.total_time_ms = max(0, output_time_ms)
-                            break
+                match chunk.type:
+                    case "assistant_message":
+                        # Start or update assistant message in new format
+                        meta = AssistantMessageMeta(
+                            sent_at=chunk.message.meta.sent_at,
+                            latency_ms=getattr(chunk.message.meta, "latency_ms", None),
+                            total_time_ms=getattr(chunk.message.meta, "output_time_ms", None),
+                        )
+                        # If we already have a current assistant message, just update its metadata
+                        if self._current_assistant_message is not None:
+                            self._current_assistant_message.meta = meta
+                        else:
+                            # Extract text content from the new message format
+                            text_content = ""
+                            if chunk.message.content:
+                                for item in chunk.message.content:
+                                    if hasattr(item, "type") and item.type == "text":
+                                        text_content = item.text
+                                        break
+                            self._start_assistant_message(text_content, meta)
+                        # Only yield assistant_message chunk if it's in includes and has content
+                        if chunk.type in includes and self._current_assistant_message is not None:
+                            # Create a new chunk with the current assistant message content
+                            updated_chunk = AssistantMessageEvent(
+                                message=self._current_assistant_message,
+                            )
+                            yield updated_chunk
+                    case "content_delta":
+                        # Accumulate text content to current assistant message
+                        self._add_text_content_to_current_assistant_message(chunk.delta)
+                        # Always yield content_delta chunk if it's in includes
+                        if chunk.type in includes:
+                            yield chunk
+                    case "function_call":
+                        # Add tool call to current assistant message
+                        # Keep arguments as string for compatibility with funcall library
+                        tool_call = AssistantToolCall(
+                            call_id=chunk.call_id,
+                            name=chunk.name,
+                            arguments=chunk.arguments or "{}",
+                        )
+                        self._add_to_current_assistant_message(tool_call)
+                        # Always yield function_call chunk if it's in includes
+                        if chunk.type in includes:
+                            yield chunk
+                    case "usage":
+                        # Update the last assistant message with usage data and output_time_ms
+                        usage_time = datetime.now(timezone.utc)
+                        for i in range(len(self.messages) - 1, -1, -1):
+                            current_message = self.messages[i]
+                            if isinstance(current_message, NewAssistantMessage):
+                                # Update usage information
+                                if current_message.meta.usage is None:
+                                    current_message.meta.usage = MessageUsage()
+                                current_message.meta.usage.input_tokens = chunk.usage.input_tokens
+                                current_message.meta.usage.output_tokens = chunk.usage.output_tokens
+                                current_message.meta.usage.total_tokens = (chunk.usage.input_tokens or 0) + (chunk.usage.output_tokens or 0)
+
+                                # Calculate output_time_ms if latency_ms is available
+                                if current_message.meta.latency_ms is not None:
+                                    # We need to calculate from first output to usage time
+                                    # We'll calculate: usage_time - (sent_at - latency_ms)
+                                    # This gives us the time from first output to usage completion
+                                    # sent_at is when the message was completed, so sent_at - latency_ms approximates first output time
+                                    first_output_time_approx = current_message.meta.sent_at - timedelta(milliseconds=current_message.meta.latency_ms)
+                                    output_time_ms = int((usage_time - first_output_time_approx).total_seconds() * 1000)
+                                    current_message.meta.total_time_ms = max(0, output_time_ms)
+                                break
+                        # Always yield usage chunk if it's in includes
+                        if chunk.type in includes:
+                            yield chunk
+                    case _ if chunk.type in includes:
+                        yield chunk
 
             # Finalize assistant message so it can be found in pending function calls
             self._finalize_assistant_message()
@@ -377,58 +387,50 @@ class Runner:
         resp = self.run(user_input, max_steps, includes, record_to=record_to)
         return await self._collect_all_chunks(resp)
 
-    def _find_pending_tool_calls(self) -> list[AssistantToolCall]:
-        """Find tool calls that don't have corresponding results yet."""
-        # Find pending calls directly in new format messages
-        pending_calls: list[AssistantToolCall] = []
-
-        # Look at the last assistant message for pending tool calls
-        if not self.messages:
-            return pending_calls
-
-        last_message = self.messages[-1]
-        if not isinstance(last_message, NewAssistantMessage):
-            return pending_calls
+    def _analyze_last_assistant_message(self) -> tuple[list[AssistantToolCall], dict[str, str]]:
+        """Analyze the last assistant message and return pending tool calls and tool call map."""
+        if not self.messages or not isinstance(self.messages[-1], NewAssistantMessage):
+            return [], {}
 
-        # Collect tool calls and results from the last assistant message
         tool_calls = {}
         tool_results = set()
+        tool_call_names = {}
 
-        for content_item in last_message.content:
+        for content_item in self.messages[-1].content:
             if content_item.type == "tool_call":
                 tool_calls[content_item.call_id] = content_item
+                tool_call_names[content_item.call_id] = content_item.name
             elif content_item.type == "tool_call_result":
                 tool_results.add(content_item.call_id)
 
-        # Return tool calls that don't have corresponding results
-        return [call for call_id, call in tool_calls.items() if call_id not in tool_results]
+        # Return pending tool calls and tool call names map
+        pending_calls = [call for call_id, call in tool_calls.items() if call_id not in tool_results]
+        return pending_calls, tool_call_names
+
+    def _find_pending_tool_calls(self) -> list[AssistantToolCall]:
+        """Find tool calls that don't have corresponding results yet."""
+        pending_calls, _ = self._analyze_last_assistant_message()
+        return pending_calls
 
     def _get_tool_call_name_by_id(self, call_id: str) -> str | None:
         """Get the tool name for a given call_id from the last assistant message."""
-        if not self.messages or not isinstance(self.messages[-1], NewAssistantMessage):
-            return None
-
-        for content_item in self.messages[-1].content:
-            if content_item.type == "tool_call" and content_item.call_id == call_id:
-                return content_item.name
-        return None
+        _, tool_call_names = self._analyze_last_assistant_message()
+        return tool_call_names.get(call_id)
 
     def _convert_tool_calls_to_tool_calls(self, tool_calls: list[AssistantToolCall]) -> list[ToolCall]:
         """Convert AssistantToolCall objects to ToolCall objects for compatibility."""
-
-        result_tool_calls = []
-        for tc in tool_calls:
-            tool_call = ToolCall(
+        return [
+            ToolCall(
                 id=tc.call_id,
                 type="function",
                 function=ToolCallFunction(
                     name=tc.name,
                     arguments=tc.arguments if isinstance(tc.arguments, str) else str(tc.arguments),
                 ),
-                index=len(result_tool_calls),
+                index=i,
             )
-            result_tool_calls.append(tool_call)
-        return result_tool_calls
+            for i, tc in enumerate(tool_calls)
+        ]
 
     def set_chat_history(self, messages: Sequence[FlexibleRunnerMessage], root_agent: Agent | None = None) -> None:
         """Set the entire chat history and track the current agent based on function calls.
@@ -691,12 +693,11 @@ class Runner:
             msg = f"Unsupported message type: {type(message)}"
             raise TypeError(msg)
 
-    async def _handle_agent_transfer(self, tool_call: ToolCall, _includes: Sequence[AgentChunkType]) -> None:
+    async def _handle_agent_transfer(self, tool_call: ToolCall) -> None:
         """Handle agent transfer when transfer_to_agent tool is called.
 
         Args:
             tool_call: The transfer_to_agent tool call
-            _includes: The types of chunks to include in output (unused)
         """
 
         # Parse the arguments to get the target agent name
@@ -771,12 +772,11 @@ class Runner:
                 output=f"Transfer failed: {e!s}",
             )
 
-    async def _handle_parent_transfer(self, tool_call: ToolCall, _includes: Sequence[AgentChunkType]) -> None:
+    async def _handle_parent_transfer(self, tool_call: ToolCall) -> None:
         """Handle parent transfer when transfer_to_parent tool is called.
 
         Args:
             tool_call: The transfer_to_parent tool call
-            _includes: The types of chunks to include in output (unused)
         """
 
         # Check if current agent has a parent

{lite_agent-0.4.0 → lite_agent-0.4.1}/src/lite_agent/stream_handlers/litellm.py

@@ -16,18 +16,27 @@ if TYPE_CHECKING:
     from aiofiles.threadpool.text import AsyncTextIOWrapper
 
 
-def ensure_record_file(record_to: Path | None) -> Path | None:
+def ensure_record_file(record_to: Path | str | None) -> Path | None:
     if not record_to:
         return None
-    if not record_to.parent.exists():
-        logger.warning('Record directory "%s" does not exist, creating it.', record_to.parent)
-        record_to.parent.mkdir(parents=True, exist_ok=True)
-    return record_to
+
+    path = Path(record_to) if isinstance(record_to, str) else record_to
+
+    # If the path is a directory, generate a filename
+    if path.is_dir():
+        path = path / "conversation.jsonl"
+
+    # Ensure parent directory exists
+    if not path.parent.exists():
+        logger.warning('Record directory "%s" does not exist, creating it.', path.parent)
+        path.parent.mkdir(parents=True, exist_ok=True)
+
+    return path
 
 
 async def litellm_completion_stream_handler(
     resp: litellm.CustomStreamWrapper,
-    record_to: Path | None = None,
+    record_to: Path | str | None = None,
 ) -> AsyncGenerator[AgentChunk, None]:
     """
     Optimized chunk handler
@@ -52,7 +61,7 @@ async def litellm_completion_stream_handler(
 
 async def litellm_response_stream_handler(
     resp: AsyncGenerator[ResponsesAPIStreamingResponse, None],
-    record_to: Path | None = None,
+    record_to: Path | str | None = None,
 ) -> AsyncGenerator[AgentChunk, None]:
     """
     Response API stream handler for processing ResponsesAPIStreamingResponse chunks

lite_agent-0.4.1/tests/unit/test_agent_additional.py

@@ -0,0 +1,182 @@
+"""
+为agent.py未覆盖部分添加的额外测试
+"""
+
+from lite_agent.agent import Agent
+from lite_agent.types import (
+    AgentAssistantMessage,
+    AgentUserMessage,
+    FlexibleRunnerMessage,
+    NewSystemMessage,
+    NewUserMessage,
+    RunnerMessage,
+    UserTextContent,
+)
+
+
+class TestAgentAdditional:
+    """Agent类的额外测试"""
+
+    def test_agent_init_with_handoffs(self):
+        """测试带交接代理的初始化"""
+        child_agent = Agent(model="gpt-3", name="Child", instructions="Child instructions")
+        parent_agent = Agent(
+            model="gpt-3",
+            name="Parent",
+            instructions="Parent instructions",
+            handoffs=[child_agent],
+        )
+        assert child_agent.parent is parent_agent
+        assert parent_agent.handoffs == [child_agent]
+
+    def test_agent_init_with_completion_condition(self):
+        """测试带完成条件的初始化"""
+        agent = Agent(
+            model="gpt-3",
+            name="TestBot",
+            instructions="Be helpful.",
+            completion_condition="call",
+        )
+        assert agent.completion_condition == "call"
+
+    def test_prepare_completion_messages_with_complex_messages(self):
+        """测试处理复杂消息格式"""
+        agent = Agent(model="gpt-3", name="TestBot", instructions="Be helpful.")
+
+        # 包含函数调用的消息
+        messages: list[FlexibleRunnerMessage] = [
+            AgentUserMessage(content=[UserTextContent(text="Hello")]),
+            AgentAssistantMessage(content=[]),
+            {
+                "type": "function_call",
+                "call_id": "call_123",
+                "name": "test_function",
+                "arguments": '{"param": "value"}',
+            },
+            {
+                "type": "function_call_output",
+                "call_id": "call_123",
+                "output": "Function result",
+            },
+        ]
+
+        result = agent.prepare_completion_messages(messages)
+
+        # 应该包含系统消息
+        assert result[0]["role"] == "system"
+
+        # 应该正确处理各种消息类型
+        assert len(result) >= 3
+
+    def test_prepare_completion_messages_with_new_message_format(self):
+        """测试新消息格式的处理"""
+        agent = Agent(model="gpt-3", name="TestBot", instructions="Be helpful.")
+
+        messages: list[RunnerMessage] = [
+            NewUserMessage(content=[UserTextContent(text="New user message")]),
+            NewSystemMessage(content="New system message"),
+        ]
+
+        result = agent.prepare_completion_messages(messages)
+
+        # 应该正确转换新格式消息
+        assert result[0]["role"] == "system"  # 代理的系统消息
+        assert len(result) >= 2
+
+    def test_agent_with_tools_registration(self):
+        """测试工具注册"""
+
+        def test_tool(param: str) -> str:
+            return f"Result: {param}"
+
+        agent = Agent(
+            model="gpt-3",
+            name="TestBot",
+            instructions="Be helpful.",
+            tools=[test_tool],
+        )
+
+        # 工具应该被注册
+        assert agent.fc is not None
+        tools = agent.fc.get_tools()
+        assert len(tools) > 0  # 至少包含注册的工具
+
+    def test_agent_handoff_tools(self):
+        """测试代理交接工具的注册"""
+        child_agent = Agent(model="gpt-3", name="Child", instructions="Child instructions")
+        parent_agent = Agent(
+            model="gpt-3",
+            name="Parent",
+            instructions="Parent instructions",
+            handoffs=[child_agent],
+        )
+
+        # 验证父子关系建立
+        assert child_agent.parent is parent_agent
+        assert parent_agent.handoffs == [child_agent]
+
+        # 验证工具注册系统工作
+        assert parent_agent.fc is not None
+        assert child_agent.fc is not None
+
+    def test_agent_wait_for_user_tool(self):
+        """测试等待用户工具的注册"""
+        agent = Agent(
+            model="gpt-3",
+            name="TestBot",
+            instructions="Be helpful.",
+            completion_condition="call",
+        )
+
+        # 验证completion_condition设置正确
+        assert agent.completion_condition == "call"
+
+        # 验证funcall系统正常工作
+        assert agent.fc is not None
+
+    def test_agent_client_property(self):
+        """测试代理客户端属性"""
+        agent = Agent(model="gpt-3", name="TestBot", instructions="Be helpful.")
+
+        # 应该有client属性
+        assert agent.client is not None
+        assert hasattr(agent.client, "model")
+
+    def test_message_conversion_edge_cases(self):
+        """测试消息转换的边界情况"""
+        agent = Agent(model="gpt-3", name="TestBot", instructions="Be helpful.")
+
+        # 测试空消息列表
+        result = agent.prepare_completion_messages([])
+        assert len(result) == 1  # 只有系统消息
+        assert result[0]["role"] == "system"
+
+        # 测试只有系统消息的情况
+        messages = [{"role": "system", "content": "Custom system message"}]
+        result = agent.prepare_completion_messages(messages)
+        assert len(result) >= 1
+
+    def test_agent_basic_properties(self):
+        """测试代理基本属性"""
+        agent = Agent(model="gpt-3", name="TestBot", instructions="Be helpful.")
+
+        assert agent.name == "TestBot"
+        assert agent.instructions == "Be helpful."
+        assert agent.completion_condition == "stop"  # 默认值
+        assert agent.handoffs == []
+        assert agent.parent is None
+
+    def test_agent_with_custom_system_message(self):
+        """测试自定义系统消息的处理"""
+        agent = Agent(model="gpt-3", name="TestBot", instructions="Be helpful.")
+
+        messages = [
+            {"role": "system", "content": "Custom system instruction"},
+            AgentUserMessage(content=[UserTextContent(text="Hello")]),
+        ]
+
+        result = agent.prepare_completion_messages(messages)
+
+        # 应该保留自定义系统消息
+        system_messages = [msg for msg in result if msg["role"] == "system"]
+        assert len(system_messages) >= 1