@xynogen/pix-toolbox 0.1.6 → 0.1.8

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@xynogen/pix-toolbox",
-	"version": "0.1.6",
+	"version": "0.1.8",
 	"description": "Pi tool — gated tool toggle UI (/toolbox)",
 	"type": "module",
 	"main": "src/index.ts",

package/src/toolbox.test.ts

@@ -41,7 +41,7 @@ describe("buildRows", () => {
 
 	test("flags MCP tools", () => {
 		const rows = buildRows([mcpToolInfo("ctx_search")]);
-		expect(rows[0].mcp).toBe(true);
+		expect((rows[0] as ToolRow).mcp).toBe(true);
 	});
 
 	test("empty input returns empty", () => {
@@ -53,12 +53,7 @@ describe("buildRows", () => {
 
 describe("parseTargets", () => {
 	test("splits on commas, spaces, newlines", () => {
-		expect(parseTargets("ls, find  grep\nfetch")).toEqual([
-			"ls",
-			"find",
-			"grep",
-			"fetch",
-		]);
+		expect(parseTargets("ls, find  grep\nfetch")).toEqual(["ls", "find", "grep", "fetch"]);
 	});
 
 	test("dedupes", () => {
@@ -202,9 +197,7 @@ function makeHost(toolNames: string[]) {
 		},
 		emit(ev: string, payload: unknown, ctx?: unknown) {
 			return Promise.all(
-				(handlers[ev] ?? []).map((f) =>
-					(f as (...args: unknown[]) => unknown)(payload, ctx),
-				),
+				(handlers[ev] ?? []).map((f) => (f as (...args: unknown[]) => unknown)(payload, ctx)),
 			);
 		},
 		getAllTools() {
@@ -226,18 +219,13 @@ function makeHost(toolNames: string[]) {
 		},
 		appendEntry() {},
 		registerTool() {},
-		registerCommand(
-			name: string,
-			def: { handler: (...args: unknown[]) => unknown },
-		) {
+		registerCommand(name: string, def: { handler: (...args: unknown[]) => unknown }) {
 			commands.push({ name, handler: def.handler });
 		},
 	} as never;
 	const emit = (ev: string, payload: unknown, ctx?: unknown) =>
 		Promise.all(
-			(handlers[ev] ?? []).map((f) =>
-				(f as (...args: unknown[]) => unknown)(payload, ctx),
-			),
+			(handlers[ev] ?? []).map((f) => (f as (...args: unknown[]) => unknown)(payload, ctx)),
 		);
 	return {
 		pi,
@@ -260,6 +248,7 @@ function makeCtx() {
 }
 
 describe("/toolbox command", () => {
+	type Note = { text: string; level?: string };
 	const ALL = ["read", "write", "bash", "grep", "find"];
 
 	async function boot() {
@@ -281,28 +270,31 @@ describe("/toolbox command", () => {
 		await host.command("toolbox")?.handler("", ctx);
 		expect(notes.length).toBe(1);
 		// only non-core tools shown, all start active
-		expect(notes[0].text).toContain("✓ active  grep");
-		expect(notes[0].text).toContain("✓ active  find");
+		const n0 = notes[0] as Note;
+		expect(n0.text).toContain("✓ active  grep");
+		expect(n0.text).toContain("✓ active  find");
 		// core tools excluded from toolbox
-		expect(notes[0].text).not.toContain("  read");
-		expect(notes[0].text).not.toContain("  bash");
+		expect(n0.text).not.toContain("  read");
+		expect(n0.text).not.toContain("  bash");
 	});
 
 	test("/toolbox list shows non-core tools with status", async () => {
 		const host = await boot();
 		const { ctx, notes } = makeCtx();
 		await host.command("toolbox")?.handler("list", ctx);
-		expect(notes[0].text).toContain("grep");
-		expect(notes[0].text).toContain("find");
-		expect(notes[0].text).not.toContain("  bash");
+		const ln0 = notes[0] as Note;
+		expect(ln0.text).toContain("grep");
+		expect(ln0.text).toContain("find");
+		expect(ln0.text).not.toContain("  bash");
 	});
 
 	test("/toolbox list <query> filters", async () => {
 		const host = await boot();
 		const { ctx, notes } = makeCtx();
 		await host.command("toolbox")?.handler("list fin", ctx);
-		expect(notes[0].text).toContain("find");
-		expect(notes[0].text).not.toContain("✓ active  read");
+		const fn0 = notes[0] as Note;
+		expect(fn0.text).toContain("find");
+		expect(fn0.text).not.toContain("✓ active  read");
 	});
 
 	test("opens interactive picker when ctx.ui.custom exists", async () => {

package/src/toolbox.ts

@@ -39,12 +39,7 @@ import { frameLines, modalWidth } from "@xynogen/pix-pretty/modal-frame";
 // ─── Constants ──────────────────────────────────────────────────────────────
 
 /** Tools that can never be disabled — always prompt-visible. */
-export const CORE_TOOLS: ReadonlySet<string> = new Set([
-	"bash",
-	"edit",
-	"read",
-	"write",
-]);
+export const CORE_TOOLS: ReadonlySet<string> = new Set(["bash", "edit", "read", "write"]);
 
 // ─── Types ──────────────────────────────────────────────────────────────────
 
@@ -134,14 +129,10 @@ export function toggleTool(
 
 	if (action === "enable") {
 		const did = ops.onActivate(name);
-		return did
-			? `Enabled ${name} — now prompt-visible.`
-			: `${name} is already active.`;
+		return did ? `Enabled ${name} — now prompt-visible.` : `${name} is already active.`;
 	}
 	const did = ops.onDeactivate(name);
-	return did
-		? `Disabled ${name} — hidden from prompt.`
-		: `${name} is already gated.`;
+	return did ? `Disabled ${name} — hidden from prompt.` : `${name} is already gated.`;
 }
 
 // ─── Persistence ───────────────────────────────────────────────────────────
@@ -173,11 +164,7 @@ function createState(pi: ExtensionAPI) {
 		try {
 			const sp = getStatePath();
 			mkdirSync(dirname(sp), { recursive: true });
-			writeFileSync(
-				sp,
-				JSON.stringify({ enabledTools: [...enabledTools] }, null, 2),
-				"utf-8",
-			);
+			writeFileSync(sp, JSON.stringify({ enabledTools: [...enabledTools] }, null, 2), "utf-8");
 		} catch (err) {
 			console.warn("toolbox: file persist failed:", err);
 		}
@@ -203,9 +190,7 @@ function createState(pi: ExtensionAPI) {
 		const fileSaved = loadFromFile();
 		if (fileSaved) {
 			const validNames = new Set((pi.getAllTools() ?? []).map((t) => t.name));
-			enabledTools = new Set(
-				fileSaved.filter((n) => validNames.has(n) || CORE_TOOLS.has(n)),
-			);
+			enabledTools = new Set(fileSaved.filter((n) => validNames.has(n) || CORE_TOOLS.has(n)));
 			for (const ct of CORE_TOOLS) enabledTools.add(ct);
 			initialized = true;
 			apply();
@@ -233,9 +218,7 @@ function createState(pi: ExtensionAPI) {
 
 		if (saved) {
 			const validNames = new Set((pi.getAllTools() ?? []).map((t) => t.name));
-			enabledTools = new Set(
-				saved.filter((n) => validNames.has(n) || CORE_TOOLS.has(n)),
-			);
+			enabledTools = new Set(saved.filter((n) => validNames.has(n) || CORE_TOOLS.has(n)));
 			for (const ct of CORE_TOOLS) enabledTools.add(ct);
 		} else {
 			const names = (pi.getAllTools() ?? []).map((t) => t.name);
@@ -351,34 +334,24 @@ export default function registerToolbox(pi: ExtensionAPI): void {
 		};
 	}): Promise<void> {
 		await ctx.ui.custom<null>(
-			(
-				tui: TUI,
-				theme: Theme,
-				_kb: KeybindingsManager,
-				done: (r: null) => void,
-			) => {
+			(tui: TUI, theme: Theme, _kb: KeybindingsManager, done: (r: null) => void) => {
 				const accent = "accent";
 				const mute = (s: string) => theme.fg("muted", s);
 
 				type RowState = "active" | "gated";
-				const stateOf = (name: string): RowState =>
-					ops.isActive(name) ? "active" : "gated";
+				const stateOf = (name: string): RowState => (ops.isActive(name) ? "active" : "gated");
 
 				const labelFor = (r: ToolRow): string => {
 					const active = stateOf(r.name) === "active";
 					const marker = active ? " " : theme.fg("warning", "#");
-					const name = active
-						? theme.fg("success", r.name)
-						: theme.fg("dim", r.name);
+					const name = active ? theme.fg("success", r.name) : theme.fg("dim", r.name);
 					const kind = mute(`[${r.mcp ? "MCP" : "tool"}]`);
 					return `${marker} ${name}  ${kind}`;
 				};
 
 				const descFor = (r: ToolRow): string => {
 					const active = stateOf(r.name) === "active";
-					const tag = active
-						? theme.fg("success", "active")
-						: theme.fg("warning", "gated");
+					const tag = active ? theme.fg("success", "active") : theme.fg("warning", "gated");
 					return `${tag} ${mute("·")} ${r.description || "(no description)"}`;
 				};
 
@@ -394,10 +367,7 @@ export default function registerToolbox(pi: ExtensionAPI): void {
 				};
 
 				const allItems = rows.map(toItem);
-				const widest = allItems.reduce(
-					(w, it) => Math.max(w, visibleWidth(it.label)),
-					0,
-				);
+				const widest = allItems.reduce((w, it) => Math.max(w, visibleWidth(it.label)), 0);
 
 				const list = new SelectList(
 					allItems,
@@ -480,10 +450,7 @@ export default function registerToolbox(pi: ExtensionAPI): void {
 						];
 						if (statusText) lines.push(statusText);
 						lines.push(
-							theme.fg(
-								"dim",
-								"↑↓ navigate · e enable · d disable · space toggle · esc close",
-							),
+							theme.fg("dim", "↑↓ navigate · e enable · d disable · space toggle · esc close"),
 						);
 						return frameLines({
 							width: mw,
@@ -499,16 +466,10 @@ export default function registerToolbox(pi: ExtensionAPI): void {
 					handleInput(data: string) {
 						if (matchesKey(data, Key.up) || matchesKey(data, Key.down)) {
 							list.handleInput?.(data);
-						} else if (
-							matchesKey(data, Key.enter) ||
-							matchesKey(data, Key.escape)
-						) {
+						} else if (matchesKey(data, Key.enter) || matchesKey(data, Key.escape)) {
 							done(null);
 							return;
-						} else if (
-							matchesKey(data, Key.space) ||
-							matchesKey(data, Key.tab)
-						) {
+						} else if (matchesKey(data, Key.space) || matchesKey(data, Key.tab)) {
 							flipSelected();
 						} else {
 							const printable = decodeKittyPrintable(data);
@@ -553,9 +514,7 @@ export default function registerToolbox(pi: ExtensionAPI): void {
 					return;
 				}
 				const rows = getRows();
-				const msg = targets
-					.map((t) => toggleTool(verb, t, rows, ops))
-					.join("\n");
+				const msg = targets.map((t) => toggleTool(verb, t, rows, ops)).join("\n");
 				ctx.ui.notify(msg, "info");
 				return;
 			}