Update GitHub Copilot CLI to version 1.0.56#319076
Open
DonJayamanne wants to merge 6 commits into
Open
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the built-in Copilot extension’s bundled Copilot CLI SDK to @github/copilot@1.0.56-1, aligning VS Code’s Copilot CLI session integration with updated SDK APIs (session/options updates, permissions API shape, and session manager hooks), and refreshes tests/packaging checks for the SDK’s changed native binary set.
Changes:
- Bump
@github/copilotdependency to1.0.56-1and adjust lockfile entries (including new transitive deps). - Refactor Copilot CLI session wiring to use new SDK APIs (
updateOptions,permissions.setRequired,loadDeferredRepoHooks(sessionId)), and centralize session typing via a sharedcommon/utilstype alias. - Update packaging/runtime checks and tests to account for new native binaries (e.g.
cli-native.node) and revised auth-info resolution.
Show a summary per file
| File | Description |
|---|---|
| extensions/copilot/src/extension/chatSessions/copilotcli/vscode-node/test/copilotCLISDKUpgrade.spec.ts | Updates native-binary allowlist to reflect SDK prebuild changes (incl. cli-native.node, mxc additions). |
| extensions/copilot/src/extension/chatSessions/copilotcli/node/test/testHelpers.ts | Updates test mocks for new session APIs (permissions/updateOptions) (currently needs fixes). |
| extensions/copilot/src/extension/chatSessions/copilotcli/node/test/exitPlanModeHandler.spec.ts | Updates tests to use the new shared Session type alias. |
| extensions/copilot/src/extension/chatSessions/copilotcli/node/test/copilotcliSession.spec.ts | Updates session tests for updated session typing/options APIs. |
| extensions/copilot/src/extension/chatSessions/copilotcli/node/test/copilotCliAuth.spec.ts | Adapts auth tests to new SDK auth resolution helper and ensures extensionPath is available. |
| extensions/copilot/src/extension/chatSessions/copilotcli/node/permissionHelpers.ts | Rebinds permission result typing to the new shared Session type alias. |
| extensions/copilot/src/extension/chatSessions/copilotcli/node/mcpHandler.ts | Switches to shared Session type alias for MCP config typing. |
| extensions/copilot/src/extension/chatSessions/copilotcli/node/exitPlanModeHandler.ts | Switches to shared Session type alias for plan-mode handling types. |
| extensions/copilot/src/extension/chatSessions/copilotcli/node/copilotcliSessionService.ts | Refactors session creation/rehydration to updated SDK hooks/options/permissions API. |
| extensions/copilot/src/extension/chatSessions/copilotcli/node/copilotcliSession.ts | Uses updateOptions({ authInfo }) instead of deprecated auth setters. |
| extensions/copilot/src/extension/chatSessions/copilotcli/node/copilotCli.ts | Improves token authInfo handling by deferring to SDK’s token→authInfo resolver when available. |
| extensions/copilot/src/extension/chatSessions/copilotcli/common/utils.ts | Introduces shared Session/LocalSession type aliases based on SDK internal session manager typing. |
| extensions/copilot/script/postinstall.ts | Ensures cli-native.node is included when copying relevant prebuild binaries. |
| extensions/copilot/package.json | Bumps @github/copilot dependency version. |
| extensions/copilot/package-lock.json | Locks upgraded Copilot package artifacts and new transitive dependency metadata. |
Copilot's findings
Files not reviewed (1)
- extensions/copilot/package-lock.json: Language not supported
- Files reviewed: 14/15 changed files
- Comments generated: 10
DonJayamanne
changed the title
dmitrivMS
approved these changes
May 30, 2026
…ssion handling - Updated @github/copilot dependency in package.json and package-lock.json to version 1.0.54. - Modified postinstall script to include 'cli-native.node' in the file copy process. - Refactored session management to utilize updated SDK methods for setting authentication info and permissions. - Improved type definitions and imports for better clarity and maintainability. - Added tests to ensure new functionality and compatibility with updated SDK.
…n and path import
DonJayamanne
force-pushed
the
don/update-copilot-cli
branch
from
ec2113e to
56540ab
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Enhance CLI session handling and improve token management. Update the GitHub Copilot dependency to the latest version, refactor session management, and add tests for new functionalities. Ensure compatibility with the updated SDK and improve type definitions for clarity.