Skip to content

Conversation

@maartenvandiemen
Copy link

@maartenvandiemen maartenvandiemen commented Nov 2, 2025

In a recent change of the GitHub MCP Server some tools are not enabled anymore by default. You need to explicitly define them: https://github.com/github/github-mcp-server?tab=readme-ov-file#tool-configuration.

With this change I introduced an extra optional parameter in which you can pass the tools that the caller want the have enabled.

In issue #138 this is the issue, since the tools for GH Actions aren't enabled by default anymore.

@maartenvandiemen maartenvandiemen requested a review from a team as a code owner November 2, 2025 22:52
Copilot AI review requested due to automatic review settings November 2, 2025 22:52
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds support for configuring GitHub MCP toolsets by introducing a new github-mcp-toolsets input parameter. This allows users to specify which toolsets (e.g., repos, issues, pull_requests, actions, code_security) should be enabled when connecting to the GitHub MCP server, providing more granular control over available tools.

Key changes:

  • Added github-mcp-toolsets input parameter to action.yml with detailed description
  • Modified connectToGitHubMCP function to accept optional toolsets parameter and include it in the X-MCP-Toolsets header when specified
  • Updated README documentation with usage examples and reference to available toolsets
  • Added comprehensive test coverage for the new functionality

Reviewed Changes

Copilot reviewed 6 out of 8 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/mcp.ts Added optional toolsets parameter to connectToGitHubMCP function and logic to conditionally set X-MCP-Toolsets header
src/main.ts Retrieved github-mcp-toolsets input and passed it to connectToGitHubMCP function
dist/index.js Compiled JavaScript output reflecting the TypeScript changes
action.yml Added new github-mcp-toolsets input with comprehensive description of available options
tests/mcp.test.ts Added three new test cases covering default, custom, and empty toolsets scenarios
tests/main.test.ts Updated existing test expectations to include empty string for toolsets parameter
README.md Added new section documenting toolsets configuration with examples, updated version references from v1/v1.2 to v2, and updated inputs table

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@abhivarma007
Copy link

@maartenvandiemen thanks for creating the fix to this issue, can I expect by when this will be resolved?

@maartenvandiemen
Copy link
Author

maartenvandiemen commented Nov 3, 2025

@maartenvandiemen thanks for creating the fix to this issue, can I expect by when this will be resolved?

That's depends on the feedback on this PR and when I get it. For the time being I use the version of my PR. Example:

      - name: Run AI inference with prompt file
        id: ai
        uses: maartenvandiemen/ai-inference@feature/pass-toolsets
        with:

Hopefully your policies allow this. Once this PR is merged you will notice it soon enough, the branch will be deleted.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants