Skip to content

feat: show worktree directory in session actions menu - #324

Merged
kbwo merged 1 commit into
mainfrom
feat/show-worktree-directory
Aug 19, 2026
Merged

feat: show worktree directory in session actions menu#324
kbwo merged 1 commit into
mainfrom
feat/show-worktree-directory

Conversation

@kbwo

@kbwo kbwo commented Aug 19, 2026

Copy link
Copy Markdown
Owner

Problem

Pressing Space on a highlighted session in Menu opens the SessionActions dialog, but it only showed the session name (or session number) — not which worktree the session belonged to. When multiple worktrees have sessions with the same name, or a reviewer wants to confirm the target worktree's path before acting, there was no way to tell.

Verification

Prerequisites:

  • None (bun run test can be run from the repo root)

Steps:

  • bun run typecheck — confirmed it exits with no errors
  • bun run lint — confirmed it exits with no errors
  • bun run test — confirmed all tests pass (1816 passed / 10 skipped)
  • Launch ccmanager, highlight a session row in Menu, press Space, and visually confirm the SessionActions dialog shows a Directory: <worktree absolute path> line below the session name

Space in Menu opens the session actions dialog (SessionActions.tsx) for
the highlighted worktree's session, but it only showed the session name,
not which worktree it belonged to.
@kbwo
kbwo marked this pull request as ready for review August 19, 2026 13:32
@kbwo
kbwo merged commit 8ad94ff into main Aug 19, 2026
1 check passed
kbwo added a commit that referenced this pull request Aug 30, 2026
Conflicts came from #324, which added a "Directory: <path>" line to the
session actions menu and a required worktreePath prop for it.

- SessionActions keeps main's worktreePath prop and dedicated directory
  line, alongside the hasSession / canDeleteWorktree props of this branch.
  sessionLabel becomes optional, since a worktree row without a session has
  no session name to show and the path already has its own line.
- App passes both: the label is main's session name / "Session #N" for a
  session row and undefined for a session-less one.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
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.

1 participant