refactor: Remove Thinking status from worktree kanban

- Remove StatusThinking from status enum
- Reorder kanban columns: Active, Waiting, Done, Paused
- Change Waiting icon from 💬 to ⧗
- Update detection logic to prioritize Waiting status
- Update all tests for new layout

Author: marcus

internal/plugins/worktree/agent.go

@@ -478,7 +478,7 @@ func detectStatus(output string) WorktreeStatus {
 	text := strings.Join(checkLines, "\n")
 	textLower := strings.ToLower(text)
 
-	// Waiting patterns (agent needs user input) - check first as these are definitive
+	// Waiting patterns (agent needs user input) - highest priority
 	waitingPatterns := []string{
 		"[y/n]",       // Claude Code permission prompt
 		"(y/n)",       // Aider style
@@ -531,22 +531,6 @@ func detectStatus(output string) WorktreeStatus {
 		}
 	}
 
-	// Thinking patterns (agent actively reasoning)
-	// Check after waiting/done/error since those are more definitive signals
-	thinkingPatterns := []string{
-		"<thinking>",           // Claude extended thinking block
-		"</thinking>",          // Claude extended thinking end (still processing)
-		"<internal_monologue>", // Alternative thinking format
-		"thinking...",          // Generic thinking indicator
-		"reasoning about",      // Aider-style reasoning
-	}
-
-	for _, pattern := range thinkingPatterns {
-		if strings.Contains(textLower, pattern) {
-			return StatusThinking
-		}
-	}
-
 	// Default: active if we have output
 	return StatusActive
 }

internal/plugins/worktree/kanban.go

@@ -1,17 +1,16 @@
 package worktree
 
 // kanbanColumnOrder defines the order of columns in kanban view.
-var kanbanColumnOrder = []WorktreeStatus{StatusActive, StatusThinking, StatusWaiting, StatusDone, StatusPaused}
+var kanbanColumnOrder = []WorktreeStatus{StatusActive, StatusWaiting, StatusDone, StatusPaused}
 
 // getKanbanColumns returns worktrees grouped by status for kanban view.
 // StatusError worktrees are grouped with StatusPaused since they require user intervention.
 func (p *Plugin) getKanbanColumns() map[WorktreeStatus][]*Worktree {
 	columns := map[WorktreeStatus][]*Worktree{
-		StatusActive:   {},
-		StatusThinking: {},
-		StatusWaiting:  {},
-		StatusDone:     {},
-		StatusPaused:   {},
+		StatusActive:  {},
+		StatusWaiting: {},
+		StatusDone:    {},
+		StatusPaused:  {},
 	}
 	for _, wt := range p.worktrees {
 		status := wt.Status

internal/plugins/worktree/kanban_test.go

@@ -60,8 +60,8 @@ func TestSyncListToKanban(t *testing.T) {
 
 	p.syncListToKanban()
 
-	if p.kanbanCol != 3 { // Done column (after Active, Thinking, Waiting)
-		t.Errorf("expected kanbanCol=3 (Done), got %d", p.kanbanCol)
+	if p.kanbanCol != 2 { // Done column
+		t.Errorf("expected kanbanCol=2 (Done), got %d", p.kanbanCol)
 	}
 	if p.kanbanRow != 0 { // First item in Done column
 		t.Errorf("expected kanbanRow=0, got %d", p.kanbanRow)
@@ -79,8 +79,8 @@ func TestSyncListToKanbanWithErrorStatus(t *testing.T) {
 
 	p.syncListToKanban()
 
-	if p.kanbanCol != 4 { // Paused column (index 4, after Active, Thinking, Waiting, Done)
-		t.Errorf("expected kanbanCol=4 (Paused), got %d", p.kanbanCol)
+	if p.kanbanCol != 3 { // Paused column (index 3)
+		t.Errorf("expected kanbanCol=3 (Paused), got %d", p.kanbanCol)
 	}
 	if p.kanbanRow != 0 {
 		t.Errorf("expected kanbanRow=0, got %d", p.kanbanRow)
@@ -113,7 +113,7 @@ func TestSyncKanbanToList(t *testing.T) {
 			{Name: "wt3", Status: StatusDone},
 			{Name: "wt4", Status: StatusPaused},
 		},
-		kanbanCol:   2, // Waiting column (after Active, Thinking)
+		kanbanCol:   1, // Waiting column
 		kanbanRow:   0, // First item (wt2)
 		selectedIdx: 0,
 	}
@@ -130,7 +130,7 @@ func TestSyncKanbanToListEmptyColumn(t *testing.T) {
 		worktrees: []*Worktree{
 			{Name: "wt1", Status: StatusActive},
 		},
-		kanbanCol:   2, // Waiting column (empty, after Active, Thinking)
+		kanbanCol:   1, // Waiting column (empty)
 		kanbanRow:   0,
 		selectedIdx: 0,
 	}
@@ -244,7 +244,7 @@ func TestSelectedKanbanWorktree(t *testing.T) {
 			{Name: "wt1", Status: StatusActive},
 			{Name: "wt2", Status: StatusWaiting},
 		},
-		kanbanCol: 2, // Waiting column (after Active, Thinking)
+		kanbanCol: 1, // Waiting column
 		kanbanRow: 0,
 	}
 
@@ -262,7 +262,7 @@ func TestSelectedKanbanWorktreeEmptyColumn(t *testing.T) {
 		worktrees: []*Worktree{
 			{Name: "wt1", Status: StatusActive},
 		},
-		kanbanCol: 2, // Waiting column (empty, after Active, Thinking)
+		kanbanCol: 1, // Waiting column (empty)
 		kanbanRow: 0,
 	}
 

internal/plugins/worktree/types.go

@@ -51,12 +51,11 @@ const (
 type WorktreeStatus int
 
 const (
-	StatusPaused   WorktreeStatus = iota // No agent, worktree exists
-	StatusActive                         // Agent running, recent output
-	StatusThinking                       // Agent actively thinking/processing (extended reasoning)
-	StatusWaiting                        // Agent waiting for input
-	StatusDone                           // Agent completed task
-	StatusError                          // Agent crashed or errored
+	StatusPaused  WorktreeStatus = iota // No agent, worktree exists
+	StatusActive                        // Agent running, recent output
+	StatusWaiting                       // Agent waiting for input
+	StatusDone                          // Agent completed task
+	StatusError                         // Agent crashed or errored
 )
 
 // String returns the display string for a WorktreeStatus.
@@ -66,8 +65,6 @@ func (s WorktreeStatus) String() string {
 		return "paused"
 	case StatusActive:
 		return "active"
-	case StatusThinking:
-		return "thinking"
 	case StatusWaiting:
 		return "waiting"
 	case StatusDone:
@@ -86,10 +83,8 @@ func (s WorktreeStatus) Icon() string {
 		return "⏸"
 	case StatusActive:
 		return "●"
-	case StatusThinking:
-		return "◐"
 	case StatusWaiting:
-		return "💬"
+		return "⧗"
 	case StatusDone:
 		return "✓"
 	case StatusError:

internal/plugins/worktree/types_test.go

@@ -122,8 +122,7 @@ func TestWorktreeStatusIcon(t *testing.T) {
 	}{
 		{StatusPaused, "⏸"},
 		{StatusActive, "●"},
-		{StatusThinking, "◐"},
-		{StatusWaiting, "💬"},
+		{StatusWaiting, "⧗"},
 		{StatusDone, "✓"},
 		{StatusError, "✗"},
 	}
@@ -144,7 +143,6 @@ func TestWorktreeStatusString(t *testing.T) {
 	}{
 		{StatusPaused, "paused"},
 		{StatusActive, "active"},
-		{StatusThinking, "thinking"},
 		{StatusWaiting, "waiting"},
 		{StatusDone, "done"},
 		{StatusError, "error"},

internal/plugins/worktree/view_kanban.go

@@ -46,18 +46,16 @@ func (p *Plugin) renderKanbanView(width, height int) string {
 
 	// Column headers and colors
 	columnTitles := map[WorktreeStatus]string{
-		StatusActive:   "● Active",
-		StatusThinking: "◐ Thinking",
-		StatusWaiting:  "💬 Waiting",
-		StatusDone:     "✓ Ready",
-		StatusPaused:   "⏸ Paused",
+		StatusActive:  "● Active",
+		StatusWaiting: "⧗ Waiting",
+		StatusDone:    "✓ Ready",
+		StatusPaused:  "⏸ Paused",
 	}
 	columnColors := map[WorktreeStatus]lipgloss.Color{
-		StatusActive:   styles.StatusCompleted.GetForeground().(lipgloss.Color), // Green
-		StatusThinking: lipgloss.Color("183"),                                   // Light purple (processing)
-		StatusWaiting:  styles.StatusModified.GetForeground().(lipgloss.Color),  // Yellow
-		StatusDone:     lipgloss.Color("81"),                                    // Cyan
-		StatusPaused:   lipgloss.Color("245"),                                   // Gray
+		StatusActive:  styles.StatusCompleted.GetForeground().(lipgloss.Color), // Green
+		StatusWaiting: styles.StatusModified.GetForeground().(lipgloss.Color),  // Yellow
+		StatusDone:    lipgloss.Color("81"),                                    // Cyan
+		StatusPaused:  lipgloss.Color("245"),                                   // Gray
 	}
 
 	// Calculate column widths (account for panel borders)