dsh-quote-followup 0.2.5 → 0.2.6

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.5";
+		const CLIENT_VERSION = "0.2.6";
 		/** Codec owner for native DSH reference chips. */
 		const QUOTE_SOURCE = "quote-followup";
 		const LOCALE_NS = "quote-followup";
@@ -44,6 +46,8 @@ window.__ModuleLoader__.load({
 				"button.title": "引用选中内容到输入框（可多次引用，发送前可编辑）",
 				"quote.header": "引用",
 				"conversation.role": "对话",
+				"conversation.role.user": "用户",
+				"conversation.role.assistant": "助手",
 				"quote.turn": "第 {n} 轮",
 				"quote.truncated": "已截断"
 			},
@@ -52,6 +56,8 @@ window.__ModuleLoader__.load({
 				"button.title": "Quote the selection into the composer (repeatable; editable before sending)",
 				"quote.header": "Quote",
 				"conversation.role": "conversation",
+				"conversation.role.user": "user",
+				"conversation.role.assistant": "assistant",
 				"quote.turn": "turn {n}",
 				"quote.truncated": "truncated"
 			}
@@ -66,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)
@@ -81,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";
@@ -92,11 +110,9 @@ 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
@@ -104,19 +120,16 @@ window.__ModuleLoader__.load({
 		 * host-side message-id API. Absent / non-numeric markers degrade to null
 		 * and the serialized frame stays exactly as before.
 		 */
-		const detectTurn = (range, transcript) => {
-			let element = elementOf(range.startContainer);
-			while (element !== null && element !== transcript) {
+		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 (Number.isSafeInteger(turn) && turn >= 0)
+					if (isValidTurn(turn))
 						return turn;
 				}
-				element = element.parentElement;
-			}
-			return null;
-		};
+				return undefined;
+			});
 		/** Bound one quote before it becomes editor state. */
 		const quotePayload = (text, role, turn) => {
 			let body = String(text ?? "");
@@ -125,28 +138,32 @@ window.__ModuleLoader__.load({
 				body = body.slice(0, QUOTE_MAX_CHARS);
 				truncated = true;
 			}
-			return { text: body, role, truncated, turn: Number.isSafeInteger(turn) && turn >= 0 ? turn : null };
+			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, turn }, t) => {
 			const lines = text.split("\n").map((line) => `> ${line}`.trimEnd());
 			const meta = [t("quote.header"), roleLabel(role, t)];
-			if (Number.isSafeInteger(turn) && turn >= 0)
+			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,
-				turn: Number.isSafeInteger(value.turn) && value.turn >= 0 ? value.turn : null
-			};
+			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);
@@ -162,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);

package/package.json

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