You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
generateInstructions uses AvailableToolsets instead of EnabledToolsetIDs #1897
In pkg/inventory/instructions.go, the generateInstructions() function iterates over inv.AvailableToolsets() which returns all toolsets that have tools defined, rather than only the enabled toolsets based on the WithToolsets() configuration.
// Collect instructions from each enabled toolsetfor_, toolset:=rangeinv.AvailableToolsets() { // <- BUG: should use enabled toolsetsiftoolset.InstructionsFunc!=nil {
iftoolsetInstructions:=toolset.InstructionsFunc(inv); toolsetInstructions!="" {
instructions=append(instructions, toolsetInstructions)
}
}
}
Expected Behavior
When building an inventory with WithToolsets([]string{"repos"}), the generated instructions should only include instructions for the repos toolset.
Actual Behavior
Instructions for all toolsets (context, issues, pull_requests, discussions, projects, etc.) are included regardless of which toolsets are enabled.
Evidence
From mcp-server-diff action comparing v0.29.0 (base) vs v0.30.1 (branch) with repos toolset only:
Base (v0.29.0): 1226 chars of instructions
Branch (v0.30.1): 5886 chars of instructions
The v0.30.1 instructions incorrectly include:
"Always call 'get_me' first..." (context toolset - not enabled)
Discussions instructions (not enabled)
Issues instructions (not enabled)
Projects instructions (not enabled)
How to Reproduce
Update the mcp-server-diff action to v2.2.0 to see server instruction diffs in CI
Build an inventory with a single toolset: WithToolsets([]string{"repos"}).WithServerInstructions()
Check inv.Instructions() - it will contain instructions for all toolsets, not just repos
Suggested Fix
Use EnabledToolsetIDs() and look up toolset metadata, or add a new EnabledToolsets() method that returns []ToolsetMetadata for only enabled toolsets:
Activity
Fix generateInstructions to use only enabled toolsets
Update mcp-server-diff action to v2.1.8
Update mcp-server-diff action to v2.2.0
SamMorrowDrums commentedon Jan 26, 2026
https://github.com/github/github-mcp-server/actions/runs/21366757011 verifies that this change is correct, and the bug is closed.
Fix generateInstructions to use only enabled toolsets
Update mcp-server-diff action to v2.2.0
Fix merge conflicts with main branch (#1949)