Skip to content

Fix TUI duplicate bead display and planning keybind#264

Merged
newhook merged 1 commit intonewhook:mainfrom
joeybaer:fix/tui-duplicate-bead-and-planning
Feb 3, 2026
Merged

Fix TUI duplicate bead display and planning keybind#264
newhook merged 1 commit intonewhook:mainfrom
joeybaer:fix/tui-duplicate-bead-and-planning

Conversation

@joeybaer
Copy link
Collaborator

@joeybaer joeybaer commented Feb 2, 2026

Summary

Fixes three TUI issues reported by a user:

  1. Duplicate bead display: Root issue was appearing twice in work details panel (once as root issue with ◆, once as unassigned bead with ○)
  2. Unfriendly worktree error: Pressing 'r' before worktree was ready showed cryptic "no worktree path configured" error
  3. 'p' key not starting planning: The UI showed [p] plan for unassigned beads but 'p' was mapped to create PR

Changes

  • internal/progress/fetch.go: Skip root issue when populating UnassignedBeads
  • internal/tui/tui_plan_work.go: Add friendly error when worktree still being created
  • internal/tui/tui_panel_work_details.go: Add WorkDetailActionPlan and context-aware 'p' handling
  • internal/tui/tui_panel_work_overview.go: Add methods to detect unassigned bead selection
  • internal/tui/tui_plan.go: Handle WorkDetailActionPlan to spawn planning session

Test plan

  • Create work from single bead, verify no duplicate in work details panel

  • Press 'r' immediately after work creation, verify friendly error message

  • Add unassigned bead to work, select it, press 'p', verify planning session starts

Notes

The 'p' key behavior may need further review - currently it is context-sensitive (plan for unassigned beads, PR otherwise). The status bar shows [p]r for work-level actions but the unassigned bead panel shows [p] plan. A future PR may want to unify this or use different keys.

Screenshots

Before

Before

After

After

- Fix root issue appearing twice in work details (as root and unassigned)
- Add friendly error when worktree is still being created
- Make 'p' key context-aware: plan for unassigned beads, PR otherwise
@newhook newhook merged commit 6a5216e into newhook:main Feb 3, 2026
3 checks passed
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