Improve logging

* Proper logging setup instead of basicConfig()
* INFO and below to stdout, WARNING and above to stderr

Author: dannyadair

src/pdfbaker/baker.py

@@ -6,15 +6,14 @@
 bake() delegates to its documents and reports back the end result.
 """
 
-import logging
 from dataclasses import dataclass
 from pathlib import Path
 from typing import Any
 
 from .config import PDFBakerConfiguration, deep_merge
 from .document import PDFBakerDocument
 from .errors import ConfigurationError
-from .logging import TRACE, LoggingMixin
+from .logging import LoggingMixin, setup_logging
 
 __all__ = ["PDFBaker", "PDFBakerOptions"]
 
@@ -86,16 +85,7 @@ def __init__(
         """
         super().__init__()
         options = options or PDFBakerOptions()
-
-        logging.basicConfig(level=logging.INFO, format="%(levelname)s: %(message)s")
-        if options.quiet:
-            logging.getLogger().setLevel(logging.ERROR)
-        elif options.trace:
-            logging.getLogger().setLevel(TRACE)
-        elif options.verbose:
-            logging.getLogger().setLevel(logging.DEBUG)
-        else:
-            logging.getLogger().setLevel(logging.INFO)
+        setup_logging(quiet=options.quiet, trace=options.trace, verbose=options.verbose)
         self.keep_build = options.keep_build
 
         base_config = DEFAULT_BAKER_CONFIG.copy()

src/pdfbaker/logging.py

@@ -1,11 +1,14 @@
 """Logging mixin for pdfbaker classes."""
 
 import logging
+import sys
 from typing import Any
 
 TRACE = 5
 logging.addLevelName(TRACE, "TRACE")
 
+__all__ = ["LoggingMixin", "setup_logging", "truncate_strings"]
+
 
 class LoggingMixin:
     """Mixin providing consistent logging functionality across pdfbaker classes."""
@@ -71,6 +74,37 @@ def log_critical(self, msg: str, *args: Any, **kwargs: Any) -> None:
         self.logger.critical(msg, *args, **kwargs)
 
 
+def setup_logging(quiet=False, trace=False, verbose=False) -> None:
+    """Set up logging for the application."""
+    logger = logging.getLogger()
+    logger.setLevel(logging.INFO)
+    formatter = logging.Formatter("%(levelname)s: %(message)s")
+
+    # stdout handler for TRACE/DEBUG/INFO
+    stdout_handler = logging.StreamHandler(sys.stdout)
+    stdout_handler.setFormatter(formatter)
+    stdout_handler.setLevel(TRACE)
+    stdout_handler.addFilter(lambda record: record.levelno < logging.WARNING)
+
+    # stderr handler for WARNING and above
+    stderr_handler = logging.StreamHandler(sys.stderr)
+    stderr_handler.setFormatter(formatter)
+    stderr_handler.setLevel(logging.WARNING)
+
+    logger.handlers.clear()
+    logger.addHandler(stdout_handler)
+    logger.addHandler(stderr_handler)
+
+    if quiet:
+        logger.setLevel(logging.ERROR)
+    elif trace:
+        logger.setLevel(TRACE)
+    elif verbose:
+        logger.setLevel(logging.DEBUG)
+    else:
+        logger.setLevel(logging.INFO)
+
+
 def truncate_strings(obj, max_chars: int) -> Any:
     """Recursively truncate strings in nested structures."""
     if isinstance(obj, str):