dsh-client-ui-quote 0.11.1 → 0.11.3

package/README.md

@@ -55,7 +55,7 @@ DSH 会把包记进 profile 的依赖并加入组合包列表，装完在侧边
 
 ### 方式三：npm 包（已上架）
 
-已发布到 npm：`dsh-client-ui-quote@0.11.1`（0.10.0 起上架）。发布由 GitHub Actions 完成（`.github/workflows/publish.yml`）：推 `v*` 标签或手动 Run `publish` workflow，`prepublishOnly` 的 290 项断言会先在 runner 上跑完，任何一项失败都不会发出去。`npm pack --dry-run` 可核对内容（0.10.0 = 14 个文件 / 97.6 kB）。
+已发布到 npm：`dsh-client-ui-quote`（0.10.0 起上架；每个 `v*` 标签由下面的 workflow 自动发布，本仓库当前 0.11.3）。发布由 GitHub Actions 完成（`.github/workflows/publish.yml`）：推 `v*` 标签或手动 Run `publish` workflow，`prepublishOnly` 的 306 项断言会先在 runner 上跑完，任何一项失败都不会发出去。`npm pack --dry-run` 可核对内容（0.10.0 = 14 个文件 / 97.6 kB）。
 
 装：
 
@@ -227,7 +227,7 @@ transcript 侧的 `.dshq-cap` 胶囊：一颗胶囊同时装引用与评论，`
 1. **语义而不是机制**：引用与评论是同一颗节点（对齐 Kimi 的 `quote` 节点 `{text, comment}`），一颗芯片 → 一个块 → 一颗胶囊 → 一张卡片 → 一次复制；机制（reference chip、DOM 增强）谁都能抄，这条链子不好抄。
 2. **引用是带坐标的活对象**：`跳回原文` 在自己的会话面板里找行、按宿主的方式对齐到 24px，行被虚拟列表藏起来或没挂载都能找回来；坐标不随引用文本发给模型（不污染 prompt）。
 3. **引用有下落**：清单页（搜索 / 跳转 / 导出 Markdown）+ 侧边线程回流（结论写回主会话草稿 / 导出线程），这两件事在 4471 条里没有对应物。
-4. **工程化**：290 项离线断言（含真跑 `sessions.retain` / `setDraft` / 下载链路）、每个宿主契约都在 README 里写明出处、每个版本都有可回读的 release。同类里多数是一两百行、没有测试的插件。
+4. **工程化**：306 项离线断言（含真跑 `sessions.retain` / `setDraft` / 下载链路）、每个宿主契约都在 README 里写明出处、每个版本都有可回读的 release。同类里多数是一两百行、没有测试的插件。
 
 反过来也要说清楚：**本插件不做的**——不做翻译/解释类选区动作（81 条同类在做）、不做多选合并引用（7 条有）、不做选区折叠/行内 diff 这类渲染花样。它只把「引用」这一件事做穿。
 
@@ -260,7 +260,7 @@ locale/zh.json        插件页卡片文案（中文）
 locale/en.json        插件页卡片文案（英文）
 test/check.mjs        离线自检（清单 + 组件契约）
 test/dom.mjs          给离线自检用的最小 DOM / MutationObserver 垫片
-test/capsule.mjs      离线自检（发送后胶囊 + 悬停卡片 + 跳回原文 + 键位 + 改这条引用 + 代码引用 + 文件路径 + 逐条回应的回复徽章，194 项断言）
+test/capsule.mjs      离线自检（发送后胶囊 + 悬停卡片 + 跳回原文 + 键位 + 改这条引用 + 代码引用 + 文件路径 + 逐条回应的回复徽章 + 浮层不随座位失效，210 项断言）
 test/panel.mjs        离线自检（引用清单：解析 / 搜索 / Markdown / 读取 / 导出 / 页面注册 + 侧边线程回流，96 项断言）
 test/capsule.html     真浏览器里跑同一套胶囊断言（可选）
 ```
@@ -308,7 +308,10 @@ React 重渲染不会出现第二颗、引用变了就地重建、消息被删
 卡片给出 `跳回原文` 与 `改这条引用` 两个动作、点后者把栏切到评论模式并**预填旧评论与旧正文**、
 按确认后 `insertReference` 恰好一次（新芯片的 `ref` 带改后的评论）、旧引用被记为「已修订」并落进 `localStorage`、
 旧胶囊挂上 `已修订` 徽标、它的卡片显示改后的评论（连正文改过的连改后原文一起显示）、空修订不算修订、
-修订不改写引用原文本身、清空引用正文时确认按钮禁用、改后的正文随新芯片发出（共 194 项，含文件路径与逐条回应的线上形态、回复徽章的建立/重建/卸载与悬停卡片复用）。
+修订不改写引用原文本身、清空引用正文时确认按钮禁用、改后的正文随新芯片发出；
+**浮层不跟着座位生死**（`conversation.input.overlay` 没挂载也能选中出栏、挂上两个会话座位既不叠加也不丢监听、
+座位不在时用输入框自己的 `actions` 落芯片、无处可插时给 toast 而不是静默返回、effect 卸载/重装后监听恰好一份）；
+（共 210 项，含文件路径与逐条回应的线上形态、回复徽章的建立/重建/卸载与悬停卡片复用，以及评论框两个输入区的自适应高度——引用正文那格曾经在被挂进栏之前就量高，读到 0 后被写成 `height:0`，首行因此被裁掉）。
 `test/capsule.html` 是同一套断言的真浏览器版，用 `file://` 打开即可（页面会把结果写进标题与 `#probe-out`）。
 
 `test/panel.mjs` 单独跑引用清单页与侧边线程回流（96 项）：只认**持久**的 `user/message`（`transient`、助手消息、`data: null` 都跳过）、
@@ -327,7 +330,7 @@ Markdown 的标题/编号/块引用/评论行/时间戳与空列表兜底、超
 `bringThreadBack` 把结论写成**草稿**（原有草稿保留、空行隔开）、外壳没有 `setDraft` 时如实失败、没有线程时不乱写；
 `exportThread` 的文件名与整条线程内容、以及 MIME 是 Markdown。
 
-合计 **290 项**（check.mjs 的契约断言 + capsule 194 + panel 96），`npm test` 全绿。
+合计 **306 项**（check.mjs 的契约断言 + capsule 210 + panel 96），`npm test` 全绿。
 
 ## 已知限制
 

package/lib/client.js

@@ -591,7 +591,15 @@ code's own spacing — collapsed keeps one clipped line, expanded scrolls. */
 			let sessionId = null;
 			if (chat !== null) {
 				sessionId = chat.getAttribute("data-conversation-session");
-				if (typeof sessionId !== "string" || !sessions.has(sessionId)) return null;
+				if (typeof sessionId !== "string" || sessionId === "") {
+					// A pane the host has not labelled yet: fall back to whichever
+					// conversation view is mounted, the way the preview path does.
+					// The bar used to demand a mounted slot entry here, which is why
+					// a selection could go dead after a session switch.
+					const active = document.querySelector(CONTENT_SELECTOR + "[data-conversation-session]");
+					sessionId = active === null ? null : active.getAttribute("data-conversation-session");
+				}
+				if (typeof sessionId !== "string" || sessionId === "") return null;
 			} else {
 				const active = document.querySelector(CONTENT_SELECTOR + "[data-conversation-session]");
 				sessionId = active === null ? null : active.getAttribute("data-conversation-session");
@@ -859,18 +867,19 @@ code's own spacing — collapsed keeps one clipped line, expanded scrolls. */
 				sourceDisposer = typeof disposer === "function" ? disposer : () => {};
 				return true;
 			} catch (error) {
-				console.warn("[ui-quote] reference source registration failed:", error);
-				return false;
+				// "#quote is already registered" is the common case (a second copy of
+				// this plugin, or a re-apply): the source object is ours and the codec
+				// is stateless, so the resident registration is good enough. Treating
+				// it as a failure is what silently downgraded quotes to raw Markdown.
+				console.warn("[ui-quote] reference source already registered; reusing it:", error);
+				sourceDisposer = () => {};
+				return true;
 			}
 		}
 
-		/**
-		 * Mint a chip through the conversation input service face: the resident
-		 * shell owns the editor that can place an atomic node. Every step is
-		 * optional — a missing service just falls back to the Markdown form.
-		 */
-		function insertChip(entry, sessionId, text, comment, code, meta) {
-			if (!ensureSource() || pluginCtx === null) return false;
+		/** The host's resident composer shell for a session, or null when absent. */
+		function inputShellFor(sessionId) {
+			if (pluginCtx === null || typeof sessionId !== "string" || sessionId === "") return null;
 			let hub = null;
 			try {
 				const conversation = pluginCtx.get("conversation");
@@ -878,14 +887,38 @@ code's own spacing — collapsed keeps one clipped line, expanded scrolls. */
 			} catch (error) {
 				hub = null;
 			}
-			if (hub === null || hub === undefined || typeof hub.shell !== "function") return false;
-			let shell = null;
+			if (hub === null || hub === undefined || typeof hub.shell !== "function") return null;
 			try {
-				shell = hub.shell(sessionId);
+				const shell = hub.shell(sessionId);
+				return shell === null || shell === undefined ? null : shell;
 			} catch (error) {
-				shell = null;
+				return null;
 			}
-			if (shell === null || shell === undefined || typeof shell.insertReference !== "function") return false;
+		}
+
+		/**
+		 * The face that can put text into a session's draft: the mounted slot
+		 * entry when there is one, else the host shell's own actions. The bar is
+		 * document-level, so an action must never depend on a slot entry still
+		 * being there — the composer outlives the entry across a session switch.
+		 */
+		function insertionFace(sessionId, shell) {
+			const entry = typeof sessionId === "string" ? sessions.get(sessionId) : undefined;
+			if (entry !== undefined && entry !== null) return entry;
+			const actions = shell === null || shell === undefined ? null : shell.actions;
+			if (actions === null || actions === undefined || typeof actions.captureInsertion !== "function") return null;
+			return { actions: () => actions };
+		}
+
+		/**
+		 * Mint a chip through the conversation input service face: the resident
+		 * shell owns the editor that can place an atomic node. Every step is
+		 * optional — a missing service just falls back to the Markdown form.
+		 */
+		function insertChip(entry, sessionId, text, comment, code, meta) {
+			if (!ensureSource() || pluginCtx === null) return false;
+			const shell = inputShellFor(sessionId);
+			if (shell === null || typeof shell.insertReference !== "function") return false;
 			const model = chipRef(text, comment, code, meta);
 			const attachment = {
 				source: SOURCE_NAME,
@@ -1711,22 +1744,28 @@ code's own spacing — collapsed keeps one clipped line, expanded scrolls. */
 		}
 
 		function onCapsuleMutations(records) {
-			for (const record of records) {
-				if (record.type === "characterData") {
-					collectRuns(record.target, capsuleDirty);
-					collectRefRuns(record.target, refDirty);
-					continue;
-				}
-				for (const node of record.addedNodes) {
-					collectRuns(node, capsuleDirty);
-					collectRefRuns(node, refDirty);
-				}
-				for (const node of record.removedNodes) {
-					if (node.nodeType === 1 && capsules.has(node)) capsuleDirty.add(node);
+			try {
+				for (const record of records) {
+					if (record.type === "characterData") {
+						collectRuns(record.target, capsuleDirty);
+						collectRefRuns(record.target, refDirty);
+						continue;
+					}
+					for (const node of record.addedNodes) {
+						collectRuns(node, capsuleDirty);
+						collectRefRuns(node, refDirty);
+					}
+					for (const node of record.removedNodes) {
+						if (node.nodeType === 1 && capsules.has(node)) capsuleDirty.add(node);
+					}
+					const target = elementOf(record.target);
+					if (target !== null && capsules.has(target)) capsuleDirty.add(target);
 				}
-				const target = elementOf(record.target);
-				if (target !== null && capsules.has(target)) capsuleDirty.add(target);
+			} catch (error) {
+				console.warn("[ui-quote] capsule scan failed:", error);
 			}
+			// Always reschedule: a throw inside the scan used to leave the pass
+			// unscheduled and the transcript raw until the next reload.
 			scheduleCapsules();
 		}
 
@@ -1737,19 +1776,34 @@ code's own spacing — collapsed keeps one clipped line, expanded scrolls. */
 
 		function flushCapsules() {
 			capsuleTimer = null;
-			dropDetachedPopover();
-			for (const run of Array.from(capsules.keys())) {
-				if (!run.isConnected) dropCapsule(run);
-			}
 			const runs = Array.from(capsuleDirty);
 			capsuleDirty.clear();
+			// A throw anywhere in this pass used to abort it silently: the rest of
+			// the transcript stayed raw (`> ❝ …` text, no pills, no hover cards)
+			// with no error the reader could see. Each stage stands on its own now.
+			try {
+				dropDetachedPopover();
+				for (const run of Array.from(capsules.keys())) {
+					if (!run.isConnected) dropCapsule(run);
+				}
+			} catch (error) {
+				console.warn("[ui-quote] capsule cleanup failed:", error);
+			}
 			for (const run of runs) {
-				if (run.isConnected) enhance(run);
-				else if (capsules.has(run)) dropCapsule(run);
+				try {
+					if (run.isConnected) enhance(run);
+					else if (capsules.has(run)) dropCapsule(run);
+				} catch (error) {
+					console.warn("[ui-quote] capsule enhance failed:", error);
+				}
+			}
+			try {
+				// Badges come last: the quotes they link to must be in the ledger first.
+				flushRefBadges();
+				wireChips();
+			} catch (error) {
+				console.warn("[ui-quote] capsule tail pass failed:", error);
 			}
-			// Badges come last: the quotes they link to must be in the ledger first.
-			flushRefBadges();
-			wireChips();
 		}
 
 		function ensureCapsuleStyle(doc) {
@@ -2018,6 +2072,14 @@ code's own spacing — collapsed keeps one clipped line, expanded scrolls. */
 		 * ------------------------------------------------------------------ */
 
 		const sessions = new Map();
+		/**
+		 * The bar's listeners are document singletons and live exactly as long as
+		 * the plugin. While they were reference-counted by the overlay slot they
+		 * could be removed for good by an overlapping mount pair, and then the bar
+		 * never appeared again until a page reload — the "quote plugin stopped
+		 * working" report.
+		 */
+		let listenersAttached = false;
 			const state = {
 				bar: null,
 				style: null,
@@ -2040,7 +2102,6 @@ code's own spacing — collapsed keeps one clipped line, expanded scrolls. */
 			/** The quote key being corrected, when the box was opened by 改这条引用. */
 			revising: null,
 			settle: null,
-			listeners: 0,
 			pointerOnTranscript: false,
 			suppress: false
 		};
@@ -2160,6 +2221,29 @@ code's own spacing — collapsed keeps one clipped line, expanded scrolls. */
 		 * its wording, not just its comment. Both halves map onto the wire form's
 		 * two marks, top to bottom.
 		 */
+		/**
+		 * Grow one field to its content. A textarea that is not laid out yet
+		 * reports a scrollHeight of 0 — writing that back as `0px` collapsed the
+		 * quote half of the box and clipped its first line, so an unmeasurable
+		 * field is left on `auto` until the bar is on screen.
+		 */
+		function growField(node) {
+			node.style.height = "auto";
+			const next = node.scrollHeight;
+			if (!Number.isFinite(next) || next <= 0) return;
+			node.style.height = Math.min(next, COMMENT_MAX_HEIGHT) + "px";
+		}
+
+		/** Every field the bar is currently showing, re-measured. */
+		function growInputs() {
+			const bar = state.bar;
+			if (bar === null || bar === undefined || typeof bar.querySelectorAll !== "function") return;
+			for (const field of Array.from(bar.querySelectorAll(".dshq-input"))) {
+				if (String(field.tagName).toLowerCase() !== "textarea") continue;
+				growField(field);
+			}
+		}
+
 		function renderComment(bar) {
 			const doc = bar.ownerDocument;
 			bar.setAttribute("role", "dialog");
@@ -2186,10 +2270,6 @@ code's own spacing — collapsed keeps one clipped line, expanded scrolls. */
 				"data-action": "confirm"
 			}, [el(doc, "span", { text: translate("addToChat") }), el(doc, "span", { class: "dshq-enter", text: "⏎" })]);
 
-			const grow = (node) => {
-				node.style.height = "auto";
-				node.style.height = Math.min(node.scrollHeight, COMMENT_MAX_HEIGHT) + "px";
-			};
 			// The quote half is mandatory now (an empty quote is nothing to send);
 			// the comment keeps Kimi's rule of being required in this box.
 			const syncConfirm = () => {
@@ -2197,12 +2277,12 @@ code's own spacing — collapsed keeps one clipped line, expanded scrolls. */
 			};
 			quoteInput.addEventListener("input", () => {
 				state.quote = quoteInput.value;
-				grow(quoteInput);
+				growField(quoteInput);
 				syncConfirm();
 			});
 			input.addEventListener("input", () => {
 				state.comment = input.value;
-				grow(input);
+				growField(input);
 				syncConfirm();
 			});
 			const onKey = (event) => {
@@ -2234,7 +2314,8 @@ code's own spacing — collapsed keeps one clipped line, expanded scrolls. */
 				el(doc, "span", { text: translate("replyToggle") })
 			]);
 
-			grow(quoteInput);
+			// No grow here: the fields are measured in `position()`, once the bar
+			// takes part in layout. Reading them while detached returned 0.
 			syncConfirm();
 
 			bar.replaceChildren(
@@ -2258,6 +2339,10 @@ code's own spacing — collapsed keeps one clipped line, expanded scrolls. */
 
 			bar.style.visibility = "hidden";
 			bar.dataset.visible = "true";
+			// The bar is in layout now (merely invisible), which is the first
+			// moment its fields can be measured — and it has to happen before the
+			// height below is read, or the box would be placed at the wrong place.
+			growInputs();
 
 			const viewport = window.visualViewport;
 			const width = viewport === undefined || viewport === null ? window.innerWidth : viewport.width;
@@ -2385,10 +2470,19 @@ code's own spacing — collapsed keeps one clipped line, expanded scrolls. */
 			const revising = state.revising;
 			const code = state.code;
 			const meta = { path: state.path, reply: state.reply === true };
-			const entry = sessionId === null ? null : sessions.get(sessionId);
+			// Resolve the insertion face before the bar is torn down: the slot entry
+			// may already be gone (a session switch) while the composer is still up.
+			const shell = inputShellFor(sessionId);
+			const entry = insertionFace(sessionId, shell);
 			const trimmed = typeof comment === "string" ? comment.trim() : "";
 			close();
-			if (typeof text !== "string" || text.trim() === "" || entry === null || entry === undefined) return;
+			if (typeof text !== "string" || text.trim() === "") return;
+			if (entry === null) {
+				// Returning silently here is what made "clicked and nothing happened"
+				// reports: the user gets told instead.
+				toast(translate("failed"));
+				return;
+			}
 
 			// Remember the row this came from before anything can fail: the jump
 			// row is keyed by the quote's own wording, so it survives the send.
@@ -2785,6 +2879,10 @@ code's own spacing — collapsed keeps one clipped line, expanded scrolls. */
 			if (!state.pointerOnTranscript) return;
 			state.pointerOnTranscript = false;
 			if (insideBar(event.target)) return;
+			// Self-healing: `close()` suppresses the settle-timer refresh so the
+			// gesture that dismissed the bar cannot reopen it. A fresh pointer
+			// gesture is always a fresh intention.
+			state.suppress = false;
 			refresh();
 		}
 
@@ -2814,12 +2912,16 @@ code's own spacing — collapsed keeps one clipped line, expanded scrolls. */
 			else close();
 		}
 
-		function attach() {
-			if (state.listeners > 0) {
-				state.listeners += 1;
-				return;
-			}
-			state.listeners = 1;
+		/**
+		 * Install the document listeners. Called once for the plugin's lifetime
+		 * from `apply()`, never per slot: a counter kept them alive only while
+		 * some overlay happened to be mounted, and one bad count removed them
+		 * forever (the bar then stayed dead until a reload).
+		 */
+		function attachListeners() {
+			if (listenersAttached) return;
+			if (typeof document === "undefined" || typeof document.addEventListener !== "function") return;
+			listenersAttached = true;
 			document.addEventListener("selectionchange", onSelectionChange);
 			document.addEventListener("keyup", onKeyUp, true);
 			document.addEventListener("pointerdown", onPointerDown, true);
@@ -2828,13 +2930,14 @@ code's own spacing — collapsed keeps one clipped line, expanded scrolls. */
 			document.addEventListener("mousedown", onMouseDown, true);
 			document.addEventListener("keydown", onKeyDown, true);
 			document.addEventListener("scroll", onScrollOrResize, true);
-			window.addEventListener("resize", onScrollOrResize);
+			if (typeof window !== "undefined" && typeof window.addEventListener === "function") {
+				window.addEventListener("resize", onScrollOrResize);
+			}
 		}
 
-		function detach() {
-			if (state.listeners === 0) return;
-			state.listeners -= 1;
-			if (state.listeners > 0) return;
+		function detachListeners() {
+			if (!listenersAttached) return;
+			listenersAttached = false;
 			document.removeEventListener("selectionchange", onSelectionChange);
 			document.removeEventListener("keyup", onKeyUp, true);
 			document.removeEventListener("pointerdown", onPointerDown, true);
@@ -2843,8 +2946,12 @@ code's own spacing — collapsed keeps one clipped line, expanded scrolls. */
 			document.removeEventListener("mousedown", onMouseDown, true);
 			document.removeEventListener("keydown", onKeyDown, true);
 			document.removeEventListener("scroll", onScrollOrResize, true);
-			window.removeEventListener("resize", onScrollOrResize);
+			if (typeof window !== "undefined" && typeof window.removeEventListener === "function") {
+				window.removeEventListener("resize", onScrollOrResize);
+			}
 			close();
+			state.pointerOnTranscript = false;
+			state.suppress = false;
 			if (state.bar !== null) {
 				state.bar.remove();
 				state.bar = null;
@@ -2882,10 +2989,8 @@ code's own spacing — collapsed keeps one clipped line, expanded scrolls. */
 					}
 				};
 				sessions.set(sessionId, entry);
-				attach();
 				return () => {
 					if (sessions.get(sessionId) === entry) sessions.delete(sessionId);
-					if (sessions.size === 0) detach();
 				};
 			}, [props.sessionId]);
 
@@ -3649,6 +3754,12 @@ border-top:1px solid color-mix(in srgb,currentColor 18%,transparent)}
 			// The sent-message capsule lives outside the composer, so it is
 			// installed with the plugin and not with a session's slot entry.
 			ctx.effect(installCapsules, "ui-quote: sent-message capsules");
+			// So does the bar: its listeners are document singletons and must not
+			// depend on any composer staying mounted.
+			ctx.effect(() => {
+				attachListeners();
+				return () => detachListeners();
+			}, "ui-quote: selection listeners");
 			// The three rows are also three keys. Without the service the menu
 			// stays the only way in — nothing here is required to work.
 			ctx.effect(() => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-client-ui-quote",
-  "version": "0.11.1",
+  "version": "0.11.3",
   "description": "Select text in the DSH transcript, a file preview or the terminal to quote it into the composer or into a side thread's composer, Kimi-Code style (comment / add to conversation / side chat), with Alt+Q/C/B, a jump back to the quoted source, a searchable quote list page that exports as Markdown, one-click back-flow of a side thread's conclusion, and revisions of a sent quote that mark the old capsule as superseded. The quoted text itself is editable in the comment box (trim a selection, correct a revision's wording), a selection from the file preview or the terminal travels to the model as a fenced code block carrying its file path and line range, and a per-quote reply toggle has the model answer quote by quote, each 「引用N」 label in the reply becoming a badge linked back to the quote.",
   "icon": "icon.svg",
   "license": "MIT",

package/test/capsule.mjs

@@ -914,6 +914,21 @@ try {
 	// this block is about what a mounted enhancer does with a revision.
 	capsuleEffect.fn();
 
+	// A textarea measures itself from its own text — but only while it is in the
+	// document, the way the real one reads 0 before the bar is laid out. That is
+	// what made the auto-grow collapse the quote field.
+	const makeTextarea = dom.document.createElement.bind(dom.document);
+	dom.document.createElement = (tag) => {
+		const node = makeTextarea(tag);
+		if (String(tag).toLowerCase() === "textarea") {
+			Object.defineProperty(node, "scrollHeight", {
+				configurable: true,
+				get: () => (node.isConnected ? 12 + 12 * String(node.value ?? "").split("\n").length : 0)
+			});
+		}
+		return node;
+	};
+
 	// A transcript of its own, with a capsule that has a comment.
 	const host2 = dom.document.createElement("div");
 	host2.setAttribute("data-conversation-content", "");
@@ -948,11 +963,16 @@ try {
 	const input = bar?.querySelector(".dshq-comment-input");
 	check("the box is prefilled with the old quote for editing", quoteBox?.value === original, JSON.stringify(quoteBox?.value));
 	check("the box is prefilled with the old comment", input?.value === oldComment, JSON.stringify(input?.value));
+	// Regression: the quote half used to be measured before it was attached, so
+	// it was written `height: 0px` and its first line showed up clipped.
+	check("the quote field is grown to its text, not collapsed", quoteBox?.style?.height === "24px", JSON.stringify(quoteBox?.style?.height));
+	check("the comment field is grown too", input?.style?.height === "24px", JSON.stringify(input?.style?.height));
 
 	// Correcting it writes a NEW quote (nothing is edited in place) and the old
 	// capsule learns it was superseded.
 	input.value = newComment;
 	input.dispatch("input", {});
+	check("typing in a field re-measures it", input?.style?.height === "24px", JSON.stringify(input?.style?.height));
 	// The bar handles clicks by delegation, so the click is dispatched on the bar
 	// with the button as its target — the shim has no bubbling of its own.
 	const confirm = bar.querySelector('[data-action="confirm"]');
@@ -1345,6 +1365,145 @@ try {
 	);
 }
 
+/* ------------------------------------------------------------------ *
+ * 浮层不跟着座位生死 — the bar is document-level, so neither its
+ * listeners nor the action behind one of its rows may depend on an
+ * overlay slot entry still being mounted.
+ *
+ * The listeners used to be reference-counted by the slot component: an
+ * overlapping mount pair (a side thread's composer over a running
+ * parent, a re-registered slot) could drive the count to zero, the
+ * document listeners were then removed for good, and the bar never came
+ * back until a page reload — the "引用插件经常失效" report. A commit
+ * whose slot entry had gone also returned silently.
+ * ------------------------------------------------------------------ */
+
+{
+	const listenersEffect = effects.find((entry) => String(entry.label).includes("selection listeners"));
+	check("the bar's listeners are installed with the plugin, not with a slot", listenersEffect !== undefined);
+	check(
+		"and they are one document-level set, never a slot-scoped count",
+		(dom.document.listeners.get("pointerup") ?? []).length === 1,
+		JSON.stringify((dom.document.listeners.get("pointerup") ?? []).length)
+	);
+
+	const pane = (sessionId, text) => {
+		const node = dom.document.createElement("div");
+		node.setAttribute("data-conversation-content", "");
+		node.setAttribute("data-conversation-session", sessionId);
+		const bubble = dom.document.createElement("div");
+		bubble.className = "bubble";
+		const run = dom.document.createElement("span");
+		run.className = "plainRun";
+		run.textContent = text;
+		bubble.append(run);
+		node.append(bubble);
+		dom.document.body.append(node);
+		return { node, run };
+	};
+	const selectIn = (run, text) => {
+		dom.document.selection = {
+			isCollapsed: false,
+			rangeCount: 1,
+			toString: () => text,
+			removeAllRanges() {
+				dom.document.selection = null;
+			},
+			getRangeAt: () => ({
+				startContainer: run.firstChild,
+				endContainer: run.firstChild,
+				intersectsNode: () => false,
+				getBoundingClientRect: () => ({ left: 100, top: 300, right: 220, bottom: 318, width: 120, height: 18 })
+			})
+		};
+		dom.document.dispatch("pointerdown", { target: run.firstChild });
+		dom.document.dispatch("pointerup", { target: run.firstChild });
+	};
+	const barOf = () => dom.document.getElementById("dsh-quote-bar");
+
+	// 1. The composer slot never mounted for this session at all.
+	const firstPane = pane("session-bar", "浮层要跟着手走的原文");
+	selectIn(firstPane.run, "浮层要跟着手走的原文");
+	check(
+		"a selection opens the bar even with no composer slot mounted for that session",
+		barOf()?.dataset.visible === "true",
+		JSON.stringify(barOf()?.dataset.visible)
+	);
+	check(
+		"the bar offers the actions",
+		barOf()?.querySelector('[data-action="add"]') !== null && barOf()?.querySelector('[data-action="comment"]') !== null
+	);
+
+	// 2. A slot comes and goes — the listeners are not tied to it.
+	const overlay = registered.find((entry) => entry.spec.name === "conversation.input.overlay");
+	check("the overlay slot is registered for mounting", overlay !== undefined);
+	dom.document.dispatch("keydown", { key: "Escape", isComposing: false, target: firstPane.run });
+	check("Escape closes the bar", barOf()?.dataset.visible === "false", JSON.stringify(barOf()?.dataset.visible));
+	if (overlay !== undefined) {
+		overlay.Component({ sessionId: "session-bar", inputActions: { captureInsertion: () => ({ draftRev: 1 }) } });
+		overlay.Component({ sessionId: "session-bar-2", inputActions: { captureInsertion: () => ({ draftRev: 2 }) } });
+	}
+	check(
+		"mounting two composers does not stack or drop the listeners",
+		(dom.document.listeners.get("pointerup") ?? []).length === 1,
+		JSON.stringify((dom.document.listeners.get("pointerup") ?? []).length)
+	);
+	selectIn(firstPane.run, "浮层要跟着手走的原文");
+	check("the bar still opens after a slot mounted", barOf()?.dataset.visible === "true", JSON.stringify(barOf()?.dataset.visible));
+
+	// 3. A session with no slot entry still lands its chip: the composer's own
+	//    actions stand in for the entry that is not there.
+	const landed = [];
+	shells.set("session-noslot", {
+		actions: { captureInsertion: () => ({ draftRev: 7 }) },
+		insertReference(attachment) {
+			landed.push(attachment);
+			return true;
+		}
+	});
+	const secondPane = pane("session-noslot", "没有座位的会话也要能引用");
+	selectIn(secondPane.run, "没有座位的会话也要能引用");
+	barOf().dispatch("click", { target: barOf().querySelector('[data-action="add"]') });
+	check(
+		"the chip lands through the composer's own actions",
+		landed.length === 1 && String(landed[0]?.ref ?? "").includes("没有座位的会话也要能引用"),
+		JSON.stringify([landed.length, String(landed[0]?.ref ?? "").slice(0, 24)])
+	);
+
+	// 4. With nowhere to insert, the reader is told instead of left wondering.
+	const thirdPane = pane("session-noshell", "没有输入框的会话");
+	selectIn(thirdPane.run, "没有输入框的会话");
+	const staleToast = dom.document.getElementById("dsh-quote-toast");
+	if (staleToast !== null) staleToast.textContent = "";
+	barOf().dispatch("click", { target: barOf().querySelector('[data-action="add"]') });
+	const toastNode = dom.document.getElementById("dsh-quote-toast");
+	check(
+		"a dead end says so instead of doing nothing",
+		String(toastNode?.textContent) === "插入失败，请重试",
+		JSON.stringify(toastNode?.textContent)
+	);
+
+	// 5. Disposal is honest: the effect takes its listeners with it, and
+	//    reinstalling puts exactly one set back.
+	listenersEffect.dispose();
+	check(
+		"disposing the effect removes the listeners",
+		(dom.document.listeners.get("pointerup") ?? []).length === 0,
+		JSON.stringify((dom.document.listeners.get("pointerup") ?? []).length)
+	);
+	listenersEffect.fn();
+	check(
+		"reinstalling brings back exactly one set",
+		(dom.document.listeners.get("pointerup") ?? []).length === 1,
+		JSON.stringify((dom.document.listeners.get("pointerup") ?? []).length)
+	);
+	selectIn(firstPane.run, "浮层要跟着手走的原文");
+	check("the bar works again after a reinstall", barOf()?.dataset.visible === "true", JSON.stringify(barOf()?.dataset.visible));
+
+	dom.document.selection = null;
+	dom.document.dispatch("keydown", { key: "Escape", isComposing: false, target: firstPane.run });
+}
+
 console.log(
 	failures.length === 0
 		? "\ndsh-client-ui-quote capsule: " + passed + " checks passed"

package/test/dom.mjs

@@ -31,6 +31,10 @@ class ShimNode {
 		return node.nodeType === 9 || node === this.ownerDocument?.documentElement;
 	}
 
+	get firstChild() {
+		return this.childNodes.length === 0 ? null : this.childNodes[0];
+	}
+
 	get lastChild() {
 		return this.childNodes.length === 0 ? null : this.childNodes[this.childNodes.length - 1];
 	}
@@ -306,6 +310,10 @@ class ShimDocument extends ShimNode {
 		this.ownerDocument = this;
 		this.mutations = [];
 		this.observers = [];
+		/** Document-level listeners (the bar's own are installed here). */
+		this.listeners = new Map();
+		/** Set by a test to hand `document.getSelection()` a live selection. */
+		this.selection = null;
 		/** Maintained by `ShimElement.focus()` / `blur()`. */
 		this.activeElement = null;
 		this.documentElement = new ShimElement(this, 'html');
@@ -357,12 +365,28 @@ class ShimDocument extends ShimNode {
 		return found;
 	}
 
-	addEventListener() {}
+	addEventListener(type, handler) {
+		// The bar's listeners are document singletons; the harness has to keep
+		// them so a test can drive a selection the way a browser would.
+		const list = this.listeners.get(type) ?? [];
+		list.push(handler);
+		this.listeners.set(type, list);
+	}
+
+	removeEventListener(type, handler) {
+		const list = this.listeners.get(type) ?? [];
+		const index = list.indexOf(handler);
+		if (index !== -1) list.splice(index, 1);
+	}
 
-	removeEventListener() {}
+	dispatch(type, event) {
+		for (const handler of [...(this.listeners.get(type) ?? [])]) handler(event);
+	}
 
 	getSelection() {
-		// A collapsed selection that can be cleared, which is all the bar asks of it.
+		// A collapsed selection that can be cleared, which is all the bar asks of
+		// it — unless a test installs one with `document.selection = …`.
+		if (this.selection !== null && this.selection !== undefined) return this.selection;
 		return { isCollapsed: true, rangeCount: 0, removeAllRanges() {} };
 	}
 }