refactor: update post_app_event naming

Author: damassi

CLAUDE.md

@@ -4,6 +4,7 @@
 
 - Read `docs/architecture.md` for an architectural overview of the project. Refactors should always start here first.
 - The project uses `uv`, `ruff` and `ty`
+- Typechecking is done via `uv run ty check`
 - Run commands should be prefixed with `uv`: `uv run ...`
 - Use `asyncio` features, if such is needed
 - Prefer early returns

Makefile

@@ -19,4 +19,4 @@ test:
 	uv run pytest
 
 type-check:
-	uv run ty check src
+	uv run ty check

src/agent_chat_cli/core/actions.py

@@ -23,7 +23,7 @@ async def post_user_message(self, message: str) -> None:
     async def post_system_message(self, message: str) -> None:
         await self.app.renderer.add_message(RoleType.SYSTEM, message)
 
-    async def handle_app_event(self, event) -> None:
+    async def post_app_event(self, event) -> None:
         await self.app.renderer.handle_app_event(event)
 
     async def interrupt(self) -> None:

src/agent_chat_cli/core/agent_loop.py

@@ -93,7 +93,7 @@ async def start(self) -> None:
 
                 await self._handle_message(message)
 
-            await self.app.actions.handle_app_event(
+            await self.app.actions.post_app_event(
                 AppEvent(type=AppEventType.RESULT, data=None)
             )
 
@@ -136,7 +136,7 @@ async def _handle_message(self, message: Message) -> None:
                     text_chunk = delta.get("text", "")
 
                     if text_chunk:
-                        await self.app.actions.handle_app_event(
+                        await self.app.actions.post_app_event(
                             AppEvent(
                                 type=AppEventType.STREAM_EVENT,
                                 data={"text": text_chunk},
@@ -164,7 +164,7 @@ async def _handle_message(self, message: Message) -> None:
                         )
 
             # Finally, post the agent assistant response
-            await self.app.actions.handle_app_event(
+            await self.app.actions.post_app_event(
                 AppEvent(
                     type=AppEventType.ASSISTANT,
                     data={"content": content},
@@ -181,7 +181,7 @@ async def _can_use_tool(
 
         # Handle permission request queue sequentially
         async with self.permission_lock:
-            await self.app.actions.handle_app_event(
+            await self.app.actions.post_app_event(
                 AppEvent(
                     type=AppEventType.TOOL_PERMISSION_REQUEST,
                     data={

tests/core/test_agent_loop.py

@@ -21,7 +21,7 @@ def mock_app():
     app = MagicMock()
     app.ui_state = MagicMock()
     app.actions = MagicMock()
-    app.actions.handle_app_event = AsyncMock()
+    app.actions.post_app_event = AsyncMock()
     app.actions.post_system_message = AsyncMock()
     return app
 
@@ -156,8 +156,8 @@ async def test_handles_text_delta_stream_event(self, mock_app, mock_config):
 
         await agent_loop._handle_message(message)
 
-        mock_app.actions.handle_app_event.assert_called_once()
-        call_arg = mock_app.actions.handle_app_event.call_args[0][0]
+        mock_app.actions.post_app_event.assert_called_once()
+        call_arg = mock_app.actions.post_app_event.call_args[0][0]
         assert call_arg.type == AppEventType.STREAM_EVENT
         assert call_arg.data == {"text": "Hello world"}
 
@@ -178,7 +178,7 @@ async def test_ignores_empty_text_delta(self, mock_app, mock_config):
 
         await agent_loop._handle_message(message)
 
-        mock_app.actions.handle_app_event.assert_not_called()
+        mock_app.actions.post_app_event.assert_not_called()
 
 
 class TestHandleMessageAssistantMessage:
@@ -193,8 +193,8 @@ async def test_handles_text_block(self, mock_app, mock_config):
 
         await agent_loop._handle_message(message)
 
-        mock_app.actions.handle_app_event.assert_called_once()
-        call_arg = mock_app.actions.handle_app_event.call_args[0][0]
+        mock_app.actions.post_app_event.assert_called_once()
+        call_arg = mock_app.actions.post_app_event.call_args[0][0]
         assert call_arg.type == AppEventType.ASSISTANT
         assert call_arg.data["content"][0]["type"] == ContentType.TEXT.value
         assert call_arg.data["content"][0]["text"] == "Assistant response"
@@ -212,8 +212,8 @@ async def test_handles_tool_use_block(self, mock_app, mock_config):
 
         await agent_loop._handle_message(message)
 
-        mock_app.actions.handle_app_event.assert_called_once()
-        call_arg = mock_app.actions.handle_app_event.call_args[0][0]
+        mock_app.actions.post_app_event.assert_called_once()
+        call_arg = mock_app.actions.post_app_event.call_args[0][0]
         assert call_arg.type == AppEventType.ASSISTANT
         assert call_arg.data["content"][0]["type"] == ContentType.TOOL_USE.value
         assert call_arg.data["content"][0]["name"] == "read_file"
@@ -287,7 +287,7 @@ async def test_posts_permission_request_message(self, mock_app, mock_config):
             _context=MagicMock(),
         )
 
-        mock_app.actions.handle_app_event.assert_called_once()
-        call_arg = mock_app.actions.handle_app_event.call_args[0][0]
+        mock_app.actions.post_app_event.assert_called_once()
+        call_arg = mock_app.actions.post_app_event.call_args[0][0]
         assert call_arg.type == AppEventType.TOOL_PERMISSION_REQUEST
         assert call_arg.data["tool_name"] == "write_file"