Add list filter tests, use red for Fail and orange for Error

- Add integration tests for --open, --closed, --unaddressed query params
  and mutual-exclusion validation (--closed + --open, --closed + --unaddressed)
- Change Fail verdict color to red (124/196) and Error status to orange
  (166/208) for clearer semantics: red = findings, orange = job failure
- Change Canceled color to gray (243/245) to avoid collision with Error

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: wesm

cmd/roborev/list_test.go

@@ -190,6 +190,36 @@ func TestListCommand(t *testing.T) {
 				}
 			},
 		},
+		{
+			name:      "--open sends addressed=false query param",
+			args:      []string{"--open"},
+			handler:   jobsHandler([]storage.ReviewJob{}, false),
+			wantQuery: []string{"addressed=false"},
+		},
+		{
+			name:      "--closed sends addressed=true query param",
+			args:      []string{"--closed"},
+			handler:   jobsHandler([]storage.ReviewJob{}, false),
+			wantQuery: []string{"addressed=true"},
+		},
+		{
+			name:      "--unaddressed alias sends addressed=false query param",
+			args:      []string{"--unaddressed"},
+			handler:   jobsHandler([]storage.ReviewJob{}, false),
+			wantQuery: []string{"addressed=false"},
+		},
+		{
+			name:      "--closed and --open conflict",
+			args:      []string{"--closed", "--open"},
+			handler:   jobsHandler([]storage.ReviewJob{}, false),
+			wantError: "if any flags in the group",
+		},
+		{
+			name:      "--closed and --unaddressed conflict",
+			args:      []string{"--closed", "--unaddressed"},
+			handler:   jobsHandler([]storage.ReviewJob{}, false),
+			wantError: "if any flags in the group",
+		},
 		{
 			name: "explicit --repo with worktree path normalizes to main repo",
 			repoSetup: func(t *testing.T) repoSetupResult {

cmd/roborev/tui/tui.go

@@ -46,11 +46,11 @@ var (
 	queuedStyle   = lipgloss.NewStyle().Foreground(lipgloss.AdaptiveColor{Light: "136", Dark: "226"}) // Yellow/Gold
 	runningStyle  = lipgloss.NewStyle().Foreground(lipgloss.AdaptiveColor{Light: "25", Dark: "33"})   // Blue
 	doneStyle     = lipgloss.NewStyle().Foreground(lipgloss.AdaptiveColor{Light: "28", Dark: "46"})   // Green
-	failedStyle   = lipgloss.NewStyle().Foreground(lipgloss.AdaptiveColor{Light: "124", Dark: "196"}) // Red (job error)
-	canceledStyle = lipgloss.NewStyle().Foreground(lipgloss.AdaptiveColor{Light: "166", Dark: "208"}) // Orange
+	failedStyle   = lipgloss.NewStyle().Foreground(lipgloss.AdaptiveColor{Light: "166", Dark: "208"}) // Orange (job error)
+	canceledStyle = lipgloss.NewStyle().Foreground(lipgloss.AdaptiveColor{Light: "243", Dark: "245"}) // Gray
 
 	passStyle      = lipgloss.NewStyle().Foreground(lipgloss.AdaptiveColor{Light: "28", Dark: "46"})   // Green
-	failStyle      = lipgloss.NewStyle().Foreground(lipgloss.AdaptiveColor{Light: "172", Dark: "214"}) // Yellow/Orange (review found issues)
+	failStyle      = lipgloss.NewStyle().Foreground(lipgloss.AdaptiveColor{Light: "124", Dark: "196"}) // Red (review found issues)
 	addressedStyle = lipgloss.NewStyle().Foreground(lipgloss.AdaptiveColor{Light: "30", Dark: "51"})   // Cyan
 
 	helpStyle = lipgloss.NewStyle().