[Bug]: Framework-written FAILED status drops status.timestamp, persisting last_updated as NULL
Mantenedores costumam responder em até 2 dias
Avaliação
Esta issue ainda não foi avaliada.
Descrição
What happened?
When the framework itself writes a terminal FAILED state (consumer failure, producer failure), the resulting TaskStatus carries no timestamp. Because TaskManager.save_task_event replaces the status wholesale via task.status.CopyFrom(event.status) (src/a2a/server/tasks/task_manager.py:265), the previously stamped status.timestamp is cleared, and DatabaseTaskStore.save derives last_updated from status.timestamp (src/a2a/server/tasks/database_task_store.py:152-156) — persisting NULL.
Consequences:
ListTasksordering (which sorts onlast_updated) drops the task to the end;- the
status_timestamp_afterfilter never matches the task again; - API responses lose the
status.timestampfield entirely.
The behavior is also internally inconsistent: the cancel-failure path _mark_task_as_failed (src/a2a/server/agent_execution/active_task.py:945-963) copies the existing status and only overwrites state (preserving the timestamp), while the consumer failure path _write_failed_status (active_task.py:161-169) and the producer failure path construct a fresh TaskStatus(state=...) with no timestamp.
Reproduction
Self-contained script (main @ 494a8ec, Python 3.10)
import asyncio
from a2a.auth.user import UnauthenticatedUser
from a2a.helpers.proto_helpers import new_task_from_user_message
from a2a.server.agent_execution import AgentExecutor
from a2a.server.context import ServerCallContext
from a2a.server.events import EventQueue
from a2a.server.request_handlers import DefaultRequestHandlerV2
from a2a.server.tasks import InMemoryTaskStore
from a2a.types.a2a_pb2 import (
AgentCapabilities,
AgentCard,
Message,
Part,
Role,
SendMessageConfiguration,
SendMessageRequest,
TaskState,
)
class ExplodingExecutor(AgentExecutor):
task_id = None
async def execute(self, context, event_queue):
task = new_task_from_user_message(context.message)
self.task_id = task.id
await event_queue.enqueue_event(task)
raise RuntimeError('agent crashed mid-flight')
async def cancel(self, context, event_queue):
pass
async def main():
task_store = InMemoryTaskStore()
executor = ExplodingExecutor()
handler = DefaultRequestHandlerV2(
agent_executor=executor,
task_store=task_store,
agent_card=AgentCard(
name='t',
version='1.0',
capabilities=AgentCapabilities(streaming=True),
),
)
params = SendMessageRequest(
message=Message(
role=Role.ROLE_USER, message_id='m1', parts=[Part(text='Hi')]
),
configuration=SendMessageConfiguration(
accepted_output_modes=['text/plain']
),
)
try:
await handler.on_message_send(
params, ServerCallContext(user=UnauthenticatedUser())
)
except Exception as e:
print(f'on_message_send raised {type(e).__name__} (expected)')
task = await task_store.get(
executor.task_id, ServerCallContext(user=UnauthenticatedUser())
)
print(
f'persisted: state={TaskState.Name(task.status.state)}, '
f'timestamp set={task.status.HasField("timestamp")}'
)
asyncio.run(main())
Observed output:
on_message_send raised RuntimeError (expected)
persisted: state=TASK_STATE_FAILED, timestamp set=False
Expected behavior
Every task state transition the framework writes — including failures — should carry a status.timestamp, matching what TaskUpdater.update_status does for agent-driven transitions.
Suggested fix
Either:
- stamp the current time in the three framework failure paths (
_write_failed_status, the producer failure path, and — for consistency —_mark_task_as_failed), or - in
TaskManager.save_task_event, preserve the previousstatus.timestampwhen the incoming status event has none.
Happy to submit a PR with tests.
Related
Observed as a secondary effect in the reproduction of #1313.
Code of Conduct
- I agree to follow this project's Code of Conduct
- Linguagem predominante
- Python
- Estrelas
- 2.2k
- Forks
- 528
- Merge médio
- 3d 10h
- PRs com merge (30d)
- 48
Preparar o ambiente
- Sem Dockerfile nem arquivo Docker Compose
- Tem um modelo de pull request
- Ler o guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de a2aproject/a2a-python
-
[Bug]: `A2A-Version` validation ignores minor versions and the query parameterTalvez já em andamento @rohityan assumiu há 63 dias. Abertacomponent: server status:awaiting response
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 75/100
a2aproject/a2a-python#1204 · 3 comentários · 1 responsável ·
Mantenedores costumam responder em até 2 dias
-
[Bug]: REST task/request id sanitizationTalvez já em andamento @Linux2010 assumiu há 121 dias. Abertamaintainers-only
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 65/100
a2aproject/a2a-python#805 · 1 comentário ·
Mantenedores costumam responder em até 2 dias
-
v0.3 REST GetTask returns 500 for a non-integer historyLengthTalvez já em andamento @rohityan assumiu há 1 dia. Abertacomponent: server
a2aproject/a2a-python#1381 · 1 responsável ·
Mantenedores costumam responder em até 2 dias
-
v0.3 pushNotificationConfig/get by task id fails with Validation failedTalvez já em andamento @rohityan assumiu há 1 dia. Abertacomponent: server
a2aproject/a2a-python#1379 · 1 responsável ·
Mantenedores costumam responder em até 2 dias
-
[Bug]: REST tenant routes are mounted before `path_prefix`, so tenant requests from the SDK client get 404Talvez já em andamento @rohityan assumiu há 2 dias. Aberta
a2aproject/a2a-python#1376 · 2 responsáveis ·
Mantenedores costumam responder em até 2 dias
Todas as issues de a2aproject/a2a-python
Issues semelhantes
-
component:benchmark
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 76/100
agentic-os-org/ANOLISA#9177 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 88/100
distrochooser/distrochooser#520 ·
Mantenedores costumam responder em até 2 dias
-
Status: Needs confirmation Status: Needs triage
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 64/100
Mantenedores costumam responder em até 1 dia
-
[BUG] Published Docker images lack the redis extra: HEADROOM_COMPRESS_SESSION_STORE stops startupTalvez já em andamento @AlessandroSpallina assumiu hoje. Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 76/100
headroomlabs-ai/headroom#4513 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 65/100
codespell-project/codespell#4060 ·
Mantenedores costumam responder em até 1 dia