diff --git a/pkg/cmd/agent-task/list/list.go b/pkg/cmd/agent-task/list/list.go index 43f409ffe..caeb02fa7 100644 --- a/pkg/cmd/agent-task/list/list.go +++ b/pkg/cmd/agent-task/list/list.go @@ -90,11 +90,13 @@ func listRun(opts *ListOptions) error { defer opts.IO.StopProgressIndicator() var sessions []*capi.Session ctx := context.Background() + + var repo ghrepo.Interface if opts.BaseRepo != nil { - repo, err := opts.BaseRepo() - if err != nil { - return err - } + repo, _ = opts.BaseRepo() + } + + if repo != nil && repo.RepoOwner() != "" && repo.RepoName() != "" { sessions, err = capiClient.ListSessionsForRepo(ctx, repo.RepoOwner(), repo.RepoName(), opts.Limit) if err != nil { return err diff --git a/pkg/cmd/agent-task/list/list_test.go b/pkg/cmd/agent-task/list/list_test.go index f05d61b9e..2d27436d2 100644 --- a/pkg/cmd/agent-task/list/list_test.go +++ b/pkg/cmd/agent-task/list/list_test.go @@ -146,10 +146,12 @@ func Test_listRun(t *testing.T) { wantOut: "no agent tasks found\n", }, { - name: "repo resolution error surfaces", + name: "repo resolution error does not surface", tty: true, baseRepoErr: errors.New("ambiguous repo"), - wantErr: errors.New("ambiguous repo"), + wantErr: nil, + stubs: func(reg *httpmock.Registry) { registerEmptySessionsMock(reg) }, + wantOut: "no agent tasks found\n", }, { name: "repo scoped many sessions (tty)",