@oh-my-pi/pi-tui 18.4.1 → 18.4.2

package/CHANGELOG.md

@@ -2,6 +2,20 @@
 
 ## [Unreleased]
 
+## [18.4.2] - 2026-09-28
+
+### Changed
+
+- Optimized activity clock rendering by caching timestamp formatting strings
+- Reduced CPU overhead during animated terminal redraws, transcript updates, inline-image bookkeeping, and status-line layout.
+- Reduced per-keystroke CPU in `^` model-mention autocomplete and per-frame CPU while streaming (status line, live transcript blocks, tool cards, frame writes).
+
+### Fixed
+
+- Fixed the `@` completion popup swallowing Tab and cursor-movement keys while a narrowed filter matched nothing: with no candidate to accept they now fall through to their normal completion and cursor roles instead of being trapped ([#13046](https://github.com/can1357/oh-my-pi/pull/13046) by [@jchanghong023](https://github.com/jchanghong023)).
+- Fixed the Esc-Esc rewind and fullscreen `/copy` selectors getting stuck at the oldest turn of a long session's recent tail; stepping past it now loads the earlier history
+- Fixed the composer attachment band showing chip `#1` (and other prefix IDs) as still present when only `#10` remained in the prompt ([#13605](https://github.com/can1357/oh-my-pi/issues/13605))
+
 ## [18.4.1] - 2026-09-28
 
 ### Breaking Changes

package/dist/types/chrome/transcript-container.d.ts

@@ -96,6 +96,16 @@ export declare class TranscriptContainer extends Container {
      * quit is pure write volume. An already offered replay batch stays valid.
      */
     cancelReplay(): void;
+    /**
+     * Open one composed frame: every live-block measurement until this frame's
+     * {@link renderViewport} returns renders against `frame` and is taken once,
+     * so the retirement peek and the viewport share each block's render.
+     *
+     * Callers MUST call {@link renderViewport} with the same `frame` in the same
+     * synchronous composition, without mutating any transcript block in
+     * between: the shared rows are only as fresh as that first measurement.
+     */
+    beginFrame(frame: AnimationFrame): void;
     /**
      * Total rows the live, un-emitted tail occupies at `width`.
      *
@@ -107,7 +117,11 @@ export declare class TranscriptContainer extends Container {
     liveRowCount(width: number, limit?: number): number;
     /** Block spans of the last `renderViewport` output, in output coordinates. Empty when the tail is empty. */
     getLastViewportSpans(): readonly TranscriptViewportSpan[];
-    /** Render the live tail, constrained to the supplied transcript height. */
+    /**
+     * Render the live tail, constrained to the supplied transcript height.
+     * Closes the frame {@link beginFrame} opened; a different `frame` discards
+     * its measurements first.
+     */
     renderViewport(width: number, rows: number, frame: AnimationFrame): readonly string[];
     /** Offers stable-head emission or the shortest finalized prefix needed under pressure. */
     peekFinalizedBatch(width: number, capacity: number): HistoryBatch | undefined;

package/dist/types/overlays/model-browser.d.ts

@@ -37,6 +37,11 @@ export interface ResolvedModelRoleValue {
 }
 /** Host-provided preferences and model-role resolution for the browser. */
 export interface ModelBrowserSource extends ModelRoleLookup {
+    /**
+     * Changes whenever any preference this source reads or resolves against changes,
+     * except the storage-backed `mruOrder` and `modelPerf`. Keys derived-scope caches.
+     */
+    readonly revision: number;
     readonly defaultThinkingLevel: string;
     readonly modelProviderOrder: readonly string[];
     readonly knownRoleIds: readonly string[];
@@ -104,6 +109,16 @@ export interface SessionModelScope {
 }
 /** Build the session picker's current scope without creating an interactive browser. */
 export declare function buildSessionModelScope(settings: ModelBrowserSource, registry: ModelBrowserRegistry, scopedModels: ReadonlyArray<Model>): SessionModelScope;
+/**
+ * {@link buildSessionModelScope} for per-keystroke callers: returns the same
+ * scope until the source revision, MRU order, scoped models, or the registry's
+ * available models, catalog, or load error change.
+ */
+export declare class SessionModelScopeCache {
+    #private;
+    constructor(settings: ModelBrowserSource, registry: ModelBrowserRegistry);
+    get(scopedModels: ReadonlyArray<Model>): SessionModelScope;
+}
 /** User affinity used to order search matches within one relevance tier. */
 export interface SearchAffinity {
     /** `provider/id` (lowercased) → rank; configured-role models first, then MRU. */

package/dist/types/overlays/rewind-selector.d.ts

@@ -22,7 +22,8 @@
  * variants at a fork and jump between user turns elsewhere, `f` opens a
  * filter (typing narrows the current path to matching items; Esc leaves the
  * filter with the selection kept), Enter rewinds to the outlined item, A loads
- * earlier turns without changing selection, Esc cancels.
+ * earlier turns without changing selection (stepping above the oldest replayed
+ * turn loads them too), Esc cancels.
  */
 import type { AgentTool } from "@oh-my-pi/pi-agent-core";
 import { type Component, type TUI } from "../index.js";

package/dist/types/prompt/composer-attachments.d.ts

@@ -37,6 +37,13 @@ export declare function attachmentSgr(kind: ChipKind, n: number): string;
 export declare const PLACEHOLDER_REGEX: RegExp;
 /** Matches an expanded attachment marker, a compact attachment chip, or a skill chip. */
 export declare const COMPOSER_TOKEN_REGEX: RegExp;
+/**
+ * Attachment indices referenced by a composer buffer, per kind. Image/video count compact chips
+ * and expanded markers; paste counts compact chips only, since `[Paste #N]` markers number the
+ * base editor's separate paste buffer. Registered labels take precedence over theme glyphs so
+ * tokens created before a theme switch keep their kind; full numbers prevent `#1` matching `#10`.
+ */
+export declare function referencedAttachments(text: string, recorded: ReadonlyMap<string, ChipKind>): Record<ChipKind, Set<number>>;
 /** Add the registered model-chip labels to the composer placeholder matcher. */
 export declare function composerTokenRegex(mentionLabels: Iterable<string>): RegExp;
 /** Offsets image marker indices, including matching `attachment://` references. */

package/dist/types/prompt/model-mention-autocomplete.d.ts

@@ -17,7 +17,10 @@ export declare function applyModelMentionCompletion(lines: string[], cursorLine:
     cursorLine: number;
     cursorCol: number;
 };
-/** Create a fresh session-scoped model candidate lookup using picker ordering. */
+/**
+ * Create a session-scoped model candidate lookup using picker ordering. The
+ * scope and search affinity are reused across queries until their inputs change.
+ */
 export declare function createModelMentionSource(host: {
     source: ModelBrowserSource;
     registry: ModelBrowserRegistry;

package/dist/types/render/width-aware-text.d.ts

@@ -19,5 +19,11 @@ export declare class WidthAwareText implements Component {
     setCustomBgFn(customBgFn?: (text: string) => string): void;
     setIgnoreTight(ignore: boolean): this;
     invalidate(): void;
+    /**
+     * Re-run the formatter on the next render because its inputs changed.
+     * Unlike {@link invalidate}, the inner `Text` keeps its wrap cache, so a
+     * reformat that yields the same string skips re-wrapping and re-tinting.
+     */
+    reformat(): void;
     render(width: number): readonly string[];
 }

package/dist/types/terminal.d.ts

@@ -276,6 +276,13 @@ export declare class ProcessTerminal implements Terminal {
     stop(): void;
     write(data: string): void;
     get columns(): number;
+    /**
+     * With the output pump, a backlog bound at or below
+     * {@link STDOUT_BACKLOG_CLEAR_BYTES} is reported as-is instead of re-read:
+     * both consumers (the render gate and the stall watchdog) act only on a
+     * backlog above that level, so the bound already decides them and spares a
+     * native read on every frame.
+     */
     get pendingOutputBytes(): number;
     get hostOwnsGridOnResize(): boolean;
     get rows(): number;

package/dist/types/theme/color.d.ts

@@ -5,6 +5,7 @@ export declare const FG_RESET = "\u001B[39m";
 export declare const BG_RESET = "\u001B[49m";
 /** Resolve theme color depth from the shared terminal capability model. */
 export declare function detectColorMode(env?: NodeJS.ProcessEnv): ColorMode;
+/** Convert a theme color to foreground SGR at the requested depth; throws for invalid colors. */
 export declare function colorToAnsi(color: string, mode: ColorMode): string;
 export declare function fgAnsi(color: string | number, mode: ColorMode): string;
 export declare function bgAnsi(color: string | number, mode: ColorMode): string;

package/dist/types/theme/session-color.d.ts

@@ -1,11 +1,11 @@
 /** Theme-derived inputs for {@link getSessionAccentHex}; see `Theme.sessionAccentInputs`. */
 export interface SessionAccentTheme {
     /** Theme accent hex; the session accent adopts its OKLCH lightness and chroma. */
-    accentHex: string;
+    readonly accentHex: string;
     /** Major theme color hexes checked for hue collision. */
-    colorHexes: string[];
+    readonly colorHexes: readonly string[];
     /** WCAG luminance of the status-line surface on light themes; undefined on dark themes. */
-    surfaceLuminance?: number;
+    readonly surfaceLuminance?: number;
 }
 /**
  * Derive a stable CSS hex accent color from a session name and the active theme.

package/dist/types/theme/shimmer.d.ts

@@ -1,6 +1,6 @@
 import type { Theme, ThemeColor } from "./theme.js";
 type ShimmerTheme = Pick<Theme, "bold" | "fg" | "getFgAnsi">;
-/** Sweep style for animated shimmer text; `disabled` renders every tier as the low color. */
+/** Sweep style for animated shimmer text; `disabled` renders every tier as the mid color. */
 export type ShimmerMode = "classic" | "kitt" | "disabled";
 /** Select the shimmer sweep style. The host pushes its `display.shimmer` preference here. */
 export declare function setShimmerMode(mode: ShimmerMode): void;

package/dist/types/theme/theme-class.d.ts

@@ -49,7 +49,7 @@ export declare class Theme {
      * Theme-derived inputs for `getSessionAccentHex`: the accent hex whose
      * OKLCH weight session accents adopt, the major colors they must not
      * hue-collide with, and the light-theme surface luminance to contrast
-     * against.
+     * against. Built once per instance and frozen; callers share the object.
      */
     get sessionAccentInputs(): SessionAccentTheme;
     fg(color: ThemeColor, text: string): string;

package/dist/types/tui.d.ts

@@ -273,8 +273,9 @@ export declare class Container implements Component {
 /**
  * Merge runs of byte-adjacent SGR sequences (`CSI [0-9;:]* m`) into one. Only
  * CSI-SGR sequences are touched; text, cursor moves, OSC, hyperlinks and image
- * payloads pass through verbatim. Returns the original reference when nothing
- * merges, so SGR-light lines incur only a single `indexOf` scan.
+ * payloads pass through verbatim. Isolated sequences require no parameter
+ * slices or arrays; output is built only at merged boundaries or empty resets
+ * that need normalization within an adjacent run.
  */
 export declare function coalesceAdjacentSgr(line: string): string;
 /**

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-tui",
-	"version": "18.4.1",
+	"version": "18.4.2",
 	"description": "Terminal User Interface library with differential rendering for efficient text-based applications",
 	"homepage": "https://omp.sh",
 	"author": {
@@ -41,14 +41,14 @@
 		"gen:glyphs": "bun scripts/gen-glyphs.ts"
 	},
 	"dependencies": {
-		"@oh-my-pi/omptype": "18.4.1",
-		"@oh-my-pi/pi-agent-core": "18.4.1",
-		"@oh-my-pi/pi-ai": "18.4.1",
-		"@oh-my-pi/pi-catalog": "18.4.1",
-		"@oh-my-pi/pi-natives": "18.4.1",
-		"@oh-my-pi/pi-utils": "18.4.1",
-		"@oh-my-pi/pi-wire": "18.4.1",
-		"@oh-my-pi/snapcompact": "18.4.1"
+		"@oh-my-pi/omptype": "18.4.2",
+		"@oh-my-pi/pi-agent-core": "18.4.2",
+		"@oh-my-pi/pi-ai": "18.4.2",
+		"@oh-my-pi/pi-catalog": "18.4.2",
+		"@oh-my-pi/pi-natives": "18.4.2",
+		"@oh-my-pi/pi-utils": "18.4.2",
+		"@oh-my-pi/pi-wire": "18.4.2",
+		"@oh-my-pi/snapcompact": "18.4.2"
 	},
 	"devDependencies": {
 		"kitty-vt-wasm": "^0.2.0"

package/src/chat/tool-execution.ts

@@ -285,6 +285,14 @@ export class ToolExecutionComponent extends Container {
 	// so a terminal resize re-shapes image-bearing results to rescale them without
 	// forcing the common image-free result to re-shape on every resize tick.
 	#renderedImageCount = 0;
+	// `stateBgKey|themeEpoch` of the tint last handed to #contentText. Re-tinting
+	// drops its wrap cache, so a rebuild whose tint is unchanged skips it and the
+	// inner Text re-wraps only when the reformatted card text actually differs.
+	#contentTextBgKey: string | undefined;
+	// Memoized #getTextOutput(), keyed by every input it reads: the result
+	// (versioned by #resultVersion), #showImages, and the image protocol.
+	#textOutput = "";
+	#textOutputKey: string | undefined;
 	#tool?: AgentTool;
 	#renderer?: ToolRenderer;
 	#ui: ToolExecutionUi;
@@ -1200,8 +1208,7 @@ export class ToolExecutionComponent extends Container {
 			// Generic fallback (no custom/built-in renderer). WidthAwareText
 			// reformats at render time so output fills the actual terminal width
 			// instead of a fixed column cap.
-			this.#contentText.setCustomBgFn(stateBgFn);
-			this.#contentText.invalidate();
+			this.#refreshContentText(stateBgKey, stateBgFn);
 		}
 
 		// Handle images (same for both custom and built-in)
@@ -1339,6 +1346,8 @@ export class ToolExecutionComponent extends Container {
 
 	#getTextOutput(): string {
 		if (!this.#result) return "";
+		const key = `${this.#resultVersion}|${this.#showImages}|${TERMINAL.imageProtocol ?? "-"}`;
+		if (key === this.#textOutputKey) return this.#textOutput;
 
 		const textBlocks = this.#result.content.filter(c => c.type === "text");
 		const imageBlocks = this.#getAllImageBlocks();
@@ -1360,9 +1369,21 @@ export class ToolExecutionComponent extends Container {
 			output = output ? `${output}\n${imageIndicators}` : imageIndicators;
 		}
 
+		this.#textOutputKey = key;
+		this.#textOutput = output;
 		return output;
 	}
 
+	/** Re-tint (only when the tint changed) and reformat the generic #contentText card. */
+	#refreshContentText(stateBgKey: string, stateBgFn: (text: string) => string): void {
+		const bgKey = `${stateBgKey}|${getThemeEpoch()}`;
+		if (bgKey !== this.#contentTextBgKey) {
+			this.#contentTextBgKey = bgKey;
+			this.#contentText.setCustomBgFn(stateBgFn);
+		}
+		this.#contentText.reformat();
+	}
+
 	/**
 	 * Format the generic call/result card at `contentWidth`. Shared by the
 	 * #contentText fallback and the benign-skip path so both render identically.
@@ -1406,8 +1427,7 @@ export class ToolExecutionComponent extends Container {
 	 */
 	#renderBenignSkipCard(stateBgFn: (text: string) => string): void {
 		if (!this.#usesContentBox) {
-			this.#contentText.setCustomBgFn(stateBgFn);
-			this.#contentText.invalidate();
+			this.#refreshContentText("toolPendingBg", stateBgFn);
 			return;
 		}
 		for (const box of this.#multiFileBoxes) {

package/src/chrome/transcript-container.ts

@@ -174,6 +174,21 @@ export class TranscriptContainer extends Container {
 	#pinnedFrontier: { index: number; since: number; logged: boolean } | undefined;
 	/** Block spans of the last `renderViewport` output, for click hit-testing. */
 	#lastViewportSpans: TranscriptViewportSpan[] = [];
+	/**
+	 * The composed frame {@link beginFrame} opened; `undefined` outside one.
+	 * {@link renderViewport} closes it, so it never outlives the synchronous
+	 * composition that opened it.
+	 */
+	#openFrame: AnimationFrame | undefined;
+	/**
+	 * Full-allocation blank-trimmed renders of the blocks measured during the
+	 * open frame, keyed by entry at {@link #frameRowsWidth}. A retirement peek
+	 * and the viewport measure the same live blocks back to back inside one
+	 * composition, with no block mutation possible in between; replaying the
+	 * first measurement spares every block its second render per frame.
+	 */
+	#frameRows = new Map<TranscriptEntry, readonly string[]>();
+	#frameRowsWidth = 0;
 	override addChild(component: Component): void {
 		if (isToolActivityComponent(component)) component.setToolActivityVisible(this.#toolActivityVisible);
 		super.addChild(component);
@@ -300,6 +315,21 @@ export class TranscriptContainer extends Container {
 		this.#replayRequested = false;
 	}
 
+	/**
+	 * Open one composed frame: every live-block measurement until this frame's
+	 * {@link renderViewport} returns renders against `frame` and is taken once,
+	 * so the retirement peek and the viewport share each block's render.
+	 *
+	 * Callers MUST call {@link renderViewport} with the same `frame` in the same
+	 * synchronous composition, without mutating any transcript block in
+	 * between: the shared rows are only as fresh as that first measurement.
+	 */
+	beginFrame(frame: AnimationFrame): void {
+		this.#lastFrame = frame;
+		this.#frameRows.clear();
+		this.#openFrame = frame;
+	}
+
 	/**
 	 * Total rows the live, un-emitted tail occupies at `width`.
 	 *
@@ -322,8 +352,33 @@ export class TranscriptContainer extends Container {
 
 	/** One live block's un-emitted rows at `width`, rendered against its full-height allocation. */
 	#liveBlockRows(entry: TranscriptEntry, index: number, width: number): readonly string[] {
+		const rows = this.#measuredRows(entry, width);
+		const emitted = this.#projectedEmittedRowCount(entry, index, width);
+		return emitted === 0 ? rows : rows.slice(emitted);
+	}
+
+	/**
+	 * One block's blank-trimmed render at its full-height allocation. Inside an
+	 * open frame the first measurement of each block is replayed to later ones.
+	 */
+	#measuredRows(entry: TranscriptEntry, width: number): readonly string[] {
 		this.#setAllocation(entry.component, Number.MAX_SAFE_INTEGER, this.#lastFrame);
-		return this.#renderEntry(entry, width).slice(this.#projectedEmittedRowCount(entry, index, width));
+		if (this.#openFrame === undefined) return this.#renderEntry(entry, width);
+		if (this.#frameRowsWidth !== width) {
+			this.#frameRows.clear();
+			this.#frameRowsWidth = width;
+		}
+		let rows = this.#frameRows.get(entry);
+		if (rows === undefined) {
+			rows = this.#renderEntry(entry, width);
+			this.#frameRows.set(entry, rows);
+		}
+		return rows;
+	}
+
+	#closeFrame(): void {
+		this.#openFrame = undefined;
+		this.#frameRows.clear();
 	}
 
 	/** Block spans of the last `renderViewport` output, in output coordinates. Empty when the tail is empty. */
@@ -349,9 +404,22 @@ export class TranscriptContainer extends Container {
 		this.#lastViewportSpans = spans;
 	}
 
-	/** Render the live tail, constrained to the supplied transcript height. */
+	/**
+	 * Render the live tail, constrained to the supplied transcript height.
+	 * Closes the frame {@link beginFrame} opened; a different `frame` discards
+	 * its measurements first.
+	 */
 	renderViewport(width: number, rows: number, frame: AnimationFrame): readonly string[] {
+		if (frame !== this.#openFrame) this.#closeFrame();
 		this.#lastFrame = frame;
+		try {
+			return this.#composeViewport(width, rows, frame);
+		} finally {
+			this.#closeFrame();
+		}
+	}
+
+	#composeViewport(width: number, rows: number, frame: AnimationFrame): readonly string[] {
 		this.#syncEntries();
 		this.#settleFinalized();
 		const live = this.#liveEntries();
@@ -522,7 +590,7 @@ export class TranscriptContainer extends Container {
 		// Only a render publishes a block's stable rows, so the head renders
 		// before its progressive-append eligibility is read.
 		const head = this.#entries[this.#frontier];
-		if (head !== undefined) this.#liveBlockRows(head, this.#frontier, width);
+		if (head !== undefined) this.#measuredRows(head, width);
 		const appendHead =
 			policy === "pressure" &&
 			head?.mode === "appendOnly" &&
@@ -723,15 +791,10 @@ export class TranscriptContainer extends Container {
 			return this.#freezeStableRows(entry, rendered, "stable rows changed within a width epoch");
 		}
 		entry.stableRows = published;
-		// Slice only when the rendered rows actually changed: same length
-		// plus prefix-equality in both directions means byte-identical, so
-		// the stored array can be reused (callers only slice/read it).
-		const priorRows = entry.renderedStableByWidth.get(width);
-		if (
-			priorRows === undefined ||
-			priorRows.length !== stableRendered.length ||
-			!isRowPrefix(priorRows, stableRendered)
-		) {
+		// The prefix check above proves equal-length rows are byte-identical.
+		// Reuse our snapshot then; copy new rows so mutable renderer buffers
+		// cannot change the bytes checked on the next frame.
+		if (priorRender === undefined || priorRender.length !== stableRendered.length) {
 			entry.renderedStableByWidth.set(width, stableRendered.slice());
 		}
 		let perCount = entry.stableRowCountByWidth.get(width);
@@ -858,8 +921,7 @@ export class TranscriptContainer extends Container {
 		while (this.#frontier < this.#entries.length) {
 			const entry = this.#entries[this.#frontier]!;
 			if (entry.mode !== "appendOnly" || entry.state !== "settled") return;
-			this.#setAllocation(entry.component, Number.MAX_SAFE_INTEGER, this.#lastFrame);
-			const rendered = this.#renderEntry(entry, width);
+			const rendered = this.#measuredRows(entry, width);
 			if (entry.emitted !== entry.stableRows.length) return;
 			if (this.#renderStablePrefix(entry, entry.emitted, width).length !== rendered.length) return;
 			this.#retireEntry(entry);

package/src/components/editor.ts

@@ -580,6 +580,8 @@ export class Editor implements Component, Focusable {
 	#wrapCache = new Map<string, WrapEntry>();
 	#wrapCacheWidth = -1;
 	#wrapCacheEpoch = -1;
+	/** Last `#getPromptGutter` result, keyed by (gutter string, clamped gutter width). */
+	#promptGutterCache: { source: string; firstLine: string; continuation: string; width: number } | undefined;
 	#paddingXOverride: number | undefined;
 	#maxHeight?: number;
 	#scrollOffset: number = 0;
@@ -1075,11 +1077,16 @@ export class Editor implements Component, Focusable {
 		if (!gutter) return undefined;
 		const gutterWidth = this.#getPromptGutterWidth(width, paddingX);
 		if (gutterWidth === 0) return undefined;
-		return {
+		const cached = this.#promptGutterCache;
+		if (cached !== undefined && cached.source === gutter && cached.width === gutterWidth) return cached;
+		const next = {
+			source: gutter,
 			firstLine: sliceByColumn(gutter, 0, gutterWidth, true),
 			continuation: padding(gutterWidth),
 			width: gutterWidth,
 		};
+		this.#promptGutterCache = next;
+		return next;
 	}
 
 	#getContentWidth(width: number, paddingX: number): number {
@@ -1682,9 +1689,16 @@ export class Editor implements Component, Focusable {
 					kb.matchesCanonical(canonical, "tui.select.pageUp") ||
 					kb.matchesCanonical(canonical, "tui.select.pageDown")
 				) {
-					this.#autocompleteList.handleInput(data);
-					this.onAutocompleteUpdate?.();
-					return;
+					// An `@` popup whose narrowing filter matched nothing holds no candidate;
+					// let the key fall through instead of swallowing it.
+					if (!this.#autocompleteList.getSelectedItem()) {
+						this.#cancelAutocomplete();
+						this.onAutocompleteUpdate?.();
+					} else {
+						this.#autocompleteList.handleInput(data);
+						this.onAutocompleteUpdate?.();
+						return;
+					}
 				}
 
 				// If Tab was pressed, always apply the selection
@@ -1699,7 +1713,14 @@ export class Editor implements Component, Focusable {
 						this.#cancelAutocomplete();
 						return;
 					}
-					if (selected && this.#autocompleteProvider) {
+					if (!selected) {
+						// An `@` popup whose narrowing filter matched nothing stays open with no
+						// candidate (see #debouncedUpdateAutocomplete). Nothing to accept: cancel the
+						// popup and fall through so Tab keeps its normal completion role and a right
+						// arrow at end of line moves the cursor.
+						this.#cancelAutocomplete();
+						this.onAutocompleteUpdate?.();
+					} else if (this.#autocompleteProvider) {
 						const shouldChainAutocomplete =
 							this.#isSlashCommandNameAutocompleteSelection() || isDirectoryCompletionValue(selected.value);
 						const result = this.#autocompleteProvider.applyCompletion(
@@ -1725,7 +1746,8 @@ export class Editor implements Component, Focusable {
 							queueMicrotask(() => void this.#tryTriggerAutocomplete());
 						}
 					}
-					return;
+					// Only an accepted candidate consumes the key; an empty list falls through.
+					if (selected) return;
 				}
 
 				// If Enter was pressed on a submitted slash command (not an absolute-path

package/src/components/image.ts

@@ -85,7 +85,7 @@ interface SurfaceSplit {
 	 * id so a partial pass reproduces the on-screen live/text split without a
 	 * full, correctly-ordered walk.
 	 */
-	suppressedIds: Set<number>;
+	readonly suppressedIds: Set<number>;
 }
 
 function newSurfaceSplit(): SurfaceSplit {
@@ -97,7 +97,7 @@ function resetSurfaceSplit(split: SurfaceSplit): void {
 	split.onTerminal = 0;
 	split.planned = 0;
 	split.lastTotal = 0;
-	split.suppressedIds = new Set();
+	if (split.suppressedIds.size > 0) split.suppressedIds.clear();
 }
 
 let nextImageBudgetSeed = Math.floor(Math.random() * 0xffffff);
@@ -284,8 +284,8 @@ export class ImageBudget {
 	 */
 	beginPass(stable = false, altScreen = false): void {
 		this.#passIds.length = 0;
-		this.#passSuppression.clear();
-		this.#passIndex.clear();
+		if (this.#passSuppression.size > 0) this.#passSuppression.clear();
+		if (this.#passIndex.size > 0) this.#passIndex.clear();
 		this.#stablePass = stable;
 		this.#surface = altScreen ? "alt" : "screen";
 		this.#split = altScreen ? this.#altSplit : this.#screenSplit;
@@ -296,7 +296,7 @@ export class ImageBudget {
 		// first. Note that leaving alt mode is not the same as unstacking a
 		// fullscreen overlay: the flush must exclude one that is still stacked
 		// from the pass itself, which is that caller's job, not this line's.
-		if (!altScreen) this.#liveIds.alt.clear();
+		if (!altScreen && this.#liveIds.alt.size > 0) this.#liveIds.alt.clear();
 		this.#applyingReset = !stable && this.#cap > 0 && this.#split.planned > this.#split.onTerminal;
 	}
 
@@ -349,7 +349,10 @@ export class ImageBudget {
 		// [0, onTerminal) is what this surface currently shows as text. Partial
 		// passes replay this per id (see #stablePass) instead of re-deriving it
 		// from a reversed, tail-only walk.
-		split.suppressedIds = new Set(this.#passIds.slice(0, split.onTerminal));
+		const suppressedIds = split.suppressedIds;
+		if (suppressedIds.size > 0) suppressedIds.clear();
+		const suppressedCount = Math.min(total, split.onTerminal);
+		for (let i = 0; i < suppressedCount; i++) suppressedIds.add(this.#passIds[i]);
 		return retry;
 	}
 
@@ -362,7 +365,12 @@ export class ImageBudget {
 	 * the next pass on the *other* surface knows what it may not destroy.
 	 */
 	limitResidentImages(): void {
-		this.#liveIds[this.#surface] = new Set(this.#passIds.filter(id => this.#passShowsLive(id)));
+		const liveIds = this.#liveIds[this.#surface];
+		if (liveIds.size > 0) liveIds.clear();
+		for (let i = 0; i < this.#passIds.length; i++) {
+			const id = this.#passIds[i];
+			if (this.#passShowsLive(id)) liveIds.add(id);
+		}
 		const transmitted = this.#transmitted[this.#surface];
 		if (this.#cap <= 0 || transmitted.size <= this.#cap) return;
 		for (const id of transmitted) {