@xynogen/pix-runtime 0.17.0 → 0.17.2

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@xynogen/pix-runtime",
-	"version": "0.17.0",
+	"version": "0.17.2",
 	"description": "Pix shared runtime — versioned pix.json config, atomic persistence, typed change events",
 	"type": "module",
 	"main": "src/index.ts",
@@ -69,6 +69,7 @@
 		"@ast-grep/napi": "^0.45.0"
 	},
 	"dependencies": {
+		"systeminformation": "5.33.19",
 		"xxhash-wasm": "1.1.0"
 	},
 	"peerDependenciesMeta": {

package/src/binaries-tab.ts

@@ -245,9 +245,10 @@ export function createBinariesTab(opts: BinariesTabOptions) {
 
 		view(width: number): TabView {
 			const nameW = Math.max(6, ...rows.map((r) => r.name.length));
+			const iconW = Math.max(2, ...Object.values(STATE_ICON).map((key) => visibleWidth(icon(key))));
 			// Frame border + padding take 4 columns; every row must fit on one line.
 			const inner = Math.max(20, width - 4);
-			const lead = 2 + 2 + nameW + 2; // cursor, glyph, name, gap
+			const lead = 2 + iconW + 1 + nameW + 2; // cursor, glyph, name, gap
 			const body: string[] = [];
 			let selectedBodyLine: number | undefined;
 			let otherHeader = false;
@@ -294,7 +295,7 @@ export function createBinariesTab(opts: BinariesTabOptions) {
 					detail = theme.fg("muted", `not used on this OS · ${row.hint}`);
 				}
 				if (sel) selectedBodyLine = body.length;
-				const line = `${cursor} ${glyph} ${name}  ${detail}`;
+				const line = `${cursor} ${glyph}${" ".repeat(iconW - visibleWidth(glyph))} ${name}  ${detail}`;
 				body.push(visibleWidth(line) > inner ? truncateToWidth(line, inner, "…") : line);
 			}
 

package/src/pix-command.ts

@@ -213,7 +213,27 @@ const SETTINGS: SettingRow<unknown>[] = [
 		const part = key as keyof typeof prettySection.defaults.footer;
 		return row({
 			section: "Footer",
-			label: part === "wrap" ? "wrap line" : part,
+			get label() {
+				const key = {
+					mode: "status.active",
+					cwd: "cwd",
+					git: "git.unstaged",
+					context: "tokens",
+					model: "model",
+					thinking: "agent",
+					score: "score",
+					statuses: "status.running",
+					tokens: "tokens",
+					cost: "data.float",
+					tps: "score",
+					system: "sys.cpu",
+					clock: "clock",
+					wrap: "turns",
+				} as const;
+				const glyph = icon(key[part]);
+				const iconW = Math.max(2, ...Object.values(key).map((key) => visibleWidth(icon(key))));
+				return `${glyph}${" ".repeat(iconW - visibleWidth(glyph))} ${part === "wrap" ? "wrap line" : part}`;
+			},
 			handle: prettySection,
 			values: part === "wrap" ? ["on", "off"] : ["show", "hide"],
 			read: (v) =>
@@ -457,14 +477,20 @@ export function registerPixCommand(pi: ExtensionAPI, runtime: PixRuntime): void
 						});
 						return binaries;
 					};
-					const tabBar = () =>
-						tabs
+					const tabBar = (width: number) => {
+						const full = tabs
 							.map((name) =>
 								name === tab
 									? theme.fg("accent", theme.bold(`  ${name}  `))
 									: theme.fg("muted", `  ${name}  `),
 							)
 							.join(theme.fg("muted", "│"));
+						if (visibleWidth(full) <= width) return full;
+						return (
+							theme.fg("accent", theme.bold(`  ${tab}  `)) +
+							theme.fg("muted", `${tabs.indexOf(tab) + 1}/${tabs.length}`)
+						);
+					};
 					const switchTab = (direction: -1 | 1) => {
 						const next = (tabs.indexOf(tab) + direction + tabs.length) % tabs.length;
 						tab = tabs[next] ?? "Settings";
@@ -548,7 +574,7 @@ export function registerPixCommand(pi: ExtensionAPI, runtime: PixRuntime): void
 								const hints = view.footer.map(([key, action]) => guide(key, action)).join(guideSep);
 								const result = frameModal({
 									...frame,
-									header: [tabBar(), "", ...view.header],
+									header: [tabBar(Math.max(1, width - 4)), "", ...view.header],
 									body: view.body,
 									footer: [
 										"",
@@ -573,19 +599,26 @@ export function registerPixCommand(pi: ExtensionAPI, runtime: PixRuntime): void
 										lines = embedded.render(width);
 									}
 									const panelWidth = visibleWidth(lines[0] ?? "");
-									lines.splice(1, 0, ...frameRows(panelWidth, [tabBar()], frame).slice(1, -1));
+									lines.splice(
+										1,
+										0,
+										...frameRows(panelWidth, [tabBar(Math.max(1, panelWidth - 4))], frame).slice(
+											1,
+											-1,
+										),
+									);
 									return lines;
 								}
 								return frameModal({
 									...frame,
-									header: [tabBar(), ""],
+									header: [tabBar(Math.max(1, width - 4)), ""],
 									body: [theme.fg("dim", `  Loading ${tab} settings…`)],
 									footer: ["", tabGuide + guideSep + guide("esc", "close")],
 									bodyOffset: 0,
 								}).lines;
 							}
 							const settings = rows();
-							const labelW = Math.max(...settings.map((r) => r.label.length));
+							const labelW = Math.max(...settings.map((r) => visibleWidth(r.label)));
 							const body: string[] = [];
 							const settingBodyLines: number[] = [];
 							let lastSection = "";
@@ -599,7 +632,10 @@ export function registerPixCommand(pi: ExtensionAPI, runtime: PixRuntime): void
 								}
 								const sel = i === selected;
 								const cursor = sel ? theme.fg("accent", "→") : " ";
-								const label = theme.fg(sel ? "accent" : "text", row.label.padEnd(labelW));
+								const label = theme.fg(
+									sel ? "accent" : "text",
+									row.label + " ".repeat(labelW - visibleWidth(row.label)),
+								);
 								const value = isThemeRow(row) ? row.current() : row.read(runtime.get(row.handle));
 								const isDefault = isThemeRow(row) || value === row.read(row.handle.defaults);
 								settingBodyLines[i] = body.length;
@@ -607,7 +643,7 @@ export function registerPixCommand(pi: ExtensionAPI, runtime: PixRuntime): void
 							}
 							const result = frameModal({
 								...frame,
-								header: [tabBar(), ""],
+								header: [tabBar(Math.max(1, width - 4)), ""],
 								body,
 								footer: [
 									"",
@@ -782,7 +818,10 @@ function frameModal(opts: RuntimeModalOptions): RuntimeModalResult {
 	const maxBodyOffset = Math.max(0, body.length - visibleBodyLines);
 	let offset = Math.min(maxBodyOffset, Math.max(0, opts.bodyOffset));
 	if (opts.selectedBodyLine !== undefined) {
-		const selected = Math.min(body.length - 1, Math.max(0, opts.selectedBodyLine));
+		const selected = Math.min(
+			body.length - 1,
+			fitLines(opts.body.slice(0, Math.max(0, opts.selectedBodyLine)), inner).length,
+		);
 		if (selected < offset) offset = selected;
 		else if (selected >= offset + visibleBodyLines) offset = selected - visibleBodyLines + 1;
 	}

package/src/sections/pretty.ts

@@ -26,7 +26,6 @@ export interface PrettyConfig {
 		| "context"
 		| "model"
 		| "thinking"
-		| "price"
 		| "score"
 		| "statuses"
 		| "tokens"
@@ -68,7 +67,6 @@ const DEFAULTS: Readonly<PrettyConfig> = {
 		context: true,
 		model: true,
 		thinking: true,
-		price: true,
 		score: true,
 		statuses: true,
 		tokens: true,

package/src/sysstat.ts

@@ -1,118 +1,69 @@
-/**
- * sysstat.ts — cheap host load sample for status bars: CPU %, RAM %, GPU %, CPU temperature.
- *
- * CPU and RAM come from `node:os`, so they work on every OS. GPU busy % and temperature
- * are read from Linux sysfs (`/sys/class/drm`, `/sys/class/hwmon`, `/sys/class/thermal`);
- * elsewhere, or when the driver exposes nothing, those fields are undefined.
- * ponytail: no NVIDIA support (its driver has no sysfs busy file). Add an `nvidia-smi`
- * catalog binary and a slow poll here if that is needed.
- */
-
-import { readdirSync, readFileSync } from "node:fs";
-import { cpus, freemem, totalmem } from "node:os";
-import { join } from "node:path";
+import { promises as fs } from "node:fs";
+import si from "systeminformation";
 
 export interface SystemStats {
-	/** Busy CPU since the previous sample, 0–100. Undefined on the first sample. */
 	cpu?: number;
-	/** Used memory (total − available), 0–100. */
-	ram?: number;
-	/** GPU busy, 0–100 (Linux amdgpu/i915/xe sysfs only). */
+	cpuTemp?: number;
 	gpu?: number;
-	/** CPU package temperature in °C (Linux hwmon/thermal only). */
-	temp?: number;
-}
-
-/** Opaque CPU counter snapshot; pass the previous one to get a CPU percent. */
-export type CpuTimes = { busy: number; total: number };
-
-export function cpuTimes(): CpuTimes {
-	let busy = 0;
-	let total = 0;
-	for (const { times } of cpus()) {
-		const sum = times.user + times.nice + times.sys + times.idle + times.irq;
-		total += sum;
-		busy += sum - times.idle;
-	}
-	return { busy, total };
-}
-
-/** Busy share between two snapshots, 0–100, or undefined when no time passed. */
-export function cpuPercent(prev: CpuTimes, next: CpuTimes): number | undefined {
-	const total = next.total - prev.total;
-	if (total <= 0) return undefined;
-	return clampPct(((next.busy - prev.busy) / total) * 100);
-}
-
-const clampPct = (n: number) => Math.min(100, Math.max(0, Math.round(n)));
-
-function readNumber(path: string): number | undefined {
-	try {
-		const n = Number(readFileSync(path, "utf8").trim());
-		return Number.isFinite(n) ? n : undefined;
-	} catch {
-		return undefined;
-	}
-}
-
-function list(dir: string): string[] {
-	try {
-		return readdirSync(dir);
-	} catch {
-		return [];
-	}
-}
-
-/** Highest `gpu_busy_percent` across DRM cards (a laptop may report an idle iGPU too). */
-export function readGpuBusy(sys = "/sys"): number | undefined {
-	let best: number | undefined;
-	for (const card of list(join(sys, "class/drm"))) {
-		if (!/^card\d+$/.test(card)) continue;
-		const n = readNumber(join(sys, "class/drm", card, "device/gpu_busy_percent"));
-		if (n !== undefined) best = Math.max(best ?? 0, clampPct(n));
-	}
-	return best;
+	gpuTemp?: number;
+	ram?: number;
 }
 
-// CPU sensor drivers, most specific first. coretemp = Intel, k10temp/zenpower = AMD.
-const CPU_HWMON = ["k10temp", "zenpower", "coretemp", "cpu_thermal", "acpitz"];
+/** Missing sensors, sentinel values and failed probes stay unavailable, never become zero. */
+const reading = (value: unknown, max: number): number | undefined =>
+	typeof value === "number" && Number.isFinite(value) && value >= 0 && value <= max
+		? Math.round(value)
+		: undefined;
 
-/** CPU temperature in °C from hwmon (known CPU drivers), then thermal_zone0. */
-export function readCpuTemp(sys = "/sys"): number | undefined {
-	const hwmon = join(sys, "class/hwmon");
-	const byName = new Map<string, string>();
-	for (const dir of list(hwmon)) {
-		try {
-			byName.set(readFileSync(join(hwmon, dir, "name"), "utf8").trim(), dir);
-		} catch {}
+async function readAmdStats(): Promise<SystemStats> {
+	const stats: SystemStats = {};
+	const read = (path: string) => fs.readFile(path, "utf8").catch(() => "");
+	const cards = await fs.readdir("/sys/class/drm").catch(() => []);
+	for (const card of cards.filter((name) => /^card\d+$/.test(name))) {
+		const device = `/sys/class/drm/${card}/device`;
+		if ((await read(`${device}/vendor`)).trim() !== "0x1002") continue;
+		const busy = (await read(`${device}/gpu_busy_percent`)).trim();
+		const gpu = busy ? reading(Number(busy), 100) : undefined;
+		if (gpu !== undefined) stats.gpu = Math.max(stats.gpu ?? 0, gpu);
+		const sensors = await fs.readdir(`${device}/hwmon`).catch(() => []);
+		for (const sensor of sensors) {
+			const raw = (await read(`${device}/hwmon/${sensor}/temp1_input`)).trim();
+			const temperature = raw ? reading(Number(raw) / 1000, 150) : undefined;
+			if (temperature !== undefined) stats.gpuTemp = Math.max(stats.gpuTemp ?? 0, temperature);
+		}
 	}
-	for (const name of CPU_HWMON) {
-		const dir = byName.get(name);
-		const milli = dir && readNumber(join(hwmon, dir, "temp1_input"));
-		if (milli) return Math.round(milli / 1000);
-	}
-	const zone = readNumber(join(sys, "class/thermal/thermal_zone0/temp"));
-	return zone ? Math.round(zone / 1000) : undefined;
+	return stats;
 }
 
-/**
- * Take one sample. Keep the returned `times` and pass it back next call so CPU % covers
- * the interval between calls. Synchronous and cheap (a few small sysfs reads).
- */
-export function readSystemStats(
-	prev?: CpuTimes,
-	sys = "/sys",
-): { stats: SystemStats; times: CpuTimes } {
-	const times = cpuTimes();
-	const total = totalmem();
-	const linux = process.platform === "linux";
+/** Read each source independently so an unsupported sensor cannot discard other readings. */
+export async function readSystemStats(): Promise<SystemStats> {
+	const [cpu, temperature, graphics, memory] = await Promise.allSettled([
+		si.currentLoad(),
+		si.cpuTemperature(),
+		si.graphics(),
+		si.mem(),
+	]);
+	const controllers = graphics.status === "fulfilled" ? graphics.value.controllers : [];
+	const highest = (key: "utilizationGpu" | "temperatureGpu", max: number) => {
+		const values = controllers
+			.map((controller) => reading(controller[key], max))
+			.filter((value): value is number => value !== undefined);
+		return values.length ? Math.max(...values) : undefined;
+	};
+	const gpu = highest("utilizationGpu", 100);
+	const gpuTemp = highest("temperatureGpu", 150);
+	const amd =
+		process.platform === "linux" && (gpu === undefined || gpuTemp === undefined)
+			? await readAmdStats()
+			: {};
 	return {
-		times,
-		stats: {
-			cpu: prev ? cpuPercent(prev, times) : undefined,
-			ram: total > 0 ? clampPct(((total - freemem()) / total) * 100) : undefined,
-			gpu: linux ? readGpuBusy(sys) : undefined,
-			temp: linux ? readCpuTemp(sys) : undefined,
-		},
+		cpu: cpu.status === "fulfilled" ? reading(cpu.value.currentLoad, 100) : undefined,
+		cpuTemp: temperature.status === "fulfilled" ? reading(temperature.value.main, 150) : undefined,
+		gpu: gpu ?? amd.gpu,
+		gpuTemp: gpuTemp ?? amd.gpuTemp,
+		ram:
+			memory.status === "fulfilled" && memory.value.total > 0
+				? reading(((memory.value.total - memory.value.available) / memory.value.total) * 100, 100)
+				: undefined,
 	};
 }