@xynogen/pix-models 0.2.6 → 0.2.8

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@xynogen/pix-models",
-	"version": "0.2.6",
+	"version": "0.2.8",
 	"description": "Pi extension — enhanced /models picker with BenchLM ranks",
 	"type": "module",
 	"main": "src/index.ts",
@@ -45,7 +45,7 @@
 	"dependencies": {
 		"@xynogen/pix-data": "^0.5.0",
 		"@xynogen/pix-pretty": "^1.11.2",
-		"@xynogen/pix-runtime": "^0.8.0"
+		"@xynogen/pix-runtime": "^0.10.0"
 	},
 	"peerDependencies": {
 		"@earendil-works/pi-coding-agent": "*",

package/src/models.ts

@@ -8,7 +8,11 @@
  * Sorted by benchlm rank when available (best first), then alphabetical.
  */
 
-import type { ExtensionAPI, ExtensionContext } from "@earendil-works/pi-coding-agent";
+import {
+	type ExtensionAPI,
+	type ExtensionContext,
+	SettingsManager,
+} from "@earendil-works/pi-coding-agent";
 import {
 	fuzzyFilter,
 	Input,
@@ -574,8 +578,23 @@ async function showEnhancedPicker(pi: ExtensionAPI, ctx: ExtensionContext): Prom
 		return;
 	}
 	const ok = await pi.setModel(picked);
-	if (ok) ctx.ui.notify(`Switched to ${picked.name ?? picked.id}`, "info");
-	else ctx.ui.notify(`Failed to switch to ${picked.id}`, "error");
+	if (!ok) {
+		ctx.ui.notify(`Failed to switch to ${picked.id}`, "error");
+		return;
+	}
+
+	const settings = SettingsManager.create(ctx.cwd);
+	settings.setDefaultModelAndProvider(picked.provider, picked.id);
+	await settings.flush();
+	const saveError = settings.drainErrors()[0];
+	if (saveError) {
+		ctx.ui.notify(
+			`Switched to ${picked.name ?? picked.id}, but failed to save the default: ${saveError.error.message}`,
+			"warning",
+		);
+		return;
+	}
+	ctx.ui.notify(`Default model: ${picked.provider}/${picked.id}`, "info");
 }
 
 export default function modelPickerExtension(pi: ExtensionAPI) {

package/src/patch-builtin.ts

@@ -19,30 +19,21 @@
  *   3. createRequire against the extension's own node_modules.
  */
 
-import {
-	accessSync,
-	constants,
-	existsSync,
-	readdirSync,
-	readFileSync,
-	realpathSync,
-	writeFileSync,
-} from "node:fs";
+import { existsSync, readdirSync, readFileSync, realpathSync, writeFileSync } from "node:fs";
 import { createRequire } from "node:module";
 import { homedir } from "node:os";
-import { delimiter, join, sep } from "node:path";
+import { join, sep } from "node:path";
+import { findExecutableSync } from "@xynogen/pix-runtime/which";
 
-/** Locate `pi` on PATH (a pure-JS `which`), returning its real (symlink-resolved) path. */
+/** Locate `pi` on PATH, 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 {}
+	const found = findExecutableSync("pi");
+	if (!found) return undefined;
+	try {
+		return realpathSync(found);
+	} catch {
+		return found;
 	}
-	return undefined;
 }
 
 // The assignment appears once per build. `\s*=\s*\[` avoids the `.map(...)`