dsh-neotui 0.0.10 → 0.0.12

package/README.md

@@ -59,7 +59,8 @@ tui/app/            dsh-neotui-app bundle：cordis.patch.yml + tui-startup/tui-r
 | 设置次级页 | 在「设置」页内 **Shift+Tab** 翻次级页（常规 / 插件） |
 | 主页切换 | **Shift+Tab** 在对话 ↔ 轨迹之间切换；顶部标签页（对话/轨迹/工作区/设置/技能/子代理）可点击直达 |
 | 输入模式 | `i` 或点击进入，**Esc 退出**（nvim normal/insert）；未聚焦时字母=快捷键，多字粘贴直接输入 |
-| 思考/工具折叠 | `t` 思考块展开折叠，`b` 工具块（bash 等）展开折叠；右键消息 →「展开 / 折叠」折叠单个输出块 |
+| 思考/工具折叠 | `t` 思考块展开折叠，`b` 工具块（bash 等）展开折叠；右键消息 →「展开 / 折叠」折叠单个输出块；正文文本块同样支持点击/右键折叠（▸/▾ + 「…共 N 字」尾注）；展开的思考块渲染**全文**，不再截断 |
+| 右键菜单 | 菜单打开后**再次右键**任意位置：立即关闭当前菜单并在新位置打开新菜单（OS 式）；左键点击菜单外仍是关闭 |
 | 对话 ↔ 轨迹转跳 | 对话里右键消息 →「转跳轨迹」定位到对应 step（自动展开+高亮）；轨迹里右键 step →「转跳对话」回到该消息 |
 | 步骤快速转跳 | **Ctrl+E**：fzf 式过滤 step 列表（编号/耗时/工具/消息预览），回车定位到该 step 的轨迹窗口 |
 | 轨迹窗口导航 | 转跳后只显示 `step±20` 窗口；**PgUp/PgDn** 上下各加载 10 步，**Home/End** 跳到最早/最新 20 步，`r` 回最新 |
@@ -100,9 +101,9 @@ tui/app/            dsh-neotui-app bundle：cordis.patch.yml + tui-startup/tui-r
 - 会话内搜索：**Ctrl+F** 模糊搜本会话消息/工具名，回车跳转
 - 轨迹详情：点击回合行弹窗（工具调用 + 耗时），`/` 过滤回合；**增量加载**（每页 20 回合逐页渲染，最近回合秒开，历史步骤按需翻页，不再阻塞等待全量历史）
 - 轨迹 详细/简略：每个 step 左键单击（▸/▾）或右键菜单第一项 →「展开（详细）/ 折叠（简略）」内联展开事件列表（仿对话页）；跳转定位的 step 自动展开 + 高亮闪烁
-- 轨迹窗口模型：转跳只显示 `step±20`；PgUp/PgDn 按 10 步扩展窗口（按需向后翻页加载），Home 加载到 step 1–20，End 回到最新 20 步，窗口指示行显示「窗口 step A–B · 共 N 步」
+- 轨迹窗口模型：转跳只显示目标 step **±20 个相邻步骤**；PgUp/PgDn 按 10 步扩展窗口（按需向后翻页加载），Home 加载到最早 20 步，End 回到最新 20 步，窗口指示行显示「窗口 #N–#M(已加载 X) · step a–b」；窗口边界基于事件序号(seq)而非 step 编号——会话压缩后 step 编号会重新计数,seq 始终唯一
 - 工作区文件搜索：工作区面板内直接打字（`/`）模糊搜文件名，回车预览
-- 消息反馈：助手消息右键 👍/👎（`messageFeedback/put` Typert RPC），已评状态回显、可删除
+- 消息反馈：助手消息右键 👍/👎（`messageFeedback/put` Typert RPC，`{request:…}` 载荷），已评状态回显、可删除
 - 图片画廊：多图时 ←/→ 切换，标题显示 (N/M) + 尺寸
 - 会话内搜索高亮：Ctrl+F 跳转后匹配词高亮（黄底），Esc 清除
 - 轨迹逐事件详情：点击回合 → 事件列表 → 选事件看全文

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-neotui",
-  "version": "0.0.10",
+  "version": "0.0.12",
   "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/panels.js

@@ -556,12 +556,12 @@ export class TrajectoryPanel extends Widget {
     this.allEvents = [];
     this.sessionId = null;
     this.expandedSteps = new Set(); // step identity keys rendered 详细 (expanded)
-    this.flashStep = null;          // step NUMBER just jumped to (brief highlight)
+    this.flashKey = null;           // step key just jumped to (brief highlight)
     this.flashUntil = 0;
     this.loadPromise = null;        // dedupes concurrent load(currentSession)
     this.loadTarget = null;
-    this.winLo = null;              // visible step-number window [winLo, winHi]; null = follow the tail
-    this.winHi = null;
+    this.winSeqLo = null;           // visible window = first-event SEQ range; null = follow the tail
+    this.winSeqHi = null;
     // LEFT click toggles a step's 详细/简略 expansion (the ▸/▾ triangle).
     this.view = new ScrollView({ x: this.x, y: this.y, w: this.w, h: this.h, showScrollbar: true, onClick: (y) => this.#clickLine(y) });
     this.stepLines = [];
@@ -665,98 +665,127 @@ export class TrajectoryPanel extends Widget {
     return -1;
   }
 
-  /** Load older pages until step `loStep` is covered (or the session's first
-   *  step is reached). Used by jumps and Home — many pages when going far. */
-  async ensureLoaded(loStep, maxPages = 80) {
+  /** Load older pages until at least `minCount` steps are loaded (or the
+   *  session's first step is reached). Used by jumps and Home. */
+  async ensureCount(minCount, maxPages = 80) {
     for (let i = 0; i < maxPages; i++) {
-      if (!this.hasMore) break;
-      if (this.steps.length && this.steps[0].step <= loStep) break;
-      this.app.setStatus(`加载更早轨迹…（当前最早 step ${this.steps[0]?.step ?? "?"}）`);
+      if (!this.hasMore || this.steps.length >= minCount) break;
+      this.app.setStatus(`加载更早轨迹…（已加载 ${this.steps.length} 步）`);
       await this.loadOlder();
     }
     this.app.setStatus("");
   }
 
-  /** Set the visible step-number window and re-render. */
-  setWindow(lo, hi) {
-    this.winLo = lo;
-    this.winHi = hi;
+  /** The visible window is a SEQ RANGE (first-event seqs are globally unique
+   *  and monotonic; the server's step numbers restart after compactions and
+   *  cannot be used as boundaries). null = follow the tail (newest 20). */
+  setWindow(loSeq, hiSeq) {
+    this.winSeqLo = loSeq;
+    this.winSeqHi = hiSeq;
     this.buildLines();
   }
 
-  /** Step number at the top of the viewport (for anchoring after growth). */
-  #topVisibleStep() {
+  /** Tail-follow window: the newest 20 loaded steps. */
+  #tailWindow() {
+    const n = this.steps.length;
+    if (n === 0) return;
+    const lo = Math.max(0, n - 20);
+    this.winSeqLo = this.stepKey(this.steps[lo]);
+    this.winSeqHi = this.stepKey(this.steps[n - 1]);
+  }
+
+  /** Seq of the step at the top of the viewport (for anchoring after growth). */
+  #topVisibleSeq() {
     const si = this.stepLines[this.view.scrollY];
-    return si !== undefined ? this.steps[si]?.step : null;
+    return si !== undefined ? this.stepKey(this.steps[si]) : null;
   }
 
-  /** Scroll so the given step number sits at the top of the viewport. */
-  #anchorScroll(stepNum) {
-    const li = this.stepLines.findIndex((si) => this.steps[si]?.step === stepNum);
-    if (li >= 0) this.view.scrollY = Math.max(0, li);
+  /** Scroll so the given step seq sits at the top of the viewport. */
+  #anchorScroll(seq) {
+    const li = this.stepLines.findIndex((si) => this.stepKey(this.steps[si]) === seq);
+    if (li >= 0) this.view.scrollY = Math.max(0, Math.min(li, this.view.maxScroll()));
   }
 
   /** Scroll to a step: open a ±20 window around it (loading older pages on
    *  demand), auto-expand and highlight the step. */
   async jumpToStep(si) {
     if (si < 0 || si >= this.steps.length) return;
-    const S = this.steps[si].step;
-    this.expandedSteps.add(this.stepKey(this.steps[si]));
-    this.flashStep = S;
+    const key = this.stepKey(this.steps[si]);
+    this.expandedSteps.add(key);
+    this.flashKey = key;
     this.flashUntil = Date.now() + 3000;
-    await this.ensureLoaded(Math.max(1, S - 20));
-    // ensureLoaded re-segments → re-find the step index by its number
-    const si2 = this.steps.findIndex((s) => s.step === S);
-    this.setWindow(Math.max(1, S - 20), Math.min(this.totalSteps(), S + 20));
-    const li = si2 >= 0 ? this.stepLines.indexOf(si2) : -1;
-    this.view.scrollY = li >= 0 ? Math.max(0, li - 2) : 0;
+    // load older pages until at least 20 steps sit above the target
+    for (let i = 0; i < 80 && this.hasMore; i++) {
+      if (this.steps.findIndex((s) => this.stepKey(s) === key) >= 20) break;
+      await this.loadOlder();
+    }
+    const idx = this.steps.findIndex((s) => this.stepKey(s) === key);
+    if (idx < 0) return;
+    const lo = Math.max(0, idx - 20), hi = Math.min(this.steps.length - 1, idx + 20);
+    this.setWindow(this.stepKey(this.steps[lo]), this.stepKey(this.steps[hi]));
+    const li = this.stepLines.indexOf(idx);
+    this.view.scrollY = li >= 0 ? Math.max(0, Math.min(li - 2, this.view.maxScroll())) : 0;
     this.app.redraw();
   }
 
   /** PgUp: extend the window 10 steps upward (loading older if needed),
    *  keeping the view anchored on the step that was at the top. */
   async extendUp() {
-    const N = this.totalSteps();
-    if (this.winLo == null) { this.winLo = Math.max(1, N - 19); this.winHi = N; }
-    if (this.winLo <= 1 && this.steps.length && this.steps[0].step <= 1) {
-      this.app.toast("已到最早步骤");
-      return;
+    if (this.winSeqLo == null) this.#tailWindow();
+    if (this.steps.length === 0) return;
+    let topIdx = this.steps.findIndex((s) => this.stepKey(s) === this.winSeqLo);
+    if (topIdx < 0) topIdx = 0;
+    if (topIdx === 0 && !this.hasMore) { this.app.toast("已到最早步骤"); return; }
+    // ensure at least 10 steps above the window top are loaded
+    for (let i = 0; i < 80 && this.hasMore && topIdx < 10; i++) {
+      await this.loadOlder();
+      topIdx = this.steps.findIndex((s) => this.stepKey(s) === this.winSeqLo);
     }
-    const anchor = this.#topVisibleStep() ?? this.winLo;
-    this.winLo = Math.max(1, this.winLo - 10);
-    if (!this.steps.length || this.steps[0].step > this.winLo) await this.ensureLoaded(this.winLo);
+    const anchorSeq = this.#topVisibleSeq();
+    this.winSeqLo = this.stepKey(this.steps[Math.max(0, topIdx - 10)]);
     this.buildLines();
-    this.#anchorScroll(anchor);
+    if (anchorSeq != null) this.#anchorScroll(anchorSeq);
     this.app.redraw();
   }
 
   /** PgDn: extend the window 10 steps downward (the newer steps are already
    *  loaded — the tail is always kept). */
   extendDown() {
-    const N = this.totalSteps();
-    if (this.winLo == null) { this.winLo = Math.max(1, N - 19); this.winHi = N; }
-    if (this.winHi >= N) { this.app.toast("已到最新步骤"); return; }
-    this.winHi = Math.min(N, this.winHi + 10);
+    if (this.winSeqLo == null) this.#tailWindow();
+    if (this.steps.length === 0) return;
+    let bottomIdx = this.steps.length - 1;
+    for (let i = this.steps.length - 1; i >= 0; i--) {
+      if (this.stepKey(this.steps[i]) <= this.winSeqHi) { bottomIdx = i; break; }
+    }
+    const target = Math.min(this.steps.length - 1, bottomIdx + 10);
+    if (target === bottomIdx) { this.app.toast("已到最新步骤"); return; }
+    this.winSeqHi = this.stepKey(this.steps[target]);
     this.buildLines();
     this.app.redraw();
   }
 
   /** Home: jump to the very first steps (loading all the way back). */
   async gotoHome() {
-    await this.ensureLoaded(1);
-    this.setWindow(1, 20);
-    const li = this.stepLines.findIndex((si) => this.steps[si]?.step === 1);
-    this.view.scrollY = li >= 0 ? li : 0;
-    this.app.toast("已跳到最早步骤（step 1–20）");
+    for (let i = 0; i < 80 && this.hasMore; i++) {
+      this.app.setStatus(`加载全部步骤…（已加载 ${this.steps.length} 步）`);
+      await this.loadOlder();
+    }
+    this.app.setStatus("");
+    if (this.steps.length === 0) return;
+    const hi = Math.min(19, this.steps.length - 1);
+    this.setWindow(this.stepKey(this.steps[0]), this.stepKey(this.steps[hi]));
+    this.view.scrollY = 0;
+    this.app.toast("已跳到最早步骤");
     this.app.redraw();
   }
 
   /** End: jump to the newest steps. */
   gotoEnd() {
-    const N = this.totalSteps();
-    this.setWindow(Math.max(1, N - 19), N);
+    if (this.steps.length === 0) return;
+    const lo = Math.max(0, this.steps.length - 20);
+    this.setWindow(this.stepKey(this.steps[lo]), this.stepKey(this.steps[this.steps.length - 1]));
     this.view.scrollY = this.view.maxScroll();
-    this.app.toast(`已跳到最新步骤（step ${Math.max(1, N - 19)}–${N}）`);
+    this.app.toast("已跳到最新步骤");
     this.app.redraw();
   }
 
@@ -773,7 +802,7 @@ export class TrajectoryPanel extends Widget {
     }
     if (si < 0 && messageId) {
       // still not found — the message is far back; scan everything (bounded)
-      await this.ensureLoaded(1, 60);
+      await this.ensureCount(Infinity, 60);
       si = this.indexOfMessage(messageId);
     }
     if (si >= 0) {
@@ -871,8 +900,17 @@ export class TrajectoryPanel extends Widget {
   buildLines() {
     const w = Math.max(40, this.w - 2);
     const N = this.totalSteps();
-    const lo = this.winLo ?? Math.max(1, N - 19);
-    const hi = this.winHi ?? N;
+    // Window boundaries are SEQ-based (step numbers restart after compaction).
+    let loSeq = this.winSeqLo, hiSeq = this.winSeqHi;
+    if (loSeq == null && this.steps.length) {
+      const lo = Math.max(0, this.steps.length - 20);
+      loSeq = this.stepKey(this.steps[lo]);
+      hiSeq = this.stepKey(this.steps[this.steps.length - 1]);
+    }
+    const winIdxLo = this.steps.findIndex((s) => this.stepKey(s) === loSeq);
+    const winIdxHi = this.steps.findIndex((s) => this.stepKey(s) === hiSeq);
+    const loStepNum = this.steps[winIdxLo]?.step ?? "?";
+    const hiStepNum = this.steps[winIdxHi]?.step ?? "?";
     const lines = [];
     lines.push([{ t: "轨迹 — 步骤时间轴（左键展开/折叠 · PgUp/PgDn 上下加载 · Home/End 首尾 · Ctrl+E 转跳 · r 刷新）", fg: K.ACCENT, bold: true }]);
     if (this.hasMore) lines.push([{ t: "▲ 更早步骤（点击 / PgUp 向上加载 10 步）", fg: K.FAINT }]);
@@ -881,7 +919,7 @@ export class TrajectoryPanel extends Widget {
     if (st) {
       lines.push([{ t: `回合 ${st.turns} · 步骤 ${st.steps} · LLM ${fmtMs(st.llmMs)} · 工具 ${fmtMs(st.toolMs)}`, fg: K.DIM }]);
     }
-    lines.push([{ t: `窗口 step ${lo}–${hi} · 共 ${N} 步${this.winLo == null ? "（跟随最新）" : ""}：`, fg: K.DIM, underline: true }]);
+    lines.push([{ t: `窗口 #${winIdxLo + 1}–#${winIdxHi + 1}（已加载 ${this.steps.length}${this.hasMore ? "+" : ""}）· step ${loStepNum}–${hiStepNum}${this.winSeqLo == null ? "（跟随最新）" : ""}：`, fg: K.DIM, underline: true }]);
     this.stepLines = [];
     const list = this.query
       ? this.steps.filter((t) => t.events.some((e) => {
@@ -889,7 +927,10 @@ export class TrajectoryPanel extends Widget {
         const hay = `${e.type} ${d.name ?? ""} ${typeof d.content === "string" ? d.content : ""}`.toLowerCase();
         return hay.includes(this.query.toLowerCase());
       }))
-      : this.steps.filter((s) => s.step >= lo && s.step <= hi);
+      : this.steps.filter((s) => {
+        const k = this.stepKey(s);
+        return k >= loSeq && k <= hiSeq;
+      });
     for (const step of list.reverse()) {
       const si = this.steps.indexOf(step);
       const tools = [...new Set(step.events.filter((e) => e.type === "tool/call").map((e) => e.data?.name))];
@@ -900,7 +941,7 @@ export class TrajectoryPanel extends Widget {
       const bg = tools.length ? (hasResult ? T.TOOLOK : T.TOOLBG) : hasReasoning ? T.THINKBG : T.CARD;
       const summary = tools.slice(0, 3).join(",") || (hasReasoning ? "模型推理" : "纯文本");
       const open = this.expandedSteps.has(this.stepKey(step));       // 详细
-      const flash = this.flashStep === step.step && Date.now() < this.flashUntil;
+      const flash = this.flashKey === this.stepKey(step) && Date.now() < this.flashUntil;
       const rowBg = flash ? T.ACCENT : bg;
       const label = `${open ? "▾" : "▸"} step ${String(step.step).padStart(3)}  ${pad(dur, 8)}  ${summary}  ${open ? "[折叠]" : "[展开]"}`;
       const segs = [{ t: label, fg: flash ? T.SELFG : K.TXT, bg: rowBg, bold: true }];
@@ -971,7 +1012,7 @@ export class TrajectoryPanel extends Widget {
     }
     if (ev.name === "backspace") { this.query = this.query.slice(0, -1); this.buildLines(); this.app.redraw(); return true; }
     if (ev.name === "char" && ev.key === "r" && !ev.ctrl) {
-      this.winLo = this.winHi = null;
+      this.winSeqLo = this.winSeqHi = null;
       this.steps = [];
       this.load(this.sessionId);
       return true;

package/src/views.js

@@ -848,9 +848,28 @@ export class ChatView extends Widget {
         if (ref) { this.app.openImage(ref, { all: node.images, index: info.imgIdx }); return true; }
       }
       const node = this.nodes[info.nodeIdx];
+      // Expand/collapse changes the line count ABOVE the viewport, which would
+      // leave scrollY pointing at unrelated content (the "chaotic" jump).
+      // Anchor: after the rebuild, scroll so the clicked block's first line
+      // sits at the top of the viewport. At the bottom the tail-follow snap
+      // in setLines is the right behavior instead.
+      const wasAtBottom = this.view.scrollY + this.view.h >= this.view.lines.length - 1;
+      const reanchor = (match) => {
+        if (wasAtBottom) return;
+        let fallback = -1;
+        for (let i = 0; i < this.lineMap.length; i++) {
+          if (!match(this.lineMap[i])) continue;
+          if ((this.lines[i] ?? []).some((g) => g.t.trim() !== "")) {
+            this.view.scrollY = Math.max(0, Math.min(i, this.view.maxScroll()));
+            return;
+          }
+          if (fallback < 0) fallback = i;
+        }
+        if (fallback >= 0) this.view.scrollY = Math.max(0, Math.min(fallback, this.view.maxScroll()));
+      };
       if (node?.kind === "assistant" && info.blockIdx !== null) {
         const b = node.blocks[info.blockIdx];
-        if (b && (b.kind === "tool" || b.kind === "reasoning" || b.kind === "other")) {
+        if (b && (b.kind === "tool" || b.kind === "reasoning" || b.kind === "other" || b.kind === "text")) {
           const key = `${info.nodeIdx}:${info.blockIdx}`;
           if (b.kind === "reasoning") {
             // clean two-state override: expand ⇄ collapse (never a no-op click)
@@ -862,6 +881,7 @@ export class ChatView extends Widget {
             else this.collapsedBlocks.add(key);
           }
           this.#rebuild();
+          reanchor((m) => m?.nodeIdx === info.nodeIdx && m?.blockIdx === info.blockIdx);
           return true;
         }
       }
@@ -869,6 +889,7 @@ export class ChatView extends Widget {
         if (this.expanded.has(info.nodeIdx)) this.expanded.delete(info.nodeIdx);
         else this.expanded.add(info.nodeIdx);
         this.#rebuild();
+        reanchor((m) => m?.nodeIdx === info.nodeIdx);
         return true;
       }
     }
@@ -891,7 +912,6 @@ export class ChatView extends Widget {
     if (this.hasMore) { lines.push([{ t: "▲ 更早的记录", fg: K.FAINT }]); mark(-1); }
     if (skipCount > 0) { lines.push([{ t: `… 更早 ${skipCount} 条记录（PgUp 加载）`, fg: K.FAINT }]); mark(-1); }
 
-    const MAX_TEXT = 4000;
     for (let ni = 0; ni < nodes.length; ni++) {
       const node = nodes[ni];
       const realIdx = ni + skipCount;
@@ -999,7 +1019,8 @@ export class ChatView extends Widget {
               lines.push([{ t: "💭 思考" + (b.streaming ? "…" : "") + thinkMeta + (open ? " [t 折叠]" : " [t 展开]"), fg: K.FAINT }]);
               mark(realIdx, bi);
               if (open) {
-                for (const ln of renderMd(truncateText(b.text, MAX_TEXT), w - 4)) { lines.push([{ t: "  " }, ...ln]); mark(realIdx, bi); }
+                // Expanded = the WHOLE reasoning (no hidden remainder).
+                for (const ln of renderMd(b.text ?? "", w - 4)) { lines.push([{ t: "  " }, ...ln]); mark(realIdx, bi); }
               } else {
                 // collapsed: three-line preview
                 for (const ln of renderMd(truncateText(b.text, 400), w - 4).slice(0, 3)) {
@@ -1059,18 +1080,27 @@ export class ChatView extends Widget {
               sep();
             } else {
               beginCard("CARD");
+              // 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.
+              const key = `${realIdx}:${bi}`;
+              const open = !this.collapsedBlocks.has(key);
               const text = b.text ?? "";
-              const md = renderMd(text, w - 4 - strWidth(stepTag));
+              const glyph = open ? "▾" : "▸";
+              const mdW = Math.max(10, w - 6 - strWidth(stepTag));
+              const md = open
+                ? renderMd(text, mdW)
+                : renderMd(truncateText(text, 600), mdW).slice(0, 3);
               if (md.length > 0) {
-                lines.push([{ t: "  " }, { t: stepTag, fg: K.FAINT }, ...md[0]]);
-                mark(realIdx);
-                for (const ln of md.slice(1)) { lines.push([{ t: "  " }, ...ln]); mark(realIdx); }
-                // Label the block's LAST line with its blockIdx IN PLACE. A
-                // pushed mark without a line would desync lineMap from lines
-                // and shift every later click mapping (the off-by-one bug).
-                lineMap[lineMap.length - 1] = { nodeIdx: realIdx, blockIdx: bi };
+                lines.push([{ t: "  " }, { t: glyph + " ", fg: K.FAINT }, { t: stepTag, fg: K.FAINT }, ...md[0]]);
+                mark(realIdx, bi);
+                for (const ln of md.slice(1)) { lines.push([{ t: "  " }, ...ln]); mark(realIdx, bi); }
               } else {
-                lines.push([{ t: "  " + stepTag, fg: K.FAINT }]);
+                lines.push([{ t: "  " + glyph + " " + stepTag, fg: K.FAINT }]);
+                mark(realIdx, bi);
+              }
+              if (!open && text.length > 0) {
+                lines.push([{ t: `  …共 ${text.length} 字（点击展开）`, fg: K.FAINT }]);
                 mark(realIdx, bi);
               }
               sep();
@@ -1298,13 +1328,13 @@ export class ChatView extends Widget {
       case "pgup": if (this.view.scrollY === 0) { this.loadOlder(); return true; } return this.view.scroll(-this.view.h);
       case "pgdn": return this.view.scroll(this.view.h);
     }
-    if (ev.name === "char" && ev.key === "g" && !ev.ctrl) {
+    if (ev.name === "char" && ev.key === "g" && !ev.ctrl && !ev.shift) {
       if (this.gKey) { this.gKey = false; this.view.scrollY = 0; return true; }
       this.gKey = true;
       this.app.toast("再按 g 回顶");
       return true;
     }
-    if (ev.name === "char" && ev.key === "G") { this.view.scrollY = this.view.maxScroll(); return true; }
+    if (ev.name === "char" && ev.key === "g" && ev.shift) { this.view.scrollY = this.view.maxScroll(); return true; }
     if (ev.name === "escape" && this.app.searchQuery) { this.app.searchQuery = null; this.queueRebuild(); return true; }
     if (ev.name === "escape" && this.selStart !== null) { this.selStart = this.selEnd = null; this.app.redraw(); return true; }
     if (ev.name === "char" && ev.key === "i" && !ev.ctrl) { this.app.focus(this.input); return true; }
@@ -1891,18 +1921,26 @@ export class App {
   async loadFeedback() {
     if (!this.currentSession) return;
     try {
-      const res = await this.api.rpcCall("messageFeedback/list", { sessionId: this.currentSession });
+      const res = await this.api.rpcCall("messageFeedback/list", { request: { sessionId: this.currentSession } });
       this.feedbackMap = new Map();
-      for (const item of res.items ?? []) this.feedbackMap.set(item.messageId, item);
+      for (const item of res?.value?.items ?? res?.items ?? []) this.feedbackMap.set(item.messageId, item);
     } catch { this.feedbackMap = new Map(); }
   }
 
   async feedback(messageId, rating) {
     const existing = this.feedbackMap?.get(messageId);
     try {
-      const item = await this.api.rpcCall("messageFeedback/put", {
-        sessionId: this.currentSession, messageId, rating, ifVersion: existing?.version ?? null,
+      const res = await this.api.rpcCall("messageFeedback/put", {
+        request: {
+          sessionId: this.currentSession, messageId, rating,
+          ifVersion: existing?.version ?? null,
+        },
       });
+      if (res?.ok === false) {
+        this.toast(`反馈失败: ${res.error?.code ?? res.error?.message ?? "unknown"}`);
+        return;
+      }
+      const item = res?.value ?? res;
       this.feedbackMap = this.feedbackMap ?? new Map();
       this.feedbackMap.set(messageId, item);
       this.toast(rating === "positive" ? "已记录 👍" : "已记录 👎");
@@ -1913,7 +1951,13 @@ export class App {
     const existing = this.feedbackMap?.get(messageId);
     if (!existing) return;
     try {
-      await this.api.rpcCall("messageFeedback/delete", { sessionId: this.currentSession, messageId, ifVersion: existing.version });
+      const res = await this.api.rpcCall("messageFeedback/delete", {
+        request: { sessionId: this.currentSession, messageId, ifVersion: existing.version },
+      });
+      if (res?.ok === false) {
+        this.toast(`删除反馈失败: ${res.error?.code ?? res.error?.message ?? "unknown"}`);
+        return;
+      }
       this.feedbackMap.delete(messageId);
       this.toast("已删除反馈");
     } catch (e) { this.toast(`删除反馈失败: ${e.message}`); }
@@ -2290,16 +2334,29 @@ export class App {
     }
     if (this.menu) {
       const before = this.menu;
-      if (ev.type === "key") this.menu.onKey(ev);
-      else if (ev.type === "mouse") {
-        if (ev.kind === "press" && ev.button === 0 && !this.menu.inside(ev.x, ev.y)) { this.menu = null; this.swallowRelease = true; }
-        else {
+      if (ev.type === "key") { this.menu.onKey(ev); this.redraw(); return; }
+      if (ev.type === "mouse") {
+        if (ev.kind === "press" && ev.button === 0 && !this.menu.inside(ev.x, ev.y)) {
+          this.menu = null; this.swallowRelease = true; this.redraw(); return;
+        }
+        if (ev.kind === "press" && ev.button === 2) {
+          if (this.menu.inside(ev.x, ev.y)) { this.redraw(); return; } // right-click on the menu itself: keep it open
+          // OS-style: a right-click anywhere else closes the current menu and
+          // the SAME press falls through to open a fresh menu at the new spot.
+          this.menu = null;
+          this.swallowRelease = true;
+          this.redraw();
+          // fall through to the normal mouse routing below
+        } else {
           this.menu.onMouse(ev);
           if (ev.kind === "press" && this.menu !== before) this.swallowRelease = true;
+          this.redraw();
+          return;
         }
+      } else {
+        this.redraw();
+        return;
       }
-      this.redraw();
-      return;
     }
     if (this.overlay) {
       const before = this.overlay;
@@ -2443,7 +2500,14 @@ export class App {
         return;
       }
       if (ev.text.length === 1) {
-        const asKey = { type: "key", name: "char", key: ev.text, text: ev.text, ctrl: false, alt: false, shift: false };
+        // Legacy terminals deliver Shift+letter as the UPPERCASE char with no
+        // modifier info — recover the shift flag from the case so Shift+T
+        // (todo fold) and G (scroll bottom) work everywhere.
+        const asKey = {
+          type: "key", name: "char",
+          key: ev.text.toLowerCase(), text: ev.text,
+          ctrl: false, alt: false, shift: ev.text !== ev.text.toLowerCase(),
+        };
         if (this.chat.onKey(asKey)) { this.redraw(); return; }
         if (this.focused && this.focused !== this.chat.input && this.focused.onKey(asKey)) { this.redraw(); return; }
         this.toast("按 i 进入输入");