Skip to content

Conversation

westey-m
Copy link
Contributor

Motivation and Context

microsoft/agent-framework#1122

Description

Add adapters to allow SK agents to be exposed as AIAgent

Contribution Checklist

@westey-m westey-m requested a review from a team as a code owner October 10, 2025 11:05
@westey-m westey-m requested a review from Copilot October 10, 2025 11:05
Copy link
Contributor

@Copilot 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 adds adapters to allow Semantic Kernel (SK) agents to be exposed as AIAgent implementations from the Microsoft Agent Framework. The implementation provides a common interface conformance pattern that enables different agent types to work through the same AIAgent abstraction.

  • Introduces AIAgentAdapter and AIAgentThreadAdapter classes to bridge SK agents with the Microsoft Agent Framework
  • Adds extension methods for each agent type (OpenAI, Azure AI, Bedrock, etc.) to convert to AIAgent
  • Creates comprehensive unit and integration tests to verify the adapter functionality

Reviewed Changes

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

Show a summary per file
File Description
dotnet/src/Agents/Abstractions/AIAgent/AIAgentAdapter.cs Core adapter class that wraps SK agents and exposes them as AIAgent
dotnet/src/Agents/Abstractions/AIAgent/AIAgentThreadAdapter.cs Thread adapter for bridging SK agent threads with AIAgent threads
dotnet/src/Agents/Abstractions/AgentExtensions.cs Base extension method for converting any SK agent to AIAgent
dotnet/src/Agents/*/Extensions.cs Agent-specific extension methods for each agent type
Unit test files Comprehensive test coverage for all adapter functionality
Integration test files End-to-end testing of adapter conformance

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

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.

2 participants