Proper set up of logging to work correctly with TUI also for logging level DEBUG.

Author: dusano

plain2code.py

@@ -43,6 +43,7 @@
     TuiLoggingHandler,
     dump_crash_logs,
     get_log_file_path,
+    LOGGER_NAME,
 )
 from plain2code_state import RunState
 from plain2code_utils import print_dry_run_output
@@ -115,6 +116,7 @@ def setup_logging(
 ):
     # Set default level to INFO for everything not explicitly configured
     logging.getLogger().setLevel(logging.INFO)
+    logging.getLogger(LOGGER_NAME).setLevel(logging.INFO)
     logging.getLogger("git").setLevel(logging.WARNING)
     logging.getLogger("repositories").setLevel(logging.WARNING)
     logging.getLogger("transitions").setLevel(logging.ERROR)
@@ -136,9 +138,9 @@ def setup_logging(
     # We add the TuiLoggingHandler to the root logger.
     # CRITICAL: We must remove existing handlers (like StreamHandler) to prevent double-logging
     # that spills into the TUI dashboard.
-    root_logger = logging.getLogger()
-    for h in root_logger.handlers[:]:
-        root_logger.removeHandler(h)
+    root_logger = logging.getLogger(LOGGER_NAME)
+    configured_log_level = root_logger.level
+    root_logger.setLevel(logging.DEBUG)  # Capture all logs; handlers will filter levels as needed
 
     formatter = IndentedFormatter("%(levelname)s:%(name)s:%(message)s")
 
@@ -151,6 +153,7 @@ def setup_logging(
         try:
             file_handler = logging.FileHandler(log_file_path, mode="w")
             file_handler.setFormatter(formatter)
+            file_handler.setLevel(configured_log_level)
             root_logger.addHandler(file_handler)
         except Exception as e:
             console.warning(f"Failed to setup file logging to {log_file_path}: {str(e)}")
@@ -159,6 +162,7 @@ def setup_logging(
         # in case we need to dump them on crash.
         crash_handler = CrashLogHandler()
         crash_handler.setFormatter(formatter)
+        crash_handler.setLevel(configured_log_level)
         root_logger.addHandler(crash_handler)
 
     root_logger.info(f"Render ID: {render_id}")  # Ensure render ID is logged in to codeplain.log file

plain2code_console.py

@@ -4,8 +4,12 @@
 from rich.style import Style
 from rich.tree import Tree
 
+import plain2code_logger
+
 CHARACTERS_TO_TOKENS_RULE_OF_THUMB_RATIO = 4
 
+logger = logging.getLogger(plain2code_logger.LOGGER_NAME)
+
 
 class Plain2CodeConsole(Console):
     INFO_STYLE = Style()
@@ -22,35 +26,35 @@ def __init__(self):
 
             self.llm_encoding = tiktoken.get_encoding("cl100k_base")
         except Exception as e:
-            logging.warning(
+            logger.warning(
                 "Failed to import optional library tiktoken. Using approximate instead of exact token count."
             )
-            logging.debug(f"Exception: {e}")
+            logger.debug(f"Exception: {e}")
             self.llm_encoding = None
 
     def info(self, *args, **kwargs):
-        logging.info(" ".join(map(str, args)))
+        logger.info(" ".join(map(str, args)))
         super().print(*args, **kwargs, style=self.INFO_STYLE)
 
     def warning(self, *args, **kwargs):
-        logging.warning(" ".join(map(str, args)))
+        logger.warning(" ".join(map(str, args)))
         super().print(*args, **kwargs, style=self.WARNING_STYLE)
 
     def error(self, *args, **kwargs):
-        logging.error(" ".join(map(str, args)))
+        logger.error(" ".join(map(str, args)))
         super().print(*args, **kwargs, style=self.ERROR_STYLE)
 
     def input(self, *args, **kwargs):
         # We also log input as info so it shows in the toggled view
-        logging.info(" ".join(map(str, args)))
+        logger.info(" ".join(map(str, args)))
         super().print(*args, **kwargs, style=self.INPUT_STYLE)
 
     def output(self, *args, **kwargs):
-        logging.info(" ".join(map(str, args)))
+        logger.info(" ".join(map(str, args)))
         super().print(*args, **kwargs, style=self.OUTPUT_STYLE)
 
     def debug(self, *args, **kwargs):
-        logging.debug(" ".join(map(str, args)))
+        logger.debug(" ".join(map(str, args)))
         super().print(*args, **kwargs, style=self.DEBUG_STYLE)
 
     def print_list(self, items, style=None):
@@ -116,7 +120,7 @@ def _count_tokens(self, text):
 
     def print_resources(self, resources_list, linked_resources):
         if len(resources_list) == 0:
-            self.input("Linked resources: None")
+            self.debug("Linked resources: None")
             return
 
         self.input("Linked resources:")

plain2code_events.py

@@ -40,7 +40,7 @@ class RenderFailed(BaseEvent):
 
 @dataclass
 class LogMessageEmitted(BaseEvent):
-    logger_name: str  # e.g., "services.langsmith.langsmith_service", "root"
+    logger_name: str  # e.g., "codeplain"
     level: str  # e.g., "INFO", "DEBUG", "ERROR"
     message: str  # The actual log message
     timestamp: str  # Formatted timestamp

plain2code_logger.py

@@ -6,6 +6,8 @@
 from event_bus import EventBus
 from plain2code_events import LogMessageEmitted
 
+LOGGER_NAME = "codeplain"
+
 
 class RetryOnlyFilter(logging.Filter):
     def filter(self, record):
@@ -105,7 +107,7 @@ def dump_crash_logs(args, formatter=None):
     if formatter is None:
         formatter = IndentedFormatter("%(levelname)s:%(name)s:%(message)s")
 
-    root_logger = logging.getLogger()
+    root_logger = logging.getLogger(LOGGER_NAME)
     crash_handler = next((h for h in root_logger.handlers if isinstance(h, CrashLogHandler)), None)
 
     if crash_handler and args.filename:

tui/components.py

@@ -488,7 +488,7 @@ class StructuredLogView(VerticalScroll):
 
     def __init__(self, **kwargs):
         super().__init__(**kwargs)
-        self.min_level = "DEBUG"  # Show all by default
+        self.min_level = "INFO"  # By default, show INFO and above
 
     def _should_show_log(self, level: str) -> bool:
         """Check if log should be shown based on minimum level."""