Skip to content

CCR feedback from https://github.com/microsoft/vscode-pull-request-github/pull/8727#8742

Merged
alexr00 merged 1 commit into
mainfrom
alexr00/vivacious-jay
May 12, 2026
Merged

CCR feedback from https://github.com/microsoft/vscode-pull-request-github/pull/8727#8742
alexr00 merged 1 commit into
mainfrom
alexr00/vivacious-jay

Conversation

@alexr00
Copy link
Copy Markdown
Member

@alexr00 alexr00 commented May 12, 2026

No description provided.

Copilot AI review requested due to automatic review settings May 12, 2026 10:10
@alexr00 alexr00 enabled auto-merge (squash) May 12, 2026 10:10
@alexr00 alexr00 self-assigned this May 12, 2026
@alexr00 alexr00 disabled auto-merge May 12, 2026 10:12
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 ReviewManager.validateState to reduce repeated GitHub lookups while still allowing local branch↔PR metadata to “self-heal” when it’s stale (e.g., pointing at a closed PR).

Changes:

  • Add _staleMetadataCheckedBranches to track per-branch one-shot GitHub re-checks when local PR metadata exists.
  • Update validateState logic to trigger GitHub checks on branch change, new PRs, or a one-time stale-metadata re-check.
Show a summary per file
File Description
src/view/reviewManager.ts Adds a per-branch “stale metadata checked” cache and updates validation logic to avoid repeated GitHub queries while enabling one-shot self-healing.

Copilot's findings

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

Comment thread src/view/reviewManager.ts
@alexr00 alexr00 enabled auto-merge (squash) May 12, 2026 13:05
@alexr00 alexr00 merged commit 7cdbb9c into main May 12, 2026
10 checks passed
@alexr00 alexr00 deleted the alexr00/vivacious-jay branch May 12, 2026 14:05
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