Remove last remnants of this

Author: omgitsads

internal/ghmcp/server.go

@@ -15,7 +15,6 @@ import (
 	"github.com/github/github-mcp-server/pkg/github"
 	mcplog "github.com/github/github-mcp-server/pkg/log"
 	"github.com/github/github-mcp-server/pkg/raw"
-	"github.com/github/github-mcp-server/pkg/toolsets"
 	"github.com/github/github-mcp-server/pkg/translations"
 	gogithub "github.com/google/go-github/v72/github"
 	"github.com/mark3labs/mcp-go/mcp"
@@ -47,9 +46,6 @@ type MCPServerConfig struct {
 
 	// Translator provides translated text for the server tooling
 	Translator translations.TranslationHelperFunc
-
-	// ToolHandlerWrapper is a function that can be used to wrap tool handlers
-	ToolHandlerWrapper toolsets.ToolHandlerWrapper
 }
 
 func NewMCPServer(cfg MCPServerConfig) (*server.MCPServer, error) {

pkg/toolsets/toolsets.go

@@ -33,8 +33,6 @@ func NewServerTool(tool mcp.Tool, handler server.ToolHandlerFunc) server.ServerT
 	return server.ServerTool{Tool: tool, Handler: handler}
 }
 
-type ToolHandlerWrapper func(handler server.ToolHandlerFunc) server.ToolHandlerFunc
-
 func NewServerResourceTemplate(resourceTemplate mcp.ResourceTemplate, handler server.ResourceTemplateHandlerFunc) ServerResourceTemplate {
 	return ServerResourceTemplate{
 		resourceTemplate: resourceTemplate,
@@ -57,7 +55,6 @@ type Toolset struct {
 	writeTools  []server.ServerTool
 	readTools   []server.ServerTool
 
-	ToolHandler ToolHandlerWrapper
 	// resources are not tools, but the community seems to be moving towards namespaces as a broader concept
 	// and in order to have multiple servers running concurrently, we want to avoid overlapping resources too.
 	resourceTemplates []ServerResourceTemplate