Skip to content

feat(CopilotCLI): support reasoning effort#308951

Merged
DonJayamanne merged 2 commits intomainfrom
don/literary-catfish
Apr 10, 2026
Merged

feat(CopilotCLI): support reasoning effort#308951
DonJayamanne merged 2 commits intomainfrom
don/literary-catfish

Conversation

@DonJayamanne
Copy link
Copy Markdown
Contributor

Fixes #304641

@DonJayamanne DonJayamanne self-assigned this Apr 10, 2026
Copilot AI review requested due to automatic review settings April 10, 2026 05:35
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

Adds support for passing “reasoning effort” (thinking effort) from VS Code chat model configuration through to Copilot CLI session creation and request execution, addressing missing reasoning-effort behavior for Copilot CLI sessions.

Changes:

  • Plumbs reasoningEffort from vscode.ChatRequest.modelConfiguration into Copilot CLI session initialization (createSession/getSession).
  • Updates request handling to pass the full { model, reasoningEffort? } object through to ICopilotCLISession.handleRequest.
  • Gates reasoning-effort handling behind chat.cli.thinkingEffort.enabled.
Show a summary per file
File Description
extensions/copilot/src/extension/chatSessions/vscode-node/copilotCLIChatSessionsContribution.ts Extracts reasoning effort from request model configuration and forwards it into Copilot CLI session lifecycle (create/get/handleRequest).

Copilot's findings

  • Files reviewed: 3/3 changed files
  • Comments generated: 1

@github-actions
Copy link
Copy Markdown
Contributor

Screenshot Changes

Base: a4f51197 Current: 1ea25af6

Changed (1)

agentSessionsViewer/ApprovalRowLongLabel/Dark
Before After
before after

@DonJayamanne DonJayamanne marked this pull request as ready for review April 10, 2026 05:44
@DonJayamanne DonJayamanne merged commit e9d8794 into main Apr 10, 2026
40 of 41 checks passed
@DonJayamanne DonJayamanne deleted the don/literary-catfish branch April 10, 2026 06:29
@vs-code-engineering vs-code-engineering bot added this to the 1.116.0 milestone Apr 10, 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.

Copilot CLI: Support Reasoning Effort

3 participants