perf(tui): stabilize long-session scrolling

tests/test_tui_gateway_server.py

@@ -347,6 +347,34 @@ def test_complete_slash_includes_provider_alias():
     assert any(item["text"] == "provider" for item in resp["result"]["items"])
 
 
+def test_complete_slash_includes_tui_details_command():
+    resp = server.handle_request(
+        {"id": "1", "method": "complete.slash", "params": {"text": "/det"}}
+    )
+
+    assert any(item["text"] == "/details" for item in resp["result"]["items"])
+
+
+def test_complete_slash_details_args():
+    resp_root = server.handle_request(
+        {"id": "0", "method": "complete.slash", "params": {"text": "/details"}}
+    )
+    resp_section = server.handle_request(
+        {"id": "1", "method": "complete.slash", "params": {"text": "/details t"}}
+    )
+    resp_mode = server.handle_request(
+        {
+            "id": "2",
+            "method": "complete.slash",
+            "params": {"text": "/details thinking e"},
+        }
+    )
+
+    assert resp_root["result"]["replace_from"] == len("/details")
+    assert any(item["text"] == "thinking" for item in resp_section["result"]["items"])
+    assert any(item["text"] == "expanded" for item in resp_mode["result"]["items"])
+
+
 def test_config_set_reasoning_updates_live_session_and_agent(tmp_path, monkeypatch):
     monkeypatch.setattr(server, "_hermes_home", tmp_path)
     agent = types.SimpleNamespace(reasoning_config=None)

tui_gateway/server.py

@@ -3710,6 +3710,65 @@ def _(rid, params: dict) -> dict:
     return _ok(rid, {"items": items})
 
 
+def _details_completion_item(value: str, meta: str = "") -> dict:
+    return {"text": value, "display": value, "meta": meta}
+
+
+def _details_completions(text: str) -> list[dict] | None:
+    if not text.lower().startswith("/details"):
+        return None
+
+    stripped = text.strip()
+    if stripped and not "/details".startswith(stripped.lower().split()[0]):
+        return None
+
+    body = text[len("/details"):]
+    if body.startswith(" "):
+        body = body[1:]
+    parts = body.split()
+    has_trailing_space = text.endswith(" ")
+    sections = ("thinking", "tools", "subagents", "activity")
+    modes = ("hidden", "collapsed", "expanded")
+
+    if not body or (len(parts) == 0 and has_trailing_space):
+        return [
+            *[_details_completion_item(mode, "global mode") for mode in modes],
+            _details_completion_item("cycle", "cycle global mode"),
+            *[_details_completion_item(section, "section override") for section in sections],
+        ]
+
+    if len(parts) == 1 and not has_trailing_space:
+        prefix = parts[0].lower()
+        candidates = [*modes, "cycle", *sections]
+        return [
+            _details_completion_item(
+                candidate,
+                "section override" if candidate in sections else "global mode",
+            )
+            for candidate in candidates
+            if candidate.startswith(prefix) and candidate != prefix
+        ]
+
+    if len(parts) == 1 and has_trailing_space and parts[0].lower() in sections:
+        return [
+            *[_details_completion_item(mode, f"set {parts[0].lower()}") for mode in modes],
+            _details_completion_item("reset", f"clear {parts[0].lower()} override"),
+        ]
+
+    if len(parts) == 2 and not has_trailing_space and parts[0].lower() in sections:
+        prefix = parts[1].lower()
+        return [
+            _details_completion_item(
+                candidate,
+                f"clear {parts[0].lower()} override" if candidate == "reset" else f"set {parts[0].lower()}",
+            )
+            for candidate in (*modes, "reset")
+            if candidate.startswith(prefix) and candidate != prefix
+        ]
+
+    return []
+
+
 @method("complete.slash")
 def _(rid, params: dict) -> dict:
     text = params.get("text", "")
@@ -3742,6 +3801,11 @@ def _(rid, params: dict) -> dict:
                 "display": "/compact",
                 "meta": "Toggle compact display mode",
             },
+            {
+                "text": "/details",
+                "display": "/details",
+                "meta": "Control agent detail visibility",
+            },
             {
                 "text": "/logs",
                 "display": "/logs",
@@ -3753,6 +3817,17 @@ def _(rid, params: dict) -> dict:
                 item["text"] == extra["text"] for item in items
             ):
                 items.append(extra)
+
+        details_items = _details_completions(text)
+        if details_items is not None:
+            return _ok(
+                rid,
+                {
+                    "items": details_items,
+                    "replace_from": text.rfind(" ") + 1 if " " in text else len(text),
+                },
+            )
+
         return _ok(
             rid,
             {"items": items, "replace_from": text.rfind(" ") + 1 if " " in text else 1},

ui-tui/packages/hermes-ink/src/ink/components/App.tsx

@@ -29,7 +29,7 @@ import {
   FOCUS_IN,
   FOCUS_OUT
 } from '../termio/csi.js'
-import { DBP, DFE, DISABLE_MOUSE_TRACKING, EBP, EFE, HIDE_CURSOR, SHOW_CURSOR } from '../termio/dec.js'
+import { DBP, DFE, DISABLE_MOUSE_TRACKING, EBP, EFE, SHOW_CURSOR } from '../termio/dec.js'
 
 import AppContext from './AppContext.js'
 import { ClockProvider } from './ClockContext.js'
@@ -206,10 +206,9 @@ export default class App extends PureComponent<Props, State> {
     )
   }
   override componentDidMount() {
-    // In accessibility mode, keep the native cursor visible for screen magnifiers and other tools
-    if (this.props.stdout.isTTY) {
-      this.props.stdout.write(HIDE_CURSOR)
-    }
+    // Keep the native terminal cursor visible. Ink parks it at the declared
+    // input caret after each frame, so the terminal emulator provides the
+    // normal blinking block/bar without React-driven blink re-renders.
   }
   override componentWillUnmount() {
     if (this.props.stdout.isTTY) {
@@ -470,7 +469,7 @@ export default class App extends PureComponent<Props, State> {
       }
 
       if (this.props.stdout.isTTY) {
-        this.props.stdout.write(HIDE_CURSOR + EFE)
+        this.props.stdout.write(EFE)
       }
 
       this.inputEmitter.emit('resume')
@@ -569,18 +568,19 @@ function processKeysInBatch(app: App, items: ParsedInput[], _unused1: undefined,
 
 /** Exported for testing. Mutates app.props.selection and click/hover state. */
 export function handleMouseEvent(app: App, m: ParsedMouse): void {
-  // Allow disabling click handling while keeping wheel scroll (which goes
-  // through the keybinding system as 'wheelup'/'wheeldown', not here).
-  if (isMouseClicksDisabled()) {
-    return
-  }
-
   const sel = app.props.selection
   // Terminal coords are 1-indexed; screen buffer is 0-indexed
   const col = m.col - 1
   const row = m.row - 1
   const baseButton = m.button & 0x03
 
+  // Allow disabling app click/selection handling while keeping wheel scroll
+  // and DOM mouse dispatch alive. Put this after coordinate/button decoding
+  // and exempt non-left buttons so scrollbar/right-click handlers still work.
+  if (isMouseClicksDisabled() && baseButton === 0) {
+    return
+  }
+
   if (m.action === 'press') {
     if ((m.button & 0x20) !== 0 && baseButton === 3) {
       if (app.mouseCaptureTarget) {

ui-tui/packages/hermes-ink/src/ink/components/ScrollBox.tsx

@@ -122,6 +122,19 @@ function ScrollBox({ children, ref, stickyScroll, ...style }: PropsWithChildren<
     })
   }
 
+  const scrollByNow = (dy: number) => {
+    const el = domRef.current
+
+    if (!el) {
+      return
+    }
+
+    el.stickyScroll = false
+    el.scrollAnchor = undefined
+    el.pendingScrollDelta = (el.pendingScrollDelta ?? 0) + Math.floor(dy)
+    scrollMutated(el)
+  }
+
   useImperativeHandle(
     ref,
     (): ScrollBoxHandle => ({
@@ -155,22 +168,7 @@ function ScrollBox({ children, ref, stickyScroll, ...style }: PropsWithChildren<
         }
         scrollMutated(box)
       },
-      scrollBy(dy: number) {
-        const el = domRef.current
-
-        if (!el) {
-          return
-        }
-
-        el.stickyScroll = false
-        // Wheel input cancels any in-flight anchor seek — user override.
-        el.scrollAnchor = undefined
-        // Accumulate in pendingScrollDelta; renderer drains it at a capped
-        // rate so fast flicks show intermediate frames. Pure accumulator:
-        // scroll-up followed by scroll-down naturally cancels.
-        el.pendingScrollDelta = (el.pendingScrollDelta ?? 0) + Math.floor(dy)
-        scrollMutated(el)
-      },
+      scrollBy: scrollByNow,
       scrollToBottom() {
         const el = domRef.current