@oh-my-pi/pi-tui 18.13.2 → 18.13.3

package/CHANGELOG.md

@@ -2,6 +2,29 @@
 
 ## [Unreleased]
 
+## [18.13.3] - 2026-10-28
+
+### Added
+
+- The resume picker renames the selected session in place: F2 opens a dialog prefilled with the current title, Enter commits, Esc cancels ([#17262](https://github.com/can1357/oh-my-pi/pull/17262) by [@Twilight0](https://github.com/Twilight0))
+
+### Changed
+
+- Updated Bun engine requirement to ≥1.4.4
+
+### Fixed
+
+- Restored WebP and JPEG images stay visible in Kitty scrollback after resuming a session ([#17220](https://github.com/can1357/oh-my-pi/issues/17220)).
+- Escape in an annotation note no longer discards it: the overlay closes and `getDraft()`/`restoreDraft()` let the host keep and reopen the note; **Discard notes and close** (confirmed by a second press) clears them instead, and code review's actions start on Paste so a stray Enter never sends the LLM review ([#17271](https://github.com/can1357/oh-my-pi/pull/17271) by [@H4vC](https://github.com/H4vC), based on [#17230](https://github.com/can1357/oh-my-pi/pull/17230) by [@Shadorain](https://github.com/Shadorain)).
+- Fixed terminal replies such as Tern's `tsp;e;{"ev":"ack",…}` showing up in the shell prompt after quitting omp; the exit drain now waits for the terminal to answer a final DA1 sentinel
+- Fixed the plan review sheet in Tern squeezing the "continue with" model tiers under a long plan, so the model name overlapped the tier buttons
+- `/usage` in Tern shows when each quota resets as a countdown (`resets in 2h30m`) instead of a clock time; hovering shows the date and time
+- Fixed extension `ui.select` dialogs treating unconfigured `j`/`k` keys as navigation, which could turn a default No into Yes when typing `ok` then Enter ([#17255](https://github.com/can1357/oh-my-pi/pull/17255) by [@SAKANA-MOONLIGHT](https://github.com/SAKANA-MOONLIGHT)).
+
+### Removed
+
+- Removed support for the TSP `t` (theme) verb, as surface theme synchronization is no longer required
+
 ## [18.13.2] - 2026-10-27
 
 ### Added
@@ -2475,17 +2498,4 @@
 - Fixed TUI shutdown placing the parent shell prompt one row below short rendered content instead of directly on the next line ([#1620](https://github.com/can1357/oh-my-pi/issues/1620)).
 - Stopped painting inline color swatches for 4-digit hex runs in Markdown rendering. The `#RGBA` CSS form collides with hashline `#TAG` snapshot tags (4 hex digits, e.g. `#6C5E`), which were sprouting spurious RGB swatches in prose and codespans. Only `#RGB`, `#RRGGBB`, and `#RRGGBBAA` qualify now.
 
-## [15.7.6] - 2026-06-01
-
-### Fixed
-
-- Fixed native Windows + Windows Terminal freezing the editor on the wrap keystroke, on `/plan`/`/resume`/model-switch/status-line toggles, and on any other offscreen structural mutation until the next prompt submit. The `15.7.5` `#1635` fix routed every viewport-saturating pure-append and structural mutation through `deferredMutation` (a literal no-op) whenever `isNativeViewportAtBottom()` returned `undefined` — which it always does under `WT_SESSION` because the kernel32 probe can't see WT host scrollback. The deferral was only ever meant for the *confirmed-scrolled* case; an unknown viewport now falls back to a non-destructive `viewportRepaint` instead, so the live UI keeps updating without emitting `\x1b[3J` and without yanking a possibly-scrolled reader. Confirmed-scrolled frames (probe returns `false`) still defer.
-- Removed the hard-coded 20-result cap on `@`-prefixed fuzzy file completion in `CombinedAutocompleteProvider.#getFuzzyFileSuggestions`. The dropdown now honors the existing `maxResults: 100` ceiling already configured for `fuzzyFind`, so projects with many files sharing a common stem (e.g. `@controller`, `@test`) surface all relevant matches instead of being silently truncated. ([#1652](https://github.com/can1357/oh-my-pi/issues/1652))
-
-## [15.7.5] - 2026-06-01
-
-### Fixed
-
-- Fixed native Windows + Windows Terminal scrollback being yanked to the top when a streaming response triggered a TUI full redraw. Under ConPTY the `kernel32` `GetConsoleScreenBufferInfo` probe answers about the pseudo-console (always at the buffer tail) and not about WT's host scrollback, so `isNativeViewportAtBottom()` falsely returned `true` while the user was scrolled up and the shrink-across-viewport branch issued a destructive `historyRebuild` (`\x1b[2J\x1b[H\x1b[3J`). The probe now short-circuits to `undefined` whenever `WT_SESSION` is set, letting the existing deferred-rebuild path keep streaming-time mutations non-destructive and reconcile native history at the next prompt-submit checkpoint. ([#1635](https://github.com/can1357/oh-my-pi/issues/1635))
-
-Older entries are archived in [packages/tui/CHANGELOG.md@3d45d8557cd7](https://github.com/can1357/oh-my-pi/blob/3d45d8557cd7493221b69c7ed3c6b6a2307f46e7/packages/tui/CHANGELOG.md).
+Older entries are archived in [packages/tui/CHANGELOG.md@3deb6751027b](https://github.com/can1357/oh-my-pi/blob/3deb6751027b11b10ac14b2a5c07303817276505/packages/tui/CHANGELOG.md).

package/dist/types/apps/session-picker.d.ts

@@ -4,6 +4,8 @@ export interface SessionPickerHost<T extends SessionSelectorEntry = SessionSelec
     loadPinnedIds?(): Promise<ReadonlySet<string>>;
     loadHistoryMatcher?(): SessionHistoryMatcher;
     deleteSession?(session: T): Promise<boolean>;
+    /** Rename the session and resolve with the title as stored, or null when it did not happen. */
+    renameSession?(session: T, title: string): Promise<string | null>;
     loadAllSessions?(): Promise<T[]>;
 }
 /** Presentation and capability controls for the standalone session picker. */
@@ -13,6 +15,8 @@ export interface SessionPickerOptions<T extends SessionSelectorEntry = SessionSe
     scopeLabel?: string | false;
     showCwd?: boolean;
     allowDelete?: boolean;
+    /** Rename the selected session; resolve with the title as stored, or null when it did not happen. */
+    onRename?: (session: T, title: string) => Promise<string | null>;
     allowGlobalScope?: boolean;
     historySearch?: boolean;
     pinnedIds?: ReadonlySet<string>;

package/dist/types/chat/assistant-message.d.ts

@@ -87,7 +87,7 @@ export declare class AssistantMessageComponent extends Container {
      */
     setExpanded(expanded: boolean): void;
     isTranscriptBlockFinalized(): boolean;
-    /** Whether a figure still waits for the raster of its final source; the transcript holds retirement meanwhile. */
+    /** Hold image and figure rendering until the completed block can enter immutable scrollback. */
     isTranscriptBlockPending(): boolean;
     render(width: number): readonly string[];
     /**

package/dist/types/chat/bash-execution.d.ts

@@ -37,6 +37,8 @@ export declare class BashExecutionComponent extends Container {
     appendOutput(chunk: string): void;
     /** Switch to PTY rendering and feed raw terminal bytes through the vterm replay. */
     appendPtyChunk(chunk: string): void;
+    /** Bytes a terminal would send for a key event, honoring the command's cursor-key mode; "" for key releases. */
+    encodePtyInput(data: string): string;
     setComplete(exitCode: number | undefined, cancelled: boolean, options?: {
         output?: string;
         truncation?: TruncationMeta;

package/dist/types/chat/display-preferences.d.ts

@@ -6,6 +6,8 @@ export interface ChatTranscriptDisplayPreferences {
     cacheMissMarker: boolean;
     showTokenUsage: boolean;
     showTurnTime: boolean;
+    /** Max rows a collapsed advisor card shows below its header; 0 = no cap. */
+    advisorCollapsedLines: number;
 }
 /** Current transcript display preferences. */
 export declare const chatTranscriptDisplayPreferences: ChatTranscriptDisplayPreferences;

package/dist/types/chat/tool-execution.d.ts

@@ -102,7 +102,7 @@ export declare class ToolExecutionComponent extends Container {
      * component-scoped so the TUI reuses every other root subtree (issue #4377).
      */
     tickSpinner(frame: number): void;
-    /** Whether the result's figure still waits for its raster; the transcript holds retirement meanwhile. */
+    /** Hold visible results until their figures and Kitty images are ready for immutable scrollback. */
     isTranscriptBlockPending(): boolean;
     /**
      * Whether this block is ready to retire as immutable history. Partial

package/dist/types/components/editor.d.ts

@@ -98,6 +98,8 @@ export declare class Editor implements Component, Focusable {
     describeDecorations: ((lines: readonly string[]) => readonly TspEditorDecoration[]) | undefined;
     /** TSP code language of the buffer (`python`, `bash`): the terminal highlights it in the mono face. */
     describeLanguage: (() => string | undefined) | undefined;
+    /** Label the TSP field to disable host-side editing while another input owner handles raw keys. */
+    nativeKeyPassthrough: boolean;
     /**
      * TSP chrome around the `editor` node (the prompt composer's chips, mode
      * chip, send/stop): the root role, the children with `input` placed where

package/dist/types/overlays/annotation-overlay.d.ts

@@ -1,7 +1,7 @@
 import { type Focusable, type TUI } from "../index.js";
 import { type KeybindingsManager } from "../app-keybindings.js";
 import { type Theme } from "../theme/theme.js";
-import type { CodeReviewAnnotation, CodeReviewOverlayResult, ReviewDiffFile, TextReviewAnnotation, TextReviewOverlayResult, TextReviewSource } from "./annotation-types.js";
+import type { AnnotationOverlayDraft, CodeReviewAnnotation, CodeReviewOverlayResult, ReviewDiffFile, TextReviewAnnotation, TextReviewOverlayResult, TextReviewSource } from "./annotation-types.js";
 import type { TspSpan } from "@oh-my-pi/pi-wire";
 import { type NativeNode, type NativeUiEvent } from "../native/node.js";
 export declare const CONTINUE_CODE_REVIEW_ACTION = "Continue with LLM review";
@@ -20,6 +20,8 @@ export interface CodeReviewOverlayCallbacks extends AnnotationOverlayCallbacks {
 export interface TextReviewOverlayCallbacks extends AnnotationOverlayCallbacks {
     onComplete(result: TextReviewOverlayResult | undefined): void;
 }
+/** Clears every note and closes; the host then has nothing to keep. Needs a second confirm. */
+export declare const DISCARD_ANNOTATIONS_ACTION = "Discard notes and close";
 /** Splits a text source into the overlay's 1-based annotatable lines. */
 export declare function splitTextLines(text: string): string[];
 /** The native sheet: full-pane glass, titled by the review kind with what is under review after it. */
@@ -38,6 +40,10 @@ export declare class AnnotationOverlay implements Focusable {
     setUseTerminalCursor(useTerminalCursor: boolean): void;
     invalidate(): void;
     dispose(): void;
+    /** Notes, the unfinished note, and any source edit, in the form {@link restoreDraft} replays. */
+    getDraft(): AnnotationOverlayDraft;
+    /** Replay a draft taken from this same source; anything anchored outside it is dropped. */
+    restoreDraft(draft: AnnotationOverlayDraft): void;
     getAnnotations(): CodeReviewAnnotation[];
     getTextAnnotations(): TextReviewAnnotation[];
     /** Repo-relative path of the diff file under the cursor, if this is a code review. */

package/dist/types/overlays/annotation-types.d.ts

@@ -84,3 +84,31 @@ export interface TextReviewOverlayResult {
     /** Source text after an external-editor edit; absent when the source was not edited. */
     editedText?: string;
 }
+export type AnnotationOverlayDraft = {
+    kind: "diff";
+    annotations: {
+        fileIndex: number;
+        sourceIndex: number;
+        annotation: CodeReviewAnnotation;
+    }[];
+    editor?: {
+        fileIndex: number;
+        sourceIndex: number;
+        scope: "file" | "line";
+        editingAnnotationIndex?: number;
+        text: string;
+    };
+} | {
+    kind: "text";
+    annotations: {
+        sourceIndex: number;
+        annotation: TextReviewAnnotation;
+    }[];
+    editedText?: string;
+    editor?: {
+        sourceIndex: number;
+        scope: "text" | "line";
+        editingAnnotationIndex?: number;
+        text: string;
+    };
+};

package/dist/types/overlays/plan-review-overlay.d.ts

@@ -74,6 +74,8 @@ export interface PlanReviewOverlayOptions {
     slider?: HookSelectorSlider;
     /** Display label for the external-editor key, surfaced in the footer help. */
     externalEditorLabel?: string;
+    /** The external editor's name ("Notepad", "nvim") for the native Edit button; `$EDITOR` when unset. */
+    externalEditorName?: string;
     /** Serializable annotations restored into this overlay instance. */
     annotationState?: PlanReviewAnnotationState;
 }

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

@@ -71,6 +71,7 @@ declare class SessionList<T extends SessionSelectorEntry> implements Component {
     onExit: () => void;
     onToggleScope?: () => void;
     onDeleteRequest?: (session: T) => void;
+    onRenameRequest?: (session: T) => void;
     /** Re-render hook for async list updates (fuzzy scan completion, history merge). */
     onRequestRender?: () => void;
     constructor(sessions: T[], showCwd?: boolean, historyMatcher?: SessionHistoryMatcher, getTerminalRows?: () => number, pinnedIds?: ReadonlySet<string>, currentSessionPath?: string | (() => string | undefined));
@@ -79,6 +80,8 @@ declare class SessionList<T extends SessionSelectorEntry> implements Component {
     /** Cancel pending async search work; idempotent, called on every picker exit path. */
     dispose(): void;
     removeSession(sessionPath: string): void;
+    /** Update a session's title in place and refresh the visible list, keeping the selection. */
+    renameSessionTitle(sessionPath: string, title: string): void;
     /** Resolve a list-local rendered-line index to a filtered-session index. */
     hitTestSession(line: number): number | undefined;
     /** Wheel notch: move the selection one step (clamped, no wrap). */
@@ -93,6 +96,8 @@ declare class SessionList<T extends SessionSelectorEntry> implements Component {
     resumeSelected(): void;
     /** Delete / Backspace on an empty query: ask the parent to confirm deleting the selected session. */
     requestDelete(): void;
+    /** F2: ask the parent to rename the selected session. */
+    requestRename(): void;
     /** Empty the search query (the empty state's "Clear search"). */
     clearSearch(): void;
     get selectedSession(): T | undefined;
@@ -120,6 +125,12 @@ declare class SessionList<T extends SessionSelectorEntry> implements Component {
 }
 export interface SessionSelectorOptions<T extends SessionSelectorEntry = SessionSelectorEntry> {
     onDelete?: (session: T) => Promise<boolean>;
+    /**
+     * Rename the selected session to a user-supplied title; resolve with the
+     * title as stored (the session canonicalizes it) or null when the rename
+     * did not happen.
+     */
+    onRename?: (session: T, title: string) => Promise<string | null>;
     historyMatcher?: SessionHistoryMatcher;
     /** Loads sessions across all projects for the all-projects scope toggle (Tab). */
     loadAllSessions?: () => Promise<T[]>;
@@ -188,8 +199,8 @@ export declare class SessionSelectorComponent<T extends SessionSelectorEntry = S
     /**
      * With the `picker` kind: the data-first session picker (§4.7). Otherwise
      * scope `tabs`, loading/error message, then the search `input` and session
-     * `list` — or the delete-confirmation dialog in their place, as on the ANSI
-     * path — and the key-hint footer.
+     * `list` — or the delete-confirmation / rename dialog in their place, as on
+     * the ANSI path — and the key-hint footer.
      */
     describe(cx: DescribeContext): NativeNode;
     /** Picker events (root keypath); fallback: scope tab → toggle scope, click on a session → resume it, exactly like a mouse click. */

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

@@ -114,29 +114,6 @@ export declare function stopThemeWatcher(): void;
  * Used by HTML export to generate CSS custom properties.
  */
 export declare function getResolvedThemeColors(themeName?: string): Promise<Record<string, string>>;
-/** One appearance variant of {@link NativeThemePalette}: token name → `#rrggbb`. */
-export type NativeThemeVariant = Record<string, string>;
-/** omp's resolved theme for a Tern Surface Protocol terminal (the `t` verb body minus `sf`). */
-export interface NativeThemePalette {
-    dark?: NativeThemeVariant;
-    light?: NativeThemeVariant;
-    name: {
-        dark?: string;
-        light?: string;
-    };
-}
-/**
- * Cheap identity of everything {@link getNativeThemePalette} depends on;
- * recompute the palette only when it changes.
- */
-export declare function getNativeThemePaletteKey(): string;
-/**
- * omp's theme for a native surface: with auto theme, the configured dark and
- * light themes; otherwise (or while the theme selector previews one) the
- * active theme under its own appearance only. Themes without a loadable
- * definition (in-memory instances) contribute nothing.
- */
-export declare function getNativeThemePalette(): NativeThemePalette;
 /**
  * Check if a theme is a "light" theme by analyzing its status-line background
  * luminance. Loads theme JSON synchronously (built-in or custom file on disk)

package/dist/types/tools/bash-interactive.d.ts

@@ -6,6 +6,12 @@ import type { Theme } from "../theme/theme.js";
 export interface BashInteractiveTerminalBackend {
     resize(columns: number, rows: number): void;
 }
+/**
+ * Translates a TUI key event into the bytes a terminal would send to a PTY
+ * child; key releases encode to "". Shared by the agent `pty` console and
+ * `!` commands (`BashExecutionComponent.encodePtyInput`).
+ */
+export declare function normalizeInputForPty(data: string, applicationCursorKeysMode: boolean): string;
 /** Interactive terminal overlay driven by an external PTY controller. */
 export declare class BashInteractiveOverlayComponent implements Component {
     #private;

package/dist/types/tui.d.ts

@@ -507,7 +507,10 @@ export declare class TUI extends Container {
      * never come.
      */
     addStartListener(listener: StartListener): () => void;
-    addInputListener(listener: InputListener): () => void;
+    /** Registers a key listener; `prepend` gives a foreground input owner first refusal over global shortcuts. */
+    addInputListener(listener: InputListener, options?: {
+        prepend?: boolean;
+    }): () => void;
     removeInputListener(listener: InputListener): void;
     stop(): void;
     /**

package/package.json

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

package/src/apps/session-picker.ts

@@ -11,6 +11,8 @@ export interface SessionPickerHost<T extends SessionSelectorEntry = SessionSelec
 	loadPinnedIds?(): Promise<ReadonlySet<string>>;
 	loadHistoryMatcher?(): SessionHistoryMatcher;
 	deleteSession?(session: T): Promise<boolean>;
+	/** Rename the session and resolve with the title as stored, or null when it did not happen. */
+	renameSession?(session: T, title: string): Promise<string | null>;
 	loadAllSessions?(): Promise<T[]>;
 }
 
@@ -21,6 +23,8 @@ export interface SessionPickerOptions<T extends SessionSelectorEntry = SessionSe
 	scopeLabel?: string | false;
 	showCwd?: boolean;
 	allowDelete?: boolean;
+	/** Rename the selected session; resolve with the title as stored, or null when it did not happen. */
+	onRename?: (session: T, title: string) => Promise<string | null>;
 	allowGlobalScope?: boolean;
 	historySearch?: boolean;
 	pinnedIds?: ReadonlySet<string>;
@@ -63,6 +67,7 @@ export async function selectSession<T extends SessionSelectorEntry>(
 				},
 				{
 					onDelete: options.allowDelete === false ? undefined : host.deleteSession,
+					onRename: options.onRename ?? host.renameSession,
 					historyMatcher,
 					loadAllSessions: options.allowGlobalScope === false ? undefined : host.loadAllSessions,
 					allSessions: options.allSessions,

package/src/chat/advisor-message.ts

@@ -2,6 +2,7 @@ import { type Component } from "../tui";
 import { Disclosure } from "../components/disclosure";
 import { visibleWidth } from "../utils";
 import type { AdvisorMessageDetails, AdvisorNote, AdvisorSeverity } from "./messages";
+import { chatTranscriptDisplayPreferences } from "./display-preferences";
 import { formatBadge, replaceTabs, type ToolUIColor, wrapTextWithAnsi } from "../render/render-utils";
 import { Ellipsis, truncateToWidth } from "../render";
 import type { Theme } from "../theme";
@@ -274,6 +275,7 @@ export function createAdvisorMessageCard(
 		body: () => new AdvisorNotes(notes, 0, uiTheme),
 		expanded: getExpanded(),
 		paddingX: 1,
+		maxCollapsedRows: chatTranscriptDisplayPreferences.advisorCollapsedLines || undefined,
 	});
 	// The tool-output toggle owns expansion state; synchronize the controlled
 	// disclosure from the callback on every render.

package/src/chat/assistant-message.ts

@@ -792,8 +792,9 @@ export class AssistantMessageComponent extends Container {
 		return this.#transcriptBlockFinalized;
 	}
 
-	/** Whether a figure still waits for the raster of its final source; the transcript holds retirement meanwhile. */
+	/** Hold image and figure rendering until the completed block can enter immutable scrollback. */
 	isTranscriptBlockPending(): boolean {
+		if (this.#showImages && this.#kittyConversionsAwaited.size > 0) return true;
 		for (const block of this.#figureBlocks.values()) if (block.pending) return true;
 		return false;
 	}
@@ -1615,6 +1616,7 @@ export class AssistantMessageComponent extends Container {
 				})
 				.catch(() => {
 					this.#kittyConversionsAwaited.delete(key);
+					this.#onImageUpdate?.();
 				});
 		}
 	}

package/src/chat/bash-execution.ts

@@ -16,6 +16,7 @@ import type { TruncationMeta } from "../tools/output-meta";
 import { resolveImageOptions } from "../render/render-utils";
 import { OutputPane } from "../render/output-pane";
 import { loadXtermTerminal, readTerminalRows, styleTerminalRow } from "../tools/terminal-output";
+import { normalizeInputForPty } from "../tools/bash-interactive";
 import { getSixelLineMask, isSixelPassthroughEnabled, sanitizeWithOptionalSixelPassthrough } from "../render/sixel";
 import {
 	buildExecutionFrame,
@@ -286,6 +287,11 @@ export class BashExecutionComponent extends Container {
 		this.#drainPtyQueue();
 	}
 
+	/** Bytes a terminal would send for a key event, honoring the command's cursor-key mode; "" for key releases. */
+	encodePtyInput(data: string): string {
+		return normalizeInputForPty(data, this.#ptyTerminal?.modes.applicationCursorKeysMode ?? false);
+	}
+
 	#drainPtyQueue(): void {
 		const terminal = this.#ptyTerminal;
 		if (!terminal || this.#ptyWriting) return;

package/src/chat/display-preferences.ts

@@ -6,6 +6,8 @@ export interface ChatTranscriptDisplayPreferences {
 	cacheMissMarker: boolean;
 	showTokenUsage: boolean;
 	showTurnTime: boolean;
+	/** Max rows a collapsed advisor card shows below its header; 0 = no cap. */
+	advisorCollapsedLines: number;
 }
 
 /** Current transcript display preferences. */
@@ -16,6 +18,7 @@ export const chatTranscriptDisplayPreferences: ChatTranscriptDisplayPreferences
 	cacheMissMarker: false,
 	showTokenUsage: false,
 	showTurnTime: false,
+	advisorCollapsedLines: 0,
 };
 
 /** Apply host display preferences without pulling settings into the renderer. */

package/src/chat/tool-execution.ts

@@ -711,14 +711,15 @@ export class ToolExecutionComponent extends Container {
 			// Convert async - catch errors from processing
 			convertImageToPngShared(image)
 				.then(converted => {
+					this.#kittyConversionsAwaited.delete(key);
 					this.#kittyConverted.set(key, converted);
 					this.#displayInputVersion++;
 					this.#updateDisplay();
 					this.#ui.requestRender();
 				})
 				.catch(() => {
-					// Ignore conversion failures - display will use original image format
 					this.#kittyConversionsAwaited.delete(key);
+					this.#ui.requestRender();
 				});
 		}
 	}
@@ -859,9 +860,12 @@ export class ToolExecutionComponent extends Container {
 		}
 	}
 
-	/** Whether the result's figure still waits for its raster; the transcript holds retirement meanwhile. */
+	/** Hold visible results until their figures and Kitty images are ready for immutable scrollback. */
 	isTranscriptBlockPending(): boolean {
-		return this.#toolActivityVisible && this.#figure?.pending === true;
+		return (
+			this.#toolActivityVisible &&
+			((this.#showImages && this.#kittyConversionsAwaited.size > 0) || this.#figure?.pending === true)
+		);
 	}
 
 	/**

package/src/components/editor.ts

@@ -776,6 +776,8 @@ export class Editor implements Component, Focusable {
 	describeDecorations: ((lines: readonly string[]) => readonly TspEditorDecoration[]) | undefined;
 	/** TSP code language of the buffer (`python`, `bash`): the terminal highlights it in the mono face. */
 	describeLanguage: (() => string | undefined) | undefined;
+	/** Label the TSP field to disable host-side editing while another input owner handles raw keys. */
+	nativeKeyPassthrough = false;
 	/**
 	 * TSP chrome around the `editor` node (the prompt composer's chips, mode
 	 * chip, send/stop): the root role, the children with `input` placed where
@@ -1894,7 +1896,7 @@ export class Editor implements Component, Focusable {
 			decor: decor.decor,
 			ghost: ghost ? plainText(ghost) : undefined,
 			placeholder: nativePlaceholder || undefined,
-			mode: vim ? VIM_MODE_LABELS[vim.mode] : undefined,
+			mode: this.nativeKeyPassthrough ? "PTY" : vim ? VIM_MODE_LABELS[vim.mode] : undefined,
 			lang: this.describeLanguage?.(),
 			maxLines: this.#maxHeight,
 		};

package/src/native/backend.ts

@@ -39,15 +39,7 @@ import {
 	type TspOp,
 } from "@oh-my-pi/pi-wire";
 import type { Terminal } from "../terminal";
-import {
-	bindTheme,
-	getCurrentThemeName,
-	getNativeThemePalette,
-	getNativeThemePaletteKey,
-	isLightTheme,
-	type NativeThemePalette,
-	setNativeSymbolPreset,
-} from "../theme/theme";
+import { getCurrentThemeName, isLightTheme, setNativeSymbolPreset } from "../theme/theme";
 import type { Component, OverlayOptions, RenderScheduler, RenderTimer } from "../tui";
 import { TspDocument } from "./apply";
 import { getNativeBlob, type NativeBlob } from "./blobs";
@@ -313,11 +305,6 @@ export class NativeBackend {
 	#sawResize = false;
 	#live = false;
 	#overlayNodes = new Map<Component, { key: string; node: NativeNode }>();
-	#unbindTheme: (() => void) | undefined;
-	#palette: NativeThemePalette | undefined;
-	#paletteKey: string | undefined;
-	/** Serialized palette last sent, to skip resends of an unchanged theme. */
-	#paletteSent: string | undefined;
 	/** Pending delivery owns bytes until sent or confirmed held; settled descriptions may already be released. */
 	#blobs = new Map<string, NativeBlob | "sent">();
 	/** `blobs` queries sent, oldest first: the terminal answers each once, in order. */
@@ -366,54 +353,11 @@ export class NativeBackend {
 		if (this.#live) return;
 		this.#live = true;
 		this.#useNerdSymbols(true);
-		this.#watchTheme();
 		this.#open(this.#inline);
 		setNativeRendering(true);
 		this.render();
 	}
 
-	/**
-	 * Any theme swap (switch, selector preview, auto light/dark, file reload)
-	 * schedules a render; `render()` resends the palette when it changed, so
-	 * bursts coalesce into one `t` and no frame is needed for it.
-	 */
-	#watchTheme(): void {
-		this.#unbindTheme?.();
-		let initial = true;
-		this.#unbindTheme = bindTheme(() => {
-			if (initial) return;
-			this.#host.requestRender();
-		});
-		initial = false;
-	}
-
-	/** The current palette, recomputed only when its inputs changed. */
-	#currentPalette(): NativeThemePalette {
-		const key = getNativeThemePaletteKey();
-		if (this.#palette === undefined || key !== this.#paletteKey) {
-			this.#paletteKey = key;
-			this.#palette = getNativeThemePalette();
-		}
-		return this.#palette;
-	}
-
-	/** Send the palette to `surface` (right after its `o`, before any `f`). */
-	#sendPalette(surface: Surface): void {
-		const palette = this.#currentPalette();
-		if (!palette.dark && !palette.light) return;
-		this.#paletteSent = JSON.stringify(palette);
-		this.#write("t", { sf: surface.id, ...palette });
-	}
-
-	/** Resend the palette to every open surface when the theme changed since it was sent. */
-	#refreshPalette(): void {
-		const key = this.#paletteKey;
-		const palette = this.#currentPalette();
-		if (key === this.#paletteKey || JSON.stringify(palette) === this.#paletteSent) return;
-		this.#sendPalette(this.#inline);
-		if (this.#screen) this.#sendPalette(this.#screen);
-	}
-
 	/**
 	 * Tern's font carries every Nerd Font glyph, and it never answers the Glyph
 	 * Protocol, so icons come from the nerd preset (process-local, not the
@@ -435,8 +379,6 @@ export class NativeBackend {
 		this.#close(this.#inline, keep);
 		this.#inline.doc?.close();
 		setNativeRendering(false);
-		this.#unbindTheme?.();
-		this.#unbindTheme = undefined;
 		this.#useNerdSymbols(false);
 		this.#clearStallTimer();
 		// Replies no longer reach the backend: forget the queries outright.
@@ -455,7 +397,6 @@ export class NativeBackend {
 		this.#applyHello(hello);
 		this.#live = true;
 		this.#useNerdSymbols(true);
-		this.#watchTheme();
 		this.#resetBlobs();
 		const surface = this.#inline;
 		surface.reconciler.detachLive();
@@ -466,7 +407,6 @@ export class NativeBackend {
 		surface.focus = null;
 		surface.dirty = false;
 		this.#write("o", { id: surface.id, mode: "inline", title: "omp", role: SESSION_ROLE, adopt: true });
-		this.#sendPalette(surface);
 		// After the `o`, as in `start()`.
 		setNativeRendering(true);
 		this.render();
@@ -508,7 +448,6 @@ export class NativeBackend {
 	/** Reconcile and send a frame, unless the target surface is out of credits. */
 	render(): void {
 		if (!this.#live) return;
-		this.#refreshPalette();
 		const overlays = this.#host.overlays();
 		let fullscreen = -1;
 		for (let i = overlays.length - 1; i >= 0; i--) {
@@ -608,7 +547,6 @@ export class NativeBackend {
 
 	#open(surface: Surface): void {
 		this.#write("o", { id: surface.id, mode: surface.mode, title: "omp", role: surface.role });
-		this.#sendPalette(surface);
 	}
 
 	#close(surface: Surface, keep: boolean): void {
@@ -776,7 +714,7 @@ export class NativeBackend {
 		}
 	}
 
-	#write(verb: "o" | "f" | "t" | "x" | "q", body: unknown): void {
+	#write(verb: "o" | "f" | "x" | "q", body: unknown): void {
 		this.#record("out", verb, undefined, body);
 		this.#host.terminal.write(encodeTspJson(verb, body, undefined, this.#limit));
 	}