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

package/CHANGELOG.md

@@ -2,6 +2,36 @@
 
 ## [Unreleased]
 
+## [18.12.2] - 2026-10-18
+
+### Added
+
+- Added `ComposerPreferences.completeScrollback` (default `false`): rows a growing HUD or dialog pushes off screen go straight to scrollback instead of being held back until it shrinks ([#15692](https://github.com/can1357/oh-my-pi/pull/15692) by [@Ilm-Alan](https://github.com/Ilm-Alan))
+
+### Fixed
+
+- Fixed table charts picking the wrong label and measure columns when numbers group thousands with spaces (`1 988`, `17 083`); they now read like `1,988` ([#16175](https://github.com/can1357/oh-my-pi/issues/16175))
+- Fixed terminal capability replies relayed through win32-input-mode appearing as typed input in WSL terminals ([#16137](https://github.com/can1357/oh-my-pi/issues/16137))
+- Fixed the status-bar activity spinner and turn timer freezing until the next keypress when the working-row loader is stopped mid-turn; an open turn window now repaints the status line on its own timer ([#16109](https://github.com/can1357/oh-my-pi/pull/16109) by [@tommymorgan](https://github.com/tommymorgan)).
+- Fixed pasting sometimes inserting a block of base64 text instead of the pasted text in terminals that deliver pastes through the Kitty clipboard protocol (OSC 5522), most often with dictation apps that paste for you; a slow paste now arrives whole, and keys typed while a paste never finishes are given back instead of lost ([#16170](https://github.com/can1357/oh-my-pi/pull/16170) by [@H4vC](https://github.com/H4vC))
+- Fixed a streaming table bouncing everything below it up and down while its last cell was still being written ([#16116](https://github.com/can1357/oh-my-pi/pull/16116) by [@galtamam-cmyk](https://github.com/galtamam-cmyk))
+- Fixed inline images in iTerm2 being sized for the wrong cell shape, which left charts smeared or offset ([#16118](https://github.com/can1357/oh-my-pi/pull/16118) by [@galtamam-cmyk](https://github.com/galtamam-cmyk))
+
+## [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
@@ -2447,79 +2477,4 @@
 
 - Increased the minimum required Bun version for the TUI package from >=1.3.7 to >=1.3.14
 
-## [14.9.8] - 2026-05-12
-
-### Added
-
-- Added `Terminal.setProgress(active)` to emit OSC 9;4 progress sequences with a ~1s keepalive interval so Ghostty does not clear the indicator during long-running work (ports pi-mono `a900d251` + `76bc605a`)
-- Added optional `argumentHint?: string` to `SlashCommand`; rendered before the description in the autocomplete dropdown (ports pi-mono `aa25726e`)
-- Added `VirtualTerminal.waitForRender()` test helper for the throttled render pipeline (ports pi-mono `41377ee8`)
-
-### Changed
-
-- `ProcessTerminal` `columns`/`rows` getters consult `Bun.env.COLUMNS` / `Bun.env.LINES` before falling back to 80×24, so piped/non-TTY runs honour environment-provided dimensions (ports pi-mono `32f7fc6a`)
-- `requestRender()` non-force calls are coalesced to a ~16ms frame budget; `requestRender(true)` still flushes immediately via `process.nextTick` (ports pi-mono `6f5f37f8`)
-- `KNOWN_TERMINALS.base` / `KNOWN_TERMINALS.trueColor` default `hyperlinks: false`; tmux and screen (`TMUX` env or `TERM` starts with `tmux`/`screen`) force `hyperlinks: false` even when the outer terminal would advertise OSC 8 (adapts pi-mono `30a8a41f`)
-- `SlashCommand.getArgumentCompletions()` may return a `Promise`; results are now awaited and non-array returns are ignored (ports pi-mono `a1e10789`)
-- Fuzzy `@` autocomplete now follows symlinked directories via `ScanOptions.follow_links` plumbed through the native walker (ports pi-mono `780d5367`)
-- Plain `@<query>` (no slash) fuzzy matches by basename only, so `@plan` no longer surfaces every file whose ancestor directories contain `plan` (ports pi-mono `968430f6`)
-- Changed slash-command autocomplete list rendering to combine command hint and description in a single displayed suggestion text
-- Changed render scheduling to throttle `requestRender` calls to roughly 60fps by batching updates
-- Changed terminal input handling to process complete cell-size responses without buffering partial input
-- Changed `KeyId` to accept super-modifier combinations and improve typed key-id validation
-
-### Fixed
-
-- Fixed editor corruption on Thai Sara Am (U+0E33) and Lao AM (U+0EB3) vowels by normalizing to their compatibility decompositions on the terminal-write path while keeping editor content logically unchanged (ports pi-mono `bc668826` + `338ce3a3` + `20ca45d5`)
-- Fixed cell-size detection (`CSI 6;h;w t` response) to consume only exact replies, so a bare `Escape` keystroke is no longer swallowed while waiting for terminal image metadata (ports pi-mono `49c0d860`)
-- Fixed Kitty CSI-u printable input duplicating on layouts (e.g. Italian) where the terminal also emits the raw character: the immediately-following matching codepoint is now suppressed (ports pi-mono `bdb416cb`)
-- Fixed bracketed-paste CSI-u `Ctrl+<letter>` re-encoding (tmux popup with `extended-keys-format=csi-u`) leaking literal `[<code>;5u` into the editor; control bytes are decoded back to their literal byte before per-char filtering (ports pi-mono `d06db09a`)
-- Fixed xterm `modifyOtherKeys` shifted printable input so uppercase letters inserted via `CSI 27;mod;codepoint~` reach the editor correctly (ports pi-mono `6b55d685`)
-- Fixed `super`-modified Kitty shortcuts (`super+k`, `ctrl+super+enter`, …) to parse and match via the new `KITTY_MOD_SUPER` mask (ports pi-mono `ddb8454c` + `5ed46003`)
-- Fixed `ctrl+alt+<letter>` in tmux falling through to CSI-u / `modifyOtherKeys` when the legacy `ESC<ctrl-char>` form does not match (ports pi-mono `6cf5098f`)
-- Fixed Markdown strikethrough requiring strict `~~text~~` delimiters with non-whitespace boundaries; single tildes no longer render strikethrough (ports pi-mono `db5274b4`)
-- Allowed `SlashCommand.getArgumentCompletions` to return asynchronous results by accepting Promise-based completions
-- Added `argumentHint` support to slash command definitions and displayed it in command suggestion descriptions
-- Added support for xterm `modifyOtherKeys` printable key sequences by decoding `CSI 27;mod;key~` into text input
-- Normalized line output during rendering to correct Thai/Lao AM glyph composition for displayed text
-- Fixed duplicated Kitty key input emissions by dropping the matching unmodified follow-up sequence after a Kitty CSI-u printable-key event
-
-## [14.9.5] - 2026-05-12
-
-### Fixed
-
-- Fixed rapidly blinking cursor artifact during task execution by consolidating cursor control sequences into the synchronized output buffer ([#992](https://github.com/can1357/oh-my-pi/issues/992))
-
-## [14.5.7] - 2026-04-29
-
-### Fixed
-
-- Fixed editor Ctrl+Enter handling to recognize NumLock and keypad Enter variants.
-
-## [14.3.0] - 2026-04-25
-
-### Fixed
-
-- Fixed shared Markdown Mermaid fenced-block rendering to resolve diagrams from fenced source text instead of external prerender state
-
-## [14.1.1] - 2026-04-14
-
-### Breaking Changes
-
-- Removed the `searchDb` constructor argument from `CombinedAutocompleteProvider`, requiring callers to use the built-in search behavior
-
-### Changed
-
-- 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@04d4ab14b45f](https://github.com/can1357/oh-my-pi/blob/04d4ab14b45f8dbd4055b675519f8815a1eb1841/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/chrome/status-notice.d.ts

@@ -21,6 +21,8 @@ export interface StatusNoticeOptions {
 export declare class StatusNotice extends Container {
     #private;
     constructor(message: string, options?: StatusNoticeOptions);
+    /** The text the notice currently shows. */
+    get message(): string;
     setMessage(message: string, options?: StatusNoticeOptions): void;
     describe(): NativeNode;
 }

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

@@ -14,6 +14,8 @@ export interface ComposerPreferences {
     readonly showHardwareCursor: boolean;
     readonly maxInlineImages: number;
     readonly resizeScrollback: ResizeScrollbackMode;
+    /** Send rows a growing HUD or dialog pushes off screen straight to scrollback instead of holding them back until it shrinks. */
+    readonly completeScrollback: boolean;
     readonly imeSafeCursor: boolean;
     readonly autocompleteMaxVisible: number;
     readonly spellingTypoDetection: boolean;

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/dist/types/tools/web-search-types.d.ts

@@ -25,6 +25,7 @@ export declare const SEARCH_PROVIDER_LABELS: {
     readonly tavily: "Tavily";
     readonly firecrawl: "Firecrawl";
     readonly brave: "Brave";
+    readonly deepseek: "DeepSeek";
     readonly kimi: "Kimi";
     readonly synthetic: "Synthetic";
     readonly ollama: "Ollama";

package/package.json

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

package/src/charts/table-data.ts

@@ -148,7 +148,17 @@ export interface TableAnalysis {
 	readonly composition?: Composition;
 }
 
-const NUMBER = String.raw`(?:\d{1,3}(?:,\d{3})+|\d+)(?:\.\d+)?(?:[eE][+-]?\d+)?|\.\d+`;
+/**
+ * A whole number as written: bare, or grouped in thousands by commas (`1,988`)
+ * or by spaces as many locales write it (`1 988`, also U+00A0 and U+202F).
+ * Space groups take exactly three digits, so `12 5` and `128 2560` stay apart.
+ */
+const INTEGER = String.raw`\d{1,3}(?:,\d{3})+|\d{1,3}(?:[ \u00A0\u202F]\d{3})+(?!\d)|\d+`;
+const NUMBER = String.raw`(?:${INTEGER})(?:\.\d+)?(?:[eE][+-]?\d+)?|\.\d+`;
+/** A cell holding one whole number and nothing else (`38`, `1,980`). */
+const WHOLE = new RegExp(String.raw`^(?:${INTEGER})$`);
+/** The first number of a figure, its whole part and its decimals apart. */
+const WRITTEN = new RegExp(String.raw`(${INTEGER})(?:\.(\d+))?`);
 const CELL = new RegExp(
 	String.raw`^(?<approx>~|≈|<=|>=|<|>|≤|≥)?\s*(?<sign>[+\-−±])?\s*(?<cur>[$€£¥])?\s*(?<num>${NUMBER})(?<plus>\+(?=\s*[A-Za-zµμ%]))?\s*(?<unit>%|[A-Za-zµμ×]+(?:\/[A-Za-z]+)?|\/s)?(?<rest>.*)$`,
 );
@@ -517,8 +527,9 @@ export function parseCell(markdown: string): Cell {
 	return { ...cell, fit: "prose" };
 }
 
+/** The value of a number {@link NUMBER} matched, its comma or space group separators removed. */
 function toNumber(written: string): number {
-	return Number(written.replaceAll(",", ""));
+	return Number(written.replace(/[,\s]/g, ""));
 }
 
 /** A number of a list or sum: its value as written, its unit (known or a noun) and how it was marked. */
@@ -632,7 +643,7 @@ function listCell(text: string, base: CellBase): NumberCell | undefined {
 	// A spaced pair of bare numbers that is no score (`25.0 / 27.8`, `12.6 / 9.4`: the second is no whole, or
 	// the first exceeds it) lists two values of the column's quantity; `27 / 30` stays a score.
 	const [first, second] = items;
-	const whole = /^\d[\d,]*$/.test(text.slice(text.indexOf("/") + 1, end).trim());
+	const whole = WHOLE.test(text.slice(text.indexOf("/") + 1, end).trim());
 	const score = whole && first!.value >= 0 && first!.value <= second!.value;
 	const pair = items.length === 2 && !marked.some(Boolean) && separators[0] !== "/" && !score;
 	if (!pair && (!marked.at(-1) || !(marked.every(Boolean) || carried))) return undefined;
@@ -876,9 +887,9 @@ const WHOLE_HEADER = /\b(?:total|all|sum|overall)\b/i;
  */
 export function writtenSlack(cell: NumberCell): number {
 	if (cell.denominator !== undefined && cell.dim === "percent") return 0;
-	const match = /(\d[\d,]*)(?:\.(\d+))?/.exec(cell.figure);
+	const match = WRITTEN.exec(cell.figure);
 	if (!match) return 0;
-	const written = Number(`${match[1]!.replaceAll(",", "")}${match[2] ? `.${match[2]}` : ""}`);
+	const written = toNumber(`${match[1]!}${match[2] ? `.${match[2]}` : ""}`);
 	const factor = written ? Math.abs(cell.value) / written : 1;
 	const decimals = match[2]?.length ?? 0;
 	if (cell.dim === "count" && decimals === 0 && factor === 1 && !cell.approx) return 0;
@@ -1097,7 +1108,7 @@ const MIN_LISTS = 2;
 /** A spaced slash between list items. */
 const LIST_SLASH = /\s+\/\s+/;
 /** A list item that is one number, as written (`12`, `~1,400`, `3.5 ms`, `40%`). */
-const LISTED_NUMBER = /^[~≈]?[$€£¥]?\d[\d,]*(?:\.\d+)?\s*(?:%|[A-Za-zµμ]+)?$/;
+const LISTED_NUMBER = new RegExp(String.raw`^[~≈]?[$€£¥]?(?:${INTEGER})(?:\.\d+)?\s*(?:%|[A-Za-zµμ]+)?$`);
 
 /**
  * Rows folding several rows into one (`response anthropic / chat / responses /

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,