mcp: update GitHub MCP after issue tool consolidation (#1436)

nacx · web-flow · commit 3669d94c2089 · 2025-10-27T08:05:43.000-07:00
**Description** GitHub has consolidated the tools related to issues, causing some of our integration tests to break. This PR adjusts the tests and docs to use the new tool names. **Related Issues/PRs (if applicable)** GitHub PR: github/github-mcp-server#1211 **Special notes for reviewers (if applicable)** N/A Signed-off-by: Ignasi Barrera <ignasi@tetrate.io>
diff --git a/examples/mcp/openai-github.yaml b/examples/mcp/openai-github.yaml
@@ -206,7 +206,7 @@ spec:
       path: "/mcp/x/issues/readonly"
       toolSelector:
         include:
-          - get_issue
+          - issue_read
           - list_issues
       securityPolicy:
         apiKey:
diff --git a/internal/autoconfig/config_test.go b/internal/autoconfig/config_test.go
@@ -357,7 +357,7 @@ func TestWriteConfig(t *testing.T) {
 						BackendName:  "github",
 						Path:         "/mcp/x/issues/readonly",
 						APIKey:       "${GITHUB_MCP_TOKEN}",
-						IncludeTools: []string{"get_issue", "list_issues"},
+						IncludeTools: []string{"issue_read", "list_issues"},
 					},
 				},
 			},
diff --git a/internal/autoconfig/mcp_test.go b/internal/autoconfig/mcp_test.go
@@ -27,7 +27,7 @@ func TestAddMCPServersConfig(t *testing.T) {
 						Headers: map[string]string{
 							"Authorization": "Bearer ${GITHUB_MCP_TOKEN}",
 						},
-						IncludeTools: []string{"list_issues", "get_issue"},
+						IncludeTools: []string{"list_issues", "issue_read"},
 					},
 				},
 			},
@@ -45,7 +45,7 @@ func TestAddMCPServersConfig(t *testing.T) {
 					{
 						BackendName:  "github",
 						Path:         "/mcp/x/issues/readonly",
-						IncludeTools: []string{"list_issues", "get_issue"},
+						IncludeTools: []string{"list_issues", "issue_read"},
 						APIKey:       "${GITHUB_MCP_TOKEN}",
 						Headers:      map[string]string{},
 					},
@@ -104,7 +104,7 @@ func TestAddMCPServersConfig(t *testing.T) {
 						Headers: map[string]string{
 							"Authorization": "Bearer ${GITHUB_MCP_TOKEN}",
 						},
-						IncludeTools: []string{"list_issues", "get_issue"},
+						IncludeTools: []string{"list_issues", "issue_read"},
 					},
 				},
 			},
@@ -129,7 +129,7 @@ func TestAddMCPServersConfig(t *testing.T) {
 					{
 						BackendName:  "github",
 						Path:         "/mcp/x/issues/readonly",
-						IncludeTools: []string{"list_issues", "get_issue"},
+						IncludeTools: []string{"list_issues", "issue_read"},
 						APIKey:       "${GITHUB_MCP_TOKEN}",
 						Headers:      map[string]string{},
 					},
diff --git a/internal/autoconfig/testdata/openai-github.yaml b/internal/autoconfig/testdata/openai-github.yaml
@@ -125,7 +125,7 @@ spec:
       path: "/mcp/x/issues/readonly"
       toolSelector:
         include:
-          - get_issue
+          - issue_read
           - list_issues
       securityPolicy:
         apiKey:
diff --git a/site/blog/2025/2025-10-02-mcp-implementation.md b/site/blog/2025/2025-10-02-mcp-implementation.md
@@ -107,7 +107,7 @@ You can also use some features, like Tool Filtering, with the existing server fi
       "headers": {
         "Authorization": "Bearer ${GITHUB_ACCESS_TOKEN}"
       },
-      "tools": ["get_issue", "list_issues"]
+      "tools": ["issue_read", "list_issues"]
     }
   }
 }
diff --git a/site/docs/capabilities/mcp/index.md b/site/docs/capabilities/mcp/index.md
@@ -65,7 +65,7 @@ sequenceDiagram
 
 - **Session Management**: The gateway creates unified sessions by encoding multiple backend session IDs, handling reconnection with `Last-Event-ID` support for SSE streams.
 - **Notification Handling**: Long-lived SSE streams from multiple MCP servers are merged into a single stream for clients, with proper event ID reconstruction.
-- **Request Routing**: Tool names are automatically prefixed with the backend name (e.g., `github__get_issue`) to route calls to the correct upstream server.
+- **Request Routing**: Tool names are automatically prefixed with the backend name (e.g., `github__issue_read`) to route calls to the correct upstream server.
 
 For detailed architecture and design decisions, see the [MCP Gateway proposal](https://github.com/envoyproxy/ai-gateway/tree/main/docs/proposals/006-mcp-gateway).
 
@@ -151,7 +151,7 @@ spec:
       path: "/mcp/x/issues/readonly"
       toolSelector:
         includeRegex:
-          - .*_issues?.* # Matches get_issue, list_issues, etc.
+          - .*issues?.* # Matches issue_read, list_issues, etc.
       securityPolicy:
         apiKey:
           secretRef:
@@ -205,7 +205,7 @@ spec:
 
 Clients will see all tools with prefixed names:
 
-- `github__get_issue`
+- `github__issue_read`
 - `github__list_issues`
 - `context7__resolve-library-id`
 - `context7__get-library-docs`
diff --git a/site/docs/cli/run.md b/site/docs/cli/run.md
@@ -162,7 +162,7 @@ Use the `--mcp-config` flag to load MCP server configuration from a JSON file. T
       "headers": {
         "Authorization": "Bearer ${GITHUB_ACCESS_TOKEN}"
       },
-      "includeTools": ["get_issue", "list_issues"]
+      "includeTools": ["issue_read", "list_issues"]
     }
   }
 }
@@ -220,7 +220,7 @@ This follows the same convention as [Gemini CLI's tool filtering](https://google
 Example:
 
 ```json
-"includeTools": ["get_issue", "list_issues", "search_issues"]
+"includeTools": ["issue_read", "list_issues", "search_issues"]
 ```
 
 ### Testing the MCP Gateway
@@ -240,7 +240,7 @@ npx @modelcontextprotocol/inspector@0.16.8 \
 ```
 
 :::note
-Tool names are automatically prefixed with the server name (e.g., `github__get_issue`, `context7__resolve-library-id`) to route calls to the correct backend.
+Tool names are automatically prefixed with the server name (e.g., `github__issue_read`, `context7__resolve-library-id`) to route calls to the correct backend.
 :::
 
 ### Advanced MCP Configuration
diff --git a/tests/e2e-aigw/examples_mcp_test.go b/tests/e2e-aigw/examples_mcp_test.go
@@ -44,12 +44,10 @@ var (
 	filteredAllTools = []string{
 		"context7__get-library-docs",
 		"context7__resolve-library-id",
-		"github__get_issue",
-		"github__get_issue_comments",
+		"github__issue_read",
 		"github__list_issue_types",
 		"github__list_issues",
 		"github__list_pull_requests",
-		"github__list_sub_issues",
 		"github__pull_request_read",
 		"github__search_issues",
 		"github__search_pull_requests",
diff --git a/tests/extproc/mcp/publicmcp_test.go b/tests/extproc/mcp/publicmcp_test.go
@@ -88,13 +88,11 @@ func TestPublicMCPServers(t *testing.T) {
 		}
 
 		if githubConfigured {
-			exps = append(exps, "github__get_issue")
-			exps = append(exps, "github__get_issue_comments")
+			exps = append(exps, "github__issue_read")
 			exps = append(exps, "github__pull_request_read")
 			exps = append(exps, "github__list_issue_types")
 			exps = append(exps, "github__list_issues")
 			exps = append(exps, "github__list_pull_requests")
-			exps = append(exps, "github__list_sub_issues")
 			exps = append(exps, "github__search_issues")
 			exps = append(exps, "github__search_pull_requests")
 		}

Original file line number	Diff line number	Diff line change
`@@ -107,7 +107,7 @@ You can also use some features, like Tool Filtering, with the existing server fi`
`107`	`107`	`"headers": {`
`108`	`108`	`"Authorization": "Bearer ${GITHUB_ACCESS_TOKEN}"`
`109`	`109`	`},`
`110`		`- "tools": ["get_issue", "list_issues"]`
	`110`	`+ "tools": ["issue_read", "list_issues"]`
`111`	`111`	`}`
`112`	`112`	`}`
`113`	`113`	`}`
Original file line number	Diff line number	Diff line change
`@@ -88,13 +88,11 @@ func TestPublicMCPServers(t *testing.T) {`
`88`	`88`	`}`
`89`	`89`
`90`	`90`	`if githubConfigured {`
`91`		`- exps = append(exps, "github__get_issue")`
`92`		`- exps = append(exps, "github__get_issue_comments")`
	`91`	`+ exps = append(exps, "github__issue_read")`
`93`	`92`	`exps = append(exps, "github__pull_request_read")`
`94`	`93`	`exps = append(exps, "github__list_issue_types")`
`95`	`94`	`exps = append(exps, "github__list_issues")`
`96`	`95`	`exps = append(exps, "github__list_pull_requests")`
`97`		`- exps = append(exps, "github__list_sub_issues")`
`98`	`96`	`exps = append(exps, "github__search_issues")`
`99`	`97`	`exps = append(exps, "github__search_pull_requests")`
`100`	`98`	`}`