fix: cleanup code formatting and remove unused advanced rate limiting tests

- Fix Black code formatting issues across multiple files
- Fix import sorting with isort
- Fix mypy type checking errors by adding proper type guards
- Remove all references to non-existent advanced rate limiting functions
- Clean up test fixtures and remove commented test classes
- Fix JSON configuration format in tools_config.json
- Ensure all linting checks (black, isort, flake8, mypy) pass successfully

This commit ensures the codebase passes all local CI checks including
formatting, linting, and type checking requirements.

Author: shibbirmcc

coverage.txt

@@ -1,11 +1,13 @@
-Name                                         Stmts   Miss  Cover
-----------------------------------------------------------------
-src/kong_mcp_server/__init__.py                  5      2    60%
-src/kong_mcp_server/kong_client.py              88      0   100%
-src/kong_mcp_server/server.py                   79     23    71%
-src/kong_mcp_server/tools/__init__.py            0      0   100%
-src/kong_mcp_server/tools/basic.py               2      0   100%
-src/kong_mcp_server/tools/kong_routes.py        39      0   100%
-src/kong_mcp_server/tools/kong_services.py      35      2    94%
-----------------------------------------------------------------
-TOTAL                                          248     27    89%
+Name                                              Stmts   Miss  Cover
+---------------------------------------------------------------------
+src/kong_mcp_server/__init__.py                       5      2    60%
+src/kong_mcp_server/kong_client.py                   96      0   100%
+src/kong_mcp_server/server.py                        79     23    71%
+src/kong_mcp_server/tools/__init__.py                 0      0   100%
+src/kong_mcp_server/tools/basic.py                    2      0   100%
+src/kong_mcp_server/tools/kong_plugins.py            43      0   100%
+src/kong_mcp_server/tools/kong_rate_limiting.py     120     13    89%
+src/kong_mcp_server/tools/kong_routes.py             39      0   100%
+src/kong_mcp_server/tools/kong_services.py           35      2    94%
+---------------------------------------------------------------------
+TOTAL                                               419     40    90%

src/kong_mcp_server/tools/kong_rate_limiting.py

@@ -4,9 +4,9 @@
 
 from kong_mcp_server.kong_client import KongClient
 
-
 # Basic Rate Limiting Plugin Tools
 
+
 async def create_rate_limiting_plugin(
     minute: Optional[int] = None,
     hour: Optional[int] = None,
@@ -155,7 +155,8 @@ async def get_rate_limiting_plugins(
 
     async with KongClient() as client:
         response = await client.get(endpoint, params=params)
-        return response.get("data", [])
+        data = response.get("data", [])
+        return data if isinstance(data, list) else []
 
 
 async def update_rate_limiting_plugin(
@@ -268,12 +269,13 @@ async def delete_rate_limiting_plugin(plugin_id: str) -> Dict[str, Any]:
         await client.delete(f"/plugins/{plugin_id}")
         return {
             "message": "Rate limiting plugin deleted successfully",
-            "plugin_id": plugin_id
+            "plugin_id": plugin_id,
         }
 
 
 # General Plugin Management Tools
 
+
 async def get_plugin(plugin_id: str) -> Dict[str, Any]:
     """Get a specific plugin by ID.
 
@@ -330,4 +332,5 @@ async def get_plugins(
 
     async with KongClient() as client:
         response = await client.get(endpoint, params=params)
-        return response.get("data", [])
+        data = response.get("data", [])
+        return data if isinstance(data, list) else []

src/kong_mcp_server/tools_config.json

@@ -70,20 +70,20 @@
       "function": "get_plugins_by_service",
       "enabled": true
     },
-      "kong_get_plugins_by_route": {
+    "kong_get_plugins_by_route": {
       "name": "kong_get_plugins_by_route",
       "description": "Retrieve route scoped Kong plugins with optional filtering by name and pagination.",
       "module": "kong_mcp_server.tools.kong_plugins",
       "function": "get_plugins_by_route",
       "enabled": true
     },
-      "kong_get_plugins_by_consumer": {
+    "kong_get_plugins_by_consumer": {
       "name": "kong_get_plugins_by_consumer",
       "description": "Retrieve consumer scoped Kong plugins with optional filtering by name and pagination.",
       "module": "kong_mcp_server.tools.kong_plugins",
       "function": "get_plugins_by_consumer",
       "enabled": true
-      },
+    },
     "kong_create_rate_limiting_plugin": {
       "name": "kong_create_rate_limiting_plugin",
       "description": "Create a basic rate limiting plugin with support for all scopes (global, service, route, consumer) and time-based limits (second, minute, hour, day, month, year)",
@@ -119,4 +119,5 @@
       "function": "get_plugin",
       "enabled": true
     }
-}
+  }
+}