Skip to content

Sessions - increase polling interval for pull request data#307608

Merged
lszomoru merged 1 commit intomainfrom
lszomoru/afraid-yak
Apr 3, 2026
Merged

Sessions - increase polling interval for pull request data#307608
lszomoru merged 1 commit intomainfrom
lszomoru/afraid-yak

Conversation

@lszomoru
Copy link
Copy Markdown
Member

@lszomoru lszomoru commented Apr 3, 2026

No description provided.

Copilot AI review requested due to automatic review settings April 3, 2026 09:20
@lszomoru lszomoru enabled auto-merge (squash) April 3, 2026 09:20
@lszomoru lszomoru self-assigned this Apr 3, 2026
@lszomoru lszomoru added this to the 1.115.0 milestone Apr 3, 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

Adjusts GitHub Sessions polling behavior to reduce network/API churn and avoid unnecessary recomputation in the Sessions “Checks” experience.

Changes:

  • Increased the default polling interval for GitHub pull request model refreshes from 30s to 60s.
  • Updated the checks view model to derive a structurally-equal PR identity object (owner/repo/headRef) to stabilize downstream CI model creation/refresh behavior.

Reviewed changes

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

File Description
src/vs/sessions/contrib/github/browser/models/githubPullRequestModel.ts Doubles the default PR polling interval to reduce fetch frequency.
src/vs/sessions/contrib/changes/browser/checksViewModel.ts Introduces a structurally-compared derived PR identity observable to prevent redundant CI model churn while still reacting to meaningful PR changes (e.g., new head SHA).

@lszomoru lszomoru merged commit a013d5c into main Apr 3, 2026
33 of 34 checks passed
@lszomoru lszomoru deleted the lszomoru/afraid-yak branch April 3, 2026 12:42
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