@xynogen/pix-core 0.2.3 → 0.3.0

package/src/commands/models/patch-builtin.test.ts

@@ -1,66 +0,0 @@
-import { describe, expect, it } from "bun:test";
-import { mkdtempSync, readFileSync, writeFileSync } from "node:fs";
-import { tmpdir } from "node:os";
-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)" },';
-
-function escapeRegExp(text: string): string {
-	return text.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
-}
-
-function patchSource(source: string): string {
-	if (!source.includes(MODEL_COMMAND_LINE)) return source;
-	return source.replace(
-		new RegExp(`[ \\t]*${escapeRegExp(MODEL_COMMAND_LINE)}\\n?`),
-		"",
-	);
-}
-
-const UNPATCHED = `export const BUILTIN_SLASH_COMMANDS = [
-    { name: "settings", description: "Open settings menu" },
-    { name: "model", description: "Select model (opens selector UI)" },
-    { name: "login", description: "Configure provider authentication" },
-];
-`;
-
-describe("patch-builtin /model removal", () => {
-	it("removes the built-in /model line and keeps neighbors", () => {
-		const out = patchSource(UNPATCHED);
-		expect(out).not.toContain('name: "model"');
-		expect(out).toContain('name: "settings"');
-		expect(out).toContain('name: "login"');
-	});
-
-	it("is idempotent — second pass is a no-op", () => {
-		const once = patchSource(UNPATCHED);
-		const twice = patchSource(once);
-		expect(twice).toBe(once);
-	});
-
-	it("leaves an already-clean file untouched", () => {
-		const clean = `export const X = [\n    { name: "login" },\n];\n`;
-		expect(patchSource(clean)).toBe(clean);
-	});
-
-	it("does not strip the plural /models entry", () => {
-		const withPlural = `[
-    { name: "models", description: "Enhanced picker" },
-    { name: "model", description: "Select model (opens selector UI)" },
-]`;
-		const out = patchSource(withPlural);
-		expect(out).toContain('name: "models"');
-		expect(out).not.toContain('{ name: "model", description');
-	});
-
-	it("round-trips through disk", () => {
-		const dir = mkdtempSync(join(tmpdir(), "pix-patch-"));
-		const file = join(dir, "slash-commands.js");
-		writeFileSync(file, UNPATCHED, "utf8");
-		writeFileSync(file, patchSource(readFileSync(file, "utf8")), "utf8");
-		expect(readFileSync(file, "utf8")).not.toContain('name: "model"');
-	});
-});

package/src/commands/models/patch-builtin.ts

@@ -1,120 +0,0 @@
-/**
- * patch-builtin.ts — strip Pi's built-in /model slash command at load time.
- *
- * Built-in commands can't be removed via the extension API, so we edit Pi's
- * compiled slash-commands.js directly. Done on every load: idempotent and
- * self-healing across Pi upgrades, so no manual repatch is ever needed.
- *
- * Resolution strategy (in order):
- *   1. Locate the `pi` binary via PATH → infer package root from its realpath.
- *      The binary is always at <pkg>/dist/cli.js so ../../ is the package root.
- *   2. Probe well-known global install locations (bun, npm).
- *   3. Fall back to createRequire against the extension's own node_modules
- *      (works when pi and the extension share the same install tree).
- */
-
-import { execSync } from "node:child_process";
-import { existsSync, readFileSync, writeFileSync } from "node:fs";
-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)" },';
-
-/** Candidate slash-commands.js paths, most-specific first. */
-function candidatePaths(): string[] {
-	const paths: string[] = [];
-
-	// 1. Resolve via the running `pi` binary → its realpath gives the dist dir.
-	try {
-		const piReal = execSync("realpath $(which pi)", {
-			encoding: "utf8",
-			stdio: ["pipe", "pipe", "pipe"],
-		}).trim();
-		if (piReal) {
-			// piReal = /.../pi-coding-agent/dist/cli.js → dist/ → ../dist/core/
-			const distCore = resolve(dirname(piReal), "core");
-			paths.push(join(distCore, "slash-commands.js"));
-		}
-	} catch {
-		// `pi` not on PATH or `which`/`realpath` unavailable — skip
-	}
-
-	// 2. Well-known global install locations.
-	const home = homedir();
-	const globalRoots = [
-		join(home, ".bun", "install", "global", "node_modules"),
-		join(home, ".npm-global", "lib", "node_modules"),
-		"/usr/local/lib/node_modules",
-		"/usr/lib/node_modules",
-	];
-	for (const root of globalRoots) {
-		paths.push(
-			join(
-				root,
-				"@earendil-works",
-				"pi-coding-agent",
-				"dist",
-				"core",
-				"slash-commands.js",
-			),
-		);
-	}
-
-	// 3. Fallback: createRequire from this file (works when extension is co-installed).
-	try {
-		const require = createRequire(import.meta.url);
-		const entry = require.resolve("@earendil-works/pi-coding-agent");
-		paths.push(resolve(dirname(entry), "core", "slash-commands.js"));
-	} catch {
-		// local resolution failed — skip
-	}
-
-	return paths;
-}
-
-/** Locate the host's compiled slash-commands.js, or null if not found. */
-function findSlashCommandsFile(): string | null {
-	for (const p of candidatePaths()) {
-		if (existsSync(p)) return p;
-	}
-	return null;
-}
-
-/**
- * Remove the built-in /model command line from Pi's slash-commands.js.
- * Idempotent: returns silently if the file is missing or already patched.
- */
-export function patchOutBuiltinModelCommand(): void {
-	const file = findSlashCommandsFile();
-	if (!file) return;
-
-	let source: string;
-	try {
-		source = readFileSync(file, "utf8");
-	} catch {
-		return;
-	}
-
-	if (!source.includes(MODEL_COMMAND_LINE)) return; // already patched
-
-	const patched = source.replace(
-		new RegExp(`[ \\t]*${escapeRegExp(MODEL_COMMAND_LINE)}\\n?`),
-		"",
-	);
-	if (patched === source) return;
-
-	try {
-		writeFileSync(file, patched, "utf8");
-	} catch {
-		// Read-only install — leave /model in place rather than crash.
-	}
-}
-
-function escapeRegExp(text: string): string {
-	return text.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
-}
-
-// Export for tests
-export { candidatePaths, findSlashCommandsFile };

package/src/commands/tools.test.ts

@@ -1,15 +0,0 @@
-/**
- * Smoke tests for the command extensions merged from pix-tools.
- * Each default export must be a registrable (pi) => void function.
- */
-
-import { describe, expect, it } from "bun:test";
-
-describe("merged pix-tools commands", () => {
-	for (const name of ["diff"]) {
-		it(`${name} exports a register function`, async () => {
-			const mod = await import(`./${name}/${name}.ts`);
-			expect(mod.default).toBeFunction();
-		});
-	}
-});

package/src/commands/update/update.test.ts

@@ -1,112 +0,0 @@
-import { describe, expect, it } from "bun:test";
-import {
-	commandFor,
-	formatUpdateSummary,
-	type InstallMethod,
-	isTransient,
-	PACKAGE_NAME,
-} from "./update.ts";
-
-describe("isTransient", () => {
-	it("matches network errors", () => {
-		expect(isTransient("ETIMEDOUT")).toBe(true);
-		expect(isTransient("ECONNRESET")).toBe(true);
-		expect(isTransient("ECONNREFUSED")).toBe(true);
-		expect(isTransient("socket hang up")).toBe(true);
-		expect(isTransient("network error occurred")).toBe(true);
-	});
-
-	it("matches HTTP status codes", () => {
-		expect(isTransient("Error 429: Too many requests")).toBe(true);
-		expect(isTransient("502 Bad Gateway")).toBe(true);
-		expect(isTransient("503 Service Unavailable")).toBe(true);
-		expect(isTransient("504 Gateway Timeout")).toBe(true);
-	});
-
-	it("matches timeout/temporary", () => {
-		expect(isTransient("Request timeout after 30s")).toBe(true);
-		expect(isTransient("temporary failure")).toBe(true);
-		expect(isTransient("EAI_AGAIN")).toBe(true);
-	});
-
-	it("returns false for permanent errors", () => {
-		expect(isTransient("permission denied")).toBe(false);
-		expect(isTransient("command not found")).toBe(false);
-		expect(isTransient("syntax error")).toBe(false);
-		expect(isTransient("")).toBe(false);
-	});
-
-	it("is case-insensitive", () => {
-		expect(isTransient("NETWORK FAILURE")).toBe(true);
-		expect(isTransient("Timeout after 30s")).toBe(true);
-	});
-});
-
-describe("commandFor", () => {
-	it("returns correct command for each method", () => {
-		const methods: InstallMethod[] = ["vp", "bun", "npm", "brew"];
-		for (const m of methods) {
-			const spec = commandFor(m);
-			expect(spec).toBeDefined();
-			expect(spec?.command).toBeTruthy();
-			expect(spec?.label).toBeTruthy();
-		}
-	});
-
-	it("vp uses vp add -g", () => {
-		const spec = commandFor("vp")!;
-		expect(spec.command).toBe("vp");
-		expect(spec.args).toContain(`${PACKAGE_NAME}@latest`);
-	});
-
-	it("bun uses bun add -g", () => {
-		const spec = commandFor("bun")!;
-		expect(spec.command).toBe("bun");
-		expect(spec.args).toContain(`${PACKAGE_NAME}@latest`);
-	});
-
-	it("npm uses npm install -g", () => {
-		const spec = commandFor("npm")!;
-		expect(spec.command).toBe("npm");
-		expect(spec.args).toContain("-g");
-		expect(spec.args).toContain(`${PACKAGE_NAME}@latest`);
-	});
-
-	it("brew uses sh -lc", () => {
-		const spec = commandFor("brew")!;
-		expect(spec.command).toBe("/bin/sh");
-		expect(spec.label).toContain("brew upgrade");
-	});
-
-	it("native returns undefined", () => {
-		expect(commandFor("native")).toBeUndefined();
-	});
-});
-
-describe("formatUpdateSummary", () => {
-	it("shows updated message when version changed", () => {
-		const msg = formatUpdateSummary("0.75.0", "0.76.0", 1);
-		expect(msg).toContain("0.75.0 → 0.76.0");
-	});
-
-	it("shows up-to-date when version unchanged", () => {
-		const msg = formatUpdateSummary("0.76.0", "0.76.0", 1);
-		expect(msg).toContain("up to date");
-		expect(msg).toContain("0.76.0");
-	});
-
-	it("shows retry count when attempts > 1", () => {
-		const msg = formatUpdateSummary("0.75.0", "0.76.0", 3);
-		expect(msg).toContain("Retried 2 transient failure");
-	});
-
-	it("no retry mention when attempts = 1", () => {
-		const msg = formatUpdateSummary("0.75.0", "0.76.0", 1);
-		expect(msg).not.toContain("Retried");
-	});
-
-	it("handles unknown versions gracefully", () => {
-		const msg = formatUpdateSummary("unknown", "unknown", 1);
-		expect(msg).toContain("up to date");
-	});
-});

package/src/commands/update/update.ts

@@ -1,271 +0,0 @@
-import type {
-	ExtensionAPI,
-	ExtensionCommandContext,
-} from "@earendil-works/pi-coding-agent";
-// ─── Pure logic (exported for tests) ─────────────────────────────────────────
-
-export const PACKAGE_NAME = "@earendil-works/pi-coding-agent";
-
-export const TRANSIENT_PATTERNS = [
-	/eai_again/i,
-	/etimedout/i,
-	/econnreset/i,
-	/econnrefused/i,
-	/socket hang up/i,
-	/network/i,
-	/timeout/i,
-	/temporar/i,
-	/too many requests/i,
-	/\b429\b/,
-	/\b502\b/,
-	/\b503\b/,
-	/\b504\b/,
-];
-
-export type InstallMethod = "vp" | "bun" | "npm" | "brew" | "native";
-
-export type CommandSpec = {
-	command: string;
-	args: string[];
-	label: string;
-};
-
-export function isTransient(output: string): boolean {
-	return TRANSIENT_PATTERNS.some((pattern) => pattern.test(output));
-}
-
-export function commandFor(method: InstallMethod): CommandSpec | undefined {
-	switch (method) {
-		case "vp":
-			return {
-				command: "vp",
-				args: ["add", "-g", `${PACKAGE_NAME}@latest`],
-				label: `vp add -g ${PACKAGE_NAME}@latest`,
-			};
-		case "bun":
-			return {
-				command: "bun",
-				args: ["add", "-g", `${PACKAGE_NAME}@latest`],
-				label: `bun add -g ${PACKAGE_NAME}@latest`,
-			};
-		case "npm":
-			return {
-				command: "npm",
-				args: ["install", "-g", `${PACKAGE_NAME}@latest`],
-				label: `npm install -g ${PACKAGE_NAME}@latest`,
-			};
-		case "brew":
-			return {
-				command: "/bin/sh",
-				args: ["-lc", "brew upgrade pi-coding-agent || brew upgrade pi"],
-				label: "brew upgrade pi-coding-agent || brew upgrade pi",
-			};
-		case "native":
-			return undefined;
-	}
-}
-
-export function formatUpdateSummary(
-	before: string,
-	after: string,
-	attempts: number,
-): string {
-	const changed =
-		before !== after && before !== "unknown" && after !== "unknown";
-	const summary = changed
-		? `Pi updated: ${before} → ${after}`
-		: `Pi is up to date (${after}).`;
-	return attempts > 1
-		? `${summary} Retried ${attempts - 1} transient failure(s).`
-		: summary;
-}
-
-async function resolveCommand(command: string, pi: ExtensionAPI) {
-	const result = await pi.exec(
-		"/bin/sh",
-		["-lc", `command -v ${command} || true`],
-		{ timeout: 10_000 },
-	);
-	return result.stdout.trim().split("\n")[0] || undefined;
-}
-
-async function currentVersion(pi: ExtensionAPI) {
-	const result = await pi.exec("pi", ["--version"], { timeout: 10_000 });
-	return result.stdout.trim() || result.stderr.trim() || "unknown";
-}
-
-async function detectInstallMethod(pi: ExtensionAPI): Promise<InstallMethod> {
-	const piPath = await resolveCommand("pi", pi);
-	const realPiPath = piPath
-		? (
-				await pi.exec(
-					"/bin/sh",
-					["-lc", `realpath ${piPath} 2>/dev/null || printf %s ${piPath}`],
-					{ timeout: 10_000 },
-				)
-			).stdout.trim()
-		: undefined;
-
-	if (piPath?.includes("/.vite-plus/") || realPiPath?.includes("/.vite-plus/"))
-		return "vp";
-	if (piPath?.includes("/.bun/") || realPiPath?.includes("/.bun/"))
-		return "bun";
-	if (
-		piPath?.includes("/Homebrew/") ||
-		piPath?.includes("/homebrew/") ||
-		realPiPath?.includes("/Homebrew/") ||
-		realPiPath?.includes("/homebrew/")
-	)
-		return "brew";
-
-	if (piPath) {
-		const hasGlobalNpm = await pi.exec(
-			"/bin/sh",
-			[
-				"-lc",
-				`p=${piPath}; i=0; while [ $i -lt 5 ]; do d=$(dirname "$p"); [ -d "$d/node_modules/${PACKAGE_NAME}" ] && exit 0; p=$d; i=$((i+1)); done; exit 1`,
-			],
-			{ timeout: 10_000 },
-		);
-		if ((hasGlobalNpm.code ?? 1) === 0) return "npm";
-	}
-
-	if (await resolveCommand("vp", pi)) return "vp";
-	if (await resolveCommand("bun", pi)) return "bun";
-	if (await resolveCommand("npm", pi)) return "npm";
-	if (await resolveCommand("brew", pi)) return "brew";
-	return "native";
-}
-
-async function runWithRetry(pi: ExtensionAPI, spec: CommandSpec) {
-	let lastOutput = "";
-	for (let attempt = 1; attempt <= 3; attempt++) {
-		const result = await pi.exec(spec.command, spec.args, { timeout: 180_000 });
-		lastOutput = [result.stdout, result.stderr]
-			.filter(Boolean)
-			.join("\n")
-			.trim();
-		if ((result.code ?? 0) === 0)
-			return { ok: true, output: lastOutput, attempts: attempt };
-		if (attempt === 3 || !isTransient(lastOutput))
-			return { ok: false, output: lastOutput, attempts: attempt };
-		await new Promise((resolve) => setTimeout(resolve, attempt * 1500));
-	}
-	return { ok: false, output: lastOutput, attempts: 3 };
-}
-
-async function updatePi(pi: ExtensionAPI, ctx: ExtensionCommandContext) {
-	await (
-		ctx as ExtensionCommandContext & { waitForIdle?: () => Promise<void> }
-	).waitForIdle?.();
-
-	const before = await currentVersion(pi).catch(() => "unknown");
-	const method = await detectInstallMethod(pi);
-	const spec = commandFor(method);
-
-	if (!spec) {
-		ctx.ui.notify(
-			`Pi ${before}; install method appears native. Please update the native binary manually.`,
-			"warning",
-		);
-		return;
-	}
-
-	ctx.ui.notify(`Updating Pi via ${method}: ${spec.label}`, "info");
-	const result = await runWithRetry(pi, spec);
-	const after = await currentVersion(pi).catch(() => "unknown");
-
-	if (!result.ok) {
-		ctx.ui.notify(
-			`Pi update failed after ${result.attempts} attempt(s). ${result.output || "No output."}`,
-			"error",
-		);
-		return;
-	}
-
-	ctx.ui.notify(formatUpdateSummary(before, after, result.attempts), "info");
-}
-
-async function updateExtensions(
-	pi: ExtensionAPI,
-	ctx: ExtensionCommandContext,
-) {
-	ctx.ui.notify("Updating Pi extensions from dotfiles setup", "info");
-	const result = await pi.exec(
-		"/bin/sh",
-		["-lc", '"$HOME/dotfiles/ai_config/pi/setup.sh"'],
-		{ timeout: 240_000 },
-	);
-	const output = [result.stdout, result.stderr]
-		.filter(Boolean)
-		.join("\n")
-		.trim();
-	if ((result.code ?? 0) !== 0) {
-		ctx.ui.notify(
-			`Pi extensions update failed. ${output || "No output."}`,
-			"error",
-		);
-		return;
-	}
-	ctx.ui.notify(
-		"Pi extensions updated. Please run /reload to apply changes.",
-		"warning",
-	);
-}
-
-async function updatePackages(pi: ExtensionAPI, ctx: ExtensionCommandContext) {
-	ctx.ui.notify("Updating pi packages (pi update --extensions)", "info");
-	const result = await pi.exec("pi", ["update", "--extensions"], {
-		timeout: 240_000,
-	});
-	const output = [result.stdout, result.stderr]
-		.filter(Boolean)
-		.join("\n")
-		.trim();
-	if ((result.code ?? 0) !== 0) {
-		ctx.ui.notify(
-			`Pi package update failed. ${output || "No output."}`,
-			"error",
-		);
-		return;
-	}
-	ctx.ui.notify(
-		"Pi packages updated. Please run /reload to apply changes.",
-		"warning",
-	);
-}
-
-async function updateAll(pi: ExtensionAPI, ctx: ExtensionCommandContext) {
-	await updatePi(pi, ctx);
-	await updateExtensions(pi, ctx);
-	await updatePackages(pi, ctx);
-}
-
-export default function (pi: ExtensionAPI) {
-	(
-		pi as ExtensionAPI & {
-			registerFlag: (name: string, opts: unknown) => void;
-		}
-	).registerFlag("update", {
-		description: "Update Pi, dotfiles extensions, and pi packages",
-		type: "boolean",
-		default: false,
-	});
-
-	pi.registerCommand("update", {
-		description: "Update Pi, dotfiles extensions, and pi packages",
-		handler: async (_args, ctx) => {
-			await updateAll(pi, ctx);
-		},
-	});
-
-	pi.on("session_start", async (_event, ctx) => {
-		const flags = pi as ExtensionAPI & {
-			getFlag?: (name: string) => boolean;
-			sendUserMessage?: (message: string, opts?: unknown) => void;
-		};
-		if (!flags.getFlag?.("update")) return;
-		flags.sendUserMessage?.("/update", { deliverAs: "followUp" });
-		ctx.ui.notify("Queued /update from --update", "info");
-	});
-}

package/src/index.ts

@@ -1,45 +0,0 @@
-/**
- * pix-core — Pi extension bundle
- *
- * Layout (grouped by concern):
- *   - ui/       — welcome (π banner + health checks), footer (status bar)
- *   - commands/ — models (/models picker), update (/update self-update),
- *                 diff (/diff)
- *   - tool/     — todo (durable execution checklist),
- *                 toolbox (/toolbox command — user toggles tools on/off),
- *                 lazy (lazy tool exposure — gates schemas out of the prompt)
- *   - nudge/    — model-steering reminders (tools / capability+skills)
- *   - lib/      — shared data layer (models.dev + BenchLM)
- *
- * Depends on pix-data (github.com/xynogen/pix-data) for shared
- * models.dev + BenchLM cache at ~/.cache/pi/.
- */
-
-import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
-import registerAgentSop from "./commands/agent-sop/agent-sop.ts";
-import registerClear from "./commands/clear/clear.ts";
-import registerDiff from "./commands/diff/diff.ts";
-import registerModels from "./commands/models/models.ts";
-import registerUpdate from "./commands/update/update.ts";
-import registerNudges from "./nudge/index.ts";
-import registerAsk from "./tool/ask/index.ts";
-import registerTodo from "./tool/todo/todo.ts";
-import registerToolbox from "./tool/toolbox/toolbox.ts";
-import registerDiagnostics from "./ui/diagnostics.ts";
-import registerFooter from "./ui/footer.ts";
-import registerWelcome from "./ui/welcome.ts";
-
-export default function (pi: ExtensionAPI): void {
-	registerAgentSop(pi);
-	registerWelcome(pi);
-	registerFooter(pi);
-	registerDiagnostics(pi);
-	registerModels(pi);
-	registerUpdate(pi);
-	registerDiff(pi);
-	registerClear(pi);
-	registerTodo(pi);
-	registerAsk(pi);
-	registerToolbox(pi);
-	registerNudges(pi);
-}

package/src/lib/data.ts

@@ -1,33 +0,0 @@
-/**
- * data.ts — model data layer (shim)
- *
- * Thin re-export of the shared data layer from @xynogen/pix-data
- * (github.com/xynogen/pix-mono/tree/main/packages/pix-data). Cache lives at
- * ~/.cache/pi/ and is shared across all Pi extensions — pix-data warms it on
- * session start; this extension reads from it.
- *
- * Consumers in this extension dir:
- *   footer.ts  — lookupModelsDev, lookupBenchmark, ModelsDevModel
- *   models.ts  — lookupModelsDev, lookupBenchmark
- */
-
-export type {
-	BenchmarkEntry,
-	ModelsDevApi,
-	ModelsDevModel,
-} from "../../../pix-data/src/index.ts";
-export {
-	benchmark,
-	buildModelsDevIndex,
-	CACHE_DIR,
-	DataSource,
-	fetchModelsDevIndex,
-	lookupBenchmark,
-	lookupInIndex,
-	lookupModelsDev,
-	modelsDev,
-} from "../../../pix-data/src/index.ts";
-
-export default function (_pi: unknown): void {
-	// pix-data warms this cache on startup — nothing to do here.
-}