Skip to content

Sessions: Report cloud session pull requests#327393

Merged
benibenj merged 3 commits into
mainfrom
benibenj/cloud-sessions-pr-reporting
Jul 26, 2026
Merged

Sessions: Report cloud session pull requests#327393
benibenj merged 3 commits into
mainfrom
benibenj/cloud-sessions-pr-reporting

Conversation

@benibenj

Copy link
Copy Markdown
Contributor

Summary

  • report provider-backed pull requests for Copilot Cloud sessions in the Agents window
  • infer the pull request from cloud repository and branch metadata when the provider omits it
  • share the Agent Host live PR polling, status icon, and persistent icon cache behavior

Testing

  • Not run locally per request; validation is deferred to CI
  • Pre-commit hygiene hook passed

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings July 24, 2026 21:12
@benibenj
benibenj enabled auto-merge (squash) July 24, 2026 21:12

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

Adds pull request reporting for Copilot Cloud sessions in the Agents window.

Changes:

  • Resolves PRs from provider metadata or repository branch data.
  • Shares live PR icon polling and caching behavior.
  • Adds documentation and provider tests.
Show a summary per file
File Description
copilotChatSessionsProvider.test.ts Tests cloud PR resolution and icon updates.
COPILOT_CHAT_SESSIONS_PROVIDER.md Documents cloud PR behavior.
copilotChatSessionsProvider.ts Adds cloud PR inference and live icons.
baseAgentHostSessionsProvider.ts Uses the shared icon helper.
pullRequestIconStatus.ts Centralizes live and cached PR icon resolution.

Review details

  • Files reviewed: 5/5 changed files
  • Comments generated: 1
  • Review effort level: Medium

aeschli
aeschli previously approved these changes Jul 24, 2026
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@dmitrivMS

Copy link
Copy Markdown
Contributor

@benibenj
Error: [core-ci ] [21:59:11] src/vs/sessions/contrib/providers/copilotChatSessions/browser/copilotChatSessionsProvider.ts(1066,3): error TS2322: Type 'IObservableWithChange<IGitHubInfo | { owner: string; repo: string; pullRequest: { number?: number | undefined; uri?: URI | undefined; baseRefOid?: string | undefined; headRefOid?: string | undefined; icon: ThemeIcon | undefined; }; } | undefined, unknown>' is not assignable to type 'IObservable<IGitHubInfo | undefined>'.

roblourens
roblourens previously approved these changes Jul 24, 2026
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@benibenj
benibenj merged commit 5121c1c into main Jul 26, 2026
29 checks passed
@benibenj
benibenj deleted the benibenj/cloud-sessions-pr-reporting branch July 26, 2026 18:33
@vs-code-engineering vs-code-engineering Bot added this to the 1.131.0 milestone Jul 26, 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.

6 participants