add author field to list_commits for filtering (#569)

Author: davidbyttow

README.md

@@ -705,6 +705,7 @@ export GITHUB_MCP_TOOL_ADD_ISSUE_COMMENT_DESCRIPTION="an alternative description
   - `owner`: Repository owner (string, required)
   - `repo`: Repository name (string, required)
   - `sha`: Branch name, tag, or commit SHA (string, optional)
+  - `author`: Author username or email address (string, optional)
   - `path`: Only commits containing this file path (string, optional)
   - `page`: Page number (number, optional)
   - `perPage`: Results per page (number, optional)

pkg/github/__toolsnaps__/list_commits.snap

@@ -28,6 +28,10 @@
       "sha": {
         "description": "SHA or Branch name",
         "type": "string"
+      },
+      "author": {
+        "description": "Author username or email address",
+        "type": "string"
       }
     },
     "required": [

pkg/github/repositories.go

@@ -108,6 +108,9 @@ func ListCommits(getClient GetClientFn, t translations.TranslationHelperFunc) (t
 			mcp.WithString("sha",
 				mcp.Description("SHA or Branch name"),
 			),
+			mcp.WithString("author",
+				mcp.Description("Author username or email address"),
+			),
 			WithPagination(),
 		),
 		func(ctx context.Context, request mcp.CallToolRequest) (*mcp.CallToolResult, error) {
@@ -123,13 +126,18 @@ func ListCommits(getClient GetClientFn, t translations.TranslationHelperFunc) (t
 			if err != nil {
 				return mcp.NewToolResultError(err.Error()), nil
 			}
+			author, err := OptionalParam[string](request, "author")
+			if err != nil {
+				return mcp.NewToolResultError(err.Error()), nil
+			}
 			pagination, err := OptionalPaginationParams(request)
 			if err != nil {
 				return mcp.NewToolResultError(err.Error()), nil
 			}
 
 			opts := &github.CommitsListOptions{
-				SHA: sha,
+				SHA:    sha,
+				Author: author,
 				ListOptions: github.ListOptions{
 					Page:    pagination.page,
 					PerPage: pagination.perPage,

pkg/github/repositories_test.go

@@ -646,6 +646,7 @@ func Test_ListCommits(t *testing.T) {
 	assert.Contains(t, tool.InputSchema.Properties, "owner")
 	assert.Contains(t, tool.InputSchema.Properties, "repo")
 	assert.Contains(t, tool.InputSchema.Properties, "sha")
+	assert.Contains(t, tool.InputSchema.Properties, "author")
 	assert.Contains(t, tool.InputSchema.Properties, "page")
 	assert.Contains(t, tool.InputSchema.Properties, "perPage")
 	assert.ElementsMatch(t, tool.InputSchema.Required, []string{"owner", "repo"})
@@ -713,6 +714,7 @@ func Test_ListCommits(t *testing.T) {
 				mock.WithRequestMatchHandler(
 					mock.GetReposCommitsByOwnerByRepo,
 					expectQueryParams(t, map[string]string{
+						"author":   "username",
 						"sha":      "main",
 						"page":     "1",
 						"per_page": "30",
@@ -722,9 +724,10 @@ func Test_ListCommits(t *testing.T) {
 				),
 			),
 			requestArgs: map[string]interface{}{
-				"owner": "owner",
-				"repo":  "repo",
-				"sha":   "main",
+				"owner":  "owner",
+				"repo":   "repo",
+				"sha":    "main",
+				"author": "username",
 			},
 			expectError:     false,
 			expectedCommits: mockCommits,
@@ -801,6 +804,7 @@ func Test_ListCommits(t *testing.T) {
 			require.NoError(t, err)
 			assert.Len(t, returnedCommits, len(tc.expectedCommits))
 			for i, commit := range returnedCommits {
+				assert.Equal(t, *tc.expectedCommits[i].Author, *commit.Author)
 				assert.Equal(t, *tc.expectedCommits[i].SHA, *commit.SHA)
 				assert.Equal(t, *tc.expectedCommits[i].Commit.Message, *commit.Commit.Message)
 				assert.Equal(t, *tc.expectedCommits[i].Author.Login, *commit.Author.Login)