dsh-quote-followup 0.2.4 → 0.2.6

package/README.md

@@ -18,11 +18,21 @@ A Web-only [DeepSeek Harness](https://github.com/deepseek-ai/deepseek-harness) p
 - Select text inside the Web conversation transcript to reveal a floating **Quote** button.
 - Append a native DSH conversation-reference chip without replacing the existing draft.
 - Reuse the same atomic `ReferenceChipNode`, conversation icon, and business color as `@file` / `@session`.
+- Keep each chip compact by showing only the excerpt; the bubble icon already conveys that it is a quote.
+- Follow the active DSH locale for the floating action and serialized quote frame.
+- Attach the conversation turn number to the serialized frame when the selection comes from a DSH chat row, so follow-ups like "revisit turn 3" stay resolvable. Selections without a turn marker keep the provenance-free frame.
 - Quote multiple excerpts; on send, the plugin codec expands each chip into a model-readable Markdown blockquote.
 - Fall back to a plain-text quote when the host lacks native chip support.
 
 TUI is intentionally unsupported.
 
+## Scope
+
+This is a Web client-side extension. The composer and send path are owned by the browser client: chips exist only in the Lexical editor, and the model only ever sees the Markdown blockquotes the codec expands at send time — never the chips themselves. No host-side plugin surface is involved.
+
+Chips are text-only by design. They carry the excerpt, a display-only role hint, the conversation turn ordinal when it is available, and a truncation flag — no global session-message references — so they stay valid across compaction folds and session rotation.
+
+
 ## Install
 
 Requires DSH `>=0.1.2-rc.1`.
@@ -46,7 +56,7 @@ Add `dsh-quote-followup` to the Web profile's `dsh.profile.bundles`, then restar
 npm test
 ```
 
-The regression harness covers native quote chips, repeated quoting, spacing after an existing draft, codec serialization, the Firefox text fallback, and current-client takeover of stale singleton/button state after a hot swap.
+The regression harness covers compact native quote chips, locale switching, turn-number provenance with graceful degradation, repeated quoting, spacing after an existing draft, codec serialization, the Firefox text fallback, and current-client takeover of stale singleton/button state after a hot swap.
 
 ## License
 

package/README.zh.md

@@ -18,11 +18,21 @@
 - 在 Web 对话区选中文本后显示浮动的 **❐ 引用** 按钮。
 - 以 DSH 原生对话引用 chip 追加到现有草稿，不覆盖已输入内容。
 - chip 使用与 `@文件` / `@对话` 相同的 `ReferenceChipNode`、对话图标和业务色，可整体删除。
+- chip 只展示摘录正文，引用语义由气泡图标表达，减少重复标签。
+- 浮动按钮和发送时的引用框架跟随 DSH 当前语言。
+- 选中内容来自 DSH 聊天行时，序列化引用会附带对话轮次编号，模型可定位"第 3 轮引用的内容"；无轮次标记时保持原有框架。
 - 可连续引用多段内容；发送时由插件 codec 将各 chip 展开为模型可读的 Markdown 引用块。
 - 旧版 DSH 缺少原生 chip 能力时，自动降级为纯文本引用。
 
 本插件不再适配 TUI。
 
+## 边界
+
+这是一个 Web 客户端侧扩展。composer 和发送路径由浏览器 client 拥有：chip 只存在于 Lexical 编辑器中，模型只会看到发送时 codec 展开的 Markdown 引用块，永远不会看到 chip 本身，不涉及任何 host 侧插件接口。
+
+chip 刻意只存文本：包含摘录正文、仅供显示的角色提示、可用的对话轮次编号和截断标记，不含全局 session-message 引用，因此在 compaction 折叠与会话轮转后依然有效。
+
+
 ## 安装
 
 要求 DSH `>=0.1.2-rc.1`。
@@ -46,7 +56,7 @@ dsh plugin --profile web add dsh-quote-followup
 npm test
 ```
 
-回归测试覆盖原生引用 chip、连续引用、已有草稿后的间距、codec 序列化、Firefox 文本降级路径，以及热替换后新版 client 接管旧按钮/单例状态。
+回归测试覆盖精简的原生引用 chip、中英文 locale 切换、轮次溯源与降级、连续引用、已有草稿后的间距、codec 序列化、Firefox 文本降级路径，以及热替换后新版 client 接管旧按钮/单例状态。
 
 ## 许可证
 

package/lib/client.js

@@ -7,9 +7,11 @@
  * can edit before sending — a visible, targeted follow-up with zero hidden
  * state and zero host-side privileges.
  *
- * Pure DOM (no React), so the module carries no platform dependencies:
- * `inject: []`. Mounted by the web client runtime through the package's
- * `dsh.client` declaration (`platform: "web"`).
+ * Pure DOM (no React). The only injected host services are DSH's
+ * `inputTriggers` codec registry and `locale` (both optional — the module
+ * degrades to English text fallbacks when either is absent). Mounted by the
+ * web client runtime through the package's `dsh.client` declaration
+ * (`platform: "web"`).
  *
  * @module dsh-quote-followup/client
  */
@@ -33,7 +35,7 @@ window.__ModuleLoader__.load({
 		];
 		const BUTTON_ID = "dsh-quote-followup-btn";
 		const BUTTON_VERSION_ATTR = "data-dsh-quote-followup-version";
-		const CLIENT_VERSION = "0.2.4";
+		const CLIENT_VERSION = "0.2.6";
 		/** Codec owner for native DSH reference chips. */
 		const QUOTE_SOURCE = "quote-followup";
 		const LOCALE_NS = "quote-followup";
@@ -44,14 +46,20 @@ window.__ModuleLoader__.load({
 				"button.title": "引用选中内容到输入框（可多次引用，发送前可编辑）",
 				"quote.header": "引用",
 				"conversation.role": "对话",
-				"quote.truncated": "，已截断"
+				"conversation.role.user": "用户",
+				"conversation.role.assistant": "助手",
+				"quote.turn": "第 {n} 轮",
+				"quote.truncated": "已截断"
 			},
 			en: {
 				"button.label": "❐ Quote",
 				"button.title": "Quote the selection into the composer (repeatable; editable before sending)",
 				"quote.header": "Quote",
 				"conversation.role": "conversation",
-				"quote.truncated": ", truncated"
+				"conversation.role.user": "user",
+				"conversation.role.assistant": "assistant",
+				"quote.turn": "turn {n}",
+				"quote.truncated": "truncated"
 			}
 		};
 		const fallbackT = (key) => LOCALE_DICT.en[key] ?? key;
@@ -64,6 +72,19 @@ window.__ModuleLoader__.load({
 			const element = elementOf(node);
 			return element === null ? null : element.closest(TRANSCRIPT_SELECTOR);
 		};
+		/** Walk from a range endpoint up to (but excluding) the transcript. */
+		const walkAncestors = (node, transcript, visit) => {
+			let element = elementOf(node);
+			while (element !== null && element !== transcript) {
+				const found = visit(element);
+				if (found !== undefined)
+					return found;
+				element = element.parentElement;
+			}
+			return null;
+		};
+		/** A turn marker is only meaningful as a non-negative safe integer. */
+		const isValidTurn = (turn) => Number.isSafeInteger(turn) && turn >= 0;
 		/** The live selection when BOTH ends sit inside the same transcript. */
 		const selectionInTranscript = (selection) => {
 			if (selection === null || selection.isCollapsed || selection.rangeCount === 0)
@@ -79,9 +100,8 @@ window.__ModuleLoader__.load({
 		 * looking for role markers (data attributes, then class-name hints).
 		 * Null → the generic "对话" label; this is display-only metadata.
 		 */
-		const detectRole = (range, transcript) => {
-			let element = elementOf(range.startContainer);
-			while (element !== null && element !== transcript) {
+		const detectRole = (range, transcript) =>
+			walkAncestors(range.startContainer, transcript, (element) => {
 				const role = element.getAttribute?.("data-role") ?? element.getAttribute?.("data-message-role");
 				if (typeof role === "string" && role !== "") {
 					if (/user|human/i.test(role)) return "user";
@@ -90,35 +110,60 @@ window.__ModuleLoader__.load({
 				const cls = String(element.className ?? "");
 				if (/(^|[\s_-])user([\s_-]|$)/i.test(cls)) return "user";
 				if (/(^|[\s_-])assistant([\s_-]|$)/i.test(cls)) return "assistant";
-				element = element.parentElement;
-			}
-			return null;
-		};
-		const roleLabel = (role, t) => role === "user" ? "user" : role === "assistant" ? "assistant" : t("conversation.role");
+				return undefined;
+			});
+		const roleLabel = (role, t) => role === "user" ? t("conversation.role.user") : role === "assistant" ? t("conversation.role.assistant") : t("conversation.role");
+		/**
+		 * Conversation-turn provenance: DSH's chat renderer stamps every message
+		 * row with `data-chat-turn` (a per-session monotonic ordinal). Resolving
+		 * it lets the model address "what you quoted in turn N" without any
+		 * host-side message-id API. Absent / non-numeric markers degrade to null
+		 * and the serialized frame stays exactly as before.
+		 */
+		const detectTurn = (range, transcript) =>
+			walkAncestors(range.startContainer, transcript, (element) => {
+				const raw = element.getAttribute?.("data-chat-turn");
+				if (typeof raw === "string" && raw !== "") {
+					const turn = Number(raw);
+					if (isValidTurn(turn))
+						return turn;
+				}
+				return undefined;
+			});
 		/** Bound one quote before it becomes editor state. */
-		const quotePayload = (text, role) => {
+		const quotePayload = (text, role, turn) => {
 			let body = String(text ?? "");
 			let truncated = false;
 			if (body.length > QUOTE_MAX_CHARS) {
 				body = body.slice(0, QUOTE_MAX_CHARS);
 				truncated = true;
 			}
-			return { text: body, role, truncated };
+			return { text: body, role, truncated, turn: isValidTurn(turn) ? turn : null };
 		};
 		/** Model / clipboard projection. The composer shows a chip, not this frame. */
-		const quoteFrame = ({ text, role, truncated }, t) => {
+		const quoteFrame = ({ text, role, truncated, turn }, t) => {
 			const lines = text.split("\n").map((line) => `> ${line}`.trimEnd());
-			return `> [${t("quote.header")} · ${roleLabel(role, t)}${truncated ? t("quote.truncated") : ""}]\n${lines.join("\n")}\n\n`;
+			const meta = [t("quote.header"), roleLabel(role, t)];
+			if (isValidTurn(turn))
+				meta.push(String(t("quote.turn")).replace("{n}", String(turn)));
+			if (truncated)
+				meta.push(t("quote.truncated"));
+			return `> [${meta.join(" · ")}]\n${lines.join("\n")}\n\n`;
 		};
 		const decodeQuote = (ref) => {
-			const value = JSON.parse(ref);
-			if (value === null || typeof value !== "object" || typeof value.text !== "string")
-				throw new Error("invalid quote-followup reference");
-			return {
-				text: value.text,
-				role: value.role === "user" || value.role === "assistant" ? value.role : null,
-				truncated: value.truncated === true
-			};
+			try {
+				const value = JSON.parse(ref);
+				if (value === null || typeof value !== "object" || typeof value.text !== "string")
+					return null;
+				return {
+					text: value.text,
+					role: value.role === "user" || value.role === "assistant" ? value.role : null,
+					truncated: value.truncated === true,
+					turn: isValidTurn(value.turn) ? value.turn : null
+				};
+			} catch {
+				return null;
+			}
 		};
 		const quoteInsert = (payload, t) => {
 			const clipboardText = quoteFrame(payload, t);
@@ -134,17 +179,24 @@ window.__ModuleLoader__.load({
 			};
 		};
 		/** Codec-only source: it owns quote chips but intentionally adds no @ candidates. */
-		const makeQuoteSource = (t) => ({
-			trigger: "@",
-			name: QUOTE_SOURCE,
-			order: 1000,
-			candidates: () => Promise.resolve([]),
-			onPick: () => void 0,
-			codec: {
-				clipboardText: (ref) => quoteFrame(decodeQuote(ref), t),
-				serialize: (ref) => Promise.resolve(quoteFrame(decodeQuote(ref), t))
-			}
-		});
+		const makeQuoteSource = (t) => {
+			/** Project a chip ref to Markdown, degrading to "" for malformed refs. */
+			const project = (ref) => {
+				const payload = decodeQuote(ref);
+				return payload === null ? "" : quoteFrame(payload, t);
+			};
+			return {
+				trigger: "@",
+				name: QUOTE_SOURCE,
+				order: 1000,
+				candidates: () => Promise.resolve([]),
+				onPick: () => void 0,
+				codec: {
+					clipboardText: project,
+					serialize: (ref) => Promise.resolve(project(ref))
+				}
+			};
+		};
 		const findComposer = () => {
 			for (const selector of COMPOSER_SELECTORS) {
 				const element = document.querySelector(selector);
@@ -378,7 +430,11 @@ window.__ModuleLoader__.load({
 				hideButton();
 				return;
 			}
-			pendingQuote = { text: text.trim(), role: detectRole(selection.getRangeAt(0), transcript) };
+			pendingQuote = {
+				text: text.trim(),
+				role: detectRole(selection.getRangeAt(0), transcript),
+				turn: detectTurn(selection.getRangeAt(0), transcript)
+			};
 			placeButton(ensureButton(), rect);
 		};
 		const onQuoteClick = () => {
@@ -395,7 +451,7 @@ window.__ModuleLoader__.load({
 			// Clear the transcript range BEFORE focusing the composer. Clearing after
 			// insertion destroys Lexical's caret and makes the next quote unreliable.
 			document.getSelection()?.removeAllRanges();
-			const payload = quotePayload(quote.text, quote.role);
+			const payload = quotePayload(quote.text, quote.role, quote.turn);
 			if (!(quoteSourceReady && appendQuoteChip(composer, payload, currentT)))
 				appendToComposer(composer, quoteFrame(payload, currentT));
 		};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-quote-followup",
-  "version": "0.2.4",
+  "version": "0.2.6",
   "description": "Quote selected Web conversation text into the DeepSeek Harness composer for targeted follow-up turns.",
   "keywords": [
     "dsh",