Add tabbed output pane to dashboard and fix CI integration test

Replace the expand/collapse host output model with a persistent tab bar
that provides a diff tab plus per-host output tabs. Add keyboard shortcuts
for tab navigation ([/], 1-9, Enter, Esc). Make status bar hints
responsive to terminal width. Gate live SSH integration test behind a
build tag so CI passes without real hosts.

Author: agent462

internal/ui/dashboard/model.go

@@ -222,16 +222,20 @@ func (m Model) handleHostTableKey(msg tea.KeyPressMsg) (tea.Model, tea.Cmd) {
 
 	switch {
 	case key.Code == tea.KeyEnter:
-		// Expand selected host in output pane.
+		// Switch to the selected host's tab in the output pane.
 		host := m.hostTable.SelectedHost()
 		if host != "" && m.lastGrouped != nil {
-			m.outputPane.ExpandHost(host, m.lastGrouped, m.lastResults)
+			if m.outputPane.ActivateHostTab(host) {
+				// Only switch focus when the host tab actually exists.
+				m.hostTable.Blur()
+				m.focused = paneOutput
+			}
 		}
 		return m, nil
 
 	case key.Code == tea.KeyEscape:
 		if m.outputPane.IsExpanded() {
-			m.outputPane.CollapseHost(m.lastGrouped, m.lastResults)
+			m.outputPane.ActivateDiffTab()
 			return m, nil
 		}
 
@@ -258,11 +262,25 @@ func (m Model) handleOutputKey(msg tea.KeyPressMsg) (tea.Model, tea.Cmd) {
 
 	if key.Code == tea.KeyEscape {
 		if m.outputPane.IsExpanded() {
-			m.outputPane.CollapseHost(m.lastGrouped, m.lastResults)
+			m.outputPane.ActivateDiffTab()
 			return m, nil
 		}
 	}
 
+	// Tab switching with [ and ].
+	switch msg.String() {
+	case "[":
+		m.outputPane.PrevTab()
+		return m, nil
+	case "]":
+		m.outputPane.NextTab()
+		return m, nil
+	case "1", "2", "3", "4", "5", "6", "7", "8", "9":
+		idx := int(msg.String()[0]-'0') - 1 // 1-based to 0-based
+		m.outputPane.SetTabIndex(idx)
+		return m, nil
+	}
+
 	// Forward to viewport for scrolling.
 	cmd := m.outputPane.Update(msg)
 	return m, cmd

internal/ui/dashboard/model_integration_test.go

@@ -1,3 +1,5 @@
+//go:build integration
+
 package dashboard
 
 import (
@@ -14,10 +16,6 @@ import (
 // TestDashboardWithLiveHosts exercises the dashboard model against real SSH hosts.
 // It verifies health checks, command execution, and message routing work end-to-end.
 func TestDashboardWithLiveHosts(t *testing.T) {
-	if testing.Short() {
-		t.Skip("skipping live SSH test in short mode")
-	}
-
 	rawHosts := []string{"signal@192.168.86.59", "signal@192.168.86.238"}
 
 	baseConf := ssh.ClientConfig{

internal/ui/dashboard/output_pane.go

@@ -13,22 +13,32 @@ import (
 	"github.com/agent462/herd/internal/grouper"
 )
 
-// outputPane wraps a bubbles/viewport for displaying grouped command results.
+// tabBarHeight is the number of rows consumed by the tab bar.
+const tabBarHeight = 2 // 1 row for tabs + 1 row for bottom border
+
+// outputPane wraps a bubbles/viewport for displaying grouped command results,
+// with a tab bar for switching between the diff view and per-host output.
 type outputPane struct {
-	viewport     viewport.Model
-	width        int
-	height       int
-	expandedHost string // when non-empty, show only this host's output
+	viewport viewport.Model
+	tabBar   tabBar
+	width    int
+	height   int
+
+	// Cached data for re-rendering when tabs switch.
+	lastGrouped *grouper.GroupedResults
+	lastResults []*executor.HostResult
+	allHosts    []string
 }
 
 func newOutputPane(width, height int) outputPane {
 	contentWidth := width - 2 // account for pane border
 	vp := viewport.New(
 		viewport.WithWidth(contentWidth),
-		viewport.WithHeight(height-2), // account for border
+		viewport.WithHeight(height-2-tabBarHeight), // border + tab bar
 	)
 	return outputPane{
 		viewport: vp,
+		tabBar:   newTabBar(contentWidth),
 		width:    contentWidth,
 		height:   height,
 	}
@@ -41,13 +51,14 @@ func (o *outputPane) Update(msg tea.Msg) tea.Cmd {
 }
 
 func (o *outputPane) View() string {
-	// Hard-clip the viewport output to prevent any line from exceeding the
-	// content width.  The viewport pads lines but does not truncate, so this
-	// catches edge cases where styled/ANSI content is wider than expected.
+	bar := o.tabBar.View()
+	var content string
 	if o.width > 0 {
-		return lipgloss.NewStyle().MaxWidth(o.width).Render(o.viewport.View())
+		content = lipgloss.NewStyle().MaxWidth(o.width).Render(o.viewport.View())
+	} else {
+		content = o.viewport.View()
 	}
-	return o.viewport.View()
+	return lipgloss.JoinVertical(lipgloss.Left, bar, content)
 }
 
 // setContent truncates each line to the viewport width (ANSI-aware) before
@@ -68,37 +79,105 @@ func (o *outputPane) Resize(width, height int) {
 	o.width = width - 2 // content width inside pane border
 	o.height = height
 	o.viewport.SetWidth(o.width)
-	o.viewport.SetHeight(height - 2)
+	o.viewport.SetHeight(height - 2 - tabBarHeight)
+	o.tabBar.Resize(o.width)
 }
 
+// SetGroupedResults updates the output pane with new execution results.
+// Rebuilds the tab bar and re-renders the active tab's content.
 func (o *outputPane) SetGroupedResults(grouped *grouper.GroupedResults, results []*executor.HostResult) {
+	o.lastGrouped = grouped
+	o.lastResults = results
+
 	if grouped == nil {
 		o.setContent("No results yet. Type a command below.")
 		return
 	}
 
-	if o.expandedHost != "" {
-		o.renderHostOutput(o.expandedHost, grouped, results)
-		return
+	// Collect all hosts in order from results.
+	hosts := make([]string, 0, len(results))
+	for _, r := range results {
+		hosts = append(hosts, r.Host)
+	}
+	o.allHosts = hosts
+	o.tabBar.SetTabs(hosts)
+
+	o.renderActiveTab()
+}
+
+// renderActiveTab dispatches to the correct renderer based on the active tab.
+func (o *outputPane) renderActiveTab() {
+	id := o.tabBar.ActiveID()
+	if id == "diff" {
+		if o.lastGrouped != nil {
+			o.renderGrouped(o.lastGrouped)
+		}
+	} else {
+		o.renderHostOutput(id, o.lastGrouped, o.lastResults)
 	}
+}
 
-	o.renderGrouped(grouped)
+// NextTab switches to the next tab and re-renders.
+func (o *outputPane) NextTab() {
+	o.tabBar.Next()
+	o.renderActiveTab()
 }
 
+// PrevTab switches to the previous tab and re-renders.
+func (o *outputPane) PrevTab() {
+	o.tabBar.Prev()
+	o.renderActiveTab()
+}
+
+// SetTabIndex jumps to a tab by index and re-renders.
+func (o *outputPane) SetTabIndex(index int) {
+	o.tabBar.SetActive(index)
+	o.renderActiveTab()
+}
+
+// ActivateHostTab switches to a specific host's tab.
+// Returns true if the host was found in the tab list.
+func (o *outputPane) ActivateHostTab(hostname string) bool {
+	if o.tabBar.SetActiveByID(hostname) {
+		o.renderActiveTab()
+		return true
+	}
+	return false
+}
+
+// ActivateDiffTab switches back to the diff output tab.
+func (o *outputPane) ActivateDiffTab() {
+	o.tabBar.SetActive(0)
+	o.renderActiveTab()
+}
+
+// ExpandHost is a backwards-compatible wrapper that activates the host's tab.
 func (o *outputPane) ExpandHost(name string, grouped *grouper.GroupedResults, results []*executor.HostResult) {
-	o.expandedHost = name
-	o.renderHostOutput(name, grouped, results)
+	o.lastGrouped = grouped
+	o.lastResults = results
+	// Ensure tabs are populated.
+	if len(o.tabBar.tabs) <= 1 {
+		hosts := make([]string, 0, len(results))
+		for _, r := range results {
+			hosts = append(hosts, r.Host)
+		}
+		o.allHosts = hosts
+		o.tabBar.SetTabs(hosts)
+	}
+	o.tabBar.SetActiveByID(name)
+	o.renderActiveTab()
 }
 
+// CollapseHost is a backwards-compatible wrapper that returns to the diff tab.
 func (o *outputPane) CollapseHost(grouped *grouper.GroupedResults, results []*executor.HostResult) {
-	o.expandedHost = ""
-	if grouped != nil {
-		o.renderGrouped(grouped)
-	}
+	o.lastGrouped = grouped
+	o.lastResults = results
+	o.ActivateDiffTab()
 }
 
+// IsExpanded returns true if viewing a specific host (not the diff tab).
 func (o *outputPane) IsExpanded() bool {
-	return o.expandedHost != ""
+	return o.tabBar.ActiveID() != "diff"
 }
 
 func (o *outputPane) renderGrouped(grouped *grouper.GroupedResults) {

internal/ui/dashboard/status_bar.go

@@ -22,14 +22,35 @@ func renderStatusBar(totalHosts, connectedHosts int, width int, groupName string
 
 	left += " │ " + connStr + disconnStr
 
-	right := helpKeyStyle.Render("Tab") + helpDescStyle.Render(" focus") +
-		"  " + helpKeyStyle.Render("f") + helpDescStyle.Render(" filter") +
-		"  " + helpKeyStyle.Render("d") + helpDescStyle.Render(" diff") +
-		"  " + helpKeyStyle.Render("?") + helpDescStyle.Render(" help") +
-		"  " + helpKeyStyle.Render("q") + helpDescStyle.Render(" quit") + " "
+	// Build right-side hints, dropping lowest-priority items (from the end)
+	// when they don't fit alongside the left side.
+	type hint struct{ key, desc string }
+	// Ordered by priority — first items survive at narrow widths.
+	hints := []hint{
+		{"q", "quit"},
+		{"Tab", "focus"},
+		{"[ ]", "tabs"},
+		{"?", "help"},
+		{"f", "filter"},
+		{"d", "diff"},
+	}
+
+	rightPadding := 1 // trailing space
+	stylePadding := statusBarStyle.GetHorizontalPadding()
+	avail := width - lipgloss.Width(left) - rightPadding - stylePadding
+	right := ""
+	for _, h := range hints {
+		item := "  " + helpKeyStyle.Render(h.key) + helpDescStyle.Render(" "+h.desc)
+		if lipgloss.Width(right)+lipgloss.Width(item) > avail {
+			break
+		}
+		right += item
+	}
+	right += " "
 
-	// Pad middle.
-	gap := width - lipgloss.Width(left) - lipgloss.Width(right)
+	// Pad middle to fill the content area (total width minus style padding).
+	contentWidth := width - stylePadding
+	gap := contentWidth - lipgloss.Width(left) - lipgloss.Width(right)
 	if gap < 0 {
 		gap = 0
 	}
@@ -47,9 +68,11 @@ func renderHelpOverlay(width, height int) string {
   Tab          Cycle focus: hosts → output → input
   q / Ctrl+C   Quit (when not typing)
   j / k        Navigate host table up/down
-  Enter        Host table: expand host output
+  Enter        Host table: jump to host tab
                Command input: execute command
-  Esc          Close overlay / collapse expanded view
+  Esc          Close overlay / back to diff tab
+  [ / ]        Previous / next output tab
+  1-9          Jump to output tab by number
   f            Toggle host filter bar
   d            Show diff for selected divergent host
   ?            Toggle this help
@@ -65,8 +88,8 @@ func renderHelpOverlay(width, height int) string {
 `
 
 	style := lipgloss.NewStyle().
-		Width(width - 4).
-		Height(height - 2).
+		Width(width-4).
+		Height(height-2).
 		Padding(1, 2).
 		Border(lipgloss.RoundedBorder()).
 		BorderForeground(colorCyan)

internal/ui/dashboard/styles.go

@@ -74,4 +74,23 @@ var (
 
 	helpDescStyle = lipgloss.NewStyle().
 			Foreground(colorSubtle)
+
+	// Tab bar styles.
+	tabActiveStyle = lipgloss.NewStyle().
+			Foreground(colorWhite).
+			Background(lipgloss.Color("#333333")).
+			Bold(true).
+			Padding(0, 1)
+
+	tabInactiveStyle = lipgloss.NewStyle().
+				Foreground(colorSubtle).
+				Padding(0, 1)
+
+	tabBarStyle = lipgloss.NewStyle().
+			BorderBottom(true).
+			BorderStyle(lipgloss.NormalBorder()).
+			BorderForeground(colorSubtle)
+
+	tabScrollIndicator = lipgloss.NewStyle().
+				Foreground(colorCyan)
 )