@xynogen/pix-optimizer 1.1.15 → 1.1.17

package/README.md

@@ -44,7 +44,7 @@ Switch the style live from the **`/optimizer` overlay** — a fifth `icons` row
 at the bottom cycles `nerd → unicode → ascii` with `←→`; the choice persists to
 `~/.pi/agent/optimizer.json`.
 
-Icons follow the **global** `pix-pretty` mode — set via `/pretty` or `PRETTY_ICONS`
+Icons follow the **global** `pix-pretty` mode — set via `/pix` or `PRETTY_ICONS`
 env var. The optimizer no longer has its own toggle.
 
 ## Features

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@xynogen/pix-optimizer",
-	"version": "1.1.15",
+	"version": "1.1.17",
 	"description": "Performance optimization suite for Pi Coding Agent - caveman mode + RTK tool rewriting + jq/TOON JSON compression + ponytail lazy-dev mode",
 	"type": "module",
 	"main": "src/index.ts",

package/src/capability.test.ts

@@ -0,0 +1,42 @@
+import { describe, expect, it, mock } from "bun:test";
+import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
+import { canExecute } from "./capability.ts";
+
+function executor(impl: ExtensionAPI["exec"]): Pick<ExtensionAPI, "exec"> {
+	return { exec: impl };
+}
+
+describe("canExecute", () => {
+	it("runs the requested capability probe without a shell", async () => {
+		const exec = mock(async () => ({
+			stdout: "jq-1.7",
+			stderr: "",
+			code: 0,
+			killed: false,
+		}));
+
+		expect(await canExecute(executor(exec), "jq", ["--version"])).toBe(true);
+		expect(exec).toHaveBeenCalledWith("jq", ["--version"], { timeout: 3000 });
+	});
+
+	it("supports a custom timeout", async () => {
+		const exec = mock(async () => ({ stdout: "2.3.0", stderr: "", code: 0, killed: false }));
+
+		expect(await canExecute(executor(exec), "toon", ["--version"], 1000)).toBe(true);
+		expect(exec).toHaveBeenCalledWith("toon", ["--version"], { timeout: 1000 });
+	});
+
+	it("returns false for a non-zero exit", async () => {
+		const exec = mock(async () => ({ stdout: "", stderr: "failed", code: 1, killed: false }));
+
+		expect(await canExecute(executor(exec), "jq", ["--version"])).toBe(false);
+	});
+
+	it("returns false when the executable cannot be spawned", async () => {
+		const exec = mock(async () => {
+			throw new Error("ENOENT");
+		});
+
+		expect(await canExecute(executor(exec), "toon", ["--version"])).toBe(false);
+	});
+});

package/src/capability.ts

@@ -0,0 +1,16 @@
+import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
+
+/** Test whether a command can perform a harmless operation through Pi's executor. */
+export async function canExecute(
+	pi: Pick<ExtensionAPI, "exec">,
+	command: string,
+	args: string[],
+	timeout = 3000,
+): Promise<boolean> {
+	try {
+		const result = await pi.exec(command, args, { timeout });
+		return result.code === 0;
+	} catch {
+		return false;
+	}
+}

package/src/caveman.ts

@@ -108,8 +108,7 @@ export function resolveLevel(arg: string): Level | null {
  * Help text shown when /caveman is run with no argument.
  */
 export function buildHelp(current: Level): string {
-	const statusLine =
-		current === "off" ? "off" : `${STATUS_LABELS[current]} (${current})`;
+	const statusLine = current === "off" ? "off" : `${STATUS_LABELS[current]} (${current})`;
 	return [
 		`Caveman mode: ${statusLine}`,
 		"",
@@ -133,10 +132,7 @@ export function toggleLevel(current: Level): Level {
 
 // ── Pi extension ────────────────────────────────────────────────────────────
 
-export function caveman(
-	pi: ExtensionAPI,
-	status: OptimizerStatus,
-): OptimizerHandle {
+export function caveman(pi: ExtensionAPI, status: OptimizerStatus): OptimizerHandle {
 	let level: Level = "off";
 
 	// -- Status: report into the shared optimizer indicator. --
@@ -179,10 +175,7 @@ export function caveman(
 
 	// -- Overlay value handler (called by the /optimizer overlay) --
 
-	async function run(
-		value: string,
-		ctx: ExtensionCommandContext,
-	): Promise<void> {
+	async function run(value: string, ctx: ExtensionCommandContext): Promise<void> {
 		const resolved = resolveLevel(value);
 		if (resolved === null) return;
 		level = resolved;
@@ -192,9 +185,7 @@ export function caveman(
 		syncStatus(ctx);
 
 		ctx.ui.notify(
-			level === "off"
-				? "Caveman mode off."
-				: `Caveman: ${STATUS_LABELS[level]}`,
+			level === "off" ? "Caveman mode off." : `Caveman: ${STATUS_LABELS[level]}`,
 			"info",
 		);
 	}

package/src/index.ts

@@ -19,15 +19,11 @@ import { json } from "./json.ts";
 import { registerOptCommand } from "./opt.ts";
 import { ponytail } from "./ponytail.ts";
 import { rtk } from "./rtk.ts";
-import {
-	type OptimizerHandle,
-	OptimizerStatus,
-	type OptimizerTool,
-} from "./status.ts";
+import { type OptimizerHandle, OptimizerStatus, type OptimizerTool } from "./status.ts";
 import { filterModelWarnings } from "./tool-result-filter.ts";
 
 export default function optimizer(pi: ExtensionAPI) {
-	// Icons follow the global pix-pretty mode (set via /pretty); no local state.
+	// Icons follow the global pix-pretty mode (set via /pix); no local state.
 	const status = new OptimizerStatus();
 
 	// Each module registers its own lifecycle hooks and returns a handle the

package/src/json.test.ts

@@ -71,9 +71,7 @@ describe("isUniformObjectArray", () => {
 	});
 
 	it("false for differing keys", () => {
-		expect(isUniformObjectArray([{ id: 1 }, { id: 2, extra: true }])).toBe(
-			false,
-		);
+		expect(isUniformObjectArray([{ id: 1 }, { id: 2, extra: true }])).toBe(false);
 	});
 
 	it("false when a value is itself an object", () => {

package/src/json.ts

@@ -22,6 +22,7 @@ import type {
 	ExtensionCommandContext,
 	ExtensionContext,
 } from "@earendil-works/pi-coding-agent";
+import { canExecute } from "./capability.ts";
 import { loadOptValue, saveOptValue } from "./persist.ts";
 import type { OptimizerHandle, OptimizerStatus } from "./status.ts";
 
@@ -182,8 +183,7 @@ export function isUniformObjectArray(arr: unknown[]): boolean {
 		for (let i = 0; i < keys.length; i++) {
 			if (elKeys[i] !== keys[i]) return false;
 			// values must be primitive for a clean tabular row
-			if (isObjectLike((el as Record<string, unknown>)[keys[i] as string]))
-				return false;
+			if (isObjectLike((el as Record<string, unknown>)[keys[i] as string])) return false;
 		}
 		return true;
 	});
@@ -209,21 +209,14 @@ export function objectDepth(value: unknown): number {
 
 // ── Pi extension ──────────────────────────────────────────────────────────────
 
-export function json(
-	pi: ExtensionAPI,
-	status: OptimizerStatus,
-): OptimizerHandle {
+export function json(pi: ExtensionAPI, status: OptimizerStatus): OptimizerHandle {
 	let enabled = true;
 	let jqAvailable: boolean | null = null;
 	let toonAvailable: boolean | null = null;
 
 	// Report into the shared optimizer indicator.
 	function syncStatus(ctx: Pick<ExtensionContext, "ui">) {
-		status.set(
-			"toon",
-			enabled && jqAvailable !== false && toonAvailable !== false,
-			ctx,
-		);
+		status.set("toon", enabled && jqAvailable !== false && toonAvailable !== false, ctx);
 	}
 
 	pi.on("session_start", async (_event, ctx) => {
@@ -249,14 +242,10 @@ export function json(
 
 		// Probe once on first JSON-relevant prompt.
 		if (jqAvailable === null || toonAvailable === null) {
-			const [jqRes, toonRes] = await Promise.all([
-				pi.exec("which", ["jq"], { timeout: 1000 }).catch(() => ({ code: 1 })),
-				pi
-					.exec("which", ["toon"], { timeout: 1000 })
-					.catch(() => ({ code: 1 })),
+			[jqAvailable, toonAvailable] = await Promise.all([
+				canExecute(pi, "jq", ["--version"]),
+				canExecute(pi, "toon", ["--version"]),
 			]);
-			jqAvailable = jqRes.code === 0;
-			toonAvailable = toonRes.code === 0;
 			if (!jqAvailable)
 				ctx.ui.notify(
 					"jq not found — JSON/TOON guidance disabled. Install: sudo apt install jq  or  brew install jq",
@@ -277,10 +266,7 @@ export function json(
 
 	// -- Overlay value handler (called by the /optimizer overlay) --
 
-	async function run(
-		value: string,
-		ctx: ExtensionCommandContext,
-	): Promise<void> {
+	async function run(value: string, ctx: ExtensionCommandContext): Promise<void> {
 		enabled = value === "on";
 		saveOptValue("toon", enabled ? "on" : "off");
 

package/src/opt.test.ts

@@ -4,11 +4,7 @@ import type { OptimizerHandle, OptimizerTool } from "./status.ts";
 
 /** Build a handle set with fixed current values + value lists. */
 function fakeHandles(): Record<OptimizerTool, OptimizerHandle> {
-	const mk = (
-		name: OptimizerTool,
-		current: string,
-		values: string[],
-	): OptimizerHandle => ({
+	const mk = (name: OptimizerTool, current: string, values: string[]): OptimizerHandle => ({
 		name,
 		help: `${name} — ${name} help`,
 		values,

package/src/opt.ts

@@ -16,11 +16,7 @@
 
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 import { frameLines } from "@xynogen/pix-pretty/modal-frame";
-import type {
-	OptimizerHandle,
-	OptimizerStatus,
-	OptimizerTool,
-} from "./status.ts";
+import type { OptimizerHandle, OptimizerStatus, OptimizerTool } from "./status.ts";
 import { toolIcon } from "./status.ts";
 
 /** Min/max content width for the overlay box (excludes 4 cols of chrome). */
@@ -28,12 +24,7 @@ const MIN_CONTENT = 28;
 const MAX_CONTENT = 60;
 
 /** Fixed render order — matches the status-bar cell. */
-const TOOL_ORDER: readonly OptimizerTool[] = [
-	"caveman",
-	"rtk",
-	"toon",
-	"ponytail",
-];
+const TOOL_ORDER: readonly OptimizerTool[] = ["caveman", "rtk", "toon", "ponytail"];
 
 /** Strip the leading "name [args] — " prefix from a handle's help string. */
 function helpSummary(help: string): string {
@@ -64,9 +55,7 @@ export function levelBar(current: string, values: readonly string[]): string {
 }
 
 /** Plain-text status fallback when there's no custom-UI host (headless/tests). */
-export function buildOptHelp(
-	handles: Record<OptimizerTool, OptimizerHandle>,
-): string {
+export function buildOptHelp(handles: Record<OptimizerTool, OptimizerHandle>): string {
 	const lines = TOOL_ORDER.map(
 		(n) => `  ${n}: ${handles[n].current()}  — ${helpSummary(handles[n].help)}`,
 	);
@@ -140,8 +129,7 @@ export function registerOptCommand(
 					};
 
 					const move = (direction: -1 | 1) => {
-						selected =
-							(selected + direction + TOOL_ORDER.length) % TOOL_ORDER.length;
+						selected = (selected + direction + TOOL_ORDER.length) % TOOL_ORDER.length;
 					};
 
 					return {
@@ -156,14 +144,8 @@ export function registerOptCommand(
 									tool.padEnd(nameWidth),
 								);
 								const cur = handles[tool].current();
-								const value = theme.fg(
-									on ? "success" : "dim",
-									cur.padEnd(valueWidth),
-								);
-								const bar = theme.fg(
-									on ? "accent" : "dim",
-									levelBar(cur, handles[tool].values),
-								);
+								const value = theme.fg(on ? "success" : "dim", cur.padEnd(valueWidth));
+								const bar = theme.fg(on ? "accent" : "dim", levelBar(cur, handles[tool].values));
 								return `${cursor} ${glyph}  ${name}  ${value}  ${bar}`;
 							});
 							const lines = [
@@ -171,12 +153,7 @@ export function registerOptCommand(
 								"",
 								...rows,
 								"",
-								theme.fg(
-									"dim",
-									helpSummary(
-										handles[TOOL_ORDER[selected] as OptimizerTool].help,
-									),
-								),
+								theme.fg("dim", helpSummary(handles[TOOL_ORDER[selected] as OptimizerTool].help)),
 								"",
 								theme.fg("dim", "←→ change · ↑↓ move · esc close"),
 							];
@@ -192,12 +169,7 @@ export function registerOptCommand(
 							if (data === "k" || data === "\u001b[A") move(-1);
 							else if (data === "j" || data === "\u001b[B") move(1);
 							else if (data === "h" || data === "\u001b[D") cycle(-1);
-							else if (
-								data === "l" ||
-								data === "\u001b[C" ||
-								data === " " ||
-								data === "\r"
-							)
+							else if (data === "l" || data === "\u001b[C" || data === " " || data === "\r")
 								cycle(1);
 							else if (data === "\u001b" || data === "q") {
 								done(null);

package/src/ponytail.ts

@@ -113,8 +113,7 @@ export function resolveLevel(arg: string): Level | null {
  * Help text shown when /opt ponytail is run with no argument.
  */
 export function buildHelp(current: Level): string {
-	const statusLine =
-		current === "off" ? "off" : `${STATUS_LABELS[current]} (${current})`;
+	const statusLine = current === "off" ? "off" : `${STATUS_LABELS[current]} (${current})`;
 	return [
 		`Ponytail mode: ${statusLine}`,
 		"",
@@ -137,10 +136,7 @@ export function toggleLevel(current: Level): Level {
 
 // ── Pi extension ────────────────────────────────────────────────────────────
 
-export function ponytail(
-	pi: ExtensionAPI,
-	status: OptimizerStatus,
-): OptimizerHandle {
+export function ponytail(pi: ExtensionAPI, status: OptimizerStatus): OptimizerHandle {
 	let level: Level = "off";
 
 	// -- Status: report into the shared optimizer indicator. --
@@ -182,10 +178,7 @@ export function ponytail(
 
 	// -- Overlay value handler (called by the /optimizer overlay) --
 
-	async function run(
-		value: string,
-		ctx: ExtensionCommandContext,
-	): Promise<void> {
+	async function run(value: string, ctx: ExtensionCommandContext): Promise<void> {
 		const resolved = resolveLevel(value);
 		if (resolved === null) return;
 		level = resolved;
@@ -195,9 +188,7 @@ export function ponytail(
 		syncStatus(ctx);
 
 		ctx.ui.notify(
-			level === "off"
-				? "Ponytail mode off."
-				: `Ponytail: ${STATUS_LABELS[level]}`,
+			level === "off" ? "Ponytail mode off." : `Ponytail: ${STATUS_LABELS[level]}`,
 			"info",
 		);
 	}

package/src/rtk.test.ts

@@ -1,9 +1,11 @@
-import { describe, expect, it } from "bun:test";
+import { describe, expect, it, mock } from "bun:test";
+import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 import {
 	applyRtkRewrite,
 	type BashCallEvent,
 	buildSudoBlockReason,
 	detectSudoSegments,
+	probeRtkAvailability,
 	rewriteChain,
 	splitChain,
 } from "./rtk.ts";
@@ -13,17 +15,41 @@ function bashEvent(command: string): BashCallEvent {
 	return { toolName: "bash", input: { command } };
 }
 
+describe("probeRtkAvailability", () => {
+	it("probes rtk directly and accepts a successful version check", async () => {
+		const exec = mock(async () => ({
+			stdout: "rtk 0.1.0",
+			stderr: "",
+			code: 0,
+			killed: false,
+		}));
+
+		expect(await probeRtkAvailability({ exec } as Pick<ExtensionAPI, "exec">)).toBe(true);
+		expect(exec).toHaveBeenCalledWith("rtk", ["--version"], { timeout: 3000 });
+	});
+
+	it("rejects an unsuccessful version check", async () => {
+		const exec = mock(async () => ({ stdout: "", stderr: "missing", code: 1, killed: false }));
+
+		expect(await probeRtkAvailability({ exec } as Pick<ExtensionAPI, "exec">)).toBe(false);
+	});
+
+	it("treats a spawn failure as unavailable", async () => {
+		const exec = mock(async () => {
+			throw new Error("ENOENT");
+		});
+
+		expect(await probeRtkAvailability({ exec } as Pick<ExtensionAPI, "exec">)).toBe(false);
+	});
+});
+
 describe("splitChain", () => {
 	it("returns single segment for plain command", () => {
 		expect(splitChain("git status")).toEqual(["git status"]);
 	});
 
 	it("splits on && keeping operator", () => {
-		expect(splitChain("git add . && git push")).toEqual([
-			"git add . ",
-			"&&",
-			" git push",
-		]);
+		expect(splitChain("git add . && git push")).toEqual(["git add . ", "&&", " git push"]);
 	});
 
 	it("splits on ||, ;, |", () => {
@@ -33,9 +59,7 @@ describe("splitChain", () => {
 	});
 
 	it("ignores operators inside double quotes", () => {
-		expect(splitChain('git commit -m "a && b"')).toEqual([
-			'git commit -m "a && b"',
-		]);
+		expect(splitChain('git commit -m "a && b"')).toEqual(['git commit -m "a && b"']);
 	});
 
 	it("ignores operators inside single quotes", () => {
@@ -59,9 +83,7 @@ describe("rewriteChain", () => {
 	});
 
 	it("prefixes mixed known commands", () => {
-		expect(rewriteChain("cargo build && npm test")).toBe(
-			"rtk cargo build && rtk npm test",
-		);
+		expect(rewriteChain("cargo build && npm test")).toBe("rtk cargo build && rtk npm test");
 	});
 
 	it("leaves unknown commands alone", () => {
@@ -74,22 +96,16 @@ describe("rewriteChain", () => {
 	});
 
 	it("only prefixes known segments in a mixed chain", () => {
-		expect(rewriteChain("cd /tmp && git status")).toBe(
-			"cd /tmp && rtk git status",
-		);
+		expect(rewriteChain("cd /tmp && git status")).toBe("cd /tmp && rtk git status");
 	});
 
 	it("does not double-prefix already-rtk commands", () => {
 		expect(rewriteChain("rtk git status")).toBe("rtk git status");
-		expect(rewriteChain("rtk git add . && git push")).toBe(
-			"rtk git add . && rtk git push",
-		);
+		expect(rewriteChain("rtk git add . && git push")).toBe("rtk git add . && rtk git push");
 	});
 
 	it("does not touch operators inside quotes", () => {
-		expect(rewriteChain('git commit -m "a && b"')).toBe(
-			'rtk git commit -m "a && b"',
-		);
+		expect(rewriteChain('git commit -m "a && b"')).toBe('rtk git commit -m "a && b"');
 	});
 
 	it("returns original on unbalanced quotes", () => {
@@ -106,9 +122,7 @@ describe("rewriteChain", () => {
 	});
 
 	it("handles pipes between known commands", () => {
-		expect(rewriteChain("git log | grep fix")).toBe(
-			"rtk git log | rtk grep fix",
-		);
+		expect(rewriteChain("git log | grep fix")).toBe("rtk git log | rtk grep fix");
 	});
 });
 
@@ -120,9 +134,7 @@ describe("detectSudoSegments", () => {
 	});
 
 	it("detects a plain sudo segment", () => {
-		expect(detectSudoSegments(["sudo apt-get install foo"])).toEqual([
-			"sudo apt-get install foo",
-		]);
+		expect(detectSudoSegments(["sudo apt-get install foo"])).toEqual(["sudo apt-get install foo"]);
 	});
 
 	it("detects sudo in a chain (operators excluded)", () => {
@@ -132,10 +144,7 @@ describe("detectSudoSegments", () => {
 
 	it("detects multiple sudo segments", () => {
 		const parts = ["sudo rm -rf /tmp ", ";", " sudo reboot"];
-		expect(detectSudoSegments(parts)).toEqual([
-			"sudo rm -rf /tmp",
-			"sudo reboot",
-		]);
+		expect(detectSudoSegments(parts)).toEqual(["sudo rm -rf /tmp", "sudo reboot"]);
 	});
 
 	it("does not match 'sudoer' or 'pseudo'", () => {
@@ -170,10 +179,7 @@ describe("buildSudoBlockReason", () => {
 	});
 
 	it("lists all blocked commands", () => {
-		const reason = buildSudoBlockReason(
-			["sudo rm -rf /tmp", "sudo reboot"],
-			true,
-		);
+		const reason = buildSudoBlockReason(["sudo rm -rf /tmp", "sudo reboot"], true);
 		expect(reason).toContain("sudo rm -rf /tmp");
 		expect(reason).toContain("sudo reboot");
 	});
@@ -254,13 +260,9 @@ describe("applyRtkRewrite (tool_call hook step)", () => {
 
 	it("handles missing / non-string command safely", () => {
 		const event: BashCallEvent = { toolName: "bash", input: {} };
-		expect(applyRtkRewrite(event, { enabled: true, rtkAvailable: true })).toBe(
-			false,
-		);
+		expect(applyRtkRewrite(event, { enabled: true, rtkAvailable: true })).toBe(false);
 		const event2: BashCallEvent = { toolName: "bash", input: { command: 123 } };
-		expect(applyRtkRewrite(event2, { enabled: true, rtkAvailable: true })).toBe(
-			false,
-		);
+		expect(applyRtkRewrite(event2, { enabled: true, rtkAvailable: true })).toBe(false);
 	});
 
 	it("leaves command unchanged on unbalanced quotes", () => {

package/src/rtk.ts

@@ -11,6 +11,7 @@ import type {
 	ExtensionCommandContext,
 	ExtensionContext,
 } from "@earendil-works/pi-coding-agent";
+import { canExecute } from "./capability.ts";
 import { loadOptValue, saveOptValue } from "./persist.ts";
 import type { OptimizerHandle, OptimizerStatus } from "./status.ts";
 
@@ -51,10 +52,7 @@ export function detectSudoSegments(parts: string[]): string[] {
  * intercepted. Directs the model to `sudo_run` when available, otherwise
  * explains the restriction clearly.
  */
-export function buildSudoBlockReason(
-	sudoCmds: string[],
-	hasSudoRunTool: boolean,
-): string {
+export function buildSudoBlockReason(sudoCmds: string[], hasSudoRunTool: boolean): string {
 	const list = sudoCmds.map((c) => `  - ${c}`).join("\n");
 	if (hasSudoRunTool) {
 		return (
@@ -139,7 +137,11 @@ const RTK_COMMANDS = new Set([
 interface RtkStatus {
 	available: boolean;
 	checkedAt: number;
-	path?: string;
+}
+
+/** Probe the command we actually use instead of relying on a platform-specific locator. */
+export function probeRtkAvailability(pi: Pick<ExtensionAPI, "exec">): Promise<boolean> {
+	return canExecute(pi, "rtk", ["--version"]);
 }
 
 /**
@@ -224,10 +226,7 @@ export function rewriteChain(command: string): string {
 	return changed ? rewritten.join("") : command;
 }
 
-export function rtk(
-	pi: ExtensionAPI,
-	status: OptimizerStatus,
-): OptimizerHandle {
+export function rtk(pi: ExtensionAPI, status: OptimizerStatus): OptimizerHandle {
 	let rtkStatus: RtkStatus | null = null;
 	let warnedMissing = false;
 	let enabled = true;
@@ -248,25 +247,12 @@ export function rtk(
 			return rtkStatus;
 		}
 
-		try {
-			const result = await pi.exec("which", ["rtk"], { timeout: 1000 });
-			if (result.code === 0 && result.stdout?.trim()) {
-				rtkStatus = {
-					available: true,
-					checkedAt: Date.now(),
-					path: result.stdout.trim(),
-				};
-				warnedMissing = false;
-				return rtkStatus;
-			}
-		} catch (_error) {
-			// which command failed
-		}
-
+		const available = await probeRtkAvailability(pi);
 		rtkStatus = {
-			available: false,
+			available,
 			checkedAt: Date.now(),
 		};
+		if (available) warnedMissing = false;
 		return rtkStatus;
 	};
 
@@ -306,10 +292,7 @@ export function rtk(
 
 	// -- Overlay value handler (called by the /optimizer overlay) --
 
-	async function run(
-		value: string,
-		ctx: ExtensionCommandContext,
-	): Promise<void> {
+	async function run(value: string, ctx: ExtensionCommandContext): Promise<void> {
 		enabled = value === "on";
 		saveOptValue("rtk", enabled ? "on" : "off");
 

package/src/status.test.ts

@@ -1,10 +1,5 @@
 import { describe, expect, it } from "bun:test";
-import {
-	OptimizerStatus,
-	renderStatus,
-	STATUS_KEY,
-	toolIcon,
-} from "./status.ts";
+import { OptimizerStatus, renderStatus, STATUS_KEY, toolIcon } from "./status.ts";
 
 /** Tagging colorizer: <accent>X</accent> / <dim>X</dim> for assertions. */
 const tag = (c: string, t: string) => `<${c}>${t}</${c}>`;
@@ -17,23 +12,13 @@ const PT = toolIcon("ponytail");
 
 describe("renderStatus", () => {
 	it("shows ALL icons in order, accent when enabled", () => {
-		expect(
-			renderStatus(
-				{ caveman: true, rtk: true, toon: true, ponytail: true },
-				tag,
-			),
-		).toBe(
+		expect(renderStatus({ caveman: true, rtk: true, toon: true, ponytail: true }, tag)).toBe(
 			`<accent>${CV}</accent>  <accent>${RK}</accent>  <accent>${TN}</accent>  <accent>${PT}</accent> `,
 		);
 	});
 
 	it("dims disabled tools but still shows them", () => {
-		expect(
-			renderStatus(
-				{ caveman: false, rtk: true, toon: true, ponytail: true },
-				tag,
-			),
-		).toBe(
+		expect(renderStatus({ caveman: false, rtk: true, toon: true, ponytail: true }, tag)).toBe(
 			`<dim>${CV}</dim>  <accent>${RK}</accent>  <accent>${TN}</accent>  <accent>${PT}</accent> `,
 		);
 	});
@@ -63,8 +48,7 @@ describe("OptimizerStatus", () => {
 		return {
 			calls,
 			ui: {
-				setStatus: (key: string, text: string | undefined) =>
-					calls.push({ key, text: text ?? "" }),
+				setStatus: (key: string, text: string | undefined) => calls.push({ key, text: text ?? "" }),
 				theme: { fg: (c: string, t: string) => `<${c}>${t}</${c}>` },
 			},
 		} as const;

package/src/status.ts

@@ -17,11 +17,7 @@ import type {
 	ExtensionContext,
 	ThemeColor,
 } from "@earendil-works/pi-coding-agent";
-import {
-	type IconKey,
-	icon,
-	onIconModeChange,
-} from "@xynogen/pix-pretty/icon-catalog";
+import { type IconKey, icon, onIconModeChange } from "@xynogen/pix-pretty/icon-catalog";
 
 /**
  * Each optimizer tool (caveman/rtk/toon/ponytail) exposes a handle so the
@@ -49,7 +45,7 @@ export type OptimizerTool = "caveman" | "rtk" | "toon" | "ponytail";
 
 /**
  * Icons come from the shared pix-pretty catalog and follow the ONE global icon
- * mode (set via /pretty). The optimizer no longer owns a mode: each tool maps
+ * mode (set via /pix). The optimizer no longer owns a mode: each tool maps
  * to a semantic catalog key, and the cell renders whatever the active mode
  * resolves — nerd / unicode / ascii — with no per-package knob.
  */
@@ -66,12 +62,7 @@ export function toolIcon(tool: OptimizerTool): string {
 }
 
 /** Fixed left-to-right order of icons in the cell. */
-const TOOL_ORDER: readonly OptimizerTool[] = [
-	"caveman",
-	"rtk",
-	"toon",
-	"ponytail",
-];
+const TOOL_ORDER: readonly OptimizerTool[] = ["caveman", "rtk", "toon", "ponytail"];
 
 /** Theme color for enabled icons. */
 const ENABLED_COLOR: ThemeColor = "accent";
@@ -85,7 +76,7 @@ export type Colorize = (color: ThemeColor, text: string) => string;
  * Build the colored status string for a set of tool states. ALL tool icons are
  * always shown, in TOOL_ORDER; each is accent-colored when its tool is enabled
  * and dim when disabled. Glyphs resolve against the active global icon mode
- * (see /pretty). `color` applies the theme color (e.g. theme.fg).
+ * (see /pix). `color` applies the theme color (e.g. theme.fg).
  *
  * Pure + exported for tests (pass a tagging colorizer to assert per-icon color).
  * A trailing space separates the cell from the next status segment.
@@ -111,7 +102,7 @@ export class OptimizerStatus {
 
 	constructor() {
 		// The cell is a pushed status (setStatus), so it does NOT auto-refresh
-		// when /pretty flips the global icon mode. Repaint it on mode change
+		// when /pix flips the global icon mode. Repaint it on mode change
 		// using the last-seen ctx (no-op until the cell has painted once).
 		onIconModeChange(() => {
 			if (this.lastCtx) this.paint(this.lastCtx);
@@ -124,11 +115,7 @@ export class OptimizerStatus {
 	}
 
 	/** Update one tool's enabled state and repaint the shared cell. */
-	set(
-		tool: OptimizerTool,
-		enabled: boolean,
-		ctx: Pick<ExtensionContext, "ui">,
-	): void {
+	set(tool: OptimizerTool, enabled: boolean, ctx: Pick<ExtensionContext, "ui">): void {
 		this.states[tool] = enabled;
 		this.paint(ctx);
 	}

package/src/tool-result-filter.test.ts

@@ -1,8 +1,5 @@
 import { describe, expect, it } from "bun:test";
-import {
-	filterModelWarnings,
-	stripModelWarningParagraphs,
-} from "./tool-result-filter.ts";
+import { filterModelWarnings, stripModelWarningParagraphs } from "./tool-result-filter.ts";
 
 // ── stripModelWarningParagraphs ───────────────────────────────────────────────
 
@@ -29,8 +26,7 @@ describe("stripModelWarningParagraphs", () => {
 	});
 
 	it("strips multiple warning paragraphs", () => {
-		const text =
-			"output\n\n⚠ BLIND WRITE — foo\n\n🔴 THRASHING — bar\n\nmore output";
+		const text = "output\n\n⚠ BLIND WRITE — foo\n\n🔴 THRASHING — bar\n\nmore output";
 		const result = stripModelWarningParagraphs(text);
 		expect(result).not.toContain("BLIND WRITE");
 		expect(result).not.toContain("THRASHING");
@@ -75,9 +71,7 @@ describe("filterModelWarnings", () => {
 		expect(result).not.toBe(content);
 		expect(result[0]?.type).toBe("text");
 		if (result[0]?.type === "text") {
-			expect((result[0] as { type: string; text: string }).text).not.toContain(
-				"BLIND WRITE",
-			);
+			expect((result[0] as { type: string; text: string }).text).not.toContain("BLIND WRITE");
 		}
 	});
 

package/src/tool-result-filter.ts

@@ -28,10 +28,7 @@ export function stripModelWarningParagraphs(text: string): string {
 	// then rejoin. Trim trailing whitespace from the result.
 	const paragraphs = text.split(/\n\n/);
 	const kept = paragraphs.filter(
-		(p) =>
-			!MODEL_WARNING_PREFIXES.some((prefix) =>
-				p.trimStart().startsWith(prefix),
-			),
+		(p) => !MODEL_WARNING_PREFIXES.some((prefix) => p.trimStart().startsWith(prefix)),
 	);
 	if (kept.length === paragraphs.length) return text;
 	return kept.join("\n\n").trimEnd();