Repository navigation
Conversation
| class GitHubMCPServer: | ||
| def __init__( | ||
| self, | ||
| binary: str = "github-mcp-server", |
There was a problem hiding this comment.
Looking at github-mcp-server, we might be able to replace all of this by deploying the official container as a service in our GCP project. That would do the work for us with almost no code on our side.
There was a problem hiding this comment.
sounds good. replaced the wrapper with a docker compose setup that runs the official gh mcp container in read-only mode for local development. I tested it against a test PR and confirmed it can read the PR and expose read-only tools. iiuc, we can connect an agent to it as part of the GitHub integration, and use the GCP service once it’s deployed
029924f to
3dfd594
Compare
There was a problem hiding this comment.
I just wanted to keep it separate from any particular agent, but I've now moved it to services/github-mcp
There was a problem hiding this comment.
What is the goal if this check?
There was a problem hiding this comment.
just a smoke test to verify that the server can read a PR and check for the tools. Forgot to remove the hardcoded test repo before opening the PR 😅 but I could also remove this if you don't think it's valuable
There was a problem hiding this comment.
We could drop it. We will later have that tested as part of the integration with the agents anyway.
3dfd594 to
2fb67e7
Compare
There was a problem hiding this comment.
We could drop it. We will later have that tested as part of the integration with the agents anyway.
| services: | ||
| github-mcp: | ||
| image: ghcr.io/github/github-mcp-server:v2.0.3 | ||
| command: ["http", "--read-only", "--toolsets=repos,pull_requests,issues"] |
There was a problem hiding this comment.
We might need to add env here to pass the auth to the mcp directly instead of having the part of the mcp request if that is supported.
There was a problem hiding this comment.
I might be misunderstanding this, but from the GitHub App auth docs, it looks like the server’s built-in App auth may only work in stdio mode, and HTTP clients need to provide a token. I also tried setting GITHUB_PERSONAL_ACCESS_TOKEN on the container, but got a 401 without an auth header. Maybe we need another way to supply the token for the GCP service?
Related to #7055
Description
Add a Docker Compose setup for running the official GitHub MCP server locally in read-only mode. The service runs independently of any agent and exposes repository, pull request, and issue tools.
Notes
Testing