Skip to content

json-rpc: Add the support for json-rpc response#43516

Merged
botengyao merged 4 commits intoenvoyproxy:mainfrom
tyxia:jsonrpc
Feb 18, 2026
Merged

json-rpc: Add the support for json-rpc response#43516
botengyao merged 4 commits intoenvoyproxy:mainfrom
tyxia:jsonrpc

Conversation

@tyxia
Copy link
Member

@tyxia tyxia commented Feb 17, 2026

jso-rpc response is important for a2a, for example Task/Message Ops response

Signed-off-by: tyxia <tyxia@google.com>
@repokitteh-read-only
Copy link

As a reminder, PRs marked as draft will not be automatically assigned reviewers,
or be handled by maintainer-oncall triage.

Please mark your PR as ready when you want it to be reviewed!

🐱

Caused by: #43516 was opened by tyxia.

see: more, trace.

Signed-off-by: tyxia <tyxia@google.com>
@tyxia tyxia changed the title json-rpc: Implement the support for json-rpc response json-rpc: Add the support for json-rpc response Feb 17, 2026
Signed-off-by: tyxia <tyxia@google.com>
@tyxia tyxia marked this pull request as ready for review February 17, 2026 16:25
@tyxia
Copy link
Member Author

tyxia commented Feb 17, 2026

/retest

Copy link
Member

@botengyao botengyao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

re the description I think the agent-card can be handled by a full buffered JSON parse, since it is a well-known json .well-known/agent-card.json which is not JSON-RPC.

But the code lgtm, thanks for iterating this!

@botengyao botengyao merged commit a898602 into envoyproxy:main Feb 18, 2026
28 checks passed
@tyxia
Copy link
Member Author

tyxia commented Feb 18, 2026

Yeah that is correct. json-rpc is used for Task/Message Ops, and plain-json is used for agent card.

Thanks!

@tyxia tyxia deleted the jsonrpc branch February 18, 2026 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants