Skip to content

.NET: Seal ChatClientAgentThread - #2842

Merged
SergeyMenshykh merged 4 commits into
mainfrom
copilot/seal-chat-client-agent-thread
Jan 6, 2026
Merged

.NET: Seal ChatClientAgentThread#2842
SergeyMenshykh merged 4 commits into
mainfrom
copilot/seal-chat-client-agent-thread

Conversation

CopilotAI commented Dec 12, 2025

Copy link
Copy Markdown
Contributor

Motivation and Context

This follows the established pattern in the codebase where related classes (ChatClientAgentOptions, ChatClientAgentRunOptions, ChatClientAgentRunResponse<T>) are already sealed.

Description

Added sealed modifier to ChatClientAgentThread class declaration.

-public class ChatClientAgentThread : AgentThread
+publicsealedclassChatClientAgentThread:AgentThread

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.
Original prompt

Seal ChatClientAgentThread


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: SergeyMenshykh <68852919+SergeyMenshykh@users.noreply.github.com>
CopilotAI changed the title [WIP] Seal ChatClientAgentThread for enhanced stabilitySeal ChatClientAgentThreadDec 12, 2025
@markwallace-microsoftMark Wallace (markwallace-microsoft) added the .NET Usage: [Issues, PRs], Target: .Net label Dec 12, 2025
@github-actionsgithub-actionsBot changed the title Seal ChatClientAgentThread.NET: Seal ChatClientAgentThreadDec 12, 2025
@SergeyMenshykh
SergeyMenshykh marked this pull request as ready for review January 5, 2026 11:39
CopilotAI review requested due to automatic review settings January 5, 2026 11:39

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

Pull request overview

This PR adds the sealed modifier to the ChatClientAgentThread class to align with the established pattern in the codebase where related classes (ChatClientAgentOptions, ChatClientAgentRunOptions, and ChatClientAgentRunResponse<T>) are already sealed.

Key changes:

  • Modified ChatClientAgentThread from public class to public sealed class

@SergeyMenshykh
SergeyMenshykh added this pull request to the merge queueJan 6, 2026
Merged via the queue into main with commit 0aa0579Jan 6, 2026
14 checks passed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

.NETUsage: [Issues, PRs], Target: .Net

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

6 participants

@rogerbarreto@westey-m@SergeyMenshykh@markwallace-microsoft