@oh-my-pi/pi-tui 15.5.15 → 15.6.0

package/CHANGELOG.md

@@ -2,6 +2,15 @@
 
 ## [Unreleased]
 
+## [15.6.0] - 2026-05-30
+### Added
+
+- Added autocomplete triggering for internal URL scheme tokens such as `local://` and `skill://` while typing in the editor
+
+### Fixed
+
+- Fixed streaming output staying invisible in Windows Terminal + WSL2 until the window was minimized + restored. The 15.5.14 WSL branch of `requiresNativeViewportProofForReplay` treated an unknown native viewport state as "scrolled into history" — but `ProcessTerminal.isNativeViewportAtBottom` can only return a real answer through `kernel32.dll` FFI, which a Linux user-space process inside WSL cannot load, so the probe was permanently `undefined`. Every row-inserting structural mutation (each new streaming token row above the bottom-anchored prompt) was therefore classified as `deferredMutation` and emitted zero bytes. Any geometry change (resize/minimize/restore) bypassed the gate via a different render intent, which is why the output became visible only on window resize. The WSL clause is removed; on platforms where the probe cannot answer, unknown is treated as at-bottom (the pre-15.5.14 behaviour) so the live render path runs again. Native Win32 keeps the conservative "assume scrolled when unknown" heuristic since `kernel32` FFI does succeed there and unknown means the probe transiently failed. ([#1534](https://github.com/can1357/oh-my-pi/issues/1534))
+
 ## [15.5.14] - 2026-05-29
 
 ### Added

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-tui",
-	"version": "15.5.15",
+	"version": "15.6.0",
 	"description": "Terminal User Interface library with differential rendering for efficient text-based applications",
 	"homepage": "https://omp.sh",
 	"author": "Can Boluk",
@@ -37,10 +37,10 @@
 		"fmt": "biome format --write ."
 	},
 	"dependencies": {
-		"@oh-my-pi/pi-natives": "15.5.15",
-		"@oh-my-pi/pi-utils": "15.5.15",
-		"lru-cache": "11.3.6",
-		"marked": "^18.0.3"
+		"@oh-my-pi/pi-natives": "15.6.0",
+		"@oh-my-pi/pi-utils": "15.6.0",
+		"lru-cache": "11.5.1",
+		"marked": "^18.0.4"
 	},
 	"devDependencies": {
 		"chalk": "^5.6.2",

package/src/components/editor.ts

@@ -1571,6 +1571,10 @@ export class Editor implements Component, Focusable {
 				else if (textBeforeCursor.match(/(?:^|[\s([{>]):[a-zA-Z0-9_+-]*$/)) {
 					this.#tryTriggerAutocomplete();
 				}
+				// Check if we're typing an internal URL scheme (e.g. local://, skill://)
+				else if (this.#textTriggersUrlAutocomplete(textBeforeCursor)) {
+					this.#tryTriggerAutocomplete();
+				}
 			}
 		} else {
 			this.#debouncedUpdateAutocomplete();
@@ -1762,6 +1766,10 @@ export class Editor implements Component, Focusable {
 			else if (textBeforeCursor.match(/#[^\s#]*$/)) {
 				this.#tryTriggerAutocomplete();
 			}
+			// internal URL scheme context (e.g. local://, skill://)
+			else if (this.#textTriggersUrlAutocomplete(textBeforeCursor)) {
+				this.#tryTriggerAutocomplete();
+			}
 		}
 	}
 
@@ -1913,6 +1921,8 @@ export class Editor implements Component, Focusable {
 				this.#tryTriggerAutocomplete();
 			} else if (textBeforeCursor.match(/#[^\s#]*$/)) {
 				this.#tryTriggerAutocomplete();
+			} else if (this.#textTriggersUrlAutocomplete(textBeforeCursor)) {
+				this.#tryTriggerAutocomplete();
 			}
 		}
 	}
@@ -2232,6 +2242,10 @@ export class Editor implements Component, Focusable {
 			else if (textBeforeCursor.match(/#[^\s#]*$/)) {
 				this.#tryTriggerAutocomplete();
 			}
+			// internal URL scheme context (e.g. local://, skill://)
+			else if (this.#textTriggersUrlAutocomplete(textBeforeCursor)) {
+				this.#tryTriggerAutocomplete();
+			}
 		}
 	}
 
@@ -2463,6 +2477,17 @@ export class Editor implements Component, Focusable {
 	}
 
 	// Autocomplete methods
+	/**
+	 * Whether the text ending at the cursor looks like a `scheme://` URL token.
+	 * Generic by design: any scheme triggers a suggestion fetch and the active
+	 * provider decides whether it has candidates (returning none is a no-op).
+	 * MUST stay in sync with the token grammar in coding-agent's
+	 * `internal-url-autocomplete.ts`.
+	 */
+	#textTriggersUrlAutocomplete(textBeforeCursor: string): boolean {
+		return /(?:^|[\s"'`(<=])[a-z][a-z0-9+.-]*:\/{1,2}[^\s"'`()<>]*$/i.test(textBeforeCursor);
+	}
+
 	async #tryTriggerAutocomplete(explicitTab: boolean = false): Promise<void> {
 		if (!this.#autocompleteProvider) return;
 		// Check if we should trigger file completion on Tab

package/src/tui.ts

@@ -154,15 +154,6 @@ function isMultiplexerSession(): boolean {
 	return Boolean(Bun.env.TMUX || Bun.env.STY || Bun.env.ZELLIJ);
 }
 
-function requiresNativeViewportProofForReplay(): boolean {
-	return (
-		process.platform === "win32" ||
-		(process.platform === "linux" &&
-			Boolean(Bun.env.WT_SESSION) &&
-			Boolean(Bun.env.WSL_DISTRO_NAME || Bun.env.WSL_INTEROP))
-	);
-}
-
 /**
  * Options for overlay positioning and sizing.
  * Values can be absolute numbers or percentage strings (e.g., "50%").
@@ -1315,10 +1306,28 @@ export class TUI extends Container {
 		const pureAppend = diff.appendedLines && diff.firstChanged === this.#previousLines.length;
 		const structuralMutation = newLines.length !== this.#previousLines.length || diff.firstChanged < prevViewportTop;
 		if (!pureAppend && structuralMutation && !isMultiplexerSession()) {
-			if (this.#nativeViewportIsScrolled(this.#readNativeViewportAtBottom())) {
+			const nativeViewportAtBottom = this.#readNativeViewportAtBottom();
+			if (this.#nativeViewportIsScrolled(nativeViewportAtBottom)) {
 				this.#markNativeScrollbackDirty();
 				return { kind: "deferredMutation" };
 			}
+			// Expanding a collapsed offscreen cell inserts rows before an unchanged
+			// suffix. A viewport-only repaint makes the live bottom look correct but
+			// leaves native scrollback holding the old collapsed rows; scrolling up then
+			// shows a splice of stale history and the new tail. Pure tail appends with an
+			// offscreen status/header tick are still handled by the append-tail path.
+			if (
+				contentGrew &&
+				diff.firstChanged < prevViewportTop &&
+				this.#canReplayNativeScrollbackAtCheckpoint(nativeViewportAtBottom, false)
+			) {
+				const appendedTailStart = diff.appendedLines ? this.#findAppendedTailStart(newLines) : newLines.length;
+				const tailAppendCount = newLines.length - appendedTailStart;
+				const addedCount = newLines.length - this.#previousLines.length;
+				if (addedCount > tailAppendCount) {
+					return { kind: "historyRebuild" };
+				}
+			}
 		}
 
 		// Height changes shift the visible window. Repaint when content didn't
@@ -1422,10 +1431,7 @@ export class TUI extends Container {
 	}
 
 	#nativeViewportIsScrolled(nativeViewportAtBottom: boolean | undefined): boolean {
-		return (
-			nativeViewportAtBottom === false ||
-			(nativeViewportAtBottom === undefined && requiresNativeViewportProofForReplay())
-		);
+		return nativeViewportAtBottom === false || (nativeViewportAtBottom === undefined && process.platform === "win32");
 	}
 
 	#nativeViewportIsAtBottom(nativeViewportAtBottom: boolean | undefined): boolean {
@@ -1438,7 +1444,7 @@ export class TUI extends Container {
 	): boolean {
 		return (
 			nativeViewportAtBottom === true ||
-			(nativeViewportAtBottom === undefined && (allowUnknownViewport || !requiresNativeViewportProofForReplay()))
+			(nativeViewportAtBottom === undefined && (allowUnknownViewport || process.platform !== "win32"))
 		);
 	}
 

package/src/utils.ts

@@ -85,6 +85,44 @@ export function padding(n: number): string {
 // Grapheme segmenter (shared instance)
 const segmenter = new Intl.Segmenter(undefined, { granularity: "grapheme" });
 
+const EXTENDED_PICTOGRAPHIC_REGEX = /\p{Extended_Pictographic}/u;
+
+// Matches CSI (`\x1b[…`) and OSC (`\x1b]…` terminated by BEL/ST) escape
+// sequences. Mirrors the standard ansi-regex coverage so visible-span
+// segmentation lines up with the native ANSI scanner.
+const ANSI_ESCAPE_REGEX =
+	/[\u001b\u009b][[\]()#;?]*(?:(?:(?:(?:;[-a-zA-Z\d/#&.:=?%@~_]+)*|[a-zA-Z\d]+(?:;[-a-zA-Z\d/#&.:=?%@~_]*)*)?\u0007)|(?:(?:\d{1,4}(?:;\d{0,4})*)?[\dA-PR-TZcf-nq-uy=><~]))/g;
+
+function pictographicSpanWidth(span: string): number {
+	let width = 0;
+	for (const { segment } of segmenter.segment(span)) {
+		width += EXTENDED_PICTOGRAPHIC_REGEX.test(segment) ? 2 : nativeVisibleWidth(segment, getDefaultTabWidth());
+	}
+	return width;
+}
+
+// Width fallback for strings that mix ANSI styling with ZWJ pictographic
+// emoji. `Intl.Segmenter` would split an escape sequence into individual
+// graphemes, so the native scanner (which only skips ANSI when handed the
+// complete sequence) double-counts the printable SGR bytes. Excise the ANSI
+// spans first — they contribute zero cells — and apply the pictographic
+// grapheme override only to the visible spans, then sum.
+function visibleWidthByGrapheme(str: string): number {
+	let width = 0;
+	let lastIndex = 0;
+	ANSI_ESCAPE_REGEX.lastIndex = 0;
+	for (let match = ANSI_ESCAPE_REGEX.exec(str); match !== null; match = ANSI_ESCAPE_REGEX.exec(str)) {
+		if (match.index > lastIndex) {
+			width += pictographicSpanWidth(str.slice(lastIndex, match.index));
+		}
+		lastIndex = ANSI_ESCAPE_REGEX.lastIndex;
+	}
+	if (lastIndex < str.length) {
+		width += lastIndex === 0 ? pictographicSpanWidth(str) : pictographicSpanWidth(str.slice(lastIndex));
+	}
+	return width;
+}
+
 /**
  * Get the shared grapheme segmenter instance.
  */
@@ -104,7 +142,7 @@ export function visibleWidthRaw(str: string): number {
 	for (let i = 0; i < str.length; i++) {
 		const code = str.charCodeAt(i);
 		if (code < 0x20 || code > 0x7e) {
-			return nativeVisibleWidth(str, getDefaultTabWidth());
+			return str.includes("\u200d") ? visibleWidthByGrapheme(str) : nativeVisibleWidth(str, getDefaultTabWidth());
 		}
 	}
 	return str.length;