feat: add GetExtendedAgentCardRequest as input parameter to GetExtendedAgentCard method by sokoliva · Pull Request #767 · a2aproject/a2a-python

sokoliva · 2026-03-04T15:58:39Z

Spec defines rpc GetExtendedAgentCard(GetExtendedAgentCardRequest) returns (AgentCard). Currently GetExtendedAgentCard does not take GetExtendedAgentCardRequest as input parameter.

Fixes #766 🦕

…edAgentCard method

gemini-code-assist · 2026-03-04T16:03:13Z

Summary of Changes

Hello, I'm Gemini Code Assist¹! 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 introduces a change to the GetExtendedAgentCard method within the A2A client library. It modifies the method signature to accept a GetExtendedAgentCardRequest object as input, bringing the implementation in line with the A2A protocol specification. This change affects the base client, client interface, transport layers (gRPC, JSONRPC, REST), and includes updates to existing tests and the addition of a new test file to ensure proper functionality and tenant isolation.

Highlights

Feature Addition: The GetExtendedAgentCard method now accepts GetExtendedAgentCardRequest as an input parameter, aligning with the A2A protocol specification.
Interface Updates: Updated the client interfaces and transport layers to include the new request parameter for the GetExtendedAgentCard method.
Test Adjustments: Modified existing tests and added a new test file to ensure the correct implementation and usage of the GetExtendedAgentCardRequest.

Changelog

src/a2a/client/base_client.py
- Modified to include GetExtendedAgentCardRequest as a parameter in the get_extended_agent_card method.
src/a2a/client/client.py
- Modified the abstract method get_extended_agent_card to include GetExtendedAgentCardRequest as a parameter.
src/a2a/client/transports/base.py
- Modified the abstract method get_extended_agent_card to include GetExtendedAgentCardRequest as a parameter.
src/a2a/client/transports/grpc.py
- Modified the get_extended_agent_card method to accept GetExtendedAgentCardRequest and pass it to the gRPC stub.
src/a2a/client/transports/jsonrpc.py
- Modified the get_extended_agent_card method to include GetExtendedAgentCardRequest as a parameter and pass it in the JSONRPC request.
src/a2a/client/transports/rest.py
- Modified the get_extended_agent_card method to include GetExtendedAgentCardRequest as a parameter and convert it to query parameters.
tests/client/transports/test_jsonrpc_client.py
- Updated test_get_card_with_extended_card_support_with_extensions to include GetExtendedAgentCardRequest.
tests/client/transports/test_rest_client.py
- Updated test_get_card_with_extended_card_support_with_extensions to include GetExtendedAgentCardRequest.
tests/client/transports/test_tenant_decorator.py
- Added a new test file to test the TenantTransportDecorator.
tests/integration/test_client_server_integration.py
- Updated integration tests to include GetExtendedAgentCardRequest in get_extended_agent_card calls.

Activity

The pull request addresses issue [Task] add GetExtendedAgentCardRequest as input parameter to GetExtendedAgentCard method #766.
The changes align the GetExtendedAgentCard method with the A2A protocol specification by adding GetExtendedAgentCardRequest as an input parameter.
The pull request includes updates to client interfaces, transport layers (gRPC, JSONRPC, REST), and tests to ensure proper functionality.

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 by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

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 pull request 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. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

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.

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. ↩

gemini-code-assist

Code Review

This pull request correctly adds the GetExtendedAgentCardRequest parameter to the get_extended_agent_card method across various client and transport layers, aligning it with the A2A specification. The changes are consistent and well-tested in the modified files.

However, a key component, TenantTransportDecorator, appears to have been missed. It needs to be updated to reflect the new method signature of get_extended_agent_card to avoid breaking its functionality. Additionally, the newly added test file for this decorator, tests/client/transports/test_tenant_decorator.py, contains a call that is inconsistent with the new method signature, which will cause test failures. Please refer to the specific comments for detailed suggestions to address these issues.

I am having trouble creating individual review comments. Click here to see my feedback.

tests/client/transports/test_tenant_decorator.py (6-20)

To use GetExtendedAgentCardRequest in this test file, it needs to be imported. Please add it to this import block, maintaining alphabetical order.

from a2a.types.a2a_pb2 import (
    AgentCard,
    CancelTaskRequest,
    CreateTaskPushNotificationConfigRequest,
    DeleteTaskPushNotificationConfigRequest,
    GetExtendedAgentCardRequest,
    GetTaskPushNotificationConfigRequest,
    GetTaskRequest,
    ListTaskPushNotificationConfigsRequest,
    ListTasksRequest,
    Message,
    Part,
    SendMessageRequest,
    StreamResponse,
    SubscribeToTaskRequest,
)

tests/client/transports/test_tenant_decorator.py (132-136)

The get_extended_agent_card method signature has been updated to require a request parameter, but this test call is missing it. This will cause a TypeError.

It also appears that TenantTransportDecorator itself has not been updated in this PR to handle this new parameter. Please ensure you update TenantTransportDecorator.get_extended_agent_card to accept and forward the request parameter to the underlying transport, and then update this test as suggested.

        # Test get_extended_agent_card
        card = MagicMock(spec=AgentCard);
        mock_transport.get_extended_agent_card.return_value = card;
        request = GetExtendedAgentCardRequest();
        res = await decorator.get_extended_agent_card(request);
        assert res is card

src/a2a/client/transports/rest.py

…T transport.

🤖 I have created a release *beep* *boop* --- ### ⚠ BREAKING CHANGES * **spec**: upgrade SDK to A2A 1.0 spec and use proto-based types ([#572](#572), [#665](#665), [#804](#804), [#765](#765)) * **client:** introduce ServiceParameters for extensions and include it in ClientCallContext ([#784](#784)) * **client:** rename "callback" -> "push_notification_config" ([#749](#749)) * **client:** transport agnostic interceptors ([#796](#796)) ([a910cbc](a910cbc)) * add `protocol_version` column to Task and PushNotificationConfig models and create a migration ([#789](#789)) ([2e2d431](2e2d431)) * **server:** implement `Resource Scoping` for tasks and push notifications ([#709](#709)) ([f0d4669](f0d4669)) ### Features * add GetExtendedAgentCardRequest as input parameter to GetExtendedAgentCard method ([#767](#767)) ([13a092f](13a092f)) * add validation for the JSON-RPC version ([#808](#808)) ([6eb7e41](6eb7e41)) * **client:** expose close() and async context manager support on abstract Client ([#719](#719)) ([e25ba7b](e25ba7b)) * **compat:** AgentCard backward compatibility helpers and tests ([#760](#760)) ([81f3494](81f3494)) * **compat:** GRPC client compatible with 0.3 server ([#779](#779)) ([0ebca93](0ebca93)) * **compat:** GRPC server compatible with 0.3 client ([#772](#772)) ([80d827a](80d827a)) * **compat:** legacy v0.3 protocol models, conversion logic and utilities ([#754](#754)) ([26835ad](26835ad)) * **compat:** REST and JSONRPC clients compatible with 0.3 servers ([#798](#798)) ([08794f7](08794f7)) * **compat:** REST and JSONRPC servers compatible with 0.3 clients ([#795](#795)) ([9856054](9856054)) * **compat:** set a2a-version header to 1.0.0 ([#764](#764)) ([4cb68aa](4cb68aa)) * **compat:** unify v0.3 REST url prefix and expand cross-version tests ([#820](#820)) ([0925f0a](0925f0a)) * database forward compatibility: make `owner` field optional ([#812](#812)) ([cc29d1f](cc29d1f)) * handle tenant in Client ([#758](#758)) ([5b354e4](5b354e4)) * implement missing push notifications related methods ([#711](#711)) ([041f0f5](041f0f5)) * implement rich gRPC error details per A2A v1.0 spec ([#790](#790)) ([245eca3](245eca3)) * **rest:** add tenant support to rest ([#773](#773)) ([4771b5a](4771b5a)) * send task as a first subscribe event ([#716](#716)) ([e71ac62](e71ac62)) * **server, grpc:** Implement tenant context propagation for gRPC requests. ([#781](#781)) ([164f919](164f919)) * **server, json-rpc:** Implement tenant context propagation for JSON-RPC requests. ([#778](#778)) ([72a330d](72a330d)) * **server:** add v0.3 legacy compatibility for database models ([#783](#783)) ([08c491e](08c491e)) * **spec:** add `tasks/list` method with filtering and pagination to the specification ([#511](#511)) ([d5818e5](d5818e5)) * use StreamResponse as push notifications payload ([#724](#724)) ([a149a09](a149a09)) * **rest:** update REST error handling to use `google.rpc.Status` ([#838](#838)) ([ea7d3ad](ea7d3ad)) ### Bug Fixes * add history length and page size validations ([#726](#726)) ([e67934b](e67934b)) * allign error codes with the latest spec ([#826](#826)) ([709b1ff](709b1ff)) * **client:** align send_message signature with BaseClient ([#740](#740)) ([57cb529](57cb529)) * get_agent_card trailing slash when agent_card_path="" ([#799](#799)) ([#800](#800)) ([a55c97e](a55c97e)) * handle parsing error in REST ([#806](#806)) ([bbd09f2](bbd09f2)) * Improve error handling for Timeout exceptions on REST and JSON-RPC clients ([#690](#690)) ([2acd838](2acd838)) * Improve streaming errors handling ([#576](#576)) ([7ea7475](7ea7475)) * properly handle unset and zero history length ([#717](#717)) ([72a1007](72a1007)) * return entire history when history_length=0 ([#537](#537)) ([acdc0de](acdc0de)) * return mandatory fields from list_tasks ([#710](#710)) ([6132053](6132053)) * taskslist error on invalid page token and response serialization ([#814](#814)) ([a102d31](a102d31)) * use correct REST path for Get Extended Agent Card operation ([#769](#769)) ([ced3f99](ced3f99)) * Use POST method for REST endpoint /tasks/{id}:subscribe ([#843](#843)) ([a0827d0](a0827d0)) --- 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: Ivan Shymko <ishymko@google.com>

feat: add GetExtendedAgentCardRequest as input parameter to GetExtend…

f4c0670

…edAgentCard method

sokoliva requested a review from a team as a code owner March 4, 2026 15:58

sokoliva requested a review from ishymko March 4, 2026 15:58

fix: delete accidentally pushed file

675cabb

fix: format

5f76f1e

gemini-code-assist bot reviewed Mar 4, 2026

View reviewed changes

fix: remove unused import, add desc

6ed09b8

sokoliva mentioned this pull request Mar 4, 2026

feat: handle tenant in Client #758

Merged

4 tasks

ishymko reviewed Mar 5, 2026

View reviewed changes

src/a2a/client/transports/rest.py Outdated Show resolved Hide resolved

ishymko approved these changes Mar 5, 2026

View reviewed changes

feat: remove query parameters from the /card GET request in the RES…

9ea0eef

…T transport.

sokoliva merged commit 13a092f into a2aproject:1.0-dev Mar 5, 2026
6 checks passed

This was referenced Mar 12, 2026

chore(1.0-dev): release 1.0.0-alpha.0 #818

Closed

chore(1.0-dev): release 1.0.0-alpha.0 #825

Closed

chore(1.0-dev): release 1.0.0-alpha.0 #828

Merged

a2a-bot mentioned this pull request Mar 17, 2026

chore(1.0-dev): release 1.0.0-alpha.0 #852

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: add GetExtendedAgentCardRequest as input parameter to GetExtendedAgentCard method#767

feat: add GetExtendedAgentCardRequest as input parameter to GetExtendedAgentCard method#767
sokoliva merged 5 commits intoa2aproject:1.0-devfrom
sokoliva:extended-card-req

sokoliva commented Mar 4, 2026

Uh oh!

gemini-code-assist bot commented Mar 4, 2026

Uh oh!

gemini-code-assist bot left a comment

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

sokoliva commented Mar 4, 2026

Uh oh!

gemini-code-assist bot commented Mar 4, 2026

Summary of Changes

Highlights

Footnotes

Uh oh!

gemini-code-assist bot left a comment

Choose a reason for hiding this comment

Code Review

tests/client/transports/test_tenant_decorator.py (6-20)

tests/client/transports/test_tenant_decorator.py (132-136)

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants