Skip to content

.NET: Add AIAgent implementation for GitHub Copilot SDK#3395

Merged
dmytrostruk merged 41 commits into
mainfrom
copilot/add-aiagent-implementation
Jan 27, 2026
Merged

.NET: Add AIAgent implementation for GitHub Copilot SDK#3395
dmytrostruk merged 41 commits into
mainfrom
copilot/add-aiagent-implementation

Conversation

Copilot AI commented Jan 22, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

GitHub Copilot SDK AIAgent Implementation

All Changes Complete

  • Fixed event handling: Added AgentResponseUpdate yields for SessionIdleEvent and SessionErrorEvent
  • Removed explicit subscription.Dispose() (handled by using statement)
  • Removed AutoStart from README example (defaults to true)
  • Moved name/description defaults to primary constructor (applies to all construction paths)
  • Made Name and Description properties non-nullable (always have defaults)
  • Enhanced UsageContent with complete usage details:
    • InputTokenCount from InputTokens
    • OutputTokenCount from OutputTokens
    • CachedInputTokenCount from CacheReadTokens
    • AdditionalCounts for CacheWriteTokens, Cost, Duration
  • All 11 tests passing across net8.0, net9.0, net10.0
  • All builds successful

Implementation Complete

All PR feedback addressed. Event handling complete, usage tracking comprehensive, defaults properly applied.

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

@stephentoub

stephentoub commented Jan 22, 2026

Copy link
Copy Markdown
Member

@westey-m I was talking with @SteveSanderson earlier today about doing exactly this. :)

Copilot AI and others added 3 commits January 22, 2026 21:10
@westey-m
Add GitHub Copilot SDK AIAgent implementation with tests
5a39df4 
Co-authored-by: westey-m <164392973+westey-m@users.noreply.github.com>
@westey-m
Add projects to solution and fix sample imports
9178095 
Co-authored-by: westey-m <164392973+westey-m@users.noreply.github.com>
@westey-m
Improve pragma comment clarity in GithubCopilotAgentThread
b365707 
Co-authored-by: westey-m <164392973+westey-m@users.noreply.github.com>
Copilot AI changed the title [WIP] Add AIAgent implementation for Github Copilot SDK .NET: Add AIAgent implementation for GitHub Copilot SDK Jan 22, 2026
Copilot AI requested a review from westey-m January 22, 2026 21:23
@markwallace-microsoft markwallace-microsoft added documentation Usage: [Issues, PRs], Target: documentation in the code base and learn docs .NET Usage: [Issues, PRs], Target: .Net labels Jan 23, 2026
westey-m
westey-m reviewed Jan 23, 2026
View reviewed changes
Comment thread dotnet/src/Microsoft.Agents.AI.GithubCopilot/GithubCopilotAgentThread.cs Outdated
Comment thread dotnet/src/Microsoft.Agents.AI.GithubCopilot/GithubCopilotAgentThread.cs Outdated
Comment thread dotnet/src/Microsoft.Agents.AI.GithubCopilot/GithubCopilotAgentThread.cs Outdated
Comment thread dotnet/src/Microsoft.Agents.AI.GithubCopilot/GithubCopilotAgentThread.cs Outdated
Comment thread dotnet/samples/GettingStarted/AgentProviders/Agent_With_GithubCopilot/Program.cs Outdated
Comment thread dotnet/samples/GettingStarted/AgentProviders/Agent_With_GithubCopilot/README.md
Comment thread dotnet/src/Microsoft.Agents.AI.GithubCopilot/GithubCopilotAgent.cs Outdated
Comment thread dotnet/src/Microsoft.Agents.AI.GithubCopilot/GithubCopilotAgent.cs Outdated
@westey-m
Address PR feedback: internal constructor/setter, remove CopilotClien…
44f3ab6 
…tOptions ctor, streaming improvements, better sample, container warning

Co-authored-by: westey-m <164392973+westey-m@users.noreply.github.com>
Copilot AI requested a review from westey-m January 23, 2026 09:42
westey-m
westey-m reviewed Jan 23, 2026
View reviewed changes
Comment thread dotnet/src/Microsoft.Agents.AI.GithubCopilot/GithubCopilotAgent.cs Outdated
@westey-m
Add ownsClient parameter to allow caller control over client disposal
52ba627 
Co-authored-by: westey-m <164392973+westey-m@users.noreply.github.com>
Copilot AI requested a review from westey-m January 23, 2026 10:49
This was referenced Mar 4, 2026
Open
Closed
Closed
This was referenced Mar 16, 2026
Open
Closed
Merged
Closed
Open
Closed
This was referenced Mar 23, 2026
Open
Open
Open
Closed
Closed
Closed
This was referenced Apr 6, 2026
Closed
Open
Open
This was referenced May 2, 2026
Closed
Closed
Merged
This was referenced May 9, 2026
Closed
Closed
Closed
Closed
Closed
This was referenced May 18, 2026
Closed
Closed
@dependabot dependabot Bot mentioned this pull request Jun 10, 2026
Closed
@dependabot dependabot Bot mentioned this pull request Jun 24, 2026
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dmytrostruk dmytrostruk dmytrostruk left review comments
Copilot code review Copilot Copilot left review comments
@stephentoub stephentoub stephentoub approved these changes
@rogerbarreto rogerbarreto rogerbarreto approved these changes
+1 more reviewer
@westey-m westey-m westey-m approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

@dmytrostruk dmytrostruk

@westey-m westey-m

Copilot code review Copilot

Labels

documentation Usage: [Issues, PRs], Target: documentation in the code base and learn docs .NET Usage: [Issues, PRs], Target: .Net

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@stephentoub @westey-m @dmytrostruk @rogerbarreto @markwallace-microsoft