Add a setting to fully disable background PR state polling (causes editor to jump/scroll on focus regain)
Maintainer antworten meist innerhalb von 1 Tag
@alexr00 arbeitet bereits daran.
Seit 20.10.2026.
Bewertung
Dieses Issue wurde noch nicht bewertet.
Beschreibung
Summary
ReviewManager polls for PR state changes on a hardcoded schedule with no way to disable it:
// src/view/reviewManager.ts
private static readonly MIN_POLL_INTERVAL_MS = 1000 * 60 * 5; // 5 min
private static readonly MAX_POLL_INTERVAL_MS = 1000 * 60 * 30; // 30 min
pollForStateChange() is started unconditionally in the constructor and re-armed on every onDidChangeWindowState focus event (with 15-60s jitter). There is no vscode.workspace.getConfiguration(...) check anywhere in this path - polling cannot be turned off short of patching the extension.
Impact
In a workspace with many git repositories (e.g. a monorepo with dozens of submodules - each gets its own FolderRepositoryManager/ReviewManager), regaining window focus causes every repo's poller to fire in a burst. For the repo with the actively checked-out PR, if hasNewPullRequests() reports activity, updateState() re-resolves the PR and re-applies CommentThreads on any currently open file with review comments. Recreating those comment threads forces VS Code to re-layout the editor, which visibly scrolls/jumps the cursor - repeatedly, every time focus returns and backoff hasn't kicked in yet.
This is a recurrence of the same user-visible symptom reported in #4159 (duplicate of #4031, fixed in 0.54.0 for a different repro involving GitLens) - but the underlying mechanism (periodic background refresh disturbing editor state) has no off switch today.
Request
Add a setting (e.g. githubPullRequests.disableBackgroundPolling or similar) to disable the automatic poll entirely, relying on the existing manual refresh command instead. Note that manual refresh already goes through a separate updateState() call path, not through doPoll(), so this wouldn't need to touch that.
Environment
- VS Code: 1.141.1
- Extension: GitHub Pull Requests and Issues
- Workspace: multi-repo (submodules), Remote
- Vorherrschende Sprache
- TypeScript
- Sterne
- 2.6k
- Forks
- 804
- Ø Merge
- 1 T. 23 Std.
- Gemergte PRs (30 T.)
- 44
Entwicklungsumgebung
Dieses Projekt bietet weder Dev-Container noch Dockerfile noch Beitragsleitfaden – die Einrichtung liegt bei Ihnen. Beginnen Sie mit der README; die allgemeinen Schritte stehen in unserem Leitfaden für den ersten Beitrag.
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus microsoft/vscode-pull-request-github
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
microsoft/vscode-pull-request-github#9072 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Issue 3 — DocumentationOffen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
microsoft/vscode-pull-request-github#8792 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Separate link openers for issues and pull requestsEvtl. vergeben @hediet hat das vor 1 Tag übernommen. Offenfeature-request
microsoft/vscode-pull-request-github#9082 · 1 zugewiesene Person ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 68/100
microsoft/vscode-pull-request-github#9076 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 55/100
microsoft/vscode-pull-request-github#9075 ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in microsoft/vscode-pull-request-github
Ähnliche Issues
-
createAI: the ai peer range allows ai@6, which the provider cannot work withEvtl. vergeben @akhil9tiet hat das heute übernommen. Offenbug
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 83/100
cloudflare/agents#2690 · 3 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
-
[fullsend] E2E: theme — global-header ignores app.branding.fullLogo (RHDHBUGS-3670)Evtl. vergeben @fullsend-ai-coder hat das heute übernommen. Offene2e-failure ready-to-code
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 78/100
redhat-developer/rhdh-plugin-export-overlays#4503 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
needs triage 📥
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 86/100
ethereum/ethereum-org-website#19601 ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 76/100
cloudscape-design/components#5184 ·
Maintainer antworten meist innerhalb von 2 Tagen
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
agilepathway/label-checker#805 ·
Maintainer antworten meist innerhalb von 1 Tag