@xynogen/pix-pretty 1.5.0 → 1.5.1

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@xynogen/pix-pretty",
-	"version": "1.5.0",
+	"version": "1.5.1",
 	"description": "Enhanced tool output rendering with syntax highlighting, file icons, tree views, FFF search, and paste chip formatting",
 	"type": "module",
 	"main": "src/index.ts",

package/src/paste-chips.test.ts

@@ -8,7 +8,7 @@
 
 import { describe, expect, it } from "bun:test";
 import { visibleWidth } from "@earendil-works/pi-tui";
-import { restyleMarkers } from "./paste-chips";
+import { endsWithMarker, restyleMarkers } from "./paste-chips";
 
 const stripAnsi = (s: string) => s.replace(/\x1b\[[0-9;]*m/g, "");
 
@@ -132,3 +132,31 @@ describe("paste-chips width safety", () => {
 		});
 	});
 });
+
+// ─── endsWithMarker (trailing-space gate) ──────────────────────────────
+
+describe("paste-chips endsWithMarker", () => {
+	it("matches a chars marker at end of string", () => {
+		expect(endsWithMarker("[paste #1 2232 chars]")).toBe(true);
+	});
+
+	it("matches a lines marker at end of string", () => {
+		expect(endsWithMarker("hello [paste #2 +42 lines]")).toBe(true);
+	});
+
+	it("matches a bare marker at end of string", () => {
+		expect(endsWithMarker("[paste #3]")).toBe(true);
+	});
+
+	it("does not match when marker is not at the end", () => {
+		expect(endsWithMarker("[paste #1 58 chars] trailing")).toBe(false);
+	});
+
+	it("does not match plain text", () => {
+		expect(endsWithMarker("just some text")).toBe(false);
+	});
+
+	it("does not match a lone trailing space", () => {
+		expect(endsWithMarker(" ")).toBe(false);
+	});
+});

package/src/paste-chips.ts

@@ -134,6 +134,11 @@ function chip(
 	return `${color}${BOLD}${icon} ${label}${RST}${FG_DIM} ${meta}${RST}`;
 }
 
+/** True when `text` ends with a Pi paste marker (chip), e.g. `[paste #1 58 chars]`. */
+export function endsWithMarker(text: string): boolean {
+	return /\[paste #\d+[^\]]*\]$/.test(text);
+}
+
 function compactNumber(raw: string): string {
 	const n = Number.parseInt(raw, 10);
 	if (!Number.isFinite(n)) return raw;
@@ -144,16 +149,42 @@ function compactNumber(raw: string): string {
 
 // ─── Custom editor ────────────────────────────────────────────────────────────
 
+// handlePaste is TS-private in CustomEditor but runtime-public JS. We patch it
+// per-instance, capturing the base implementation, so large text pastes get the
+// same trailing-space treatment as image chips.
+type PasteHandler = { handlePaste(text: string): void };
+
 class ChipEditor extends CustomEditor {
 	private readonly imageIds = new Set<number>();
 
+	constructor(...args: ConstructorParameters<typeof CustomEditor>) {
+		super(...args);
+		this.patchHandlePaste();
+	}
+
 	override insertTextAtCursor(text: string): void {
 		const internals = this as unknown as EditorInternals;
 		const replaced = replaceImagePaths(text, internals, this.imageIds);
 		// Append a trailing space when the insertion ends with a paste marker so
 		// the cursor lands after the chip rather than inside it.
-		const needsSpace = /\[paste #\d+[^\]]*\]$/.test(replaced);
-		super.insertTextAtCursor(needsSpace ? `${replaced} ` : replaced);
+		super.insertTextAtCursor(
+			endsWithMarker(replaced) ? `${replaced} ` : replaced,
+		);
+	}
+
+	private patchHandlePaste(): void {
+		const self = this as unknown as PasteHandler;
+		const base = self.handlePaste.bind(self);
+		self.handlePaste = (pastedText: string) => {
+			const before = this.getText();
+			base(pastedText);
+			const after = this.getText();
+			// Only nudge a space when the paste collapsed into a marker chip;
+			// inline small pastes (no marker) are left untouched.
+			if (endsWithMarker(after) && after !== before) {
+				super.insertTextAtCursor(" ");
+			}
+		};
 	}
 
 	override render(width: number): string[] {