@oh-my-pi/pi-tui 18.12.0 → 18.12.1

package/CHANGELOG.md

@@ -2,6 +2,21 @@
 
 ## [Unreleased]
 
+## [18.12.1] - 2026-10-17
+
+### Breaking Changes
+
+- `numberedCode`, `readContentCode` and `nativeDiff` take the file's path (`path`) instead of a language, and `describeGroupedOutput` drops its `lang` option: Tern picks the grammar of file contents from their path
+
+### Changed
+
+- Advisor notes in Tern read as a callout tinted by their worst severity: its icon and a tally (`Concern`, `1 blocker · 2 concerns`) head the card with the advisor's name at the end, and several notes each lead with their own severity icon
+
+### Fixed
+
+- Fixed images rendering blank inside tmux on Kitty-capable terminals when tmux's `allow-passthrough` is off (the default); omp now falls back to SIXEL or the text placeholder instead of sending Kitty graphics tmux drops
+- Fixed Tern drawing tool cards' file contents (write, read, edit, grep, find, ast-grep, ast-edit) plain or in the wrong language when omp did not know the file's extension (`.ptx`) or mapped it to a neighbor (`.cu` as C++); omp now sends the file's path and Tern picks the grammar
+
 ## [18.12.0] - 2026-10-17
 
 ### Breaking Changes
@@ -2512,14 +2527,4 @@
 
 - Changed truncation debug logging to run only when `debugRedraw` is enabled
 
-## [14.0.5] - 2026-04-11
-
-### Changed
-
-- Updated hash computation to use `Bun.hash()` instead of `Bun.hash.xxHash64()`, which may return `number` in addition to `bigint`
-- Simplified cache key computation in Box component by removing intermediate hash updates and consolidating hash operations
-- Wrapped native text utility functions (`sliceWithWidth`, `truncateToWidth`, `wrapTextWithAnsi`, `extractSegments`) to automatically pass the current default tab width, simplifying the API for consumers
-- Added `getIndentationNoescape` wrapper that uses `process.cwd()` as the project root for relative file paths
-- Re-export `getDefaultTabWidth`, `getIndentation`, and `setDefaultTabWidth` from `@oh-my-pi/pi-utils`; native text helpers still receive tab width via wrappers that read the JS default
-
-Older entries are archived in [packages/tui/CHANGELOG.md@691b3d448f3d](https://github.com/can1357/oh-my-pi/blob/691b3d448f3da1f99010854dea4a562395fd190d/packages/tui/CHANGELOG.md).
+Older entries are archived in [packages/tui/CHANGELOG.md@200e7d2c6ebb](https://github.com/can1357/oh-my-pi/blob/200e7d2c6ebb2e187b167f807b9f68f779c0bf4b/packages/tui/CHANGELOG.md).

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

@@ -1,6 +1,6 @@
 import { type Component } from "../tui.js";
 import type { AdvisorMessageDetails } from "./messages.js";
-import { type Theme } from "../theme/index.js";
+import type { Theme } from "../theme/index.js";
 import { type NativeUiEvent } from "../native/node.js";
 /** The advisor card: shared by the controlled expansion getter and native toggles. */
 export interface AdvisorMessageCard extends Component {
@@ -14,6 +14,9 @@ export interface AdvisorMessageCard extends Component {
  * output (whose `thinkingText` color equals `toolOutput` in most themes):
  * a bold `customMessageLabel` header tag (skill-card convention), a heavy
  * rail tinted per-note severity, and the note body on the default text color.
+ * Natively it is a callout tinted by its worst note: that severity's icon and
+ * the tally head it, the advisor's name ends the head, and with several notes
+ * each leads with its own severity icon.
  *
  * `getExpanded` owns expansion; a native toggle overrides it for this card
  * until the next transcript-wide change.

package/dist/types/chat/messages.d.ts

@@ -173,7 +173,9 @@ export declare function isUserInvokedSkillPrompt(message: CustomMessage): boolea
  * reminders, and auto-continues are not turn starts.
  */
 export declare function isUserTurnInitiator(message: CustomMessage): boolean;
-export type AdvisorSeverity = "nit" | "concern" | "blocker";
+/** Advisor note severities, weakest first; ranks severity thresholds such as `advisor.continueSeverity`. */
+export declare const ADVISOR_SEVERITIES: readonly ["nit", "concern", "blocker"];
+export type AdvisorSeverity = (typeof ADVISOR_SEVERITIES)[number];
 /** One queued advice note. */
 export interface AdvisorNote {
     note: string;

package/dist/types/chrome/diff.d.ts

@@ -7,10 +7,9 @@ import type { NativeNode } from "../native/node.js";
  * becomes one hunk; the terminal draws gutters, word emphasis and highlighting.
  * Removed rows carry old line numbers, added rows new ones and context rows
  * old ones, so the other side is derived from the running line delta.
+ * Tern picks the grammar from `path`.
  */
-export declare function nativeDiff(diffText: string, options?: {
-    filePath?: string;
-} & TspProps<"diff">): NativeNode;
+export declare function nativeDiff(diffText: string, props?: TspProps<"diff">): NativeNode;
 export interface RenderDiffOptions {
     /** File path used to resolve indentation (.editorconfig + defaults) */
     filePath?: string;

package/dist/types/tmux.d.ts

@@ -45,3 +45,12 @@ export declare function resolveTmuxClientTerminalNames(env?: NodeJS.ProcessEnv):
  * 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 tmuxPaneAllowsPassthrough(env?: NodeJS.ProcessEnv): boolean;

package/dist/types/tools/eval.d.ts

@@ -13,30 +13,60 @@ export interface EvalStatusEvent {
     resolvedThinkingLevel?: ConfiguredThinkingLevel;
     [key: string]: unknown;
 }
+/** One column of a table view. */
+export interface EvalTableColumn {
+    head: string;
+    /** Numeric values: renderers right-align the column. */
+    numeric?: boolean;
+}
+/**
+ * A pretty rendering of something a cell printed, shown to the user in place
+ * of its text; the model only ever sees the text.
+ * - `table`: a data frame or `console.table`; leading index columns are plain
+ *   columns, and a truncated frame's elided rows/columns are `…` cells.
+ * - `json`: a structured value (dict/list, object/array) as a collapsible tree.
+ * - `markdown`: an object's Markdown representation.
+ */
+export type EvalView = {
+    kind: "table";
+    columns: EvalTableColumn[];
+    rows: string[][];
+} | {
+    kind: "json";
+    value: unknown;
+} | {
+    kind: "markdown";
+    text: string;
+};
+/**
+ * A view and the exact printed text it stands for. Renderers locate `text` in
+ * the cell's `output` (entries in print order) and show the view there; an
+ * entry whose text is no longer in the output (truncated) is skipped.
+ */
+export interface EvalViewEntry {
+    text: string;
+    view: EvalView;
+}
 /** Per-cell execution result for transcript rendering. */
 export interface EvalCellResult {
     index: number;
     title?: string;
     code: string;
     language?: EvalLanguage;
+    /** What the model saw: the cell's printed text plus image notes. */
     output: string;
-    /**
-     * `output` without its `display[N]:` JSON sections, which renderers show
-     * from `jsonOutputs` instead. Absent when the cell displayed no JSON.
-     */
-    textOutput?: string;
+    /** Pretty views of spans of `output`, in print order. */
+    views?: EvalViewEntry[];
     status: "pending" | "running" | "complete" | "error";
     durationMs?: number;
     exitCode?: number;
     statusEvents?: EvalStatusEvent[];
     /** Discrete events dropped from the front of `statusEvents` (see {@link recordStatusEvent}). */
     statusEventsElided?: number;
-    hasMarkdown?: boolean;
 }
 /** Tool result detail object surfaced to the UI/transcript. */
 export interface EvalToolDetails {
     cells?: EvalCellResult[];
-    jsonOutputs?: unknown[];
     images?: ImageContent[];
     statusEvents?: EvalStatusEvent[];
     /** Discrete events dropped from the front of `statusEvents` (see {@link recordStatusEvent}). */
@@ -105,6 +135,20 @@ export declare function upsertStatusEvent(events: EvalStatusEvent[], event: Eval
  * events behind an "… N earlier" row.
  */
 export declare function recordStatusEvent(log: StatusEventLog, event: EvalStatusEvent): void;
+/** One ordered piece of a cell's output: printed text, or a view shown in place of the text it matched. */
+export type EvalOutputPart = {
+    kind: "text";
+    text: string;
+} | {
+    kind: "view";
+    view: EvalView;
+};
+/**
+ * Split `output` around its views, in order: each view's `text` is searched
+ * from the end of the previous match, unmatched entries are skipped, and
+ * blank text between views is dropped (newlines bordering a view trimmed).
+ */
+export declare function splitEvalOutput(output: string, views: readonly EvalViewEntry[] | undefined): EvalOutputPart[];
 /** Render eval code cells, structured display output, and progress events. */
 export declare const evalToolRenderer: {
     animatedPendingPreview: true;

package/dist/types/tools/grouped-file-output.d.ts

@@ -110,11 +110,12 @@ export interface NumberedLine {
  * Numbered `code` for possibly non-contiguous source lines: one block per run
  * of consecutive line numbers (`start` = the run's first line, match lines
  * `mark`ed), with a `…` row wherever the numbers jump or an elision marker
- * (`n: null`) sits. No path header: the row or head above names the file.
+ * (`n: null`) sits. `path` is the file the lines come from: Tern picks their
+ * grammar from it and draws no header (the row or head above names the file).
  * `wrap` soft-wraps long lines instead of scrolling the block sideways.
  */
 export declare function numberedCode(lines: readonly NumberedLine[], opts?: {
-    lang?: string;
+    path?: string;
     wrap?: boolean;
 }): NativeNode[];
 /**
@@ -128,7 +129,6 @@ export declare function numberedCode(lines: readonly NumberedLine[], opts?: {
  * same file row as a directory search's.
  */
 export declare function describeGroupedOutput(lines: readonly string[], options?: {
-    lang?: (path: string) => string | undefined;
     maxFiles?: number;
     file?: string;
 }): NativeNode[];

package/dist/types/tools/read.d.ts

@@ -136,7 +136,7 @@ export declare function readResultIsMarkdown(details: ReadToolDetails | undefine
  * block; runs split with `…` between) when they cover every line, else count
  * up from `start`.
  */
-export declare function readContentCode(content: string, start: number, numbers: readonly (number | null)[] | undefined, lang: string | undefined): NativeNode[];
+export declare function readContentCode(content: string, start: number, numbers: readonly (number | null)[] | undefined, path: string | undefined): NativeNode[];
 /** Render file, image, and URL reads in the transcript. */
 export declare const readToolRenderer: {
     activitySummary(args: unknown, _context: ToolActivityContext): ToolActivitySummary;

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-tui",
-	"version": "18.12.0",
+	"version": "18.12.1",
 	"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.12.0",
-		"@oh-my-pi/pi-agent-core": "18.12.0",
-		"@oh-my-pi/pi-ai": "18.12.0",
-		"@oh-my-pi/pi-catalog": "18.12.0",
-		"@oh-my-pi/pi-natives": "18.12.0",
-		"@oh-my-pi/pi-utils": "18.12.0",
-		"@oh-my-pi/pi-wire": "18.12.0",
-		"@oh-my-pi/snapcompact": "18.12.0"
+		"@oh-my-pi/omptype": "18.12.1",
+		"@oh-my-pi/pi-agent-core": "18.12.1",
+		"@oh-my-pi/pi-ai": "18.12.1",
+		"@oh-my-pi/pi-catalog": "18.12.1",
+		"@oh-my-pi/pi-natives": "18.12.1",
+		"@oh-my-pi/pi-utils": "18.12.1",
+		"@oh-my-pi/pi-wire": "18.12.1",
+		"@oh-my-pi/snapcompact": "18.12.1"
 	},
 	"devDependencies": {
 		"kitty-vt-wasm": "^0.2.0"

package/src/chat/advisor-message.ts

@@ -4,9 +4,9 @@ import { visibleWidth } from "../utils";
 import type { AdvisorMessageDetails, AdvisorNote, AdvisorSeverity } from "./messages";
 import { formatBadge, replaceTabs, type ToolUIColor, wrapTextWithAnsi } from "../render/render-utils";
 import { Ellipsis, truncateToWidth } from "../render";
-import { getThemeEpoch, type Theme } from "../theme";
-import type { TspSpan } from "@oh-my-pi/pi-wire";
-import { card, span, text } from "../native/describe";
+import type { Theme } from "../theme";
+import type { TspSpan, TspTone } from "@oh-my-pi/pi-wire";
+import { card, node, span, text } from "../native/describe";
 import { type NativeNode, type NativeUiEvent, rootToggleExpanded } from "../native/node";
 import { plainText } from "../native/spans";
 import { Memo } from "../native/memo";
@@ -111,13 +111,16 @@ class AdvisorNotes implements Component {
 	}
 }
 
+/** The advisor a note came from; undefined for the implicit single ("default") advisor, which goes unlabeled. */
+function sourceOf(entry: AdvisorNote): string | undefined {
+	return entry.advisor && entry.advisor !== "default" ? entry.advisor : undefined;
+}
+
 /** Wrapped, truncated rail rows for a single note; shared by both branches. */
 function renderAdvisorNote(entry: AdvisorNote, width: number, uiTheme: Theme): string[] {
 	const badge = entry.severity ? `${formatBadge(entry.severity, severityColor(entry.severity), uiTheme)} ` : "";
-	// Multi-advisor: attribute the note to its source. The implicit
-	// single ("default") advisor renders unlabeled, as before.
-	const who =
-		entry.advisor && entry.advisor !== "default" ? `${uiTheme.fg("dim", `[${replaceTabs(entry.advisor)}]`)} ` : "";
+	const source = sourceOf(entry);
+	const who = source ? `${uiTheme.fg("dim", `[${replaceTabs(source)}]`)} ` : "";
 	const age = entry.turnsAgo !== undefined ? `${uiTheme.fg("dim", `T-${entry.turnsAgo}`)} ` : "";
 	const railGlyph = uiTheme.symbol("advisor.rail");
 	const rail = uiTheme.fg(severityColor(entry.severity), railGlyph);
@@ -145,16 +148,47 @@ function renderAdvisorNote(entry: AdvisorNote, width: number, uiTheme: Theme): s
 	);
 }
 
-/** Native spans for one note: severity badge, advisor attribution, then the note text. */
-function advisorNoteSpans(entry: AdvisorNote): TspSpan[] {
+/** A note's rank in the native callout: its severity, `note` when it has none. */
+type AdvisorRank = AdvisorSeverity | "note";
+
+/** Ranks worst first: the callout wears the worst rank among its notes. */
+const RANKS: readonly AdvisorRank[] = ["blocker", "concern", "nit", "note"];
+
+/** Each rank's native look, GitHub-alert style: card/icon tone, Tern icon name, label and its span style. */
+const RANK_LOOK: Record<AdvisorRank, { tone: TspTone; icon: string; label: string; style: string }> = {
+	blocker: { tone: "error", icon: "octagon-alert", label: "Blocker", style: "error strong" },
+	concern: { tone: "warning", icon: "warn", label: "Concern", style: "warning strong" },
+	nit: { tone: "neutral", icon: "lightbulb", label: "Nit", style: "strong" },
+	note: { tone: "neutral", icon: "info", label: "Note", style: "strong" },
+};
+
+function rankOf(entry: AdvisorNote): AdvisorRank {
+	return entry.severity ?? "note";
+}
+
+/** The callout's title: one note's rank (`Concern`), else a tally worst first (`1 blocker · 2 concerns`). */
+function tallySpans(notes: readonly AdvisorNote[]): TspSpan[] {
+	if (notes.length === 1) {
+		const { label, style } = RANK_LOOK[rankOf(notes[0]!)];
+		return [span(label, style)];
+	}
 	const spans: TspSpan[] = [];
-	if (entry.severity)
-		spans.push(span(entry.severity.toUpperCase(), `${severityColor(entry.severity)} strong`), span(" "));
-	if (entry.advisor && entry.advisor !== "default") spans.push(span(`[${entry.advisor}] `, "dim"));
-	spans.push(span(plainText(entry.note), "customMessageText"));
+	for (const rank of RANKS) {
+		const count = notes.filter(entry => rankOf(entry) === rank).length;
+		if (count === 0) continue;
+		if (spans.length > 0) spans.push(span(" · ", "dim"));
+		spans.push(span(`${count} ${rank}${count === 1 ? "" : "s"}`, RANK_LOOK[rank].style));
+	}
 	return spans;
 }
 
+/** Native spans for one note: its advisor when the callout mixes several, then the text. */
+function noteSpans(entry: AdvisorNote, attribute: boolean): TspSpan[] {
+	const who = attribute ? sourceOf(entry) : undefined;
+	const body = span(plainText(entry.note));
+	return who ? [span(`${who} · `, "dim"), body] : [body];
+}
+
 /** The advisor card: shared by the controlled expansion getter and native toggles. */
 export interface AdvisorMessageCard extends Component {
 	/** Transcript-wide expansion (`Ctrl+O`); clears a toggle made in the terminal. */
@@ -168,6 +202,9 @@ export interface AdvisorMessageCard extends Component {
  * output (whose `thinkingText` color equals `toolOutput` in most themes):
  * a bold `customMessageLabel` header tag (skill-card convention), a heavy
  * rail tinted per-note severity, and the note body on the default text color.
+ * Natively it is a callout tinted by its worst note: that severity's icon and
+ * the tally head it, the advisor's name ends the head, and with several notes
+ * each leads with its own severity icon.
  *
  * `getExpanded` owns expansion; a native toggle overrides it for this card
  * until the next transcript-wide change.
@@ -193,24 +230,40 @@ export function createAdvisorMessageCard(
 	};
 	const nativeMemo = new Memo();
 	const describeCard = (isExpanded: boolean): NativeNode => {
-		const head: TspSpan[] = [
-			span(`${uiTheme.status.info} Advisor`, "customMessageLabel strong"),
-			span(` ${notes.length} ${notes.length === 1 ? "note" : "notes"}`, "dim"),
-		];
-		if (blockers > 0) head.push(span(`${uiTheme.sep.dot}${blockers} blocker${blockers === 1 ? "" : "s"}`, "error"));
-		const body = notes.map((entry, index) =>
-			text(advisorNoteSpans(entry), { wrap: "word", role: "omp.advisor.note", key: `n${index}` }),
+		const look = RANK_LOOK[RANKS.find(rank => notes.some(entry => rankOf(entry) === rank)) ?? "note"];
+		const sources = new Set(notes.map(sourceOf));
+		const [sole] = sources;
+		const head = node(
+			"row",
+			{ gap: "sm", align: "center", role: "omp.advisor.head" },
+			[
+				node("icon", { name: look.icon, tone: look.tone }),
+				text(tallySpans(notes), { grow: 1 }),
+				text([span(sources.size === 1 && sole ? sole : "Advisor", "dim")], { role: "omp.advisor.source" }),
+			],
+			"head",
 		);
+		const attribute = sources.size > 1;
+		const body = notes.map((entry, index) => {
+			const spans = noteSpans(entry, attribute);
+			if (notes.length === 1) return text(spans, { wrap: "word", role: "omp.advisor.note", key: `n${index}` });
+			const { tone, icon, label } = RANK_LOOK[rankOf(entry)];
+			return node(
+				"row",
+				{ gap: "sm", align: "start", role: "omp.advisor.note", tone },
+				[node("icon", { name: icon, tone, aria: label }), text(spans, { wrap: "word", grow: 1, shrink: 1 })],
+				`n${index}`,
+			);
+		});
 		return card(
 			{
 				role: "omp.advisor",
-				tone: blockers > 0 ? "error" : "info",
-				head,
+				tone: look.tone,
 				collapsible: notes.length > COLLAPSED_NOTES,
 				collapsed: notes.length > COLLAPSED_NOTES ? !isExpanded : undefined,
 				preview: notes.length > COLLAPSED_NOTES ? "auto" : undefined,
 			},
-			body,
+			[head, ...body],
 		);
 	};
 
@@ -231,7 +284,7 @@ export function createAdvisorMessageCard(
 		},
 		describe(): NativeNode {
 			const isExpanded = expanded();
-			return nativeMemo.get([isExpanded, getThemeEpoch()], () => describeCard(isExpanded));
+			return nativeMemo.get([isExpanded], () => describeCard(isExpanded));
 		},
 		setExpanded(value: boolean): void {
 			lastGlobal = value;

package/src/chat/messages.ts

@@ -286,7 +286,10 @@ export function isUserTurnInitiator(message: CustomMessage): boolean {
 	);
 }
 
-export type AdvisorSeverity = "nit" | "concern" | "blocker";
+/** Advisor note severities, weakest first; ranks severity thresholds such as `advisor.continueSeverity`. */
+export const ADVISOR_SEVERITIES = ["nit", "concern", "blocker"] as const;
+
+export type AdvisorSeverity = (typeof ADVISOR_SEVERITIES)[number];
 
 /** One queued advice note. */
 export interface AdvisorNote {

package/src/chat/read-tool-group.ts

@@ -761,7 +761,7 @@ export class ReadToolGroupComponent extends Container implements ToolExecutionHa
 			lines.slice(0, shown).join("\n"),
 			entry.codeStartLine ?? firstSelectorLine(split.sel) ?? 1,
 			entry.codeLineNumbers?.slice(0, shown),
-			getLanguageFromPath(split.path),
+			split.path,
 		);
 		const hidden = lines.length - shown;
 		if (hidden > 0) {
@@ -836,7 +836,7 @@ export class ReadToolGroupComponent extends Container implements ToolExecutionHa
 						previewsMarkdown(entry)
 							? md(entry.contentText ?? "")
 							: code(entry.contentText ?? "", {
-									lang: getLanguageFromPath(split.path),
+									path: split.path,
 									start: entry.codeStartLine,
 									numbers: true,
 									wrap: true,

package/src/chrome/diff.ts

@@ -2,7 +2,7 @@ import { diffWords } from "@oh-my-pi/pi-natives";
 import { DEFAULT_TAB_WIDTH, sanitizeText } from "@oh-my-pi/pi-utils";
 import type { TspDiffHunk, TspProps } from "@oh-my-pi/pi-wire";
 import { theme as activeTheme, getLanguageFromPath, highlightCode, type Theme } from "../theme/index";
-import { type CodeFrameMarker, formatCodeFrameLine, replaceTabs, shortenPath } from "../render/render-utils";
+import { type CodeFrameMarker, formatCodeFrameLine, replaceTabs } from "../render/render-utils";
 import { node } from "../native/describe";
 import type { NativeNode } from "../native/node";
 /** SGR dim on / normal intensity — additive, preserves fg/bg colors. */
@@ -106,9 +106,9 @@ function renderIntraLineDiff(
  * becomes one hunk; the terminal draws gutters, word emphasis and highlighting.
  * Removed rows carry old line numbers, added rows new ones and context rows
  * old ones, so the other side is derived from the running line delta.
+ * Tern picks the grammar from `path`.
  */
-export function nativeDiff(diffText: string, options: { filePath?: string } & TspProps<"diff"> = {}): NativeNode {
-	const { filePath, ...props } = options;
+export function nativeDiff(diffText: string, props: TspProps<"diff"> = {}): NativeNode {
 	const hunks: TspDiffHunk[] = [];
 	let lines: string[] = [];
 	let oldStart = 0;
@@ -148,12 +148,7 @@ export function nativeDiff(diffText: string, options: { filePath?: string } & Ts
 		lines.push(`${parsed.prefix}${parsed.content}`);
 	}
 	close();
-	return node("diff", {
-		...props,
-		hunks,
-		path: props.path ?? (filePath ? shortenPath(filePath) : undefined),
-		lang: props.lang ?? (filePath ? getLanguageFromPath(filePath) : undefined),
-	});
+	return node("diff", { ...props, hunks });
 }
 
 export interface RenderDiffOptions {

package/src/terminal-capabilities.ts

@@ -17,7 +17,7 @@ import {
 	terminalMultiplexerForTerm,
 	terminalMultiplexerSessions,
 } from "./terminal-multiplexer";
-import { resolveTmuxClientTerminalNames, wrapTmuxPassthroughIfNeeded } from "./tmux";
+import { resolveTmuxClientTerminalNames, tmuxPaneAllowsPassthrough, wrapTmuxPassthroughIfNeeded } from "./tmux";
 import { detectTerminalBackground } from "./theme/theme";
 import type { HangulCompatibilityJamoWidth } from "./utils";
 
@@ -560,6 +560,12 @@ export function resolveImageProtocol(
 	) {
 		return null;
 	}
+	// Kitty graphics reach a tmux client only through the DCS passthrough
+	// envelope, which tmux drops unless `allow-passthrough` is enabled. A Kitty
+	// client identity alone would otherwise send images into the void.
+	if (imageProtocol === ImageProtocol.Kitty && env.TMUX && !tmuxPaneAllowsPassthrough(env)) {
+		return null;
+	}
 	return imageProtocol;
 }
 

package/src/tmux.ts

@@ -150,3 +150,33 @@ export function tmuxClientRendersSixel(env: NodeJS.ProcessEnv = Bun.env): boolea
 	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/ast-edit.ts

@@ -25,7 +25,6 @@ import type {
 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 {
@@ -155,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. */

package/src/tools/ast-grep.ts

@@ -14,7 +14,6 @@ import {
 	toPathList,
 } from "../render/render-utils";
 import { classifyGroupedLines, describeGroupedOutput, groupLineIndicesByBlank } from "./grouped-file-output";
-import { getLanguageFromPath } from "../lang-from-path";
 import { code, compact, node, span } from "../native/describe";
 import type { NativeChild } from "../native/node";
 import { errorText, noteText, resultText, toolHead } from "./native-view";
@@ -279,7 +278,7 @@ export const astGrepToolRenderer = {
 		return {
 			head,
 			body: compact<NativeChild>([
-				...describeGroupedOutput(kept, { lang: getLanguageFromPath }),
+				...describeGroupedOutput(kept),
 				limitReached && noteText("limit reached; narrow path or use skip", "warning"),
 				parseNote,
 			]),

package/src/tools/edit.ts

@@ -1222,7 +1222,7 @@ export const editToolRenderer = {
 				diffText = args.previewDiff ?? args.diff ?? "";
 				body.push(editDiff(diffText, rawPath));
 			} else if (args.diff || args.newText || args.patch) {
-				body.push(code(args.diff ?? args.newText ?? args.patch ?? "", { lang: getLanguageFromPath(rawPath) }));
+				body.push(code(args.diff ?? args.newText ?? args.patch ?? "", { path: rawPath || undefined }));
 			} else if (renderContext?.editDiffPreview) {
 				const preview = renderContext.editDiffPreview;
 				if ("error" in preview && preview.error) body.push(errorText(preview.error));
@@ -1362,9 +1362,9 @@ function editToolHead(facts: EditHeadFacts): NativeToolHead {
 	};
 }
 
-/** A file's diff: hunks highlighted by the path's language, no path header (the head or section names the file). */
+/** A file's diff: Tern highlights the hunks by the file's grammar and draws no path header (the head or section names the file). */
 function editDiff(diffText: string, filePath: string): NativeNode {
-	return nativeDiff(diffText, { lang: filePath ? getLanguageFromPath(filePath) : undefined });
+	return nativeDiff(diffText, { path: filePath || undefined });
 }
 
 /** One file's edit result for native views: head facts plus its body (diff or error, then diagnostics). */

package/src/tools/eval.ts

@@ -36,7 +36,6 @@ import {
 } from "./json-tree";
 import { formatStyledTruncationWarning, stripOutputNotice } from "./output-meta";
 import { splitPathAndSel } from "./read";
-import { getLanguageFromPath } from "../lang-from-path";
 import {
 	DEFAULT_TERMINAL_PREVIEW_LINES,
 	PREVIEW_LIMITS,
@@ -65,31 +64,57 @@ export interface EvalStatusEvent {
 	[key: string]: unknown;
 }
 
+/** One column of a table view. */
+export interface EvalTableColumn {
+	head: string;
+	/** Numeric values: renderers right-align the column. */
+	numeric?: boolean;
+}
+
+/**
+ * A pretty rendering of something a cell printed, shown to the user in place
+ * of its text; the model only ever sees the text.
+ * - `table`: a data frame or `console.table`; leading index columns are plain
+ *   columns, and a truncated frame's elided rows/columns are `…` cells.
+ * - `json`: a structured value (dict/list, object/array) as a collapsible tree.
+ * - `markdown`: an object's Markdown representation.
+ */
+export type EvalView =
+	| { kind: "table"; columns: EvalTableColumn[]; rows: string[][] }
+	| { kind: "json"; value: unknown }
+	| { kind: "markdown"; text: string };
+
+/**
+ * A view and the exact printed text it stands for. Renderers locate `text` in
+ * the cell's `output` (entries in print order) and show the view there; an
+ * entry whose text is no longer in the output (truncated) is skipped.
+ */
+export interface EvalViewEntry {
+	text: string;
+	view: EvalView;
+}
+
 /** Per-cell execution result for transcript rendering. */
 export interface EvalCellResult {
 	index: number;
 	title?: string;
 	code: string;
 	language?: EvalLanguage;
+	/** What the model saw: the cell's printed text plus image notes. */
 	output: string;
-	/**
-	 * `output` without its `display[N]:` JSON sections, which renderers show
-	 * from `jsonOutputs` instead. Absent when the cell displayed no JSON.
-	 */
-	textOutput?: string;
+	/** Pretty views of spans of `output`, in print order. */
+	views?: EvalViewEntry[];
 	status: "pending" | "running" | "complete" | "error";
 	durationMs?: number;
 	exitCode?: number;
 	statusEvents?: EvalStatusEvent[];
 	/** Discrete events dropped from the front of `statusEvents` (see {@link recordStatusEvent}). */
 	statusEventsElided?: number;
-	hasMarkdown?: boolean;
 }
 
 /** Tool result detail object surfaced to the UI/transcript. */
 export interface EvalToolDetails {
 	cells?: EvalCellResult[];
-	jsonOutputs?: unknown[];
 	images?: ImageContent[];
 	statusEvents?: EvalStatusEvent[];
 	/** Discrete events dropped from the front of `statusEvents` (see {@link recordStatusEvent}). */
@@ -593,8 +618,7 @@ function formatCellOutputLines(
 	theme: Theme,
 	width: number,
 ): { lines: readonly string[]; hiddenCount: number } {
-	// JSON displays render once, as the `jsonOutputs` tree below the last cell.
-	const output = cell.textOutput ?? cell.output;
+	const output = cell.output;
 	if (!output) {
 		return { lines: [], hiddenCount: 0 };
 	}
@@ -607,23 +631,47 @@ function formatCellOutputLines(
 	// exact hidden-count row, so the pane marker stays off here.
 	const innerWidth = outputBlockContentWidth(width);
 
-	if (cell.hasMarkdown && cell.status !== "error") {
-		const md = new Markdown(output, 0, 0, getMarkdownTheme());
-		const allLines = md.render(innerWidth);
+	const isError = cell.status === "error";
+	const styleText = (line: string): string => theme.fg(isError ? "error" : "toolOutput", replaceTabs(line));
+	// A table's printed text is already a terminal table, aligned to the line it starts on: keep it as text.
+	const parts = splitEvalOutput(
+		output,
+		cell.views?.filter(entry => entry.view.kind !== "table"),
+	);
+	if (parts.some(part => part.kind === "view")) {
+		// Printed text keeps its place; json and markdown views replace theirs with a tree / rendered Markdown.
+		const treeDepth = expanded ? JSON_TREE_MAX_DEPTH_EXPANDED : JSON_TREE_MAX_DEPTH_COLLAPSED;
+		const treeLineCap = expanded ? JSON_TREE_MAX_LINES_EXPANDED : JSON_TREE_MAX_LINES_COLLAPSED;
+		const treeScalarLen = expanded ? JSON_TREE_SCALAR_LEN_EXPANDED : JSON_TREE_SCALAR_LEN_COLLAPSED;
+		const allLines: string[] = [];
+		for (const part of parts) {
+			if (part.kind === "text") {
+				for (const line of part.text.split("\n")) allLines.push(styleText(line));
+				continue;
+			}
+			const view = part.view;
+			if (view.kind === "json") {
+				const tree = renderJsonTreeLines(view.value, theme, treeDepth, treeLineCap, treeScalarLen);
+				allLines.push(...tree.lines);
+				if (tree.truncated) allLines.push(theme.fg("dim", "…"));
+			} else if (view.kind === "markdown") {
+				allLines.push(...new Markdown(view.text, 0, 0, getMarkdownTheme()).render(innerWidth));
+			}
+		}
 		const formatted = formatOutputPaneLines(
 			{
 				lines: allLines,
 				expanded,
 				collapsedMaxLines: previewLines,
 				edge: "tail",
+				visual: true,
+				width: innerWidth,
 				showHiddenMarker: false,
 			},
 			theme,
 		);
 		return { lines: formatted.lines, hiddenCount: formatted.hiddenCount };
 	}
-
-	const isError = cell.status === "error";
 	const formatted = formatOutputPaneLines(
 		{
 			lines: output.split("\n"),
@@ -632,7 +680,7 @@ function formatCellOutputLines(
 			edge: "tail",
 			visual: true,
 			width: innerWidth,
-			styleLine: line => theme.fg(isError ? "error" : "toolOutput", replaceTabs(line)),
+			styleLine: styleText,
 			showHiddenMarker: false,
 		},
 		theme,
@@ -664,13 +712,12 @@ function describeStatusEvent(event: EvalStatusEvent, expanded: boolean): NativeN
 	if (!preview) return line;
 	const head = cappedHeadLines(sanitizeDisplayLines(preview), PREVIEW_LIMITS.OUTPUT_EXPANDED).lines;
 	const path = eventString(event.path);
-	const lang = path ? getLanguageFromPath(splitPathAndSel(path).path) : undefined;
 	return node("col", { role: "omp.tool.eval.status.event", gap: "none" }, [
 		keyed(line, "line"),
 		keyed(
 			code(head.join("\n"), {
 				role: "omp.tool.eval.status.preview",
-				lang,
+				path: path ? splitPathAndSel(path).path : undefined,
 				wrap: false,
 				preview: { lines: PREVIEW_LIMITS.COLLAPSED_LINES },
 			}),
@@ -702,96 +749,79 @@ function evalToolHead(
 	};
 }
 
-/** Box-drawing `console.table` output split out of plain text: `text` runs and parsed tables, in order. */
-type EvalOutputPart =
-	| { readonly kind: "text"; readonly text: string }
-	| { readonly kind: "table"; readonly head: readonly string[]; readonly rows: readonly string[][] };
-
-/** The cells of a `│ a │ b │` row, trimmed; undefined when the line is not a table row. */
-function tableRowCells(line: string): string[] | undefined {
-	const plain = plainText(line).trim();
-	if (plain.length < 2 || !plain.startsWith("│") || !plain.endsWith("│")) return undefined;
-	return plain
-		.slice(1, -1)
-		.split("│")
-		.map(cell => cell.trim());
-}
+/** One ordered piece of a cell's output: printed text, or a view shown in place of the text it matched. */
+export type EvalOutputPart = { kind: "text"; text: string } | { kind: "view"; view: EvalView };
 
 /**
- * Split `console.table` blocks (`┌…┐` header `├…┤` rows `└…┘`, as Node/Bun
- * print them) out of cell output so they render as a `table`; anything that
- * does not parse as one stays text.
+ * Split `output` around its views, in order: each view's `text` is searched
+ * from the end of the previous match, unmatched entries are skipped, and
+ * blank text between views is dropped (newlines bordering a view trimmed).
  */
-function splitConsoleTables(output: string): EvalOutputPart[] {
-	const lines = output.split("\n");
+export function splitEvalOutput(output: string, views: readonly EvalViewEntry[] | undefined): EvalOutputPart[] {
 	const parts: EvalOutputPart[] = [];
-	let textStart = 0;
-	const flushText = (end: number): void => {
-		const chunk = lines
-			.slice(textStart, end)
-			.join("\n")
-			.replace(/^\n+|\s+$/g, "");
-		if (chunk.length > 0) parts.push({ kind: "text", text: chunk });
+	const pushText = (chunk: string, trimStart: boolean, trimEnd: boolean): void => {
+		let text = chunk;
+		if (trimStart) text = text.replace(/^\n+/, "");
+		if (trimEnd) text = text.replace(/\n+$/, "");
+		if (text.trim().length > 0) parts.push({ kind: "text", text });
 	};
-	for (let i = 0; i < lines.length; i++) {
-		if (!plainText(lines[i]!).trimStart().startsWith("┌")) continue;
-		const head = tableRowCells(lines[i + 1] ?? "");
-		if (
-			!head ||
-			!plainText(lines[i + 2] ?? "")
-				.trimStart()
-				.startsWith("├")
-		)
-			continue;
-		const rows: string[][] = [];
-		let end = i + 3;
-		for (; end < lines.length; end++) {
-			const cells = tableRowCells(lines[end]!);
-			if (!cells) break;
-			rows.push(cells);
-		}
-		if (
-			!plainText(lines[end] ?? "")
-				.trimStart()
-				.startsWith("└")
-		)
-			continue;
-		if (rows.some(row => row.length !== head.length)) continue;
-		flushText(i);
-		parts.push({ kind: "table", head, rows });
-		i = end;
-		textStart = end + 1;
+	let cursor = 0;
+	for (const entry of views ?? []) {
+		if (!entry.text) continue;
+		const at = output.indexOf(entry.text, cursor);
+		if (at < 0) continue;
+		pushText(output.slice(cursor, at), cursor > 0, true);
+		parts.push({ kind: "view", view: entry.view });
+		cursor = at + entry.text.length;
 	}
-	flushText(lines.length);
+	pushText(output.slice(cursor), cursor > 0, cursor > 0);
 	return parts;
 }
 
+/** A table view as a native `table`; numeric columns align to the end. */
+function evalTableNode(view: Extract<EvalView, { kind: "table" }>, key: string): NativeNode {
+	return node(
+		"table",
+		{
+			cols: view.columns.map((column, c) => ({
+				id: `c${c}`,
+				head: column.head,
+				align: column.numeric ? "end" : undefined,
+			})),
+			rows: view.rows.map((cells, r) => ({
+				id: `r${r}`,
+				cells: Object.fromEntries(cells.map((cell, c) => [`c${c}`, cell])),
+			})),
+			role: "omp.tool.eval.table",
+		},
+		undefined,
+		key,
+	);
+}
+
 /**
- * One cell's outputs: markdown, else terminal text (following its tail,
- * clamped to the last `previewLines` while folded) with any `console.table`
- * blocks as tables.
+ * One cell's outputs in order: printed text (following its tail, clamped to
+ * the last `previewLines` while folded) with each view in place of its text.
  */
-function evalOutputNodes(output: string, markdown: boolean, previewLines: number): NativeNode[] {
+function evalOutputNodes(
+	output: string,
+	views: readonly EvalViewEntry[] | undefined,
+	previewLines: number,
+): NativeNode[] {
 	if (output.trim().length === 0) return [];
-	if (markdown) return [keyed(md(output), "output")];
-	return splitConsoleTables(output).map((part, i) =>
-		part.kind === "text"
-			? runOutput(part.text, { role: "omp.tool.eval.output", previewLines, key: i === 0 ? "output" : `output-${i}` })
-			: node(
-					"table",
-					{
-						// Node's index column header is noise in a real table.
-						cols: part.head.map((head, c) => ({ id: `c${c}`, head: head === "(index)" ? "" : head })),
-						rows: part.rows.map((cells, r) => ({
-							id: `r${r}`,
-							cells: Object.fromEntries(cells.map((cell, c) => [`c${c}`, cell])),
-						})),
-						role: "omp.tool.eval.table",
-					},
-					undefined,
-					`table-${i}`,
-				),
-	);
+	return splitEvalOutput(output, views).map((part, i) => {
+		if (part.kind === "text") {
+			return runOutput(part.text, {
+				role: "omp.tool.eval.output",
+				previewLines,
+				key: i === 0 ? "output" : `output-${i}`,
+			});
+		}
+		const view = part.view;
+		if (view.kind === "table") return evalTableNode(view, `table-${i}`);
+		if (view.kind === "json") return keyed(describeJsonTree(view.value, { hiddenRootKeys: [] }), `json-${i}`);
+		return keyed(md(view.text), `markdown-${i}`);
+	});
 }
 
 /** Status lines a folded cell keeps: the newest ones. */
@@ -958,18 +988,6 @@ export const evalToolRenderer = {
 		// the styled `warningLine` below carries the same text in ⟨…⟩ form.
 		const output = stripOutputNotice(rawOutput, details?.meta).trimEnd();
 
-		const jsonOutputs = details?.jsonOutputs ?? [];
-		const treeExpanded = options.renderContext?.expanded ?? options.expanded;
-		const treeDepth = treeExpanded ? JSON_TREE_MAX_DEPTH_EXPANDED : JSON_TREE_MAX_DEPTH_COLLAPSED;
-		const treeLineCap = treeExpanded ? JSON_TREE_MAX_LINES_EXPANDED : JSON_TREE_MAX_LINES_COLLAPSED;
-		const treeScalarLen = treeExpanded ? JSON_TREE_SCALAR_LEN_EXPANDED : JSON_TREE_SCALAR_LEN_COLLAPSED;
-		const labelOutputs = jsonOutputs.length > 1;
-		const jsonLines = jsonOutputs.flatMap((value, index) => {
-			const tree = renderJsonTreeLines(value, uiTheme, treeDepth, treeLineCap, treeScalarLen);
-			const body = tree.truncated ? [...tree.lines, uiTheme.fg("dim", "…")] : tree.lines;
-			return labelOutputs ? [uiTheme.fg("dim", `display[${index + 1}]`), ...body] : body;
-		});
-
 		const timeoutSeconds = options.renderContext?.timeout;
 		const timeoutLine =
 			typeof timeoutSeconds === "number"
@@ -1093,7 +1111,6 @@ export const evalToolRenderer = {
 							lines.push("");
 						}
 					}
-					lines.push(...jsonLines);
 					if (timeoutLine) {
 						lines.push(timeoutLine);
 					}
@@ -1116,9 +1133,6 @@ export const evalToolRenderer = {
 			});
 		}
 
-		const displayOutput = output;
-		const combinedOutput = [displayOutput, ...jsonLines].filter(Boolean).join("\n");
-
 		const statusEvents = details?.statusEvents ?? [];
 		const statusLines = renderStatusEvents(
 			statusEvents,
@@ -1127,12 +1141,12 @@ export const evalToolRenderer = {
 			details?.statusEventsElided,
 		);
 
-		if (!combinedOutput && statusLines.length === 0) {
+		if (!output && statusLines.length === 0) {
 			const lines = [timeoutLine, noticeLine, asyncLine, warningLine].filter(Boolean) as string[];
 			return new Text(lines.join("\n"), 0, 0);
 		}
 
-		if (!combinedOutput && statusLines.length > 0) {
+		if (!output && statusLines.length > 0) {
 			const lines = [
 				uiTheme.fg("dim", "Status"),
 				...statusLines,
@@ -1145,7 +1159,7 @@ export const evalToolRenderer = {
 		}
 
 		if (options.renderContext?.expanded ?? options.expanded) {
-			const styledOutput = combinedOutput
+			const styledOutput = output
 				.split("\n")
 				.map(line => uiTheme.fg("toolOutput", line))
 				.join("\n");
@@ -1160,7 +1174,7 @@ export const evalToolRenderer = {
 			return new Text(lines.join("\n"), 0, 0);
 		}
 
-		const styledOutput = combinedOutput
+		const styledOutput = output
 			.split("\n")
 			.map(line => uiTheme.fg("toolOutput", line))
 			.join("\n");
@@ -1250,9 +1264,6 @@ export const evalToolRenderer = {
 		const cancelled = options.cancelled === true;
 		const expanded = options.renderContext?.expanded ?? options.expanded;
 		const previewLines = options.renderContext?.previewLines ?? EVAL_DEFAULT_PREVIEW_LINES;
-		const jsonNodes = (details?.jsonOutputs ?? []).map((value, index) =>
-			keyed(describeJsonTree(value, { hiddenRootKeys: [] }), `display-${index}`),
-		);
 		// The call's notice and truncation join the last run's foot facts (a background job is a head badge).
 		const callFacts = { facts: compact([details?.notice]), meta: details?.meta };
 		const cellResults = details?.cells;
@@ -1278,15 +1289,7 @@ export const evalToolRenderer = {
 						language,
 						code: formatEvalCodeForDisplay(cell.code, language),
 						title: cell.title,
-						outputs: [
-							...evalOutputNodes(
-								cell.textOutput ?? cell.output,
-								cell.hasMarkdown === true && cell.status !== "error",
-								previewLines,
-							),
-
-							...(i === cellResults.length - 1 ? jsonNodes : []),
-						],
+						outputs: [...evalOutputNodes(cell.output, cell.views, previewLines)],
 						status: evalStatusSection(cell, expanded),
 						foot: state && {
 							state,
@@ -1311,10 +1314,7 @@ export const evalToolRenderer = {
 		const rawOutput = options.renderContext?.output ?? resultText(result).trimEnd();
 		const state: RunState = cancelled ? "cancelled" : isPartial ? "running" : result.isError ? "failed" : "done";
 		const last: Omit<EvalCellRun, "language" | "code" | "title"> = {
-			outputs: [
-				...evalOutputNodes(stripOutputNotice(rawOutput, details?.meta).trimEnd(), false, previewLines),
-				...jsonNodes,
-			],
+			outputs: evalOutputNodes(stripOutputNotice(rawOutput, details?.meta).trimEnd(), undefined, previewLines),
 			status: evalStatusSection(details ?? {}, expanded),
 			foot: { state, elapsedMs: options.elapsedMs, ...callFacts },
 		};

package/src/tools/find.ts

@@ -37,7 +37,6 @@ import type {
 import type { TspTone } from "@oh-my-pi/pi-wire";
 import { code, col, compact, keyed, node, span, text } from "../native/describe";
 import type { NativeChild, NativeNode } from "../native/node";
-import { getLanguageFromPath } from "../lang-from-path";
 import { fileHref, fileRow, inlineErrorView } from "./native-view";
 import { splitUrlScheme } from "./url-scheme-host";
 
@@ -214,10 +213,14 @@ function describeHit(hit: FindHit, rangeLimit: number, cwd: string | undefined,
 		href: isUrlHit || cwd === undefined ? undefined : fileHref(path.join(cwd, rawRel)),
 		key: "hit",
 	});
-	const lang = getLanguageFromPath(hit.rel);
 	const snippets = ranges.map(range =>
 		keyed(
-			code(range.snippet.trimEnd(), { lang, start: range.start, numbers: true, title: range.p.toFixed(2) }),
+			code(range.snippet.trimEnd(), {
+				path: hit.rel,
+				start: range.start,
+				numbers: true,
+				title: range.p.toFixed(2),
+			}),
 			`${range.start}-${range.end}`,
 		),
 	);

package/src/tools/grep.ts

@@ -9,7 +9,6 @@ import type {
 	RenderResultOptions,
 	ToolRenderer,
 } from "./renderer";
-import { getLanguageFromPath } from "../lang-from-path";
 import { compact } from "../native/describe";
 import type { NativeChild } from "../native/node";
 import { footnoteText, inlineErrorView, resultText } from "./native-view";
@@ -557,7 +556,6 @@ export const grepToolRenderer = {
 			inline: true,
 			body: compact<NativeChild>([
 				...describeGroupedOutput(textContent.split("\n"), {
-					lang: getLanguageFromPath,
 					maxFiles: options.expanded ? undefined : NATIVE_COLLAPSED_FILES,
 					file: details?.files?.length === 1 ? details.files[0] : undefined,
 				}),

package/src/tools/grouped-file-output.ts

@@ -314,12 +314,13 @@ export interface NumberedLine {
  * Numbered `code` for possibly non-contiguous source lines: one block per run
  * of consecutive line numbers (`start` = the run's first line, match lines
  * `mark`ed), with a `…` row wherever the numbers jump or an elision marker
- * (`n: null`) sits. No path header: the row or head above names the file.
+ * (`n: null`) sits. `path` is the file the lines come from: Tern picks their
+ * grammar from it and draws no header (the row or head above names the file).
  * `wrap` soft-wraps long lines instead of scrolling the block sideways.
  */
 export function numberedCode(
 	lines: readonly NumberedLine[],
-	opts: { lang?: string; wrap?: boolean } = {},
+	opts: { path?: string; wrap?: boolean } = {},
 ): NativeNode[] {
 	const out: NativeNode[] = [];
 	let run: { start: number; text: string[]; marks: { line: number; tone: TspTone }[] } | undefined;
@@ -332,7 +333,7 @@ export function numberedCode(
 				code(run.text.join("\n"), {
 					start: run.start,
 					numbers: true,
-					lang: opts.lang,
+					path: opts.path,
 					wrap: opts.wrap,
 					marks: run.marks.length > 0 ? run.marks : undefined,
 				}),
@@ -367,7 +368,7 @@ export function numberedCode(
  */
 export function describeGroupedOutput(
 	lines: readonly string[],
-	options: { lang?: (path: string) => string | undefined; maxFiles?: number; file?: string } = {},
+	options: { maxFiles?: number; file?: string } = {},
 ): NativeNode[] {
 	const files: NativeNode[] = [];
 	const loose: NativeNode[] = [];
@@ -380,7 +381,7 @@ export function describeGroupedOutput(
 	const flushLines = () => {
 		const target = current?.children ?? loose;
 		if (numbered.length > 0) {
-			target.push(...numberedCode(numbered, { lang: current ? options.lang?.(current.path) : undefined }));
+			target.push(...numberedCode(numbered, { path: current?.path }));
 			numbered = [];
 		}
 		if (plain.length > 0) {

package/src/tools/read.ts

@@ -431,12 +431,12 @@ export function readContentCode(
 	content: string,
 	start: number,
 	numbers: readonly (number | null)[] | undefined,
-	lang: string | undefined,
+	path: string | undefined,
 ): NativeNode[] {
 	const lines = content.split("\n");
 	return numberedCode(
 		lines.map((text, i) => ({ n: numbers && numbers.length === lines.length ? numbers[i]! : start + i, text })),
-		{ lang, wrap: true },
+		{ path, wrap: true },
 	);
 }
 
@@ -766,7 +766,7 @@ export const readToolRenderer = {
 					contentText,
 					details?.displayContent?.startLine ?? readRangeStart(rawPath, args),
 					details?.displayContent?.lineNumbers,
-					getLanguageFromPath(renderPath.path),
+					renderPath.path,
 				);
 		return { tool: head, inline: true, body: compact<NativeChild>([media, ...content, foot]) };
 	},

package/src/tools/write.ts

@@ -164,10 +164,12 @@ function describeFigure(figure: ToolFigure | undefined): NativeNode | undefined
 	return { ...md(text, { role: "omp.tool.write.figure", stream: !figure.closed }), key: "figure" };
 }
 
-/** The written content as numbered code, keyed so it keeps its node as the figure above it comes and goes. */
+/**
+ * The written content as numbered code, highlighted by Tern from the file's path,
+ * keyed so it keeps its node as the figure above it comes and goes.
+ */
 function describeSource(rawPath: string, content: string): NativeNode {
-	const lang = rawPath ? getLanguageFromPath(rawPath) : undefined;
-	return { ...code(content, { lang, numbers: true, wrap: true }), key: "source" };
+	return { ...code(content, { path: rawPath || undefined, numbers: true, wrap: true }), key: "source" };
 }
 
 function countLines(text: string): number {