keep scope minial

hungran · hungran · commit 20d248cd6260 · 2025-07-11T20:11:22.000+07:00
Signed-off-by: hungran &lt;26101787+hungran@users.noreply.github.com&gt;
diff --git a/internal/ghmcp/server.go b/internal/ghmcp/server.go
@@ -141,11 +141,11 @@ func NewMCPServer(cfg MCPServerConfig) (*server.MCPServer, error) {
 	}
 
 	// Register all mcp functionality with the server
-	tsg.RegisterAllWithPrefix(ghServer, cfg.ToolPrefix)
+	tsg.RegisterAll(ghServer, cfg.ToolPrefix)
 
 	if cfg.DynamicToolsets {
 		dynamic := github.InitDynamicToolset(ghServer, tsg, cfg.Translator)
-		dynamic.RegisterToolsWithPrefix(ghServer, cfg.ToolPrefix)
+		dynamic.RegisterTools(ghServer, cfg.ToolPrefix)
 	}
 
 	return ghServer, nil
diff --git a/pkg/toolsets/toolsets.go b/pkg/toolsets/toolsets.go
@@ -103,11 +103,7 @@ func (t *Toolset) GetAvailableTools() []server.ServerTool {
 	return append(t.readTools, t.writeTools...)
 }
 
-func (t *Toolset) RegisterTools(s *server.MCPServer) {
-	t.RegisterToolsWithPrefix(s, "")
-}
-
-func (t *Toolset) RegisterToolsWithPrefix(s *server.MCPServer, prefix string) {
+func (t *Toolset) RegisterTools(s *server.MCPServer, prefix string) {
 	if !t.Enabled {
 		return
 	}
@@ -276,13 +272,9 @@ func (tg *ToolsetGroup) EnableToolset(name string) error {
 	return nil
 }
 
-func (tg *ToolsetGroup) RegisterAll(s *server.MCPServer) {
-	tg.RegisterAllWithPrefix(s, "")
-}
-
-func (tg *ToolsetGroup) RegisterAllWithPrefix(s *server.MCPServer, prefix string) {
+func (tg *ToolsetGroup) RegisterAll(s *server.MCPServer, prefix string) {
 	for _, toolset := range tg.Toolsets {
-		toolset.RegisterToolsWithPrefix(s, prefix)
+		toolset.RegisterTools(s, prefix)
 		toolset.RegisterResourcesTemplates(s)
 		toolset.RegisterPrompts(s)
 	}

Original file line number	Diff line number	Diff line change
`@@ -141,11 +141,11 @@ func NewMCPServer(cfg MCPServerConfig) (*server.MCPServer, error) {`
`141`	`141`	`}`
`142`	`142`
`143`	`143`	`// Register all mcp functionality with the server`
`144`		`- tsg.RegisterAllWithPrefix(ghServer, cfg.ToolPrefix)`
	`144`	`+ tsg.RegisterAll(ghServer, cfg.ToolPrefix)`
`145`	`145`
`146`	`146`	`if cfg.DynamicToolsets {`
`147`	`147`	`dynamic := github.InitDynamicToolset(ghServer, tsg, cfg.Translator)`
`148`		`- dynamic.RegisterToolsWithPrefix(ghServer, cfg.ToolPrefix)`
	`148`	`+ dynamic.RegisterTools(ghServer, cfg.ToolPrefix)`
`149`	`149`	`}`
`150`	`150`
`151`	`151`	`return ghServer, nil`