Skip to content

Conversation

@DonJayamanne
Copy link
Collaborator

@DonJayamanne DonJayamanne self-assigned this Dec 4, 2025
@DonJayamanne DonJayamanne marked this pull request as ready for review December 4, 2025 19:56
Copilot AI review requested due to automatic review settings December 4, 2025 19:56
@vs-code-engineering vs-code-engineering bot added this to the November 2025 milestone Dec 4, 2025
Copilot finished reviewing on behalf of DonJayamanne December 4, 2025 19:58
Copy link
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 extends the chat session options API to support rich option values (not just strings) for background worktree sessions. It enables displaying locked worktree branch names for isolated sessions with visual indicators.

Key changes:

  • Updated the ChatSessionOptionChangeEvent type to allow ChatSessionProviderOptionItem objects as option values
  • Added logic to trigger option change events when a session with an active worktree is opened, displaying the worktree branch name as a locked option
  • Enhanced agent option items to include descriptions

Reviewed changes

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

File Description
src/extension/vscode.proposed.chatSessionsProvider.d.ts Extended ChatSessionOptionChangeEvent.value type to support ChatSessionProviderOptionItem objects in addition to strings
src/extension/chatSessions/vscode-node/copilotCLIChatSessionsContribution.ts Updated method signature to match new type, added worktree option change notifications for sessions with active worktrees, added description to agent items, and included commented code for model tracking

@DonJayamanne DonJayamanne added this pull request to the merge queue Dec 4, 2025
Merged via the queue into main with commit c88a966 Dec 4, 2025
16 checks passed
@DonJayamanne DonJayamanne deleted the don/energetic-galliform branch December 4, 2025 20:46
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.

Worktree info is not updated immediately in new background agent session

3 participants