🎨 Palette: Hide terminal cursor during CLI spinner animation

💡 What: Hides the terminal cursor (`\x1b[?25l`) while the `Spinner` is active and restores it (`\x1b[?25h`) when the spinner finishes, fails, or goes out of scope (via the `Drop` trait).
🎯 Why: Prevents the terminal cursor from jumping around or rendering alongside the spinner frames during long-running tasks, creating a smoother and more polished CLI experience. The `Drop` implementation ensures safety so the cursor is not permanently hidden if the CLI crashes or is interrupted.
📸 Before/After: Visual noise during "🦀 Thinking..." is reduced.

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

Author: badMade

.Jules/palette.md

@@ -0,0 +1,3 @@
+## $(date +%Y-%m-%d) - [CLI Cursor UX Fix]
+**Learning:** This app is a Rust CLI, not a web frontend. Standard web accessibility rules (ARIA labels, DOM structure) do not apply. UX improvements here involve terminal manipulation (using `crossterm` for ANSI output, cursor hiding, colors, text alignment). Hiding the terminal cursor during a `Spinner` animation prevents the cursor from awkwardly jumping or rendering alongside spinner frames.
+**Action:** When implementing CLI UX changes that modify terminal state (like hiding a cursor), ALWAYS implement a `Drop` trait to ensure the state (like cursor visibility) is predictably restored if the process is interrupted or panics.

.port_sessions/108fa52a890542199d833cd93ed0429f.json

@@ -0,0 +1,8 @@
+{
+  "session_id": "108fa52a890542199d833cd93ed0429f",
+  "messages": [
+    "review MCP tool"
+  ],
+  "input_tokens": 3,
+  "output_tokens": 13
+}

.port_sessions/200c7e63f6f04a57ac6211a90e4e35d0.json

@@ -0,0 +1,9 @@
+{
+  "session_id": "200c7e63f6f04a57ac6211a90e4e35d0",
+  "messages": [
+    "review MCP tool",
+    "review MCP tool"
+  ],
+  "input_tokens": 6,
+  "output_tokens": 32
+}

.port_sessions/59b491f8b3b2439a957e0c680cd193bc.json

@@ -0,0 +1,9 @@
+{
+  "session_id": "59b491f8b3b2439a957e0c680cd193bc",
+  "messages": [
+    "review MCP tool",
+    "review MCP tool"
+  ],
+  "input_tokens": 6,
+  "output_tokens": 32
+}

.port_sessions/d5662f5a103444a9984129c1c8b01597.json

@@ -0,0 +1,9 @@
+{
+  "session_id": "d5662f5a103444a9984129c1c8b01597",
+  "messages": [
+    "review MCP tool",
+    "review MCP tool"
+  ],
+  "input_tokens": 6,
+  "output_tokens": 32
+}

rust/.claw/sessions/session-1775386832313-0.jsonl

@@ -1,7 +1,7 @@
 use std::fmt::Write as FmtWrite;
 use std::io::{self, Write};
 
-use crossterm::cursor::{MoveToColumn, RestorePosition, SavePosition};
+use crossterm::cursor::{Hide, MoveToColumn, RestorePosition, SavePosition, Show};
 use crossterm::style::{Color, Print, ResetColor, SetForegroundColor, Stylize};
 use crossterm::terminal::{Clear, ClearType};
 use crossterm::{execute, queue};
@@ -47,6 +47,16 @@ impl Default for ColorTheme {
 #[derive(Debug, Default, Clone, PartialEq, Eq)]
 pub struct Spinner {
     frame_index: usize,
+    cursor_hidden: bool,
+}
+
+impl Drop for Spinner {
+    fn drop(&mut self) {
+        if self.cursor_hidden {
+            let mut out = io::stdout();
+            let _ = execute!(out, Show);
+        }
+    }
 }
 
 impl Spinner {
@@ -63,6 +73,10 @@ impl Spinner {
         theme: &ColorTheme,
         out: &mut impl Write,
     ) -> io::Result<()> {
+        if !self.cursor_hidden {
+            queue!(out, Hide)?;
+            self.cursor_hidden = true;
+        }
         let frame = Self::FRAMES[self.frame_index % Self::FRAMES.len()];
         self.frame_index += 1;
         queue!(
@@ -85,13 +99,15 @@ impl Spinner {
         out: &mut impl Write,
     ) -> io::Result<()> {
         self.frame_index = 0;
+        self.cursor_hidden = false;
         execute!(
             out,
             MoveToColumn(0),
             Clear(ClearType::CurrentLine),
             SetForegroundColor(theme.spinner_done),
             Print(format!("✔ {label}\n")),
-            ResetColor
+            ResetColor,
+            Show
         )?;
         out.flush()
     }
@@ -103,13 +119,15 @@ impl Spinner {
         out: &mut impl Write,
     ) -> io::Result<()> {
         self.frame_index = 0;
+        self.cursor_hidden = false;
         execute!(
             out,
             MoveToColumn(0),
             Clear(ClearType::CurrentLine),
             SetForegroundColor(theme.spinner_failed),
             Print(format!("✘ {label}\n")),
-            ResetColor
+            ResetColor,
+            Show
         )?;
         out.flush()
     }