[Bug]: Streaming follow-up on an existing task does not begin with a Task; enqueuing the current task drops the follow-up message from history
维护者通常 2 天内回复
评估
这个 Issue 还没有评估数据。
描述
What happened?
When a client continues an existing task with SendStreamingMessage (for example, after TASK_STATE_INPUT_REQUIRED), the stream does not begin with a Task object. The spec requires one, in §3.1.2 Send Streaming Message, Behavior 2:
If the agent returns a Task, the stream MUST begin with the Task object, followed by zero or more TaskStatusUpdateEvent or TaskArtifactUpdateEvent objects.
The executor cannot fix this from its side. Enqueueing the current task first, as the v1.0 migration guide shows (task = context.current_task or new_task_from_user_message(...) followed by enqueue_event(task)), makes the stream start with a Task. But the follow-up user message is then never written to the task history, and an ERROR is logged.
Reproduced on a2a-sdk 1.1.5 and 1.2.1. I found no change to the affected code on main since 1.2.1.
Expected: a streamed follow-up on an existing task begins with a Task (its current state), and the follow-up user message is added to Task.history.
Actual (reproduction below):
| Executor | 2nd stream | history after 2nd turn |
|---|---|---|
enqueues Task only for a new task (as in tests/integration/test_end_to_end.py) |
status_update, status_update, no leading Task |
user 1, agent, user 2 |
always enqueues context.current_task first (migration guide) |
task, status_update, status_update |
user 1, agent. User 2 is missing, and ERROR … already exists. Ignoring task replacement. is logged |
Where it comes from (v1.2.1)
default_request_handler_v2.py#L466:on_message_send_streamsubscribes withinclude_initial_task=False, so the handler never emits the stored task. The leadingTaskhas to come from the executor.active_task.py#L274-L283: when the executor enqueues aTaskand the task already exists,_handle_initial_tasklogs the ERROR, ignores the event, and still setsself.message_to_save = None. The follow-up message was stored inmessage_to_savewhen the request started, so_handle_task_modification_eventno longer adds it to history.active_task.py#L291-L299: theInvalidAgentResponseErrorcheck added in #979 ("Agent should enqueue Task before …") only fires when no task exists yet, so a continuation without a leadingTaskpasses silently.test_end_to_end.py#L589-L596:test_end_to_end_input_requiredcurrently expects the follow-up stream to bestatus_update, artifact_update, status_update.
Related
- #965 was fixed for new tasks by #979.
- The earlier attempt #964 injected
Task(SUBMITTED)in the handler, but only whennot params.message.task_id, so continuations were out of scope there too. - #1188 concerns resuming from a stale snapshot after
input_required. It's related but different: that report is about multi-replica state, and this one reproduces on a single in-memory replica.
Possible directions (maintainers will know better)
- Handler: for a message that carries
task_idof an existing task, emit the stored task as the first stream event, ason_subscribe_to_taskalready does withinclude_initial_task=True. Ideally that snapshot would already include the follow-up message. - Consumer: when the executor enqueues a
Taskwhose id matches the existing task, treat it as "emit current state": keep or applymessage_to_saveand don't log an ERROR. That would make the migration-guide pattern correct.
I'm happy to test a fix. Which direction do you prefer?
Reproduction
Self-contained. Needs a2a-sdk[http-server] (and httpx). Run python repro.py for row 1 and python repro.py --always for row 2.
import asyncio
import logging
import sys
import httpx
from a2a.client import ClientConfig, ClientFactory
from a2a.helpers.proto_helpers import new_task_from_user_message
from a2a.server.agent_execution import AgentExecutor, RequestContext
from a2a.server.events import EventQueue
from a2a.server.request_handlers import DefaultRequestHandler
from a2a.server.routes import create_agent_card_routes, create_jsonrpc_routes
from a2a.server.tasks import TaskUpdater
from a2a.server.tasks.inmemory_task_store import InMemoryTaskStore
from a2a.types import (
AgentCapabilities,
AgentCard,
AgentInterface,
GetTaskRequest,
Message,
Part,
Role,
SendMessageRequest,
TaskState,
)
from a2a.utils import TransportProtocol
from starlette.applications import Starlette
ALWAYS_ENQUEUE_TASK = '--always' in sys.argv
class Executor(AgentExecutor):
async def execute(self, context: RequestContext, event_queue: EventQueue) -> None:
task = context.current_task or new_task_from_user_message(context.message)
if ALWAYS_ENQUEUE_TASK or context.current_task is None:
await event_queue.enqueue_event(task)
updater = TaskUpdater(event_queue, task.id, task.context_id)
await updater.update_status(TaskState.TASK_STATE_WORKING)
if context.current_task is None:
await updater.update_status(
TaskState.TASK_STATE_INPUT_REQUIRED,
message=updater.new_agent_message([Part(text='Which one?')]),
)
else:
await updater.complete()
async def cancel(self, context: RequestContext, event_queue: EventQueue) -> None:
raise NotImplementedError
async def main() -> None:
card = AgentCard(
name='repro',
description='repro',
version='1.0.0',
capabilities=AgentCapabilities(streaming=True),
default_input_modes=['text/plain'],
default_output_modes=['text/plain'],
supported_interfaces=[
AgentInterface(protocol_binding=TransportProtocol.JSONRPC, url='http://testserver')
],
)
handler = DefaultRequestHandler(
agent_executor=Executor(), task_store=InMemoryTaskStore(), agent_card=card
)
app = Starlette(
routes=[
*create_agent_card_routes(agent_card=card, card_url='/'),
*create_jsonrpc_routes(request_handler=handler, rpc_url='/'),
]
)
http = httpx.AsyncClient(transport=httpx.ASGITransport(app=app), base_url='http://testserver')
client = ClientFactory(
ClientConfig(httpx_client=http, supported_protocol_bindings=[TransportProtocol.JSONRPC])
).create(card)
async def send(text: str, task_id: str | None = None) -> list:
message = Message(role=Role.ROLE_USER, message_id=f'msg-{text}', parts=[Part(text=text)])
if task_id:
message.task_id = task_id
return [e async for e in client.send_message(SendMessageRequest(message=message))]
first = await send('first')
task_id = first[0].task.id
second = await send('second', task_id)
task = await client.get_task(GetTaskRequest(id=task_id))
print('first stream: ', [e.WhichOneof('payload') for e in first])
print('second stream:', [e.WhichOneof('payload') for e in second])
print('final state: ', TaskState.Name(task.status.state))
print('history: ', [(Role.Name(m.role), m.message_id) for m in task.history])
if __name__ == '__main__':
logging.basicConfig(level=logging.ERROR, format='%(levelname)s %(name)s: %(message)s')
asyncio.run(main())
Environment: a2a-sdk 1.1.5 (PyPI) and 1.2.1 (tag v1.2.1), Python 3.14, macOS, JSON-RPC transport, InMemoryTaskStore.
Relevant log output
$ python repro.py
first stream: ['task', 'status_update', 'status_update']
second stream: ['status_update', 'status_update']
final state: TASK_STATE_COMPLETED
history: [('ROLE_USER', 'msg-first'), ('ROLE_AGENT', '<uuid>'), ('ROLE_USER', 'msg-second')]
$ python repro.py --always
ERROR a2a.server.agent_execution.active_task: Task <uuid> already exists. Ignoring task replacement.
first stream: ['task', 'status_update', 'status_update']
second stream: ['task', 'status_update', 'status_update']
final state: TASK_STATE_COMPLETED
history: [('ROLE_USER', 'msg-first'), ('ROLE_AGENT', '<uuid>')]
Code of Conduct
- I agree to follow this project's Code of Conduct
- 主要语言
- Python
- 星标
- 2.2k
- 派生
- 509
- 平均合并
- 2 天 22 小时
- 30 天内合并 PR
- 43
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
a2aproject/a2a-python 的其他 Issue
-
[Bug]: REST task/request id sanitization可能已有人在做 @Linux2010 于 107 天前认领。 未关闭maintainers-only
难度 2/5 1-3 小时 新手友好度 65/100
a2aproject/a2a-python#805 · 1 条评论 ·
维护者通常 2 天内回复
-
[Bug]: Agent card signature verification raises raw binascii.Error on a malformed protected header instead of SignatureVerificationError可能已有人在做 @Iwaniukooo11 今天认领。 未关闭
a2aproject/a2a-python#1332 · 已指派 2 人 ·
维护者通常 2 天内回复
-
[Bug]: Framework-written FAILED status drops status.timestamp, persisting last_updated as NULL可能已有人在做 @rohityan 于 3 天前认领。 未关闭
a2aproject/a2a-python#1331 · 已指派 2 人 ·
维护者通常 2 天内回复
-
[Bug]: message/send hangs forever when the AgentExecutor returns without producing events可能已有人在做 @Iwaniukooo11 今天认领。 未关闭
a2aproject/a2a-python#1330 · 已指派 2 人 ·
维护者通常 2 天内回复
-
Whole numbers in a data Part are returned as floats (e.g. 4326 becomes 4326.0)可能已有人在做 @rohityan 于 3 天前认领。 未关闭
a2aproject/a2a-python#1328 · 已指派 1 人 ·
维护者通常 2 天内回复
查看 a2aproject/a2a-python 的全部 Issue
相似的 Issue
-
bug
难度 1/5 1 小时以内 新手友好度 88/100
openwisp/openwisp-utils#777 ·
-
难度 2/5 1-3 小时 新手友好度 63/100
google-research/weatherbenchX#310 ·
维护者通常 3 天内回复
-
难度 2/5 1-3 小时 新手友好度 78/100
huggingface/peft#3955 ·
维护者通常 1 天内回复
-
author-page correction
难度 2/5 1-3 小时 新手友好度 60/100
acl-org/acl-anthology#10313 ·
维护者通常 1 天内回复
-
bug
难度 2/5 1-3 小时 新手友好度 76/100
AOSSIE-Org/Gitcord-GithubDiscordBot#147 · 1 条评论 ·
维护者通常 1 天内回复