fix: resolve code formatting and type checking issues

- Fix black formatting for server.py and test_custom_endpoints.py
- Remove unused import (patch) from test file
- Add proper type annotations for all async test functions
- Fix return type for apis_discovery endpoint (Response vs JSONResponse)
- Add noqa comment for acceptable long line in f-string
- All linting and type checking now passes locally:
  * black --check ✓
  * isort --check-only ✓
  * flake8 ✓
  * mypy (for modified files) ✓
  * pytest: 92 passed, 2 skipped, 96% coverage ✓

Author: shibbirmcc

src/kong_mcp_server/server.py

@@ -9,7 +9,7 @@
 
 from fastmcp import FastMCP
 from starlette.requests import Request
-from starlette.responses import JSONResponse
+from starlette.responses import JSONResponse, Response
 
 mcp = FastMCP("Kong Rate Limiter MCP Server")
 
@@ -39,7 +39,7 @@ async def api_discovery(request: Request) -> JSONResponse:
 
 
 @mcp.custom_route("/apis", methods=["GET"])
-async def apis_discovery(request: Request) -> JSONResponse:
+async def apis_discovery(request: Request) -> Response:
     """Alternative API discovery endpoint for compatibility."""
     return await api_discovery(request)
 

tests/test_custom_endpoints.py

@@ -11,7 +11,7 @@
 
 
 @pytest.mark.asyncio
-async def test_api_discovery():
+async def test_api_discovery() -> None:
     """Test /api discovery endpoint."""
     request = Mock(spec=Request)
     response = await api_discovery(request)
@@ -29,7 +29,7 @@ async def test_api_discovery():
 
 
 @pytest.mark.asyncio
-async def test_apis_discovery():
+async def test_apis_discovery() -> None:
     """Test /apis alternative discovery endpoint."""
     request = Mock(spec=Request)
     response = await apis_discovery(request)
@@ -40,7 +40,7 @@ async def test_apis_discovery():
 
 
 @pytest.mark.asyncio
-async def test_sse_ping():
+async def test_sse_ping() -> None:
     """Test /sse/ping endpoint."""
     request = Mock(spec=Request)
     response = await sse_ping(request)
@@ -56,11 +56,11 @@ async def test_sse_ping():
 
 
 @pytest.mark.asyncio
-async def test_sse_request_default():
+async def test_sse_request_default() -> None:
     """Test /sse/request endpoint with default request."""
     request = Mock(spec=Request)
 
-    async def mock_json():
+    async def mock_json() -> dict[str, str | int]:
         return {"jsonrpc": "2.0", "method": "test_method", "id": 1}
 
     request.json = mock_json
@@ -77,11 +77,11 @@ async def mock_json():
 
 
 @pytest.mark.asyncio
-async def test_sse_request_tool_call():
+async def test_sse_request_tool_call() -> None:
     """Test /sse/request endpoint with tool call."""
     request = Mock(spec=Request)
 
-    async def mock_json():
+    async def mock_json() -> dict[str, str | int | dict[str, str]]:
         return {
             "jsonrpc": "2.0",
             "method": "tools/call",
@@ -104,7 +104,7 @@ async def mock_json():
 
 
 @pytest.mark.asyncio
-async def test_sse_request_error_handling():
+async def test_sse_request_error_handling() -> None:
     """Test /sse/request endpoint error handling."""
     request = Mock(spec=Request)
     request.json = Mock(side_effect=Exception("Invalid JSON"))