Apply code quality fixes and Python 3.10+ compatibility (#4)

Run black, ruff, and fix linting issues across all example server
code and tests.

Code formatting:
- Apply black formatting to all Python files
- Fix import sorting with ruff isort

Linting fixes:
- Remove unused imports
- Fix unused variable assignments
- Update imports to use collections.abc.Callable

Configuration updates:
- Update pyproject.toml ruff config to use lint section
- Fix deprecated ruff configuration format

Python 3.10+ compatibility:
- Add `from __future__ import annotations` for union type hints
- Required for `X | Y` syntax on Python versions < 3.10

Author: RahulHere

examples/auth/py_auth_mcp_server/app.py

@@ -3,8 +3,9 @@
 Creates and configures the Flask application for the Auth MCP Server.
 """
 
+from __future__ import annotations
+
 from pathlib import Path
-from typing import Any
 
 from flask import Flask
 from flask_cors import CORS
@@ -79,7 +80,9 @@ def create_app(
                 if config.jwks_auto_refresh:
                     auth_client.set_option("auto_refresh", "true")
                 if config.request_timeout > 0:
-                    auth_client.set_option("request_timeout", str(config.request_timeout))
+                    auth_client.set_option(
+                        "request_timeout", str(config.request_timeout)
+                    )
 
                 app.config["AUTH_CLIENT"] = auth_client
             except Exception as e:

examples/auth/py_auth_mcp_server/config.py

@@ -4,7 +4,9 @@
 /gopher-orch/examples/auth/auth_server_config.h
 """
 
-from dataclasses import dataclass, field
+from __future__ import annotations
+
+from dataclasses import dataclass
 from pathlib import Path
 from typing import Any
 
@@ -157,9 +159,7 @@ def build_config(config_map: dict[str, str]) -> AuthServerConfig:
     # Derive endpoints from auth_server_url if not explicitly set
     if config.auth_server_url:
         if not config.jwks_uri:
-            config.jwks_uri = (
-                f"{config.auth_server_url}/protocol/openid-connect/certs"
-            )
+            config.jwks_uri = f"{config.auth_server_url}/protocol/openid-connect/certs"
         if not config.token_endpoint:
             config.token_endpoint = (
                 f"{config.auth_server_url}/protocol/openid-connect/token"

examples/auth/py_auth_mcp_server/middleware/oauth_auth.py

@@ -4,8 +4,11 @@
 Mirrors OAuthAuthFilter from the C++ example.
 """
 
+from __future__ import annotations
+
+from collections.abc import Callable
 from functools import wraps
-from typing import Any, Callable
+from typing import Any
 
 from flask import Response, g, jsonify, request
 

examples/auth/py_auth_mcp_server/routes/health.py

@@ -4,6 +4,8 @@
 and load balancer health checks.
 """
 
+from __future__ import annotations
+
 import time
 from dataclasses import dataclass
 from datetime import datetime, timezone

examples/auth/py_auth_mcp_server/routes/mcp_handler.py

@@ -4,10 +4,13 @@
 for tool registration and invocation.
 """
 
+from __future__ import annotations
+
+from collections.abc import Callable
 from dataclasses import dataclass, field
-from typing import Any, Callable, TypedDict
+from typing import Any, TypedDict
 
-from flask import Blueprint, Flask, Response, g, jsonify, request
+from flask import Blueprint, Flask, Response, jsonify, request
 
 
 class JsonRpcErrorCode:
@@ -238,16 +241,12 @@ def handle_request(self, body: Any) -> JsonRpcResponse:
         request_id = rpc_request.id
 
         try:
-            result = self._dispatch_method(
-                rpc_request.method, rpc_request.params or {}
-            )
+            result = self._dispatch_method(rpc_request.method, rpc_request.params or {})
             return JsonRpcResponse(jsonrpc="2.0", id=request_id, result=result)
         except JsonRpcError as e:
             return JsonRpcResponse(jsonrpc="2.0", id=request_id, error=e)
         except Exception as e:
-            error = JsonRpcError(
-                code=JsonRpcErrorCode.INTERNAL_ERROR, message=str(e)
-            )
+            error = JsonRpcError(code=JsonRpcErrorCode.INTERNAL_ERROR, message=str(e))
             return JsonRpcResponse(jsonrpc="2.0", id=request_id, error=error)
 
     def _parse_request(self, body: Any) -> JsonRpcRequest | JsonRpcError:
@@ -285,9 +284,7 @@ def _parse_request(self, body: Any) -> JsonRpcRequest | JsonRpcError:
             params=params,
         )
 
-    def _dispatch_method(
-        self, method: str, params: dict[str, Any]
-    ) -> Any:
+    def _dispatch_method(self, method: str, params: dict[str, Any]) -> Any:
         """Dispatch a method call to the appropriate handler."""
         if method == "initialize":
             return self._handle_initialize(params)

examples/auth/py_auth_mcp_server/routes/oauth_endpoints.py

@@ -8,6 +8,8 @@
 - /oauth/register (RFC 7591 dynamic registration)
 """
 
+from __future__ import annotations
+
 import math
 import time
 from typing import Any
@@ -28,9 +30,7 @@ def _set_cors_headers(response: Response) -> Response:
         Response with CORS headers set.
     """
     response.headers["Access-Control-Allow-Origin"] = "*"
-    response.headers["Access-Control-Allow-Methods"] = (
-        "GET, POST, PUT, DELETE, OPTIONS"
-    )
+    response.headers["Access-Control-Allow-Methods"] = "GET, POST, PUT, DELETE, OPTIONS"
     response.headers["Access-Control-Allow-Headers"] = (
         "Authorization, Content-Type, Accept, Origin, X-Requested-With"
     )

examples/auth/py_auth_mcp_server/tools/weather_tools.py

@@ -4,6 +4,8 @@
 Mirrors the weather tools from the C++ auth example.
 """
 
+from __future__ import annotations
+
 import json
 from typing import Any
 
@@ -168,10 +170,12 @@ def access_denied(scope: str) -> ToolResult:
         content=[
             ToolContentItem(
                 type="text",
-                text=json.dumps({
-                    "error": "access_denied",
-                    "message": f"Access denied. Required scope: {scope}",
-                }),
+                text=json.dumps(
+                    {
+                        "error": "access_denied",
+                        "message": f"Access denied. Required scope: {scope}",
+                    }
+                ),
             ),
         ],
         is_error=True,

examples/auth/pyproject.toml

@@ -61,6 +61,8 @@ exclude = '''
 [tool.ruff]
 line-length = 88
 target-version = "py310"
+
+[tool.ruff.lint]
 select = [
     "E",   # pycodestyle errors
     "W",   # pycodestyle warnings
@@ -74,7 +76,7 @@ ignore = [
     "E501",  # line too long (handled by black)
 ]
 
-[tool.ruff.isort]
+[tool.ruff.lint.isort]
 known-first-party = ["py_auth_mcp_server", "gopher_mcp_python"]
 
 [tool.mypy]

examples/auth/tests/conftest.py

@@ -1,11 +1,12 @@
 """Pytest fixtures for py_auth_mcp_server tests."""
 
+from __future__ import annotations
+
 import pytest
 from flask import Flask
 from flask.testing import FlaskClient
 
 from gopher_mcp_python.ffi.auth import AuthContext
-
 from py_auth_mcp_server import AuthServerConfig, create_app, create_default_config
 from py_auth_mcp_server.routes.mcp_handler import McpHandler
 

examples/auth/tests/test_config.py

@@ -1,7 +1,6 @@
 """Tests for configuration module."""
 
 import tempfile
-from pathlib import Path
 
 import pytest
 
@@ -68,7 +67,7 @@ def test_empty_content(self):
     def test_only_comments(self):
         """Test parsing content with only comments."""
         content = "# comment 1\n# comment 2"
-        result = parse_config_file("")
+        result = parse_config_file(content)
         assert result == {}
 
 
@@ -86,11 +85,13 @@ def test_default_values(self):
 
     def test_custom_values(self):
         """Test custom values override defaults."""
-        config = build_config({
-            "host": "127.0.0.1",
-            "port": "8080",
-            "auth_disabled": "true",
-        })
+        config = build_config(
+            {
+                "host": "127.0.0.1",
+                "port": "8080",
+                "auth_disabled": "true",
+            }
+        )
         assert config.host == "127.0.0.1"
         assert config.port == 8080
 
@@ -102,10 +103,12 @@ def test_port_type_conversion(self):
 
     def test_boolean_type_conversion(self):
         """Test boolean values are converted correctly."""
-        config = build_config({
-            "auth_disabled": "true",
-            "jwks_auto_refresh": "false",
-        })
+        config = build_config(
+            {
+                "auth_disabled": "true",
+                "jwks_auto_refresh": "false",
+            }
+        )
         assert config.auth_disabled is True
         assert config.jwks_auto_refresh is False
 
@@ -116,10 +119,12 @@ def test_server_url_default(self):
 
     def test_server_url_custom(self):
         """Test custom server_url."""
-        config = build_config({
-            "server_url": "https://api.example.com",
-            "auth_disabled": "true",
-        })
+        config = build_config(
+            {
+                "server_url": "https://api.example.com",
+                "auth_disabled": "true",
+            }
+        )
         assert config.server_url == "https://api.example.com"
 
 
@@ -128,44 +133,52 @@ class TestEndpointDerivation:
 
     def test_derives_jwks_uri(self):
         """Test jwks_uri is derived from auth_server_url."""
-        config = build_config({
-            "auth_server_url": "https://auth.example.com/realms/test",
-            "client_id": "test-client",
-            "client_secret": "secret",
-        })
+        config = build_config(
+            {
+                "auth_server_url": "https://auth.example.com/realms/test",
+                "client_id": "test-client",
+                "client_secret": "secret",
+            }
+        )
         assert config.jwks_uri == (
             "https://auth.example.com/realms/test/protocol/openid-connect/certs"
         )
 
     def test_derives_token_endpoint(self):
         """Test token_endpoint is derived from auth_server_url."""
-        config = build_config({
-            "auth_server_url": "https://auth.example.com/realms/test",
-            "client_id": "test-client",
-            "client_secret": "secret",
-        })
+        config = build_config(
+            {
+                "auth_server_url": "https://auth.example.com/realms/test",
+                "client_id": "test-client",
+                "client_secret": "secret",
+            }
+        )
         assert config.token_endpoint == (
             "https://auth.example.com/realms/test/protocol/openid-connect/token"
         )
 
     def test_derives_issuer(self):
         """Test issuer is derived from auth_server_url."""
-        config = build_config({
-            "auth_server_url": "https://auth.example.com/realms/test",
-            "client_id": "test-client",
-            "client_secret": "secret",
-        })
+        config = build_config(
+            {
+                "auth_server_url": "https://auth.example.com/realms/test",
+                "client_id": "test-client",
+                "client_secret": "secret",
+            }
+        )
         assert config.issuer == "https://auth.example.com/realms/test"
 
     def test_explicit_values_not_overwritten(self):
         """Test explicit values are not overwritten by derivation."""
-        config = build_config({
-            "auth_server_url": "https://auth.example.com/realms/test",
-            "jwks_uri": "https://custom.example.com/jwks",
-            "issuer": "https://custom-issuer.example.com",
-            "client_id": "test-client",
-            "client_secret": "secret",
-        })
+        config = build_config(
+            {
+                "auth_server_url": "https://auth.example.com/realms/test",
+                "jwks_uri": "https://custom.example.com/jwks",
+                "issuer": "https://custom-issuer.example.com",
+                "client_id": "test-client",
+                "client_secret": "secret",
+            }
+        )
         assert config.jwks_uri == "https://custom.example.com/jwks"
         assert config.issuer == "https://custom-issuer.example.com"