Function log_error

logging_utils/core/error_handler.py:17–90

Centralized error logging with automatic stack trace capture. This function ensures consistent error logging across the codebase: - Always includes exc_info for full stack traces - Optionally saves error snapshots for debugging - Integrates with request context for tracing

(
    logger: logging.Logger,
    message: str,
    exception: Optional[BaseException] = None,
    *,
    save_snapshot: bool = False,
    req_id: str = "",
    exc_info: bool = True,
)

def log_error(
    logger: logging.Logger,
    message: str,
    exception: Optional[BaseException] = None,
    *,
    save_snapshot: bool = False,
    req_id: str = "",
    exc_info: bool = True,
) -> None:
    """
    Centralized error logging with automatic stack trace capture.

    This function ensures consistent error logging across the codebase:
    - Always includes exc_info for full stack traces
    - Optionally saves error snapshots for debugging
    - Integrates with request context for tracing

    Args:
        logger: Logger instance to use
        message: Error message to log
        exception: Optional exception object (used for snapshot)
        save_snapshot: Whether to save an error snapshot (screenshot, DOM, etc.)
        req_id: Optional request ID override (uses context var if empty)
        exc_info: Whether to include exception info (default True)

    Usage:
        try:
            risky_operation()
        except Exception as e:
            log_error(logger, f"Operation failed: {e}", e, save_snapshot=True)
    """
    # Get request ID from context if not provided
    if not req_id:
        try:
            req_id = request_id_var.get()
        except LookupError:
            req_id = "unknown"

    # Log with exc_info for full stack trace
    logger.error(message, exc_info=exc_info)

    # Save error snapshot if requested
    if save_snapshot:
        try:
            # Lazy import to avoid circular dependencies
            from browser_utils.debug_utils import save_error_snapshot_enhanced

            # Generate error name from message (first 30 chars, sanitized)
            error_name = (
                message[:30].replace(" ", "_").replace(":", "").replace("/", "_")
            )
            # This is async, but we're in sync context - schedule it
            try:
                loop = asyncio.get_running_loop()
                loop.create_task(
                    save_error_snapshot_enhanced(
                        error_name=error_name,
                        error_exception=exception