Skip to content

feat(agent-manager): show worktree name on hover card - #12634

Merged
marius-kilocode merged 2 commits into
mainfrom
include-worktree-name-agent-manager-session-card
Jul 29, 2026
Merged

feat(agent-manager): show worktree name on hover card#12634
marius-kilocode merged 2 commits into
mainfrom
include-worktree-name-agent-manager-session-card

Conversation

@marius-kilocode

Copy link
Copy Markdown
Collaborator

The Agent Manager worktree hover card showed the branch, base, session count, and git stats, but not the worktree's own directory name. When the display label is a custom name or a session title, and the branch is a long auto-generated name, there is no way to see the actual directory the worktree lives in (e.g. decorous-taker under .kilo/worktrees/) without leaving the UI and checking the filesystem.

This adds a Worktree row to the hover card showing the directory basename computed from worktree.path, placed between the branch header and the Base row. Basename extraction handles POSIX and Windows separators and trailing slashes. The new agentManager.hoverCard.worktree i18n key is added to all 20 Agent Manager locale files (value kept as "Worktree", matching how existing rows treat the term), which the i18n split test requires for keyset alignment.

Agent Manager worktree hover card showing the new Worktree row with the directory name

Close-up of the hover card: Worktree row between branch header and Base row

@marius-kilocode
marius-kilocode enabled auto-merge (squash) July 29, 2026 08:43
@kilo-code-bot

kilo-code-bot Bot commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (21 files)
  • packages/kilo-vscode/webview-ui/agent-manager/WorktreeItem.tsx
  • .changeset/worktree-hover-card-name.md
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/ar.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/br.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/bs.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/da.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/de.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/en.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/es.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/fr.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/it.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/ja.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/ko.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/nl.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/no.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/pl.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/ru.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/th.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/tr.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/uk.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/zh.ts
  • packages/kilo-vscode/webview-ui/agent-manager/i18n/zht.ts

Notes: the basename logic correctly handles POSIX/Windows separators and trailing slashes, worktree.path is a required non-optional field so no null-guard is needed, the new hover-card row reuses existing am-hover-card-row/am-hover-card-divider CSS classes with no new styles added, the i18n key is present in all 20 locale files (matching the split-keyset test requirement), and a concise user-facing changeset is included. This PR is confined to packages/kilo-vscode/, so no fork-merge-hygiene concerns apply.


Reviewed by claude-sonnet-5 · Input: 50 · Output: 7.6K · Cached: 1.6M

Review guidance: REVIEW.md from base branch main

@marius-kilocode
marius-kilocode merged commit 4c5c242 into main Jul 29, 2026
23 checks passed
@marius-kilocode
marius-kilocode deleted the include-worktree-name-agent-manager-session-card branch July 29, 2026 09:26
t7tran pushed a commit to t7tran/kilocode that referenced this pull request Aug 14, 2026
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