remove unused imports and clean up module docstrings

Author: shahar4499

fastapi_mcp/__init__.py

@@ -13,14 +13,10 @@
     __version__ = "0.0.0.dev0"
 
 from .server import add_mcp_server, create_mcp_server, mount_mcp_server
-from .openapi.convert import convert_openapi_to_mcp_tools
-from .execute import execute_api_tool
 
 
 __all__ = [
     "add_mcp_server",
     "create_mcp_server",
     "mount_mcp_server",
-    "convert_openapi_to_mcp_tools",
-    "execute_api_tool",
 ]

fastapi_mcp/execute.py

@@ -1,10 +1,3 @@
-"""
-Direct OpenAPI to MCP Tools Conversion Module.
-
-This module provides functionality for directly converting OpenAPI schema to MCP tool specifications
-and for executing HTTP tools.
-"""
-
 import json
 import logging
 from typing import Any, Dict, List, Union
@@ -14,7 +7,7 @@
 import mcp.types as types
 
 
-logger = logging.getLogger("fastapi_mcp")
+logger = logging.getLogger(__name__)
 
 
 async def execute_api_tool(

fastapi_mcp/openapi/convert.py

@@ -1,10 +1,3 @@
-"""
-Direct OpenAPI to MCP Tools Conversion Module.
-
-This module provides functionality for directly converting OpenAPI schema to MCP tool specifications
-and for executing HTTP tools.
-"""
-
 import json
 import logging
 from typing import Any, Dict, List, Tuple
@@ -18,7 +11,7 @@
     get_single_param_type_from_schema,
 )
 
-logger = logging.getLogger("fastapi_mcp")
+logger = logging.getLogger(__name__)
 
 
 def convert_openapi_to_mcp_tools(
@@ -50,11 +43,13 @@ def convert_openapi_to_mcp_tools(
         for method, operation in path_item.items():
             # Skip non-HTTP methods
             if method not in ["get", "post", "put", "delete", "patch"]:
+                logger.warning(f"Skipping non-HTTP method: {method}")
                 continue
 
             # Get operation metadata
             operation_id = operation.get("operationId")
             if not operation_id:
+                logger.warning(f"Skipping operation with no operationId: {operation}")
                 continue
 
             # Save operation details for later HTTP calls
@@ -78,8 +73,8 @@ def convert_openapi_to_mcp_tools(
             if responses:
                 response_info = "\n\n### Responses:\n"
 
-                # Find the success response (usually 200 or 201)
-                success_codes = ["200", "201", "202", 200, 201, 202]
+                # Find the success response
+                success_codes = range(200, 300)
                 success_response = None
                 for status_code in success_codes:
                     if str(status_code) in responses:

fastapi_mcp/openapi/utils.py

@@ -1,9 +1,3 @@
-"""
-OpenAPI utility functions for FastAPI-MCP.
-
-This module provides utility functions for working with OpenAPI schemas.
-"""
-
 from typing import Any, Dict, List, Optional
 
 

fastapi_mcp/server.py

@@ -1,9 +1,3 @@
-"""
-Server module for FastAPI-MCP.
-
-This module provides functionality for creating and mounting MCP servers to FastAPI applications.
-"""
-
 from contextlib import asynccontextmanager
 from typing import Dict, Optional, Any, Tuple, List, Union, AsyncIterator