@membank/cli 0.0.1 → 0.0.3

package/dist/index.mjs

@@ -7,6 +7,8 @@ import { dirname, join } from "node:path";
 import * as readline from "node:readline";
 import { createInterface } from "node:readline";
 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";
 //#region src/commands/add.ts
@@ -80,12 +82,12 @@ const MEMORY_TYPES = new Set([
 function isValidRecord(r) {
 	if (typeof r !== "object" || r === null) return false;
 	const rec = r;
-	return typeof rec["id"] === "string" && rec["id"].length > 0 && typeof rec["content"] === "string" && typeof rec["type"] === "string" && MEMORY_TYPES.has(rec["type"]) && typeof rec["scope"] === "string" && rec["scope"].length > 0;
+	return typeof rec.id === "string" && rec.id.length > 0 && typeof rec.content === "string" && typeof rec.type === "string" && MEMORY_TYPES.has(rec.type) && typeof rec.scope === "string" && rec.scope.length > 0;
 }
 function isExportFile(parsed) {
 	if (typeof parsed !== "object" || parsed === null) return false;
 	const obj = parsed;
-	return obj["version"] === 1 && Array.isArray(obj["memories"]);
+	return obj.version === 1 && Array.isArray(obj.memories);
 }
 async function importCommand(filePath, db, formatter, prompt) {
 	let raw;
@@ -314,19 +316,50 @@ var PromptHelper = class {
 	}
 };
 //#endregion
+//#region src/utils/execFileNoThrow.ts
+const execFileAsync = promisify(execFile);
+function resolveCmd(cmd) {
+	return process.platform === "win32" ? `${cmd}.cmd` : cmd;
+}
+async function execFileNoThrow(cmd, args) {
+	try {
+		const { stdout, stderr } = await execFileAsync(resolveCmd(cmd), args, { encoding: "utf8" });
+		return {
+			stdout: stdout ?? "",
+			stderr: stderr ?? "",
+			exitCode: 0
+		};
+	} catch (err) {
+		if (err !== null && typeof err === "object" && "code" in err) {
+			const e = err;
+			if (e.code === "ENOENT") return {
+				stdout: "",
+				stderr: `Command not found: ${cmd}`,
+				exitCode: 127
+			};
+			return {
+				stdout: typeof e.stdout === "string" ? e.stdout : "",
+				stderr: typeof e.stderr === "string" ? e.stderr : "",
+				exitCode: typeof e.code === "number" ? e.code : 1
+			};
+		}
+		return {
+			stdout: "",
+			stderr: err instanceof Error ? err.message : String(err),
+			exitCode: 1
+		};
+	}
+}
+//#endregion
 //#region src/setup/harness-config-writer.ts
 const defaultPathResolver = {
 	home: () => {
-		const h = process.env["HOME"] ?? process.env["USERPROFILE"];
+		const h = process.env.HOME ?? process.env.USERPROFILE;
 		if (!h) throw new Error("Cannot determine home directory");
 		return h;
 	},
 	cwd: () => process.cwd()
 };
-const MEMBANK_ENTRY = {
-	command: "npx",
-	args: ["@membank/cli", "--mcp"]
-};
 function readJson(path) {
 	try {
 		return JSON.parse(readFileSync(path, "utf8"));
@@ -343,66 +376,133 @@ function writeJsonAtomic(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`);
+}
+const MEMBANK_NPX_ARGS = [
+	"npx",
+	"@membank/cli@latest",
+	"--mcp"
+];
 const writers = {
-	"claude-code": {
-		configPath: (r) => join(r.home(), ".claude", "settings.json"),
-		isConfigured: (cfg) => hasKey(cfg["mcpServers"], "membank"),
-		merge: (cfg) => ({
-			...cfg,
-			mcpServers: {
-				...cfg["mcpServers"],
-				membank: MEMBANK_ENTRY
-			}
-		})
-	},
-	vscode: {
-		configPath: (r) => join(r.cwd(), ".vscode", "mcp.json"),
-		isConfigured: (cfg) => hasKey(cfg["servers"], "membank"),
-		merge: (cfg) => ({
-			...cfg,
-			servers: {
-				...cfg["servers"],
-				membank: MEMBANK_ENTRY
-			}
-		})
-	},
-	codex: {
-		configPath: (r) => join(r.home(), ".codex", "config.json"),
-		isConfigured: (cfg) => hasKey(cfg["mcpServers"], "membank"),
-		merge: (cfg) => ({
-			...cfg,
-			mcpServers: {
-				...cfg["mcpServers"],
-				membank: MEMBANK_ENTRY
-			}
-		})
-	},
-	opencode: {
-		configPath: (r) => join(r.home(), ".config", "opencode", "config.json"),
-		isConfigured: (cfg) => hasKey(cfg["mcp"], "membank"),
-		merge: (cfg) => ({
+	"claude-code": { async write(resolver, run, { overwrite = false } = {}) {
+		const configured = hasKey(readJson(join(resolver.home(), ".claude.json")).mcpServers, "membank");
+		if (configured && !overwrite) return { status: "already-configured" };
+		if (configured) {
+			const remove = await run("claude", [
+				"mcp",
+				"remove",
+				"--scope",
+				"user",
+				"membank"
+			]);
+			assertCliFound(remove, "claude");
+			if (remove.exitCode !== 0) throw new Error(`claude mcp remove failed: ${remove.stderr}`);
+		}
+		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" };
+	} },
+	vscode: { async write(resolver, run, { overwrite = false } = {}) {
+		const cfgPath = join(resolver.cwd(), ".vscode", "mcp.json");
+		const cfg = readJson(cfgPath);
+		const configured = hasKey(cfg.servers, "membank");
+		if (configured && !overwrite) return { status: "already-configured" };
+		if (configured) {
+			writeJsonAtomic(cfgPath, {
+				...cfg,
+				servers: {
+					...cfg.servers,
+					membank: {
+						command: "npx",
+						args: ["@membank/cli@latest", "--mcp"]
+					}
+				}
+			});
+			return { status: "written" };
+		}
+		const payload = JSON.stringify({
+			name: "membank",
+			command: "npx",
+			args: ["@membank/cli@latest", "--mcp"]
+		});
+		const result = await run("code", [
+			"--folder-uri",
+			resolver.cwd(),
+			"--add-mcp",
+			payload
+		]);
+		assertCliFound(result, "code");
+		if (result.exitCode !== 0) throw new Error(`code --add-mcp failed: ${result.stderr || result.stdout}`);
+		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", [
+				"mcp",
+				"remove",
+				"membank"
+			]);
+			assertCliFound(remove, "codex");
+			if (remove.exitCode !== 0) throw new Error(`codex mcp remove failed: ${remove.stderr}`);
+		}
+		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(cfgPath);
+		if (hasKey(cfg.mcp, "membank") && !overwrite) return { status: "already-configured" };
+		writeJsonAtomic(cfgPath, {
 			...cfg,
 			mcp: {
-				...cfg["mcp"],
-				membank: MEMBANK_ENTRY
+				...cfg.mcp,
+				membank: {
+					type: "local",
+					command: [
+						"npx",
+						"@membank/cli@latest",
+						"--mcp"
+					]
+				}
 			}
-		})
-	}
+		});
+		return { status: "written" };
+	} }
 };
 const SUPPORTED_HARNESSES = Object.keys(writers);
 var HarnessConfigWriter = class {
 	#resolver;
-	constructor(resolver = defaultPathResolver) {
+	#run;
+	constructor(resolver = defaultPathResolver, run = execFileNoThrow) {
 		this.#resolver = resolver;
+		this.#run = run;
 	}
-	write(harness, { overwrite = false } = {}) {
+	async write(harness, { overwrite = false } = {}) {
 		const writer = writers[harness];
 		if (!writer) throw new Error(`Unknown harness: ${harness}`);
-		const path = writer.configPath(this.#resolver);
-		const existing = readJson(path);
-		if (!overwrite && writer.isConfigured(existing)) return { status: "already-configured" };
-		writeJsonAtomic(path, writer.merge(existing));
-		return { status: "written" };
+		return writer.write(this.#resolver, this.#run, { overwrite });
 	}
 };
 //#endregion
@@ -481,7 +581,8 @@ function harnessConfigs(resolver) {
 	return [
 		{
 			name: "claude-code",
-			configPath: join(home, ".claude", "settings.json")
+			configPath: join(home, ".claude.json"),
+			fallbackPaths: [join(home, ".claude", "settings.json")]
 		},
 		{
 			name: "vscode",
@@ -489,16 +590,18 @@ function harnessConfigs(resolver) {
 		},
 		{
 			name: "codex",
-			configPath: join(home, ".codex", "config.json")
+			configPath: join(home, ".codex", "config.toml"),
+			fallbackPaths: [join(home, ".codex", "config.json")]
 		},
 		{
 			name: "opencode",
-			configPath: join(home, ".config", "opencode", "config.json")
+			configPath: join(home, ".config", "opencode", "opencode.json"),
+			fallbackPaths: [join(home, ".config", "opencode", "config.json")]
 		}
 	];
 }
 function detectHarnesses(resolver = defaultResolver) {
-	return harnessConfigs(resolver).filter((h) => existsSync(h.configPath)).map((h) => ({
+	return harnessConfigs(resolver).filter((h) => existsSync(h.configPath) || (h.fallbackPaths?.some((p) => existsSync(p)) ?? false)).map((h) => ({
 		name: h.name,
 		configPath: h.configPath
 	}));
@@ -582,7 +685,7 @@ var SetupOrchestrator = class {
 		for (const h of detected) {
 			let writeResult;
 			try {
-				writeResult = this.#writer.write(h.name);
+				writeResult = await this.#writer.write(h.name);
 			} catch (err) {
 				const msg = err instanceof Error ? err.message : String(err);
 				out(`  ✗ ${h.name}: ${msg}`);
@@ -605,7 +708,7 @@ var SetupOrchestrator = class {
 					continue;
 				}
 				try {
-					this.#writer.write(h.name, { overwrite: true });
+					await this.#writer.write(h.name, { overwrite: true });
 					out(`  ✓ ${h.name}: written (overwritten)`);
 					results.push({
 						harness: h.name,
@@ -812,6 +915,6 @@ program.on("command:*", () => {
 	program.outputHelp();
 	process.exit(1);
 });
-program.parse(process.argv);
+if (!process.argv.includes("--mcp")) program.parse(process.argv);
 //#endregion
 export {};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@membank/cli",
-  "version": "0.0.1",
+  "version": "0.0.3",
   "type": "module",
   "repository": {
     "type": "git",
@@ -17,8 +17,8 @@
     "@huggingface/transformers": "^4.2.0",
     "commander": "^14.0.3",
     "ora": "^9.4.0",
-    "@membank/core": "0.0.1",
-    "@membank/mcp": "0.0.1"
+    "@membank/core": "0.0.3",
+    "@membank/mcp": "0.0.3"
   },
   "devDependencies": {
     "@types/node": "^25.6.0",