Skip to content

Python: Update GitHub Copilot SDK to 1.0.11 - #8002

Merged
Evan Mattson (moonbox3) merged 6 commits into
microsoft:mainfrom
droideronline:update-copilot-sdk-1-0-11
Sep 4, 2026
Merged

Python: Update GitHub Copilot SDK to 1.0.11#8002
Evan Mattson (moonbox3) merged 6 commits into
microsoft:mainfrom
droideronline:update-copilot-sdk-1-0-11

Conversation

@droideronline

Copy link
Copy Markdown
Contributor

Motivation & Context

The Python GitHub Copilot adapter is pinned to github-copilot-sdk==1.0.2, which prevents consumers from using the stable SDK's BYOK bearer token provider callback. SDK 1.0.11 adds ProviderConfig.bearer_token_provider and the async-capable BearerTokenProvider type.

The Copilot SDK changelog was reviewed for the releases between 1.0.2 and 1.0.11. The upgrade is additive for this adapter's existing usage, and the adapter already forwards its generic session options.

Description & Review Guide

  • What are the major changes? Update the package dependency declaration and the corresponding uv.lock entry from 1.0.2 to 1.0.11.
  • What is the impact of these changes? Consumers can provide rotating synchronous or asynchronous BYOK bearer tokens through the SDK provider configuration. Existing adapter behavior remains unchanged.
  • What do you want reviewers to focus on? Confirm the lockfile metadata and that no runtime adapter changes are needed for the SDK's additive provider callback API.

Related Issue

Fixes #8001

Contribution Checklist

  • The code builds clean without any errors or warnings
  • All unit tests pass, and I have added new tests where possible
  • The PR follows the Contribution Guidelines
  • This PR is linked to an issue and there is no other open PR for this issue (see Related Issue above).
  • This is not a breaking change.

Copilot AI balanced review requested due to automatic review settings September 1, 2026 17:31

Copilot AI 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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

auto-merge was automatically disabled September 2, 2026 08:29

Head branch was pushed to by a user without write access

auto-merge was automatically disabled September 2, 2026 08:40

Head branch was pushed to by a user without write access

@moonbox3

Copy link
Copy Markdown
Contributor

Dineshsuriya D (@droideronline) please have a look at the file conflict.

@droideronline

Dineshsuriya D (droideronline) commented Sep 4, 2026

Copy link
Copy Markdown
Contributor Author

Eduard van Valkenburg (@eavanvalkenburg) Evan Mattson (@moonbox3) could you please take another look at this PR?

I resolved the main merge conflict and pushed the update in commit 81aee5340. I also isolated the compatibility Any used by the permission-handler boundary into a private _PermissionHandlerContext alias in commit 1e7178ae4, preserving support for both the legacy dictionary context and SDK 1.0.11 PermissionInvocation context without changing runtime behavior.

Validation is clean: formatting/lint, Pyright, mypy, ty, pyrefly, zuban, and the GitHub Copilot tests (183 passed, 6 deselected, 95% coverage).

@moonbox3
Evan Mattson (moonbox3) added this pull request to the merge queue Sep 4, 2026
Merged via the queue into microsoft:main with commit e5b55c7 Sep 4, 2026
37 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

python Usage: [Issues, PRs], Target: Python

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Python: Update GitHub Copilot SDK to support BYOK token providers

4 participants