Missing Feature: ACP SessionUpdate notifications not emitted
Found by: test_acp_session_updates.py (2 xfail)
Missing notifications:
user_message_chunk — AgentPool ACP server does not emit user_message_chunk SessionUpdate notification when user sends a message
tool_call_start / tool_call_end — AgentPool does not emit tool_call_start/end SessionUpdate notifications when a tool is invoked
Location: src/agentpool_server/acp_server/event_converter.py — SessionUpdate conversion logic
Impact: ACP clients (e.g., Zed) cannot see user message echo or tool call progress via SessionUpdate notifications.
Tests:
tests/e2e/test_acp_session_updates.py (user_message_chunk)
tests/e2e/test_acp_session_updates.py::test_tool_call_start_end_notifications
Severity: Low-Medium — these are optional ACP v1 notifications; 9 other SessionUpdate variants ARE emitted correctly
Missing Feature: ACP SessionUpdate notifications not emitted
Found by:
test_acp_session_updates.py(2 xfail)Missing notifications:
user_message_chunk— AgentPool ACP server does not emituser_message_chunkSessionUpdate notification when user sends a messagetool_call_start/tool_call_end— AgentPool does not emit tool_call_start/end SessionUpdate notifications when a tool is invokedLocation:
src/agentpool_server/acp_server/event_converter.py— SessionUpdate conversion logicImpact: ACP clients (e.g., Zed) cannot see user message echo or tool call progress via SessionUpdate notifications.
Tests:
tests/e2e/test_acp_session_updates.py(user_message_chunk)tests/e2e/test_acp_session_updates.py::test_tool_call_start_end_notificationsSeverity: Low-Medium — these are optional ACP v1 notifications; 9 other SessionUpdate variants ARE emitted correctly