refactor(error_handler): implement structured logging for errors

Author: stickerdaniel

linkedin_mcp_server/drivers/chrome.py

@@ -10,14 +10,6 @@
 from typing import Dict, Optional
 
 import inquirer  # type: ignore
-from selenium import webdriver
-from selenium.common.exceptions import WebDriverException
-from selenium.webdriver.chrome.options import Options
-from selenium.webdriver.chrome.service import Service
-
-from linkedin_mcp_server.config import get_config
-from linkedin_mcp_server.config.providers import clear_credentials_from_keyring
-from linkedin_mcp_server.config.secrets import get_credentials
 from linkedin_scraper.exceptions import (
     CaptchaRequiredError,
     InvalidCredentialsError,
@@ -26,6 +18,14 @@
     SecurityChallengeError,
     TwoFactorAuthError,
 )
+from selenium import webdriver
+from selenium.common.exceptions import WebDriverException
+from selenium.webdriver.chrome.options import Options
+from selenium.webdriver.chrome.service import Service
+
+from linkedin_mcp_server.config import get_config
+from linkedin_mcp_server.config.providers import clear_credentials_from_keyring
+from linkedin_mcp_server.config.secrets import get_credentials
 from linkedin_mcp_server.exceptions import (
     CredentialsNotFoundError,
     DriverInitializationError,
@@ -133,9 +133,6 @@ def get_or_create_driver() -> Optional[webdriver.Chrome]:
             raise WebDriverException(error_msg)
 
 
-# Remove this function - linkedin-scraper now handles all error detection
-
-
 def login_to_linkedin(driver: webdriver.Chrome) -> bool:
     """
     Log in to LinkedIn using stored or provided credentials.

linkedin_mcp_server/error_handler.py

@@ -5,6 +5,7 @@
 eliminating code duplication and ensuring consistent error responses.
 """
 
+import logging
 from typing import Any, Dict, List
 
 from linkedin_scraper.exceptions import (
@@ -135,8 +136,16 @@ def convert_exception_to_response(
         return {"error": "linkedin_error", "message": str(exception)}
 
     else:
-        # Generic error handling
-        print(f"❌ Error in {context}: {exception}")
+        # Generic error handling with structured logging
+        logger = logging.getLogger(__name__)
+        logger.error(
+            f"Error in {context}: {exception}",
+            extra={
+                "context": context,
+                "exception_type": type(exception).__name__,
+                "exception_message": str(exception),
+            },
+        )
         return {
             "error": "unknown_error",
             "message": f"Failed to execute {context}: {str(exception)}",