dsh-neotui 0.0.23 → 0.0.25

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-neotui",
-  "version": "0.0.23",
+  "version": "0.0.25",
   "description": "Neo-TUI: mouse-driven terminal UI client for DeepSeek Harness (B-tier per dsh-tui-design.md)",
   "type": "module",
   "bin": { "dsh-neotui": "bin/dsh-tui.js" },

package/src/views.js

@@ -565,9 +565,12 @@ export class ChatView extends Widget {
     this.expanded = new Set();   // node indexes (user-message full text)
     this.expandedTools = new Set();
     this.collapsedBlocks = new Set(); // per-block COLLAPSE (default expanded): `${realIdx}:${bi}`
+    this.blockHeights = new Map();    // collapsed block → its rendered height when collapsed
+                                       // (fixed-height fold: the space below stays put)
     this.thinkMode = "expanded";        // think blocks: expanded by default (t toggles)
     this.bashMode = "expanded";         // tool blocks: expanded | collapsed (b toggles)
     this.todosVisible = true;           // todo block above the input (Shift+T toggles)
+    this.todoSeen = false;             // once seen, the todo box keeps its height
     this.running = false;
     this.hasMore = false;
     this.loadingOlder = false;
@@ -715,11 +718,16 @@ export class ChatView extends Widget {
     }
   }
 
-  /** Height of the collapsible todo block (0 when empty or collapsed). */
+  /** Height of the collapsible todo block. FROZEN at the max (1 header + 6
+   *  items) once the session has ever shown todos: the harness updates the
+   *  list in the background while the user reads, and a height that tracks
+   *  the count reflows the whole layout every time (the idle 2-line shifts). */
   todoHeight() {
     const todos = this.app.todos;
-    if (!this.todosVisible || !todos || todos.length === 0) return 0;
-    return Math.min(todos.length, 6) + 1; // 1 header row + up to 6 todos
+    if (!this.todosVisible) return 0;
+    if (!todos || todos.length === 0) return this.todoSeen ? 7 : 0;
+    this.todoSeen = true;
+    return 7; // fixed max height — short lists just show fewer rows
   }
 
   inputChanged() {
@@ -906,28 +914,11 @@ export class ChatView extends Widget {
       this.#clickLog(`toggle mark=${JSON.stringify(info)} kind=${node.kind} blockIdx=${info.blockIdx} preHeaderRow=${preHeaderRow} topKey=${topKey} topOffset=${topOffset}`);
     }
     const reanchor = () => {
-      // COLLAPSING a block by clicking one of its CONTENT lines: the clicked
-      // text itself vanishes, and the content below slides up by the fold
-      // size — the "4-line offset" the user sees. Land the fold's LAST line
-      // (the trailer/summary) exactly at the clicked row so the eye stays
-      // anchored and nothing below visibly shifts.
-      if (collapsing && pressY !== null && pressY !== preHeaderIdx && pressRow !== null && pressRow >= 0) {
-        let lastIdx = -1;
-        for (let i = 0; i < this.lineMap.length; i++) {
-          if (match(this.lineMap[i])) lastIdx = i;
-        }
-        if (lastIdx >= 0) {
-          const sy = lastIdx - pressRow;
-          this.view.scrollY = Math.max(0, sy);
-          if (sy > this.view.maxScroll()) this.view.anchorLock = sy;
-          return;
-        }
-      }
+      // Fixed-height folds keep the buffer size UNCHANGED (the collapsed
+      // block pads its recorded height with ghost rows), so nothing below
+      // the fold can move — the anchor is only a safety net now.
       // primary anchor: the clicked block's header stays at its pre-click
       // viewport row (zero shift) — only when the header was ON-SCREEN.
-      // When the fold removed the tail content below, the anchored position
-      // may exceed maxScroll: hold it via the view's anchorLock instead of
-      // clamping (the clamp is exactly the slide the user sees).
       if (preHeaderRow !== null && preHeaderRow >= 0 && preHeaderIdx >= 0) {
         const h2 = firstNonEmpty(match);
         if (h2 >= 0) {
@@ -951,17 +942,25 @@ export class ChatView extends Widget {
       if (target > this.view.maxScroll()) this.view.anchorLock = target;
     };
     // never park the viewport on a blank separator row — it reads as a
-    // spurious offset right after the click
+    // spurious offset right after the click. Ghost rows of a fixed-height
+    // fold are INTENTIONAL blanks (marked with a blockIdx) — leave them.
     const nudge = () => {
       while (
         this.view.scrollY < this.lineMap.length - 1 &&
-        !(this.lines[this.view.scrollY] ?? []).some((g) => g.t.trim() !== "")
+        !(this.lines[this.view.scrollY] ?? []).some((g) => g.t.trim() !== "") &&
+        (this.lineMap[this.view.scrollY]?.blockIdx ?? null) === null
       ) this.view.scrollY++;
     };
     if (node.kind === "assistant" && info.blockIdx !== null) {
       const b = node.blocks[info.blockIdx];
       if (b && (b.kind === "tool" || b.kind === "reasoning" || b.kind === "other" || b.kind === "text")) {
         const key = `${info.nodeIdx}:${info.blockIdx}`;
+        // fixed-height fold: remember how tall the block is right now so the
+        // collapsed form can pad the SAME height (nothing below moves)
+        let firstM = -1, lastM = -1;
+        for (let i = 0; i < this.lineMap.length; i++) {
+          if (match(this.lineMap[i])) { if (firstM < 0) firstM = i; lastM = i; }
+        }
         if (b.kind === "reasoning") {
           // clean two-state override: expand ⇄ collapse (never a no-op click)
           const open = this.expanded.has(key) || (!this.collapsedBlocks.has(key) && this.thinkMode === "expanded");
@@ -973,6 +972,8 @@ export class ChatView extends Widget {
           if (this.collapsedBlocks.has(key)) this.collapsedBlocks.delete(key);
           else this.collapsedBlocks.add(key);
         }
+        if (collapsing && firstM >= 0) this.blockHeights.set(key, lastM - firstM + 1);
+        else if (!collapsing) this.blockHeights.delete(key);
         this.#rebuild();
         reanchor(); nudge();
         if (process.env.DSH_TUI_DEBUG_CLICK) {
@@ -1116,6 +1117,7 @@ export class ChatView extends Widget {
                 timing = " 已完成";
               }
               const thinkMeta = `${stepTag}（${b.text?.length ?? 0} 字）${timing}`;
+              const blkLines0 = lines.length;
               lines.push([{ t: "💭 思考" + (b.streaming ? "…" : "") + thinkMeta + (open ? " [t 折叠]" : " [t 展开]"), fg: K.FAINT }]);
               mark(realIdx, bi);
               if (open) {
@@ -1128,6 +1130,13 @@ export class ChatView extends Widget {
                   mark(realIdx, bi);
                 }
               }
+              // fixed-height fold: pad the recorded height with ghost rows so
+              // nothing below the block moves
+              if (!open) {
+                const h = this.blockHeights.get(key) ?? 0;
+                const filler = Math.max(0, h - (lines.length - blkLines0));
+                for (let fi = 0; fi < filler; fi++) { lines.push([{ t: "" }]); mark(realIdx, bi); }
+              }
               sep();
             } else if (b.kind === "tool") {
               const key = `${realIdx}:${bi}`;
@@ -1142,6 +1151,7 @@ export class ChatView extends Widget {
               const glyph = orphan ? "✗" : running ? "⏳" : exitCode !== undefined && exitCode !== 0 ? "✗" : "✓";
               const card = b.view ? renderToolCard(b.view, w, open) : [];
               beginCard(status);
+              const blkLines0 = lines.length;
               let timing = "";
               if (running) {
                 timing = ` 已经过 ${fmtDuration(Date.now() - (b.startedAt ?? Date.now()))}`;
@@ -1165,6 +1175,11 @@ export class ChatView extends Widget {
                   lines.push([{ t: "  " + truncate(summary, w - 6), fg: K.FAINT }]);
                   mark(realIdx, bi);
                 }
+                // fixed-height fold: pad the recorded height with ghost rows so
+                // nothing below the block moves
+                const h = this.blockHeights.get(key) ?? 0;
+                const filler = Math.max(0, h - (lines.length - blkLines0));
+                for (let fi = 0; fi < filler; fi++) { lines.push([{ t: "" }]); mark(realIdx, bi); }
               }
               if (open) {
                 for (const ln of card) { lines.push(ln); mark(realIdx, bi); }
@@ -1187,6 +1202,7 @@ export class ChatView extends Widget {
               sep();
             } else {
               beginCard("CARD");
+              const blkLines0 = lines.length;
               // Text blocks collapse per-block like tool/reasoning blocks:
               // collapsed shows a 3-line preview + trailer, expanded the whole
               // text. Clicking (or the right-click 展开/折叠) toggles it.
@@ -1210,6 +1226,13 @@ export class ChatView extends Widget {
                 lines.push([{ t: `  …共 ${text.length} 字（点击展开）`, fg: K.FAINT }]);
                 mark(realIdx, bi);
               }
+              // fixed-height fold: pad the recorded height with ghost rows so
+              // nothing below the block moves
+              if (!open) {
+                const h = this.blockHeights.get(key) ?? 0;
+                const filler = Math.max(0, h - (lines.length - blkLines0));
+                for (let fi = 0; fi < filler; fi++) { lines.push([{ t: "" }]); mark(realIdx, bi); }
+              }
               sep();
             }
           }
@@ -1702,7 +1725,9 @@ export class App {
   }
 
   footerHeight() {
-    return 2 + (this.jobs?.length ? 1 : 0);
+    // Constant 3 rows: the jobs summary row is always present, so jobs
+    // arriving/completing in the background never reflow the layout.
+    return 3;
   }
 
   layout() {
@@ -2271,6 +2296,7 @@ export class App {
     this.chat.cache.clear();
     this.chat.nodes = [];
     this.chat.collapsedBlocks.clear();
+    this.chat.blockHeights.clear();
     this.chat.expanded.clear();
     this.chat.expandedTools.clear();
     this.chat.queueRebuild();
@@ -2841,11 +2867,13 @@ export class App {
       if (stats.ttftMs) row1.right.push({ t: ` 首响${Math.round(stats.ttftMs / stats.ttftSteps)}ms `, fg: T.FAINT, bg: T.STATUSBG });
     }
     rows.push(row1);
-    // ── row 2: background jobs — one summary line, never per-job noise ──
-    if (this.jobs?.length) {
-      const running = this.jobs.filter((j) => j.status === "running").length;
-      const done = this.jobs.filter((j) => j.status === "completed").length;
-      const failed = this.jobs.filter((j) => j.status === "failed").length;
+    // ── row 2: background jobs — one summary line, always present so its
+    //    appearance/disappearance never reflows the layout ──
+    {
+      const jobs = this.jobs ?? [];
+      const running = jobs.filter((j) => j.status === "running").length;
+      const done = jobs.filter((j) => j.status === "completed").length;
+      const failed = jobs.filter((j) => j.status === "failed").length;
       const row2 = { left: [], right: [] };
       row2.left.push({
         t: ` ${running > 0 ? `${running} 个任务正在后台运行` : "没有任务正在后台运行"} `,