@xynogen/pix-core 0.2.3 → 0.3.0

package/package.json

@@ -1,26 +1,15 @@
 {
 	"name": "@xynogen/pix-core",
-	"version": "0.2.3",
-	"description": "Pi extension — core UI/UX bundle (welcome banner, footer, model picker, self-update)",
+	"version": "0.3.0",
+	"description": "Pi extension bundle — installs all core pix-* packages",
 	"type": "module",
-	"main": "src/index.ts",
 	"scripts": {
 		"test": "bun test"
 	},
 	"files": [
-		"src",
-		"skills",
 		"README.md",
 		"LICENSE"
 	],
-	"pi": {
-		"extensions": [
-			"src/index.ts"
-		],
-		"skills": [
-			"./skills"
-		]
-	},
 	"keywords": [
 		"pi",
 		"pi-package",
@@ -42,12 +31,17 @@
 		"access": "public"
 	},
 	"dependencies": {
-		"@xynogen/pix-data": "*",
+		"@xynogen/pix-welcome": "*",
+		"@xynogen/pix-footer": "*",
+		"@xynogen/pix-diagnostics": "*",
+		"@xynogen/pix-prompts": "*",
 		"@xynogen/pix-skills": "*",
-		"typebox": "^1.1.38"
+		"@xynogen/pix-models": "*",
+		"@xynogen/pix-update": "*",
+		"@xynogen/pix-commands": "*",
+		"@xynogen/pix-nudge": "*"
 	},
 	"peerDependencies": {
-		"@earendil-works/pi-coding-agent": "*",
-		"@earendil-works/pi-tui": "*"
+		"@earendil-works/pi-coding-agent": "*"
 	}
 }

package/skills/ask-user/SKILL.md

@@ -1,48 +0,0 @@
----
-name: ask-user
-description: "MUST use before high-stakes/irreversible decisions or when requirements are ambiguous. Gather context, present 2-5 options via ask_user, get explicit choice, then proceed."
-metadata:
-  short-description: Decision gate for ambiguity and high-stakes choices
----
-
-# ask_user decision gate
-
-Decision control, not chit-chat.
-
-## Gate (call ask_user before proceeding if ANY true)
-- changes architecture/schema/API/deploy/security
-- costly to undo (big refactor, migration, destructive edit, prod behavior)
-- requirements unclear/conflicting/missing
-- multiple valid options, trade-off is preference-dependent
-- about to assume something that changes implementation
-
-Skip only if user already gave an explicit decision for THIS exact trade-off.
-
-## Handshake
-1. classify step: high_stakes | ambiguous | both | clear. clear → no gate.
-2. gather evidence first (read/bash/web/ref). don't ask blind.
-3. synthesize: 3-7 bullets — state, constraints, trade-offs, recommendation.
-4. ask ONE focused question: `question`, `context`(summary), `options`(2-5),
-   `allowMultiple:false` unless truly independent, `allowFreeform:true`.
-   `inline` displayMode when the preceding summary must stay visible.
-5. commit: restate decision, say next step, proceed.
-6. re-ask only on materially new ambiguity. no confirm loops.
-
-## Budget (anti-overasking)
-- max 1 call per boundary normally; max 2 if first is unclear/cancelled.
-- never re-ask same trade-off without new evidence.
-- attempt 2 = narrower question: [Proceed w/ recommended] [Choose other (freeform)] [Stop].
-- after attempt 2: high_stakes/both → STOP, mark blocked.
-  ambiguous-only + user says "your call" → take most reversible default, state assumptions.
-
-## Quality
-- question: concrete decision boundary, one decision only.
-- options: short, outcome-oriented, explicit trade-offs; add description when non-obvious.
-
-## Anti-patterns
-asking without context · trivial formatting choices · forcing options when freeform fits ·
-repeat questions w/o new info · proceeding high-stakes after unclear/cancelled answer.
-
-## On cancel / unclear
-Pause, explain what's blocked. At most one narrower follow-up.
-Then: high-stakes → stay blocked until explicit decision; ambiguity-only → proceed only if user delegated.

package/src/commands/agent-sop/agent-sop.ts

@@ -1,58 +0,0 @@
-/**
- * agent-sop — inject AGENT.md (from pix-skills) into system prompt
- *
- * Reads AGENT.md from the @xynogen/pix-skills package and appends it to the
- * system prompt on every agent start via `before_agent_start`.
- *
- * This is the "register skill" mechanism for the agent operating spec — no
- * static SKILL.md file needed. The content becomes part of the model's
- * standing instructions.
- */
-
-import { existsSync, readFileSync } from "node:fs";
-import { createRequire } from "node:module";
-import { resolve } from "node:path";
-import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
-
-/** Resolve the absolute path to AGENT.md inside @xynogen/pix-skills. */
-function resolveAgentMdPath(): string | null {
-	try {
-		const require = createRequire(import.meta.url);
-		const pkgJson = require.resolve("@xynogen/pix-skills/package.json");
-		return resolve(pkgJson, "..", "AGENT.md");
-	} catch {
-		return null;
-	}
-}
-
-/** Read and return AGENT.md content, or null if unavailable. */
-function loadAgentMd(): string | null {
-	const p = resolveAgentMdPath();
-	if (!p || !existsSync(p)) return null;
-	try {
-		return readFileSync(p, "utf-8");
-	} catch {
-		return null;
-	}
-}
-
-export default function registerAgentSop(pi: ExtensionAPI): void {
-	// Load once at startup (content is static per session).
-	const agentMdContent = loadAgentMd();
-
-	if (!agentMdContent) {
-		// Silent skip — pix-skills might not be installed.
-		return;
-	}
-
-	pi.on("before_agent_start", async (event) => {
-		// Skip if already injected (idempotent check via a simple marker).
-		if (event.systemPrompt.includes("pix-agent-sop")) {
-			return;
-		}
-
-		return {
-			systemPrompt: `${event.systemPrompt}\n\n<pix-agent-sop>\n${agentMdContent}\n</pix-agent-sop>`,
-		};
-	});
-}

package/src/commands/clear/clear.ts

@@ -1,32 +0,0 @@
-import type {
-	ExtensionAPI,
-	ExtensionCommandContext,
-} from "@earendil-works/pi-coding-agent";
-
-async function clearCache(pi: ExtensionAPI, ctx: ExtensionCommandContext) {
-	ctx.ui.notify("Clearing ~/.cache/pi", "info");
-	const result = await pi.exec("/bin/sh", ["-lc", 'rm -rf "$HOME/.cache/pi"'], {
-		timeout: 10_000,
-	});
-	const output = [result.stdout, result.stderr]
-		.filter(Boolean)
-		.join("\n")
-		.trim();
-	if ((result.code ?? 0) !== 0) {
-		ctx.ui.notify(`Cache clear failed. ${output || "No output."}`, "error");
-		return;
-	}
-	ctx.ui.notify(
-		"~/.cache/pi cleared. Run /reload to apply changes.",
-		"warning",
-	);
-}
-
-export default function (pi: ExtensionAPI) {
-	pi.registerCommand("clear", {
-		description: "Remove ~/.cache/pi and reload",
-		handler: async (_args, ctx) => {
-			await clearCache(pi, ctx);
-		},
-	});
-}

package/src/commands/diff/diff.ts

@@ -1,32 +0,0 @@
-/**
- * /diff — explain unstaged git diff with per-file +/- counts.
- *
- * The agent runs `git status` + `git diff`, then replies with:
- *   1. 1–2 sentence explanation of what changed
- *   2. Per-file +/- line counts
- *   3. Total +/- line count
- */
-
-import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
-
-const DIFF_PROMPT = `Run git status and inspect the unstaged git diff, then respond with only:
-
-1. A short 1-2 sentence explanation of what changed and why it matters.
-2. A list of changed unstaged files with their +/- line counts.
-3. A total +/- line count at the bottom.
-
-Keep it concise. Use git commands to calculate the line counts. Base the summary on the actual diff, not only filenames. Do not include staged changes unless they also have unstaged modifications.`;
-
-export default function (pi: ExtensionAPI) {
-	pi.registerCommand("diff", {
-		description: "Explain unstaged git diff with per-file +/- counts",
-		handler: async (_args, ctx) => {
-			if (!ctx.isIdle()) {
-				pi.sendUserMessage(DIFF_PROMPT, { deliverAs: "followUp" });
-				ctx.ui.notify("Queued /diff after the current turn finishes.", "info");
-				return;
-			}
-			pi.sendUserMessage(DIFF_PROMPT);
-		},
-	});
-}

package/src/commands/models/models.test.ts

@@ -1,95 +0,0 @@
-import { describe, expect, it } from "bun:test";
-import { benchStars, fmtCost, fmtCtx, sortModels } from "./models.ts";
-
-describe("fmtCtx", () => {
-	it("formats 0 as 0", () => expect(fmtCtx(0)).toBe("0"));
-	it("formats small numbers as-is", () => expect(fmtCtx(512)).toBe("512"));
-	it("formats thousands as Nk", () => {
-		expect(fmtCtx(128_000)).toBe("128k");
-		expect(fmtCtx(8_192)).toBe("8k");
-	});
-	it("formats millions as NM", () => {
-		expect(fmtCtx(1_000_000)).toBe("1M");
-		expect(fmtCtx(2_000_000)).toBe("2M");
-		expect(fmtCtx(1_500_000)).toBe("1.5M");
-	});
-});
-
-describe("fmtCost", () => {
-	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");
-	});
-	it("formats input/output costs", () => {
-		expect(fmtCost({ cost: { input: 3, output: 15 } })).toBe("3.00/15.00");
-	});
-	it("handles missing input/output as 0", () => {
-		expect(fmtCost({ cost: {} })).toBe("free");
-	});
-});
-
-describe("benchStars", () => {
-	it("gives 5 stars for score >= 90", () => {
-		expect(benchStars(95).filled).toBe(5);
-		expect(benchStars(90).filled).toBe(5);
-	});
-	it("gives 4 stars for 80-89", () => {
-		expect(benchStars(85).filled).toBe(4);
-		expect(benchStars(80).filled).toBe(4);
-	});
-	it("gives 3 stars for 70-79", () => {
-		expect(benchStars(75).filled).toBe(3);
-	});
-	it("gives 2 stars for 50-69", () => {
-		expect(benchStars(60).filled).toBe(2);
-		expect(benchStars(50).filled).toBe(2);
-	});
-	it("gives 1 star for score < 50", () => {
-		expect(benchStars(30).filled).toBe(1);
-		expect(benchStars(0).filled).toBe(1);
-	});
-	it("gives 1 star for null/undefined", () => {
-		expect(benchStars(null).filled).toBe(1);
-		expect(benchStars(undefined).filled).toBe(1);
-	});
-	it("filled + empty always = 5", () => {
-		for (const s of [0, 50, 70, 80, 90, 100]) {
-			const { filled, empty } = benchStars(s);
-			expect(filled + empty).toBe(5);
-		}
-	});
-});
-
-describe("sortModels", () => {
-	const models = [
-		{ provider: "a", id: "m1", name: "Zebra", score: 80 },
-		{ provider: "a", id: "m2", name: "Alpha", score: 95 },
-		{ provider: "a", id: "m3", name: "Middle", score: null },
-		{ provider: "a", id: "m4", name: "Beta", score: 80 },
-	];
-
-	it("sorts by score descending", () => {
-		const sorted = sortModels(models);
-		expect(sorted[0].name).toBe("Alpha"); // score 95
-	});
-
-	it("breaks score ties alphabetically by name", () => {
-		const sorted = sortModels(models);
-		const tiedIdx = sorted.findIndex((m) => m.name === "Beta");
-		const zebraIdx = sorted.findIndex((m) => m.name === "Zebra");
-		expect(tiedIdx).toBeLessThan(zebraIdx); // Beta before Zebra, both score 80
-	});
-
-	it("puts null score models last", () => {
-		const sorted = sortModels(models);
-		expect(sorted[sorted.length - 1].name).toBe("Middle");
-	});
-
-	it("does not mutate the original array", () => {
-		const original = [...models];
-		sortModels(models);
-		expect(models).toEqual(original);
-	});
-});

package/src/commands/models/models.ts

@@ -1,367 +0,0 @@
-/**
- * models.ts — enhanced /models command with benchlm rank + score
- *
- * Replaces (or supplements) the built-in /model selector by registering
- * /models (plural). Each row shows:
- *   <name>  <provider> · <ctx> · <cost> · 🏅 #rank score
- *
- * Sorted by benchlm rank when available (best first), then alphabetical.
- */
-
-import type {
-	ExtensionAPI,
-	ExtensionContext,
-} from "@earendil-works/pi-coding-agent";
-import { DynamicBorder } from "@earendil-works/pi-coding-agent";
-import {
-	Container,
-	fuzzyFilter,
-	Input,
-	matchesKey,
-	type SelectItem,
-	SelectList,
-	Text,
-	visibleWidth,
-} from "@earendil-works/pi-tui";
-import { lookupBenchmark, lookupModelsDev } from "../../lib/data";
-import { patchOutBuiltinModelCommand } from "./patch-builtin";
-
-// ─── Pure logic (exported for tests) ─────────────────────────────────────────
-
-export function fmtCtx(n: number): string {
-	if (!n || n < 1_000) return `${n}`;
-	if (n >= 1_000_000) {
-		const m = n / 1_000_000;
-		return `${Number.isInteger(m) ? m.toFixed(0) : m.toFixed(1).replace(/\.0$/, "")}M`;
-	}
-	return `${Math.round(n / 1_000)}k`;
-}
-
-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;
-	if (i === 0 && o === 0) return "free";
-	return `${i.toFixed(2)}/${o.toFixed(2)}`;
-}
-
-export function benchStars(score: number | null | undefined): {
-	filled: number;
-	empty: number;
-} {
-	const total = 5;
-	let filled = 1;
-	if (typeof score === "number") {
-		if (score >= 90) filled = 5;
-		else if (score >= 80) filled = 4;
-		else if (score >= 70) filled = 3;
-		else if (score >= 50) filled = 2;
-	}
-	return { filled, empty: total - filled };
-}
-
-export type SortableModel = {
-	provider: string;
-	id: string;
-	name?: string;
-	score?: number | null;
-};
-
-export function sortModels<T extends SortableModel>(models: T[]): T[] {
-	return [...models].sort((a, b) => {
-		const sa = a.score ?? -1;
-		const sb = b.score ?? -1;
-		if (sa !== sb) return sb - sa;
-		return (a.name ?? a.id).localeCompare(b.name ?? b.id);
-	});
-}
-
-export 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
-	// extension load order vs oauth/auth resolution, can omit oauth providers
-	// whose models were registered as built-ins but resolved after the last
-	// load (notably `openai-codex`). 9router survives because it's registered
-	// as a custom provider with an env-key apiKey. refresh() rebuilds the model
-	// list (resetOAuthProviders → loadModels → re-apply registered providers)
-	// so oauth-backed codex models reappear, exactly as the built-in does.
-	//
-	// The public ExtensionContext type narrows modelRegistry to a sync
-	// getAvailable() only; at runtime ctx.modelRegistry is the full
-	// 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>;
-	const registry = ctx.modelRegistry as unknown as {
-		refresh?: () => void;
-		getAvailable(): AvailableModels | Promise<AvailableModels>;
-	};
-	registry.refresh?.();
-	const available = await registry.getAvailable();
-	if (available.length === 0) {
-		ctx.ui.notify("No models with configured auth.", "warning");
-		return;
-	}
-
-	const current = ctx.model;
-
-	// Build items with benchmark data
-	type Row = {
-		m: (typeof available)[number];
-		dev: ReturnType<typeof lookupModelsDev>;
-		bench: ReturnType<typeof lookupBenchmark>;
-	};
-	const rows: Row[] = available.map((m) => ({
-		m,
-		dev: lookupModelsDev(m.provider, m.id),
-		bench: lookupBenchmark(m.name ?? m.id),
-	}));
-
-	// Sort: by score desc (highest first), unscored last alphabetical
-	rows.sort((a, b) => {
-		const sa = a.bench?.overallScore ?? -1;
-		const sb = b.bench?.overallScore ?? -1;
-		if (sa !== sb) return sb - sa;
-		return (a.m.name ?? a.m.id).localeCompare(b.m.name ?? b.m.id);
-	});
-
-	// Show all models (no deduplication)
-	const dedupedRows = rows;
-
-	// items built inside the custom() factory so we have theme access for colors
-
-	const result = await ctx.ui.custom<string | null>(
-		(_tui, theme, _kb, done) => {
-			const container = new Container();
-			const accent = "accent";
-
-			// Find max rank width across all benchmarked rows for # padding
-			const maxRankWidth = Math.max(
-				...dedupedRows.map((r) => (r.bench ? String(r.bench.rank).length : 0)),
-				1,
-			);
-
-			// Mute low-info parts (separators, padding, #, ☆) so the actual values pop.
-			const mute = (s: string) => theme.fg("muted", s);
-			const sep = mute(" · ");
-
-			// Track rank per item value so fuzzy results can prioritize ranked models.
-			const rankByValue = new Map<string, number>();
-			for (const { m, bench } of dedupedRows) {
-				if (bench) rankByValue.set(`${m.provider}/${m.id}`, bench.rank);
-			}
-
-			const items: SelectItem[] = dedupedRows.map(({ m, dev, bench }) => {
-				const isCurrent =
-					current && m.provider === current.provider && m.id === current.id;
-
-				// Label: marker + muted '#' + bright rank + accent-colored model name
-				const marker = isCurrent ? theme.fg(accent, "▶") : " ";
-				let rankPrefix: string;
-				if (bench) {
-					const rankStr = String(bench.rank).padEnd(maxRankWidth);
-					rankPrefix = mute("#") + theme.fg("warning", rankStr);
-				} else {
-					rankPrefix = " ".repeat(maxRankWidth + 1);
-				}
-				// 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;
-					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("warning", "★".repeat(filled)) +
-						mute("☆".repeat(5 - filled));
-					benchSeg = `⚡${theme.fg("warning", String(score))} ${starBar}`;
-				}
-				const desc = [ctxStr, costSeg, benchSeg].filter(Boolean).join(sep);
-
-				return {
-					value: `${m.provider}/${m.id}`,
-					label,
-					description: desc,
-				};
-			});
-
-			const currentIdx = current
-				? items.findIndex(
-						(it) => it.value === `${current.provider}/${current.id}`,
-					)
-				: 0;
-
-			container.addChild(new DynamicBorder((s) => theme.fg(accent, s)));
-			container.addChild(
-				new Text(theme.fg(accent, theme.bold("󰚩  Select model"))),
-			);
-			container.addChild(
-				new Text(
-					theme.fg(
-						"dim",
-						"context & pricing from models.dev · ranks from benchlm.ai",
-					),
-				),
-			);
-
-			// 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 search = new Input();
-			const list = new SelectList(
-				items,
-				Math.min(items.length, 14),
-				{
-					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),
-					noMatch: (t) => theme.fg("warning", t),
-				},
-				{
-					minPrimaryColumnWidth: widestLabel + 2,
-					maxPrimaryColumnWidth: widestLabel + 2,
-				},
-			);
-			if (currentIdx >= 0) list.setSelectedIndex(currentIdx);
-
-			list.onSelect = (item) => done(item.value);
-			list.onCancel = () => done(null);
-			search.onEscape = () => done(null);
-
-			const applyFuzzy = (query: string) => {
-				const internal = list as unknown as {
-					items: SelectItem[];
-					filteredItems: SelectItem[];
-					selectedIndex: number;
-					invalidate(): void;
-				};
-				const q = query.trim();
-				let next: SelectItem[];
-				if (q.length === 0) {
-					next = internal.items;
-				} 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,
-					);
-				} else {
-					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 }))
-						.sort((a, b) => {
-							const ra = rankByValue.get(a.it.value) ?? Infinity;
-							const rb = rankByValue.get(b.it.value) ?? Infinity;
-							if (ra !== rb) return ra - rb;
-							return a.i - b.i;
-						})
-						.map(({ it }) => it);
-				}
-				internal.filteredItems = next;
-				internal.selectedIndex = 0;
-				internal.invalidate();
-			};
-
-			container.addChild(new Text(theme.fg("muted", "Search:")));
-			container.addChild(search);
-			container.addChild(list);
-			container.addChild(
-				new Text(
-					theme.fg(
-						"dim",
-						"fuzzy search · ↑↓ navigate · enter select · esc cancel",
-					),
-				),
-			);
-			container.addChild(new DynamicBorder((s) => theme.fg(accent, s)));
-
-			return {
-				render(w: number) {
-					return container.render(w);
-				},
-				invalidate() {
-					container.invalidate();
-				},
-				handleInput(data: string) {
-					// Detect keys via pi-tui's own parser — the same recognition
-					// SelectList uses. Arrows arrive as named keys ("up"/"down"),
-					// not raw escape sequences, so string-equality checks fail.
-					const isNav = matchesKey(data, "up") || matchesKey(data, "down");
-					if (isNav || matchesKey(data, "enter")) {
-						list.handleInput?.(data);
-					} else if (matchesKey(data, "escape")) {
-						done(null);
-					} else {
-						search.handleInput?.(data);
-						applyFuzzy(search.getValue?.() ?? "");
-					}
-					container.invalidate();
-				},
-			};
-		},
-	);
-
-	if (!result) return;
-
-	// Apply selection
-	const [provider, ...rest] = result.split("/");
-	const id = rest.join("/");
-	const picked = available.find((m) => m.provider === provider && m.id === id);
-	if (!picked) {
-		ctx.ui.notify(`Model not found: ${result}`, "error");
-		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");
-}
-
-export default function modelPickerExtension(pi: ExtensionAPI) {
-	// Remove Pi's built-in /model so only the enhanced /models picker remains.
-	// Self-healing: re-applies on every load, so a Pi upgrade can't restore it.
-	patchOutBuiltinModelCommand();
-
-	const handler = async (_args: unknown, ctx: ExtensionContext) => {
-		await showEnhancedPicker(pi, ctx);
-	};
-	pi.registerCommand("models", {
-		description: "Enhanced model picker — shows benchlm rank + score",
-		handler,
-	});
-}