@xynogen/pix-optimizer 1.1.8 → 1.1.10

package/README.md

@@ -4,10 +4,10 @@ Token-optimization suite for Pi Coding Agent. Four tools wired into one
 extension via `src/index.ts`, fronted by a single `/optimizer` command and one
 shared status-bar cell:
 
-- **Caveman** (`󰜐`) — terse-output system prompt
-- **RTK** (`󰓥`) — prefixes shell commands with `rtk` + injects RTK prompt
-- **TOON** (`󰗀`) — jq + TOON guidance for dense JSON (skill lives in pix-skills)
-- **Ponytail** (`󰆐`) — lazy-senior-dev system prompt (minimal code, YAGNI)
+- **Caveman** (`Cv`) — terse-output system prompt
+- **RTK** (`Rk`) — prefixes shell commands with `rtk` + injects RTK prompt
+- **TOON** (`Tn`) — jq + TOON guidance for dense JSON (skill lives in pix-skills)
+- **Ponytail** (`Pt`) — lazy-senior-dev system prompt (minimal code, YAGNI)
 
 ## Command
 
@@ -25,12 +25,31 @@ State persists to `~/.pi/agent/optimizer.json` (caveman/rtk/toon/ponytail).
 
 ## Status bar
 
-A single cell always shows the enabled icons in a fixed order (`󰜐 󰓥 󰗀 󰆐`), color-
-coded by state: **accent** when the tool is enabled, **dim** when disabled.
+A single cell always shows all four tool icons in a fixed order, color-coded by
+state: **accent** when the tool is enabled, **dim** when disabled.
+
+### Icon style (Nerd Font, Unicode, or ASCII)
+
+The default glyphs (Nerd Font PUA codepoints) are **Nerd Font** symbols and require a patched
+font (e.g. MesloLGS NF). Terminals without one render them as missing-glyph
+“tofu” boxes. Two font-independent fallbacks are available:
+
+| Mode | Glyphs | Needs Nerd Font? |
+|---|---|---|
+| `nerd` (default) | Nerd Font PUA glyphs | yes |
+| `unicode` | `♤ ♡ ♢ ♧` (outline card suits) | no |
+| `ascii` | `Cv Rk Tn Pt` | no |
+
+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`
+env var. The optimizer no longer has its own toggle.
 
 ## Features
 
-### Caveman Mode (`󰜐`)
+### Caveman Mode (`Cv`)
 
 Cuts ~75% of output tokens while keeping full technical accuracy.
 
@@ -44,7 +63,7 @@ Cuts ~75% of output tokens while keeping full technical accuracy.
 The `/optimizer` overlay opens a settings dialog when needed. Default level
 for new sessions is restored from `~/.pi/agent/optimizer.json`.
 
-### RTK Tool Rewriting (`󰓥`)
+### RTK Tool Rewriting (`Rk`)
 
 Two layers, both active automatically:
 
@@ -65,7 +84,7 @@ Two layers, both active automatically:
 cargo install rtk-ai
 ```
 
-### TOON / JSON Compression (`󰗀`)
+### TOON / JSON Compression (`Tn`)
 
 Guidance for handling information-dense JSON via `jq` (query/reshape) and
 `toon` (compress). The system-prompt nudge is injected **only when the user
@@ -82,7 +101,7 @@ The `toon-json` skill (full workflow + when-NOT-to-use guidance) is bundled in
 npm i -g @toon-format/cli
 ```
 
-### Ponytail Mode (`󰆐`)
+### Ponytail Mode (`Pt`)
 
 "Lazy senior dev" mode. Governs **what** the agent builds (minimal code,
 YAGNI), orthogonal to Caveman which governs **how** it talks — they pair. Before
@@ -117,7 +136,7 @@ pi install npm:@xynogen/pix-optimizer
 |-------------------|-----------------------------------------------------------|
 | `src/index.ts`    | Wires the four tools + shared status, registers `/optimizer` |
 | `src/opt.ts`      | The `/optimizer` overlay UI (keyboard nav + cycling)     |
-| `src/status.ts`   | Shared status-bar cell + `OptimizerHandle` contract       |
+| `src/status.ts`   | Shared status-bar cell (`toolIcon()` → shared `pix-pretty` catalog) |
 | `src/caveman.ts`  | Caveman logic, levels, prompt                            |
 | `src/rtk.ts`      | RTK prompt + bash command rewriting                       |
 | `src/json.ts`     | jq+TOON guidance, heuristics, system-prompt injection     |

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@xynogen/pix-optimizer",
-	"version": "1.1.8",
+	"version": "1.1.10",
 	"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/index.ts

@@ -7,9 +7,10 @@
  *   - toon:     jq + TOON guidance for dense JSON (+ bundled skill)
  *   - ponytail: lazy-senior-dev system prompt (minimal code, YAGNI)
  *
- * They share ONE status-bar cell (󰜐 󰓥 󰗀 󰆐, all four always shown — dimmed when off, accented when on) and ONE
- * command (/optimizer — an interactive overlay). index.ts wires lifecycle hooks
- * via each module, then registers the overlay command from their handles.
+ * They share ONE status-bar cell (all four icons always shown — dimmed when
+ * off, accented when on; icon style is nerd/unicode/ascii, set via /optimizer)
+ * and ONE command (/optimizer — an interactive overlay). index.ts wires
+ * lifecycle hooks via each module, then registers the overlay command.
  */
 
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
@@ -26,6 +27,7 @@ import {
 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.
 	const status = new OptimizerStatus();
 
 	// Each module registers its own lifecycle hooks and returns a handle the

package/src/opt.ts

@@ -16,12 +16,12 @@
 
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 import { frameLines } from "@xynogen/pix-pretty/modal-frame";
-import {
-	type OptimizerHandle,
-	type OptimizerStatus,
-	type OptimizerTool,
-	TOOL_ICONS,
+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). */
 const MIN_CONTENT = 28;
@@ -150,7 +150,7 @@ export function registerOptCommand(
 								const on = handles[tool].current() !== "off";
 								const sel = i === selected;
 								const cursor = sel ? theme.fg("accent", "→") : " ";
-								const icon = theme.fg(on ? "accent" : "dim", TOOL_ICONS[tool]);
+								const glyph = theme.fg(on ? "accent" : "dim", toolIcon(tool));
 								const name = theme.fg(
 									sel ? "accent" : on ? "text" : "muted",
 									tool.padEnd(nameWidth),
@@ -164,7 +164,7 @@ export function registerOptCommand(
 									on ? "accent" : "dim",
 									levelBar(cur, handles[tool].values),
 								);
-								return `${cursor} ${icon}  ${name}  ${value}  ${bar}`;
+								return `${cursor} ${glyph}  ${name}  ${value}  ${bar}`;
 							});
 							const lines = [
 								theme.fg("accent", theme.bold("󱎫  Optimizer")),

package/src/status.test.ts

@@ -3,12 +3,18 @@ import {
 	OptimizerStatus,
 	renderStatus,
 	STATUS_KEY,
-	TOOL_ICONS,
+	toolIcon,
 } from "./status.ts";
 
 /** Tagging colorizer: <accent>X</accent> / <dim>X</dim> for assertions. */
 const tag = (c: string, t: string) => `<${c}>${t}</${c}>`;
 
+// Icons resolve against the global pix-pretty mode (default "nerd" in tests).
+const CV = toolIcon("caveman");
+const RK = toolIcon("rtk");
+const TN = toolIcon("toon");
+const PT = toolIcon("ponytail");
+
 describe("renderStatus", () => {
 	it("shows ALL icons in order, accent when enabled", () => {
 		expect(
@@ -17,7 +23,7 @@ describe("renderStatus", () => {
 				tag,
 			),
 		).toBe(
-			`<accent>${TOOL_ICONS.caveman}</accent>  <accent>${TOOL_ICONS.rtk}</accent>  <accent>${TOOL_ICONS.toon}</accent>  <accent>${TOOL_ICONS.ponytail}</accent> `,
+			`<accent>${CV}</accent>  <accent>${RK}</accent>  <accent>${TN}</accent>  <accent>${PT}</accent> `,
 		);
 	});
 
@@ -28,21 +34,26 @@ describe("renderStatus", () => {
 				tag,
 			),
 		).toBe(
-			`<dim>${TOOL_ICONS.caveman}</dim>  <accent>${TOOL_ICONS.rtk}</accent>  <accent>${TOOL_ICONS.toon}</accent>  <accent>${TOOL_ICONS.ponytail}</accent> `,
+			`<dim>${CV}</dim>  <accent>${RK}</accent>  <accent>${TN}</accent>  <accent>${PT}</accent> `,
 		);
 	});
 
 	it("all dim when nothing enabled (cell never empty)", () => {
 		expect(renderStatus({}, tag)).toBe(
-			`<dim>${TOOL_ICONS.caveman}</dim>  <dim>${TOOL_ICONS.rtk}</dim>  <dim>${TOOL_ICONS.toon}</dim>  <dim>${TOOL_ICONS.ponytail}</dim> `,
+			`<dim>${CV}</dim>  <dim>${RK}</dim>  <dim>${TN}</dim>  <dim>${PT}</dim> `,
 		);
 	});
 
 	it("preserves fixed order regardless of insertion order", () => {
 		expect(renderStatus({ toon: true, caveman: true }, tag)).toBe(
-			`<accent>${TOOL_ICONS.caveman}</accent>  <dim>${TOOL_ICONS.rtk}</dim>  <accent>${TOOL_ICONS.toon}</accent>  <dim>${TOOL_ICONS.ponytail}</dim> `,
+			`<accent>${CV}</accent>  <dim>${RK}</dim>  <accent>${TN}</accent>  <dim>${PT}</dim> `,
 		);
 	});
+
+	it("resolves icons via the shared catalog (non-empty glyphs)", () => {
+		// toolIcon delegates to pix-pretty's icon(); every tool has a glyph.
+		expect([CV, RK, TN, PT].every((g) => g.length > 0)).toBe(true);
+	});
 });
 
 describe("OptimizerStatus", () => {
@@ -67,7 +78,7 @@ describe("OptimizerStatus", () => {
 		expect(last.key).toBe(STATUS_KEY);
 		// caveman + toon + ponytail still unset (dim), rtk accent.
 		expect(last.text).toBe(
-			`<dim>${TOOL_ICONS.caveman}</dim>  <accent>${TOOL_ICONS.rtk}</accent>  <dim>${TOOL_ICONS.toon}</dim>  <dim>${TOOL_ICONS.ponytail}</dim> `,
+			`<dim>${CV}</dim>  <accent>${RK}</accent>  <dim>${TN}</dim>  <dim>${PT}</dim> `,
 		);
 	});
 
@@ -78,7 +89,7 @@ describe("OptimizerStatus", () => {
 		status.set("toon", true, ctx as never);
 		const last = ctx.calls.at(-1)!;
 		expect(last.text).toBe(
-			`<accent>${TOOL_ICONS.caveman}</accent>  <dim>${TOOL_ICONS.rtk}</dim>  <accent>${TOOL_ICONS.toon}</accent>  <dim>${TOOL_ICONS.ponytail}</dim> `,
+			`<accent>${CV}</accent>  <dim>${RK}</dim>  <accent>${TN}</accent>  <dim>${PT}</dim> `,
 		);
 	});
 
@@ -89,7 +100,7 @@ describe("OptimizerStatus", () => {
 		status.set("rtk", false, ctx as never);
 		const last = ctx.calls.at(-1)!;
 		expect(last.text).toBe(
-			`<dim>${TOOL_ICONS.caveman}</dim>  <dim>${TOOL_ICONS.rtk}</dim>  <dim>${TOOL_ICONS.toon}</dim>  <dim>${TOOL_ICONS.ponytail}</dim> `,
+			`<dim>${CV}</dim>  <dim>${RK}</dim>  <dim>${TN}</dim>  <dim>${PT}</dim> `,
 		);
 	});
 });

package/src/status.ts

@@ -17,6 +17,11 @@ import type {
 	ExtensionContext,
 	ThemeColor,
 } from "@earendil-works/pi-coding-agent";
+import {
+	type IconKey,
+	icon,
+	onIconModeChange,
+} from "@xynogen/pix-pretty/icon-catalog";
 
 /**
  * Each optimizer tool (caveman/rtk/toon/ponytail) exposes a handle so the
@@ -42,13 +47,24 @@ export const STATUS_KEY = "pix-optimizer";
 /** Tools that participate in the shared indicator, in render order. */
 export type OptimizerTool = "caveman" | "rtk" | "toon" | "ponytail";
 
-export const TOOL_ICONS: Record<OptimizerTool, string> = {
-	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
+ * to a semantic catalog key, and the cell renders whatever the active mode
+ * resolves — nerd / unicode / ascii — with no per-package knob.
+ */
+const TOOL_ICON_KEY: Record<OptimizerTool, IconKey> = {
+	caveman: "opt.caveman",
+	rtk: "opt.rtk",
+	toon: "opt.toon",
+	ponytail: "opt.ponytail",
 };
 
+/** Current glyph for a tool, resolved against the active global icon mode. */
+export function toolIcon(tool: OptimizerTool): string {
+	return icon(TOOL_ICON_KEY[tool]);
+}
+
 /** Fixed left-to-right order of icons in the cell. */
 const TOOL_ORDER: readonly OptimizerTool[] = [
 	"caveman",
@@ -68,7 +84,8 @@ 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. `color` applies the theme color (e.g. theme.fg).
+ * and dim when disabled. Glyphs resolve against the active global icon mode
+ * (see /pretty). `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.
@@ -78,7 +95,7 @@ export function renderStatus(
 	color: Colorize,
 ): string {
 	return `${TOOL_ORDER.map((t) =>
-		color(states[t] === true ? ENABLED_COLOR : DISABLED_COLOR, TOOL_ICONS[t]),
+		color(states[t] === true ? ENABLED_COLOR : DISABLED_COLOR, toolIcon(t)),
 	).join("  ")} `;
 }
 
@@ -89,6 +106,17 @@ export function renderStatus(
  */
 export class OptimizerStatus {
 	private states: Partial<Record<OptimizerTool, boolean>> = {};
+	/** Latest ui ctx seen, so an icon-mode change can repaint without a toggle. */
+	private lastCtx: Pick<ExtensionContext, "ui"> | undefined;
+
+	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
+		// using the last-seen ctx (no-op until the cell has painted once).
+		onIconModeChange(() => {
+			if (this.lastCtx) this.paint(this.lastCtx);
+		});
+	}
 
 	/** Current enabled state for one tool (undefined until first set). */
 	get(tool: OptimizerTool): boolean | undefined {
@@ -107,6 +135,7 @@ export class OptimizerStatus {
 
 	/** Repaint the shared cell from current states. */
 	paint(ctx: Pick<ExtensionContext, "ui">): void {
+		this.lastCtx = ctx;
 		const text = renderStatus(this.states, (c, t) => ctx.ui.theme.fg(c, t));
 		ctx.ui.setStatus(STATUS_KEY, text);
 	}