refactor: cleanup more naming

Author: damassi

docs/architecture.md

@@ -75,7 +75,7 @@ Routes messages from the AgentLoop to appropriate UI components:
 
 **Actions** (`core/actions.py`)
 User-initiated action handlers:
-- `submit_user_message()`: Posts user message and queries agent
+- `post_user_message()`: Posts user message and queries agent
 - `interrupt()`: Cancels current agent operation
 - `new()`: Starts new conversation, clears history
 - `respond_to_tool_permission()`: Handles permission prompt responses
@@ -90,7 +90,7 @@ Manages the Claude Agent SDK client lifecycle:
 ### Message Flow
 
 1. User types in `UserInput` and presses Enter
-2. `Actions.submit_user_message()` posts to UI and enqueues to `AgentLoop.query_queue`
+2. `Actions.post_user_message()` posts to UI and enqueues to `AgentLoop.query_queue`
 3. `AgentLoop` sends query to Claude Agent SDK and streams responses
 4. Responses flow through `Actions.render_message()` to update UI
 5. Tool use triggers permission prompt via `UIState.show_permission_prompt()`

src/agent_chat_cli/components/user_input.py

@@ -120,4 +120,4 @@ async def action_submit(self) -> None:
             return
 
         input_widget.clear()
-        await self.actions.submit_user_message(user_message)
+        await self.actions.post_user_message(user_message)

src/agent_chat_cli/core/actions.py

@@ -1,7 +1,6 @@
 from typing import TYPE_CHECKING
 
 from agent_chat_cli.utils.enums import ControlCommand
-from agent_chat_cli.components.chat_history import ChatHistory
 from agent_chat_cli.components.messages import RoleType
 from agent_chat_cli.components.tool_permission_prompt import ToolPermissionPrompt
 from agent_chat_cli.utils.logger import log_json
@@ -17,7 +16,7 @@ def __init__(self, app: "AgentChatCLIApp") -> None:
     def quit(self) -> None:
         self.app.exit()
 
-    async def submit_user_message(self, message: str) -> None:
+    async def post_user_message(self, message: str) -> None:
         await self.app.renderer.add_message(RoleType.USER, message)
         await self._query(message)
 
@@ -32,14 +31,12 @@ async def interrupt(self) -> None:
         if permission_prompt.is_visible:
             return
 
+        self.app.ui_state.stop_thinking()
         self.app.ui_state.set_interrupting(True)
         await self.app.agent_loop.client.interrupt()
-        self.app.ui_state.stop_thinking()
 
     async def clear(self) -> None:
-        chat_history = self.app.query_one(ChatHistory)
-        await chat_history.remove_children()
-
+        await self.app.renderer.reset_chat_history()
         self.app.ui_state.stop_thinking()
 
     async def new(self) -> None:
@@ -64,7 +61,7 @@ async def respond_to_tool_permission(self, response: str) -> None:
             if normalized in ["n", "no", "deny"]:
                 await self._query("The user has denied the tool")
             else:
-                await self.submit_user_message(response)
+                await self.post_user_message(response)
 
     async def _query(self, user_input: str) -> None:
         await self.app.agent_loop.query_queue.put(user_input)

src/agent_chat_cli/core/renderer.py

@@ -73,6 +73,10 @@ async def add_message(self, type: RoleType, content: str) -> None:
         self.app.ui_state.start_thinking()
         await self.app.ui_state.scroll_to_bottom()
 
+    async def reset_chat_history(self) -> None:
+        chat_history = self.app.query_one(ChatHistory)
+        await chat_history.remove_children()
+
     async def _render_stream_event(self, event: AppEvent) -> None:
         text_chunk = event.data.get("text", "")
 

tests/components/test_user_input.py

@@ -16,7 +16,7 @@ def __init__(self):
         self.mock_actions.interrupt = AsyncMock()
         self.mock_actions.new = AsyncMock()
         self.mock_actions.clear = AsyncMock()
-        self.mock_actions.submit_user_message = AsyncMock()
+        self.mock_actions.post_user_message = AsyncMock()
 
     def compose(self) -> ComposeResult:
         yield UserInput(actions=self.mock_actions)
@@ -31,7 +31,7 @@ async def test_empty_submit_does_nothing(self, app):
         async with app.run_test() as pilot:
             await pilot.press("enter")
 
-            app.mock_actions.submit_user_message.assert_not_called()
+            app.mock_actions.post_user_message.assert_not_called()
 
     async def test_submits_message(self, app):
         async with app.run_test() as pilot:
@@ -41,7 +41,7 @@ async def test_submits_message(self, app):
 
             await pilot.press("enter")
 
-            app.mock_actions.submit_user_message.assert_called_with("Hello agent")
+            app.mock_actions.post_user_message.assert_called_with("Hello agent")
 
     async def test_clears_input_after_submit(self, app):
         async with app.run_test() as pilot:

tests/core/test_actions.py

@@ -96,7 +96,7 @@ async def test_adds_user_message_to_chat(self, mock_agent_loop, mock_config):
         async with app.run_test():
             chat_history = app.query_one(ChatHistory)
 
-            await app.actions.submit_user_message("Hello agent")
+            await app.actions.post_user_message("Hello agent")
 
             widgets = chat_history.query(UserMessage)
             assert len(widgets) == 1
@@ -107,15 +107,15 @@ async def test_starts_thinking_indicator(self, mock_agent_loop, mock_config):
 
         app = AgentChatCLIApp()
         async with app.run_test():
-            await app.actions.submit_user_message("Hello agent")
+            await app.actions.post_user_message("Hello agent")
 
             thinking_indicator = app.query_one(ThinkingIndicator)
             assert thinking_indicator.is_thinking is True
 
     async def test_queues_message_to_agent_loop(self, mock_agent_loop, mock_config):
         app = AgentChatCLIApp()
         async with app.run_test():
-            await app.actions.submit_user_message("Hello agent")
+            await app.actions.post_user_message("Hello agent")
 
             mock_agent_loop.query_queue.put.assert_called_with("Hello agent")
 

tests/core/test_agent_loop.py

@@ -272,7 +272,9 @@ async def test_denies_tool_on_custom_response(self, mock_app, mock_config):
         )
 
         assert result.behavior == "deny"
-        assert result.message == "do something else instead"
+        assert (
+            hasattr(result, "message") and result.message == "do something else instead"
+        )
 
     async def test_posts_permission_request_message(self, mock_app, mock_config):
         agent_loop = AgentLoop(app=mock_app)