Streamline TUI candidate inspection

Author: Chris Jean

README.md

@@ -82,7 +82,7 @@ crucible tui
 crucible tui --run previous
 ```
 
-The TUI loads the selected run archive directly from `.crucible/runs/`, shows run status, leaderboard rows, selected candidate details, and forms for auto run setup, discovery, evaluator generation, competitor generation, adoption, promotion, evaluation, next-round preparation, evolution, reports, index rebuilds, archive queries, and candidate inspection. The new-run form defaults to `run --auto`, so typing the optimization request and pressing Enter is enough to create the run, generate and validate an evaluator, record baseline metrics, and return to a leaderboard. The forms execute the same command paths as the shell CLI, show a live spinner with elapsed time while actions run, allow cancellation requests for long-running actions, and show captured command output when the action finishes. Canceled evaluation work records an event and marks affected unevaluated candidates as `canceled` without overwriting completed `passed` or `failed` results. Canceled generation also records valid unadopted candidate artifacts as `canceled` and records partial candidate directories in the cancellation event.
+The TUI loads the selected run archive directly from `.crucible/runs/`, shows run status, leaderboard rows, selected candidate details, and forms for auto run setup, discovery, evaluator generation, competitor generation, adoption, promotion, evaluation, next-round preparation, evolution, reports, index rebuilds, archive queries, and candidate inspection. When a non-baseline candidate has passed, the dashboard selects it by default; otherwise it selects the first non-baseline candidate before falling back to the baseline. The candidate table is the picker for candidate-specific actions, so inspecting the selected candidate runs directly from the table instead of asking for an ID. The new-run form defaults to `run --auto`, so typing the optimization request and pressing Enter is enough to create the run, generate and validate an evaluator, record baseline metrics, and return to a leaderboard. The forms execute the same command paths as the shell CLI, show a live spinner with elapsed time while actions run, allow cancellation requests for long-running actions, and show captured command output when the action finishes. Canceled evaluation work records an event and marks affected unevaluated candidates as `canceled` without overwriting completed `passed` or `failed` results. Canceled generation also records valid unadopted candidate artifacts as `canceled` and records partial candidate directories in the cancellation event.
 
 Create a tournament run from inside an existing project:
 
@@ -707,6 +707,7 @@ Current priorities:
 - [x] Add prompt and TUI actions for evaluator generation on existing runs
 - [x] Fail fast with clear guidance when a selected agent executable is unavailable
 - [x] Add CLI and TUI candidate promotion back into the source project with dry-run previews and archived promotion reports
+- [x] Make TUI candidate inspection act on the selected row without requiring candidate ID entry
 
 Deferred roadmap:
 

docs/tui.md

@@ -19,7 +19,7 @@ Code Crucible uses Bubble Tea v1.2.x and Bubbles v0.20.x because that pair suppo
 
 - Keep `crucible tui` as the explicit entrypoint instead of replacing bare `crucible`.
 - Use the existing `internal/cli.WorkflowController` from prompt mode and the TUI so both interfaces share command construction and execution.
-- The first implementation includes a run dashboard: latest or selected run status, leaderboard rows, selected candidate detail, and common next actions. Actions that have TUI forms are shown as key-first prompts; workflows that are not yet implemented in the TUI use short CLI hints instead of long absolute commands.
+- The first implementation includes a run dashboard: latest or selected run status, leaderboard rows, selected candidate detail, and common next actions. The dashboard defaults to a passed non-baseline candidate when one exists, and the candidate table is the picker for candidate-specific actions such as inspect. Actions that have TUI forms are shown as key-first prompts; workflows that are not yet implemented in the TUI use short CLI hints instead of long absolute commands.
 - Forms cover low-prompt auto run setup, discovery, evaluator generation, generation, adoption, promotion, evaluation, next-round preparation, evolution, reports, index rebuilds, archive queries, and inspection. The new-run form defaults to `crucible run --auto`, so the operator can type only the optimization request and press Enter to create the run, generate and validate an evaluator, record baseline metrics, and refresh the dashboard. Forms use Bubbles text inputs, execute through the shared CLI controller, show a live spinner with elapsed time while actions run, allow cancellation requests for long-running actions, and show captured output in a scrollable viewport after completion.
 - Cancellation is conservative: Code Crucible records a cancellation event and marks affected unevaluated evaluation candidates as `canceled`, but does not overwrite completed `passed` or `failed` results. Canceled generation records valid unadopted candidate artifacts as `canceled` and records partial candidate directories in the same event stream.
 - Candidate selection uses the Bubbles table widget, and candidate details use a scrollable viewport.

internal/cli/tui.go

@@ -203,10 +203,11 @@ func loadTUIDashboard(projectDir, runSelector string) (tuiDashboardData, error)
 
 func newTUIDashboardModel(data tuiDashboardData, message string) tuiDashboardModel {
 	m := tuiDashboardModel{
-		data:    data,
-		width:   100,
-		message: message,
-		spinner: spinner.New(),
+		data:     data,
+		selected: defaultTUISelectedIndex(data.Results),
+		width:    100,
+		message:  message,
+		spinner:  spinner.New(),
 	}
 	m.configureBubbles()
 	return m
@@ -323,7 +324,12 @@ func (m tuiDashboardModel) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
 		case "s":
 			m.openForm(tuiActionQuery)
 		case "p":
-			m.openForm(tuiActionInspect)
+			if m.selectedCandidateID() == "" {
+				m.message = "No candidate is selected."
+				return m, nil
+			}
+			m.form = m.newForm(tuiActionInspect)
+			return m.submitForm()
 		case "up", "down", "k", "j", "home", "end", "pgup", "pgdown":
 			var cmd tea.Cmd
 			m.table, cmd = m.table.Update(msg)
@@ -1392,6 +1398,20 @@ func (m tuiDashboardModel) selectedCandidateID() string {
 	return m.data.Results[selected].Candidate.ID
 }
 
+func defaultTUISelectedIndex(results []model.CandidateResult) int {
+	for i, result := range results {
+		if result.Status == model.CandidateStatusPassed && !result.Candidate.Baseline {
+			return i
+		}
+	}
+	for i, result := range results {
+		if !result.Candidate.Baseline {
+			return i
+		}
+	}
+	return 0
+}
+
 func tuiStatusCounts(results []model.CandidateResult) (passed, failed, canceled, pending int) {
 	for _, result := range results {
 		switch result.Status {

internal/cli/tui_test.go

@@ -91,6 +91,8 @@ func TestTUIDashboardNavigationChangesSelectedCandidate(t *testing.T) {
 			{Candidate: model.Candidate{ID: "candidate-0001"}},
 		},
 	}, "")
+	dashboard.selected = 0
+	dashboard.configureBubbles()
 
 	updated, _ := dashboard.Update(tea.KeyMsg{Type: tea.KeyRunes, Runes: []rune{'j'}})
 	navigated := updated.(tuiDashboardModel)
@@ -108,6 +110,64 @@ func TestTUIDashboardNavigationChangesSelectedCandidate(t *testing.T) {
 	}
 }
 
+func TestTUIDashboardDefaultsToFirstNonBaselinePassedCandidate(t *testing.T) {
+	dashboard := newTUIDashboardModel(tuiDashboardData{
+		Config: model.RunConfig{ID: "run-1"},
+		Results: []model.CandidateResult{
+			{
+				Candidate: model.Candidate{ID: "candidate-0000-baseline", Baseline: true},
+				Status:    model.CandidateStatusPassed,
+				Score:     100,
+			},
+			{
+				Candidate: model.Candidate{ID: "candidate-0001"},
+				Status:    model.CandidateStatusPassed,
+				Score:     95,
+			},
+		},
+	}, "")
+
+	if dashboard.selected != 1 {
+		t.Fatalf("selected = %d, want first non-baseline passed candidate", dashboard.selected)
+	}
+	if got := dashboard.selectedCandidateID(); got != "candidate-0001" {
+		t.Fatalf("selected candidate = %q, want candidate-0001", got)
+	}
+}
+
+func TestTUIInspectKeyRunsSelectedCandidateDirectly(t *testing.T) {
+	dashboard := newTUIDashboardModel(tuiDashboardData{
+		ProjectDir: "/tmp/code crucible fixture",
+		Config: model.RunConfig{
+			ID: "run-1",
+		},
+		Results: []model.CandidateResult{
+			{
+				Candidate: model.Candidate{ID: "candidate-0000-baseline", Baseline: true},
+				Status:    model.CandidateStatusPassed,
+			},
+			{
+				Candidate: model.Candidate{ID: "candidate-0001"},
+				Status:    model.CandidateStatusPassed,
+			},
+		},
+	}, "")
+
+	updated, cmd := dashboard.Update(tea.KeyMsg{Type: tea.KeyRunes, Runes: []rune{'p'}})
+	running := updated.(tuiDashboardModel)
+	if cmd == nil {
+		t.Fatal("inspect key did not start an action")
+	}
+	if !running.busy {
+		t.Fatal("inspect key did not enter busy action state")
+	}
+	for _, want := range []string{"crucible inspect", "--run run-1", "candidate-0001"} {
+		if !strings.Contains(running.actionCmd, want) {
+			t.Fatalf("inspect command did not contain %q:\n%s", want, running.actionCmd)
+		}
+	}
+}
+
 func TestTUIStartsWithoutExistingRun(t *testing.T) {
 	projectDir := t.TempDir()
 	data, message, err := loadInitialTUIDashboard(projectDir, "latest")