fix(tui): preserve selections during refresh (#591)

closes #586 

## Summary

- Preserve last queue entry visible in distraction-free mode during job
refresh
- Preserve review view selection during job refresh so it doesn't jump
back to the first item

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

Author: mariusvniekerk

cmd/roborev/tui/queue_test.go

@@ -347,6 +347,46 @@ func TestTUIQueueDistractionFreeToggle(t *testing.T) {
 	assert.Contains(t, output, "JobID")
 }
 
+// Regression test for #586: distraction-free mode lost the last queue entry.
+// The lipgloss table drops the last data row when Headers() is not called,
+// so compact mode must still supply (empty) headers and strip the resulting
+// blank line.
+func TestTUIQueueDistractionFreePreservesLastJob(t *testing.T) {
+	assert := assert.New(t)
+	for _, h := range []int{20, 24, 30} {
+		for _, n := range []int{10, 25, 40} {
+			t.Run(fmt.Sprintf("h%d_jobs%d", h, n), func(t *testing.T) {
+				m := newTuiModel("http://localhost")
+				m.currentView = tuiViewQueue
+				m.width = 120
+				m.height = h
+				for i := range n {
+					m.jobs = append(m.jobs, makeJob(int64(9300+i)))
+				}
+				lastJobID := fmt.Sprintf("%d", 9300+n-1)
+				// Select near the end so the last job is in the scroll window.
+				m.selectedIdx = n - 2
+				m.selectedJobID = int64(9300 + n - 2)
+
+				normalOutput := m.View()
+				normalHasLast := strings.Contains(normalOutput, lastJobID)
+
+				m2, _ := updateModel(t, m, tea.KeyMsg{Type: tea.KeyRunes, Runes: []rune{'D'}})
+				compactOutput := m2.View()
+				compactHasLast := strings.Contains(compactOutput, lastJobID)
+
+				if normalHasLast {
+					assert.True(compactHasLast,
+						"h=%d n=%d: job %s visible before D but missing after", h, n, lastJobID)
+				}
+				compactLines := strings.Count(compactOutput, "\n") + 1
+				assert.LessOrEqual(compactLines, h,
+					"h=%d n=%d: output %d lines exceeds terminal height", h, n, compactLines)
+			})
+		}
+	}
+}
+
 func TestTUITasksMouseClickSelectsRow(t *testing.T) {
 	m := newTuiModel("http://localhost")
 	m.currentView = tuiViewTasks

cmd/roborev/tui/render_queue.go

@@ -530,16 +530,26 @@ func (m model) renderQueueView() string {
 				return s
 			})
 
+		// Always set headers — lipgloss table drops the last data row
+		// when Headers() is not called.
+		headers := make([]string, len(visCols))
 		if !compact {
-			headers := make([]string, len(visCols))
 			for vi, c := range visCols {
 				headers[vi] = allHeaders[c]
 			}
-			t = t.Headers(headers...)
 		}
+		t = t.Headers(headers...)
 		t = t.Rows(rows...)
 
 		tableStr := t.Render()
+
+		// In compact mode, strip the empty header line we added as a
+		// workaround (it renders as a row of spaces).
+		if compact {
+			if idx := strings.Index(tableStr, "\n"); idx >= 0 {
+				tableStr = tableStr[idx+1:]
+			}
+		}
 		b.WriteString(tableStr)
 		b.WriteString("\x1b[K\n")