Skip to content

Sessions - update order for setting the default value for the branch picker#307053

Merged
lszomoru merged 1 commit intomainfrom
lszomoru/external-partridge
Apr 1, 2026
Merged

Sessions - update order for setting the default value for the branch picker#307053
lszomoru merged 1 commit intomainfrom
lszomoru/external-partridge

Conversation

@lszomoru
Copy link
Copy Markdown
Member

@lszomoru lszomoru commented Apr 1, 2026

No description provided.

Copilot AI review requested due to automatic review settings April 1, 2026 07:49
@lszomoru lszomoru enabled auto-merge (squash) April 1, 2026 07:49
@lszomoru lszomoru self-assigned this Apr 1, 2026
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

Updates the branch defaulting logic used by the Copilot CLI session branch picker, to prefer common default branches (main/master) ahead of the current HEAD branch.

Changes:

  • Reorders the defaultBranch selection logic to try main, then master, then HEAD, then fallback.
  • Removes all contents (including the required copyright header) from src/vs/sessions/contrib/chat/browser/branchPicker.ts, leaving it effectively empty.

Reviewed changes

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

File Description
src/vs/sessions/contrib/copilotChatSessions/browser/copilotChatSessionsProvider.ts Changes default-branch selection precedence used when initializing branch state.
src/vs/sessions/contrib/chat/browser/branchPicker.ts File content/header removed, leaving an empty file (needs delete or restore).
Comments suppressed due to low confidence (1)

src/vs/sessions/contrib/chat/browser/branchPicker.ts:1

  • This file now appears to be empty (only a blank line remains) and the standard Microsoft copyright header was removed. If the file is no longer needed, please delete it from the repo; otherwise restore the header and the intended implementation to avoid leaving a dead/placeholder file behind.

@lszomoru lszomoru merged commit accaa4d into main Apr 1, 2026
22 checks passed
@lszomoru lszomoru deleted the lszomoru/external-partridge branch April 1, 2026 08:16
@vs-code-engineering vs-code-engineering bot added this to the 1.115.0 milestone Apr 1, 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.

3 participants