@xynogen/pix-pretty 1.7.9 → 1.7.12

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@xynogen/pix-pretty",
-	"version": "1.7.9",
+	"version": "1.7.12",
 	"description": "Enhanced tool output rendering with syntax highlighting, file icons, tree views, diff rendering, and FFF search",
 	"type": "module",
 	"main": "src/index.ts",

package/src/ansi.ts

@@ -23,7 +23,7 @@ function parseAnsiRgb(
 	const m = ansi.match(
 		new RegExp(`${ESC_RE}\\[(?:38|48);2;(\\d+);(\\d+);(\\d+)m`),
 	);
-	return m ? { r: +m[1], g: +m[2], b: +m[3] } : null;
+	return m ? { r: +(m[1] ?? 0), g: +(m[2] ?? 0), b: +(m[3] ?? 0) } : null;
 }
 
 function getThemeBgAnsi(theme: BgTheme, key: string): string | null {
@@ -63,7 +63,7 @@ function isLowContrastShikiFg(params: string): boolean {
 	const parts = params.split(";").map(Number);
 	if (parts.length !== 5 || parts.some((n) => !Number.isFinite(n)))
 		return false;
-	const [, , r, g, b] = parts;
+	const [, , r = 0, g = 0, b = 0] = parts;
 	const luminance = 0.2126 * r + 0.7152 * g + 0.0722 * b;
 	return luminance < 72;
 }

package/src/commands/pretty.ts

@@ -62,7 +62,7 @@ export function registerPrettyCommand(pi: PiPrettyApi): void {
 			// Headless / no custom-UI host: cycle to the next mode + notify.
 			if (typeof ui.custom !== "function") {
 				const cur = ICON_MODES.indexOf(getIconMode());
-				const next = ICON_MODES[(cur + 1) % ICON_MODES.length]!;
+				const next = ICON_MODES[(cur + 1) % ICON_MODES.length] ?? "nerd";
 				applyMode(next);
 				ui.notify(`pix-pretty icons: ${next}`, "info");
 				return;
@@ -82,7 +82,7 @@ export function registerPrettyCommand(pi: PiPrettyApi): void {
 
 					const choose = (i: number) => {
 						selected = (i + ICON_MODES.length) % ICON_MODES.length;
-						applyMode(ICON_MODES[selected]!);
+						applyMode(ICON_MODES[selected] ?? "nerd");
 					};
 
 					return {

package/src/diff-render.ts

@@ -464,9 +464,12 @@ function injectBg(
 				continue;
 			}
 		}
-		while (ri < ranges.length && vis >= ranges[ri][1]) ri++;
+		while (ri < ranges.length && vis >= (ranges[ri] as [number, number])[1])
+			ri++;
 		const want =
-			ri < ranges.length && vis >= ranges[ri][0] && vis < ranges[ri][1];
+			ri < ranges.length &&
+			vis >= (ranges[ri] as [number, number])[0] &&
+			vis < (ranges[ri] as [number, number])[1];
 		if (want !== inHL) {
 			inHL = want;
 			out += inHL ? hlBg : baseBg;
@@ -497,6 +500,12 @@ function plainWordDiff(
 	return { old: o, new: n };
 }
 
+/** Type-safe index into an array that noUncheckedIndexedAccess marks as T|undefined.
+ *  Only call when the index is provably in-bounds (loop condition, length check, etc.). */
+function at<T>(arr: T[], i: number): T {
+	return arr[i] as T;
+}
+
 // ---------------------------------------------------------------------------
 // Split-vs-unified decision
 // ---------------------------------------------------------------------------
@@ -605,7 +614,7 @@ export async function renderUnified(
 	}
 
 	while (idx < vis.length) {
-		const l = vis[idx];
+		const l = at(vis, idx);
 
 		if (l.type === "sep") {
 			const gap = l.newNum;
@@ -638,29 +647,35 @@ export async function renderUnified(
 		}
 
 		const dels: Array<{ l: DiffLine; hl: string }> = [];
-		while (idx < vis.length && vis[idx].type === "del") {
-			dels.push({ l: vis[idx], hl: oldHL[oI] ?? vis[idx].content });
+		while (idx < vis.length) {
+			const entry = at(vis, idx);
+			if (entry.type !== "del") break;
+			dels.push({ l: entry, hl: oldHL[oI] ?? entry.content });
 			oI++;
 			idx++;
 		}
 		const adds: Array<{ l: DiffLine; hl: string }> = [];
-		while (idx < vis.length && vis[idx].type === "add") {
-			adds.push({ l: vis[idx], hl: newHL[nI] ?? vis[idx].content });
+		while (idx < vis.length) {
+			const entry = at(vis, idx);
+			if (entry.type !== "add") break;
+			adds.push({ l: entry, hl: newHL[nI] ?? entry.content });
 			nI++;
 			idx++;
 		}
 
 		const isPaired = dels.length === 1 && adds.length === 1;
 		const wd = isPaired
-			? wordDiffAnalysis(dels[0].l.content, adds[0].l.content)
+			? wordDiffAnalysis(at(dels, 0).l.content, at(adds, 0).l.content)
 			: null;
 		const wdBalanced = wd && wd.oldRanges.length > 0 && wd.newRanges.length > 0;
 
 		if (isPaired && wdBalanced && wd.similarity >= WORD_DIFF_MIN_SIM && canHL) {
-			const delBody = injectBg(dels[0].hl, wd.oldRanges, BG_DEL, BG_DEL_W);
-			const addBody = injectBg(adds[0].hl, wd.newRanges, BG_ADD, BG_ADD_W);
+			const del0 = at(dels, 0);
+			const add0 = at(adds, 0);
+			const delBody = injectBg(del0.hl, wd.oldRanges, BG_DEL, BG_DEL_W);
+			const addBody = injectBg(add0.hl, wd.newRanges, BG_ADD, BG_ADD_W);
 			emitRow(
-				dels[0].l.oldNum,
+				del0.l.oldNum,
 				"-",
 				BG_GUTTER_DEL,
 				`${dc.fgDel}${BOLD}`,
@@ -668,7 +683,7 @@ export async function renderUnified(
 				BG_DEL,
 			);
 			emitRow(
-				adds[0].l.newNum,
+				add0.l.newNum,
 				"+",
 				BG_GUTTER_ADD,
 				`${dc.fgAdd}${BOLD}`,
@@ -683,9 +698,11 @@ export async function renderUnified(
 			wd.similarity >= WORD_DIFF_MIN_SIM &&
 			!canHL
 		) {
-			const pwd = plainWordDiff(dels[0].l.content, adds[0].l.content);
+			const del0 = at(dels, 0);
+			const add0 = at(adds, 0);
+			const pwd = plainWordDiff(del0.l.content, add0.l.content);
 			emitRow(
-				dels[0].l.oldNum,
+				del0.l.oldNum,
 				"-",
 				BG_GUTTER_DEL,
 				`${dc.fgDel}${BOLD}`,
@@ -693,7 +710,7 @@ export async function renderUnified(
 				BG_DEL,
 			);
 			emitRow(
-				adds[0].l.newNum,
+				add0.l.newNum,
 				"+",
 				BG_GUTTER_ADD,
 				`${dc.fgAdd}${BOLD}`,
@@ -755,7 +772,7 @@ export async function renderSplit(
 	const rows: Row[] = [];
 	let i = 0;
 	while (i < diff.lines.length) {
-		const l = diff.lines[i];
+		const l = at(diff.lines, i);
 		if (l.type === "sep" || l.type === "ctx") {
 			rows.push({ left: l, right: l });
 			i++;
@@ -763,12 +780,16 @@ export async function renderSplit(
 		}
 		const dels: DiffLine[] = [];
 		const adds: DiffLine[] = [];
-		while (i < diff.lines.length && diff.lines[i].type === "del") {
-			dels.push(diff.lines[i]);
+		while (i < diff.lines.length) {
+			const entry = at(diff.lines, i);
+			if (entry.type !== "del") break;
+			dels.push(entry);
 			i++;
 		}
-		while (i < diff.lines.length && diff.lines[i].type === "add") {
-			adds.push(diff.lines[i]);
+		while (i < diff.lines.length) {
+			const entry = at(diff.lines, i);
+			if (entry.type !== "add") break;
+			adds.push(entry);
 			i++;
 		}
 		const n = Math.max(dels.length, adds.length);

package/src/diff.ts

@@ -4,6 +4,12 @@
 
 import * as Diff from "diff";
 
+/** Type-safe index into an array that noUncheckedIndexedAccess marks as T|undefined.
+ *  Only call when the index is provably in-bounds (loop condition, length check, etc.). */
+function at<T>(arr: T[], i: number): T {
+	return arr[i] as T;
+}
+
 export interface DiffLine {
 	type: "add" | "del" | "ctx" | "sep";
 	oldNum: number | null;
@@ -35,8 +41,9 @@ export function parseDiff(
 
 	for (let hi = 0; hi < patch.hunks.length; hi++) {
 		if (hi > 0) {
-			const prev = patch.hunks[hi - 1];
-			const gap = patch.hunks[hi].oldStart - (prev.oldStart + prev.oldLines);
+			const prev = at(patch.hunks, hi - 1);
+			const gap =
+				at(patch.hunks, hi).oldStart - (prev.oldStart + prev.oldLines);
 			lines.push({
 				type: "sep",
 				oldNum: null,
@@ -44,7 +51,7 @@ export function parseDiff(
 				content: "",
 			});
 		}
-		const h = patch.hunks[hi];
+		const h = at(patch.hunks, hi);
 		const shift = baseLine > 0 ? baseLine - 1 : 0;
 		let oL = h.oldStart + shift;
 		let nL = h.newStart + shift;

package/src/icon-catalog.ts

@@ -74,6 +74,9 @@ const CATALOG = {
 	"opt.toon": { nerd: "\u{F05C0}", unicode: `\u2662${VS}`, ascii: "Tn" },
 	"opt.ponytail": { nerd: "\u{F0190}", unicode: `\u2667${VS}`, ascii: "Pt" },
 	"opt.title": { nerd: "\u{F0DAB}", unicode: `\u25C8${VS}`, ascii: "*" },
+
+	// ── subagent widget (pix-subagent) ────────────────────────────────────
+	agent: { nerd: "\u{F0BA0}", unicode: `\u2699${VS}`, ascii: "@" },
 } as const;
 
 /** Every valid semantic icon key. */

package/src/icon-persist.ts

@@ -16,6 +16,10 @@ import { dirname, join } from "node:path";
 import { getAgentDir } from "@earendil-works/pi-coding-agent";
 import { ICON_MODES, type IconMode, setIconMode } from "./icon-catalog.js";
 
+function isIconMode(m: string): m is IconMode {
+	return (ICON_MODES as readonly string[]).includes(m);
+}
+
 function statePath(): string {
 	return join(getAgentDir(), "pretty.json");
 }
@@ -27,9 +31,8 @@ export function loadIconMode(): IconMode | undefined {
 		if (!existsSync(p)) return undefined;
 		const raw = JSON.parse(readFileSync(p, "utf-8")) as { icons?: string };
 		const mode = raw?.icons;
-		return ICON_MODES.includes(mode as IconMode)
-			? (mode as IconMode)
-			: undefined;
+		if (mode == null) return undefined;
+		return isIconMode(mode) ? mode : undefined;
 	} catch {
 		return undefined;
 	}

package/src/progress.ts

@@ -89,7 +89,7 @@ export function openProgress(
 						width: mw,
 						lines: [
 							theme.fg(accent, theme.bold(title)),
-							`${theme.fg(accent, SPINNER[frame])} ${theme.fg("muted", labelValue)}`,
+							`${theme.fg(accent, SPINNER[frame] ?? "")} ${theme.fg("muted", labelValue)}`,
 						],
 						color: (s) => theme.fg(accent, s),
 						bg: (s) => theme.bg("customMessageBg", s),

package/src/renderers.ts

@@ -92,7 +92,7 @@ export function renderTree(text: string, _basePath: string): string {
 	const show = lines.slice(0, MAX_PREVIEW_LINES);
 
 	for (let i = 0; i < show.length; i++) {
-		const entry = show[i].trim();
+		const entry = (show[i] ?? "").trim();
 		const isLast = i === show.length - 1 && total <= MAX_PREVIEW_LINES;
 		const prefix = isLast ? "└── " : "├── ";
 		const connector = `${FG_RULE}${prefix}${RST}`;