Skip to content

fix(rest): support application/a2a+json content type - #1181

Merged
ehsavoie merged 2 commits into
mainfrom
jakubworek/a2a-content-type
Sep 25, 2026
Merged

ehsavoie merged 2 commits into
mainfrom
jakubworek/a2a-content-type

Conversation

@JakubWorek

Copy link
Copy Markdown
Member

Description

According to spec https://a2a-protocol.org/latest/specification/#111-protocol-requirements HTTP+JSON should support Content Type application/a2a+json and ACTS runner rely on that.

Adding support of this Content Type brings ACTS results
From:

========================================================
ACROSS 3 BINDING(S)
========================================================
  111 test(s) in all, each scored on the binding(s) it targets
  jsonrpc   93/101 passed   must 57/59   8 failed, 0 error(s)   NOT CONFORMANT
  grpc       83/88 passed   must 46/47   5 failed, 0 error(s)   NOT CONFORMANT
  rest       23/92 passed   must 13/49   69 failed, 0 error(s)   NOT CONFORMANT

To:

========================================================
ACROSS 3 BINDING(S)
========================================================
  111 test(s) in all, each scored on the binding(s) it targets
  jsonrpc   93/101 passed   must 57/59   8 failed, 0 error(s)   NOT CONFORMANT
  grpc       83/88 passed   must 46/47   5 failed, 0 error(s)   NOT CONFORMANT
  rest       86/92 passed   must 47/49   6 failed, 0 error(s)   NOT CONFORMANT

@ehsavoie

Copy link
Copy Markdown
Collaborator

@JakubWorek this is not in the 1.0.0 version of the specs https://a2a-protocol.org/v1.0.0/specification/#111-protocol-requirements. We can accept it on the server side but for the response I would still keep the initial one.
@kabir WDYT ?

@ehsavoie
ehsavoie requested review from ehsavoie, jmesnil and kabir and removed request for ehsavoie September 25, 2026 11:58
@JakubWorek

Copy link
Copy Markdown
Member Author

@JakubWorek this is not in the 1.0.0 version of the specs https://a2a-protocol.org/v1.0.0/specification/#111-protocol-requirements. We can accept it on the server side but for the response I would still keep the initial one. @kabir WDYT ?

Agree on that, I'll wait for response from @kabir and adjust my changes if this solution will be accepted

@kabir

kabir commented Sep 25, 2026

Copy link
Copy Markdown
Collaborator

I agree with @ehsavoie

@kabir

kabir commented Sep 25, 2026 •

Copy link
Copy Markdown
Collaborator

@JakubWorek BTW the 1.0.1 version of the spec is very confusing :-) It changes a bunch of stuff, while both 1.0.0 and 1.0.1 have 1.0 in the version header, so there is nothing to distinguish between the two.
When looking at https://a2a-protocol.org/latest/specification/, it seems to still be 1.0.0. So we've been ignoring 1.0.1 since we don't know what to do with it.

@JakubWorek

Copy link
Copy Markdown
Member Author

I agree with @ehsavoie

Okay, I adjusted mine solution

@ehsavoie
ehsavoie merged commit c78c472 into main Sep 25, 2026
21 checks passed
@ehsavoie
ehsavoie deleted the jakubworek/a2a-content-type branch September 25, 2026 16:53
@ehsavoie

Copy link
Copy Markdown
Collaborator

Thanks @JakubWorek

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