a2aproject / a2aproject/a2a-python

[Bug]: Response Stream Array-Wrapping Crash (Transport Layer Mismatch)

Aberta
#1,089 6 comentários 0 reações 1 responsável Reivindicada por @ishymko Ver no GitHub
component: client
Linguagem predominante
Python
Estrelas
2.1k
Forks
496
Merge médio
4d 17h
PRs com merge (30d)
12

Descrição

### What happened?

This is continuation from #1074 as splitting into two issues are suggested by @ishymko

Copied the issue description below for context

##### Response Stream Array-Wrapping Crash (Transport Layer Mismatch)
By default, Google wraps streaming RPC responses (`Flow`) inside a JSON array `[ {"task": ...} ]` to frame stream chunk boundaries on the wire.
* **The Mismatch**: The SDK's legacy compatibility transport (`CompatRestTransport` inside `a2a/compat/v0_3/rest_transport.py`) fails to request standard SSE transcoding from Google because it does not append the standard **`?alt=sse`** query parameter to the stream endpoint URL. This forces Google to fall back to default plain JSON array-wrapped streams.
* **The Crash**: Standard transports in `a2a/client/transports/rest.py` parse stream events using standard `Parse` (from `google.protobuf.json_format`):
```python
event: StreamResponse = Parse(sse_data, StreamResponse())
```
Because standard `Parse` strictly expects a single JSON object `{...}`, the leading and trailing array brackets (`[` and `]`) cause `json_format.Parse` to crash immediately on the very first chunk:
```python
google.protobuf.json_format.ParseError: Failed to parse {'task': {...}} field: unhashable type: 'dict'.
```

----

Thanks for pointing this issue is not specific to v0.3 but v1 as well. Regarding the workaround mentioned in https://github.com/a2aproject/a2a-python/issues/1074#issuecomment-4555290758 , yes it will work with the workaround but this mean all the customer/developer who integrated with Google A2A APIs will require this workaround introduces developer friction. Is there any way we could make this smoother so developer can use SDK with Google A2A API out of the box? Thanks!

### Code of Conduct

- [x] I agree to follow this project's Code of Conduct

Guia de contribuição

Abrir o guia de contribuição

Avaliação

Esta issue ainda não foi avaliada.

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.