fix(watcher): escape exception strings to prevent rich MarkupError crashes

Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com>
Co-authored-by: shenald-dev <245350826+shenald-dev@users.noreply.github.com>

Author: shenald-dev

src/echo/watcher.py

@@ -161,7 +161,7 @@ def _run_command(self, event_path):
                     else:
                         console.print(f"[red]✖ Command failed with exit code {process.returncode}.[/red]")
         except Exception as e:
-            console.print(f"[bold red]Error executing command: {e}[/bold red]")
+            console.print(f"[bold red]Error executing command: {escape(str(e))}[/bold red]")
 
     def _is_ignored_impl(self, path: str) -> bool:
         if path.startswith(self._abs_base_path):

tests/test_markup.py

@@ -1,5 +1,7 @@
 import time
 from echo.watcher import CommandRunnerHandler
+import subprocess
+
 from unittest.mock import MagicMock
 
 def test_rich_markup_crash_escaping():
@@ -24,3 +26,23 @@ def test_rich_markup_crash_escaping():
     if handler.current_process:
         handler.current_process.terminate()
         handler.current_process.wait()
+
+
+def test_rich_error_crash():
+    handler = CommandRunnerHandler("echo test")
+
+    class MockException(Exception):
+        def __str__(self):
+            return "File not found: [missing.py]"
+
+    original_popen = subprocess.Popen
+
+    def fake_popen(*args, **kwargs):
+        raise MockException()
+
+    subprocess.Popen = fake_popen
+    try:
+        # this should crash rich with MarkupError if not escaped
+        handler._run_command("test.txt")
+    finally:
+        subprocess.Popen = original_popen