Skip to content

Agents - create session from pull request should not leverage the IGitService - #331644

Merged
Ladislau Szomoru (lszomoru) merged 2 commits into
mainfrom
lszomoru/wilful-bonobo
Aug 19, 2026
Merged

Agents - create session from pull request should not leverage the IGitService#331644
Ladislau Szomoru (lszomoru) merged 2 commits into
mainfrom
lszomoru/wilful-bonobo

Conversation

@lszomoru

Copy link
Copy Markdown
Member

No description provided.

Copilot AI balanced review requested due to automatic review settings August 19, 2026 14:33

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Removes the pull-request session action’s dependency on IGitService, relying solely on hydrated session metadata.

Changes:

  • Removes Git remote parsing and repository fallback logic.
  • Renames the action and command.
  • Updates repository-resolution tests.
Show a summary per file
File Description
createSessionFromPullRequestAction.ts Removes IGitService usage and renames the action.
pullRequestPicker.ts Resolves repositories exclusively from session metadata.
pullRequestPicker.test.ts Updates resolution tests and removes remote parsing tests.

Review details

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

  • Files reviewed: 3/3 changed files
  • Comments generated: 4
  • Review effort level: Balanced

Comment thread src/vs/sessions/contrib/github/browser/pullRequestPicker.ts
@lszomoru
Ladislau Szomoru (lszomoru) merged commit 0927290 into main Aug 19, 2026
27 checks passed
@lszomoru
Ladislau Szomoru (lszomoru) deleted the lszomoru/wilful-bonobo branch August 19, 2026 18:50
@vs-code-engineering vs-code-engineering Bot added this to the 1.135.0 milestone Aug 19, 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.

4 participants