pi-copy-message 2.1.0 → 3.0.0

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # Changelog
 
+## 3.0.0 - 2026-09-26
+
+- Require Node.js 24.15 or newer for repository tooling; `.nvmrc` pins 24.21.0 and `packageManager` is npm 12.1.0.
+- Run tests with Node's built-in `node --test` and type stripping instead of `tsx`; typecheck with TypeScript 7.0.2 and `erasableSyntaxOnly`.
+- Refresh the development baseline to Pi 0.87.1, qualified against official Pi and the maintained fork.
+- Guard `package-lock.json` against private-registry URLs.
+- First npm release containing the 2.1.1 clipboard change (2.1.1 was tagged but never published to npm).
+
+## 2.1.1 - 2026-08-25
+
+- Reuse Pi's clipboard implementation instead of maintaining a duplicate command runner, enabling Pi's OSC 52 fallback on Android/Termux.
+
 ## 2.1.0 - 2026-08-17
 
 - Add an `Alt+C` picker filter and `MessageVisibility.showCustom` option for showing or hiding custom messages.

package/README.md

@@ -16,6 +16,7 @@ A [pi](https://github.com/earendil-works/pi-mono) extension that adds `/copy-mes
 - Supports type-to-filter search across role and message text, with `time:<term>` for timestamp search
 - Supports Home/End jumps for oldest/newest visible messages
 - Uses Pi's configured selection bindings for navigation, copy, and cancel
+- Uses Pi's clipboard implementation, including its OSC 52 terminal fallback
 - Includes fast paths: `/copy-message latest`, `/copy-message last`, and `/copy-message newest`
 - Supports direct numbered copies like `/copy-message 3`
 - Supports metadata copies with `--with-meta`, `--with-metadata`, or `--with-role`
@@ -34,11 +35,7 @@ Or install it directly from GitHub with pi:
 pi install https://github.com/fitchmultz/pi-copy-message
 ```
 
-Then reload pi from inside the app:
-
-```text
-/reload
-```
+Restart Pi after installing or updating extension code or dependencies. The maintained fork's `/reload` refreshes resources and reinitializes cached code; it does not apply code updates.
 
 If you prefer to load it directly from a local checkout during development:
 
@@ -97,7 +94,7 @@ Copy with role and timestamp metadata instead of raw text only:
 | `Ctrl+PageUp` / `Ctrl+PageDown` | Move by one visible page in fullscreen mode |
 | `Home` / `End` | Jump to oldest/newest visible message in regular mode |
 | `Ctrl+Home` / `Ctrl+End` | Jump to oldest/newest visible message in fullscreen mode |
-| Type text | Filter visible messages |
+| Type or paste text | Filter visible messages; pasted line breaks and tabs become spaces |
 | `time:<term>` | Search timestamps |
 | `Backspace` | Delete one search character |
 | `Ctrl+U` | Toggle user messages |
@@ -115,6 +112,7 @@ Copy with role and timestamp metadata instead of raw text only:
 - The picker caps visible rows and scrolls instead of filling the screen.
 - Search preserves your original selected message and restores it when the search is cleared.
 - General search does not match timestamps; use `time:<term>` when you want to search by displayed time.
+- Picker previews and copy notifications remove terminal control characters; the copied text stays raw.
 - Filter labels honor the active pi theme.
 - Selection key hints show the active Pi bindings. Configured selection actions take precedence if they collide with the picker's filter, preview, or format shortcuts.
 - Only custom messages with `display: true` can appear or be copied from current `custom_message` entries or legacy message-wrapped entries; visible ones can be toggled with `Alt+C`.
@@ -125,18 +123,28 @@ Copy with role and timestamp metadata instead of raw text only:
 
 ## Compatibility
 
-- Requires and is tested with Pi 0.84.0 or later
+- Declared Pi floor: 0.84.0. Qualified on official Pi 0.87.1 and the maintained fork's `main` (not a claim that every later release has been tested).
+- The maintained fork persists in-progress assistant snapshots and context-window boundaries; the picker coalesces those so each response appears once. Official Pi writes neither, and the same code handles both.
+- Android/Termux clipboard writes use Pi's normal fallback chain, including OSC 52 when native clipboard commands are unavailable.
 - Pi includes `Ctrl+X` for copying the latest assistant response; this extension remains useful for searchable history, other roles, metadata, and direct selectors.
-- Supported Node.js range for local repo tooling: `>=22.19.0`
-- `.nvmrc` pins Node 22.19.0 for local development
+- Node.js `>=24.15.0`. `.nvmrc` pins Node 24.21.0 for local development.
 
 This package keeps pi core packages as wildcard peers (`*`) per pi package guidance. Pi aliases these imports to its own bundled copies when loading the extension, so the package never bundles or shadows pi core. Local development uses `@earendil-works/pi-coding-agent` and `@earendil-works/pi-tui` as dev dependencies for typechecking and tests.
 
 ## Development
 
 ```bash
-npm install
-npm run check
+npm ci
+npm run check # node --test + tsc --noEmit + pack dry-run
+```
+
+No production build or `prepare` is required. Tests run TypeScript directly with Node's type stripping, so source must stay erasable (`erasableSyntaxOnly`). Qualification uses the selected host installed in this checkout's dependency graph. The tests cover picker behavior and formatting without clipboard writes; native clipboard delivery, including Android/Termux and OSC 52, requires separate platform qualification.
+
+`package-lock.json` must only reference `registry.npmjs.org`; a test fails on private-registry URLs. Behind a registry proxy that rewrites tarball URLs, refresh the lock from public metadata and install through the proxy:
+
+```bash
+npm install --package-lock-only --ignore-scripts --registry=https://registry.npmjs.org/
+npm ci
 ```
 
 Key files:

package/extensions/copy-message.ts

@@ -1,5 +1,4 @@
-import { spawnSync } from "node:child_process";
-import type { ExtensionAPI, ExtensionCommandContext, KeybindingsManager } from "@earendil-works/pi-coding-agent";
+import { copyToClipboard, type ExtensionAPI, type ExtensionCommandContext, type KeybindingsManager } from "@earendil-works/pi-coding-agent";
 import type { AutocompleteItem, TuiMode } from "@earendil-works/pi-tui";
 import { decodeKittyPrintable, matchesKey, truncateToWidth, visibleWidth, wrapTextWithAnsi } from "@earendil-works/pi-tui";
 
@@ -33,7 +32,7 @@ function textFromContent(content: unknown): string {
 			);
 		})
 		.map((part) => part.text)
-		.join("\n\n");
+		.join("");
 }
 
 function textFromMessage(message: Record<string, unknown>): string {
@@ -56,14 +55,10 @@ function textFromMessage(message: Record<string, unknown>): string {
 	return textFromContent(message.content);
 }
 
-type SegmenterCtor = new (locale?: string, options?: { granularity?: "grapheme" }) => {
-	segment(input: string): Iterable<{ segment: string }>;
-};
+const graphemeSegmenter = new Intl.Segmenter(undefined, { granularity: "grapheme" });
 
 function splitGraphemes(text: string): string[] {
-	const Segmenter = (Intl as unknown as { Segmenter?: SegmenterCtor }).Segmenter;
-	if (!Segmenter) return Array.from(text);
-	return Array.from(new Segmenter(undefined, { granularity: "grapheme" }).segment(text), (part) => part.segment);
+	return Array.from(graphemeSegmenter.segment(text), (part) => part.segment);
 }
 
 function truncateGraphemes(text: string, max: number): string {
@@ -74,8 +69,12 @@ function truncateGraphemes(text: string, max: number): string {
 	return `${graphemes.slice(0, max - 1).join("")}…`;
 }
 
+function safeDisplayText(text: string): string {
+	return text.replace(/[\x00-\x08\x0b\x0c\x0e-\x1f\x7f-\x9f]/g, "");
+}
+
 function compactPreview(text: string, max = 96): string {
-	const preview = text.replace(/\s+/gu, " ").trim();
+	const preview = safeDisplayText(text).replace(/\s+/gu, " ").trim();
 	return truncateGraphemes(preview, max);
 }
 
@@ -96,7 +95,7 @@ function roleLabel(role: string): string {
 		case "compactionSummary":
 			return "compaction";
 		default:
-			return role || "message";
+			return safeDisplayText(role).replace(/\s+/gu, " ").trim() || "message";
 	}
 }
 
@@ -138,7 +137,33 @@ function entryToCopyableMessage(entry: unknown): CopyableMessage | undefined {
 }
 
 export function collectCopyableMessages(ctx: { sessionManager: { getBranch(): unknown[] } }): CopyableMessage[] {
-	return ctx.sessionManager.getBranch().flatMap((entry) => {
+	const entries: unknown[] = [];
+	// Pi persists assistant snapshots separately and coalesces them within each context window.
+	const responses = new Map<string, number>();
+	for (const entry of ctx.sessionManager.getBranch()) {
+		const record = entry as {
+			type?: string;
+			message?: { role?: string; responseId?: string; stopReason?: string; content?: unknown[] };
+		} | null;
+		if (record?.type === "context_window") responses.clear();
+		const responseId =
+			record?.type === "message" && record.message?.role === "assistant" && typeof record.message.responseId === "string"
+				? record.message.responseId
+				: undefined;
+		const previous = responseId ? responses.get(responseId) : undefined;
+		if (previous !== undefined) {
+			const earlier = entries[previous] as typeof record;
+			if (
+				record?.message?.stopReason !== "pending" ||
+				(earlier?.message?.stopReason === "pending" && (earlier.message.content?.length ?? 0) <= (record.message.content?.length ?? 0))
+			) entries[previous] = entry;
+			continue;
+		}
+		if (responseId) responses.set(responseId, entries.length);
+		entries.push(entry);
+	}
+
+	return entries.flatMap((entry) => {
 		const message = entryToCopyableMessage(entry);
 		return message ? [message] : [];
 	});
@@ -181,58 +206,6 @@ export function getMostRecentUserMessage(ctx: { sessionManager: { getBranch(): u
 	return sawUserMessage ? { kind: "no-text" } : { kind: "no-user-message" };
 }
 
-type ClipboardCommand = {
-	name: string;
-	args: string[];
-	enabled: () => boolean;
-};
-
-const clipboardCommands: ClipboardCommand[] = [
-	{ name: "pbcopy", args: [], enabled: () => process.platform === "darwin" },
-	{ name: "termux-clipboard-set", args: [], enabled: () => Boolean(process.env.TERMUX_VERSION) },
-	{ name: "wl-copy", args: [], enabled: () => true },
-	{ name: "xclip", args: ["-selection", "clipboard"], enabled: () => true },
-	{ name: "xsel", args: ["--clipboard", "--input"], enabled: () => true },
-];
-
-const CLIPBOARD_TIMEOUT_MS = 3000;
-const commandExistsCache = new Map<string, boolean>();
-
-function commandExists(command: string): boolean {
-	const cached = commandExistsCache.get(command);
-	if (cached !== undefined) return cached;
-
-	const result = spawnSync("sh", ["-c", "command -v \"$1\" >/dev/null 2>&1", "sh", command], {
-		stdio: "ignore",
-		timeout: CLIPBOARD_TIMEOUT_MS,
-		killSignal: "SIGKILL",
-	});
-	const exists = result.status === 0;
-	commandExistsCache.set(command, exists);
-	return exists;
-}
-
-function copyWith(command: string, args: string[], text: string): boolean {
-	const result = spawnSync(command, args, { input: text, encoding: "utf8", timeout: CLIPBOARD_TIMEOUT_MS, killSignal: "SIGKILL" });
-	return !result.error && result.status === 0;
-}
-
-function copyToClipboard(text: string): string | undefined {
-	const failedCommands: string[] = [];
-
-	for (const command of clipboardCommands) {
-		if (!command.enabled() || !commandExists(command.name)) continue;
-		if (copyWith(command.name, command.args, text)) return undefined;
-		failedCommands.push(command.name);
-	}
-
-	if (failedCommands.length > 0) {
-		return `Clipboard command${failedCommands.length === 1 ? "" : "s"} failed (${failedCommands.join(", ")})`;
-	}
-
-	return "No clipboard command found (tried pbcopy, termux-clipboard-set, wl-copy, xclip, xsel)";
-}
-
 function isToolMessage(message: CopyableMessage): boolean {
 	return message.role === "toolResult" || message.role === "bashExecution";
 }
@@ -353,7 +326,7 @@ function renderMessageLine(
 
 function renderPeekLines(message: CopyableMessage, width: number, theme: CopyMessageTheme, format: CopyFormat): string[] {
 	const contentWidth = Math.max(1, width - 2);
-	const text = formatMessageForCopy(message, format);
+	const text = safeDisplayText(formatMessageForCopy(message, format));
 	const wrapped = wrapTextWithAnsi(styleRoleText(theme, message.role, text, false), contentWidth);
 	const shown = wrapped.slice(0, MAX_PEEK_LINES);
 	const remaining = wrapped.length - shown.length;
@@ -440,8 +413,10 @@ export class CopyMessagePickerState {
 	format: CopyFormat;
 	peek = false;
 	private searchAnchorId: string | undefined;
+	private readonly messages: CopyableMessage[];
 
-	constructor(private readonly messages: CopyableMessage[], initialFormat: CopyFormat = "raw") {
+	constructor(messages: CopyableMessage[], initialFormat: CopyFormat = "raw") {
+		this.messages = messages;
 		this.format = initialFormat;
 		this.visibleMessages = filteredMessages(messages, this.visibility, this.search);
 		this.selectedIndex = Math.max(0, this.visibleMessages.length - 1);
@@ -496,6 +471,13 @@ export class CopyMessagePickerState {
 	}
 
 	handleInput(data: string, keybindings?: PickerKeybindings): PickerInputResult {
+		// Pi delivers each paste as one input wrapped in bracketed-paste markers.
+		if (data.startsWith("\x1b[200~") && data.endsWith("\x1b[201~")) {
+			const text = data.slice(6, -6).replace(/\s+/gu, " ");
+			if (!isPrintableSearchInput(text)) return "none";
+			this.setSearch(this.search + text);
+			return "render";
+		}
 		if (keybindings?.matches(data, "tui.select.up")) {
 			this.move(-1);
 			return "render";
@@ -545,7 +527,7 @@ export class CopyMessagePickerState {
 			return "render";
 		}
 		if (matchesKey(data, "backspace") || data === "\x7f") {
-			this.setSearch(this.search.slice(0, -1));
+			this.setSearch(splitGraphemes(this.search).slice(0, -1).join(""));
 			return "render";
 		}
 		const printable = decodeKittyPrintable(data) ?? data;
@@ -708,17 +690,16 @@ function copyNotificationText(selected: CopyableMessage): string {
 	return `Copied ${roleLabel(selected.role)} message: “${compactPreview(selected.text, 48)}”`;
 }
 
-function copySelectedMessage(ctx: Pick<ExtensionCommandContext, "ui">, selected: CopyableMessage, text = selected.text) {
-	const error = copyToClipboard(text);
-	if (error) {
-		ctx.ui.notify(error, "error");
-		return;
+async function copySelectedMessage(ctx: Pick<ExtensionCommandContext, "ui">, selected: CopyableMessage, text = selected.text) {
+	try {
+		await copyToClipboard(text);
+		ctx.ui.notify(copyNotificationText(selected), "info");
+	} catch (error) {
+		ctx.ui.notify(error instanceof Error ? error.message : "Failed to copy to clipboard", "error");
 	}
-
-	ctx.ui.notify(copyNotificationText(selected), "info");
 }
 
-function copyMostRecentUserMessage(ctx: Pick<ExtensionCommandContext, "sessionManager" | "ui">, format: CopyFormat) {
+async function copyMostRecentUserMessage(ctx: Pick<ExtensionCommandContext, "sessionManager" | "ui">, format: CopyFormat) {
 	const result = getMostRecentUserMessage(ctx);
 	if (result.kind === "no-user-message") {
 		ctx.ui.notify("No user messages found", "warning");
@@ -729,7 +710,7 @@ function copyMostRecentUserMessage(ctx: Pick<ExtensionCommandContext, "sessionMa
 		return;
 	}
 
-	copySelectedMessage(ctx, result.message, formatMessageForCopy(result.message, format));
+	await copySelectedMessage(ctx, result.message, formatMessageForCopy(result.message, format));
 }
 
 export default function copyMessageExtension(pi: Pick<ExtensionAPI, "registerCommand">) {
@@ -746,7 +727,7 @@ export default function copyMessageExtension(pi: Pick<ExtensionAPI, "registerCom
 
 			if (parsedArgs.selector === "latest") {
 				const latestVisible = latestDefaultMessage(messages);
-				if (latestVisible) copySelectedMessage(ctx, latestVisible, formatMessageForCopy(latestVisible, parsedArgs.format));
+				if (latestVisible) await copySelectedMessage(ctx, latestVisible, formatMessageForCopy(latestVisible, parsedArgs.format));
 				return;
 			}
 
@@ -756,7 +737,7 @@ export default function copyMessageExtension(pi: Pick<ExtensionAPI, "registerCom
 					ctx.ui.notify(`No default visible message #${parsedArgs.selector.number} (found ${defaultVisibleMessages(messages).length})`, "warning");
 					return;
 				}
-				copySelectedMessage(ctx, selected, formatMessageForCopy(selected, parsedArgs.format));
+				await copySelectedMessage(ctx, selected, formatMessageForCopy(selected, parsedArgs.format));
 				return;
 			}
 
@@ -768,7 +749,7 @@ export default function copyMessageExtension(pi: Pick<ExtensionAPI, "registerCom
 			const selected = await pickMessage(ctx, messages, parsedArgs.format);
 			if (!selected) return;
 
-			copySelectedMessage(ctx, selected.message, selected.text);
+			await copySelectedMessage(ctx, selected.message, selected.text);
 		},
 	});
 
@@ -776,7 +757,7 @@ export default function copyMessageExtension(pi: Pick<ExtensionAPI, "registerCom
 		description: "Copy the most recent user message to the clipboard",
 		getArgumentCompletions: (argumentPrefix) => copyArgumentCompletions(argumentPrefix, false),
 		handler: async (args, ctx) => {
-			copyMostRecentUserMessage(ctx, parseCopyArgs(args).format);
+			await copyMostRecentUserMessage(ctx, parseCopyArgs(args).format);
 		},
 	});
 }

package/package.json

@@ -1,12 +1,12 @@
 {
   "name": "pi-copy-message",
-  "version": "2.1.0",
+  "version": "3.0.0",
   "description": "Pi extension with a searchable TUI picker for copying raw session messages",
   "author": "Mitch Fultz (https://github.com/fitchmultz)",
   "license": "MIT",
   "type": "module",
   "engines": {
-    "node": ">=22.19.0"
+    "node": ">=24.15.0"
   },
   "keywords": [
     "pi-package",
@@ -32,10 +32,11 @@
   },
   "homepage": "https://github.com/fitchmultz/pi-copy-message#readme",
   "scripts": {
-    "test": "tsx tests/copy-message.test.ts",
+    "test": "node --test tests/*.test.ts",
     "typecheck": "tsc --noEmit",
-    "check": "npm test && npm run typecheck",
-    "prepublishOnly": "npm run check && npm pack --dry-run"
+    "check": "npm test && npm run typecheck && npm pack --dry-run",
+    "check:compat": "npm run check",
+    "prepublishOnly": "npm run check"
   },
   "files": [
     "extensions",
@@ -44,18 +45,17 @@
     "LICENSE"
   ],
   "devDependencies": {
-    "@earendil-works/pi-coding-agent": "0.84.0",
-    "@earendil-works/pi-tui": "0.84.0",
-    "@types/node": "^25.9.1",
-    "tsx": "^4.22.4",
-    "typescript": "^6.0.3"
+    "@earendil-works/pi-coding-agent": "0.87.1",
+    "@earendil-works/pi-tui": "0.87.1",
+    "@types/node": "24.19.0",
+    "typescript": "7.0.2"
   },
   "pi": {
     "extensions": [
       "./extensions"
     ]
   },
-  "packageManager": "npm@11.16.0",
+  "packageManager": "npm@12.1.0",
   "peerDependencies": {
     "@earendil-works/pi-coding-agent": "*",
     "@earendil-works/pi-tui": "*"