remove both default and all

tommaso-moro · tommaso-moro · commit c683f9e5598c · 2025-11-13T12:51:11.000Z
diff --git a/internal/ghmcp/server.go b/internal/ghmcp/server.go
@@ -111,9 +111,10 @@ func NewMCPServer(cfg MCPServerConfig) (*server.MCPServer, error) {
 
 	enabledToolsets := cfg.EnabledToolsets
 
-	// If dynamic toolsets are enabled, remove "all" from the enabled toolsets
+	// If dynamic toolsets are enabled, remove "all" and "default" from the enabled toolsets
 	if cfg.DynamicToolsets {
 		enabledToolsets = github.RemoveToolset(enabledToolsets, github.ToolsetMetadataAll.ID)
+		enabledToolsets = github.RemoveToolset(enabledToolsets, github.ToolsetMetadataDefault.ID)
 	}
 
 	// Clean up the passed toolsets

Original file line number	Diff line number	Diff line change
`@@ -111,9 +111,10 @@ func NewMCPServer(cfg MCPServerConfig) (*server.MCPServer, error) {`
`111`	`111`
`112`	`112`	`enabledToolsets := cfg.EnabledToolsets`
`113`	`113`
`114`		`- // If dynamic toolsets are enabled, remove "all" from the enabled toolsets`
	`114`	`+ // If dynamic toolsets are enabled, remove "all" and "default" from the enabled toolsets`
`115`	`115`	`if cfg.DynamicToolsets {`
`116`	`116`	`enabledToolsets = github.RemoveToolset(enabledToolsets, github.ToolsetMetadataAll.ID)`
	`117`	`+ enabledToolsets = github.RemoveToolset(enabledToolsets, github.ToolsetMetadataDefault.ID)`
`117`	`118`	`}`
`118`	`119`
`119`	`120`	`// Clean up the passed toolsets`