Skip to content

Agents - fix isolation/branch picker regression#309153

Merged
lszomoru merged 1 commit intomainfrom
lszomoru/instant-sailfish
Apr 11, 2026
Merged

Agents - fix isolation/branch picker regression#309153
lszomoru merged 1 commit intomainfrom
lszomoru/instant-sailfish

Conversation

@lszomoru
Copy link
Copy Markdown
Member

No description provided.

@lszomoru lszomoru enabled auto-merge (squash) April 11, 2026 08:22
@lszomoru lszomoru self-assigned this Apr 11, 2026
Copilot AI review requested due to automatic review settings April 11, 2026 08:22
@github-actions
Copy link
Copy Markdown
Contributor

Screenshot Changes

Base: 63f7113c Current: 7b8c702a

Changed (1)

editor/inlineChatAffordance/InlineChatOverlay/Light
Before After
before after

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

Fixes a regression affecting Agents’ isolation/branch selection by ensuring Git repositories returned from the main-thread Git bridge have an initialized HEAD state before consumers use them.

Changes:

  • Add waitForState usage to delay openRepository resolution until repository.state.HEAD is available.
  • Import waitForState from the observable utilities.
Show a summary per file
File Description
src/vs/workbench/api/browser/mainThreadGitExtensionService.ts Waits for HEAD to be present on the observable repository state before returning the repository from openRepository.

Copilot's findings

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

Comment thread src/vs/workbench/api/browser/mainThreadGitExtensionService.ts
@lszomoru lszomoru merged commit 0284705 into main Apr 11, 2026
44 of 45 checks passed
@lszomoru lszomoru deleted the lszomoru/instant-sailfish branch April 11, 2026 10:00
@vs-code-engineering vs-code-engineering Bot added this to the 1.116.0 milestone Apr 11, 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