Skip to content

feat(agent-manager): periodic full sync of PR status for all worktrees#8655

Merged
marius-kilocode merged 1 commit into
mainfrom
aloud-honesty
Apr 9, 2026
Merged

feat(agent-manager): periodic full sync of PR status for all worktrees#8655
marius-kilocode merged 1 commit into
mainfrom
aloud-honesty

Conversation

@marius-kilocode

Copy link
Copy Markdown
Collaborator

Summary

  • Add a 2-minute background sync interval that polls PR status for all worktrees, not just the active one. Previously inactive worktree badges went stale until selected.
  • Full syncs use bounded concurrency (3 parallel gh calls) to cap process burst across all platforms.
  • Active worktree still gets the fast 15s poll cycle unchanged.

How it works

Tick type Frequency Targets Comments fetched
Normal tick Every 15s Active worktree only Yes (active only)
Full sync tick Every 2 min All worktrees Only for active
Initial tick First poll All worktrees Only for active
Visibility resume On panel show All worktrees Only for active

The settled() helper runs thunks with bounded concurrency so a full sync of 8+ worktrees doesn't spike 16-24 simultaneous gh/git processes.

Previously PR badges only updated for the active worktree (every 15s).
Inactive worktrees went stale until selected. Add a 2-minute full sync
interval so CI results and review status stay current across all sessions.

Full syncs use bounded concurrency (3) to cap process burst on all
platforms.
@marius-kilocode marius-kilocode enabled auto-merge (squash) April 9, 2026 10:00
Comment thread packages/kilo-vscode/src/agent-manager/PRStatusPoller.ts
@kilo-code-bot

kilo-code-bot Bot commented Apr 9, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: 1 Issues Found | Recommendation: Address before merge

Overview

Severity Count
CRITICAL 0
WARNING 1
SUGGESTION 0

Fix these issues in Kilo Cloud

Issue Details (click to expand)

WARNING

File Line Issue
packages/kilo-vscode/src/agent-manager/PRStatusPoller.ts 185 Full-sync still allows more than 3 concurrent gh/git subprocesses because each limited fetchOne() may spawn multiple child commands internally.
Other Observations (not in diff)

Issues found in unchanged code that cannot receive inline comments:

File Line Issue
Files Reviewed (1 files)
  • packages/kilo-vscode/src/agent-manager/PRStatusPoller.ts - 1 issue

Reviewed by gpt-5.4-20260305 · 90,256 tokens

@marius-kilocode marius-kilocode merged commit 80f8271 into main Apr 9, 2026
17 checks passed
@marius-kilocode marius-kilocode deleted the aloud-honesty branch April 9, 2026 10:21
jliounis pushed a commit to jliounis/kilocode that referenced this pull request May 18, 2026
Kilo-Org#8655)

Previously PR badges only updated for the active worktree (every 15s).
Inactive worktrees went stale until selected. Add a 2-minute full sync
interval so CI results and review status stay current across all sessions.

Full syncs use bounded concurrency (3) to cap process burst on all
platforms.
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.

2 participants