refactor: better specificity in method

Author: damassi

src/agent_chat_cli/core/actions.py

@@ -17,7 +17,9 @@ def __init__(self, app: "AgentChatCLIApp") -> None:
     def quit(self) -> None:
         self.app.exit()
 
-    async def add_message_to_chat(self, type: MessageType, content: str) -> None:
+    async def add_message_to_chat_history(
+        self, type: MessageType, content: str
+    ) -> None:
         match type:
             case MessageType.USER:
                 message = Message.user(content)
@@ -39,7 +41,7 @@ async def submit_user_message(self, message: str) -> None:
         await self._query(message)
 
     async def post_system_message(self, message: str) -> None:
-        await self.add_message_to_chat(MessageType.SYSTEM, message)
+        await self.add_message_to_chat_history(MessageType.SYSTEM, message)
 
     async def render_message(self, message) -> None:
         await self.app.renderer.render_message(message)

src/agent_chat_cli/core/renderer.py

@@ -102,14 +102,18 @@ async def _render_system_message(self, message: AgentMessage) -> None:
             message.data if isinstance(message.data, str) else str(message.data)
         )
 
-        await self.app.actions.add_message_to_chat(MessageType.SYSTEM, system_content)
+        await self.app.actions.add_message_to_chat_history(
+            MessageType.SYSTEM, system_content
+        )
 
     async def _render_user_message(self, message: AgentMessage) -> None:
         user_content = (
             message.data if isinstance(message.data, str) else str(message.data)
         )
 
-        await self.app.actions.add_message_to_chat(MessageType.USER, user_content)
+        await self.app.actions.add_message_to_chat_history(
+            MessageType.USER, user_content
+        )
 
     async def _render_tool_permission_request(self, message: AgentMessage) -> None:
         log_json(

tests/core/test_actions.py

@@ -42,7 +42,7 @@ async def test_adds_user_message(self, mock_agent_loop, mock_config):
         async with app.run_test():
             chat_history = app.query_one(ChatHistory)
 
-            await app.actions.add_message_to_chat(MessageType.USER, "Hello")
+            await app.actions.add_message_to_chat_history(MessageType.USER, "Hello")
 
             widgets = chat_history.query(UserMessage)
             assert len(widgets) == 1
@@ -53,7 +53,9 @@ async def test_adds_system_message(self, mock_agent_loop, mock_config):
         async with app.run_test():
             chat_history = app.query_one(ChatHistory)
 
-            await app.actions.add_message_to_chat(MessageType.SYSTEM, "System alert")
+            await app.actions.add_message_to_chat_history(
+                MessageType.SYSTEM, "System alert"
+            )
 
             widgets = chat_history.query(SystemMessage)
             assert len(widgets) == 1
@@ -64,7 +66,9 @@ async def test_adds_agent_message(self, mock_agent_loop, mock_config):
         async with app.run_test():
             chat_history = app.query_one(ChatHistory)
 
-            await app.actions.add_message_to_chat(MessageType.AGENT, "I can help")
+            await app.actions.add_message_to_chat_history(
+                MessageType.AGENT, "I can help"
+            )
 
             widgets = chat_history.query(AgentMessage)
             assert len(widgets) == 1
@@ -74,7 +78,9 @@ async def test_raises_for_unsupported_type(self, mock_agent_loop, mock_config):
         app = AgentChatCLIApp()
         async with app.run_test():
             with pytest.raises(ValueError, match="Unsupported message type"):
-                await app.actions.add_message_to_chat(MessageType.TOOL, "tool content")
+                await app.actions.add_message_to_chat_history(
+                    MessageType.TOOL, "tool content"
+                )
 
 
 class TestActionsPostSystemMessage: