clean comments

Author: key4ng

sgl-router/src/mcp/manager.rs

@@ -130,9 +130,6 @@ impl McpManager {
                 },
             )
             .await?;
-
-        // Note: We don't add dynamic server tools to the inventory
-        // They are fetched directly from the client in list_tools_for_request()
         Ok(client)
     }
 
@@ -165,14 +162,6 @@ impl McpManager {
     ///
     /// Returns a HashMap mapping (server_label, tool_name) -> (Tool, server_url)
     /// Includes both static tools (from config) and dynamic tools (from request servers)
-    ///
-    /// # Arguments
-    /// * `dynamic_servers` - List of (server_label, server_url) pairs from the request
-    ///
-    /// # Returns
-    /// HashMap where:
-    /// - Key: (server_label, tool_name) - composite key for unique identification
-    /// - Value: (Tool, server_url) - tool definition and URL for execution
     pub async fn list_tools_for_request(
         &self,
         dynamic_servers: &[(String, String)],
@@ -261,17 +250,6 @@ impl McpManager {
     ///
     /// This method is used for dynamic MCP servers where the tool is identified
     /// by both server_url and tool_name (rather than from the static inventory).
-    ///
-    /// # Arguments
-    /// * `server_url` - URL of the MCP server to call the tool on
-    /// * `tool_name` - Name of the tool to call
-    /// * `args_map` - Optional map of arguments to pass to the tool
-    ///
-    /// # Returns
-    /// Tuple of (output_str, success, error):
-    /// - output_str: JSON string of the result or error message
-    /// - success: true if the call succeeded, false otherwise
-    /// - error: Optional error message if the call failed
     pub async fn call_tool_by_url(
         &self,
         server_url: &str,