Skip to content

fix(spec): Add Transport enum to specification#909

Merged
holtskinner merged 2 commits into
mainfrom
add-transport
Jul 24, 2025
Merged

fix(spec): Add Transport enum to specification#909
holtskinner merged 2 commits into
mainfrom
add-transport

Conversation

@holtskinner

@holtskinner holtskinner commented Jul 24, 2025

Copy link
Copy Markdown
Member

Adds Enum of supported Transports to Specification, with extensibility for proprietary protocols.
Adds URL formatting requirement for AgentInterface.url and AgentCard.url

Co-authored-by: Stefano Maestri stefano.maestri@javalinux.it

Co-authored-by: Stefano Maestri <stefano.maestri@javalinux.it>
@holtskinner holtskinner requested a review from a team as a code owner July 24, 2025 18:55

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Summary of Changes

Hello @holtskinner, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

This pull request enhances the A2A specification by formalizing the definition of supported transport protocols. It introduces a dedicated enum for transports and significantly improves the documentation and validation of transport-related fields within the AgentCard and AgentInterface schemas. This change aims to provide clearer guidance for agents and clients on how to communicate and negotiate transport methods.

Highlights

  • New TransportProtocol Enum: Introduced a new TransportProtocol enum in types/src/types.ts and its corresponding definition in specification/json/a2a.json. This enum formalizes supported communication methods like JSONRPC, GRPC, and HTTP+JSON, improving clarity and standardization.
  • Enhanced AgentInterface Definition: The AgentInterface's transport field now leverages the new TransportProtocol enum (while retaining string extensibility). Its description has been improved, and url and transport fields now include examples and format annotations for better schema validation and documentation.
  • Improved AgentCard Properties: The AgentCard's preferredTransport and url properties have been updated with more detailed descriptions, examples, and default values. The additionalInterfaces property's description has been significantly expanded to include best practices for declaring multiple transport options, aiding clients in transport negotiation and fallback scenarios.
Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point in your pull request via creating an issue comment (i.e. comment on the pull request page) using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in issue comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments or fill out our survey to provide feedback.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

@holtskinner holtskinner merged commit e834347 into main Jul 24, 2025
4 checks passed

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request introduces a TransportProtocol enum to both the TypeScript types and the JSON schema to standardize supported transport protocols. By referencing the TransportProtocol enum, the schema can be more robust.

Comment on lines 182 to 191
"preferredTransport": {
"description": "The transport protocol for the preferred endpoint. Defaults to 'JSONRPC' if not specified.",
"default": "JSONRPC",
"description": "The transport protocol for the preferred endpoint (the main 'url' field).\nIf not specified, defaults to 'JSONRPC'.\n\nIMPORTANT: The transport specified here MUST be available at the main 'url'.\nThis creates a binding between the main URL and its supported transport protocol.\nClients should prefer this transport and URL combination when both are supported.",
"examples": [
"JSONRPC",
"GRPC",
"HTTP+JSON"
],
"type": "string"
},

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

To ensure consistency and leverage the new TransportProtocol enum for validation, the $ref should be used to reference the enum, instead of defining default, description, and examples inline.

                    "description": "The transport protocol for the preferred endpoint (the main 'url' field).\nIf not specified, defaults to 'JSONRPC'.\n\nIMPORTANT: The transport specified here MUST be available at the main 'url'.\nThis creates a binding between the main URL and its supported transport protocol.\nClients should prefer this transport and URL combination when both are supported.",
                    "$ref": "#/definitions/TransportProtocol"

Comment on lines 297 to 305
"transport": {
"description": "The transport protocol supported at this URL. This is a string to allow for future\nextension. Core supported transports include 'JSONRPC', 'GRPC', and 'HTTP+JSON'.",
"description": "The transport protocol supported at this URL.",
"examples": [
"JSONRPC",
"GRPC",
"HTTP+JSON"
],
"type": "string"
},

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

To leverage the newly added TransportProtocol enum for validation, the $ref should be used to reference the enum, instead of defining description and examples inline.

                    "description": "The transport protocol supported at this URL.",
                    "$ref": "#/definitions/TransportProtocol"

@holtskinner holtskinner deleted the add-transport branch July 24, 2025 18:56
holtskinner pushed a commit that referenced this pull request Jul 30, 2025
🤖 I have created a release *beep* *boop*
---


## [0.3.0](v0.2.6...v0.3.0)
(2025-07-30)


### ⚠ BREAKING CHANGES

* Add mTLS to SecuritySchemes, add oauth2 metadata url field, allow
Skills to specify Security
([#901](#901))
* Change Well-Known URI for Agent Card hosting from `agent.json` to
`agent-card.json` ([#841](#841))
* Add method for fetching extended card
([#929](#929))

### Features

* Add `signatures` to the `AgentCard`
([#917](#917))
([ef4a305](ef4a305))
* Add method for fetching extended card
([#929](#929))
([2cd7d98](2cd7d98))
* Add mTLS to SecuritySchemes, add oauth2 metadata url field, allow
Skills to specify Security
([#901](#901))
([e162c0c](e162c0c))


### Bug Fixes

* **spec:** Add `SendMessageRequest.request` `json_name` mapping to
`message` ([#904](#904))
([2eef3f6](2eef3f6))
* **spec:** Add Transport enum to specification
([#909](#909))
([e834347](e834347))


### Code Refactoring

* Change Well-Known URI for Agent Card hosting from `agent.json` to
`agent-card.json` ([#841](#841))
([0858ddb](0858ddb))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
holtskinner pushed a commit to a2aproject/a2a-python that referenced this pull request Jul 31, 2025
🤖 I have created a release *beep* *boop*
---


##
[0.3.0](v0.2.16...v0.3.0)
(2025-07-31)


### ⚠ BREAKING CHANGES

* **deps:** Make opentelemetry an optional dependency
([#369](#369))
* **spec:** Update Agent Card Well-Known Path to
`/.well-known/agent-card.json`
([#320](#320))
* Remove custom `__getattr__` and `__setattr__` for `camelCase` fields
in `types.py`
([#335](#335))
* Add mTLS to SecuritySchemes, add oauth2 metadata url field, allow
Skills to specify Security
([#362](#362))
* Support for serving agent card at deprecated path
([#352](#352))

### Features

* Add `metadata` as parameter to `TaskUpdater.update_status()`
([#371](#371))
([9444ed6](9444ed6))
* Add mTLS to SecuritySchemes, add oauth2 metadata url field, allow
Skills to specify Security
([#362](#362))
([be6c517](be6c517))
* Add RESTful API Serving
([#348](#348))
([82a6b7c](82a6b7c))
* Add server-side support for plumbing requested and activated
extensions ([#333](#333))
([4d5b92c](4d5b92c))
* Allow agent cards (default and extended) to be dynamic
([#365](#365))
([ee92aab](ee92aab))
* Support for serving agent card at deprecated path
([#352](#352))
([2444034](2444034))
* support non-blocking `sendMessage`
([#349](#349))
([70b4999](70b4999))
* Type update to support fetching extended card
([#361](#361))
([83304bb](83304bb))


### Bug Fixes

* Add Input Validation for Task Context IDs in new_task Function
([#340](#340))
([a7ed7ef](a7ed7ef))
* **deps:** Reduce FastAPI library required version to `0.95.0`
([#372](#372))
([a319334](a319334))
* Remove `DeprecationWarning` for regular properties
([#345](#345))
([2806f3e](2806f3e))
* **spec:** Add `SendMessageRequest.request` `json_name` mapping to
`message` proto
([bc97cba](bc97cba))
* **spec:** Add Transport enum to specification
(a2aproject/A2A#909)
([d9e463c](d9e463c))


### Documentation

* Address typos in docstrings and docs.
([#370](#370))
([ee48d68](ee48d68))


### Miscellaneous Chores

* Add support for authenticated extended card method
([#356](#356))
([b567e80](b567e80))


### Code Refactoring

* **deps:** Make opentelemetry an optional dependency
([#369](#369))
([9ad8b96](9ad8b96))
* Remove custom `__getattr__` and `__setattr__` for `camelCase` fields
in `types.py`
([#335](#335))
([cd94167](cd94167))
* **spec:** Update Agent Card Well-Known Path to
`/.well-known/agent-card.json`
([#320](#320))
([270ea9b](270ea9b))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
debug313 pushed a commit to debug313/agent-to-agent that referenced this pull request Apr 12, 2026
🤖 I have created a release *beep* *boop*
---


## [0.3.0](a2aproject/A2A@v0.2.6...v0.3.0)
(2025-07-30)


### ⚠ BREAKING CHANGES

* Add mTLS to SecuritySchemes, add oauth2 metadata url field, allow
Skills to specify Security
([#901](a2aproject/A2A#901))
* Change Well-Known URI for Agent Card hosting from `agent.json` to
`agent-card.json` ([#841](a2aproject/A2A#841))
* Add method for fetching extended card
([#929](a2aproject/A2A#929))

### Features

* Add `signatures` to the `AgentCard`
([#917](a2aproject/A2A#917))
([e799342](a2aproject/A2A@e799342))
* Add method for fetching extended card
([#929](a2aproject/A2A#929))
([ae89f89](a2aproject/A2A@ae89f89))
* Add mTLS to SecuritySchemes, add oauth2 metadata url field, allow
Skills to specify Security
([#901](a2aproject/A2A#901))
([a5c881e](a2aproject/A2A@a5c881e))


### Bug Fixes

* **spec:** Add `SendMessageRequest.request` `json_name` mapping to
`message` ([#904](a2aproject/A2A#904))
([273a1f1](a2aproject/A2A@273a1f1))
* **spec:** Add Transport enum to specification
([#909](a2aproject/A2A#909))
([3076ae6](a2aproject/A2A@3076ae6))


### Code Refactoring

* Change Well-Known URI for Agent Card hosting from `agent.json` to
`agent-card.json` ([#841](a2aproject/A2A#841))
([c20c28b](a2aproject/A2A@c20c28b))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
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.

1 participant