@membank/cli 0.4.0 → 0.5.0

package/dist/index.mjs

@@ -1,17 +1,20 @@
 #!/usr/bin/env node
+import { cancel, confirm, intro, isCancel, multiselect, note, outro } from "@clack/prompts";
 import { DatabaseManager, EmbeddingService, MEMORY_TYPE_VALUES, MemoryRepository, QueryEngine, SessionContextBuilder, resolveScope } from "@membank/core";
 import { startServer } from "@membank/mcp";
+import chalk from "chalk";
 import { Command } from "commander";
+import ora from "ora";
 import { startDashboard } from "@membank/dashboard";
 import { existsSync, mkdirSync, mkdtempSync, readFileSync, readdirSync, renameSync, writeFileSync } from "node:fs";
 import { dirname, join } from "node:path";
-import * as readline from "node:readline";
-import { createInterface } from "node:readline";
+import Table from "cli-table3";
 import { homedir, tmpdir } from "node:os";
 import { execFile } from "node:child_process";
 import { promisify } from "node:util";
 import { EventEmitter } from "node:events";
 import { pipeline } from "@huggingface/transformers";
+import { createInterface } from "node:readline";
 //#region src/commands/add.ts
 async function addCommand(content, options, formatter, db, embeddingService) {
 	const ownDb = db === void 0;
@@ -19,12 +22,14 @@ async function addCommand(content, options, formatter, db, embeddingService) {
 	try {
 		const repo = new MemoryRepository(resolvedDb, embeddingService ?? new EmbeddingService());
 		const tags = options.tags !== void 0 ? options.tags.split(",").map((t) => t.trim()) : [];
+		const spinner = formatter.isJson ? null : ora("Saving memory…").start();
 		const memory = await repo.save({
 			content,
 			type: options.type,
 			tags,
 			scope: options.scope
 		});
+		spinner?.succeed("Memory saved");
 		formatter.outputMemory(memory);
 	} finally {
 		if (ownDb) resolvedDb.close();
@@ -44,7 +49,7 @@ async function deleteCommand(id, db, formatter, prompt) {
 	}
 	if (!await prompt.confirm(`Delete memory ${id}?`)) return;
 	await new MemoryRepository(db, new EmbeddingService()).delete(id);
-	process.stdout.write(`Deleted memory: ${id}\n`);
+	process.stdout.write(`${chalk.green("✓")} Deleted memory: ${chalk.dim(id)}\n`);
 }
 //#endregion
 //#region src/commands/export.ts
@@ -134,7 +139,7 @@ async function importCommand(filePath, db, formatter, prompt) {
 }
 //#endregion
 //#region src/commands/inject.ts
-const MEMORY_GUIDANCE = "[Memory Guidance]: Persistent memory is available via query_memory, save_memory, update_memory, delete_memory. Skipping save_memory when the user gives a correction or preference means they have to repeat themselves next session — that is the failure mode to avoid. Skipping query_memory on topics that touch prior decisions means contradicting yourself. Default to saving (type: correction|preference|decision|learning|fact) when in doubt; rely on dedup to handle redundancy. Pin anything that should appear at every session start.";
+const MEMORY_GUIDANCE = "[Memory Guidance]: Call save_memory when ANY of these happen: (1) user states a preference or makes a decision; (2) user corrects you; (3) you discover a working fix after a tool error; (4) you learn a non-obvious project fact. Type ∈ correction|preference|decision|learning|fact. Call query_memory before answering anything that might touch prior decisions. When unsure, save.";
 function formatContext(ctx) {
 	const lines = [];
 	const statParts = Object.entries(ctx.stats).filter(([, count]) => count > 0).map(([type, count]) => `${count} ${type}${count !== 1 ? "s" : ""}`);
@@ -202,7 +207,7 @@ function pinCommand(id, formatter, db) {
 			process.exit(2);
 		} else {
 			resolvedDb.db.prepare("UPDATE memories SET pinned = 1 WHERE id = ?").run(id);
-			process.stdout.write(`Pinned: ${id}\n`);
+			process.stdout.write(`${chalk.green("✓")} Pinned: ${chalk.dim(id)}\n`);
 		}
 	} finally {
 		if (ownDb) resolvedDb.close();
@@ -216,11 +221,13 @@ async function queryCommand(queryText, options, formatter) {
 		const embedding = new EmbeddingService();
 		const engine = new QueryEngine(db, embedding, new MemoryRepository(db, embedding));
 		const limit = options.limit !== void 0 ? Number.parseInt(options.limit, 10) : 10;
+		const spinner = formatter.isJson ? null : ora("Searching memories…").start();
 		const results = await engine.query({
 			query: queryText,
 			type: options.type,
 			limit
 		});
+		spinner?.succeed(`${results.length} result${results.length === 1 ? "" : "s"} found`);
 		formatter.outputQueryResults(results);
 	} finally {
 		db.close();
@@ -248,7 +255,7 @@ function unpinCommand(id, formatter, db) {
 			process.exit(2);
 		} else {
 			resolvedDb.db.prepare("UPDATE memories SET pinned = 0 WHERE id = ?").run(id);
-			process.stdout.write(`Unpinned: ${id}\n`);
+			process.stdout.write(`${chalk.green("✓")} Unpinned: ${chalk.dim(id)}\n`);
 		}
 	} finally {
 		if (ownDb) resolvedDb.close();
@@ -256,6 +263,19 @@ function unpinCommand(id, formatter, db) {
 }
 //#endregion
 //#region src/formatter.ts
+const TYPE_COLORS = {
+	correction: chalk.yellow,
+	preference: chalk.cyan,
+	decision: chalk.blue,
+	learning: chalk.green,
+	fact: chalk.dim
+};
+function colorType(type) {
+	return TYPE_COLORS[type](type);
+}
+function truncate(str, max) {
+	return str.length > max ? `${str.slice(0, max - 1)}…` : str;
+}
 var Formatter = class Formatter {
 	#isJson;
 	constructor(isJson) {
@@ -272,7 +292,12 @@ var Formatter = class Formatter {
 			process.stdout.write(`${JSON.stringify(memory)}\n`);
 			return;
 		}
-		this.#writeMemoryBlock(memory, `  Pinned  : ${memory.pinned}\n`);
+		const tags = memory.tags.length > 0 ? memory.tags.join(", ") : "(none)";
+		process.stdout.write("\n");
+		process.stdout.write(`  ${colorType(memory.type)}  ${chalk.dim(memory.id)}\n`);
+		process.stdout.write(`  ${memory.content}\n`);
+		process.stdout.write(`  ${chalk.dim("Tags:")} ${tags}  ${chalk.dim("Scope:")} ${memory.scope}\n`);
+		process.stdout.write(`\n  ${chalk.dim(`Hint: pin with  membank pin ${memory.id}`)}\n\n`);
 	}
 	outputMemories(memories) {
 		if (this.#isJson) {
@@ -280,26 +305,51 @@ var Formatter = class Formatter {
 			return;
 		}
 		if (memories.length === 0) {
-			process.stdout.write("No memories found.\n");
+			process.stdout.write(`${chalk.dim("No memories found.")}\n`);
 			return;
 		}
-		for (const memory of memories) this.#writeMemoryBlock(memory);
-		process.stdout.write("\n");
+		const table = new Table({
+			head: [
+				"Type",
+				"ID",
+				"Content",
+				"Pinned"
+			].map((h) => chalk.bold(h)),
+			style: {
+				head: [],
+				border: []
+			}
+		});
+		for (const m of memories) {
+			const tags = m.tags.length > 0 ? m.tags.join(", ") : "(none)";
+			const meta = `${truncate(m.content, 45)}\n${chalk.dim(`${tags} · ${m.scope}`)}`;
+			table.push([
+				colorType(m.type),
+				chalk.dim(m.id),
+				meta,
+				m.pinned ? "📌" : ""
+			]);
+		}
+		process.stdout.write(`\n${table.toString()}\n\n`);
 	}
 	outputStats(stats) {
 		if (this.#isJson) {
 			process.stdout.write(`${JSON.stringify(stats)}\n`);
 			return;
 		}
-		for (const type of [
+		const types = [
 			"correction",
 			"preference",
 			"decision",
 			"learning",
 			"fact"
-		]) process.stdout.write(`  ${type.padEnd(12)}: ${stats.byType[type]}\n`);
-		process.stdout.write(`\n  total       : ${stats.total}\n`);
-		process.stdout.write(`  needs_review: ${stats.needsReview}\n`);
+		];
+		process.stdout.write("\n");
+		for (const type of types) process.stdout.write(`  ${TYPE_COLORS[type](type.padEnd(14))}  ${stats.byType[type]}\n`);
+		process.stdout.write(`\n  ${chalk.dim("─".repeat(24))}\n`);
+		process.stdout.write(`  ${"total".padEnd(14)}  ${stats.total}\n`);
+		if (stats.needsReview > 0) process.stdout.write(`  ${chalk.yellow("⚠")} ${"needs_review".padEnd(12)}  ${stats.needsReview}\n\n`);
+		else process.stdout.write(`  ${"  needs_review".padEnd(14)}  ${stats.needsReview}\n\n`);
 	}
 	outputQueryResults(results) {
 		if (this.#isJson) {
@@ -307,24 +357,38 @@ var Formatter = class Formatter {
 			return;
 		}
 		if (results.length === 0) {
-			process.stdout.write("No memories found.\n");
+			process.stdout.write(`${chalk.dim("No memories found.")}\n`);
 			return;
 		}
-		for (const result of results) this.#writeMemoryBlock(result, `  Score   : ${result.score.toFixed(4)}\n`);
-		process.stdout.write("\n");
+		const table = new Table({
+			head: [
+				"Type",
+				"ID",
+				"Content",
+				"Score"
+			].map((h) => chalk.bold(h)),
+			style: {
+				head: [],
+				border: []
+			}
+		});
+		for (const r of results) {
+			const scoreStr = r.score.toFixed(4);
+			const score = r.score >= .85 ? chalk.bold(scoreStr) : r.score < .75 ? chalk.dim(scoreStr) : scoreStr;
+			const tags = r.tags.length > 0 ? r.tags.join(", ") : "(none)";
+			const meta = `${truncate(r.content, 45)}\n${chalk.dim(`${tags} · ${r.scope}`)}`;
+			table.push([
+				colorType(r.type),
+				chalk.dim(r.id),
+				meta,
+				score
+			]);
+		}
+		process.stdout.write(`\n${table.toString()}\n\n`);
 	}
 	error(msg) {
 		if (this.#isJson) process.stderr.write(`${JSON.stringify({ error: msg })}\n`);
-		else process.stderr.write(`Error: ${msg}\n`);
-	}
-	#writeMemoryBlock(memory, extra) {
-		const tags = memory.tags.length > 0 ? memory.tags.join(", ") : "(none)";
-		process.stdout.write(`\n[${memory.type}] ${memory.id}\n`);
-		process.stdout.write(`  Content : ${memory.content}\n`);
-		process.stdout.write(`  Tags    : ${tags}\n`);
-		process.stdout.write(`  Scope   : ${memory.scope}\n`);
-		if (extra !== void 0) process.stdout.write(extra);
-		process.stdout.write("\n");
+		else process.stderr.write(`${chalk.red("Error:")} ${msg}\n`);
 	}
 };
 //#endregion
@@ -333,18 +397,11 @@ var PromptHelper = class {
 	constructor(autoConfirm) {
 		this.autoConfirm = autoConfirm;
 	}
-	confirm(message) {
-		if (this.autoConfirm) return Promise.resolve(true);
-		return new Promise((resolve) => {
-			const rl = readline.createInterface({
-				input: process.stdin,
-				output: process.stdout
-			});
-			rl.question(`${message} [y/N] `, (answer) => {
-				rl.close();
-				resolve(answer.trim().toLowerCase() === "y");
-			});
-		});
+	async confirm(message) {
+		if (this.autoConfirm) return true;
+		const result = await confirm({ message });
+		if (typeof result === "symbol") return false;
+		return result;
 	}
 };
 //#endregion
@@ -384,6 +441,14 @@ async function execFileNoThrow(cmd, args) {
 }
 //#endregion
 //#region src/setup/harness-config-writer.ts
+var CommandError = class extends Error {
+	command;
+	constructor(message, command) {
+		super(message);
+		this.name = "CommandError";
+		this.command = command;
+	}
+};
 const defaultPathResolver$1 = {
 	home: () => {
 		const h = process.env.HOME ?? process.env.USERPROFILE;
@@ -408,103 +473,150 @@ function writeJsonAtomic$1(path, data) {
 function hasKey(container, key) {
 	return container !== null && typeof container === "object" && key in container;
 }
-function assertCliFound(result, cli) {
-	if (result.exitCode === 127) throw new Error(`${cli} CLI not found — install ${cli} first`);
+function assertCliFound(result, cli, command) {
+	if (result.exitCode === 127) throw new CommandError(`${cli} CLI not found — install ${cli} first`, command);
 }
 const MEMBANK_NPX_ARGS = [
 	"npx",
-	"@membank/cli@latest",
+	"-y",
+	"@membank/cli",
 	"--mcp"
 ];
 const writers$1 = {
-	"claude-code": { async write(resolver, run, { overwrite = false } = {}) {
-		const configured = hasKey(readJson$1(join(resolver.home(), ".claude.json")).mcpServers, "membank");
-		if (configured && !overwrite) return { status: "already-configured" };
-		if (configured) {
-			const remove = await run("claude", [
+	"claude-code": {
+		preview(resolver) {
+			return {
+				configPath: join(resolver.home(), ".claude.json"),
+				cliCommand: "claude mcp add --scope user membank -- npx -y @membank/cli --mcp"
+			};
+		},
+		async write(resolver, run, { overwrite = false } = {}) {
+			const configured = hasKey(readJson$1(join(resolver.home(), ".claude.json")).mcpServers, "membank");
+			if (configured && !overwrite) return { status: "already-configured" };
+			if (configured) {
+				const removeArgs = [
+					"mcp",
+					"remove",
+					"--scope",
+					"user",
+					"membank"
+				];
+				const removeCmd = `claude ${removeArgs.join(" ")}`;
+				const remove = await run("claude", [...removeArgs]);
+				assertCliFound(remove, "claude", removeCmd);
+				if (remove.exitCode !== 0) throw new CommandError(`claude mcp remove failed: ${remove.stderr}`, removeCmd);
+			}
+			const addArgs = [
 				"mcp",
-				"remove",
+				"add",
 				"--scope",
 				"user",
-				"membank"
-			]);
-			assertCliFound(remove, "claude");
-			if (remove.exitCode !== 0) throw new Error(`claude mcp remove failed: ${remove.stderr}`);
+				"membank",
+				"--",
+				...MEMBANK_NPX_ARGS
+			];
+			const addCmd = `claude ${addArgs.join(" ")}`;
+			const add = await run("claude", [...addArgs]);
+			assertCliFound(add, "claude", addCmd);
+			if (add.exitCode !== 0) throw new CommandError(`claude mcp add failed: ${add.stderr || add.stdout}`, addCmd);
+			return { status: "written" };
 		}
-		const add = await run("claude", [
-			"mcp",
-			"add",
-			"--scope",
-			"user",
-			"membank",
-			"--",
-			...MEMBANK_NPX_ARGS
-		]);
-		assertCliFound(add, "claude");
-		if (add.exitCode !== 0) throw new Error(`claude mcp add failed: ${add.stderr || add.stdout}`);
-		return { status: "written" };
-	} },
-	copilot: { async write(resolver, _run, { overwrite = false } = {}) {
-		const cfgPath = join(resolver.home(), ".copilot", "mcp-config.json");
-		const cfg = readJson$1(cfgPath);
-		if (hasKey(cfg.mcpServers, "membank") && !overwrite) return { status: "already-configured" };
-		writeJsonAtomic$1(cfgPath, {
-			...cfg,
-			mcpServers: {
-				...cfg.mcpServers,
-				membank: {
-					command: "npx",
-					args: ["@membank/cli@latest", "--mcp"]
+	},
+	copilot: {
+		preview(resolver) {
+			return {
+				configPath: join(resolver.home(), ".copilot", "mcp-config.json"),
+				cliCommand: null
+			};
+		},
+		async write(resolver, _run, { overwrite = false } = {}) {
+			const cfgPath = join(resolver.home(), ".copilot", "mcp-config.json");
+			const cfg = readJson$1(cfgPath);
+			if (hasKey(cfg.mcpServers, "membank") && !overwrite) return { status: "already-configured" };
+			writeJsonAtomic$1(cfgPath, {
+				...cfg,
+				mcpServers: {
+					...cfg.mcpServers,
+					membank: {
+						command: "npx",
+						args: [
+							"-y",
+							"@membank/cli",
+							"--mcp"
+						]
+					}
 				}
+			});
+			return { status: "written" };
+		}
+	},
+	codex: {
+		preview(_resolver) {
+			return {
+				configPath: null,
+				cliCommand: "codex mcp add membank -- npx -y @membank/cli --mcp"
+			};
+		},
+		async write(_resolver, run, { overwrite = false } = {}) {
+			const listCmd = "codex mcp list";
+			const list = await run("codex", ["mcp", "list"]);
+			assertCliFound(list, "codex", listCmd);
+			const configured = list.exitCode === 0 && list.stdout.includes("membank");
+			if (configured && !overwrite) return { status: "already-configured" };
+			if (configured) {
+				const removeArgs = [
+					"mcp",
+					"remove",
+					"membank"
+				];
+				const removeCmd = `codex ${removeArgs.join(" ")}`;
+				const remove = await run("codex", [...removeArgs]);
+				assertCliFound(remove, "codex", removeCmd);
+				if (remove.exitCode !== 0) throw new CommandError(`codex mcp remove failed: ${remove.stderr}`, removeCmd);
 			}
-		});
-		return { status: "written" };
-	} },
-	codex: { async write(_resolver, run, { overwrite = false } = {}) {
-		const list = await run("codex", ["mcp", "list"]);
-		assertCliFound(list, "codex");
-		const configured = list.exitCode === 0 && list.stdout.includes("membank");
-		if (configured && !overwrite) return { status: "already-configured" };
-		if (configured) {
-			const remove = await run("codex", [
+			const addArgs = [
 				"mcp",
-				"remove",
-				"membank"
-			]);
-			assertCliFound(remove, "codex");
-			if (remove.exitCode !== 0) throw new Error(`codex mcp remove failed: ${remove.stderr}`);
+				"add",
+				"membank",
+				"--",
+				...MEMBANK_NPX_ARGS
+			];
+			const addCmd = `codex ${addArgs.join(" ")}`;
+			const add = await run("codex", [...addArgs]);
+			assertCliFound(add, "codex", addCmd);
+			if (add.exitCode !== 0) throw new CommandError(`codex mcp add failed: ${add.stderr || add.stdout}`, addCmd);
+			return { status: "written" };
 		}
-		const add = await run("codex", [
-			"mcp",
-			"add",
-			"membank",
-			"--",
-			...MEMBANK_NPX_ARGS
-		]);
-		assertCliFound(add, "codex");
-		if (add.exitCode !== 0) throw new Error(`codex mcp add failed: ${add.stderr || add.stdout}`);
-		return { status: "written" };
-	} },
-	opencode: { async write(resolver, _run, { overwrite = false } = {}) {
-		const cfgPath = join(resolver.home(), ".config", "opencode", "opencode.json");
-		const cfg = readJson$1(cfgPath);
-		if (hasKey(cfg.mcp, "membank") && !overwrite) return { status: "already-configured" };
-		writeJsonAtomic$1(cfgPath, {
-			...cfg,
-			mcp: {
-				...cfg.mcp,
-				membank: {
-					type: "local",
-					command: [
-						"npx",
-						"@membank/cli@latest",
-						"--mcp"
-					]
+	},
+	opencode: {
+		preview(resolver) {
+			return {
+				configPath: join(resolver.home(), ".config", "opencode", "opencode.json"),
+				cliCommand: null
+			};
+		},
+		async write(resolver, _run, { overwrite = false } = {}) {
+			const cfgPath = join(resolver.home(), ".config", "opencode", "opencode.json");
+			const cfg = readJson$1(cfgPath);
+			if (hasKey(cfg.mcp, "membank") && !overwrite) return { status: "already-configured" };
+			writeJsonAtomic$1(cfgPath, {
+				...cfg,
+				mcp: {
+					...cfg.mcp,
+					membank: {
+						type: "local",
+						command: [
+							"npx",
+							"-y",
+							"@membank/cli",
+							"--mcp"
+						]
+					}
 				}
-			}
-		});
-		return { status: "written" };
-	} }
+			});
+			return { status: "written" };
+		}
+	}
 };
 const SUPPORTED_HARNESSES = Object.keys(writers$1);
 var HarnessConfigWriter = class {
@@ -514,6 +626,11 @@ var HarnessConfigWriter = class {
 		this.#resolver = resolver;
 		this.#run = run;
 	}
+	preview(harness) {
+		const writer = writers$1[harness];
+		if (!writer) throw new Error(`Unknown harness: ${harness}`);
+		return writer.preview(this.#resolver);
+	}
 	async write(harness, { overwrite = false } = {}) {
 		const writer = writers$1[harness];
 		if (!writer) throw new Error(`Unknown harness: ${harness}`);
@@ -582,12 +699,14 @@ function pruneFlatEvent(hooks, eventKey) {
 const writers = {
 	"claude-code": {
 		inspect(resolver) {
-			const hooks = readJson(join(resolver.home(), ".claude", "settings.json")).hooks ?? {};
+			const cfgPath = join(resolver.home(), ".claude", "settings.json");
+			const hooks = readJson(cfgPath).hooks ?? {};
 			return {
 				status: "ready",
+				configPath: cfgPath,
 				hooks: [{
 					event: "SessionStart",
-					command: "npx @membank/cli@latest inject --harness claude-code",
+					command: "npx -y @membank/cli inject --harness claude-code",
 					existingCommand: extractInjectCommand((Array.isArray(hooks.SessionStart) ? hooks.SessionStart : []).flatMap(getHooksArray)) || null
 				}]
 			};
@@ -603,7 +722,7 @@ const writers = {
 				matcher: "",
 				hooks: [{
 					type: "command",
-					command: "npx @membank/cli@latest inject --harness claude-code"
+					command: "npx -y @membank/cli inject --harness claude-code"
 				}]
 			}];
 			writeJsonAtomic(cfgPath, {
@@ -615,12 +734,14 @@ const writers = {
 	},
 	"copilot-cli": {
 		inspect(resolver) {
-			const hooks = readJson(join(resolver.home(), ".copilot", "settings.json")).hooks ?? {};
+			const cfgPath = join(resolver.home(), ".copilot", "settings.json");
+			const hooks = readJson(cfgPath).hooks ?? {};
 			return {
 				status: "ready",
+				configPath: cfgPath,
 				hooks: [{
 					event: "sessionStart",
-					command: "npx @membank/cli@latest inject --harness copilot-cli",
+					command: "npx -y @membank/cli inject --harness copilot-cli",
 					existingCommand: extractInjectCommand(Array.isArray(hooks.sessionStart) ? hooks.sessionStart : []) || null
 				}]
 			};
@@ -634,7 +755,7 @@ const writers = {
 			pruneFlatEvent(newHooks, "postToolUseFailure");
 			if (events.includes("sessionStart")) newHooks.sessionStart = [...filterOutMembankFlat(Array.isArray(hooks.sessionStart) ? hooks.sessionStart : []), {
 				type: "command",
-				bash: "npx @membank/cli@latest inject --harness copilot-cli",
+				bash: "npx -y @membank/cli inject --harness copilot-cli",
 				timeoutSec: 30
 			}];
 			writeJsonAtomic(cfgPath, {
@@ -647,12 +768,14 @@ const writers = {
 	},
 	codex: {
 		inspect(resolver) {
-			const hooks = readJson(join(resolver.home(), ".codex", "hooks.json")).hooks ?? {};
+			const cfgPath = join(resolver.home(), ".codex", "hooks.json");
+			const hooks = readJson(cfgPath).hooks ?? {};
 			return {
 				status: "ready",
+				configPath: cfgPath,
 				hooks: [{
 					event: "SessionStart",
-					command: "npx @membank/cli@latest inject --harness codex",
+					command: "npx -y @membank/cli inject --harness codex",
 					existingCommand: extractInjectCommand((Array.isArray(hooks.SessionStart) ? hooks.SessionStart : []).flatMap(getHooksArray)) || null
 				}]
 			};
@@ -668,7 +791,7 @@ const writers = {
 				matcher: "",
 				hooks: [{
 					type: "command",
-					command: "npx @membank/cli@latest inject --harness codex",
+					command: "npx -y @membank/cli inject --harness codex",
 					timeout: 30
 				}]
 			}];
@@ -688,6 +811,7 @@ const writers = {
 			}
 			return {
 				status: "ready",
+				configPath: pluginPath,
 				hooks: [{
 					event: "plugin",
 					command: pluginPath,
@@ -709,7 +833,7 @@ function newOpencodePlugin() {
 		"export default {",
 		"  hooks: {",
 		"    \"session.start\": async ({ $ }) => {",
-		"      return await $`npx @membank/cli@latest inject`.text();",
+		"      return await $`npx -y @membank/cli inject`.text();",
 		"    },",
 		"  },",
 		"};"
@@ -757,6 +881,12 @@ var ModelDownloader = class extends EventEmitter {
 		super();
 		this.modelPath = modelPath ?? defaultModelPath();
 	}
+	isAlreadyCached() {
+		return isCached(this.modelPath);
+	}
+	get cachePath() {
+		return this.modelPath;
+	}
 	async download() {
 		if (isCached(this.modelPath)) return { skipped: true };
 		const startTime = Date.now();
@@ -852,6 +982,7 @@ var SetupOrchestrator = class {
 	#writer;
 	#hookWriter;
 	#prompter;
+	#harnessSelector;
 	#modelDownloader;
 	#out;
 	#progressWrite;
@@ -860,6 +991,7 @@ var SetupOrchestrator = class {
 		this.#writer = deps.writer;
 		this.#hookWriter = deps.hookWriter;
 		this.#prompter = deps.prompter ?? defaultPrompter;
+		this.#harnessSelector = deps.harnessSelector;
 		this.#modelDownloader = deps.modelDownloader;
 		this.#out = deps.out ?? ((msg) => process.stdout.write(`${msg}\n`));
 		this.#progressWrite = deps.progressWrite ?? ((text) => process.stdout.write(text));
@@ -885,6 +1017,13 @@ var SetupOrchestrator = class {
 			}));
 			return [];
 		}
+		if (this.#harnessSelector !== void 0 && !json) {
+			detected = await this.#harnessSelector(detected);
+			if (detected.length === 0) {
+				out("No harnesses selected.");
+				return [];
+			}
+		}
 		if (!json) {
 			out("Detected harnesses:");
 			for (const h of detected) out(`  • ${h.name}  (${h.configPath})`);
@@ -893,11 +1032,27 @@ var SetupOrchestrator = class {
 		if (dryRun) {
 			out("Planned changes (dry-run — no files written):");
 			for (const h of detected) {
-				out(`  ⚠ ${h.name}: would write MCP config`);
-				if (this.#hookWriter) out(`  ⚠ ${h.name}: would write injection hook config`);
+				const mcpPreview = this.#writer.preview(h.name);
+				const mcpTarget = mcpPreview.configPath ?? (mcpPreview.cliCommand ? "(via CLI)" : "");
+				out(`  ⚠ ${h.name}  MCP config → ${mcpTarget}`);
+				if (mcpPreview.cliCommand) out(`      via: ${mcpPreview.cliCommand}`);
+				if (this.#hookWriter) {
+					const inspected = this.#hookWriter.inspect(h.name);
+					if (inspected.status === "ready") {
+						out(`  ⚠ ${h.name}  injection hook → ${inspected.configPath}`);
+						for (const hook of inspected.hooks) {
+							out(`      event:   ${hook.event}`);
+							out(`      command: ${hook.command}`);
+						}
+					}
+				}
 			}
 			out("");
-			out("  ⚠ Model download: skipped (dry-run)");
+			if (this.#modelDownloader) {
+				const cached = this.#modelDownloader.isAlreadyCached();
+				out(`  ⚠ Model: ${MODEL_NAME} → ${this.#modelDownloader.cachePath}`);
+				out(`      status: ${cached ? "already cached — would skip" : "not cached — would download"}`);
+			} else out("  ⚠ Model download: skipped (dry-run)");
 			return detected.map((h) => ({
 				harness: h.name,
 				status: "skipped"
@@ -910,6 +1065,8 @@ var SetupOrchestrator = class {
 			}
 		}
 		const results = [];
+		const totalSteps = this.#hookWriter !== void 0 ? 3 : this.#modelDownloader !== void 0 ? 2 : 1;
+		out(`Step 1/${totalSteps}  Writing MCP configs`);
 		for (const h of detected) {
 			let writeResult;
 			try {
@@ -917,6 +1074,7 @@ var SetupOrchestrator = class {
 			} catch (err) {
 				const msg = err instanceof Error ? err.message : String(err);
 				out(`  ✗ ${h.name}: ${msg}`);
+				if (err instanceof CommandError) out(`    Command: ${err.command}`);
 				results.push({
 					harness: h.name,
 					status: "error",
@@ -945,6 +1103,7 @@ var SetupOrchestrator = class {
 				} catch (err) {
 					const msg = err instanceof Error ? err.message : String(err);
 					out(`  ✗ ${h.name}: ${msg}`);
+					if (err instanceof CommandError) out(`    Command: ${err.command}`);
 					results.push({
 						harness: h.name,
 						status: "error",
@@ -962,12 +1121,15 @@ var SetupOrchestrator = class {
 		out("");
 		const injectionHooksConfigured = [];
 		if (this.#hookWriter) {
+			out(`Step 2/${totalSteps}  Installing injection hooks`);
 			injectionHooksConfigured.push(...await this.#runHookSetup(detected, yes, out));
 			out("");
 		}
 		let modelDownloaded = false;
-		if (this.#modelDownloader) modelDownloaded = !(await this.#runModelDownload(this.#modelDownloader, out)).skipped;
-		else out("Model download step: see DRA-52");
+		if (this.#modelDownloader) {
+			out(`Step ${this.#hookWriter !== void 0 ? 3 : 2}/${totalSteps}  Embedding model`);
+			modelDownloaded = !(await this.#runModelDownload(this.#modelDownloader, out)).skipped;
+		} else out("Model download step: see DRA-52");
 		const written = results.filter((r) => r.status === "written").length;
 		const skipped = results.filter((r) => r.status === "already-configured").length;
 		const errors = results.filter((r) => r.status === "error").length;
@@ -988,6 +1150,7 @@ var SetupOrchestrator = class {
 	async #runHookSetup(detected, yes, out) {
 		const configured = [];
 		const w = this.#hookWriter;
+		if (w === void 0) return configured;
 		for (const h of detected) try {
 			const inspected = w.inspect(h.name);
 			if (inspected.status === "not-supported") continue;
@@ -1012,6 +1175,7 @@ var SetupOrchestrator = class {
 		} catch (err) {
 			const msg = err instanceof Error ? err.message : String(err);
 			out(`  ✗ ${h.name} injection hooks: ${msg}`);
+			if (err instanceof CommandError) out(`    Command: ${err.command}`);
 		}
 		return configured;
 	}
@@ -1157,28 +1321,65 @@ program.command("setup").description("detect installed harnesses and write MCP c
 	const globalOpts = program.opts();
 	const autoYes = cmdOptions.yes === true || globalOpts.yes === true;
 	const formatter = Formatter.create(globalOpts.json === true);
+	const interactive = !formatter.isJson && !autoYes && cmdOptions.harness === void 0;
 	if (cmdOptions.harness !== void 0) {
 		if (!SUPPORTED_HARNESSES.some((h) => h === cmdOptions.harness)) {
 			formatter.error(`Unknown harness: "${cmdOptions.harness}". Supported: ${SUPPORTED_HARNESSES.join(", ")}`);
 			process.exit(1);
 		}
 	}
+	if (!formatter.isJson) intro(chalk.bold("  membank  setup  "));
+	function decoratedOut(msg) {
+		const decorated = msg.replace(/✓/g, chalk.green("✓")).replace(/✗/g, chalk.red("✗")).replace(/⚠/g, chalk.yellow("⚠"));
+		const styled = /^Step \d/.test(msg) ? `\n${chalk.bold(decorated)}` : decorated;
+		process.stdout.write(`${styled}\n`);
+	}
 	const writer = new HarnessConfigWriter();
 	const hookWriter = new InjectionHookWriter();
 	const promptHelper = new PromptHelper(autoYes);
+	let harnessSelector;
+	if (interactive) harnessSelector = async (detected) => {
+		const selected = await multiselect({
+			message: "Which harnesses to configure?",
+			options: SUPPORTED_HARNESSES.map((name) => {
+				const found = detected.find((d) => d.name === name);
+				return {
+					value: found ?? {
+						name,
+						configPath: ""
+					},
+					label: name,
+					hint: found !== void 0 ? found.configPath : "(not detected)"
+				};
+			}),
+			initialValues: detected
+		});
+		if (isCancel(selected)) {
+			cancel("Setup cancelled.");
+			process.exit(0);
+		}
+		return selected;
+	};
 	const orchestrator = new SetupOrchestrator({
 		writer,
 		hookWriter,
 		prompter: (question) => promptHelper.confirm(question),
-		modelDownloader: new ModelDownloader()
+		harnessSelector,
+		modelDownloader: new ModelDownloader(),
+		out: formatter.isJson ? void 0 : decoratedOut
 	});
 	try {
-		if ((await orchestrator.run({
+		const results = await orchestrator.run({
 			yes: autoYes,
 			dryRun: cmdOptions.dryRun,
 			harness: cmdOptions.harness,
 			json: formatter.isJson
-		})).some((r) => r.status === "error")) process.exit(1);
+		});
+		if (!formatter.isJson && !cmdOptions.dryRun && results.length > 0) {
+			note("Start a new session to activate injection\nRun  membank query \"test\"  to verify", "Next steps");
+			outro(`${chalk.green("✓")} Setup complete`);
+		}
+		if (results.some((r) => r.status === "error")) process.exit(1);
 	} catch (err) {
 		formatter.error(err instanceof Error ? err.message : String(err));
 		process.exit(2);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@membank/cli",
-  "version": "0.4.0",
+  "version": "0.5.0",
   "type": "module",
   "repository": {
     "type": "git",
@@ -14,12 +14,15 @@
     "dist"
   ],
   "dependencies": {
+    "@clack/prompts": "^1.3.0",
     "@huggingface/transformers": "^4.2.0",
+    "chalk": "^5.6.2",
+    "cli-table3": "^0.6.5",
     "commander": "^14.0.3",
     "ora": "^9.4.0",
-    "@membank/core": "0.4.0",
-    "@membank/dashboard": "0.2.0",
-    "@membank/mcp": "0.4.0"
+    "@membank/dashboard": "0.2.2",
+    "@membank/mcp": "0.5.0",
+    "@membank/core": "0.5.0"
   },
   "devDependencies": {
     "@types/node": "^25.6.0",