fix: resolve code formatting and linting issues

- Fix black formatting for server.py and test_custom_endpoints.py
- Remove unused import (patch) from test file
- Add noqa comment for acceptable long line in f-string
- All linting checks now pass (black, isort, flake8)
- Tests still passing (92 passed, 2 skipped, 96% coverage)

Author: shibbirmcc

src/kong_mcp_server/server.py

@@ -18,22 +18,24 @@
 @mcp.custom_route("/api", methods=["GET"])
 async def api_discovery(request: Request) -> JSONResponse:
     """Standard MCP API discovery endpoint."""
-    return JSONResponse({
-        "name": "Kong Rate Limiter MCP Server",
-        "version": "0.1.2",
-        "protocol_version": "2025-06-18",
-        "capabilities": {
-            "tools": True,
-            "resources": False,
-            "prompts": False,
-            "sampling": False
-        },
-        "endpoints": {
-            "sse": "/sse/",
-            "ping": "/sse/ping",
-            "request": "/sse/request"
+    return JSONResponse(
+        {
+            "name": "Kong Rate Limiter MCP Server",
+            "version": "0.1.2",
+            "protocol_version": "2025-06-18",
+            "capabilities": {
+                "tools": True,
+                "resources": False,
+                "prompts": False,
+                "sampling": False,
+            },
+            "endpoints": {
+                "sse": "/sse/",
+                "ping": "/sse/ping",
+                "request": "/sse/request",
+            },
         }
-    })
+    )
 
 
 @mcp.custom_route("/apis", methods=["GET"])
@@ -45,52 +47,53 @@ async def apis_discovery(request: Request) -> JSONResponse:
 @mcp.custom_route("/sse/ping", methods=["GET", "POST"])
 async def sse_ping(request: Request) -> JSONResponse:
     """SSE ping endpoint for health checks."""
-    return JSONResponse({
-        "jsonrpc": "2.0",
-        "method": "ping",
-        "result": {
-            "status": "ok",
-            "timestamp": time.time()
+    return JSONResponse(
+        {
+            "jsonrpc": "2.0",
+            "method": "ping",
+            "result": {"status": "ok", "timestamp": time.time()},
         }
-    })
+    )
 
 
 @mcp.custom_route("/sse/request", methods=["POST"])
 async def sse_request(request: Request) -> JSONResponse:
     """SSE request endpoint for MCP JSON-RPC messages."""
     try:
         body = await request.json()
-        
+
         # Handle tool calls
         if body.get("method") == "tools/call":
             tool_name = body.get("params", {}).get("name")
             if tool_name:
-                return JSONResponse({
-                    "jsonrpc": "2.0",
-                    "id": body.get("id"),
-                    "result": {
-                        "content": [
-                            {"type": "text", "text": f"Tool {tool_name} executed via SSE request"}
-                        ]
+                return JSONResponse(
+                    {
+                        "jsonrpc": "2.0",
+                        "id": body.get("id"),
+                        "result": {
+                            "content": [
+                                {
+                                    "type": "text",
+                                    "text": f"Tool {tool_name} executed via SSE request",  # noqa: E501
+                                }
+                            ]
+                        },
                     }
-                })
-        
+                )
+
         # Default response for other requests
-        return JSONResponse({
-            "jsonrpc": "2.0",
-            "id": body.get("id"),
-            "result": {"status": "received"}
-        })
-        
+        return JSONResponse(
+            {"jsonrpc": "2.0", "id": body.get("id"), "result": {"status": "received"}}
+        )
+
     except Exception as e:
-        return JSONResponse({
-            "jsonrpc": "2.0",
-            "error": {
-                "code": -32603,
-                "message": "Internal error",
-                "data": str(e)
-            }
-        }, status_code=500)
+        return JSONResponse(
+            {
+                "jsonrpc": "2.0",
+                "error": {"code": -32603, "message": "Internal error", "data": str(e)},
+            },
+            status_code=500,
+        )
 
 
 def load_tools_config() -> Dict[str, Any]:

tests/test_custom_endpoints.py

@@ -1,7 +1,7 @@
 """Tests for custom MCP endpoints added to the server."""
 
 import json
-from unittest.mock import Mock, patch
+from unittest.mock import Mock
 
 import pytest
 from starlette.requests import Request
@@ -15,10 +15,10 @@ async def test_api_discovery():
     """Test /api discovery endpoint."""
     request = Mock(spec=Request)
     response = await api_discovery(request)
-    
+
     assert isinstance(response, JSONResponse)
     assert response.status_code == 200
-    
+
     # Parse response body
     body = json.loads(response.body)
     assert body["name"] == "Kong Rate Limiter MCP Server"
@@ -33,7 +33,7 @@ async def test_apis_discovery():
     """Test /apis alternative discovery endpoint."""
     request = Mock(spec=Request)
     response = await apis_discovery(request)
-    
+
     # Should be identical to api_discovery
     api_response = await api_discovery(request)
     assert response.body == api_response.body
@@ -44,10 +44,10 @@ async def test_sse_ping():
     """Test /sse/ping endpoint."""
     request = Mock(spec=Request)
     response = await sse_ping(request)
-    
+
     assert isinstance(response, JSONResponse)
     assert response.status_code == 200
-    
+
     body = json.loads(response.body)
     assert body["jsonrpc"] == "2.0"
     assert body["method"] == "ping"
@@ -59,21 +59,17 @@ async def test_sse_ping():
 async def test_sse_request_default():
     """Test /sse/request endpoint with default request."""
     request = Mock(spec=Request)
-    
+
     async def mock_json():
-        return {
-            "jsonrpc": "2.0",
-            "method": "test_method",
-            "id": 1
-        }
-    
+        return {"jsonrpc": "2.0", "method": "test_method", "id": 1}
+
     request.json = mock_json
-    
+
     response = await sse_request(request)
-    
+
     assert isinstance(response, JSONResponse)
     assert response.status_code == 200
-    
+
     body = json.loads(response.body)
     assert body["jsonrpc"] == "2.0"
     assert body["id"] == 1
@@ -84,22 +80,22 @@ async def mock_json():
 async def test_sse_request_tool_call():
     """Test /sse/request endpoint with tool call."""
     request = Mock(spec=Request)
-    
+
     async def mock_json():
         return {
             "jsonrpc": "2.0",
             "method": "tools/call",
             "params": {"name": "test_tool"},
-            "id": 1
+            "id": 1,
         }
-    
+
     request.json = mock_json
-    
+
     response = await sse_request(request)
-    
+
     assert isinstance(response, JSONResponse)
     assert response.status_code == 200
-    
+
     body = json.loads(response.body)
     assert body["jsonrpc"] == "2.0"
     assert body["id"] == 1
@@ -112,13 +108,13 @@ async def test_sse_request_error_handling():
     """Test /sse/request endpoint error handling."""
     request = Mock(spec=Request)
     request.json = Mock(side_effect=Exception("Invalid JSON"))
-    
+
     response = await sse_request(request)
-    
+
     assert isinstance(response, JSONResponse)
     assert response.status_code == 500
-    
+
     body = json.loads(response.body)
     assert body["jsonrpc"] == "2.0"
     assert "error" in body
-    assert body["error"]["code"] == -32603
+    assert body["error"]["code"] == -32603