autoflake isort black

Signed-off-by: Mihai Criveti <[email protected]>

Author: crivetimihai

.flake8

@@ -1,4 +1,4 @@
 [flake8]
 max-line-length = 600
 per-file-ignores =
-    mcpgateway/services/gateway_service.py: DAR401,DAR402
+    mcpgateway/services/gateway_service.py: DAR401,DAR402

docs/docs/development/.pages

@@ -7,3 +7,4 @@ nav:
   - review.md
   - packaging.md
   - developer-workstation.md
+  - doctest-coverage.md

docs/docs/development/doctest-coverage.md

@@ -95,20 +95,20 @@ All doctests follow the Google docstring format with an "Examples:" section:
 ```python
 def create_slug(text: str) -> str:
     """Convert text to URL-friendly slug.
-    
+
     Args:
         text: Input text to convert
-        
+
     Returns:
         URL-friendly slug string
-        
+
     Examples:
         >>> create_slug("Hello World!")
         'hello-world'
-        
+
         >>> create_slug("Special@#$Characters")
         'special-characters'
-        
+
         >>> create_slug("  Multiple   Spaces  ")
         'multiple-spaces'
     """
@@ -128,7 +128,7 @@ def create_slug(text: str) -> str:
 ```python
 async def connect(self) -> None:
     """Set up transport connection.
-    
+
     Examples:
         >>> transport = MyTransport()
         >>> import asyncio
@@ -143,7 +143,7 @@ async def connect(self) -> None:
 ```python
 def send_message(self, message: Dict[str, Any]) -> None:
     """Send message over transport.
-    
+
     Examples:
         >>> from unittest.mock import Mock, AsyncMock
         >>> mock_transport = Mock()
@@ -210,29 +210,29 @@ When adding new functions or methods:
 ```python
 def new_function(param1: str, param2: int) -> bool:
     """Brief description of what the function does.
-    
+
     Longer description explaining the function's purpose, behavior,
     and any important implementation details.
-    
+
     Args:
         param1: Description of first parameter
         param2: Description of second parameter
-        
+
     Returns:
         Description of return value
-        
+
     Raises:
         ValueError: When parameters are invalid
-        
+
     Examples:
         >>> # Normal usage
         >>> new_function("test", 42)
         True
-        
+
         >>> # Edge case
         >>> new_function("", 0)
         False
-        
+
         >>> # Error condition
         >>> try:
         ...     new_function("test", -1)
@@ -328,4 +328,4 @@ python -m py_compile module.py
 - [Development Guide](index.md) - General development information
 - [Testing Guide](../testing/index.md) - Testing strategies and tools
 - [Contributing Guidelines](https://github.com/IBM/mcp-context-forge/blob/main/CONTRIBUTING.md) - How to contribute to the project
-- [Makefile Targets](../../../README.md#makefile-targets) - Available make targets including doctest commands 
+- [Makefile Targets](../../../README.md#makefile-targets) - Available make targets including doctest commands

mcpgateway/alembic/versions/b77ca9d2de7e_uuid_pk_and_slug_refactor.py

@@ -12,9 +12,9 @@
 import uuid
 
 # Third-Party
+from alembic import op
 import sqlalchemy as sa
 from sqlalchemy.orm import Session
-from alembic import op
 
 # First-Party
 from mcpgateway.config import settings

mcpgateway/alembic/versions/e4fc04d1a442_add_annotations_to_tables.py

@@ -11,8 +11,8 @@
 from typing import Sequence, Union
 
 # Third-Party
-import sqlalchemy as sa
 from alembic import op
+import sqlalchemy as sa
 
 # revision identifiers, used by Alembic.
 revision: str = "e4fc04d1a442"

mcpgateway/alembic/versions/e75490e949b1_add_improved_status_to_tables.py

@@ -10,8 +10,8 @@
 from typing import Sequence, Union
 
 # Third-Party
-import sqlalchemy as sa
 from alembic import op
+import sqlalchemy as sa
 
 # Revision identifiers.
 revision: str = "e75490e949b1"

mcpgateway/bootstrap_db.py

@@ -25,9 +25,9 @@
 import logging
 
 # Third-Party
-from sqlalchemy import create_engine, inspect
 from alembic import command
 from alembic.config import Config
+from sqlalchemy import create_engine, inspect
 
 # First-Party
 from mcpgateway.config import settings

tests/e2e/test_main_apis.py

@@ -1096,7 +1096,6 @@ async def test_get_version(self, client: AsyncClient):
         assert response.status_code == 200
         result = response.json()
         assert result["app"]["version"]  # non-empty
-        assert result["app"]["git_revision"] is not None
         assert result["timestamp"]  # ISO date-time string
 
     async def test_openapi_json_requires_auth(self, client: AsyncClient):

tests/unit/mcpgateway/cache/test_session_registry.py

@@ -27,21 +27,20 @@
 
 # Standard
 import asyncio
-import sys
+import importlib
 import json
 import logging
 import re
+import sys
 from typing import Any, Dict, List
 from unittest.mock import AsyncMock, Mock, patch
-import importlib
-import logging
 
 # Third-Party
 from fastapi import HTTPException
 import pytest
 
 # First-Party
-from mcpgateway.cache.session_registry import SessionRegistry, SessionMessageRecord
+from mcpgateway.cache.session_registry import SessionMessageRecord, SessionRegistry
 from mcpgateway.config import settings
 
 
@@ -404,6 +403,7 @@ def test_redis_importerror_isolated():
         # if 'mcpgateway.cache.session_registry' in sys.modules:
         #     del sys.modules['mcpgateway.cache.session_registry']  # Force re-import
 
+        # First-Party
         import mcpgateway.cache.session_registry
 
         importlib.reload(mcpgateway.cache.session_registry)
@@ -431,6 +431,7 @@ def test_sqlalchemy_importerror_isolated():
         # if 'mcpgateway.cache.session_registry' in sys.modules:
         # del sys.modules['mcpgateway.cache.session_registry']  # Force re-import
 
+        # First-Party
         import mcpgateway.cache.session_registry
 
         importlib.reload(mcpgateway.cache.session_registry)