@xynogen/pix-models 0.2.1 → 0.2.3

package/package.json

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

package/src/models.ts

@@ -231,6 +231,7 @@ async function showEnhancedPicker(pi: ExtensionAPI, ctx: ExtensionContext): Prom
 	// ModelRegistry instance (verified in runner.js) with refresh() and an
 	// async-capable getAvailable(). Reach through the narrowed type.
 	type AvailableModels = ReturnType<typeof ctx.modelRegistry.getAvailable>;
+	// SAFETY: Runtime ModelRegistry includes refresh and async availability missing from public types.
 	const registry = ctx.modelRegistry as unknown as {
 		refresh?: () => void;
 		getAvailable(): AvailableModels | Promise<AvailableModels>;
@@ -313,8 +314,8 @@ async function showEnhancedPicker(pi: ExtensionAPI, ctx: ExtensionContext): Prom
 			// Mute low-info parts (separators, padding, #, ☆) so the actual values pop.
 			const mute = (s: string) => theme.fg("muted", s);
 			const guide = (key: string, action: string) =>
-				theme.fg("text", key) + theme.fg("dim", ` ${action}`);
-			const guideSep = theme.fg("dim", " · ");
+				theme.fg("text", key) + theme.fg("muted", ` ${action}`);
+			const guideSep = theme.fg("muted", " · ");
 
 			// Track rank per item value so fuzzy results can prioritize ranked models.
 			const rankByValue = new Map<string, number>();
@@ -369,7 +370,7 @@ async function showEnhancedPicker(pi: ExtensionAPI, ctx: ExtensionContext): Prom
 				const rawCost = fmtCost(dev);
 				let costSeg: string;
 				if (rawCost === "—") {
-					costSeg = theme.fg("dim", "—".padEnd(maxCostWidth));
+					costSeg = theme.fg("muted", "—".padEnd(maxCostWidth));
 				} else if (rawCost === "free") {
 					costSeg = mute("free".padEnd(maxCostWidth));
 				} else {
@@ -415,7 +416,7 @@ async function showEnhancedPicker(pi: ExtensionAPI, ctx: ExtensionContext): Prom
 					selectedPrefix: (t) => theme.fg(accent, t),
 					selectedText: (t) => theme.fg(accent, t),
 					description: (t) => t, // raw — per-segment colors set in items.map
-					scrollInfo: (t) => theme.fg("dim", t),
+					scrollInfo: (t) => theme.fg("muted", t),
 					noMatch: (t) => theme.fg("warning", t),
 				},
 				{
@@ -430,6 +431,7 @@ async function showEnhancedPicker(pi: ExtensionAPI, ctx: ExtensionContext): Prom
 			search.onEscape = () => done(null);
 
 			const applyFuzzy = (query: string) => {
+				// SAFETY: SelectList exposes these stable fields internally for in-place filtering.
 				const internal = list as unknown as {
 					items: SelectItem[];
 					filteredItems: SelectItem[];
@@ -464,11 +466,11 @@ async function showEnhancedPicker(pi: ExtensionAPI, ctx: ExtensionContext): Prom
 					? theme.getThinkingBorderColor(resolved)(label)
 					: theme.fg("dim", label);
 				return (
-					theme.fg("muted", "Thinking: ") +
+					theme.fg("dim", "Thinking: ") +
 					coloredLabel +
-					theme.fg("dim", "  (") +
+					theme.fg("muted", "  (") +
 					guide("←/→", "adjust") +
-					theme.fg("dim", ")")
+					theme.fg("muted", ")")
 				);
 			};
 
@@ -489,6 +491,7 @@ async function showEnhancedPicker(pi: ExtensionAPI, ctx: ExtensionContext): Prom
 						],
 						body: list.render(inner),
 						selectedBodyRange: (() => {
+							// SAFETY: SelectList tracks selectedIndex internally for pager synchronization.
 							const internal = list as unknown as { selectedIndex: number };
 							return pager.selectedRange({
 								start: internal.selectedIndex,

package/src/patch-builtin.ts

@@ -19,11 +19,31 @@
  *   3. createRequire against the extension's own node_modules.
  */
 
-import { execSync } from "node:child_process";
-import { existsSync, readdirSync, readFileSync, writeFileSync } from "node:fs";
+import {
+	accessSync,
+	constants,
+	existsSync,
+	readdirSync,
+	readFileSync,
+	realpathSync,
+	writeFileSync,
+} from "node:fs";
 import { createRequire } from "node:module";
 import { homedir } from "node:os";
-import { join, sep } from "node:path";
+import { delimiter, join, sep } from "node:path";
+
+/** Locate `pi` on PATH (a pure-JS `which`), returning its real (symlink-resolved) path. */
+function resolvePiBinary(): string | undefined {
+	for (const dir of (process.env.PATH ?? "").split(delimiter)) {
+		if (!dir) continue;
+		const candidate = join(dir, "pi");
+		try {
+			accessSync(candidate, constants.X_OK);
+			return realpathSync(candidate);
+		} catch {}
+	}
+	return undefined;
+}
 
 // The assignment appears once per build. `\s*=\s*\[` avoids the `.map(...)`
 // references and tolerates both `= [` (pretty) and `=[` (minified) spacing.
@@ -42,16 +62,11 @@ function packageRoots(): string[] {
 		if (r && !roots.includes(r)) roots.push(r);
 	};
 
-	// 1. Resolve via the running `pi` binary → realpath → strip at /dist/.
-	try {
-		const piReal = execSync("realpath $(which pi)", {
-			encoding: "utf8",
-			stdio: ["pipe", "pipe", "pipe"],
-		}).trim();
+	// 1. Resolve via the `pi` binary on PATH → realpath → strip at /dist/.
+	const piReal = resolvePiBinary();
+	if (piReal) {
 		const idx = piReal.indexOf(`${sep}dist${sep}`);
 		if (idx >= 0) pushRoot(piReal.slice(0, idx));
-	} catch {
-		// `pi` not on PATH or `which`/`realpath` unavailable — skip
 	}
 
 	// 2. Well-known global install locations.