Fix: disable interactive TUI picker when --output-file is set

When --output-file is used, the command is intended for non-interactive
use (keeping stdout clean for callers). Previously, the interactive job
picker could still activate if there were multiple ambiguous jobs,
causing scripts to hang waiting for input.

This fix ensures that logTargetResolutionOptionsForOutput also checks
if outputFile is set and disables interactive mode accordingly, just
like it does for JSON output mode.

Author: cursoragent

pkg/cmd/ci/logs.go

@@ -129,7 +129,7 @@ ID, use --job and --workflow to disambiguate by workflow job key.`,
 			reporter := newLogFollowReporter(reporterWriter, reporterInteractive)
 
 			// First, try resolving as a run ID (or job ID — the API accepts both).
-			resolutionOptions := logTargetResolutionOptionsForOutput(outputOptions)
+			resolutionOptions := logTargetResolutionOptionsForOutput(outputOptions, outputFile)
 			resp, runErr := ciGetRunStatus(ctx, tokenVal, orgID, id)
 			if runErr == nil {
 				target, err := resolveLogTargetWithOptions(resp, id, job, workflow, resolutionOptions)
@@ -807,8 +807,8 @@ type logTargetResolutionOptions struct {
 	allowInteractive bool
 }
 
-func logTargetResolutionOptionsForOutput(outputOptions logOutputOptions) logTargetResolutionOptions {
-	return logTargetResolutionOptions{allowInteractive: !outputOptions.json()}
+func logTargetResolutionOptionsForOutput(outputOptions logOutputOptions, outputFile string) logTargetResolutionOptions {
+	return logTargetResolutionOptions{allowInteractive: !outputOptions.json() && outputFile == ""}
 }
 
 // jobKeyShort returns the short form of a job key (after the first colon),

pkg/cmd/ci/logs_test.go

@@ -75,7 +75,7 @@ func TestResolveLogTargetJSONOptionsReturnNonInteractiveAmbiguity(t *testing.T)
 		},
 	}
 
-	options := logTargetResolutionOptionsForOutput(logOutputOptions{output: logOutputJSON})
+	options := logTargetResolutionOptionsForOutput(logOutputOptions{output: logOutputJSON}, "")
 	if options.allowInteractive {
 		t.Fatal("json output should disable interactive job resolution")
 	}
@@ -89,6 +89,34 @@ func TestResolveLogTargetJSONOptionsReturnNonInteractiveAmbiguity(t *testing.T)
 	}
 }
 
+func TestResolveLogTargetOutputFileOptionsReturnNonInteractiveAmbiguity(t *testing.T) {
+	resp := &civ1.GetRunStatusResponse{
+		RunId: "run-1",
+		Workflows: []*civ1.WorkflowStatus{
+			{
+				WorkflowPath: ".depot/workflows/ci.yml",
+				Jobs: []*civ1.JobStatus{
+					{JobId: "job-1", JobKey: "build", Status: "finished"},
+					{JobId: "job-2", JobKey: "test", Status: "running"},
+				},
+			},
+		},
+	}
+
+	options := logTargetResolutionOptionsForOutput(logOutputOptions{output: logOutputText}, "/tmp/logs.txt")
+	if options.allowInteractive {
+		t.Fatal("output-file should disable interactive job resolution")
+	}
+
+	_, err := resolveLogTargetWithOptions(resp, "run-1", "", "", options)
+	if err == nil {
+		t.Fatal("expected ambiguity error")
+	}
+	if !strings.Contains(err.Error(), "run has multiple jobs, specify one with --job") {
+		t.Fatalf("expected multiple-jobs ambiguity error, got: %v", err)
+	}
+}
+
 func TestFindLogsJob_MatchByJobKey(t *testing.T) {
 	resp := &civ1.GetRunStatusResponse{
 		RunId: "run-1",