Commit

Author: jy-tan

internal/tui/components/content_panel.go

@@ -1,11 +1,10 @@
 package components
 
 import (
-	"os/exec"
-	"runtime"
 	"strings"
 	"time"
 
+	"github.com/atotto/clipboard"
 	"github.com/charmbracelet/bubbles/viewport"
 	tea "github.com/charmbracelet/bubbletea"
 	"github.com/charmbracelet/lipgloss"
@@ -579,6 +578,15 @@ func (cp *ContentPanel) GetViewportWidth() int {
 	return cp.viewport.Width
 }
 
+// GetRawContent returns the raw content without ANSI codes
+func (cp *ContentPanel) GetRawContent() string {
+	var lines []string
+	for _, line := range cp.contentLines {
+		lines = append(lines, stripAnsi(line))
+	}
+	return strings.Join(lines, "\n")
+}
+
 // UpdateContentLines updates content lines without clearing selection
 // Use this when content changes but you want to preserve any active selection
 func (cp *ContentPanel) UpdateContentLines(lines []string) {
@@ -614,30 +622,21 @@ func stripAnsi(s string) string {
 }
 
 // copyToClipboard copies text to system clipboard
-func (cp *ContentPanel) copyToClipboard(text string) tea.Cmd {
+func (cp *ContentPanel) copyToClipboard(text string) {
 	if text == "" {
-		return nil
-	}
-
-	var cmd *exec.Cmd
-
-	switch runtime.GOOS {
-	case "darwin":
-		cmd = exec.Command("pbcopy")
-	case "linux":
-		if _, err := exec.LookPath("xclip"); err == nil {
-			cmd = exec.Command("xclip", "-selection", "clipboard")
-		} else if _, err := exec.LookPath("xsel"); err == nil {
-			cmd = exec.Command("xsel", "--clipboard", "--input")
-		} else {
-			return nil
-		}
-	default:
-		return nil
+		return
 	}
 
-	cmd.Stdin = strings.NewReader(text)
-	_ = cmd.Run()
+	_ = clipboard.WriteAll(text)
 	cp.showCopied = true
-	return nil
+}
+
+// CopyText copies the given text to clipboard and shows the "[copied]" indicator.
+// Returns a tea.Cmd to reset the indicator after a timeout.
+func (cp *ContentPanel) CopyText(text string) tea.Cmd {
+	cp.copyToClipboard(text)
+	return tea.Tick(time.Second, func(t time.Time) tea.Msg {
+		cp.showCopied = false
+		return struct{}{}
+	})
 }

internal/tui/components/details_panel.go

@@ -71,3 +71,9 @@ func (dp *DetailsPanel) SetSize(width, height int) {
 func (dp *DetailsPanel) View() string {
 	return dp.ContentPanel.View(dp.width, dp.height)
 }
+
+// CopyAllContent copies all panel content to the clipboard
+func (dp *DetailsPanel) CopyAllContent() tea.Cmd {
+	text := dp.ContentPanel.GetRawContent()
+	return dp.ContentPanel.CopyText(text)
+}

internal/tui/components/log_panel.go

@@ -15,6 +15,11 @@ type LogPanelComponent struct {
 	testLogs      map[string][]string
 	currentTestID string
 	logMutex      sync.RWMutex
+
+	// Caching to avoid expensive rebuilds on every View() call
+	lastWrapWidth     int
+	contentNeedsSync  bool
+	pendingGotoBottom bool
 }
 
 // NewLogPanelComponent creates a new log panel
@@ -39,7 +44,22 @@ func (lp *LogPanelComponent) View(width, height int) string {
 	lp.logMutex.Lock()
 	defer lp.logMutex.Unlock()
 
-	lp.rebuildContent(false)
+	// Only rebuild content if something actually changed
+	currentWrapWidth := lp.ContentPanel.GetViewportWidth() - 2
+	if currentWrapWidth <= 0 {
+		currentWrapWidth = 70
+	}
+
+	// Check if we need to rebuild: width changed or content was marked dirty
+	widthChanged := lp.lastWrapWidth != currentWrapWidth
+	if lp.contentNeedsSync || widthChanged {
+		// Go to bottom if new content was added, but not if just width changed
+		gotoBottom := lp.pendingGotoBottom && !widthChanged
+		lp.rebuildContent(gotoBottom)
+		lp.lastWrapWidth = currentWrapWidth
+		lp.contentNeedsSync = false
+		lp.pendingGotoBottom = false
+	}
 
 	return lp.ContentPanel.View(width, height)
 }
@@ -56,7 +76,8 @@ func (lp *LogPanelComponent) AddServiceLog(line string) {
 	}
 
 	if lp.currentTestID == "" {
-		lp.rebuildContent(true)
+		lp.contentNeedsSync = true
+		lp.pendingGotoBottom = true
 	}
 }
 
@@ -76,7 +97,8 @@ func (lp *LogPanelComponent) AddTestLog(testID, line string) {
 	}
 
 	if lp.currentTestID == testID {
-		lp.rebuildContent(true)
+		lp.contentNeedsSync = true
+		lp.pendingGotoBottom = true
 	}
 }
 
@@ -106,7 +128,8 @@ func (lp *LogPanelComponent) SetCurrentTest(testID string) {
 
 	lp.currentTestID = testID
 	lp.updateTitle()
-	lp.rebuildContent(true)
+	lp.contentNeedsSync = true
+	lp.pendingGotoBottom = true
 }
 
 // SetOffset sets the panel's position on screen (for mouse coordinate translation)
@@ -170,3 +193,9 @@ func (lp *LogPanelComponent) updateTitle() {
 		lp.ContentPanel.SetTitle(title)
 	}
 }
+
+// CopyAllLogs copies all currently visible logs to the clipboard
+func (lp *LogPanelComponent) CopyAllLogs() tea.Cmd {
+	text := lp.GetRawLogs()
+	return lp.ContentPanel.CopyText(text)
+}

internal/tui/list.go

@@ -146,7 +146,7 @@ func (m *listModel) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
 			case "enter", "d", "D":
 				m.sizeWarning.Dismiss()
 				return m, nil
-			case "q", "ctrl+c", "esc":
+			case "q", "ctrl+c":
 				return m, tea.Quit
 			}
 			return m, nil
@@ -155,7 +155,7 @@ func (m *listModel) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
 		switch m.state {
 		case listView:
 			switch msg.String() {
-			case "q", "ctrl+c", "esc":
+			case "q", "ctrl+c":
 				return m, tea.Quit
 			case "enter":
 				if m.cursor >= 0 &&
@@ -254,11 +254,13 @@ func (m *listModel) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
 					}
 				}
 				return m, nil
+			case "y":
+				return m, m.detailsPanel.CopyAllContent()
 			}
 		case testExecutionView:
 			if m.testExecutor != nil && m.testExecutor.state == stateCompleted {
 				switch msg.String() {
-				case "q", "ctrl+c", "esc", "enter", " ":
+				case "q", "ctrl+c", "enter", " ":
 					// Clean up and return to list
 					m.testExecutor.cleanup()
 					log.SetTUILogger(nil)
@@ -478,7 +480,7 @@ func (m *listModel) View() string {
 		availableWidthForHelp = max(availableWidthForHelp, 20)
 
 		footer := utils.TruncateWithEllipsis(
-			testCount+"• j/k: select • u/d: scroll • g/G: top/bottom • J/K/U/D: scroll details • ←/→: expand • enter: run • q: quit",
+			testCount+"• j/k: select • u/d: scroll • g/G: top/bottom • J/K/U/D: scroll details • ←/→: expand • y: copy • enter: run • q: quit",
 			availableWidthForHelp,
 		)
 

internal/tui/test_executor.go

@@ -375,7 +375,7 @@ func (m *testExecutorModel) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
 			case "enter", "d", "D":
 				m.sizeWarning.Dismiss()
 				return m, nil
-			case "q", "ctrl+c", "esc":
+			case "q", "ctrl+c":
 				m.cleanup()
 				return m, tea.Quit
 			}
@@ -399,7 +399,6 @@ func (m *testExecutorModel) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
 				} else if selectedTest := m.testTable.GetSelectedTest(); selectedTest != nil {
 					m.logPanel.SetCurrentTest(selectedTest.TraceID)
 				}
-				return m, nil
 			case tea.MouseButtonWheelDown:
 				// Scroll viewport only, clamp cursor to visible bounds
 				m.testTable.ScrollDown(3)
@@ -409,17 +408,18 @@ func (m *testExecutorModel) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
 				} else if selectedTest := m.testTable.GetSelectedTest(); selectedTest != nil {
 					m.logPanel.SetCurrentTest(selectedTest.TraceID)
 				}
-				return m, nil
-			}
-		} else {
-			// - X: leftWidth + border(1) + padding(1) = leftWidth + 2
-			// - Y: headerHeight + title(1) + empty line(1) = headerHeight + 2
-			m.logPanel.SetOffset(leftWidth+2, headerHeight+2)
-			if cmd := m.logPanel.Update(msg); cmd != nil {
-				return m, cmd
 			}
+			// Always return for left-side mouse events to prevent fall-through
 			return m, nil
 		}
+		// Right panel - handle mouse events there
+		// - X: leftWidth + border(1) + padding(1) = leftWidth + 2
+		// - Y: headerHeight + title(1) + empty line(1) = headerHeight + 2
+		m.logPanel.SetOffset(leftWidth+2, headerHeight+2)
+		if cmd := m.logPanel.Update(msg); cmd != nil {
+			return m, cmd
+		}
+		return m, nil
 
 	case testsLoadedMsg:
 		// Inject tests into model and start execution
@@ -726,7 +726,10 @@ func (m *testExecutorModel) handleTableNavigation(msg tea.KeyMsg) (tea.Model, te
 		m.updateLogPanelFromSelection()
 		return m, nil
 
-	case "q", "ctrl+c", "esc":
+	case "y":
+		return m, m.logPanel.CopyAllLogs()
+
+	case "q", "ctrl+c":
 		m.cleanup()
 		return m, tea.Quit
 	}
@@ -745,7 +748,7 @@ func (m *testExecutorModel) updateLogPanelFromSelection() {
 
 func (m *testExecutorModel) getFooterText() string {
 	testCount := fmt.Sprintf("%d TESTS ", len(m.tests))
-	return testCount + "• j/k: select • u/d: scroll • g/G: top/bottom • J/K/U/D: scroll logs • q: quit"
+	return testCount + "• j/k: select • u/d: scroll • g/G: top/bottom • J/K/U/D: scroll logs • y: copy logs • q: quit"
 }
 
 func (m *testExecutorModel) View() string {