Update utils/logging.py

utils/logging.py

@@ -1,55 +1,56 @@
 import logging
 import sys
-import traceback
 from datetime import datetime
-import os
+from pathlib import Path
 
-def setup_logging():
-    """Setup logging configuration for the application"""
-    # Create logs directory if it doesn't exist
-    os.makedirs('logs', exist_ok=True)
-    
-    # Configure logging
+# Global logger configuration
+LOG_FORMAT = "%(asctime)s - %(name)s - %(levelname)s - %(message)s"
+LOG_LEVEL = logging.INFO
+
+def setup_logging(log_level=LOG_LEVEL):
+    """Setup logging configuration"""
     logging.basicConfig(
-        level=logging.INFO,
-        format='%(asctime)s - %(name)s - %(levelname)s - %(message)s',
+        level=log_level,
+        format=LOG_FORMAT,
         handlers=[
             logging.StreamHandler(sys.stdout),
-            logging.FileHandler(f'logs/app_{datetime.now().strftime("%Y%m%d")}.log', mode='a')
         ]
     )
-    
-    logger = logging.getLogger('mona_app')
-    logger.info("Logging setup completed successfully")
-    return logger
 
-def get_logger(name=None):
+def get_logger(name):
     """Get a logger instance"""
-    if name is None:
-        name = 'mona_app'
     return logging.getLogger(name)
 
-def log_error(message, error=None, logger_name=None):
-    """Log error with optional traceback"""
-    logger = get_logger(logger_name)
+def log_error(logger, error_msg, exception=None, context=None):
+    """Log error with additional context"""
+    full_msg = f"{error_msg}"
+    if context:
+        full_msg += f" | Context: {context}"
+    if exception:
+        full_msg += f" | Exception: {str(exception)}"
     
-    if error:
-        logger.error(f"{message}: {str(error)}")
-        logger.error(f"Traceback: {traceback.format_exc()}")
-    else:
-        logger.error(message)
+    logger.error(full_msg)
 
-def log_info(message, logger_name=None):
-    """Log info message"""
-    logger = get_logger(logger_name)
-    logger.info(message)
+def log_info(logger, msg, context=None):
+    """Log info with additional context"""
+    full_msg = f"{msg}"
+    if context:
+        full_msg += f" | Context: {context}"
+    
+    logger.info(full_msg)
 
-def log_warning(message, logger_name=None):
-    """Log warning message"""
-    logger = get_logger(logger_name)
-    logger.warning(message)
+def log_warning(logger, msg, context=None):
+    """Log warning with additional context"""
+    full_msg = f"{msg}"
+    if context:
+        full_msg += f" | Context: {context}"
+    
+    logger.warning(full_msg)
 
-def log_debug(message, logger_name=None):
-    """Log debug message"""
-    logger = get_logger(logger_name)
-    logger.debug(message)
+def log_debug(logger, msg, context=None):
+    """Log debug with additional context"""
+    full_msg = f"{msg}"
+    if context:
+        full_msg += f" | Context: {context}"
+    
+    logger.debug(full_msg)