Skip to content

Commit a683447

Browse files
JoannaaKLJoannaaKL
authored
Merge branch 'main' into fix/logrus-to-slog
2 parents bc7c54b + 8aa3379 commit a683447

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

47 files changed

+832
-284
lines changed

‎README.md‎

Lines changed: 10 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -458,7 +458,7 @@ The following sets of tools are available (all are on by default):
458458

459459
- **list_discussion_categories** - List discussion categories
460460
- `owner`: Repository owner (string, required)
461-
- `repo`: Repository name (string, required)
461+
- `repo`: Repository name. If not provided, discussion categories will be queried at the organisation level. (string, optional)
462462

463463
- **list_discussions** - List discussions
464464
- `after`: Cursor for pagination. Use the endCursor from the previous page's PageInfo for GraphQL APIs. (string, optional)
@@ -539,15 +539,15 @@ The following sets of tools are available (all are on by default):
539539
- `repo`: Repository name (string, required)
540540

541541
- **list_issues** - List issues
542-
- `direction`: Sort direction (string, optional)
542+
- `after`: Cursor for pagination. Use the endCursor from the previous page's PageInfo for GraphQL APIs. (string, optional)
543+
- `direction`: Order direction. If provided, the 'orderBy' also needs to be provided. (string, optional)
543544
- `labels`: Filter by labels (string[], optional)
545+
- `orderBy`: Order issues by field. If provided, the 'direction' also needs to be provided. (string, optional)
544546
- `owner`: Repository owner (string, required)
545-
- `page`: Page number for pagination (min 1) (number, optional)
546547
- `perPage`: Results per page for pagination (min 1, max 100) (number, optional)
547548
- `repo`: Repository name (string, required)
548549
- `since`: Filter by date (ISO 8601 timestamp) (string, optional)
549-
- `sort`: Sort order (string, optional)
550-
- `state`: Filter by state (string, optional)
550+
- `state`: Filter by state, by default both open and closed issues are returned when not provided (string, optional)
551551

552552
- **list_sub_issues** - List sub-issues
553553
- `issue_number`: Issue number (number, required)
@@ -572,11 +572,11 @@ The following sets of tools are available (all are on by default):
572572

573573
- **search_issues** - Search issues
574574
- `order`: Sort order (string, optional)
575-
- `owner`: Optional repository owner. If provided with repo, only notifications for this repository are listed. (string, optional)
575+
- `owner`: Optional repository owner. If provided with repo, only issues for this repository are listed. (string, optional)
576576
- `page`: Page number for pagination (min 1) (number, optional)
577577
- `perPage`: Results per page for pagination (min 1, max 100) (number, optional)
578578
- `query`: Search query using GitHub issues search syntax (string, required)
579-
- `repo`: Optional repository name. If provided with owner, only notifications for this repository are listed. (string, optional)
579+
- `repo`: Optional repository name. If provided with owner, only issues for this repository are listed. (string, optional)
580580
- `sort`: Sort field by number of matches of categories, defaults to best match (string, optional)
581581

582582
- **update_issue** - Edit issue
@@ -744,11 +744,11 @@ The following sets of tools are available (all are on by default):
744744

745745
- **search_pull_requests** - Search pull requests
746746
- `order`: Sort order (string, optional)
747-
- `owner`: Optional repository owner. If provided with repo, only notifications for this repository are listed. (string, optional)
747+
- `owner`: Optional repository owner. If provided with repo, only pull requests for this repository are listed. (string, optional)
748748
- `page`: Page number for pagination (min 1) (number, optional)
749749
- `perPage`: Results per page for pagination (min 1, max 100) (number, optional)
750750
- `query`: Search query using GitHub pull request search syntax (string, required)
751-
- `repo`: Optional repository name. If provided with owner, only notifications for this repository are listed. (string, optional)
751+
- `repo`: Optional repository name. If provided with owner, only pull requests for this repository are listed. (string, optional)
752752
- `sort`: Sort field by number of matches of categories, defaults to best match (string, optional)
753753

754754
- **submit_pending_pull_request_review** - Submit the requester's latest pending pull request review
@@ -766,6 +766,7 @@ The following sets of tools are available (all are on by default):
766766
- `owner`: Repository owner (string, required)
767767
- `pullNumber`: Pull request number to update (number, required)
768768
- `repo`: Repository name (string, required)
769+
- `reviewers`: GitHub usernames to request reviews from (string[], optional)
769770
- `state`: New state (string, optional)
770771
- `title`: New title (string, optional)
771772

‎cmd/github-mcp-server/generate_docs.go‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ import (
1313
"github.com/github/github-mcp-server/pkg/raw"
1414
"github.com/github/github-mcp-server/pkg/toolsets"
1515
"github.com/github/github-mcp-server/pkg/translations"
16-
gogithub "github.com/google/go-github/v73/github"
16+
gogithub "github.com/google/go-github/v74/github"
1717
"github.com/mark3labs/mcp-go/mcp"
1818
"github.com/shurcooL/githubv4"
1919
"github.com/spf13/cobra"

‎e2e/e2e_test.go‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ import (
1818
"github.com/github/github-mcp-server/internal/ghmcp"
1919
"github.com/github/github-mcp-server/pkg/github"
2020
"github.com/github/github-mcp-server/pkg/translations"
21-
gogithub "github.com/google/go-github/v73/github"
21+
gogithub "github.com/google/go-github/v74/github"
2222
mcpClient "github.com/mark3labs/mcp-go/client"
2323
"github.com/mark3labs/mcp-go/mcp"
2424
"github.com/stretchr/testify/require"

‎go.mod‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ module github.com/github/github-mcp-server
33
go 1.23.7
44

55
require (
6-
github.com/google/go-github/v73 v73.0.0
6+
github.com/google/go-github/v74 v74.0.0
77
github.com/josephburnett/jd v1.9.2
88
github.com/mark3labs/mcp-go v0.32.0
99
github.com/migueleliasweb/go-github-mock v1.3.0

‎go.sum‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,8 +20,8 @@ github.com/google/go-cmp v0.7.0 h1:wk8382ETsv4JYUZwIsn6YpYiWiBsYLSJiTsyBybVuN8=
2020
github.com/google/go-cmp v0.7.0/go.mod h1:pXiqmnSA92OHEEa9HXL2W4E7lf9JzCmGVUdgjX3N/iU=
2121
github.com/google/go-github/v71 v71.0.0 h1:Zi16OymGKZZMm8ZliffVVJ/Q9YZreDKONCr+WUd0Z30=
2222
github.com/google/go-github/v71 v71.0.0/go.mod h1:URZXObp2BLlMjwu0O8g4y6VBneUj2bCHgnI8FfgZ51M=
23-
github.com/google/go-github/v73 v73.0.0 h1:aR+Utnh+Y4mMkS+2qLQwcQ/cF9mOTpdwnzlaw//rG24=
24-
github.com/google/go-github/v73 v73.0.0/go.mod h1:fa6w8+/V+edSU0muqdhCVY7Beh1M8F1IlQPZIANKIYw=
23+
github.com/google/go-github/v74 v74.0.0 h1:yZcddTUn8DPbj11GxnMrNiAnXH14gNs559AsUpNpPgM=
24+
github.com/google/go-github/v74 v74.0.0/go.mod h1:ubn/YdyftV80VPSI26nSJvaEsTOnsjrxG3o9kJhcyak=
2525
github.com/google/go-querystring v1.1.0 h1:AnCroh3fv4ZBgVIf1Iwtovgjaw/GiKJo8M8yD/fhyJ8=
2626
github.com/google/go-querystring v1.1.0/go.mod h1:Kcdr2DB4koayq7X8pmAG4sNG59So17icRSOU623lUBU=
2727
github.com/google/uuid v1.6.0 h1:NIvaJDMOsjHA8n1jAhLSgzrAzy1Hgr+hNrb57e+94F0=

‎internal/ghmcp/server.go‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ import (
1818
mcplog "github.com/github/github-mcp-server/pkg/log"
1919
"github.com/github/github-mcp-server/pkg/raw"
2020
"github.com/github/github-mcp-server/pkg/translations"
21-
gogithub "github.com/google/go-github/v73/github"
21+
gogithub "github.com/google/go-github/v74/github"
2222
"github.com/mark3labs/mcp-go/mcp"
2323
"github.com/mark3labs/mcp-go/server"
2424
"github.com/shurcooL/githubv4"

‎pkg/errors/error.go‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ import (
44
"context"
55
"fmt"
66

7-
"github.com/google/go-github/v73/github"
7+
"github.com/google/go-github/v74/github"
88
"github.com/mark3labs/mcp-go/mcp"
99
)
1010

‎pkg/errors/error_test.go‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ import (
66
"net/http"
77
"testing"
88

9-
"github.com/google/go-github/v73/github"
9+
"github.com/google/go-github/v74/github"
1010
"github.com/stretchr/testify/assert"
1111
"github.com/stretchr/testify/require"
1212
)

‎pkg/github/__toolsnaps__/list_issues.snap‎

Lines changed: 19 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -3,14 +3,18 @@
33
"title": "List issues",
44
"readOnlyHint": true
55
},
6-
"description": "List issues in a GitHub repository.",
6+
"description": "List issues in a GitHub repository. For pagination, use the 'endCursor' from the previous response's 'pageInfo' in the 'after' parameter.",
77
"inputSchema": {
88
"properties": {
9+
"after": {
10+
"description": "Cursor for pagination. Use the endCursor from the previous page's PageInfo for GraphQL APIs.",
11+
"type": "string"
12+
},
913
"direction": {
10-
"description": "Sort direction",
14+
"description": "Order direction. If provided, the 'orderBy' also needs to be provided.",
1115
"enum": [
12-
"asc",
13-
"desc"
16+
"ASC",
17+
"DESC"
1418
],
1519
"type": "string"
1620
},
@@ -21,15 +25,18 @@
2125
},
2226
"type": "array"
2327
},
28+
"orderBy": {
29+
"description": "Order issues by field. If provided, the 'direction' also needs to be provided.",
30+
"enum": [
31+
"CREATED_AT",
32+
"UPDATED_AT"
33+
],
34+
"type": "string"
35+
},
2436
"owner": {
2537
"description": "Repository owner",
2638
"type": "string"
2739
},
28-
"page": {
29-
"description": "Page number for pagination (min 1)",
30-
"minimum": 1,
31-
"type": "number"
32-
},
3340
"perPage": {
3441
"description": "Results per page for pagination (min 1, max 100)",
3542
"maximum": 100,
@@ -44,21 +51,11 @@
4451
"description": "Filter by date (ISO 8601 timestamp)",
4552
"type": "string"
4653
},
47-
"sort": {
48-
"description": "Sort order",
49-
"enum": [
50-
"created",
51-
"updated",
52-
"comments"
53-
],
54-
"type": "string"
55-
},
5654
"state": {
57-
"description": "Filter by state",
55+
"description": "Filter by state, by default both open and closed issues are returned when not provided",
5856
"enum": [
59-
"open",
60-
"closed",
61-
"all"
57+
"OPEN",
58+
"CLOSED"
6259
],
6360
"type": "string"
6461
}

‎pkg/github/__toolsnaps__/search_issues.snap‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@
1515
"type": "string"
1616
},
1717
"owner": {
18-
"description": "Optional repository owner. If provided with repo, only notifications for this repository are listed.",
18+
"description": "Optional repository owner. If provided with repo, only issues for this repository are listed.",
1919
"type": "string"
2020
},
2121
"page": {
@@ -34,7 +34,7 @@
3434
"type": "string"
3535
},
3636
"repo": {
37-
"description": "Optional repository name. If provided with owner, only notifications for this repository are listed.",
37+
"description": "Optional repository name. If provided with owner, only issues for this repository are listed.",
3838
"type": "string"
3939
},
4040
"sort": {

0 commit comments

Comments
 (0)