Skip to content

Revert "Fix UBB model details in Copilot CLI (#316452)"#316846

Closed
roblourens wants to merge 1 commit into
mainfrom
roblou/revert-a1803b2
Closed

Revert "Fix UBB model details in Copilot CLI (#316452)"#316846
roblourens wants to merge 1 commit into
mainfrom
roblou/revert-a1803b2

Conversation

@roblourens
Copy link
Copy Markdown
Member

Reverts commit a1803b2 (PR #316452).

(Written by Copilot)

Copilot AI review requested due to automatic review settings May 17, 2026 03:43
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

This PR reverts the Copilot CLI SDK update from PR #316452, restoring the previous SDK version and removing the associated model pricing/detail handling added for that upgrade.

Changes:

  • Downgrades @github/copilot from 1.0.48 to 1.0.39.
  • Removes Copilot CLI model token-cost and price-category propagation.
  • Restores the SDK native-binary allowlist expectations for the older SDK contents.
Show a summary per file
File Description
src/vs/workbench/contrib/chat/browser/widget/input/chatModelPicker.ts Restores unknown price-category handling to omit labels.
extensions/copilot/src/extension/chatSessions/copilotcli/vscode-node/test/copilotCLISDKUpgrade.spec.ts Reverts native-binary expectations for the older Copilot SDK package.
extensions/copilot/src/extension/chatSessions/copilotcli/node/copilotCli.ts Removes CLI model pricing fields and normalization logic.
extensions/copilot/package.json Downgrades the Copilot SDK dependency.
extensions/copilot/package-lock.json Updates locked Copilot SDK package versions and integrity hashes.

Copilot's findings

Files not reviewed (1)
  • extensions/copilot/package-lock.json: Language not supported
  • Files reviewed: 4/5 changed files
  • Comments generated: 0

@roblourens roblourens closed this May 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants