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

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: soothe-cli
-Version: 0.4.7
+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.7 → soothe_cli-0.4.8}/src/soothe_cli/cli/execution/daemon.py

@@ -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.7 → 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.7 → 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.7 → 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.7 → soothe_cli-0.4.8}/src/soothe_cli/shared/tools/tool_formatters/file_ops.py

@@ -13,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.
     """
@@ -46,6 +46,26 @@ class FileOpsFormatter(BaseFormatter):
             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"):
@@ -143,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.
 
@@ -157,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 text_looks_like_error(result):
-            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:

{soothe_cli-0.4.7 → soothe_cli-0.4.8}/src/soothe_cli/tui/_session_stats.py

@@ -9,10 +9,13 @@ config, no widget imports) so that `app.py` can import `SessionStats` and
 from __future__ import annotations
 
 from dataclasses import dataclass, field
-from typing import Literal
 
-SpinnerStatus = Literal["Thinking", "Offloading"] | None
-"""Valid spinner display states, or `None` to hide."""
+SpinnerStatus = str | None
+"""Spinner line label, or `None` to hide.
+
+Common values include ``Thinking``, ``Offloading``, ``Writing`` (assistant streaming),
+``Tools`` (tool execution), and ``Synthesizing`` (goal-completion stream).
+"""
 
 
 @dataclass

{soothe_cli-0.4.7 → soothe_cli-0.4.8}/src/soothe_cli/tui/app.py

@@ -21,7 +21,7 @@ from typing import TYPE_CHECKING, Any, ClassVar, Literal
 
 from textual.app import App, ScreenStackError
 from textual.binding import Binding, BindingType
-from textual.containers import Container, VerticalScroll
+from textual.containers import Container, Vertical, VerticalScroll
 from textual.content import Content
 from textual.css.query import NoMatches
 from textual.message import Message
@@ -780,15 +780,16 @@ class SootheApp(App):
         # Main chat area with scrollable messages
         # VerticalScroll tracks user scroll intent for better auto-scroll behavior
         with VerticalScroll(id="chat"):
-            yield WelcomeBanner(
-                thread_id=self._lc_loop_id,
-                mcp_tool_count=self._mcp_tool_count,
-                connecting=self._connecting,
-                resuming=self._resume_thread_intent is not None,
-                local_server=self._server_kwargs is not None,
-                id="welcome-banner",
-            )
-            yield Container(id="messages")
+            with Vertical(id="chat-body"):
+                yield WelcomeBanner(
+                    thread_id=self._lc_loop_id,
+                    mcp_tool_count=self._mcp_tool_count,
+                    connecting=self._connecting,
+                    resuming=self._resume_thread_intent is not None,
+                    local_server=self._server_kwargs is not None,
+                    id="welcome-banner",
+                )
+                yield Container(id="messages")
         with Container(id="bottom-app-container"):
             yield Container(id="thinking-status")
             yield ChatInput(
@@ -1965,9 +1966,12 @@ class SootheApp(App):
 
         if self._loading_widget is None:
             # Create new
-            self._loading_widget = LoadingWidget(status)
+            turn_mono = self._inflight_turn_start if self._agent_running else None
+            self._loading_widget = LoadingWidget(status, turn_start_mono=turn_mono)
             await thinking_status.mount(self._loading_widget)
         else:
+            if self._agent_running:
+                self._loading_widget.set_turn_start_mono(self._inflight_turn_start)
             # Update existing
             self._loading_widget.set_status(status)
         # NOTE: Don't call anchor() here - it would re-anchor and drag user back

{soothe_cli-0.4.7 → soothe_cli-0.4.8}/src/soothe_cli/tui/app.tcss

@@ -16,14 +16,24 @@ Screen {
 /* Chat area - main scrollable messages area */
 #chat {
     height: 1fr;
-    padding: 1 2;
+    padding: 0 2;
     background: $background;
 }
 
+/* Fills at least the scroll viewport; bottom-aligns welcome + messages when the
+   thread is short so dead space sits above the banner instead of above the input. */
+#chat-body {
+    width: 1fr;
+    min-height: 100%;
+    height: auto;
+    layout: vertical;
+    align-vertical: bottom;
+}
+
 /* Welcome banner */
 #welcome-banner {
     height: auto;
-    margin-bottom: 1;
+    margin-bottom: 0;
 }
 
 /* Messages area — uses undocumented "stream" layout (Textual ≥5.2.0) for
@@ -36,14 +46,14 @@ Screen {
 /* Bottom app container - holds ChatInput (now inside scroll) */
 #bottom-app-container {
     height: auto;
-    margin-top: 1;
+    margin-top: 0;
     padding: 0 1;
 }
 
 /* Sticky thinking row directly above input box */
 #thinking-status {
     height: auto;
-    min-height: 1;
+    min-height: 0;
     margin-bottom: 0;
 }
 
@@ -175,7 +185,7 @@ QueuedUserMessage.-ascii {
 }
 
 ToolCallMessage.-ascii {
-    border-left: ascii $panel;
+    border-left: ascii $primary;
 }
 
 ToolCallMessage.-ascii:hover {