remove unused function

Author: tommaso-moro

internal/ghmcp/server.go

@@ -176,7 +176,6 @@ func NewMCPServer(cfg MCPServerConfig) (*mcp.Server, error) {
 
 	// Register specific tools if configured
 	if len(cfg.EnabledTools) > 0 {
-		// Clean tool names and resolve deprecated aliases
 		enabledTools := github.CleanTools(cfg.EnabledTools)
 		enabledTools, _ = tsg.ResolveToolAliases(enabledTools)
 

pkg/toolsets/toolsets.go

@@ -193,7 +193,7 @@ func (t *Toolset) AddReadTools(tools ...ServerTool) *Toolset {
 
 type ToolsetGroup struct {
 	Toolsets          map[string]*Toolset
-	deprecatedAliases map[string]string // oldName → newName for renamed tools
+	deprecatedAliases map[string]string
 	everythingOn      bool
 	readOnly          bool
 }
@@ -213,13 +213,6 @@ func (tg *ToolsetGroup) AddDeprecatedToolAliases(aliases map[string]string) {
 	}
 }
 
-// IsDeprecatedToolAlias checks if a tool name is a deprecated alias.
-// Returns the canonical name and true if it's an alias, or empty string and false otherwise.
-func (tg *ToolsetGroup) IsDeprecatedToolAlias(toolName string) (canonicalName string, isAlias bool) {
-	canonicalName, isAlias = tg.deprecatedAliases[toolName]
-	return canonicalName, isAlias
-}
-
 func (tg *ToolsetGroup) AddToolset(ts *Toolset) {
 	if tg.readOnly {
 		ts.SetReadOnly()

pkg/toolsets/toolsets_test.go

@@ -302,29 +302,6 @@ func TestAddDeprecatedToolAliases(t *testing.T) {
 	}
 }
 
-func TestIsDeprecatedToolAlias(t *testing.T) {
-	tsg := NewToolsetGroup(false)
-	tsg.AddDeprecatedToolAliases(map[string]string{"old_tool": "new_tool"})
-
-	// Test with a deprecated alias
-	canonical, isAlias := tsg.IsDeprecatedToolAlias("old_tool")
-	if !isAlias {
-		t.Error("expected 'old_tool' to be recognized as an alias")
-	}
-	if canonical != "new_tool" {
-		t.Errorf("expected canonical name 'new_tool', got '%s'", canonical)
-	}
-
-	// Test with a non-alias
-	canonical, isAlias = tsg.IsDeprecatedToolAlias("some_other_tool")
-	if isAlias {
-		t.Error("expected 'some_other_tool' to not be an alias")
-	}
-	if canonical != "" {
-		t.Errorf("expected empty canonical name, got '%s'", canonical)
-	}
-}
-
 func TestResolveToolAliases(t *testing.T) {
 	tsg := NewToolsetGroup(false)
 	tsg.AddDeprecatedToolAliases(map[string]string{