@xynogen/pix-models 0.1.15 → 0.1.16

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@xynogen/pix-models",
-	"version": "0.1.15",
+	"version": "0.1.16",
 	"description": "Pi extension — enhanced /models picker with BenchLM ranks",
 	"type": "module",
 	"main": "src/index.ts",

package/src/models.test.ts

@@ -16,8 +16,7 @@ describe("fmtCtx", () => {
 });
 
 describe("fmtCost", () => {
-	it("returns — for undefined entry", () =>
-		expect(fmtCost(undefined)).toBe("—"));
+	it("returns — for undefined entry", () => expect(fmtCost(undefined)).toBe("—"));
 	it("returns — when no cost field", () => expect(fmtCost({})).toBe("—"));
 	it("returns free when both 0", () => {
 		expect(fmtCost({ cost: { input: 0, output: 0 } })).toBe("free");
@@ -118,11 +117,6 @@ describe("sortModels", () => {
 			{ provider: "a", id: "m4", name: "Delta", score: 60, tier: 0 },
 		];
 		const sorted = sortModels(mixed);
-		expect(sorted.map((m) => m.name)).toEqual([
-			"Alpha",
-			"Delta",
-			"Beta",
-			"Gamma",
-		]);
+		expect(sorted.map((m) => m.name)).toEqual(["Alpha", "Delta", "Beta", "Gamma"]);
 	});
 });

package/src/models.ts

@@ -8,10 +8,7 @@
  * Sorted by benchlm rank when available (best first), then alphabetical.
  */
 
-import type {
-	ExtensionAPI,
-	ExtensionContext,
-} from "@earendil-works/pi-coding-agent";
+import type { ExtensionAPI, ExtensionContext } from "@earendil-works/pi-coding-agent";
 import {
 	fuzzyFilter,
 	Input,
@@ -20,11 +17,7 @@ import {
 	SelectList,
 	visibleWidth,
 } from "@earendil-works/pi-tui";
-import {
-	benchScoreColor,
-	lookupBenchmark,
-	lookupModelsDev,
-} from "@xynogen/pix-data";
+import { benchScoreColor, lookupBenchmark, lookupModelsDev } from "@xynogen/pix-data";
 import { icon } from "@xynogen/pix-pretty/icon-catalog";
 import { frameLines, modalWidth } from "@xynogen/pix-pretty/modal-frame";
 import { patchOutBuiltinModelCommand } from "./patch-builtin";
@@ -40,9 +33,7 @@ export function fmtCtx(n: number): string {
 	return `${Math.round(n / 1_000)}k`;
 }
 
-export function fmtCost(
-	entry: { cost?: { input?: number; output?: number } } | undefined,
-): string {
+export function fmtCost(entry: { cost?: { input?: number; output?: number } } | undefined): string {
 	if (!entry?.cost) return "\u2014";
 	const i = entry.cost.input ?? 0;
 	const o = entry.cost.output ?? 0;
@@ -96,10 +87,7 @@ export function sortModels<T extends SortableModel>(models: T[]): T[] {
 	});
 }
 
-async function showEnhancedPicker(
-	pi: ExtensionAPI,
-	ctx: ExtensionContext,
-): Promise<void> {
+async function showEnhancedPicker(pi: ExtensionAPI, ctx: ExtensionContext): Promise<void> {
 	// Mirror the built-in /model selector, which calls refresh() then awaits
 	// getAvailable() (see model-selector.js). Without refresh(), this extension
 	// reads whatever `this.models` was last loaded into — which, depending on
@@ -182,9 +170,7 @@ async function showEnhancedPicker(
 
 			// Find max rank width across all benchmarked rows for # padding
 			const maxRankWidth = Math.max(
-				...dedupedRows.map((r) =>
-					r.localRank ? String(r.localRank).length : 0,
-				),
+				...dedupedRows.map((r) => (r.localRank ? String(r.localRank).length : 0)),
 				1,
 			);
 
@@ -198,84 +184,74 @@ async function showEnhancedPicker(
 				if (localRank) rankByValue.set(`${m.provider}/${m.id}`, localRank);
 			}
 
-			const items: SelectItem[] = dedupedRows.map(
-				({ m, dev, bench, localRank }) => {
-					const isCurrent =
-						current && m.provider === current.provider && m.id === current.id;
-
-					// Label: marker + rank cell + accent-colored model name.
-					// Ranked models show muted '#' + colored rank. Unranked (no
-					// modelgrep entry) show a muted em-dash sized to the rank
-					// column, so the model name aligns across rows.
-					const marker = isCurrent ? theme.fg(accent, "▶") : " ";
-					let rankPrefix: string;
-					if (localRank) {
-						const rankStr = String(localRank).padEnd(maxRankWidth);
-						// Color rank by the model's bench score (same scale as ⚡score),
-						// not by list position — keeps the two colors consistent.
-						const rankColor = benchScoreColor(bench?.overallScore);
-						rankPrefix = mute("#") + theme.fg(rankColor, rankStr);
-					} else {
-						// Width = "#" + maxRankWidth chars (e.g. "#   " or "#——" for 2-digit ranks).
-						const dash = "—".padEnd(maxRankWidth, " ");
-						rankPrefix = mute("#") + mute(dash);
-					}
-					// Display model id only; m.provider is routing provider, not part of id.
-					const idColored = theme.fg(accent, m.id);
-					const label = `${marker} ${rankPrefix} ${idColored}`;
-
-					// Description: ctx · cost · score stars
-					// Colors: ctx muted · cost success (free muted) · score+stars warning
-					const ctxRaw = fmtCtx(dev?.limit?.context ?? 0);
-					const ctxStr = mute(ctxRaw.padStart(4));
-					const rawCost = fmtCost(dev);
-					let costSeg: string;
-					if (rawCost === "—") {
-						costSeg = theme.fg("dim", "—".padEnd(10));
-					} else if (rawCost === "free") {
-						costSeg = mute("free".padEnd(10));
-					} else {
-						costSeg = theme.fg("success", rawCost.padEnd(10));
-					}
-					let benchSeg = "";
-					if (bench) {
-						const score = bench.overallScore ?? "?";
-						const s = bench.overallScore;
-						const scoreColor = benchScoreColor(s);
-						let filled = 1;
-						if (typeof s === "number") {
-							if (s >= 90) filled = 5;
-							else if (s >= 80) filled = 4;
-							else if (s >= 70) filled = 3;
-							else if (s >= 50) filled = 2;
-						}
-						const starBar =
-							theme.fg(scoreColor, "★".repeat(filled)) +
-							mute("☆".repeat(5 - filled));
-						benchSeg = `⚡${theme.fg(scoreColor, String(score))} ${starBar}`;
+			const items: SelectItem[] = dedupedRows.map(({ m, dev, bench, localRank }) => {
+				const isCurrent = current && m.provider === current.provider && m.id === current.id;
+
+				// Label: marker + rank cell + accent-colored model name.
+				// Ranked models show muted '#' + colored rank. Unranked (no
+				// modelgrep entry) show a muted em-dash sized to the rank
+				// column, so the model name aligns across rows.
+				const marker = isCurrent ? theme.fg(accent, "▶") : " ";
+				let rankPrefix: string;
+				if (localRank) {
+					const rankStr = String(localRank).padEnd(maxRankWidth);
+					// Color rank by the model's bench score (same scale as ⚡score),
+					// not by list position — keeps the two colors consistent.
+					const rankColor = benchScoreColor(bench?.overallScore);
+					rankPrefix = mute("#") + theme.fg(rankColor, rankStr);
+				} else {
+					// Width = "#" + maxRankWidth chars (e.g. "#   " or "#——" for 2-digit ranks).
+					const dash = "—".padEnd(maxRankWidth, " ");
+					rankPrefix = mute("#") + mute(dash);
+				}
+				// Display model id only; m.provider is routing provider, not part of id.
+				const idColored = theme.fg(accent, m.id);
+				const label = `${marker} ${rankPrefix} ${idColored}`;
+
+				// Description: ctx · cost · score stars
+				// Colors: ctx muted · cost success (free muted) · score+stars warning
+				const ctxRaw = fmtCtx(dev?.limit?.context ?? 0);
+				const ctxStr = mute(ctxRaw.padStart(4));
+				const rawCost = fmtCost(dev);
+				let costSeg: string;
+				if (rawCost === "—") {
+					costSeg = theme.fg("dim", "—".padEnd(10));
+				} else if (rawCost === "free") {
+					costSeg = mute("free".padEnd(10));
+				} else {
+					costSeg = theme.fg("success", rawCost.padEnd(10));
+				}
+				let benchSeg = "";
+				if (bench) {
+					const score = bench.overallScore ?? "?";
+					const s = bench.overallScore;
+					const scoreColor = benchScoreColor(s);
+					let filled = 1;
+					if (typeof s === "number") {
+						if (s >= 90) filled = 5;
+						else if (s >= 80) filled = 4;
+						else if (s >= 70) filled = 3;
+						else if (s >= 50) filled = 2;
 					}
-					const desc = [ctxStr, costSeg, benchSeg].filter(Boolean).join(sep);
+					const starBar = theme.fg(scoreColor, "★".repeat(filled)) + mute("☆".repeat(5 - filled));
+					benchSeg = `⚡${theme.fg(scoreColor, String(score))} ${starBar}`;
+				}
+				const desc = [ctxStr, costSeg, benchSeg].filter(Boolean).join(sep);
 
-					return {
-						value: `${m.provider}/${m.id}`,
-						label,
-						description: desc,
-					};
-				},
-			);
+				return {
+					value: `${m.provider}/${m.id}`,
+					label,
+					description: desc,
+				};
+			});
 
 			const currentIdx = current
-				? items.findIndex(
-						(it) => it.value === `${current.provider}/${current.id}`,
-					)
+				? items.findIndex((it) => it.value === `${current.provider}/${current.id}`)
 				: 0;
 
 			// Widest label (visible width, ANSI-stripped) so the model name
 			// column never truncates to "…". Add gap headroom.
-			const widestLabel = items.reduce(
-				(w, it) => Math.max(w, visibleWidth(it.label)),
-				0,
-			);
+			const widestLabel = items.reduce((w, it) => Math.max(w, visibleWidth(it.label)), 0);
 
 			const search = new Input();
 			const list = new SelectList(
@@ -313,15 +289,9 @@ async function showEnhancedPicker(
 				} else if (/^\d+$/.test(q)) {
 					// Pure number → match by benchlm rank, not name.
 					const wanted = Number(q);
-					next = internal.items.filter(
-						(it) => rankByValue.get(it.value) === wanted,
-					);
+					next = internal.items.filter((it) => rankByValue.get(it.value) === wanted);
 				} else {
-					next = fuzzyFilter(
-						internal.items,
-						q,
-						(it) => `${it.label} ${it.description ?? ""}`,
-					);
+					next = fuzzyFilter(internal.items, q, (it) => `${it.label} ${it.description ?? ""}`);
 					// Stable sort: ranked models (by rank asc) before unranked.
 					next = next
 						.map((it, i) => ({ it, i }))
@@ -343,21 +313,12 @@ async function showEnhancedPicker(
 					const mw = modalWidth(w);
 					const inner = mw - 4; // CHROME = 2 border + 2 padding
 					const lines: string[] = [
-						theme.fg(
-							accent,
-							theme.bold(`${icon("picker.model")}  Select model`),
-						),
-						theme.fg(
-							"dim",
-							"context · pricing · coding rank & score from modelgrep.com",
-						),
+						theme.fg(accent, theme.bold(`${icon("picker.model")}  Select model`)),
+						theme.fg("dim", "context · pricing · coding rank & score from modelgrep.com"),
 						theme.fg("muted", "Search:"),
 						...search.render(inner),
 						...list.render(inner),
-						theme.fg(
-							"dim",
-							"fuzzy search · ↑↓ navigate · enter select · esc cancel",
-						),
+						theme.fg("dim", "fuzzy search · ↑↓ navigate · enter select · esc cancel"),
 					];
 					return frameLines({
 						width: mw,

package/src/patch-builtin.test.ts

@@ -5,8 +5,7 @@ import { join } from "node:path";
 
 // Pure replacement tested in isolation (the exported fn resolves the host
 // package, which isn't present in the test sandbox).
-const MODEL_COMMAND_LINE =
-	'{ name: "model", description: "Select model (opens selector UI)" },';
+const MODEL_COMMAND_LINE = '{ name: "model", description: "Select model (opens selector UI)" },';
 
 function escapeRegExp(text: string): string {
 	return text.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
@@ -14,10 +13,7 @@ function escapeRegExp(text: string): string {
 
 function patchSource(source: string): string {
 	if (!source.includes(MODEL_COMMAND_LINE)) return source;
-	return source.replace(
-		new RegExp(`[ \\t]*${escapeRegExp(MODEL_COMMAND_LINE)}\\n?`),
-		"",
-	);
+	return source.replace(new RegExp(`[ \\t]*${escapeRegExp(MODEL_COMMAND_LINE)}\\n?`), "");
 }
 
 const UNPATCHED = `export const BUILTIN_SLASH_COMMANDS = [

package/src/patch-builtin.ts

@@ -19,8 +19,7 @@ import { createRequire } from "node:module";
 import { homedir } from "node:os";
 import { dirname, join, resolve } from "node:path";
 
-const MODEL_COMMAND_LINE =
-	'{ name: "model", description: "Select model (opens selector UI)" },';
+const MODEL_COMMAND_LINE = '{ name: "model", description: "Select model (opens selector UI)" },';
 
 /** Candidate slash-commands.js paths, most-specific first. */
 function candidatePaths(): string[] {
@@ -51,14 +50,7 @@ function candidatePaths(): string[] {
 	];
 	for (const root of globalRoots) {
 		paths.push(
-			join(
-				root,
-				"@earendil-works",
-				"pi-coding-agent",
-				"dist",
-				"core",
-				"slash-commands.js",
-			),
+			join(root, "@earendil-works", "pi-coding-agent", "dist", "core", "slash-commands.js"),
 		);
 	}
 
@@ -99,10 +91,7 @@ export function patchOutBuiltinModelCommand(): void {
 
 	if (!source.includes(MODEL_COMMAND_LINE)) return; // already patched
 
-	const patched = source.replace(
-		new RegExp(`[ \\t]*${escapeRegExp(MODEL_COMMAND_LINE)}\\n?`),
-		"",
-	);
+	const patched = source.replace(new RegExp(`[ \\t]*${escapeRegExp(MODEL_COMMAND_LINE)}\\n?`), "");
 	if (patched === source) return;
 
 	try {