Skip to content

Conversation

@westey-m
Copy link
Contributor

Motivation and Context

#2542

Description

  • Expose metadata from A2AAgent and seal AIAgentMetadata

Contribution Checklist

  • The code builds clean without any errors or warnings
  • The PR follows the Contribution Guidelines
  • All unit tests pass, and I have added new tests where possible
  • Is this a breaking change? If yes, add "[BREAKING]" prefix to the title of the PR.

Copilot AI review requested due to automatic review settings January 23, 2026 20:22
@github-actions github-actions bot changed the title Expose metadata from A2AAgent and seal AIAgentMetadata .NET: Expose metadata from A2AAgent and seal AIAgentMetadata Jan 23, 2026
@westey-m westey-m mentioned this pull request Jan 23, 2026
61 tasks
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR exposes metadata from the A2AAgent class and seals the AIAgentMetadata class as part of the .NET Single Agent API review (issue #2542).

Changes:

  • Sealed the AIAgentMetadata class to prevent inheritance
  • Added GetService method override to A2AAgent to expose A2AClient and AIAgentMetadata
  • Added comprehensive unit tests for the GetService method in A2AAgent

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
dotnet/src/Microsoft.Agents.AI.Abstractions/AIAgentMetadata.cs Sealed the AIAgentMetadata class to prevent inheritance
dotnet/src/Microsoft.Agents.AI.A2A/A2AAgent.cs Added GetService override to expose A2AClient and AIAgentMetadata with static metadata instance
dotnet/tests/Microsoft.Agents.AI.A2A.UnitTests/A2AAgentTests.cs Added comprehensive unit tests for GetService method covering all code paths

@westey-m westey-m added this pull request to the merge queue Jan 26, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jan 26, 2026
@westey-m westey-m added this pull request to the merge queue Jan 27, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jan 27, 2026
@westey-m westey-m added this pull request to the merge queue Jan 27, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jan 27, 2026
@westey-m westey-m added this pull request to the merge queue Jan 27, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jan 27, 2026
@westey-m westey-m added this pull request to the merge queue Jan 27, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jan 27, 2026
@westey-m westey-m added this pull request to the merge queue Jan 27, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jan 27, 2026
@westey-m westey-m added this pull request to the merge queue Jan 27, 2026
@westey-m westey-m removed this pull request from the merge queue due to a manual request Jan 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants