Skip to content

.NET: feat: Update Github Copilot SDK to 1.0.0-beta.2#5699

Merged
rogerbarreto merged 5 commits intomainfrom
dev/update_github_copilot_sdk
May 7, 2026
Merged

.NET: feat: Update Github Copilot SDK to 1.0.0-beta.2#5699
rogerbarreto merged 5 commits intomainfrom
dev/update_github_copilot_sdk

Conversation

@lokitoth
Copy link
Copy Markdown
Member

@lokitoth lokitoth commented May 7, 2026

Motivation and Context

The current GH Copilot SDK reference is multiple months old, and is also referencing a package triggering a NuGet warning-as-error on build.

  • 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 May 7, 2026 15:55
@moonbox3 moonbox3 added the .NET label May 7, 2026
@lokitoth lokitoth enabled auto-merge May 7, 2026 15:56
@github-actions github-actions Bot changed the title feat: Update Github Copilot SDK to 1.0.0-beta.2 .NET: feat: Update Github Copilot SDK to 1.0.0-beta.2 May 7, 2026
Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

Automated Code Review

Reviewers: 4 | Confidence: 84% | Result: All clear

Reviewed: Correctness, Security Reliability, Test Coverage, Design Approach


Automated review by lokitoth's agents

Copy link
Copy Markdown
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

Updates the .NET GitHub Copilot integration to use GitHub.Copilot.SDK 1.0.0-beta.2, including a small API adaptation to the SDK’s attachment model.

Changes:

  • Bump GitHub.Copilot.SDK from 0.1.29 to 1.0.0-beta.2 via central package management.
  • Update GitHubCopilotAgent to use the new UserMessageAttachmentFile type for DataContent-backed attachments.
  • Minor formatting change in a unit test file.

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 1 comment.

File Description
dotnet/Directory.Packages.props Updates the centralized NuGet version for GitHub.Copilot.SDK to 1.0.0-beta.2.
dotnet/src/Microsoft.Agents.AI.GitHub.Copilot/GitHubCopilotAgent.cs Adapts attachment creation to the new SDK attachment type used when sending messages.
dotnet/tests/Microsoft.Agents.AI.GitHub.Copilot.UnitTests/GitHubCopilotAgentTests.cs Contains a whitespace-only change in the CopySessionConfig_CopiesAllProperties test.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@lokitoth lokitoth self-assigned this May 7, 2026
@lokitoth lokitoth added this pull request to the merge queue May 7, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks May 7, 2026
@lokitoth lokitoth added this pull request to the merge queue May 7, 2026
@lokitoth lokitoth removed this pull request from the merge queue due to a manual request May 7, 2026
@lokitoth lokitoth added this pull request to the merge queue May 7, 2026
@rogerbarreto rogerbarreto removed this pull request from the merge queue due to a manual request May 7, 2026
@rogerbarreto rogerbarreto merged commit 1489d66 into main May 7, 2026
24 checks passed
@rogerbarreto
Copy link
Copy Markdown
Member

Force merge to avoid the dead lock with the hosted-it failure which fix is also locked out due to the vulnerability.

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.

5 participants