fix: address agent tasks review comments

Author: danyalahmed1995

.golangci.yml

@@ -277,6 +277,7 @@ linters:
             - ActionsCacheListOptions.Key
             - ActionsCacheListOptions.Ref
             - ActionsCacheListOptions.Sort
+            - AgentTaskListByRepoOptions.CreatorID # TODO: AgentTasks
             - ActiveCommittersListOptions.AdvancedSecurityProduct
             - AnalysesListOptions.Ref
             - AnalysesListOptions.SarifID

github/agent_tasks.go

@@ -158,12 +158,12 @@ func (s *AgentTasksService) Create(ctx context.Context, owner, repo string, opts
 	return task, resp, nil
 }
 
-// GetByRepo gets a repository task by ID.
+// GetByRepoAndID gets a repository task by ID.
 //
 // GitHub API docs: https://docs.github.com/rest/agent-tasks/agent-tasks?apiVersion=2026-03-10#get-a-task-by-repo
 //
 //meta:operation GET /agents/repos/{owner}/{repo}/tasks/{task_id}
-func (s *AgentTasksService) GetByRepo(ctx context.Context, owner, repo, taskID string) (*AgentTask, *Response, error) {
+func (s *AgentTasksService) GetByRepoAndID(ctx context.Context, owner, repo, taskID string) (*AgentTask, *Response, error) {
 	u := fmt.Sprintf("agents/repos/%v/%v/tasks/%v", owner, repo, taskID)
 
 	req, err := s.client.NewRequest(ctx, "GET", u, nil, WithVersion("2026-03-10"))

github/agent_tasks_test.go

@@ -151,7 +151,7 @@ func TestAgentTasksService_ListByRepo(t *testing.T) {
 			Since:       Ptr(time.Date(2025, time.January, 1, 0, 0, 0, 0, time.UTC)),
 			ListOptions: ListOptions{Page: 2, PerPage: 1},
 		},
-		CreatorID: 1,
+		CreatorID: Ptr(int64(1)),
 	}
 
 	ctx := t.Context()
@@ -227,7 +227,7 @@ func TestAgentTasksService_Create(t *testing.T) {
 	})
 }
 
-func TestAgentTasksService_GetByRepo(t *testing.T) {
+func TestAgentTasksService_GetByRepoAndID(t *testing.T) {
 	t.Parallel()
 	client, mux, _ := setup(t)
 
@@ -238,22 +238,22 @@ func TestAgentTasksService_GetByRepo(t *testing.T) {
 	})
 
 	ctx := t.Context()
-	task, _, err := client.AgentTasks.GetByRepo(ctx, "o", "r", agentTaskID)
+	task, _, err := client.AgentTasks.GetByRepoAndID(ctx, "o", "r", agentTaskID)
 	if err != nil {
-		t.Fatalf("AgentTasks.GetByRepo returned error: %v", err)
+		t.Fatalf("AgentTasks.GetByRepoAndID returned error: %v", err)
 	}
 	if diff := cmp.Diff(agentTask(true), task, cmpJSONRawMessageComparator()); diff != "" {
-		t.Errorf("AgentTasks.GetByRepo mismatch (-want +got):\n%v", diff)
+		t.Errorf("AgentTasks.GetByRepoAndID mismatch (-want +got):\n%v", diff)
 	}
 
-	const methodName = "GetByRepo"
+	const methodName = "GetByRepoAndID"
 	testBadOptions(t, methodName, func() error {
-		_, _, err := client.AgentTasks.GetByRepo(ctx, "\n", "\n", "\n")
+		_, _, err := client.AgentTasks.GetByRepoAndID(ctx, "\n", "\n", "\n")
 		return err
 	})
 
 	testNewRequestAndDoFailure(t, methodName, client, func() (*Response, error) {
-		got, resp, err := client.AgentTasks.GetByRepo(ctx, "o", "r", agentTaskID)
+		got, resp, err := client.AgentTasks.GetByRepoAndID(ctx, "o", "r", agentTaskID)
 		if got != nil {
 			t.Errorf("testNewRequestAndDoFailure %v = %#v, want nil", methodName, got)
 		}