Skip to content

Conversation

@SergeyMenshykh
Copy link
Member

Contributes to: #2542

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 seals three classes that are not intended for inheritance, improving code clarity and following best practices for C# class design.

  • Sealed ChatClientAgentOptions class to prevent unintended inheritance
  • Sealed InvokingContext and InvokedContext nested classes within AIContextProvider to clarify they are not meant to be extended

Reviewed changes

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

File Description
dotnet/src/Microsoft.Agents.AI/ChatClient/ChatClientAgentOptions.cs Sealed ChatClientAgentOptions class to prevent inheritance
dotnet/src/Microsoft.Agents.AI.Abstractions/AIContextProvider.cs Sealed nested InvokingContext and InvokedContext classes to prevent inheritance

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants