reduce duplicate read write tool loops

Signed-off-by: hungran <[email protected]>

Author: hungran

pkg/toolsets/toolsets.go

@@ -107,22 +107,21 @@ func (t *Toolset) RegisterTools(s *server.MCPServer, prefix string) {
 	if !t.Enabled {
 		return
 	}
-	for _, tool := range t.readTools {
+	registerToolWithPrefix := func(tool server.ServerTool) {
 		toolToRegister := tool.Tool
 		if prefix != "" {
 			// Create a new tool with the prefixed name
 			toolToRegister = createToolWithPrefixedName(tool.Tool, prefix)
 		}
 		s.AddTool(toolToRegister, tool.Handler)
 	}
+
+	for _, tool := range t.readTools {
+		registerToolWithPrefix(tool)
+	}
 	if !t.readOnly {
 		for _, tool := range t.writeTools {
-			toolToRegister := tool.Tool
-			if prefix != "" {
-				// Create a new tool with the prefixed name
-				toolToRegister = createToolWithPrefixedName(tool.Tool, prefix)
-			}
-			s.AddTool(toolToRegister, tool.Handler)
+			registerToolWithPrefix(tool)
 		}
 	}
 }