log: pass exception via exc_info instead of string interpolation

routes/chat_routes.py

@@ -127,7 +127,7 @@ def _clear_orphaned_session_endpoint(sess, owner: str | None = None) -> bool:
         sess.headers = {}
         return True
     except Exception as e:
-        logger.warning("Failed to clear orphaned session endpoint: %s", e)
+        logger.warning("Failed to clear orphaned session endpoint", exc_info=e)
         db.rollback()
         return False
     finally:
@@ -146,7 +146,7 @@ def _endpoint_cache_contains_model(endpoint, model: str) -> bool:
     try:
         models = json.loads(raw) if isinstance(raw, str) else raw
     except Exception as e:
-        logger.warning("Failed to parse cached models list, treating as containing model: %s", e)
+        logger.warning("Failed to parse cached models list, treating as containing model", exc_info=e)
         return True
     if not isinstance(models, list) or not models:
         return True
@@ -239,7 +239,7 @@ def _recover_empty_session_model(sess, session_id: str, owner: str | None = None
         try:
             cached = json.loads(ep.cached_models) if isinstance(ep.cached_models, str) else (ep.cached_models or [])
         except Exception as e:
-            logger.warning("Failed to parse cached_models for endpoint %r: %s", getattr(ep, "id", "?"), e)
+            logger.warning("Failed to parse cached_models for endpoint %r", getattr(ep, "id", "?"), exc_info=e)
             cached = []
         if not cached:
             visible = []
@@ -650,7 +650,7 @@ def setup_chat_routes(
             try:
                 att_ids = [str(x) for x in json.loads(attachments)]
             except Exception as e:
-                logger.warning("Failed to parse attachments JSON, ignoring attachments: %s", e)
+                logger.warning("Failed to parse attachments JSON, ignoring attachments", exc_info=e)
 
         no_memory = str(form_data.get("no_memory", "")).lower() == "true"
         pre_context_tool_policy = build_effective_tool_policy(

routes/document_routes.py

@@ -504,7 +504,7 @@ def setup_document_routes(session_manager, upload_handler=None) -> APIRouter:
         try:
             data = await request.json()
         except Exception as e:
-            logger.warning("Failed to parse export request body, defaulting to empty: %s", e)
+            logger.warning("Failed to parse export request body, defaulting to empty", exc_info=e)
             data = {}
         ids = data.get("ids") or []
         if not ids:
@@ -647,7 +647,7 @@ def setup_document_routes(session_manager, upload_handler=None) -> APIRouter:
                         from src.agent_tools.document_tools import clear_active_document
                         clear_active_document(doc_id)
                     except Exception as e:
-                        logger.warning("Failed to clear active document %r on detach: %s", doc_id, e)
+                        logger.warning("Failed to clear active document %r on detach", doc_id, exc_info=e)
             db.commit()
             db.refresh(doc)
             return _doc_to_dict(doc)

routes/email_routes.py

@@ -80,7 +80,7 @@ def _email_tag_owner_aliases(account_id: str | None, owner: str = "") -> list[st
                         cfg.get("from_address") or "",
                     ])
                 except Exception as _e:
-                    logger.warning("Failed to resolve email account alias: %s", _e)
+                    logger.warning("Failed to resolve email account alias", exc_info=_e)
                     resolved_account_id = None
             row = db.get(_EA, resolved_account_id) if resolved_account_id else None
             if row:
@@ -88,7 +88,7 @@ def _email_tag_owner_aliases(account_id: str | None, owner: str = "") -> list[st
         finally:
             db.close()
     except Exception as _e:
-        logger.warning("Failed to load email aliases: %s", _e)
+        logger.warning("Failed to load email aliases", exc_info=_e)
     out = []
     for a in aliases:
         a = (a or "").strip()