fix: improve error messages and fix formatting issues

Author: madebygps

agents/langchainv1_keycloak.py

@@ -50,7 +50,7 @@
         azure.identity.DefaultAzureCredential(), "https://cognitiveservices.azure.com/.default"
     )
     base_model = ChatOpenAI(
-        model=os.environ.get("AZURE_OPENAI_CHAT_DEPLOYMENT"),
+        model=os.environ.get("AZURE_OPENAI_CHAT_DEPLOYMENT", "gpt-4o"),
         base_url=os.environ["AZURE_OPENAI_ENDPOINT"] + "/openai/v1/",
         api_key=token_provider,
     )
@@ -88,7 +88,12 @@ async def register_client_via_dcr() -> tuple[str, str]:
         )
 
         if response.status_code not in (200, 201):
-            raise Exception(f"DCR failed: {response.status_code} - {response.text}")
+            raise Exception(
+                f"DCR registration failed at {dcr_url}: "
+                f"status={response.status_code}, "
+                f"response={response.text}, "
+                f"headers={dict(response.headers)}"
+            )
 
         data = response.json()
         client_id = data["client_id"]
@@ -116,7 +121,12 @@ async def get_keycloak_token(client_id: str, client_secret: str) -> str:
         )
 
         if response.status_code != 200:
-            raise Exception(f"Failed to get token: {response.status_code} - {response.text}")
+            raise Exception(
+                f"Token request failed at {token_url}: "
+                f"status={response.status_code}, "
+                f"response={response.text}, "
+                f"headers={dict(response.headers)}"
+            )
 
         token_data = response.json()
         access_token = token_data["access_token"]
@@ -137,7 +147,7 @@ async def run_agent() -> None:
     logger.info(f"📡 Connecting to MCP server: {MCP_SERVER_URL}")
 
     # Initialize MCP client with Bearer token auth
-    client = MultiServerMCPClient(
+    client = MultiServerMCPClient(  # type: ignore[arg-type]
         {
             "expenses": {
                 "url": MCP_SERVER_URL,

servers/keycloak_deployed_mcp.py

@@ -18,13 +18,15 @@
 logging.basicConfig(level=logging.INFO, format="%(asctime)s - %(message)s")
 logger = logging.getLogger("ExpensesMCP")
 
+
 def require_env_var(name: str) -> str:
     value = os.getenv(name)
     if value is None or value.strip() == "":
         logger.error(f"Missing required environment variable: {name}")
         exit(1)
     return value
 
+
 # Cosmos DB configuration from environment variables
 AZURE_COSMOSDB_ACCOUNT = require_env_var("AZURE_COSMOSDB_ACCOUNT")
 AZURE_COSMOSDB_DATABASE = require_env_var("AZURE_COSMOSDB_DATABASE")
@@ -33,7 +35,9 @@ def require_env_var(name: str) -> str:
 AZURE_CLIENT_ID = os.getenv("AZURE_CLIENT_ID", "")
 
 # Keycloak authentication configuration
-KEYCLOAK_REALM_URL = require_env_var("KEYCLOAK_REALM_URL")  # e.g., https://routeconfig.<env>.azurecontainerapps.io/auth/realms/mcp
+KEYCLOAK_REALM_URL = require_env_var(
+    "KEYCLOAK_REALM_URL"
+)  # e.g., https://routeconfig.<env>.azurecontainerapps.io/auth/realms/mcp
 MCP_SERVER_BASE_URL = require_env_var("MCP_SERVER_BASE_URL")  # e.g., https://routeconfig.<env>.azurecontainerapps.io
 MCP_SERVER_AUDIENCE = os.getenv("MCP_SERVER_AUDIENCE", "mcp-server")