First pass of removing files and attributes related to the Murfey TUI app

Author: tieneupin

pyproject.toml

@@ -82,7 +82,6 @@ Documentation = "https://github.com/DiamondLightSource/python-murfey"
 GitHub = "https://github.com/DiamondLightSource/python-murfey"
 [project.scripts]
 "murfey.add_user" = "murfey.cli.add_user:run"
-"murfey.client" = "murfey.client.tui.main:run"
 "murfey.create_db" = "murfey.cli.create_db:run"
 "murfey.db_sql" = "murfey.cli.murfey_db_sql:run"
 "murfey.decrypt_password" = "murfey.cli.decrypt_db_password:run"
@@ -121,7 +120,6 @@ include-package-data = true
 zip-safe = false
 
 [tool.setuptools.package-data]
-"murfey.client.tui" = ["*.css"]
 "murfey.util" = ["route_manifest.yaml"]
 
 [tool.setuptools.packages.find]

src/murfey/client/customlogging.py

@@ -3,10 +3,6 @@
 import json
 import logging
 
-from rich.logging import RichHandler
-
-from murfey.client.tui.screens import LogBook
-
 logger = logging.getLogger("murfey.client.customlogging")
 
 
@@ -30,23 +26,3 @@ def emit(self, record):
             self._callback(self.prepare(record))
         except Exception:
             self.handleError(record)
-
-
-class DirectableRichHandler(RichHandler):
-    def __init__(self, text_log: LogBook | None = None, **kwargs):
-        super().__init__(**kwargs)
-        self.text_log = text_log
-        self.redirect = False
-        self._last_time = None
-
-    def emit(self, record):
-        try:
-            if self.text_log:
-                message = self.format(record)
-                message_renderable = self.render_message(record, message)
-                log_renderable = self.render(
-                    record=record, traceback=None, message_renderable=message_renderable
-                )
-                self.text_log.post_message(self.text_log.Log(log_renderable))
-        except Exception:
-            self.handleError(record)

src/murfey/client/rsync.py

@@ -19,7 +19,6 @@
 from typing import Awaitable, Callable, List, NamedTuple
 from urllib.parse import ParseResult
 
-from murfey.client.tui.status_bar import StatusBar
 from murfey.util.client import Observer
 
 logger = logging.getLogger("murfey.client.rsync")
@@ -59,7 +58,6 @@ def __init__(
         server_url: ParseResult,
         stop_callback: Callable = lambda *args, **kwargs: None,
         local: bool = False,
-        status_bar: StatusBar | None = None,
         do_transfer: bool = True,
         remove_files: bool = False,
         required_substrings_for_removal: List[str] = [],
@@ -107,7 +105,6 @@ def __init__(
         )
         self._stopping = False
         self._halt_thread = False
-        self._statusbar = status_bar
 
     def __repr__(self) -> str:
         return f"<RSyncer ({self._basepath} → {self._remote}) [{self.status}]"
@@ -116,15 +113,12 @@ def __repr__(self) -> str:
     def from_rsyncer(cls, rsyncer: RSyncer, **kwargs):
         kwarguments_from_rsyncer = {
             "local": rsyncer._local,
-            "status_bar": rsyncer._statusbar,
             "do_transfer": rsyncer._do_transfer,
             "remove_files": rsyncer._remove_files,
             "notify": rsyncer._notify,
         }
         kwarguments_from_rsyncer.update(kwargs)
         assert isinstance(kwarguments_from_rsyncer["local"], bool)
-        if kwarguments_from_rsyncer["status_bar"] is not None:
-            assert isinstance(kwarguments_from_rsyncer["status_bar"], StatusBar)
         assert isinstance(kwarguments_from_rsyncer["do_transfer"], bool)
         assert isinstance(kwarguments_from_rsyncer["remove_files"], bool)
         assert isinstance(kwarguments_from_rsyncer["notify"], bool)
@@ -134,7 +128,6 @@ def from_rsyncer(cls, rsyncer: RSyncer, **kwargs):
             rsyncer._rsync_module,
             rsyncer._server_url,
             local=kwarguments_from_rsyncer["local"],
-            status_bar=kwarguments_from_rsyncer["status_bar"],
             do_transfer=kwarguments_from_rsyncer["do_transfer"],
             remove_files=kwarguments_from_rsyncer["remove_files"],
             notify=kwarguments_from_rsyncer["notify"],
@@ -398,12 +391,6 @@ def parse_stdout(line: str):
                     return
 
                 self._files_transferred += 1
-                if self._statusbar:
-                    with self._statusbar.lock:
-                        self._statusbar.transferred = [
-                            self._statusbar.transferred[0] + 1,
-                            self._statusbar.transferred[1],
-                        ]
                 current_outstanding = self.queue.unfinished_tasks - (
                     self._files_transferred - previously_transferred
                 )