soothe-cli 0.4.6__tar.gz → 0.4.8__tar.gz

{soothe_cli-0.4.6 → soothe_cli-0.4.8}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: soothe-cli
-Version: 0.4.6
+Version: 0.4.8
 Summary: Soothe CLI client - communicates with daemon via WebSocket
 Project-URL: Homepage, https://github.com/OpenSoothe/soothe
 Project-URL: Documentation, https://soothe.readthedocs.io

{soothe_cli-0.4.6 → soothe_cli-0.4.8}/src/soothe_cli/cli/execution/daemon.py

@@ -20,8 +20,8 @@ from soothe_sdk.client import (
 
 from soothe_cli.cli.execution.headless_renderer import HeadlessCliRenderer
 from soothe_cli.shared import EventProcessor
-from soothe_cli.shared.presentation_engine import PresentationEngine
-from soothe_cli.shared.subagent_routing import parse_subagent_from_input
+from soothe_cli.shared.commands.subagent_routing import parse_subagent_from_input
+from soothe_cli.shared.core.presentation_engine import PresentationEngine
 
 logger = logging.getLogger(__name__)
 
@@ -47,7 +47,6 @@ async def run_headless_via_daemon(
 
     ws_url = websocket_url_from_config(cfg)
     client = WebSocketClient(url=ws_url)
-    final_output_mode = getattr(cfg, "final_output_mode", "streaming")
 
     try:
         await connect_websocket_with_retries(client)
@@ -85,7 +84,6 @@ async def run_headless_via_daemon(
         renderer = HeadlessCliRenderer()
         processor = EventProcessor(
             renderer,
-            final_output_mode=final_output_mode,
             presentation_engine=presentation,
             headless_output=True,
         )

{soothe_cli-0.4.6 → soothe_cli-0.4.8}/src/soothe_cli/cli/execution/headless_renderer.py

@@ -12,7 +12,7 @@ from typing import Any
 
 from rich.console import Console
 
-from soothe_cli.shared.renderer_base import RendererBase
+from soothe_cli.shared.rendering.renderer_base import RendererBase
 
 
 class HeadlessCliRenderer(RendererBase):

{soothe_cli-0.4.6 → soothe_cli-0.4.8}/src/soothe_cli/cli/stream/pipeline.py

@@ -30,13 +30,13 @@ from soothe_cli.cli.stream.formatter import (
     format_subagent_done,
     format_subagent_milestone,
 )
-from soothe_cli.shared.essential_events import (
+from soothe_cli.shared.core.presentation_engine import PresentationEngine
+from soothe_cli.shared.events.essential_events import (
     LOOP_REASON_EVENT_TYPE,
     is_goal_start_event_type,
     is_step_complete_event_type,
     is_step_start_event_type,
 )
-from soothe_cli.shared.presentation_engine import PresentationEngine
 
 logger = logging.getLogger(__name__)
 

{soothe_cli-0.4.6 → soothe_cli-0.4.8}/src/soothe_cli/config/cli_config.py

@@ -27,8 +27,6 @@ class CLIConfig:
     # logging_level: DEBUG/INFO/… for ~/.soothe/logs/soothe-cli.log; None = default INFO.
     logging_level: str | None = None
 
-    final_output_mode: str = "streaming"
-
     # Output streaming overrides (RFC-614)
     output_streaming_enabled: bool | None = None
     """Override daemon streaming enabled setting."""
@@ -107,26 +105,15 @@ class CLIConfig:
         transports = daemon_section.get("transports", {})
         websocket = transports.get("websocket", {})
         websocket_legacy = data.get("websocket", {})
-        ui_section = data.get("ui", {})
 
         raw_level = data.get("logging_level")
         if raw_level is not None and not isinstance(raw_level, str):
             raw_level = None
 
-        raw_final_output_mode = data.get("final_output_mode")
-        if raw_final_output_mode is None and isinstance(ui_section, dict):
-            raw_final_output_mode = ui_section.get("final_output_mode")
-        if not isinstance(raw_final_output_mode, str):
-            raw_final_output_mode = "streaming"
-        final_output_mode = raw_final_output_mode.strip().lower()
-        if final_output_mode not in {"streaming", "batch"}:
-            final_output_mode = "streaming"
-
         return cls(
             daemon_host=websocket.get("host", websocket_legacy.get("host", "127.0.0.1")),
             daemon_port=websocket.get("port", websocket_legacy.get("port", 8765)),
             logging_level=raw_level,
-            final_output_mode=final_output_mode,
             soothe_home=Path(data.get("home", str(Path.home() / ".soothe"))),
         )
 

{soothe_cli-0.4.6 → soothe_cli-0.4.8}/src/soothe_cli/shared/core/event_processor.py

@@ -82,7 +82,6 @@ class EventProcessor:
         self,
         renderer: RendererProtocol,
         *,
-        final_output_mode: str = "streaming",
         presentation_engine: PresentationEngine | None = None,
         tui_debug: bool = False,
         headless_output: bool = False,
@@ -98,9 +97,6 @@ class EventProcessor:
         """
         self._renderer = renderer
         self._headless_output = headless_output
-        self._final_output_mode = (
-            final_output_mode if final_output_mode in {"streaming", "batch"} else "streaming"
-        )
         self._tui_debug = tui_debug
 
         rebind = getattr(renderer, "_rebind_presentation", None)
@@ -1103,12 +1099,10 @@ class EventProcessor:
             Dict with enabled, mode, and synthesis_streaming fields.
         """
         # Use defaults - streaming is enabled by default per RFC-614
-        # final_output_mode controls batch/streaming display mode
+        # Always use streaming mode
         config = {
             "enabled": True,
-            "mode": self._final_output_mode
-            if self._final_output_mode in {"streaming", "batch"}
-            else "streaming",
+            "mode": "streaming",
             "synthesis_streaming": True,
         }
 

{soothe_cli-0.4.6 → soothe_cli-0.4.8}/src/soothe_cli/shared/core/processor_state.py

@@ -13,7 +13,7 @@ from typing import TYPE_CHECKING, Any
 if TYPE_CHECKING:
     from soothe_sdk.client.schemas import Plan
 
-    from soothe_cli.shared.events.stream_accumulator import StreamingTextAccumulator
+from soothe_cli.shared.events.stream_accumulator import StreamingTextAccumulator
 
 
 @dataclass
@@ -56,9 +56,7 @@ class ProcessorState:
 
     # Unified streaming text accumulator (RFC-614)
     streaming_accumulator: StreamingTextAccumulator = field(
-        default_factory=lambda: __import__(
-            "soothe_cli.shared.stream_accumulator", fromlist=["StreamingTextAccumulator"]
-        ).StreamingTextAccumulator()
+        default_factory=StreamingTextAccumulator
     )
     """Unified streaming text accumulator with namespace isolation."""
 

soothe_cli-0.4.8/src/soothe_cli/shared/tools/__init__.py

@@ -0,0 +1,96 @@
+"""Tool call/result handling utilities."""
+
+from soothe_cli.shared.tools.message_processing import (
+    accumulate_tool_call_chunks,
+    coerce_tool_call_args_to_dict,
+    extract_tool_args_dict,
+    extract_tool_brief,
+    finalize_pending_tool_call,
+    format_tool_call_args,
+    normalize_tool_calls_list,
+    strip_internal_tags,
+    tool_calls_have_any_arg_dict,
+    try_parse_pending_tool_call_args,
+)
+from soothe_cli.shared.tools.rendering import update_name_map_from_tool_calls
+from soothe_cli.shared.tools.tool_call_resolution import (
+    build_streaming_args_overlay,
+    infer_tool_name_from_call_id,
+    materialize_ai_blocks_with_resolved_tools,
+    tool_args_meaningful,
+)
+from soothe_cli.shared.tools.tool_card_payload import (
+    ToolResultCardPayload,
+    extract_tool_result_card_payload,
+    infer_tool_output_suggests_error,
+)
+from soothe_cli.shared.tools.tool_card_visibility import (
+    should_elide_completed_tool_call_message,
+    should_elide_stream_tool_card_mount,
+    should_elide_tool_card_no_info,
+)
+from soothe_cli.shared.tools.tool_formatters import (
+    BaseFormatter,
+    ExecutionFormatter,
+    FallbackFormatter,
+    FileOpsFormatter,
+    GoalFormatter,
+    MediaFormatter,
+    StructuredFormatter,
+    SubagentFormatter,
+    WebFormatter,
+)
+from soothe_cli.shared.tools.tool_message_format import (
+    format_content_block_for_tool_display,
+    format_tool_message_content,
+    run_python_envelope_indicates_failure,
+    try_parse_run_python_result_envelope,
+)
+from soothe_cli.shared.tools.tool_output_formatter import ToolBrief, ToolOutputFormatter
+
+__all__ = [
+    # Message processing
+    "accumulate_tool_call_chunks",
+    "coerce_tool_call_args_to_dict",
+    "extract_tool_args_dict",
+    "extract_tool_brief",
+    "finalize_pending_tool_call",
+    "format_tool_call_args",
+    "normalize_tool_calls_list",
+    "strip_internal_tags",
+    "tool_calls_have_any_arg_dict",
+    "try_parse_pending_tool_call_args",
+    # Rendering
+    "update_name_map_from_tool_calls",
+    # Tool call resolution
+    "build_streaming_args_overlay",
+    "infer_tool_name_from_call_id",
+    "materialize_ai_blocks_with_resolved_tools",
+    "tool_args_meaningful",
+    # Tool card payload
+    "ToolResultCardPayload",
+    "extract_tool_result_card_payload",
+    "infer_tool_output_suggests_error",
+    # Tool card visibility
+    "should_elide_completed_tool_call_message",
+    "should_elide_stream_tool_card_mount",
+    "should_elide_tool_card_no_info",
+    # Tool message format
+    "format_content_block_for_tool_display",
+    "format_tool_message_content",
+    "run_python_envelope_indicates_failure",
+    "try_parse_run_python_result_envelope",
+    # Formatter
+    "ToolBrief",
+    "ToolOutputFormatter",
+    # Formatters
+    "BaseFormatter",
+    "ExecutionFormatter",
+    "FallbackFormatter",
+    "FileOpsFormatter",
+    "GoalFormatter",
+    "MediaFormatter",
+    "SubagentFormatter",
+    "StructuredFormatter",
+    "WebFormatter",
+]

soothe_cli-0.4.8/src/soothe_cli/shared/tools/_utils.py

@@ -0,0 +1,30 @@
+"""Internal utilities for tool formatting (not part of public API)."""
+
+
+def normalize_tool_name(tool_name: str) -> str:
+    """Normalize tool name to snake_case for comparison and lookup.
+
+    Args:
+        tool_name: Raw tool name (may contain dashes or spaces).
+
+    Returns:
+        Lowercase snake_case name.
+    """
+    return tool_name.lower().replace("-", "_").replace(" ", "_")
+
+
+def text_looks_like_error(text: str) -> bool:
+    """Return True if text content suggests a tool failure.
+
+    Checks for common error indicator substrings.
+
+    Args:
+        text: Tool output text to inspect.
+
+    Returns:
+        True if any error indicator is found.
+    """
+    if not text:
+        return False
+    lowered = text.lower()
+    return any(indicator in lowered for indicator in ("error", "failed", "exception", "traceback"))

{soothe_cli-0.4.6 → soothe_cli-0.4.8}/src/soothe_cli/shared/tools/message_processing.py

@@ -392,7 +392,7 @@ def format_tool_call_args(tool_name: str, tool_call: dict[str, Any]) -> str:
     def _is_path_arg_name(key: str) -> bool:
         return _PATH_ARG_PATTERN.match(key) is not None
 
-    max_value_length = 40  # Max length for displayed values
+    max_value_length = 50  # Max length for displayed values
 
     def _display_path_value(raw: str) -> str:
         out = convert_and_abbreviate_path(raw)

{soothe_cli-0.4.6 → soothe_cli-0.4.8}/src/soothe_cli/shared/tools/tool_card_payload.py

@@ -13,6 +13,7 @@ from collections.abc import Mapping
 from dataclasses import dataclass
 from typing import Any
 
+from soothe_cli.shared.tools._utils import text_looks_like_error
 from soothe_cli.shared.tools.tool_call_resolution import infer_tool_name_from_call_id
 from soothe_cli.shared.tools.tool_message_format import (
     format_tool_message_content,
@@ -38,8 +39,7 @@ def infer_tool_output_suggests_error(output_display: str, _tool_name: str = "")
 
     if not output_display:
         return False
-    lowered = output_display.lower()
-    return any(indicator in lowered for indicator in ("error", "failed", "exception", "traceback"))
+    return text_looks_like_error(output_display)
 
 
 @dataclass(frozen=True, slots=True)

{soothe_cli-0.4.6 → soothe_cli-0.4.8}/src/soothe_cli/shared/tools/tool_formatters/execution.py

@@ -4,6 +4,7 @@ from __future__ import annotations
 
 from typing import Any
 
+from soothe_cli.shared.tools._utils import normalize_tool_name, text_looks_like_error
 from soothe_cli.shared.tools.tool_formatters.base import BaseFormatter
 from soothe_cli.shared.tools.tool_message_format import try_parse_run_python_result_envelope
 from soothe_cli.shared.tools.tool_output_formatter import ToolBrief
@@ -37,10 +38,11 @@ class ExecutionFormatter(BaseFormatter):
             '✓ Done'
         """
         # Normalize tool name
-        normalized = tool_name.lower().replace("-", "_").replace(" ", "_")
+        normalized = normalize_tool_name(tool_name)
 
         # Route to specific formatter
-        if normalized == "run_command":
+        # execute/shell/bash are aliases of run_command (per TOOL_REGISTRY)
+        if normalized in ("run_command", "execute", "shell", "bash"):
             return self._format_run_command(result)
         if normalized == "run_python":
             return self._format_run_python(result)
@@ -83,9 +85,7 @@ class ExecutionFormatter(BaseFormatter):
             )
 
         # Check for other error patterns
-        error_indicators = ["failed:", "exception:", "traceback", "command not found"]
-        result_lower = result.lower()
-        if any(indicator in result_lower for indicator in error_indicators):
+        if text_looks_like_error(result):
             # Extract first line as error message
             first_line = result.partition("\n")[0].strip()
             return ToolBrief(
@@ -170,13 +170,7 @@ class ExecutionFormatter(BaseFormatter):
 
         # Handle string result (fallback — not the standard run_python JSON envelope)
         if isinstance(result, str):
-            lowered = result.lower()
-            if (
-                "traceback" in lowered
-                or "exception:" in lowered
-                or "failed:" in lowered
-                or result.startswith("Error:")
-            ):
+            if text_looks_like_error(result) or result.startswith("Error:"):
                 return ToolBrief(
                     icon="✗",
                     summary="Execution failed",
@@ -240,7 +234,7 @@ class ExecutionFormatter(BaseFormatter):
 
         # Handle string result (fallback)
         if isinstance(result, str):
-            if "error" in result.lower() or "failed" in result.lower():
+            if text_looks_like_error(result):
                 return ToolBrief(
                     icon="✗",
                     summary="Start failed",
@@ -280,7 +274,7 @@ class ExecutionFormatter(BaseFormatter):
             'Terminated PID 12345'
         """
         # Check for error
-        if "error" in result.lower() or "failed" in result.lower() or "not found" in result.lower():
+        if text_looks_like_error(result) or "not found" in result.lower():
             return ToolBrief(
                 icon="✗",
                 summary="Termination failed",

{soothe_cli-0.4.6 → soothe_cli-0.4.8}/src/soothe_cli/shared/tools/tool_formatters/fallback.py

@@ -6,6 +6,7 @@ from typing import Any
 
 from soothe_sdk.client.protocol import preview_first
 
+from soothe_cli.shared.tools._utils import text_looks_like_error
 from soothe_cli.shared.tools.tool_formatters.base import BaseFormatter
 from soothe_cli.shared.tools.tool_output_formatter import ToolBrief
 
@@ -40,8 +41,7 @@ class FallbackFormatter(BaseFormatter):
         # Handle string results
         if isinstance(result, str):
             # Check for error indicators
-            error_indicators = ["error:", "failed:", "exception:", "traceback"]
-            is_error = any(indicator in result.lower() for indicator in error_indicators)
+            is_error = text_looks_like_error(result)
 
             if is_error:
                 # Extract error message (first line or first 80 chars)

{soothe_cli-0.4.6 → soothe_cli-0.4.8}/src/soothe_cli/shared/tools/tool_formatters/file_ops.py

@@ -5,6 +5,7 @@ from __future__ import annotations
 import re
 from typing import Any
 
+from soothe_cli.shared.tools._utils import normalize_tool_name, text_looks_like_error
 from soothe_cli.shared.tools.tool_formatters.base import BaseFormatter
 from soothe_cli.shared.tools.tool_output_formatter import ToolBrief
 
@@ -12,8 +13,8 @@ from soothe_cli.shared.tools.tool_output_formatter import ToolBrief
 class FileOpsFormatter(BaseFormatter):
     """Formatter for file operation tools.
 
-    Handles: read_file, write_file, delete_file, list_files, search_files, glob,
-    grep, ls, file_info
+    Handles: read_file, write_file, edit_file, delete_file, list_files, search_files,
+    glob, grep, ls, file_info, edit_file_lines, insert_lines, delete_lines, apply_diff
 
     Provides semantic summaries with size, line count, and item count metrics.
     """
@@ -38,13 +39,33 @@ class FileOpsFormatter(BaseFormatter):
             '✓ Read 12 B (2 lines)'
         """
         # Normalize tool name
-        normalized = tool_name.lower().replace("-", "_").replace(" ", "_")
+        normalized = normalize_tool_name(tool_name)
 
         # Route to specific formatter
         if normalized == "read_file":
             return self._format_read_file(result)
         if normalized == "write_file":
             return self._format_write_file(result)
+        if normalized == "edit_file":
+            return self._format_file_mutation(
+                result, success_summary="Edited file", failure_summary="Edit failed"
+            )
+        if normalized == "edit_file_lines":
+            return self._format_file_mutation(
+                result, success_summary="Updated file", failure_summary="Line edit failed"
+            )
+        if normalized == "insert_lines":
+            return self._format_file_mutation(
+                result, success_summary="Inserted lines", failure_summary="Insert failed"
+            )
+        if normalized == "delete_lines":
+            return self._format_file_mutation(
+                result, success_summary="Deleted lines", failure_summary="Delete lines failed"
+            )
+        if normalized == "apply_diff":
+            return self._format_file_mutation(
+                result, success_summary="Applied patch", failure_summary="Patch failed"
+            )
         if normalized == "delete_file":
             return self._format_delete_file(result)
         if normalized in ("list_files", "ls"):
@@ -142,6 +163,28 @@ class FileOpsFormatter(BaseFormatter):
             metrics={"size_bytes": size_bytes, "lines": lines},
         )
 
+    def _format_file_mutation(
+        self,
+        result: str,
+        *,
+        success_summary: str,
+        failure_summary: str,
+    ) -> ToolBrief:
+        """Format success/error for tools that mutate file content (write, edit, patch, …)."""
+        if text_looks_like_error(result):
+            return ToolBrief(
+                icon="✗",
+                summary=failure_summary,
+                detail=self._truncate_text(result, 80),
+                metrics={"error": True},
+            )
+        return ToolBrief(
+            icon="✓",
+            summary=success_summary,
+            detail=None,
+            metrics={},
+        )
+
     def _format_write_file(self, result: str) -> ToolBrief:
         """Format write_file result.
 
@@ -156,25 +199,10 @@ class FileOpsFormatter(BaseFormatter):
         Example:
             >>> brief = formatter._format_write_file("Successfully wrote to file")
             >>> brief.summary
-            'Wrote 0 B'
+            'Wrote file'
         """
-        # Check for error
-        if "error" in result.lower() or "failed" in result.lower():
-            return ToolBrief(
-                icon="✗",
-                summary="Write failed",
-                detail=self._truncate_text(result, 80),
-                metrics={"error": True},
-            )
-
-        # Try to extract size from result (if available)
-        # Common patterns: "Wrote X bytes", "Successfully wrote X"
-        # For now, show simple success
-        return ToolBrief(
-            icon="✓",
-            summary="Wrote file",
-            detail=None,
-            metrics={},
+        return self._format_file_mutation(
+            result, success_summary="Wrote file", failure_summary="Write failed"
         )
 
     def _format_delete_file(self, result: str) -> ToolBrief:
@@ -194,7 +222,7 @@ class FileOpsFormatter(BaseFormatter):
             'Deleted'
         """
         # Check for error
-        if "error" in result.lower() or "failed" in result.lower():
+        if text_looks_like_error(result):
             return ToolBrief(
                 icon="✗",
                 summary="Delete failed",
@@ -226,7 +254,7 @@ class FileOpsFormatter(BaseFormatter):
             'Found 3 items'
         """
         # Check for error
-        if "error" in result.lower() or "failed" in result.lower():
+        if text_looks_like_error(result):
             return ToolBrief(
                 icon="✗",
                 summary="List failed",
@@ -265,7 +293,7 @@ class FileOpsFormatter(BaseFormatter):
             'Found 2 matches'
         """
         # Check for error
-        if "error" in result.lower() or "failed" in result.lower():
+        if text_looks_like_error(result):
             return ToolBrief(
                 icon="✗",
                 summary="Search failed",
@@ -311,12 +339,14 @@ class FileOpsFormatter(BaseFormatter):
             >>> brief.summary
             'Found 2 files'
         """
-        # Check for error
-        if "error" in result.lower() or "failed" in result.lower():
+        # Check for actual error message (not file paths containing error-like words)
+        # Glob results are newline-separated paths; errors start with "Error:"
+        if result.startswith("Error:"):
+            error_msg = result[6:].strip()
             return ToolBrief(
                 icon="✗",
                 summary="Glob failed",
-                detail=self._truncate_text(result, 80),
+                detail=self._truncate_text(error_msg, 80),
                 metrics={"error": True},
             )
 

{soothe_cli-0.4.6 → soothe_cli-0.4.8}/src/soothe_cli/shared/tools/tool_formatters/goal_formatter.py

@@ -4,6 +4,7 @@ from __future__ import annotations
 
 from typing import Any
 
+from soothe_cli.shared.tools._utils import normalize_tool_name, text_looks_like_error
 from soothe_cli.shared.tools.tool_formatters.base import BaseFormatter
 from soothe_cli.shared.tools.tool_output_formatter import ToolBrief
 
@@ -36,7 +37,7 @@ class GoalFormatter(BaseFormatter):
             '✓ Created goal g1'
         """
         # Normalize tool name
-        normalized = tool_name.lower().replace("-", "_").replace(" ", "_")
+        normalized = normalize_tool_name(tool_name)
 
         # Route to specific formatter
         if normalized == "create_goal":
@@ -100,7 +101,7 @@ class GoalFormatter(BaseFormatter):
 
         # Handle string result (fallback)
         if isinstance(result, str):
-            if "error" in result.lower() or "failed" in result.lower():
+            if text_looks_like_error(result):
                 return ToolBrief(
                     icon="✗",
                     summary="Create failed",
@@ -167,7 +168,7 @@ class GoalFormatter(BaseFormatter):
 
         # Handle string result (fallback)
         if isinstance(result, str):
-            if "error" in result.lower() or "failed" in result.lower():
+            if text_looks_like_error(result):
                 return ToolBrief(
                     icon="✗",
                     summary="List failed",
@@ -234,7 +235,7 @@ class GoalFormatter(BaseFormatter):
 
         # Handle string result (fallback)
         if isinstance(result, str):
-            if "error" in result.lower() or "failed" in result.lower():
+            if text_looks_like_error(result):
                 return ToolBrief(
                     icon="✗",
                     summary="Complete failed",
@@ -307,7 +308,7 @@ class GoalFormatter(BaseFormatter):
 
         # Handle string result (fallback)
         if isinstance(result, str):
-            if "error" in result.lower() or "failed" in result.lower():
+            if text_looks_like_error(result):
                 return ToolBrief(
                     icon="✗",
                     summary="Fail operation failed",

{soothe_cli-0.4.6 → soothe_cli-0.4.8}/src/soothe_cli/shared/tools/tool_formatters/media.py

@@ -4,6 +4,7 @@ from __future__ import annotations
 
 from typing import Any
 
+from soothe_cli.shared.tools._utils import normalize_tool_name, text_looks_like_error
 from soothe_cli.shared.tools.tool_formatters.base import BaseFormatter
 from soothe_cli.shared.tools.tool_output_formatter import ToolBrief
 
@@ -36,7 +37,7 @@ class MediaFormatter(BaseFormatter):
             '✓ Transcribed 45.2s (en)'
         """
         # Normalize tool name
-        normalized = tool_name.lower().replace("-", "_").replace(" ", "_")
+        normalized = normalize_tool_name(tool_name)
 
         # Route to specific formatter
         if normalized == "transcribe_audio":
@@ -104,7 +105,7 @@ class MediaFormatter(BaseFormatter):
 
         # Handle string result (fallback)
         if isinstance(result, str):
-            if "error" in result.lower() or "failed" in result.lower():
+            if text_looks_like_error(result):
                 return ToolBrief(
                     icon="✗",
                     summary="Transcription failed",
@@ -182,7 +183,7 @@ class MediaFormatter(BaseFormatter):
 
         # Handle string result (fallback)
         if isinstance(result, str):
-            if "error" in result.lower() or "failed" in result.lower():
+            if text_looks_like_error(result):
                 return ToolBrief(
                     icon="✗",
                     summary="Video info failed",
@@ -267,7 +268,7 @@ class MediaFormatter(BaseFormatter):
 
         # Handle string result (fallback)
         if isinstance(result, str):
-            if "error" in result.lower() or "failed" in result.lower():
+            if text_looks_like_error(result):
                 return ToolBrief(
                     icon="✗",
                     summary="Image analysis failed",

{soothe_cli-0.4.6 → soothe_cli-0.4.8}/src/soothe_cli/shared/tools/tool_formatters/subagent.py

@@ -7,6 +7,7 @@ from __future__ import annotations
 
 from typing import Any
 
+from soothe_cli.shared.tools._utils import text_looks_like_error
 from soothe_cli.shared.tools.tool_formatters.base import BaseFormatter
 from soothe_cli.shared.tools.tool_output_formatter import ToolBrief
 
@@ -36,8 +37,7 @@ class SubagentFormatter(BaseFormatter):
         # Handle string results
         if isinstance(result, str):
             # Check for error indicators
-            error_indicators = ["error:", "failed:", "exception:", "traceback"]
-            is_error = any(indicator in result.lower() for indicator in error_indicators)
+            is_error = text_looks_like_error(result)
 
             if is_error:
                 # Extract first line of error