@oh-my-pi/pi-tui 18.6.3 → 18.8.0

package/src/apps/git/diff-pane.ts

@@ -21,7 +21,7 @@ import { centerLine, sliceWithWidth, truncateToWidth, visibleWidth } from "../..
 import { formatBytes } from "@oh-my-pi/pi-utils";
 import { sanitizeDisplayText } from "../../overlays/extensions/display-text";
 import { getLanguageFromPath } from "../../lang-from-path";
-import { createHighlightStream, theme } from "../../theme/theme";
+import { createHighlightStream, getThemeEpoch, theme } from "../../theme/theme";
 import { bgAnsiHex, canvasHex, fgAnsiHex, mixHex, pill, selectionBgAnsi, textHex, withBg } from "./colors";
 import { DIFF_CONTEXT_LINES, type FileAssetSide, type FileStreamUpdate } from "./state";
 
@@ -502,9 +502,12 @@ interface DiffPalette {
 	gutterDel: string;
 }
 
-let paletteCache: { key: string; palette: DiffPalette } | undefined;
+let paletteCache: { epoch: number; theme: object; palette: DiffPalette } | undefined;
 
+/** Diff colors for the active theme, rebuilt only when the theme instance or epoch changes. */
 function palette(): DiffPalette {
+	const epoch = getThemeEpoch();
+	if (paletteCache?.epoch === epoch && paletteCache.theme === theme) return paletteCache.palette;
 	const added = theme.getColorHex("toolDiffAdded");
 	const removed = theme.getColorHex("toolDiffRemoved");
 	const accent = theme.getColorHex("accent");
@@ -512,8 +515,6 @@ function palette(): DiffPalette {
 	const dark = luminance === undefined || luminance <= 0.5;
 	const canvas = canvasHex();
 	const text = textHex();
-	const key = `${added}\u0000${removed}\u0000${accent}\u0000${dark}\u0000${canvas}\u0000${text}`;
-	if (paletteCache?.key === key) return paletteCache.palette;
 	const built: DiffPalette = {
 		addSoft: bgAnsiHex(mixHex(canvas, added, dark ? 0.18 : 0.24)),
 		addStrong: bgAnsiHex(mixHex(canvas, added, dark ? 0.42 : 0.48)),
@@ -529,7 +530,7 @@ function palette(): DiffPalette {
 		gutterAdd: added,
 		gutterDel: removed,
 	};
-	paletteCache = { key, palette: built };
+	paletteCache = { epoch, theme, palette: built };
 	return built;
 }
 
@@ -599,6 +600,10 @@ export class DiffPane {
 	#lastHeight = 1;
 	#lastWidth = 0;
 	#layoutCache: { key: string; visuals: Visual[] } | undefined;
+	/** Minimap column for (layout, height, scroll offset, palette). */
+	#minimapCache:
+		| { visuals: Visual[]; height: number; scrollTop: number; colors: DiffPalette; lines: string[] }
+		| undefined;
 	/** Per visible row: clickable hunk-button ranges recorded during render. */
 	#hits: ({ hunk: number; primary?: [number, number]; discard?: [number, number] } | undefined)[] = [];
 	/**
@@ -1488,6 +1493,22 @@ export class DiffPane {
 	}
 
 	#renderMinimap(visuals: Visual[], height: number, colors: DiffPalette): string[] {
+		const cached = this.#minimapCache;
+		if (
+			cached &&
+			cached.visuals === visuals &&
+			cached.height === height &&
+			cached.scrollTop === this.scrollTop &&
+			cached.colors === colors
+		) {
+			return cached.lines;
+		}
+		const lines = this.#buildMinimap(visuals, height, colors);
+		this.#minimapCache = { visuals, height, scrollTop: this.scrollTop, colors, lines };
+		return lines;
+	}
+
+	#buildMinimap(visuals: Visual[], height: number, colors: DiffPalette): string[] {
 		const total = visuals.length;
 		const lines: string[] = [];
 		const bandKind = (band: number): RowKind | "hunk" | null => {

package/src/autocomplete.ts

@@ -263,8 +263,11 @@ export interface AutocompleteProvider {
 
 	/** Get inline hint text to show as dim ghost text after the cursor */
 	getInlineHint?(lines: string[], cursorLine: number, cursorCol: number): string | null;
-	/** Synchronously try to complete a slash command at the start of a line (no async I/O). */
-	/** Returns matched items and the full prefix, or null if not applicable. */
+	/**
+	 * Synchronously list slash command-name completions for a leading slash token (no async I/O).
+	 * Mirrors the command-name branch of {@link getSuggestions}, including the bare `/` listing and
+	 * the collapsed `/skill:` namespace row. Returns null outside a command-name token or with no match.
+	 */
 	trySyncSlashCompletion?(textBeforeCursor: string): { items: AutocompleteItem[]; prefix: string } | null;
 	/**
 	 * Synchronously try to expand text immediately before the cursor (no async I/O).
@@ -589,6 +592,22 @@ function buildMidPromptSkillCompletions(commands: CommandEntry[], lowerPrefix: s
 	);
 }
 
+const DIR_CACHE_MAX = 100;
+const DIR_CACHE_EVICT = 50;
+
+/**
+ * Drops the `count` oldest entries once `map` exceeds `max`. Callers delete a
+ * key before re-setting it, so Map insertion order is refresh order.
+ */
+function evictOldest(map: Map<string, unknown>, max: number, count: number): void {
+	if (map.size <= max) return;
+	let remaining = count;
+	for (const key of map.keys()) {
+		if (remaining-- === 0) break;
+		map.delete(key);
+	}
+}
+
 // Combined provider that handles both slash commands and file paths.
 export class CombinedAutocompleteProvider implements AutocompleteProvider {
 	#commands: CommandEntry[];
@@ -598,6 +617,9 @@ export class CombinedAutocompleteProvider implements AutocompleteProvider {
 	// per-directory readdir fast-path for prefix completions. Global fuzzy
 	// discovery continues to use native fuzzyFind + shared scan cache.
 	#dirCache: Map<string, { entries: fs.Dirent[]; timestamp: number }> = new Map();
+	// Whether a scoped fuzzy query's base (`src/` in `@src/foo`) is a directory;
+	// stat'ing it on every keystroke is redundant while the base is unchanged.
+	#scopedBaseCache: Map<string, { isDirectory: boolean; timestamp: number }> = new Map();
 	readonly #DIR_CACHE_TTL = 2000; // 2 seconds
 
 	constructor(
@@ -806,7 +828,7 @@ export class CombinedAutocompleteProvider implements AutocompleteProvider {
 			};
 		}
 
-		// Slash command suggestions can be accepted before the debounced refresh
+		// Slash command suggestions can be accepted before an async refresh
 		// catches up to newly typed characters. Replace the live command token,
 		// not only the prefix captured when the suggestion list was rendered.
 		// Absolute-path completions share the leading-slash prefix shape but
@@ -976,11 +998,21 @@ export class CombinedAutocompleteProvider implements AutocompleteProvider {
 			baseDir = path.join(this.#basePath, displayBase);
 		}
 
-		try {
-			if (!(await fs.promises.stat(baseDir)).isDirectory()) {
-				return null;
+		const now = Date.now();
+		let cachedBase = this.#scopedBaseCache.get(baseDir);
+		if (!cachedBase || now - cachedBase.timestamp >= this.#DIR_CACHE_TTL) {
+			let isDirectory = false;
+			try {
+				isDirectory = (await fs.promises.stat(baseDir)).isDirectory();
+			} catch {
+				// Missing or inaccessible base: not a scope.
 			}
-		} catch {
+			cachedBase = { isDirectory, timestamp: now };
+			this.#scopedBaseCache.delete(baseDir);
+			this.#scopedBaseCache.set(baseDir, cachedBase);
+			evictOldest(this.#scopedBaseCache, DIR_CACHE_MAX, DIR_CACHE_EVICT);
+		}
+		if (!cachedBase.isDirectory) {
 			return null;
 		}
 
@@ -1003,17 +1035,9 @@ export class CombinedAutocompleteProvider implements AutocompleteProvider {
 		}
 
 		const entries = await fs.promises.readdir(searchDir, { withFileTypes: true });
+		this.#dirCache.delete(searchDir);
 		this.#dirCache.set(searchDir, { entries, timestamp: now });
-
-		if (this.#dirCache.size > 100) {
-			const sortedKeys = [...this.#dirCache.entries()]
-				.sort((a, b) => a[1].timestamp - b[1].timestamp)
-				.slice(0, 50)
-				.map(([key]) => key);
-			for (const key of sortedKeys) {
-				this.#dirCache.delete(key);
-			}
-		}
+		evictOldest(this.#dirCache, DIR_CACHE_MAX, DIR_CACHE_EVICT);
 
 		return entries;
 	}
@@ -1021,8 +1045,10 @@ export class CombinedAutocompleteProvider implements AutocompleteProvider {
 	invalidateDirCache(dir?: string): void {
 		if (dir) {
 			this.#dirCache.delete(dir);
+			this.#scopedBaseCache.delete(dir);
 		} else {
 			this.#dirCache.clear();
+			this.#scopedBaseCache.clear();
 		}
 	}
 
@@ -1087,26 +1113,29 @@ export class CombinedAutocompleteProvider implements AutocompleteProvider {
 			const entries = await this.#getCachedDirEntries(searchDir);
 			const suggestions: AutocompleteItem[] = [];
 
-			for (const entry of entries) {
-				if (!entry.name.toLowerCase().startsWith(searchPrefix.toLowerCase())) {
-					continue;
-				}
-				// Skip .git directory
-				if (entry.name === ".git") {
-					continue;
-				}
+			const lowerSearchPrefix = searchPrefix.toLowerCase();
+			// Skip .git directory
+			const matched = entries.filter(
+				entry => entry.name !== ".git" && entry.name.toLowerCase().startsWith(lowerSearchPrefix),
+			);
+			// Directory flag per match; symlinks are stat'ed concurrently to learn whether
+			// they point at a directory. `null` drops a broken symlink, a file deleted
+			// between readdir and stat, or a permission error.
+			const directoryFlags = await Promise.all(
+				matched.map(entry =>
+					!entry.isDirectory() && entry.isSymbolicLink()
+						? fs.promises.stat(path.join(searchDir, entry.name)).then(
+								stats => stats.isDirectory(),
+								() => null,
+							)
+						: entry.isDirectory(),
+				),
+			);
 
-				// Check if entry is a directory (or a symlink pointing to a directory)
-				let isDirectory = entry.isDirectory();
-				if (!isDirectory && entry.isSymbolicLink()) {
-					try {
-						const fullPath = path.join(searchDir, entry.name);
-						isDirectory = (await fs.promises.stat(fullPath)).isDirectory();
-					} catch {
-						// Broken symlink, file deleted between readdir and stat, or permission error
-						continue;
-					}
-				}
+			for (let index = 0; index < matched.length; index++) {
+				const isDirectory = directoryFlags[index];
+				if (typeof isDirectory !== "boolean") continue;
+				const entry = matched[index]!;
 
 				let relativePath: string;
 				const name = entry.name;
@@ -1300,19 +1329,14 @@ export class CombinedAutocompleteProvider implements AutocompleteProvider {
 		const slashStart = findLeadingSlashCommandStart(textBeforeCursor);
 		if (slashStart === null) return null;
 		const commandText = textBeforeCursor.slice(slashStart);
-		if (commandText.length <= 1) return null; // Bare "/" alone, don't auto-complete
 		if (commandText.includes(" ")) return null; // Only complete command name, not args
 
-		const prefix = commandText.slice(1);
-		const lowerPrefix = prefix.toLowerCase();
-
-		// The `/skill:` namespace row is excluded here: the sync path submits
-		// immediately after applying, and the bare namespace is not a command.
+		const lowerPrefix = commandText.slice(1).toLowerCase();
 		const matches = buildSlashCommandCompletions(
 			collapseSkillNamespace(this.#commands, lowerPrefix),
 			lowerPrefix,
 			this.#commandUsage,
-		).filter(item => item.value !== SKILL_NAMESPACE);
+		);
 
 		if (matches.length === 0) return null;
 		// Mirror `getSuggestions`: preserve leading whitespace so the editor's

package/src/charts/chart-plan.ts

@@ -0,0 +1,367 @@
+/**
+ * Which chart a table becomes and the data it plots. {@link planChart} is the
+ * local best guess over a {@link TableAnalysis}: the chart kind, the label
+ * column and the measure columns to draw. A host may substitute its own
+ * {@link ChartPlan} (a model's pick for a multi-series table); either way
+ * {@link buildChart} turns it into the {@link ChartSpec} that
+ * {@link renderChartSvg} draws, and {@link worthCharting} decides whether
+ * the picture says more than the table already does.
+ *
+ * The heuristics were fitted on ~7k tables from real assistant answers: the
+ * label is the first text column (index columns like `#`/`Rank` never are),
+ * table order is kept (most tables are deliberately unsorted), total rows stay
+ * out of the scale, and units never share an axis.
+ */
+import {
+	type Dimension,
+	isMonotonic,
+	isNumber,
+	type NumberCell,
+	type TableAnalysis,
+	type TableColumn,
+} from "./table-data";
+
+/**
+ * Chart kinds, by the table shapes they fit:
+ * - `bar`: one measure across categories.
+ * - `paired`: two same-unit measures per category (before/after, A vs B), or `a → b` cells.
+ * - `grouped`: three or four same-unit measures per category.
+ * - `line`: measures over an ordered axis (dates, runs, sizes).
+ * - `heatmap`: a matrix of same-unit values.
+ * - `multiples`: measures in different units, each scaled on its own; transposed when each row is its own metric.
+ * - `share`: parts of a whole (percentages summing to 100).
+ * - `diverging`: signed changes around zero.
+ * - `scatter`: two continuous measures per item.
+ */
+export type ChartKind =
+	| "bar"
+	| "paired"
+	| "grouped"
+	| "line"
+	| "heatmap"
+	| "multiples"
+	| "share"
+	| "diverging"
+	| "scatter";
+
+/** What to draw from a table: the kind, the column naming each category, and the columns plotted. */
+export interface ChartPlan {
+	readonly kind: ChartKind;
+	/** Column naming each category (or holding the x values of a line); rows are numbered without one. */
+	readonly label: number | undefined;
+	/** Plotted columns, in order. */
+	readonly series: readonly number[];
+	/** Each data row is its own series (a metric with its own unit) and the plotted columns are the categories. */
+	readonly transpose: boolean;
+}
+
+/** One plotted value. */
+export interface ChartPoint {
+	/** Value in base units (seconds, bytes, …). */
+	readonly value: number;
+	/** The number as the table wrote it (`~$250`, `12.5 ms`), shown as the value label. */
+	readonly text: string;
+	/** The table bolded the cell. */
+	readonly emphasis: boolean;
+	/** Upper end of a range cell (`24–72 h`). */
+	readonly upper?: number;
+}
+
+export interface ChartSeries {
+	readonly name: string;
+	readonly dim: Dimension;
+	/** Currency symbol or rate unit the axis prints; empty for other dimensions. */
+	readonly unit: string;
+	/** One per category; `null` where the cell holds no number of this series' dimension. */
+	readonly points: readonly (ChartPoint | null)[];
+	/** Status words standing in for a value (`TIMEOUT`, `OOM`), one per category. */
+	readonly notes: readonly (string | undefined)[];
+}
+
+/** The data a chart draws, independent of how it is drawn. */
+export interface ChartSpec {
+	readonly kind: ChartKind;
+	readonly categories: readonly string[];
+	readonly series: readonly ChartSeries[];
+	/** Numeric x positions of a line or the x measure of a scatter, one per category. */
+	readonly x?: {
+		readonly name: string;
+		readonly dim: Dimension;
+		readonly unit: string;
+		readonly values: readonly number[];
+	};
+	/** Total/summary rows, printed under the chart instead of skewing its scale. */
+	readonly caption?: string;
+	/** Name of the category axis (the label column's header). */
+	readonly axis: string;
+}
+
+const BEFORE =
+	/\b(?:before|old|baseline|base|main|master|prev|previous|original|orig|cold|was|from|v1|stock|default|unpatched|without)\b/i;
+const AFTER = /\b(?:after|new|patched|optimi[sz]ed|fixed|warm|now|to|v2|with|proposed|branch|pr|ours)\b/i;
+const DELTA = /(?:Δ|delta|change|diff|improvement|speedup|reduction|savings|saved|gain|regression|% chg)/i;
+/** Aggregate columns that restate the others; dropped when enough same-unit series remain. */
+const AGGREGATE = /^(?:total|sum|cumulative|cum\.?|running total|avg|average|mean)\b/i;
+/** Row labels that name an ordered step (`Run 3`, `Day 2`, `N=1000`, `Q3`). */
+const SEQUENCE_LABEL =
+	/^(?:run|day|week|month|iter(?:ation)?|round|step|phase|epoch|v|version|n\s*=|wave|attempt|pass|batch|q[1-4]|#)\s*\d/i;
+const MONTH = /^(?:jan|feb|mar|apr|may|jun|jul|aug|sep|oct|nov|dec)[a-z]*\b/i;
+/** Most panels a small-multiples grid stays legible with. */
+const MAX_PANELS = 6;
+/** Most lines one line chart keeps apart. */
+const MAX_LINES = 6;
+
+/** The local best guess at a chart for `table`, or `undefined` when no chart fits. */
+export function planChart(table: TableAnalysis): ChartPlan | undefined {
+	const { rows } = table;
+	let label = table.label;
+	let measures = dropAggregates(table.measures);
+	if (measures.length === 0 || rows.length < 2) return undefined;
+
+	let order = axisOrder(table, label);
+	// Without a name column, a monotonic leading measure (`Workers | p50 | p99`) is the
+	// x axis of the rest; beside one it is just a sorted measure (`% | Self | Function`).
+	if (!order && label?.role !== "label" && measures.length >= 2 && measures[0]!.index === 0 && rows.length >= 4) {
+		const xs = columnValues(measures[0]!, rows);
+		if (xs.length === rows.length && isMonotonic(xs)) {
+			label = measures[0];
+			measures = measures.slice(1);
+			order = "numeric";
+		}
+	}
+	const plan = (kind: ChartKind, series: readonly TableColumn[], transpose = false): ChartPlan => ({
+		kind,
+		label: label?.index,
+		series: series.map(column => column.index),
+		transpose,
+	});
+
+	const mixed = measures.filter(column => column.mixed);
+	if (mixed.length > 0 && mixed.length * 2 >= measures.length) return plan("multiples", measures, true);
+	if (measures.length === 1 && measures[0]!.arrowShare >= 0.6) return plan("paired", measures);
+
+	const dims = new Set(measures.map(column => column.dim));
+	if (dims.size === 1 && measures.length >= 2 && measures.length <= 4) {
+		const before = measures.find(column => BEFORE.test(column.header));
+		const after = measures.find(column => column !== before && AFTER.test(column.header));
+		if (before && after) return plan("paired", [before, after]);
+	}
+	if (rows.length === 2 && measures.length === 1) return undefined;
+	if (order && rows.length >= 4) {
+		const lead = measures[0]!.dim;
+		return plan("line", measures.filter(column => column.dim === lead).slice(0, MAX_LINES));
+	}
+	if (measures.length <= 2) {
+		const delta = measures.find(
+			column =>
+				(column.signedShare >= 0.6 || DELTA.test(column.header)) &&
+				columnValues(column, rows).some(value => value < 0),
+		);
+		if (delta) return plan("diverging", [delta]);
+	}
+	const share = measures.find(column => column.dim === "percent");
+	if (share && rows.length <= 10 && (measures.length === 1 || (measures.length === 2 && dims.size === 2))) {
+		const values = columnValues(share, rows);
+		const sum = values.reduce((total, value) => total + value, 0);
+		if (values.every(value => value >= 0) && sum >= 97 && sum <= 103) return plan("share", [share]);
+	}
+	if (dims.size === 1 && measures.length >= 4 && rows.length >= 3) return plan("heatmap", measures);
+	if (!label && measures.length === 2 && rows.length >= 6) return plan("scatter", measures);
+	if (dims.size >= 2) return plan("multiples", measures.slice(0, MAX_PANELS));
+	if (measures.length === 2) return plan("paired", measures);
+	if (measures.length >= 3) return plan("grouped", measures);
+	return plan("bar", measures);
+}
+
+/**
+ * The data `plan` draws from `table`, or `undefined` when it leaves fewer than
+ * two categories or no series with two values. The kind is fitted to the
+ * series it ends up with ({@link fitKind}), so any plan — a model's pick
+ * included — draws as something the renderer supports.
+ */
+export function buildChart(table: TableAnalysis, plan: ChartPlan): ChartSpec | undefined {
+	const label = plan.label === undefined ? undefined : table.columns[plan.label];
+	const columns = plan.series.map(index => table.columns[index]).filter(column => column !== undefined);
+	if (columns.length === 0) return undefined;
+	const rowName = (row: number, at: number) => (label ? label.cells[row]!.text : "") || `#${at + 1}`;
+
+	let categories: string[];
+	let series: ChartSeries[];
+	if (plan.transpose) {
+		categories = columns.map(column => column.header);
+		series = table.rows.map((row, at) => rowSeries(rowName(row, at), columns, row));
+	} else if (plan.kind === "paired" && columns.length === 1) {
+		// `a → b` cells: the column splits into its from and to values.
+		const column = columns[0]!;
+		const [from, to] = column.header.split(/\s*(?:→|->|⇒)\s*/);
+		const before = columnSeries(column, table.rows, from && to ? from : "before");
+		const after: ChartSeries = {
+			...before,
+			name: from && to ? to : "after",
+			points: table.rows.map(row => {
+				const cell = column.cells[row];
+				return isNumber(cell) && cell.to !== undefined
+					? { value: cell.to, text: cell.toFigure ?? "", emphasis: cell.emphasis }
+					: null;
+			}),
+		};
+		categories = table.rows.map(rowName);
+		series = [before, after];
+	} else {
+		categories = table.rows.map(rowName);
+		series = columns.map(column => columnSeries(column, table.rows, column.header));
+	}
+
+	let x: ChartSpec["x"];
+	if (plan.kind === "scatter" && series.length >= 2) {
+		const xs = series[0]!;
+		x = { name: xs.name, dim: xs.dim, unit: xs.unit, values: xs.points.map(entry => entry?.value ?? Number.NaN) };
+		series = [series[1]!];
+	} else if (plan.kind === "line" && label && !plan.transpose) {
+		const values = table.rows.map(row => label.cells[row]).map(cell => (isNumber(cell) ? cell.value : Number.NaN));
+		if (values.every(Number.isFinite) && isMonotonic(values)) {
+			const first = label.cells[table.rows[0]!];
+			x = {
+				name: label.header,
+				dim: isNumber(first) ? first.dim : "count",
+				unit: label.unit,
+				values,
+			};
+		}
+	}
+
+	series = series.filter(entry => entry.points.filter(point => point !== null).length >= 2);
+	if (categories.length < 2 || series.length === 0) return undefined;
+	const kind = plan.transpose ? "multiples" : fitKind(plan.kind, series, x !== undefined);
+	// A share or a delta reads one series; the rest would overdraw it.
+	if (kind === "share" || kind === "diverging") series = series.slice(0, 1);
+	return {
+		kind,
+		categories,
+		series,
+		x,
+		caption: totalsCaption(table, label, columns, plan.transpose),
+		axis: label?.header ?? "",
+	};
+}
+
+/**
+ * Whether a chart of `spec` reads faster than its table: at least four
+ * categories, and either nine plotted values or a 3× spread within a series.
+ * On a model-judged sample of assistant tables this gate kept 85% of the
+ * tables a chart clearly helped while rejecting most two- and three-row ones.
+ */
+export function worthCharting(spec: ChartSpec): boolean {
+	if (spec.categories.length < 4) return false;
+	let points = 0;
+	let spread = 1;
+	for (const entry of spec.series) {
+		const positive: number[] = [];
+		for (const point of entry.points) {
+			if (!point) continue;
+			points++;
+			if (point.value > 0) positive.push(point.value);
+		}
+		if (positive.length >= 2) spread = Math.max(spread, Math.max(...positive) / Math.min(...positive));
+	}
+	return points >= 9 || spread >= 3;
+}
+
+/**
+ * The nearest kind `kind` draws as for `series`: a shared axis needs one unit
+ * (else small multiples), bars take one series, a dumbbell two, grouped bars
+ * up to four (more become a heatmap), and a scatter needs its x measure.
+ */
+function fitKind(kind: ChartKind, series: readonly ChartSeries[], hasX: boolean): ChartKind {
+	if (kind === "multiples") return kind;
+	if (kind === "scatter" && hasX) return kind;
+	if (new Set(series.map(entry => `${entry.dim}:${entry.unit}`)).size > 1) return "multiples";
+	if (kind === "share" || kind === "diverging" || kind === "line" || kind === "heatmap") return kind;
+	const count = series.length;
+	return count === 1 ? "bar" : count === 2 ? "paired" : count <= 4 ? "grouped" : "heatmap";
+}
+
+/** Same-unit aggregate columns (`Total`, `avg`) restate the series beside them. */
+function dropAggregates(measures: readonly TableColumn[]): readonly TableColumn[] {
+	return measures.filter(
+		column =>
+			!AGGREGATE.test(column.header) ||
+			measures.filter(other => other !== column && other.dim === column.dim).length < 2,
+	);
+}
+
+/** How the label column orders the rows: by time, by step, by a numeric size — or not at all. */
+function axisOrder(
+	table: TableAnalysis,
+	label: TableColumn | undefined,
+): "temporal" | "sequence" | "numeric" | undefined {
+	if (!label) return undefined;
+	if (label.role === "temporal") return "temporal";
+	if (label.role === "sequence") return "sequence";
+	const names = table.rows.map(row => label.cells[row]!.text);
+	const mostly = (pattern: RegExp) => names.filter(name => pattern.test(name)).length >= names.length * 0.8;
+	if (mostly(SEQUENCE_LABEL)) return "sequence";
+	if (mostly(MONTH)) return "temporal";
+	const values = columnValues(label, table.rows);
+	if (values.length === names.length && values.length >= 3 && isMonotonic(values)) return "numeric";
+	return undefined;
+}
+
+function columnValues(column: TableColumn, rows: readonly number[]): number[] {
+	const values: number[] = [];
+	for (const row of rows) {
+		const cell = column.cells[row];
+		if (isNumber(cell)) values.push(cell.value);
+	}
+	return values;
+}
+
+/** A column as a series: its cells of the column's dimension, other cells as gaps (status words kept as notes). */
+function columnSeries(column: TableColumn, rows: readonly number[], name: string): ChartSeries {
+	const points: (ChartPoint | null)[] = [];
+	const notes: (string | undefined)[] = [];
+	for (const row of rows) {
+		const cell = column.cells[row]!;
+		const fits = isNumber(cell) && cell.dim === column.dim;
+		points.push(fits ? point(cell) : null);
+		notes.push(!fits && cell.kind !== "missing" && !isNumber(cell) ? cell.text : undefined);
+	}
+	return { name, dim: column.dim ?? "count", unit: column.unit, points, notes };
+}
+
+/** One row of a metric-per-row table as a series over the plotted columns, in the row's dominant dimension. */
+function rowSeries(name: string, columns: readonly TableColumn[], row: number): ChartSeries {
+	const cells = columns.map(column => column.cells[row]);
+	const tally = new Map<Dimension, number>();
+	for (const cell of cells) if (isNumber(cell)) tally.set(cell.dim, (tally.get(cell.dim) ?? 0) + 1);
+	let dim: Dimension = "count";
+	for (const [key, count] of tally) if (count > (tally.get(dim) ?? 0)) dim = key;
+	const unitCell = cells.find(cell => isNumber(cell) && cell.dim === dim);
+	return {
+		name,
+		dim,
+		unit: isNumber(unitCell) && (dim === "currency" || dim === "rate") ? unitCell.unit : "",
+		points: cells.map(cell => (isNumber(cell) && cell.dim === dim ? point(cell) : null)),
+		notes: cells.map(cell => (cell && !isNumber(cell) && cell.kind !== "missing" ? cell.text : undefined)),
+	};
+}
+
+/** A numeric cell as a plotted point labelled with its written figure. */
+function point(cell: NumberCell): ChartPoint {
+	return { value: cell.value, text: cell.figure, emphasis: cell.emphasis, upper: cell.upper };
+}
+
+/** `Total: 2,964 · 2,310` for each summary row, over the plotted columns. */
+function totalsCaption(
+	table: TableAnalysis,
+	label: TableColumn | undefined,
+	columns: readonly TableColumn[],
+	transpose: boolean,
+): string | undefined {
+	if (transpose || !label || table.totals.length === 0) return undefined;
+	const lines = table.totals.slice(0, 2).flatMap(row => {
+		const values = columns.map(column => column.cells[row]?.text ?? "").filter(Boolean);
+		return values.length ? [`${label.cells[row]!.text}: ${values.join(" · ")}`] : [];
+	});
+	return lines.length ? lines.join("   ") : undefined;
+}