@oh-my-pi/pi-tui 15.13.3 → 16.0.1

package/CHANGELOG.md

@@ -2,6 +2,16 @@
 
 ## [Unreleased]
 
+## [16.0.1] - 2026-06-15
+
+### Added
+
+- Added Zellij and WezTerm pane environment fallbacks for terminal-specific session continuation when no TTY path is available.
+
+### Fixed
+
+- Fixed slash command autocomplete acceptance replacing only a stale rendered prefix, which could leave fast-typed characters before `/skills:` completions and corrupt the submitted command ([#1745](https://github.com/can1357/oh-my-pi/issues/1745)).
+
 ## [15.13.1] - 2026-06-15
 
 ### Added
@@ -382,7 +392,7 @@
 
 ### Changed
 
-- Changed native-scrollback safety defaults to treat unknown POSIX, SSH, and multiplexer-shaped terminals as ED3-risk for passive rendering; checkpoint replay now requires a positive at-tail viewport proof instead of assuming prompt submit makes host scrollback safe.
+- Changed native-scrollback safety defaults to treat unknown POSIX, SSH, and multiplexer-shaped terminals as ED3-risk for passive rendering; checkpoint replay now requires a positive at-tail viewport proof instead of assuming prompt submit makes host scrollback safe ([#1799](https://github.com/can1357/oh-my-pi/issues/1799)).
 - Changed synchronized-output defaults to a conservative opt-in profile: DEC 2026 paint wrappers stay disabled for remote/multiplexer/VTE/unknown terminals unless explicitly forced, while the autowrap guards remain active.
 
 ### Fixed

package/package.json

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

package/src/autocomplete.ts

@@ -413,27 +413,39 @@ export class CombinedAutocompleteProvider implements AutocompleteProvider {
 		prefix: string,
 	): { lines: string[]; cursorLine: number; cursorCol: number } {
 		const currentLine = lines[cursorLine] || "";
-		const beforePrefix = currentLine.slice(0, cursorCol - prefix.length);
+		const textBeforeCursor = currentLine.slice(0, cursorCol);
 		const afterCursor = currentLine.slice(cursorCol);
 
-		// Check if we're completing a slash command (prefix starts with "/" but NOT a file path)
-		// Slash commands are at the start of the line and don't contain path separators after the first /
-		const isSlashCommand = prefix.startsWith("/") && beforePrefix.trim() === "" && !prefix.slice(1).includes("/");
-		if (isSlashCommand) {
-			// This is a command name completion
-			const newLine = `${beforePrefix}/${item.value} ${afterCursor}`;
-			const newLines = [...lines];
-			newLines[cursorLine] = newLine;
+		const slashStart = textBeforeCursor.indexOf("/");
+		const hasOnlyWhitespaceBeforeSlash = slashStart >= 0 && textBeforeCursor.slice(0, slashStart).trim() === "";
 
-			return {
-				lines: newLines,
-				cursorLine,
-				cursorCol: beforePrefix.length + item.value.length + 2, // +2 for "/" and space
-			};
+		// Slash command suggestions can be accepted before the debounced refresh
+		// catches up to newly typed characters. Replace the live command token,
+		// not only the prefix captured when the suggestion list was rendered.
+		if (prefix.startsWith("/") && hasOnlyWhitespaceBeforeSlash) {
+			const slashPrefix = textBeforeCursor.slice(slashStart);
+			if (!slashPrefix.includes(" ") && !slashPrefix.slice(1).includes("/")) {
+				const beforeSlash = currentLine.slice(0, slashStart);
+				const newLine = `${beforeSlash}/${item.value} ${afterCursor}`;
+				const newLines = [...lines];
+				newLines[cursorLine] = newLine;
+
+				return {
+					lines: newLines,
+					cursorLine,
+					cursorCol: beforeSlash.length + item.value.length + 2, // +2 for "/" and space
+				};
+			}
 		}
 
+		let beforePrefix = currentLine.slice(0, cursorCol - prefix.length);
+
 		// Check if we're completing a file attachment (prefix starts with "@")
 		if (prefix.startsWith("@")) {
+			const liveAtPrefix = this.#extractAtPrefix(textBeforeCursor);
+			if (liveAtPrefix) {
+				beforePrefix = currentLine.slice(0, cursorCol - liveAtPrefix.length);
+			}
 			// This is a file attachment completion
 			const newLine = `${beforePrefix + item.value} ${afterCursor}`;
 			const newLines = [...lines];
@@ -446,21 +458,10 @@ export class CombinedAutocompleteProvider implements AutocompleteProvider {
 			};
 		}
 
-		// Check if we're in a slash command context (beforePrefix contains "/command ")
-		const textBeforeCursor = currentLine.slice(0, cursorCol);
-		if (textBeforeCursor.includes("/") && textBeforeCursor.includes(" ")) {
-			// This is likely a command argument completion
-			const newLine = beforePrefix + item.value + afterCursor;
-			const newLines = [...lines];
-			newLines[cursorLine] = newLine;
-
-			return {
-				lines: newLines,
-				cursorLine,
-				cursorCol: beforePrefix.length + item.value.length,
-			};
-		}
-
+		// Slash command argument and plain file path completion both fall through
+		// to the path-completion tail below — `beforePrefix` already covers the
+		// rendered prefix, which preserves earlier arguments (e.g. accepting
+		// `package.json` for `/swarm run pac<Tab>` keeps the `run` token intact).
 		// For file paths, complete the path
 		const newLine = beforePrefix + item.value + afterCursor;
 		const newLines = [...lines];

package/src/ttyid.ts

@@ -51,15 +51,29 @@ export function getTerminalId(): string | null {
 
 	// Fallback to terminal-specific env vars
 	// Prefer inner multiplexers over host terminal emulators when stdin has no TTY path.
+	const zellijPane = process.env.ZELLIJ_PANE_ID;
+	if (zellijPane) {
+		// Session names are user-chosen (`zellij -s …`) and the id is used as a
+		// breadcrumb filename — normalize path separators like the TTY branch does.
+		const zellijSession = process.env.ZELLIJ_SESSION_NAME?.replace(/[\\/]/g, "-");
+		return zellijSession ? `zellij-${zellijSession}-${zellijPane}` : `zellij-${zellijPane}`;
+	}
+
 	const tmuxPane = process.env.TMUX_PANE;
 	if (tmuxPane) return `tmux-${tmuxPane}`;
 
 	const cmuxSurface = process.env.CMUX_SURFACE_ID;
 	if (cmuxSurface) return `cmux-${cmuxSurface}`;
 
+	// Kitty before WezTerm/others, matching terminal-capabilities.ts detection
+	// order. Inherited env makes either order wrong for some nesting; staying
+	// consistent with the capability detector keeps the two answers aligned.
 	const kittyId = process.env.KITTY_WINDOW_ID;
 	if (kittyId) return `kitty-${kittyId}`;
 
+	const weztermPane = process.env.WEZTERM_PANE;
+	if (weztermPane) return `wezterm-${weztermPane}`;
+
 	const terminalSessionId = process.env.TERM_SESSION_ID; // macOS Terminal.app
 	if (terminalSessionId) return `apple-${terminalSessionId}`;