@linxiraos/pi-tui 1.2.1 → 1.3.1

package/src/theme/theme.ts

@@ -5,7 +5,7 @@ import type { Terminal, TerminalAppearance } from "../terminal";
 import { colorLuma } from "@linxiraos/pi-utils/color";
 import { getCustomThemesDir } from "@linxiraos/pi-utils/dirs";
 import * as logger from "@linxiraos/pi-utils/logger";
-import { terminalMultiplexerSessions } from "../terminal-multiplexer";
+import { isTerminalBackgroundQueryTrusted } from "../terminal-multiplexer";
 import { setActiveSymbolTheme } from "./active-symbols";
 import { ansi256ToHex, resolveThemeColors, resolveVarRefs } from "./color";
 import {
@@ -50,18 +50,28 @@ var terminalReportedAppearance: "dark" | "light" | undefined;
 /** Appearance reported by the macOS fallback observer, or undefined if not yet available. */
 var macOSReportedAppearance: "dark" | "light" | undefined;
 
-function shouldUseMacOSAppearanceFallback(): boolean {
-	// Some multiplexers break OSC 11 passthrough on some platforms (Zellij on
-	// macOS), so terminal-derived appearance cannot be trusted there. Fall back
-	// to host macOS appearance without letting it override valid terminal
-	// signals elsewhere.
-	const platform = process.platform;
-	return terminalMultiplexerSessions().some(multiplexer => multiplexer.backgroundQueryBrokenOn?.includes(platform));
+const backgroundFallbackListeners = new Set<() => void>();
+
+/**
+ * Subscribe to native fallback appearance changes independently of theme changes.
+ * Consumers must re-check the effective background: COLORFGBG can override it.
+ */
+export function onTerminalBackgroundFallbackChange(callback: () => void): () => void {
+	backgroundFallbackListeners.add(callback);
+	return () => {
+		backgroundFallbackListeners.delete(callback);
+	};
 }
 
-function detectTerminalBackground(): "dark" | "light" {
+/**
+ * Detect appearance from trusted OSC 11 reports, then COLORFGBG, then native
+ * macOS appearance only on known-broken multiplexer background-query paths.
+ * Defaults to dark when none of those signals is available.
+ */
+export function detectTerminalBackground(): "dark" | "light" {
+	const backgroundQueryTrusted = isTerminalBackgroundQueryTrusted();
 	// Tier 1: terminal-reported appearance from OSC 11 luminance.
-	if (!shouldUseMacOSAppearanceFallback() && terminalReportedAppearance) {
+	if (backgroundQueryTrusted && terminalReportedAppearance) {
 		return terminalReportedAppearance;
 	}
 
@@ -76,7 +86,7 @@ function detectTerminalBackground(): "dark" | "light" {
 	}
 
 	// Tier 3: host macOS appearance for known-broken terminal paths only.
-	if (shouldUseMacOSAppearanceFallback()) {
+	if (!backgroundQueryTrusted) {
 		const macAppearance = macOSReportedAppearance ?? detectMacOSAppearance();
 		if (macAppearance) return macAppearance;
 	}
@@ -214,14 +224,14 @@ export async function initTheme(
 		assignTheme(await loadTheme(name, getCurrentThemeOptions()));
 		if (enableWatcher) {
 			await startThemeWatcher();
-			startSigwinchListener();
 		}
 	} catch (err) {
 		logger.debug("Theme loading failed, falling back to dark theme", { error: String(err) });
 		currentThemeName = "dark";
 		assignTheme(await loadTheme("dark", getCurrentThemeOptions()));
-		// Don't start watcher for fallback theme
+		// Skip file watching for the fallback theme, not appearance tracking.
 	}
+	if (enableWatcher) startSigwinchListener();
 }
 
 export async function setTheme(
@@ -239,6 +249,14 @@ export async function setTheme(
 		assignTheme(loadedTheme);
 		if (enableWatcher) {
 			await startThemeWatcher();
+			// startThemeWatcher() calls stopThemeWatcher(), which tears down
+			// the SIGWINCH handler and the macOS appearance observer that
+			// feeds the SIXEL background fallback. Restart it like initTheme()
+			// does so a runtime theme change does not silence fallback
+			// background repaints. Without this, `setTheme(name, true)` after
+			// startup would leave the SIXEL repaint path dead until the next
+			// process restart.
+			startSigwinchListener();
 		}
 		notifyThemeChange();
 		return { success: true };
@@ -667,12 +685,26 @@ var macObserver: { stop(): void } | undefined;
 
 function startMacAppearanceObserver(): void {
 	stopMacAppearanceObserver();
-	if (!shouldUseMacOSAppearanceFallback()) return;
+	if (isTerminalBackgroundQueryTrusted()) return;
 	try {
 		macOSReportedAppearance = detectMacOSAppearance() ?? undefined;
 		macObserver = MacAppearanceObserver.start((err, appearance) => {
 			if (!err && (appearance === "dark" || appearance === "light")) {
+				const changed = appearance !== macOSReportedAppearance;
 				macOSReportedAppearance = appearance;
+				if (changed) {
+					// oxlint-disable-next-line unicorn/no-useless-spread -- listeners may unsubscribe while reporting
+					for (const listener of [...backgroundFallbackListeners]) {
+						try {
+							listener();
+						} catch (err) {
+							// Debug-level: a listener failure must not break sibling
+							// listeners or the appearance observer. Visible only in
+							// local debug logs, never in normal runs.
+							logger.debug("macOS appearance fallback listener failed", { err });
+						}
+					}
+				}
 				reevaluateAutoThemeForAppearance("macOS fallback");
 			}
 		});

package/src/thinking.d.ts

@@ -37,6 +37,11 @@ export declare function shouldDisableReasoning(level: ThinkingLevel | undefined)
  * Resolves a selector against the current model while preserving explicit "off".
  */
 export declare function resolveThinkingLevelForModel(model: Model | undefined, level: ThinkingLevel | undefined): ResolvedThinkingLevel | undefined;
+/**
+ * Level applied to `model` when the user has not picked one for it (carried,
+ * global default, auto). Unverified thinking stays unset until the user picks.
+ */
+export declare function automaticThinkingLevel<T>(model: Model | undefined, fallback: T): T | undefined;
 /**
  * Sentinel selector for the coding-agent "auto" thinking mode. Kept entirely
  * inside the coding-agent layer: it is never an {@link Effort} or

package/src/thinking.ts

@@ -132,6 +132,14 @@ export function resolveThinkingLevelForModel(
 	return clampThinkingLevelForModel(model, level);
 }
 
+/**
+ * Level applied to `model` when the user has not picked one for it (carried,
+ * global default, auto). Unverified thinking stays unset until the user picks.
+ */
+export function automaticThinkingLevel<T>(model: Model | undefined, fallback: T): T | undefined {
+	return model?.thinking?.unverified ? undefined : fallback;
+}
+
 /**
  * Sentinel selector for the coding-agent "auto" thinking mode. Kept entirely
  * inside the coding-agent layer: it is never an {@link Effort} or
@@ -374,7 +382,7 @@ export function modelSupportsEffortCeiling(model: Model, ceiling: Effort): boole
  * models.
  */
 export function resolveProvisionalAutoLevel(model: Model | undefined): Effort | undefined {
-	if (!model?.reasoning) return undefined;
+	if (!model?.reasoning || model.thinking?.unverified) return undefined;
 	const preferred = model.thinking?.defaultLevel ?? Effort.High;
 	return clampAutoThinkingEffort(model, preferred === Effort.Max ? Effort.XHigh : preferred, Effort.XHigh);
 }

package/src/tmux.d.ts

@@ -1,6 +1,14 @@
 export { wrapTmuxPassthrough } from "./multiplexers/tmux";
 /** Pass a control sequence through tmux, leaving direct-terminal output unchanged. */
 export declare function wrapTmuxPassthroughIfNeeded(payload: string, env?: NodeJS.ProcessEnv): string;
+/**
+ * Emulator name candidates from a `#{client_termtype}<TAB>#{client_termname}`
+ * line, best first: the terminal-type reply (tmux 3.4+, XTVERSION), then the
+ * emulator the client's `TERM` names when it ships its own terminfo entry. Both
+ * are kept because the reply can name an unrecognized program (a nested tmux
+ * answers `tmux 3.4`) while `TERM` still identifies the emulator.
+ */
+export declare function tmuxClientProgramsFromReport(report: string): string[];
 /**
  * Whether the terminal hosting this tmux session draws TSP programs natively:
  * it sets the session user option `@tern-tsp` while it watches the session.
@@ -12,10 +20,37 @@ export declare function tmuxSessionSpeaksTsp(env?: NodeJS.ProcessEnv): boolean;
  */
 export declare function setTmuxPaneApcTitles(on: boolean, env?: NodeJS.ProcessEnv): void;
 /**
- * Resolve the terminal emulator name recorded for this tmux client.
+ * Resolve the terminal emulator names recorded for this tmux client, best first.
  *
  * tmux overwrites pane identity variables with its own values, but retains the
- * attached client's terminal-type reply in `#{client_termtype}`. The local IPC
- * query runs once per process and degrades to `null` when unavailable.
+ * attached client's terminal-type reply in `#{client_termtype}` and its `TERM`
+ * in `#{client_termname}` (see {@link tmuxClientProgramsFromReport}). The local
+ * IPC query runs once per process and degrades to `[]` when unavailable.
+ */
+export declare function resolveTmuxClientTerminalNames(env?: NodeJS.ProcessEnv): readonly string[];
+/**
+ * Whether tmux draws SIXEL to the client attached to this session: its
+ * effective terminal description for that client carries `Sxl`.
+ *
+ * A tmux built with SIXEL answers every pane's DA1 with attribute 4 and draws a
+ * `SIXEL IMAGE (WxH)` text placeholder for clients without SIXEL, so the pane's
+ * DA1 says nothing about the outer terminal. tmux itself renders SIXEL only
+ * when the client's terminal has `Sxl`, whether from terminfo,
+ * `terminal-overrides`, or the `sixel` feature (DA1 or `terminal-features`),
+ * and `show-messages -T` reports that merged description. The TUI consults
+ * this before trusting the pane's DA1 attribute. Queried once per tmux
+ * session; `false` outside tmux, with no attached client, or when the query
+ * fails. Nested tmux usually reports no `Sxl`: the inner client's terminal is
+ * the outer tmux, whose DA1 starts with level 1, which tmux does not parse
+ * for features.
+ */
+export declare function tmuxClientRendersSixel(env?: NodeJS.ProcessEnv): boolean;
+/**
+ * Whether tmux forwards this pane's `\x1bPtmux;…\x1b\\` passthrough envelopes
+ * to the attached client. tmux 3.3+ defaults `allow-passthrough` to `off` and
+ * then silently drops every wrapped sequence, so Kitty graphics sent through
+ * the envelope never reach a Kitty-capable outer terminal. Resolves the
+ * effective pane value (pane → window → global). Queried once per pane;
+ * `false` outside tmux or when the query fails.
  */
-export declare function resolveTmuxClientTerminalName(env?: NodeJS.ProcessEnv): string | null;
+export declare function tmuxPaneAllowsPassthrough(env?: NodeJS.ProcessEnv): boolean;

package/src/tmux.ts

@@ -10,24 +10,55 @@ export function wrapTmuxPassthroughIfNeeded(payload: string, env: NodeJS.Process
 }
 
 const CLIENT_TERMTYPE_NAME = /^([A-Za-z][A-Za-z0-9._+-]*)(?=\s|\(|$)/u;
-const CLIENT_TERMTYPE_TIMEOUT_MS = 500;
-let cachedClientTerminalName: string | null | undefined;
+// One `tmux display-message` round trip over the server socket, which answers in
+// single-digit milliseconds when healthy. 500ms kept losing the race on a loaded
+// machine (fork/exec of the tmux client alone can exceed it on a saturated box),
+// and a lost race is indistinguishable from "not tmux": the process then falls
+// back to the true-color profile for the whole session. 2s bounds the startup
+// stall only when tmux is genuinely wedged.
+const CLIENT_TERMINAL_TIMEOUT_MS = 2_000;
+/** Client `TERM` values that name their emulator; tmux records this on every version. */
+const CLIENT_TERMNAME_PROGRAMS: ReadonlyArray<readonly [RegExp, string]> = [
+	[/^xterm-kitty$/u, "kitty"],
+	[/^xterm-ghostty$/u, "ghostty"],
+	[/^wezterm$/u, "wezterm"],
+];
+let cachedClientTerminalNames: readonly string[] | undefined;
 
-function queryTmuxClientTerminalName(env: NodeJS.ProcessEnv): string | null {
+/**
+ * Emulator name candidates from a `#{client_termtype}<TAB>#{client_termname}`
+ * line, best first: the terminal-type reply (tmux 3.4+, XTVERSION), then the
+ * emulator the client's `TERM` names when it ships its own terminfo entry. Both
+ * are kept because the reply can name an unrecognized program (a nested tmux
+ * answers `tmux 3.4`) while `TERM` still identifies the emulator.
+ */
+export function tmuxClientProgramsFromReport(report: string): string[] {
+	// Split before trimming: an empty terminal type leaves the line starting with the tab.
+	const [termtype = "", termname = ""] = report.split("\t");
+	const programs: string[] = [];
+	const program = CLIENT_TERMTYPE_NAME.exec(termtype.trim())?.[1];
+	if (program) programs.push(program);
+	const term = termname.trim().toLowerCase();
+	const termProgram = CLIENT_TERMNAME_PROGRAMS.find(([pattern]) => pattern.test(term))?.[1];
+	if (termProgram) programs.push(termProgram);
+	return programs;
+}
+
+function queryTmuxClientTerminalNames(env: NodeJS.ProcessEnv): readonly string[] {
 	const tmux = $which("tmux", { PATH: env.PATH });
-	if (!tmux) return null;
+	if (!tmux) return [];
 	try {
-		const result = Bun.spawnSync([tmux, "display-message", "-p", "#{client_termtype}"], {
+		const result = Bun.spawnSync([tmux, "display-message", "-p", "#{client_termtype}\t#{client_termname}"], {
 			env,
 			stdout: "pipe",
 			stderr: "ignore",
-			timeout: CLIENT_TERMTYPE_TIMEOUT_MS,
+			timeout: CLIENT_TERMINAL_TIMEOUT_MS,
 			killSignal: "SIGKILL",
 		});
-		if (result.exitCode !== 0) return null;
-		return CLIENT_TERMTYPE_NAME.exec(result.stdout.toString().trim())?.[1] ?? null;
+		if (result.exitCode !== 0) return [];
+		return tmuxClientProgramsFromReport(result.stdout.toString());
 	} catch {
-		return null;
+		return [];
 	}
 }
 
@@ -75,14 +106,77 @@ export function setTmuxPaneApcTitles(on: boolean, env: NodeJS.ProcessEnv = Bun.e
 }
 
 /**
- * Resolve the terminal emulator name recorded for this tmux client.
+ * Resolve the terminal emulator names recorded for this tmux client, best first.
  *
  * tmux overwrites pane identity variables with its own values, but retains the
- * attached client's terminal-type reply in `#{client_termtype}`. The local IPC
- * query runs once per process and degrades to `null` when unavailable.
+ * attached client's terminal-type reply in `#{client_termtype}` and its `TERM`
+ * in `#{client_termname}` (see {@link tmuxClientProgramsFromReport}). The local
+ * IPC query runs once per process and degrades to `[]` when unavailable.
  */
-export function resolveTmuxClientTerminalName(env: NodeJS.ProcessEnv = Bun.env): string | null {
-	if (!tmuxMultiplexer.isInside(env) || isBunTestRuntime()) return null;
-	if (cachedClientTerminalName === undefined) cachedClientTerminalName = queryTmuxClientTerminalName(env);
-	return cachedClientTerminalName;
+export function resolveTmuxClientTerminalNames(env: NodeJS.ProcessEnv = Bun.env): readonly string[] {
+	if (!tmuxMultiplexer.isInside(env) || isBunTestRuntime()) return [];
+	cachedClientTerminalNames ??= queryTmuxClientTerminalNames(env);
+	return cachedClientTerminalNames;
+}
+
+const SIXEL_CAPABILITY = /^\s*\d+: Sxl: \(flag\) true$/mu;
+let cachedClientSixel: { tmux: string; renders: boolean } | undefined;
+
+function queryTmuxClientRendersSixel(env: NodeJS.ProcessEnv): boolean {
+	const client = runTmux(["display-message", "-p", "#{client_name}"], env)?.trim();
+	if (!client) return false;
+	return SIXEL_CAPABILITY.test(runTmux(["show-messages", "-T", "-t", client], env) ?? "");
+}
+
+/**
+ * Whether tmux draws SIXEL to the client attached to this session: its
+ * effective terminal description for that client carries `Sxl`.
+ *
+ * A tmux built with SIXEL answers every pane's DA1 with attribute 4 and draws a
+ * `SIXEL IMAGE (WxH)` text placeholder for clients without SIXEL, so the pane's
+ * DA1 says nothing about the outer terminal. tmux itself renders SIXEL only
+ * when the client's terminal has `Sxl`, whether from terminfo,
+ * `terminal-overrides`, or the `sixel` feature (DA1 or `terminal-features`),
+ * and `show-messages -T` reports that merged description. The TUI consults
+ * this before trusting the pane's DA1 attribute. Queried once per tmux
+ * session; `false` outside tmux, with no attached client, or when the query
+ * fails. Nested tmux usually reports no `Sxl`: the inner client's terminal is
+ * the outer tmux, whose DA1 starts with level 1, which tmux does not parse
+ * for features.
+ */
+export function tmuxClientRendersSixel(env: NodeJS.ProcessEnv = Bun.env): boolean {
+	const tmux = env.TMUX;
+	if (!tmux) return false;
+	if (cachedClientSixel?.tmux !== tmux) cachedClientSixel = { tmux, renders: queryTmuxClientRendersSixel(env) };
+	return cachedClientSixel.renders;
+}
+
+let cachedPanePassthrough: { key: string; allowed: boolean } | undefined;
+
+function queryTmuxPaneAllowsPassthrough(env: NodeJS.ProcessEnv): boolean {
+	const pane = env.TMUX_PANE;
+	const target = pane ? ["-t", pane] : [];
+	const value = runTmux(["show-options", "-Apqv", ...target, "allow-passthrough"], env)?.trim();
+	if (value === undefined) return false;
+	// tmux before 3.3 has no `allow-passthrough` option (`-q` prints nothing)
+	// and always forwards DCS passthrough.
+	return value === "" || value === "on" || value === "all";
+}
+
+/**
+ * Whether tmux forwards this pane's `\x1bPtmux;…\x1b\\` passthrough envelopes
+ * to the attached client. tmux 3.3+ defaults `allow-passthrough` to `off` and
+ * then silently drops every wrapped sequence, so Kitty graphics sent through
+ * the envelope never reach a Kitty-capable outer terminal. Resolves the
+ * effective pane value (pane → window → global). Queried once per pane;
+ * `false` outside tmux or when the query fails.
+ */
+export function tmuxPaneAllowsPassthrough(env: NodeJS.ProcessEnv = Bun.env): boolean {
+	const tmux = env.TMUX;
+	if (!tmux) return false;
+	const key = `${tmux}\0${env.TMUX_PANE ?? ""}`;
+	if (cachedPanePassthrough?.key !== key) {
+		cachedPanePassthrough = { key, allowed: queryTmuxPaneAllowsPassthrough(env) };
+	}
+	return cachedPanePassthrough.allowed;
 }

package/src/tools/ask.ts

@@ -23,6 +23,7 @@ import { getMarkdownTheme, type Theme } from "../theme/theme";
 
 import { outputBlockContentWidth, renderStatusLine } from "../render";
 import { framedToolCard } from "../render/tool-card";
+import { wrapPrefixedLine } from "../render/hanging-wrap";
 
 import { formatErrorMessage, formatMeta, formatTitle, sanitizeCarriageReturns } from "../render/render-utils";
 import { tuiText, tuiTextFmt } from "../i18n";
@@ -203,20 +204,27 @@ export function optionMarker(uiTheme: Theme, multi: boolean | undefined, selecte
 	return selected ? uiTheme.radio.selected : uiTheme.radio.unselected;
 }
 
+function wrapOptionLabel(marker: string, label: string, width: number): string[] {
+	const prefix = ` ${marker} `;
+	return wrapPrefixedLine(prefix, " ".repeat(visibleWidth(prefix)), label, width);
+}
+
 /** Render the offered options for a question form as flat marker bullets (no tree guides). */
 function renderQuestionOptionLines(
 	uiTheme: Theme,
 	mdTheme: MarkdownTheme,
 	options: AskRenderOption[],
 	multi: boolean | undefined,
+	width: number,
 ): string[] {
 	const out: string[] = [];
 	for (const opt of options) {
 		const optLabel = renderInlineMarkdown(opt.label, mdTheme, t => uiTheme.fg("muted", t));
-		out.push(` ${uiTheme.fg("dim", optionMarker(uiTheme, multi, false))} ${optLabel}`);
+		out.push(...wrapOptionLabel(uiTheme.fg("dim", optionMarker(uiTheme, multi, false)), optLabel, width));
 		if (opt.description?.trim()) {
 			const description = renderInlineMarkdown(opt.description.trim(), mdTheme, t => uiTheme.fg("dim", t));
-			out.push(`   ${uiTheme.fg("dim", "↳")} ${description}`);
+			const prefix = `   ${uiTheme.fg("dim", "↳")} `;
+			out.push(...wrapPrefixedLine(prefix, " ".repeat(visibleWidth(prefix)), description, width));
 		}
 	}
 	return out;
@@ -261,6 +269,7 @@ function renderAnswerOptionLines(
 	customInput: string | undefined,
 	note: string | undefined,
 	width: number,
+	contentWidth: number,
 	selectedIndices?: ReadonlySet<number>,
 ): string[] {
 	const selected = new Set(selectedOptions ?? []);
@@ -283,7 +292,7 @@ function renderAnswerOptionLines(
 		const labelStyled = renderInlineMarkdown(label, mdTheme, t =>
 			isSelected ? uiTheme.fg("toolOutput", t) : uiTheme.fg("muted", t),
 		);
-		out.push(` ${markerStyled} ${labelStyled}`);
+		out.push(...wrapOptionLabel(markerStyled, labelStyled, contentWidth));
 	}
 	if (customInput !== undefined) out.push(...renderCustomInputLines(uiTheme, customInput));
 	if (note !== undefined) out.push(...renderNoteLines(uiTheme, note, width));
@@ -461,7 +470,7 @@ export const askToolRenderer = {
 					? tuiText("askQuestionOne", "1 question")
 					: tuiTextFmt("askQuestionManyFmt", "%d questions", questions.length),
 			)}`;
-			return framedToolCard(uiTheme, ({ width }) => {
+			return framedToolCard(uiTheme, ({ width, contentWidth }) => {
 				const sections = questions.map(q => {
 					const meta: string[] = [];
 					if (q.multi) meta.push(tuiText("askMultiMeta", "multi"));
@@ -470,7 +479,7 @@ export const askToolRenderer = {
 					// md() returns a shared cached array (module-level Markdown LRU) — copy before appending.
 					const mdLines = md(q.question, width);
 					const lines = q.options?.length
-						? [...mdLines, ...renderQuestionOptionLines(uiTheme, mdTheme, q.options, q.multi)]
+						? [...mdLines, ...renderQuestionOptionLines(uiTheme, mdTheme, q.options, q.multi, contentWidth)]
 						: mdLines;
 					return { label: `${uiTheme.fg("dim", `[${q.id}]`)}${metaStr}`, content: lines };
 				});
@@ -496,11 +505,11 @@ export const askToolRenderer = {
 		if (questionOptions?.length) meta.push(`options:${questionOptions.length}`);
 		const header = `${label}${formatMeta(meta, uiTheme)}`;
 		const multi = args.multi;
-		return framedToolCard(uiTheme, ({ width }) => {
+		return framedToolCard(uiTheme, ({ width, contentWidth }) => {
 			// md() returns a shared cached array (module-level Markdown LRU) — copy before appending.
 			const mdLines = md(question, width);
 			const bodyLines = questionOptions?.length
-				? [...mdLines, ...renderQuestionOptionLines(uiTheme, mdTheme, questionOptions, multi)]
+				? [...mdLines, ...renderQuestionOptionLines(uiTheme, mdTheme, questionOptions, multi, contentWidth)]
 				: mdLines;
 			return {
 				header,
@@ -570,7 +579,7 @@ export const askToolRenderer = {
 				},
 				uiTheme,
 			);
-			return framedToolCard(uiTheme, ({ width }) => {
+			return framedToolCard(uiTheme, ({ width, contentWidth }) => {
 				const rawResults = rawDetails.results ?? [];
 				const sections = results.map((r, index) => {
 					// Sanitizing preserves order and length, so raw indices align with `r`.
@@ -587,6 +596,7 @@ export const askToolRenderer = {
 							r.customInput,
 							r.note,
 							width,
+							contentWidth,
 							selectedIndicesFor(raw?.options, raw?.selectedOptions),
 						),
 					];
@@ -625,7 +635,7 @@ export const askToolRenderer = {
 		const dCustom = details.customInput;
 		const dNote = details.note;
 		const dTimedOut = details.timedOut;
-		return framedToolCard(uiTheme, ({ width }) => {
+		return framedToolCard(uiTheme, ({ width, contentWidth }) => {
 			// md() returns a shared cached array (module-level Markdown LRU) — copy before appending.
 			const bodyLines = [
 				...md(question, width),
@@ -638,6 +648,7 @@ export const askToolRenderer = {
 					dCustom,
 					dNote,
 					width,
+					contentWidth,
 					selectedIndicesFor(rawDetails.options, rawDetails.selectedOptions),
 				),
 			];

package/src/tools/ast-edit.d.ts

@@ -1,7 +1,7 @@
 import type { Component } from "../tui";
 import type { Theme } from "../theme/theme";
 import type { OutputMeta } from "./output-meta";
-import type { NativeToolView, RenderResultOptions } from "./renderer";
+import type { NativeToolView, RenderResultContextOptions, RenderResultOptions } from "./renderer";
 /** Display metadata returned by ast-edit. */
 export interface AstEditToolDetails {
     totalReplacements: number;
@@ -47,7 +47,7 @@ export declare const astEditToolRenderer: {
         }>;
         details?: AstEditToolDetails;
         isError?: boolean;
-    }, options: RenderResultOptions, uiTheme: Theme, args?: AstEditRenderArgs): Component;
+    }, options: RenderResultContextOptions, uiTheme: Theme, args?: AstEditRenderArgs): Component;
     describeCall(args: AstEditRenderArgs): NativeToolView;
     describeResult(result: {
         content: Array<{

package/src/tools/ast-edit.ts

@@ -1,3 +1,4 @@
+import { isRecord, sanitizeText } from "@linxiraos/pi-utils";
 import type { Component } from "../tui";
 import { Text } from "../components/text";
 import { replaceTabs } from "../utils";
@@ -14,11 +15,16 @@ import {
 } from "../render/render-utils";
 import { classifyGroupedLines, groupLineIndicesByBlank } from "./grouped-file-output";
 import type { OutputMeta } from "./output-meta";
-import type { NativeToolHead, NativeToolView, RenderResultOptions, ToolRenderer } from "./renderer";
+import type {
+	NativeToolHead,
+	NativeToolView,
+	RenderResultContextOptions,
+	RenderResultOptions,
+	ToolRenderer,
+} from "./renderer";
 import { code, compact, diff, node, span, text } from "../native/describe";
 import type { NativeChild, NativeNode } from "../native/node";
 import { diffStatsMeta, displayPath, errorText, fileDiffSection, noteText, resultText, statsText } from "./native-view";
-import { getLanguageFromPath } from "../lang-from-path";
 
 /** Display metadata returned by ast-edit. */
 export interface AstEditToolDetails {
@@ -148,9 +154,9 @@ function parseAstEditChanges(lines: readonly string[]): AstEditFileDiff[] {
 	return files;
 }
 
-/** A file's proposed rewrite as a `diff`: highlighted by the path's language, no path header. */
+/** A file's proposed rewrite as a `diff`: Tern highlights it by the file's grammar and draws no path header. */
 function astEditDiff(file: AstEditFileDiff): NativeNode {
-	return diff(file.lines.join("\n"), { lang: getLanguageFromPath(file.path) });
+	return diff(file.lines.join("\n"), { path: file.path });
 }
 
 /** The single rewrite pattern (`target:"pattern"`), or the rewrite count, for the head. */
@@ -190,7 +196,7 @@ export const astEditToolRenderer = {
 
 	renderResult(
 		result: { content: Array<{ type: string; text?: string }>; details?: AstEditToolDetails; isError?: boolean },
-		options: RenderResultOptions,
+		options: RenderResultContextOptions,
 		uiTheme: Theme,
 		args?: AstEditRenderArgs,
 	): Component {
@@ -245,6 +251,19 @@ export const astEditToolRenderer = {
 		// Resolve hyperlinks over the whole output so nested directory headers
 		// reconstruct across the blank-line groups the tree list collapses by.
 		const contexts = classifyGroupedLines(allLines, details?.cwd ?? details?.searchPath, details?.searchPath);
+		// The display lines are sanitized; reconstruct link targets from the original
+		// grouped headers only when they still align with those display lines.
+		const rawDetails = options.renderContext?.rawDetails;
+		const rawContent =
+			isRecord(rawDetails) && typeof rawDetails.displayContent === "string" ? rawDetails.displayContent : undefined;
+		const rawLines = rawContent?.split("\n");
+		const rawCwd = isRecord(rawDetails) && typeof rawDetails.cwd === "string" ? rawDetails.cwd : undefined;
+		const rawSearchPath =
+			isRecord(rawDetails) && typeof rawDetails.searchPath === "string" ? rawDetails.searchPath : undefined;
+		const rawContexts =
+			rawContent !== undefined && rawLines?.length === allLines.length && sanitizeText(rawContent) === textContent
+				? classifyGroupedLines(rawLines, rawCwd ?? rawSearchPath, rawSearchPath)
+				: undefined;
 		// Style lazily: collapsed bodies show only the first groups.
 		const styledLines: (string | undefined)[] = Array.from({ length: allLines.length }, () => undefined);
 		const styleLine = (index: number): string => {
@@ -255,12 +274,16 @@ export const astEditToolRenderer = {
 			// second vertical bar inside the frame border.
 			const display = replaceTabs(allLines[index]!.replace("│", " "));
 			let styled: string;
+			const linkPath =
+				ctx.headerPath && rawContexts?.[index]?.kind === ctx.kind && rawContexts[index]?.depth === ctx.depth
+					? (rawContexts[index]?.headerPath ?? ctx.headerPath)
+					: ctx.headerPath;
 			if (ctx.kind === "dir") {
 				const accent = uiTheme.fg("accent", display);
-				styled = ctx.headerPath ? fileHyperlink(ctx.headerPath, accent) : accent;
+				styled = linkPath ? fileHyperlink(linkPath, accent) : accent;
 			} else if (ctx.kind === "file") {
 				const tinted = uiTheme.fg(ctx.depth === 1 ? "accent" : "dim", display);
-				styled = ctx.headerPath ? fileHyperlink(ctx.headerPath, tinted) : tinted;
+				styled = linkPath ? fileHyperlink(linkPath, tinted) : tinted;
 			} else if (display.startsWith("+")) {
 				styled = uiTheme.fg("toolDiffAdded", display);
 			} else if (display.startsWith("-")) {

package/src/tools/ast-grep.d.ts

@@ -1,7 +1,7 @@
 import type { Component } from "../tui";
 import type { Theme } from "../theme/theme";
 import type { OutputMeta } from "./output-meta";
-import type { RenderResultOptions, NativeToolView } from "./renderer";
+import type { NativeToolView, RenderResultContextOptions, RenderResultOptions } from "./renderer";
 /** Display metadata returned by ast-grep. */
 export interface AstGrepToolDetails {
     matchCount: number;
@@ -46,7 +46,7 @@ export declare const astGrepToolRenderer: {
         }>;
         details?: AstGrepToolDetails;
         isError?: boolean;
-    }, options: RenderResultOptions, uiTheme: Theme, args?: AstGrepRenderArgs): Component;
+    }, options: RenderResultContextOptions, uiTheme: Theme, args?: AstGrepRenderArgs): Component;
     describeCall(args: AstGrepRenderArgs): NativeToolView;
     describeResult(result: {
         content: Array<{