@xynogen/pix-core 0.1.0 → 0.1.1

package/package.json

@@ -1,42 +1,51 @@
 {
-  "name": "@xynogen/pix-core",
-  "version": "0.1.0",
-  "description": "Pi extension — core UI/UX bundle (welcome banner, footer, model picker, self-update)",
-  "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", "pi-extension", "pix"],
-  "author": "xynogen",
-  "license": "MIT",
-  "repository": {
-    "type": "git",
-    "url": "git+https://github.com/xynogen/pix-mono.git",
-    "directory": "packages/pix-core"
-  },
-  "homepage": "https://github.com/xynogen/pix-mono/tree/main/packages/pix-core#readme",
-  "bugs": {
-    "url": "https://github.com/xynogen/pix-mono/issues"
-  },
-  "publishConfig": {
-    "access": "public"
-  },
-  "dependencies": {
-    "typebox": "^1.1.38"
-  },
-  "peerDependencies": {
-    "@earendil-works/pi-coding-agent": "*",
-    "@earendil-works/pi-tui": "*"
-  }
+	"name": "@xynogen/pix-core",
+	"version": "0.1.1",
+	"description": "Pi extension — core UI/UX bundle (welcome banner, footer, model picker, self-update)",
+	"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",
+		"pi-extension",
+		"pix"
+	],
+	"author": "xynogen",
+	"license": "MIT",
+	"repository": {
+		"type": "git",
+		"url": "git+https://github.com/xynogen/pix-mono.git",
+		"directory": "packages/pix-core"
+	},
+	"homepage": "https://github.com/xynogen/pix-mono/tree/main/packages/pix-core#readme",
+	"bugs": {
+		"url": "https://github.com/xynogen/pix-mono/issues"
+	},
+	"publishConfig": {
+		"access": "public"
+	},
+	"dependencies": {
+		"typebox": "^1.1.38"
+	},
+	"peerDependencies": {
+		"@earendil-works/pi-coding-agent": "*",
+		"@earendil-works/pi-tui": "*"
+	}
 }

package/src/commands/clear/clear.ts

@@ -12,7 +12,7 @@ async function clearCache(pi: ExtensionAPI, ctx: ExtensionCommandContext) {
 		.filter(Boolean)
 		.join("\n")
 		.trim();
-	if ((result.exitCode ?? 0) !== 0) {
+	if ((result.code ?? 0) !== 0) {
 		ctx.ui.notify(`Cache clear failed. ${output || "No output."}`, "error");
 		return;
 	}

package/src/commands/copy-all/copy-all.ts

@@ -16,7 +16,12 @@ function textFromContent(content: unknown): string {
 	return content
 		.map((block) => {
 			if (!block || typeof block !== "object" || !("type" in block)) return "";
-			if (block.type === "text" && "text" in block && typeof block.text === "string") return block.text;
+			if (
+				block.type === "text" &&
+				"text" in block &&
+				typeof block.text === "string"
+			)
+				return block.text;
 			if (block.type === "image") return "[image]";
 			return "";
 		})
@@ -43,11 +48,14 @@ function copyToClipboard(text: string): Promise<void> {
 		}
 		const child = spawn(c.cmd, c.args);
 		let stderr = "";
-		child.stderr.on("data", (chunk) => { stderr += String(chunk); });
+		child.stderr.on("data", (chunk) => {
+			stderr += String(chunk);
+		});
 		child.on("error", reject);
 		child.on("close", (code) => {
 			if (code === 0) resolve();
-			else reject(new Error(stderr.trim() || `${c.cmd} exited with code ${code}`));
+			else
+				reject(new Error(stderr.trim() || `${c.cmd} exited with code ${code}`));
 		});
 		child.stdin.end(text);
 	});
@@ -55,7 +63,8 @@ function copyToClipboard(text: string): Promise<void> {
 
 export default function (pi: ExtensionAPI) {
 	pi.registerCommand("copy-all", {
-		description: "Copy all user/assistant messages in this thread to the clipboard",
+		description:
+			"Copy all user/assistant messages in this thread to the clipboard",
 		handler: async (_args, ctx) => {
 			await ctx.waitForIdle();
 
@@ -80,9 +89,15 @@ export default function (pi: ExtensionAPI) {
 
 			try {
 				await copyToClipboard(text);
-				ctx.ui.notify(`📋 Copied ${messages.length} messages to clipboard`, "info");
+				ctx.ui.notify(
+					`📋 Copied ${messages.length} messages to clipboard`,
+					"info",
+				);
 			} catch (err) {
-				ctx.ui.notify(`Clipboard copy failed: ${err instanceof Error ? err.message : String(err)}`, "error");
+				ctx.ui.notify(
+					`Clipboard copy failed: ${err instanceof Error ? err.message : String(err)}`,
+					"error",
+				);
 			}
 		},
 	});

package/src/commands/diff/diff.ts

@@ -18,7 +18,8 @@ import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 const COMMAND = "diff";
 
 function getStringPath(input: unknown): string | undefined {
-	if (!input || typeof input !== "object" || !("path" in input)) return undefined;
+	if (!input || typeof input !== "object" || !("path" in input))
+		return undefined;
 	const p = (input as { path?: unknown }).path;
 	return typeof p === "string" ? p : undefined;
 }
@@ -45,8 +46,15 @@ function parseGitStatus(output: string, cwd: string): Set<string> {
 	return files;
 }
 
-async function getGitChanged(pi: ExtensionAPI, cwd: string): Promise<Set<string>> {
-	const r = await pi.exec("git", ["status", "--porcelain", "--untracked-files=all"], { cwd, timeout: 5000 });
+async function getGitChanged(
+	pi: ExtensionAPI,
+	cwd: string,
+): Promise<Set<string>> {
+	const r = await pi.exec(
+		"git",
+		["status", "--porcelain", "--untracked-files=all"],
+		{ cwd, timeout: 5000 },
+	);
 	if (r.code !== 0) return new Set();
 	return parseGitStatus(r.stdout, cwd);
 }
@@ -56,7 +64,8 @@ function diff(current: Set<string>, baseline: Set<string>): Set<string> {
 }
 
 function pickEditor(): { cmd: string; args: (file: string) => string[] } {
-	const env = process.env.PI_DIFF_EDITOR || process.env.VISUAL || process.env.EDITOR;
+	const env =
+		process.env.PI_DIFF_EDITOR || process.env.VISUAL || process.env.EDITOR;
 	if (env) {
 		const parts = env.split(/\s+/);
 		const cmd = parts[0];
@@ -88,12 +97,16 @@ export default function (pi: ExtensionAPI) {
 		const now = await getGitChanged(pi, ctx.cwd);
 		changed = new Set([...diff(now, baseline), ...touched]);
 		if (changed.size > 0) {
-			ctx.ui.notify(`📝 ${changed.size} changed file(s). Run /${COMMAND} to view/open.`, "info");
+			ctx.ui.notify(
+				`📝 ${changed.size} changed file(s). Run /${COMMAND} to view/open.`,
+				"info",
+			);
 		}
 	});
 
 	pi.registerCommand(COMMAND, {
-		description: "Show files changed by the last agent run and open one in your editor",
+		description:
+			"Show files changed by the last agent run and open one in your editor",
 		handler: async (args, ctx) => {
 			await ctx.waitForIdle();
 			const arg = (args ?? "").trim();
@@ -106,19 +119,30 @@ export default function (pi: ExtensionAPI) {
 				return;
 			}
 
-			const files = [...changed].sort((a, b) => toRel(ctx.cwd, a).localeCompare(toRel(ctx.cwd, b)));
+			const files = [...changed].sort((a, b) =>
+				toRel(ctx.cwd, a).localeCompare(toRel(ctx.cwd, b)),
+			);
 			if (files.length === 0) {
-				ctx.ui.notify("No changed files tracked from the last agent run", "info");
+				ctx.ui.notify(
+					"No changed files tracked from the last agent run",
+					"info",
+				);
 				return;
 			}
 
 			if (arg === "list") {
-				ctx.ui.notify(`Changed files:\n${files.map((f) => `- ${toRel(ctx.cwd, f)}`).join("\n")}`, "info");
+				ctx.ui.notify(
+					`Changed files:\n${files.map((f) => `- ${toRel(ctx.cwd, f)}`).join("\n")}`,
+					"info",
+				);
 				return;
 			}
 
 			if (arg) {
-				ctx.ui.notify(`Unknown /${COMMAND} argument: ${arg}. Try /${COMMAND}, /${COMMAND} list, /${COMMAND} clear.`, "warning");
+				ctx.ui.notify(
+					`Unknown /${COMMAND} argument: ${arg}. Try /${COMMAND}, /${COMMAND} list, /${COMMAND} clear.`,
+					"warning",
+				);
 				return;
 			}
 
@@ -130,9 +154,17 @@ export default function (pi: ExtensionAPI) {
 			if (!file) return;
 
 			const ed = pickEditor();
-			const r = await pi.exec(ed.cmd, ed.args(file), { cwd: ctx.cwd, timeout: 5000 });
-			if (r.code === 0) ctx.ui.notify(`Opened ${selected} in ${ed.cmd}`, "info");
-			else ctx.ui.notify(r.stderr.trim() || `Failed to open ${selected} in ${ed.cmd}`, "error");
+			const r = await pi.exec(ed.cmd, ed.args(file), {
+				cwd: ctx.cwd,
+				timeout: 5000,
+			});
+			if (r.code === 0)
+				ctx.ui.notify(`Opened ${selected} in ${ed.cmd}`, "info");
+			else
+				ctx.ui.notify(
+					r.stderr.trim() || `Failed to open ${selected} in ${ed.cmd}`,
+					"error",
+				);
 		},
 	});
 }

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

@@ -1,5 +1,5 @@
-import { describe, it, expect } from "bun:test";
-import { fmtCtx, fmtCost, benchStars, sortModels } from "./models.ts";
+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"));

package/src/commands/models/models.ts

@@ -23,7 +23,7 @@ import {
 	Text,
 	visibleWidth,
 } from "@earendil-works/pi-tui";
-import { lookupModelsDev, lookupBenchmark } from "../../lib/data";
+import { lookupBenchmark, lookupModelsDev } from "../../lib/data";
 
 // ─── Pure logic (exported for tests) ─────────────────────────────────────────
 

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

@@ -1,10 +1,10 @@
-import { describe, it, expect } from "bun:test";
+import { describe, expect, it } from "bun:test";
 import {
-	isTransient,
 	commandFor,
 	formatUpdateSummary,
-	PACKAGE_NAME,
 	type InstallMethod,
+	isTransient,
+	PACKAGE_NAME,
 } from "./update.ts";
 
 describe("isTransient", () => {
@@ -48,28 +48,28 @@ describe("commandFor", () => {
 		for (const m of methods) {
 			const spec = commandFor(m);
 			expect(spec).toBeDefined();
-			expect(spec!.command).toBeTruthy();
-			expect(spec!.label).toBeTruthy();
+			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");
+		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");
+		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");
+		expect(spec.args).toContain(`${PACKAGE_NAME}@latest`);
 	});
 
 	it("brew uses sh -lc", () => {

package/src/commands/update/update.ts

@@ -127,7 +127,7 @@ async function detectInstallMethod(pi: ExtensionAPI): Promise<InstallMethod> {
 			],
 			{ timeout: 10_000 },
 		);
-		if ((hasGlobalNpm.exitCode ?? 1) === 0) return "npm";
+		if ((hasGlobalNpm.code ?? 1) === 0) return "npm";
 	}
 
 	if (await resolveCommand("vp", pi)) return "vp";
@@ -145,7 +145,7 @@ async function runWithRetry(pi: ExtensionAPI, spec: CommandSpec) {
 			.filter(Boolean)
 			.join("\n")
 			.trim();
-		if ((result.exitCode ?? 0) === 0)
+		if ((result.code ?? 0) === 0)
 			return { ok: true, output: lastOutput, attempts: attempt };
 		if (attempt === 3 || !isTransient(lastOutput))
 			return { ok: false, output: lastOutput, attempts: attempt };
@@ -200,7 +200,7 @@ async function updateExtensions(
 		.filter(Boolean)
 		.join("\n")
 		.trim();
-	if ((result.exitCode ?? 0) !== 0) {
+	if ((result.code ?? 0) !== 0) {
 		ctx.ui.notify(
 			`Pi extensions update failed. ${output || "No output."}`,
 			"error",
@@ -222,7 +222,7 @@ async function updatePackages(pi: ExtensionAPI, ctx: ExtensionCommandContext) {
 		.filter(Boolean)
 		.join("\n")
 		.trim();
-	if ((result.exitCode ?? 0) !== 0) {
+	if ((result.code ?? 0) !== 0) {
 		ctx.ui.notify(
 			`Pi package update failed. ${output || "No output."}`,
 			"error",

package/src/index.ts

@@ -16,20 +16,20 @@
  */
 
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
-import registerWelcome from "./ui/welcome.ts";
-import registerFooter from "./ui/footer.ts";
-import registerDiagnostics from "./ui/diagnostics.ts";
+import registerClear from "./commands/clear/clear.ts";
+import registerCopyAll from "./commands/copy-all/copy-all.ts";
+import registerDiff from "./commands/diff/diff.ts";
+import registerLg from "./commands/lg/lg.ts";
 import registerModels from "./commands/models/models.ts";
 import registerUpdate from "./commands/update/update.ts";
-import registerLg from "./commands/lg/lg.ts";
 import registerYeet from "./commands/yeet/yeet.ts";
-import registerCopyAll from "./commands/copy-all/copy-all.ts";
-import registerDiff from "./commands/diff/diff.ts";
-import registerClear from "./commands/clear/clear.ts";
-import registerTodo from "./tool/todo/todo.ts";
+import registerNudges from "./nudge/index.ts";
 import registerAsk from "./tool/ask/ask.ts";
+import registerTodo from "./tool/todo/todo.ts";
 import registerToolbox from "./tool/toolbox/toolbox.ts";
-import registerNudges from "./nudge/index.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 {
 	registerWelcome(pi);

package/src/lib/data.ts

@@ -9,8 +9,8 @@
  *   models.ts  — lookupModelsDev, lookupBenchmark
  */
 
-import { mkdir, readFile, writeFile } from "node:fs/promises";
 import { existsSync, readFileSync } from "node:fs";
+import { mkdir, readFile, writeFile } from "node:fs/promises";
 import { homedir } from "node:os";
 import { dirname, join } from "node:path";
 

package/src/nudge/capability.test.ts

@@ -18,7 +18,6 @@ const skill = (
 		disableModelInvocation,
 		filePath: "",
 		baseDir: "",
-		// biome-ignore lint: test fixture
 		sourceInfo: {} as never,
 	}) as never;
 
@@ -29,7 +28,6 @@ const tool = (name: string, source: string) =>
 		description: `${name} desc.`,
 		parameters: {},
 		sourceInfo: { source, path: "", scope: "user", origin: "package" },
-		// biome-ignore lint: test fixture
 	}) as never;
 
 describe("CAPABILITY_REMINDER", () => {
@@ -176,18 +174,17 @@ describe("buildOrientation", () => {
 	test("lists invocable skill names, sorted, excluding user-only", () => {
 		const out = buildOrientation(
 			[],
-			[
-				skill("zebra", "z."),
-				skill("alpha", "a."),
-				skill("hidden", "h.", true),
-			],
+			[skill("zebra", "z."), skill("alpha", "a."), skill("hidden", "h.", true)],
 		);
 		expect(out).toContain("Skills: alpha, zebra.");
 		expect(out).not.toContain("hidden");
 	});
 
 	test("omits the skills line when no invocable skills", () => {
-		const out = buildOrientation([tool("read", "builtin")], [skill("x", ".", true)]);
+		const out = buildOrientation(
+			[tool("read", "builtin")],
+			[skill("x", ".", true)],
+		);
 		expect(out).not.toContain("Skills:");
 	});
 

package/src/nudge/capability.ts

@@ -32,7 +32,9 @@ export const CAPABILITY_REMINDER =
 	"All tools are always callable via function definitions.";
 
 /** Count model-invocable skills (excludes user-only /skill:name entries). */
-export function countInvocableSkills(skills: LoadedSkill[] | undefined): number {
+export function countInvocableSkills(
+	skills: LoadedSkill[] | undefined,
+): number {
 	return (skills ?? []).filter((s) => !s.disableModelInvocation).length;
 }
 

package/src/nudge/index.ts

@@ -8,8 +8,8 @@
  */
 
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
-import registerToolsNudge from "./tools.ts";
 import registerCapabilityNudge from "./capability.ts";
+import registerToolsNudge from "./tools.ts";
 
 export default function registerNudges(pi: ExtensionAPI): void {
 	registerToolsNudge(pi);

package/src/nudge/tools.test.ts

@@ -114,15 +114,23 @@ describe("classifyCompound", () => {
 });
 
 describe("nudgeReason", () => {
-		test("active tool: point straight at it, no toolbox", () => {
-		const msg = nudgeReason("Searching file contents via bash grep/rg.", "grep", true);
+	test("active tool: point straight at it, no toolbox", () => {
+		const msg = nudgeReason(
+			"Searching file contents via bash grep/rg.",
+			"grep",
+			true,
+		);
 		expect(msg).toContain("Use `grep` instead");
 		expect(msg).toContain("function definitions");
 		expect(msg).not.toContain("toolbox");
 	});
 
-		test("gated tool: route through toolbox enable, not a direct call", () => {
-		const msg = nudgeReason("Listing a directory via bash ls/tree.", "ls", false);
+	test("gated tool: route through toolbox enable, not a direct call", () => {
+		const msg = nudgeReason(
+			"Listing a directory via bash ls/tree.",
+			"ls",
+			false,
+		);
 		expect(msg).toContain('toolbox(action:"enable", name:"ls")');
 		expect(msg).toContain("prompt-hidden");
 		expect(msg).toContain("function definitions");
@@ -131,13 +139,17 @@ describe("nudgeReason", () => {
 	});
 
 	test("gated find tool names itself in the enable hint", () => {
-		expect(nudgeReason("Locating files via bash find/fd.", "find", false)).toContain(
-			'toolbox(action:"enable", name:"find")',
-		);
+		expect(
+			nudgeReason("Locating files via bash find/fd.", "find", false),
+		).toContain('toolbox(action:"enable", name:"find")');
 	});
 
-		test("is a single short line — no inventory dump, no newlines", () => {
-		const msg = nudgeReason("Listing a directory via bash ls/tree.", "ls", false);
+	test("is a single short line — no inventory dump, no newlines", () => {
+		const msg = nudgeReason(
+			"Listing a directory via bash ls/tree.",
+			"ls",
+			false,
+		);
 		expect(msg).not.toContain("\n");
 		expect(msg).not.toContain("Available tools");
 		expect(msg.length).toBeLessThan(400);

package/src/nudge/tools.ts

@@ -28,7 +28,6 @@ import {
 	isToolCallEventType,
 } from "@earendil-works/pi-coding-agent";
 
-
 /** Categories that map a raw shell command to a dedicated Pi tool. */
 type Category = "read" | "ls" | "grep" | "find" | "edit";
 
@@ -116,7 +115,6 @@ export function splitSegments(command: string): Segment[] {
 	const re = /(\|\||&&|[|;&\n])/g;
 	let last = 0;
 	let m: RegExpExecArray | null;
-	// biome-ignore lint/suspicious/noAssignInExpressions: standard regex walk
 	while ((m = re.exec(command)) !== null) {
 		out.push({ text: command.slice(last, m.index), followedBy: m[1] });
 		last = m.index + m[1].length;