Update list_pull_requests tool to add client-side author filtering

Author: brettfire

README.md

@@ -1063,6 +1063,7 @@ The following sets of tools are available:
 
 - **list_pull_requests** - List pull requests
   - **Required OAuth Scopes**: `repo`
+  - `author`: Filter by PR author username (client-side filter) (string, optional)
   - `base`: Filter by base branch (string, optional)
   - `direction`: Sort direction (string, optional)
   - `head`: Filter by head user/org and branch (string, optional)

pkg/github/pullrequests.go

@@ -1047,6 +1047,10 @@ func ListPullRequests(t translations.TranslationHelperFunc) inventory.ServerTool
 				Description: "Sort direction",
 				Enum:        []any{"asc", "desc"},
 			},
+			"author": {
+				Type:        "string",
+				Description: "Filter by PR author username (client-side filter)",
+			},
 		},
 		Required: []string{"owner", "repo"},
 	}
@@ -1056,7 +1060,7 @@ func ListPullRequests(t translations.TranslationHelperFunc) inventory.ServerTool
 		ToolsetMetadataPullRequests,
 		mcp.Tool{
 			Name:        "list_pull_requests",
-			Description: t("TOOL_LIST_PULL_REQUESTS_DESCRIPTION", "List pull requests in a GitHub repository. If the user specifies an author, then DO NOT use this tool and use the search_pull_requests tool instead. If you receive a 422 error from search_pull_requests, then use the get_prs_reviewed_by tool instead."),
+			Description: t("TOOL_LIST_PULL_REQUESTS_DESCRIPTION", "List pull requests in a GitHub repository. If the user specifies an author, then DO NOT use this tool and use the search_pull_requests tool instead. If you receive a 422 error from search_pull_requests, then use the get_prs_reviewed_by tool (to list by reviewer), or this tool with the author parameter (for filtering by author) depending on what you need."),
 			Annotations: &mcp.ToolAnnotations{
 				Title:        t("TOOL_LIST_PULL_REQUESTS_USER_TITLE", "List pull requests"),
 				ReadOnlyHint: true,
@@ -1093,6 +1097,10 @@ func ListPullRequests(t translations.TranslationHelperFunc) inventory.ServerTool
 			if err != nil {
 				return utils.NewToolResultError(err.Error()), nil, nil
 			}
+			author, err := OptionalParam[string](args, "author")
+			if err != nil {
+				return utils.NewToolResultError(err.Error()), nil, nil
+			}
 			pagination, err := OptionalPaginationParams(args)
 			if err != nil {
 				return utils.NewToolResultError(err.Error()), nil, nil
@@ -1132,6 +1140,18 @@ func ListPullRequests(t translations.TranslationHelperFunc) inventory.ServerTool
 				return ghErrors.NewGitHubAPIStatusErrorResponse(ctx, "failed to list pull requests", resp, bodyBytes), nil, nil
 			}
 
+			// Filter by author if specified (client-side filtering)
+			if author != "" {
+				filtered := make([]*github.PullRequest, 0)
+				for _, pr := range prs {
+					if pr != nil && pr.User != nil && pr.User.Login != nil &&
+						strings.EqualFold(*pr.User.Login, author) {
+						filtered = append(filtered, pr)
+					}
+				}
+				prs = filtered
+			}
+
 			// sanitize title/body on each PR
 			for _, pr := range prs {
 				if pr == nil {

pkg/github/pullrequests_test.go

@@ -978,6 +978,120 @@ func Test_MergePullRequest(t *testing.T) {
 	}
 }
 
+func Test_ListPullRequests_AuthorFilter(t *testing.T) {
+	// Setup mock PRs from multiple authors
+	mockPRs := []*github.PullRequest{
+		{
+			Number:  github.Ptr(42),
+			Title:   github.Ptr("PR by user1"),
+			State:   github.Ptr("open"),
+			HTMLURL: github.Ptr("https://github.com/owner/repo/pull/42"),
+			User: &github.User{
+				Login: github.Ptr("user1"),
+			},
+		},
+		{
+			Number:  github.Ptr(43),
+			Title:   github.Ptr("PR by user2"),
+			State:   github.Ptr("open"),
+			HTMLURL: github.Ptr("https://github.com/owner/repo/pull/43"),
+			User: &github.User{
+				Login: github.Ptr("user2"),
+			},
+		},
+		{
+			Number:  github.Ptr(44),
+			Title:   github.Ptr("Another PR by user1"),
+			State:   github.Ptr("closed"),
+			HTMLURL: github.Ptr("https://github.com/owner/repo/pull/44"),
+			User: &github.User{
+				Login: github.Ptr("user1"),
+			},
+		},
+	}
+
+	tests := []struct {
+		name            string
+		author          string
+		expectedCount   int
+		expectedNumbers []int
+	}{
+		{
+			name:            "filter by user1 returns 2 PRs",
+			author:          "user1",
+			expectedCount:   2,
+			expectedNumbers: []int{42, 44},
+		},
+		{
+			name:            "filter by user2 returns 1 PR",
+			author:          "user2",
+			expectedCount:   1,
+			expectedNumbers: []int{43},
+		},
+		{
+			name:            "filter by USER1 (case insensitive) returns 2 PRs",
+			author:          "USER1",
+			expectedCount:   2,
+			expectedNumbers: []int{42, 44},
+		},
+		{
+			name:            "filter by nonexistent user returns 0 PRs",
+			author:          "nonexistent",
+			expectedCount:   0,
+			expectedNumbers: []int{},
+		},
+		{
+			name:            "no author filter returns all PRs",
+			author:          "",
+			expectedCount:   3,
+			expectedNumbers: []int{42, 43, 44},
+		},
+	}
+
+	for _, tc := range tests {
+		t.Run(tc.name, func(t *testing.T) {
+			mockedClient := MockHTTPClientWithHandlers(map[string]http.HandlerFunc{
+				GetReposPullsByOwnerByRepo: mockResponse(t, http.StatusOK, mockPRs),
+			})
+
+			client := github.NewClient(mockedClient)
+			serverTool := ListPullRequests(translations.NullTranslationHelper)
+			deps := BaseDeps{
+				Client: client,
+			}
+			handler := serverTool.Handler(deps)
+
+			requestArgs := map[string]interface{}{
+				"owner": "owner",
+				"repo":  "repo",
+			}
+			if tc.author != "" {
+				requestArgs["author"] = tc.author
+			}
+
+			request := createMCPRequest(requestArgs)
+			result, err := handler(ContextWithDeps(context.Background(), deps), &request)
+
+			require.NoError(t, err)
+			require.False(t, result.IsError)
+
+			textContent := getTextResult(t, result)
+			var returnedPRs []*github.PullRequest
+			err = json.Unmarshal([]byte(textContent.Text), &returnedPRs)
+			require.NoError(t, err)
+
+			assert.Len(t, returnedPRs, tc.expectedCount)
+
+			// Verify the expected PR numbers are returned
+			returnedNumbers := make([]int, len(returnedPRs))
+			for i, pr := range returnedPRs {
+				returnedNumbers[i] = *pr.Number
+			}
+			assert.ElementsMatch(t, tc.expectedNumbers, returnedNumbers)
+		})
+	}
+}
+
 func Test_SearchPullRequests(t *testing.T) {
 	serverTool := SearchPullRequests(translations.NullTranslationHelper)
 	tool := serverTool.Tool