refactor: apply safe api lint cleanup

Author: francescobianco

src/openapi_mcp_sdk/apis/async_tool.py

@@ -1,16 +1,22 @@
+"""Async status tools."""
+
 import logging
-logging.getLogger(__name__).debug("module loaded")
-from fastmcp import Context
 from typing import Any
+
+from fastmcp import Context  # pylint: disable=import-error
+
 from ..mcp_core import mcp
 from ..memory_store import get_callback_result
 
 logger = logging.getLogger(__name__)
+logger.debug("module loaded")
+
 
 @mcp.tool
 async def check_async_status(request_id: str, ctx: Context) -> Any:
     """Show the last status of an async request
     Args:
         request_id: required, returned by an async downgraded request to the mcp server
     """
+    del ctx
     return get_callback_result(request_id)

src/openapi_mcp_sdk/apis/automotive.py

@@ -1,20 +1,37 @@
+"""Automotive tools."""
+
+# MCP tool names and parameter names intentionally mirror the public API.
+# pylint: disable=invalid-name,redefined-builtin
+
 import logging
-logging.getLogger(__name__).debug("module loaded")
-from fastmcp import Context
 from typing import Any
+
+from fastmcp import Context  # pylint: disable=import-error
+
 from ..mcp_core import make_api_call, mcp
 from ..memory_store import OPENAPI_HOST_PREFIX
 
 logger = logging.getLogger(__name__)
+logger.debug("module loaded")
+
 
 @mcp.tool
-async def check_license_plate(countryCode: str, type: str, licensePlate: str, ctx: Context) -> Any:
-    """Retrieve informations about type=car,bike,insurance,mot from a countryCode=IT,FR,UK,DE,PT,ES and a licensePlate.
-    Available Combinations: IT-car, IT-bike, IT-insurance,FR-car,FR-bike,UK-car,UK-bike,UK-mot,PT-car,PT-insurance,ES-car,ES-bike
+async def check_license_plate(
+    countryCode: str, type: str, licensePlate: str, ctx: Context
+) -> Any:
+    """Retrieve vehicle data for a supported country and plate.
+
+    Available combinations:
+    IT-car, IT-bike, IT-insurance, FR-car, FR-bike, UK-car, UK-bike,
+    UK-mot, PT-car, PT-insurance, ES-car, ES-bike.
+
     Args:
         countryCode: required, 2 digit country code (IT|FR|UK|DE|PT|ES)
         type: required, type of information needed (car|bike|insurance|mot)
         licensePlate: required, the license plate to check
     """
-    url = f"https://{OPENAPI_HOST_PREFIX}automotive.openapi.com/{countryCode}-{type}/{licensePlate}"
+    url = (
+        f"https://{OPENAPI_HOST_PREFIX}automotive.openapi.com/"
+        f"{countryCode}-{type}/{licensePlate}"
+    )
     return make_api_call(ctx, "GET", url)

src/openapi_mcp_sdk/apis/cap.py

@@ -1,11 +1,19 @@
+"""Italian postal code and municipality tools."""
+
+# MCP tool names and parameter names intentionally mirror the public API.
+# pylint: disable=invalid-name
+
 import logging
-logging.getLogger(__name__).debug("module loaded")
-from fastmcp import Context
 from typing import Any
+
+from fastmcp import Context  # pylint: disable=import-error
+
 from ..mcp_core import make_api_call, mcp
 from ..memory_store import OPENAPI_HOST_PREFIX
 
 logger = logging.getLogger(__name__)
+logger.debug("module loaded")
+
 
 @mcp.tool
 async def get_IT_regions_list(ctx: Context) -> Any:
@@ -15,6 +23,7 @@ async def get_IT_regions_list(ctx: Context) -> Any:
     url = f"https://{OPENAPI_HOST_PREFIX}cap.openapi.it/regioni"
     return make_api_call(ctx, "GET", url)
 
+
 @mcp.tool
 async def get_IT_provinces_list(ctx: Context) -> Any:
     """
@@ -23,6 +32,7 @@ async def get_IT_provinces_list(ctx: Context) -> Any:
     url = f"https://{OPENAPI_HOST_PREFIX}cap.openapi.it/province"
     return make_api_call(ctx, "GET", url)
 
+
 @mcp.tool
 async def get_IT_metropolitan_cities_list(ctx: Context) -> Any:
     """
@@ -31,6 +41,7 @@ async def get_IT_metropolitan_cities_list(ctx: Context) -> Any:
     url = f"https://{OPENAPI_HOST_PREFIX}cap.openapi.it/citta_metropolitane"
     return make_api_call(ctx, "GET", url)
 
+
 @mcp.tool
 async def get_suppressed_italian_municipalities(ctx: Context) -> Any:
     """
@@ -39,6 +50,7 @@ async def get_suppressed_italian_municipalities(ctx: Context) -> Any:
     url = f"https://{OPENAPI_HOST_PREFIX}cap.openapi.it/comuni_soppressi"
     return make_api_call(ctx, "GET", url)
 
+
 @mcp.tool
 async def find_IT_istat_by_comune_name(comune: str, ctx: Context) -> Any:
     """
@@ -48,6 +60,7 @@ async def find_IT_istat_by_comune_name(comune: str, ctx: Context) -> Any:
     params = {"comune": comune}
     return make_api_call(ctx, "GET", url, params=params)
 
+
 @mcp.tool
 async def find_IT_municipality_by_istat(istatCode: str, ctx: Context) -> Any:
     """
@@ -76,6 +89,7 @@ async def find_IT_municipality_by_istat(istatCode: str, ctx: Context) -> Any:
     url = f"https://{OPENAPI_HOST_PREFIX}cap.openapi.it/comuni_advance/{istatCode}"
     return make_api_call(ctx, "GET", url)
 
+
 @mcp.tool
 async def find_IT_municipalities_by_zip(zip_code: str, ctx: Context) -> Any:
     """