@xynogen/pix-optimizer 1.1.14 → 1.1.16

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@xynogen/pix-optimizer",
-	"version": "1.1.14",
+	"version": "1.1.16",
 	"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",
@@ -49,7 +49,7 @@
 	},
 	"dependencies": {
 		"@xynogen/pix-pretty": "^1.7.9",
-		"@xynogen/pix-data": "workspace:*"
+		"@xynogen/pix-data": "^0.3.0"
 	},
 	"peerDependencies": {
 		"@earendil-works/pi-coding-agent": "*",

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,11 +19,7 @@ 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) {

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

@@ -182,8 +182,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 +208,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) => {
@@ -251,9 +243,7 @@ export function json(
 		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 })),
+				pi.exec("which", ["toon"], { timeout: 1000 }).catch(() => ({ code: 1 })),
 			]);
 			jqAvailable = jqRes.code === 0;
 			toonAvailable = toonRes.code === 0;
@@ -277,10 +267,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

@@ -19,11 +19,7 @@ describe("splitChain", () => {
 	});
 
 	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 +29,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 +53,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 +66,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 +92,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 +104,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 +114,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 +149,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 +230,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

@@ -51,10 +51,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 (
@@ -224,10 +221,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;
@@ -306,10 +300,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
@@ -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";
@@ -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();