fix(preview): handle ansi changes properly

Author: NSPC911

src/rovr/components/text_preview.py

@@ -8,6 +8,7 @@
 from rich.syntax import Syntax
 from rich.text import Text
 from textual.geometry import Size
+from textual.reactive import reactive
 from textual.scroll_view import ScrollView
 from textual.strip import Strip
 
@@ -107,6 +108,8 @@ class WindowedTextPreview(ScrollView):
     }
     """
 
+    ansi_color: reactive[bool | None] = reactive(None, init=False)
+
     def __init__(
         self,
         lines: Sequence[str] | Sequence[Text],
@@ -123,6 +126,13 @@ def __init__(
         self._rendered_window: tuple[int, int, int, int, list[Strip]] | None = None
         self.update_preview(lines, language=language, line_numbers=line_numbers)
 
+    def watch_ansi_color(self, ansi_color: bool | None) -> None:
+        # just re-render window because bg can be weird
+        self._rendered_window = None
+        self.update_preview(
+            self._lines, language=self._language, line_numbers=self._line_numbers
+        )
+
     def update_preview(
         self,
         lines: Sequence[str] | Sequence[Text],

src/rovr/core/preview_container.py

@@ -1038,7 +1038,19 @@ def show_bat_file_preview(self) -> bool:
         Raises:
             ExitNow: If this preview request is no longer active.
         """
-        bat_executable = config["plugins"]["bat"]["executable"]
+        from shutil import which
+
+        # luckily shutil.which handles absolute path for us, thanks
+        if not which(bat_executable := config["plugins"]["bat"]["executable"]):
+            config["plugins"]["bat"]["enabled"] = False
+            self.notify(
+                f"Bat executable not found: {bat_executable}.\nDisabling bat plugin.",
+                title="Plugins: Bat",
+                severity="warning",
+                markup=False,
+            )
+            return False
+
         command = [
             bat_executable,
             "--force-colorization",
@@ -1115,9 +1127,9 @@ def request_page(page: int) -> None:
                     return False
 
                 text_preview = self.call_from_thread(
-                    WindowedTextPreview,
-                    source,
-                    classes="text_preview bat_preview",
+                    lambda: WindowedTextPreview(
+                        source, classes="text_preview bat_preview"
+                    ).data_bind(type(self.app).ansi_color)
                 )
                 self.call_from_thread(self.mount, text_preview)
                 if should_cancel():
@@ -1173,7 +1185,11 @@ def show_normal_file_preview(self) -> None:
         max_file_size = config["interface"]["preview_text"]["max_file_size"]
         signature = (str(max_file_size), TEXT_PREVIEW_CACHE_VERSION)
         content: str | None = load_from_cache(
-            realpath, "windowed_text", stat_result, signature, pass_as=str
+            realpath,
+            "windowed_text",
+            stat_result,
+            signature,
+            pass_as=str,
         )
         if content is None:
             try:
@@ -1239,7 +1255,7 @@ def show_normal_file_preview(self) -> None:
                         language=language,
                         line_numbers=config["interface"]["show_line_numbers"],
                         classes="text_preview",
-                    )
+                    ).data_bind(type(self.app).ansi_color)
                 )
             )