Skip to content

fix(vscode): remove duplicate Agent Manager empty state - #12857

Merged
marius-kilocode merged 1 commit into
mainfrom
fix-worktree-removal-duplicate-template-ui
Aug 4, 2026
Merged

fix(vscode): remove duplicate Agent Manager empty state#12857
marius-kilocode merged 1 commit into
mainfrom
fix-worktree-removal-duplicate-template-ui

Conversation

@marius-kilocode

@marius-kilocode marius-kilocode commented Aug 4, 2026

Copy link
Copy Markdown
Collaborator

When the last managed worktree is removed, selection moves to an empty local context. Agent Manager currently renders both the legacy empty-state block and the newer detail-stack empty state, producing duplicate empty-state CTAs.

This removes the obsolete top-level render so the detail stack is the sole owner of session-less selected contexts while preserving setup-terminal layout behavior.

Before and after:

Agent Manager showing duplicate No sessions open templates before the fix\n\nAgent Manager showing one No sessions open template after the fix

@kilo-code-bot

kilo-code-bot Bot commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

lgtm — the removed top-level empty state is fully covered by the detail-stack empty state: contextEmpty() can only be true when a selection exists, which guarantees showDetailStack() renders the remaining panel. All CSS classes, the noSessions i18n key, and the contextEmpty memo remain in use, and the changeset is present and user-facing.

Files Reviewed (2 files)
  • .changeset/single-agent-manager-empty-state.md
  • packages/kilo-vscode/webview-ui/agent-manager/AgentManagerApp.tsx

Reviewed by kimi-k3 · Input: 59.6K · Output: 4.9K · Cached: 218.5K

Review guidance: REVIEW.md from base branch main

@marius-kilocode
marius-kilocode merged commit 692b89c into main Aug 4, 2026
24 checks passed
@marius-kilocode
marius-kilocode deleted the fix-worktree-removal-duplicate-template-ui branch August 4, 2026 12:34
t7tran pushed a commit to t7tran/kilocode that referenced this pull request Aug 14, 2026
…duplicate-template-ui

fix(vscode): remove duplicate Agent Manager empty state
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