a2aproject / a2aproject/a2a-js

[Bug]: JSON-RPC SSE error events lose their typed A2A error

オープン
#712 コメント 2 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
TypeScript
スター
613
フォーク
169
平均マージ
1日 6時間
マージ済み PR(30日)
21

説明

## What happened

The JSON-RPC client maps an error response from a unary request to a typed A2A error, but wraps the same error envelope from an SSE stream in a generic `Error`.

For example, JSON-RPC error code `-32001` is exposed as `TaskNotFoundError` by `SendMessage`. When it arrives as an SSE error event from `SendStreamingMessage`, the caught error instead has `name === "Error"` and fails `instanceof JsonRpcTaskNotFoundError`; the typed error is only available through `error.cause`.

## What I expected

The same JSON-RPC error envelope should produce the same typed A2A error regardless of whether it arrives through a unary response or an SSE error event. Callers should be able to use the SDK's error classes consistently for branching, retries, and recovery.

## Steps to reproduce

1. Create a `JsonRpcTransport` with a fetch implementation that returns HTTP 200 and `Content-Type: text/event-stream`.
2. Return an SSE error event whose data is a matching JSON-RPC response with error code `-32001` and message `"Task not found"`.
3. Iterate `sendMessageStream()` and catch the thrown error.
4. Observe that the error is a generic `Error`; its `cause` is the typed `TaskNotFoundError`.
5. Return the same JSON-RPC error envelope from `sendMessage()` and observe that the typed error is thrown directly.

## Additional context

Minimal comparison:

```text
unary: name=TaskNotFoundError, typed=true
streaming: name=Error, typed=false, cause=TaskNotFoundError
```

This makes streaming and non-streaming calls expose different public error contracts for the same protocol failure. Mapping and throwing the SSE error envelope directly would preserve the semantic and JSON-RPC-specific error information.

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

Look at the JSON-RPC client code, likely in a transport or client module, to see how unary errors are mapped to typed A2A errors (like TaskNotFoundError). Then find the SSE stream handling, likely in a streaming transport or event source adapter, to see where error events are parsed and thrown. Compare the two error-wrapping paths. The fix is to ensure the SSE error envelope is mapped to the same typed error class before being thrown. Test by running the provided reproduction steps, checking the error's name and instanceof behavior.

索引モデルが issue の本文から書いたものです。

評価

技術スタック
javascript, typescript
領域
api, backend-api-design
issue の種類
バグ
難易度
3/5
見積もり時間
1〜2日
活発さ
活発
明瞭さ
明確に書かれている
初心者へのやさしさ
65/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。