From: Christian Heller <c.heller@plomlompom.de>
Date: Sun, 16 Nov 2025 09:54:33 +0000 (+0100)
Subject: Don't write TUI-internal chatter to log files
X-Git-Url: https://plomlompom.com/repos/templates?a=commitdiff_plain;h=574c764220b3b624121bf6ca0bcece11fed2bb46;p=ircplom

Don't write TUI-internal chatter to log files
---

diff --git a/src/ircplom/client_tui.py b/src/ircplom/client_tui.py
index 908f530..2a47337 100644
--- a/src/ircplom/client_tui.py
+++ b/src/ircplom/client_tui.py
@@ -13,7 +13,7 @@ from ircplom.db_primitives import AutoAttrMixin, Dict, DictItem
 from ircplom.irc_conn import IrcMessage
 from ircplom.tui_base import (
         BaseTui, FormattingString, PromptWidget, TuiEvent, Window,
-        CMD_SHORTCUTS, LOG_FMT_ATTRS, LOG_FMT_TAG_ALERT)
+        CMD_SHORTCUTS, LOG_FMT_ATTRS, LOG_FMT_TAG_ALERT, LOG_PREFIX_DEFAULT)
 
 CMD_SHORTCUTS['disconnect'] = 'window.disconnect'
 CMD_SHORTCUTS['join'] = 'window.join'
@@ -62,8 +62,9 @@ class _ClientWindow(Window, ClientQueueMixin):
 
     def log(self, msg: FormattingString) -> None:
         super().log(msg)
-        if self._path_logs is None:
-            return
+        if self._path_logs is None\
+                or msg.stripped().startswith(LOG_PREFIX_DEFAULT):  # ignore TUI
+            return                                                 # chatter
 
         def escaped(word: str) -> str:
             ret = ''
diff --git a/src/ircplom/tui_base.py b/src/ircplom/tui_base.py
index 68decea..8a9f6e5 100644
--- a/src/ircplom/tui_base.py
+++ b/src/ircplom/tui_base.py
@@ -13,12 +13,12 @@ from blessed import Terminal as BlessedTerminal
 # ourselves
 from ircplom.events import AffectiveEvent, Loop, QueueMixin, QuitEvent
 
-_LOG_PREFIX_DEFAULT = '#'
+LOG_PREFIX_DEFAULT = '#'
 LOG_FMT_SEP = ' '
 LOG_FMT_TAG_ALERT = 'alert'
 LOG_FMT_ATTRS: dict[str, tuple[str, ...]] = {
     LOG_FMT_TAG_ALERT: ('bold', 'bright_red'),
-    _LOG_PREFIX_DEFAULT: ('bright_cyan',),
+    LOG_PREFIX_DEFAULT: ('bright_cyan',),
 }
 _WRAP_INDENT = 2
 
@@ -685,7 +685,7 @@ class BaseTui(QueueMixin):
         if isinstance(msg, str):
             msg = FormattingString(msg)
         if prefix_char is None:
-            prefix_char = _LOG_PREFIX_DEFAULT
+            prefix_char = LOG_PREFIX_DEFAULT
         now = str(datetime.now())
         today, time = now[:10], now[11:19]
         msg = FormattingString(f'{prefix_char}{LOG_FMT_SEP}{time} ') + msg