Skip to content

fix(agent-manager): clear stale child activity - #13775

Merged
marius-kilocode merged 2 commits into
mainfrom
fix-stale-worktree-status
Sep 4, 2026
Merged

fix(agent-manager): clear stale child activity#13775
marius-kilocode merged 2 commits into
mainfrom
fix-stale-worktree-status

Conversation

@marius-kilocode

Copy link
Copy Markdown
Collaborator

What Problem This Solves

Agent Manager worktrees could remain In Progress after a background child completed while another project was active.

Why This Change Was Made

Released child sessions were removed from active tracking before their terminal status reached the owning project provider. The provider now retains bounded ownership for running children and accepts only matching terminal status events.

User Impact

Completed background worktrees show their green completion badge instead of a stale running indicator. Unrelated project events remain filtered.

Evidence

  • Reproduced the issue on the installed snapshot with a real two-project background-agent flow.
  • Repeated the same flow with this fix and observed the green completion badge.
  • Full extension unit suite: 4,724 passed, 1 skipped.
  • Typecheck, lint, build, Knip, and merge-marker checks passed.

@kilo-code-bot

kilo-code-bot Bot commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (4 files)
  • .changeset/fix-stale-worktree-status.md
  • packages/kilo-vscode/src/KiloProvider.ts
  • packages/kilo-vscode/tests/unit/kilo-provider-followup.test.ts
  • packages/kilo-vscode/tests/unit/kilo-provider-load-messages.test.ts
Previous Review Summary (commit d842b97)

Current summary above is authoritative. Previous snapshots are kept for context only.

Previous review (commit d842b97)

Status: No Issues Found | Recommendation: Merge

Files Reviewed (4 files)
  • .changeset/fix-stale-worktree-status.md
  • packages/kilo-vscode/src/KiloProvider.ts
  • packages/kilo-vscode/tests/unit/kilo-provider-followup.test.ts
  • packages/kilo-vscode/tests/unit/kilo-provider-load-messages.test.ts

Reviewed by grok-4.6 · Input: 67.4K · Output: 4.3K · Cached: 183.9K

Review guidance: REVIEW.md from base branch main

@marius-kilocode
marius-kilocode merged commit dc8caf9 into main Sep 4, 2026
25 checks passed
@marius-kilocode
marius-kilocode deleted the fix-stale-worktree-status branch September 4, 2026 08:55
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