nlm-memory 0.4.2 → 0.5.1

package/src/install/claude-code.ts

@@ -1,128 +0,0 @@
-/**
- * `nlm connect claude-code` / `nlm disconnect claude-code` — writes the
- * nlm-memory MCP server block into ~/.mcp.json and removes it on disconnect.
- *
- * ~/.mcp.json is the global MCP config file that Claude Code reads on
- * startup. We merge our entry into the existing mcpServers object rather
- * than replacing the file, so other MCP servers the user has configured are
- * preserved.
- */
-
-import { existsSync, mkdirSync, readFileSync, writeFileSync } from "node:fs";
-import { homedir } from "node:os";
-import { dirname, join } from "node:path";
-import type { ClaudeHookEvent } from "../core/hook/claude-settings.js";
-
-export interface ConnectClaudeCodeOptions {
-  readonly nlmBinPath: string;
-  readonly nodeExecPath: string;
-  readonly dryRun?: boolean;
-}
-
-export interface ConnectClaudeCodeReport {
-  readonly mcpConfigPath: string;
-  readonly alreadyPresent: boolean;
-  readonly written: boolean;
-  readonly dryRun: boolean;
-}
-
-export interface DisconnectClaudeCodeReport {
-  readonly mcpConfigPath: string;
-  readonly removed: boolean;
-  readonly dryRun: boolean;
-}
-
-export function mcpConfigPath(): string {
-  return process.env["NLM_MCP_CONFIG"] ?? join(homedir(), ".mcp.json");
-}
-
-function readConfig(path: string): Record<string, unknown> {
-  if (!existsSync(path)) return {};
-  try {
-    return JSON.parse(readFileSync(path, "utf8")) as Record<string, unknown>;
-  } catch {
-    throw new Error(`${path} is not valid JSON. Fix or remove it, then re-run \`nlm connect claude-code\`.`);
-  }
-}
-
-export function connectClaudeCode(opts: ConnectClaudeCodeOptions): ConnectClaudeCodeReport {
-  const configPath = mcpConfigPath();
-  const config = readConfig(configPath);
-  const mcpServers = (config["mcpServers"] ?? {}) as Record<string, unknown>;
-  const alreadyPresent = "nlm-memory" in mcpServers;
-
-  if (!opts.dryRun) {
-    mcpServers["nlm-memory"] = {
-      command: opts.nodeExecPath,
-      args: [opts.nlmBinPath, "mcp"],
-    };
-    config["mcpServers"] = mcpServers;
-    mkdirSync(dirname(configPath), { recursive: true });
-    writeFileSync(configPath, JSON.stringify(config, null, 2) + "\n", "utf8");
-  }
-
-  return { mcpConfigPath: configPath, alreadyPresent, written: !opts.dryRun, dryRun: opts.dryRun ?? false };
-}
-
-// ── Hook install helper (shared by setup wizard + connect --with-hooks) ──────
-
-export interface HookSpec {
-  readonly event: ClaudeHookEvent;
-  readonly script: string;
-  readonly label: string;
-}
-
-export interface HookInstallOptions {
-  readonly nodeExecPath: string;
-  readonly hooks: ReadonlyArray<HookSpec>;
-  readonly settingsPath: string;
-  readonly hookLogPath: string;
-  readonly addHook: (path: string, command: string, event?: ClaudeHookEvent) => void;
-  readonly removeHook: (path: string, event?: ClaudeHookEvent | "*") => void;
-  readonly buildHookCommand: (nodeExec: string, script: string, mode: "shadow" | "live") => string;
-  readonly smokeTestHookCommand: (command: string, logPath: string) => { ok: boolean; reason?: string; stderr?: string };
-}
-
-export interface HookInstallResult {
-  readonly ok: boolean;
-  readonly count: number;
-  readonly failedLabel?: string;
-  readonly errorMessage?: string;
-}
-
-export function installClaudeCodeHooks(opts: HookInstallOptions): HookInstallResult {
-  const installed: HookSpec[] = [];
-  for (const spec of opts.hooks) {
-    try {
-      const command = opts.buildHookCommand(opts.nodeExecPath, spec.script, "shadow");
-      opts.addHook(opts.settingsPath, command, spec.event);
-      const smoke = opts.smokeTestHookCommand(command, opts.hookLogPath);
-      if (!smoke.ok) {
-        for (const prior of [...installed, spec]) opts.removeHook(opts.settingsPath, prior.event);
-        const result: HookInstallResult = { ok: false, count: installed.length, failedLabel: spec.label };
-        return smoke.reason ? { ...result, errorMessage: smoke.reason } : result;
-      }
-      installed.push(spec);
-    } catch (e) {
-      return { ok: false, count: installed.length, failedLabel: spec.label, errorMessage: e instanceof Error ? e.message : String(e) };
-    }
-  }
-  return { ok: true, count: installed.length };
-}
-
-export function disconnectClaudeCode(opts?: { dryRun?: boolean }): DisconnectClaudeCodeReport {
-  const configPath = mcpConfigPath();
-  const config = readConfig(configPath);
-  const mcpServers = config["mcpServers"] as Record<string, unknown> | undefined;
-
-  if (!mcpServers || !("nlm-memory" in mcpServers)) {
-    return { mcpConfigPath: configPath, removed: false, dryRun: opts?.dryRun ?? false };
-  }
-
-  if (!opts?.dryRun) {
-    delete mcpServers["nlm-memory"];
-    writeFileSync(configPath, JSON.stringify(config, null, 2) + "\n", "utf8");
-  }
-
-  return { mcpConfigPath: configPath, removed: true, dryRun: opts?.dryRun ?? false };
-}

package/src/install/codex.ts

@@ -1,367 +0,0 @@
-/**
- * `nlm connect codex` / `nlm disconnect codex` — installs nlm-memory as a
- * Codex CLI plugin via the marketplace mechanism.
- *
- * Two distribution surfaces:
- *
- * 1. The plugin path (default). Registers a Codex marketplace pointing at
- *    pbmagnet4/nlm-memory-ts and installs the `nlm-memory` plugin from it.
- *    Codex prompts for hook trust on first invocation; once trusted,
- *    UserPromptSubmit + Stop hooks fire, and the .mcp.json wires the
- *    `nlm-memory` MCP server alongside.
- *
- * 2. The legacy hooks.json fallback (--with-hooks). For Codex Desktop
- *    builds where openai/codex#16430 blocks plugin-local hook dispatch,
- *    additionally writes absolute paths into ~/.codex/hooks.json so the
- *    hooks fire via the project-local code path. MCP still comes through
- *    the plugin's .mcp.json.
- *
- * Marketplace + plugin add are delegated to the `codex` binary rather than
- * mutating ~/.codex/config.toml directly — the binary owns the trust state
- * machine and the snapshot fetch flow, and writing TOML by hand would race
- * against codex's own writes. The legacy hooks.json IS authored directly
- * because it's a project-local file the binary doesn't manage.
- */
-
-import { spawnSync } from "node:child_process";
-import {
-  existsSync,
-  mkdirSync,
-  readFileSync,
-  writeFileSync,
-} from "node:fs";
-import { homedir } from "node:os";
-import { dirname, join, resolve } from "node:path";
-
-const DEFAULT_SOURCE = "pbmagnet4/nlm-memory-ts";
-const PLUGIN_NAME = "nlm-memory";
-// Marketplace name is derived from the source's basename by codex when
-// `codex plugin marketplace add <source>` runs without a --name flag. For
-// owner/repo this is the repo name; for a local path it's the directory
-// basename. Both resolve to "nlm-memory-ts" in our case.
-const MARKETPLACE_NAME = "nlm-memory-ts";
-
-// Marker substring identifying entries this CLI owns in ~/.codex/hooks.json
-// so disconnect can strip only our entries and leave anything the user
-// added by hand intact.
-const LEGACY_HOOK_MARKER = "/plugin/scripts/";
-
-// Sentinels bracketing the [mcp_servers.nlm-memory] block we manage in
-// ~/.codex/config.toml. Sentinel-bracketed regions are removed atomically
-// on disconnect and replaced atomically on connect — no TOML parser
-// required, no risk of mangling user-authored entries above or below.
-const MCP_SENTINEL_BEGIN = "# >>> nlm-memory (managed by nlm connect codex)";
-const MCP_SENTINEL_END = "# <<< nlm-memory";
-
-export interface ConnectOptions {
-  readonly source?: string;
-  readonly withHooks?: boolean;
-  readonly dryRun?: boolean;
-}
-
-export interface DisconnectOptions {
-  readonly withHooks?: boolean;
-  readonly dryRun?: boolean;
-}
-
-export interface CodexCommandResult {
-  readonly status: number | null;
-  readonly stdout: string;
-  readonly stderr: string;
-}
-
-function runCodex(args: ReadonlyArray<string>): CodexCommandResult {
-  const result = spawnSync("codex", args, { encoding: "utf8" });
-  return {
-    status: result.status,
-    stdout: result.stdout ?? "",
-    stderr: result.stderr ?? "",
-  };
-}
-
-export function codexBinaryAvailable(): boolean {
-  const r = spawnSync("codex", ["--version"], { encoding: "utf8" });
-  return r.status === 0;
-}
-
-export function codexHooksPath(): string {
-  return process.env["NLM_CODEX_HOOKS"] ?? join(homedir(), ".codex", "hooks.json");
-}
-
-export function codexConfigPath(): string {
-  return process.env["NLM_CODEX_CONFIG"] ?? join(homedir(), ".codex", "config.toml");
-}
-
-/**
- * Idempotently insert (or update) the [mcp_servers.nlm-memory] block in
- * ~/.codex/config.toml. The block is bracketed by sentinel comments so a
- * later disconnect can strip the exact region without touching anything
- * else. MCP wiring is universal infrastructure — every runtime gets its
- * MCP server registered in its native format. Codex's is TOML in
- * config.toml; we write that directly rather than relying on the plugin
- * system's .mcp.json indirection (which we can't currently verify works
- * outside the upstream plugin pipeline).
- */
-export function writeMcpServerToConfig(configPath: string): void {
-  mkdirSync(dirname(configPath), { recursive: true });
-  const existing = existsSync(configPath) ? readFileSync(configPath, "utf8") : "";
-
-  const block = `${MCP_SENTINEL_BEGIN}\n[mcp_servers.nlm-memory]\ncommand = "nlm"\nargs = ["mcp"]\n${MCP_SENTINEL_END}\n`;
-
-  const next = stripSentinelBlock(existing);
-  const sep = next.length > 0 && !next.endsWith("\n\n") ? (next.endsWith("\n") ? "\n" : "\n\n") : "";
-  writeFileSync(configPath, next + sep + block, "utf8");
-}
-
-export function removeMcpServerFromConfig(configPath: string): boolean {
-  if (!existsSync(configPath)) return false;
-  const existing = readFileSync(configPath, "utf8");
-  const next = stripSentinelBlock(existing);
-  if (next === existing) return false;
-  writeFileSync(configPath, next, "utf8");
-  return true;
-}
-
-/**
- * Remove our sentinel-bracketed region from a config.toml string. Tolerant
- * of an unterminated begin sentinel (treats it as a no-op rather than
- * eating the rest of the file) so a corrupted config never amplifies.
- */
-function stripSentinelBlock(content: string): string {
-  const beginIdx = content.indexOf(MCP_SENTINEL_BEGIN);
-  if (beginIdx < 0) return content;
-  const endMarker = MCP_SENTINEL_END;
-  const endIdx = content.indexOf(endMarker, beginIdx + MCP_SENTINEL_BEGIN.length);
-  if (endIdx < 0) return content; // unterminated — refuse to mutate
-  let cutEnd = endIdx + endMarker.length;
-  if (content[cutEnd] === "\n") cutEnd += 1;
-  let cutStart = beginIdx;
-  // Also eat the single leading newline that connected this block to the
-  // prior section, so repeated connect/disconnect cycles don't accrete blanks.
-  if (cutStart > 0 && content[cutStart - 1] === "\n") cutStart -= 1;
-  return content.slice(0, cutStart) + content.slice(cutEnd);
-}
-
-interface CodexHookEntry {
-  readonly type: string;
-  readonly command: string;
-  readonly statusMessage?: string;
-}
-
-interface CodexHookGroup {
-  readonly matcher?: string;
-  readonly hooks: CodexHookEntry[];
-}
-
-interface CodexHooksFile {
-  hooks?: Record<string, CodexHookGroup[]>;
-  [k: string]: unknown;
-}
-
-function readHooksFile(path: string): CodexHooksFile {
-  if (!existsSync(path)) return {};
-  try {
-    return JSON.parse(readFileSync(path, "utf8")) as CodexHooksFile;
-  } catch {
-    // Treat a malformed hooks.json as empty rather than silently
-    // overwriting the user's intent. The legacy writer below merges
-    // entries — if the file is broken we'd rather error than clobber.
-    throw new Error(`~/.codex/hooks.json is not valid JSON: ${path}`);
-  }
-}
-
-function writeHooksFile(path: string, content: CodexHooksFile): void {
-  mkdirSync(dirname(path), { recursive: true });
-  writeFileSync(path, JSON.stringify(content, null, 2) + "\n", "utf8");
-}
-
-/**
- * Append our two hook entries into ~/.codex/hooks.json without touching any
- * pre-existing entries. Idempotent: a second call replaces our entries
- * rather than duplicating them (matched by LEGACY_HOOK_MARKER substring).
- */
-export function writeLegacyHooks(
-  pluginScriptsDir: string,
-  hooksPath: string,
-): void {
-  const file = readHooksFile(hooksPath);
-  const hooks = (file.hooks ??= {});
-
-  const ourEntries: Record<string, CodexHookGroup> = {
-    UserPromptSubmit: {
-      hooks: [
-        {
-          type: "command",
-          command: `node "${join(pluginScriptsDir, "prompt-recall-hook.mjs")}"`,
-          statusMessage: "nlm-memory: recalling prior sessions",
-        },
-      ],
-    },
-    Stop: {
-      hooks: [
-        {
-          type: "command",
-          command: `node "${join(pluginScriptsDir, "stop-hook.mjs")}"`,
-        },
-      ],
-    },
-  };
-
-  for (const [event, ourGroup] of Object.entries(ourEntries)) {
-    const existing = hooks[event] ?? [];
-    const kept = existing.filter(
-      (group) =>
-        !group.hooks.some((h) => h.command.includes(LEGACY_HOOK_MARKER)),
-    );
-    kept.push(ourGroup);
-    hooks[event] = kept;
-  }
-
-  writeHooksFile(hooksPath, file);
-}
-
-export function removeLegacyHooks(hooksPath: string): boolean {
-  if (!existsSync(hooksPath)) return false;
-  const file = readHooksFile(hooksPath);
-  const hooks = file.hooks;
-  if (!hooks) return false;
-
-  let mutated = false;
-  for (const [event, groups] of Object.entries(hooks)) {
-    const kept = groups.filter(
-      (group) =>
-        !group.hooks.some((h) => h.command.includes(LEGACY_HOOK_MARKER)),
-    );
-    if (kept.length !== groups.length) mutated = true;
-    if (kept.length === 0) delete hooks[event];
-    else hooks[event] = kept;
-  }
-
-  if (mutated) writeHooksFile(hooksPath, file);
-  return mutated;
-}
-
-export interface ConnectReport {
-  readonly source: string;
-  readonly marketplaceName: string;
-  readonly pluginName: string;
-  readonly marketplaceAdd: CodexCommandResult | null;
-  readonly pluginAdd: CodexCommandResult | null;
-  readonly legacyHooksWritten: string | null;
-  readonly mcpServerWritten: string | null;
-  readonly dryRun: boolean;
-}
-
-export function connectCodex(
-  opts: ConnectOptions,
-  pluginScriptsDir: string,
-): ConnectReport {
-  const source = opts.source ?? DEFAULT_SOURCE;
-  const marketplaceName = MARKETPLACE_NAME;
-  const pluginName = PLUGIN_NAME;
-
-  if (opts.dryRun) {
-    return {
-      source,
-      marketplaceName,
-      pluginName,
-      marketplaceAdd: null,
-      pluginAdd: null,
-      legacyHooksWritten: opts.withHooks ? codexHooksPath() : null,
-      mcpServerWritten: codexConfigPath(),
-      dryRun: true,
-    };
-  }
-
-  // Marketplace add is idempotent at the codex layer; a re-add of the same
-  // source no-ops or refreshes the snapshot depending on the binary.
-  const marketplaceAdd = runCodex(["plugin", "marketplace", "add", source]);
-  // plugin add is the action that triggers the trust-prompt path on first
-  // run. We let codex's exit code propagate to the caller.
-  const pluginAdd = runCodex([
-    "plugin",
-    "add",
-    `${pluginName}@${marketplaceName}`,
-  ]);
-
-  let legacyHooksWritten: string | null = null;
-  if (opts.withHooks) {
-    const hooksPath = codexHooksPath();
-    writeLegacyHooks(pluginScriptsDir, hooksPath);
-    legacyHooksWritten = hooksPath;
-  }
-
-  // MCP wiring is always written directly to config.toml — it's the
-  // universal infrastructure that should work whether or not the plugin
-  // system honors the bundled .mcp.json indirection.
-  const configPath = codexConfigPath();
-  writeMcpServerToConfig(configPath);
-
-  return {
-    source,
-    marketplaceName,
-    pluginName,
-    marketplaceAdd,
-    pluginAdd,
-    legacyHooksWritten,
-    mcpServerWritten: configPath,
-    dryRun: false,
-  };
-}
-
-export interface DisconnectReport {
-  readonly marketplaceName: string;
-  readonly pluginName: string;
-  readonly pluginRemove: CodexCommandResult | null;
-  readonly marketplaceRemove: CodexCommandResult | null;
-  readonly legacyHooksRemoved: boolean;
-  readonly mcpServerRemoved: boolean;
-  readonly dryRun: boolean;
-}
-
-export function disconnectCodex(opts: DisconnectOptions): DisconnectReport {
-  if (opts.dryRun) {
-    return {
-      marketplaceName: MARKETPLACE_NAME,
-      pluginName: PLUGIN_NAME,
-      pluginRemove: null,
-      marketplaceRemove: null,
-      legacyHooksRemoved: opts.withHooks ?? false,
-      mcpServerRemoved: true,
-      dryRun: true,
-    };
-  }
-
-  const pluginRemove = runCodex([
-    "plugin",
-    "remove",
-    `${PLUGIN_NAME}@${MARKETPLACE_NAME}`,
-  ]);
-  const marketplaceRemove = runCodex([
-    "plugin",
-    "marketplace",
-    "remove",
-    MARKETPLACE_NAME,
-  ]);
-
-  let legacyHooksRemoved = false;
-  if (opts.withHooks) {
-    legacyHooksRemoved = removeLegacyHooks(codexHooksPath());
-  }
-
-  const mcpServerRemoved = removeMcpServerFromConfig(codexConfigPath());
-
-  return {
-    marketplaceName: MARKETPLACE_NAME,
-    pluginName: PLUGIN_NAME,
-    pluginRemove,
-    marketplaceRemove,
-    legacyHooksRemoved,
-    mcpServerRemoved,
-    dryRun: false,
-  };
-}
-
-export function pluginScriptsDir(repoRoot: string): string {
-  return resolve(repoRoot, "plugin", "scripts");
-}

package/src/install/hermes-agent.ts

@@ -1,76 +0,0 @@
-/**
- * `nlm connect hermes-agent` / `nlm disconnect hermes-agent` — installs the
- * nlm-memory plugin into NousResearch Hermes Agent's plugin directory and
- * optionally enables it via the `hermes` binary.
- *
- * The plugin lives in plugin-hermes-agent/ at the repo root. `connect`
- * copies it to ~/.hermes/plugins/nlm-memory/ (flat layout, one category
- * level max per Hermes plugin discovery rules). `disconnect` removes that
- * directory.
- *
- * MCP server wiring (the [mcp_servers.nlm-memory] block in
- * ~/.hermes/config.yaml) is handled separately by `nlm connect hermes`.
- */
-
-import { cpSync, existsSync, mkdirSync, rmSync } from "node:fs";
-import { homedir } from "node:os";
-import { dirname, join } from "node:path";
-import { spawnSync } from "node:child_process";
-
-export interface ConnectHermesAgentOptions {
-  readonly pluginSrcDir: string;
-  readonly dryRun?: boolean;
-  readonly enableViaCliIfAvailable?: boolean;
-}
-
-export interface ConnectHermesAgentReport {
-  readonly destDir: string;
-  readonly copied: boolean;
-  readonly alreadyPresent: boolean;
-  readonly enabledViaCli: boolean;
-  readonly dryRun: boolean;
-}
-
-export interface DisconnectHermesAgentReport {
-  readonly destDir: string;
-  readonly removed: boolean;
-  readonly dryRun: boolean;
-}
-
-export function hermesAgentPluginDir(): string {
-  return process.env["NLM_HERMES_PLUGIN_DIR"] ?? join(homedir(), ".hermes", "plugins", "nlm-memory");
-}
-
-export function connectHermesAgent(opts: ConnectHermesAgentOptions): ConnectHermesAgentReport {
-  const destDir = hermesAgentPluginDir();
-  const alreadyPresent = existsSync(destDir);
-
-  if (!opts.dryRun) {
-    mkdirSync(dirname(destDir), { recursive: true });
-    cpSync(opts.pluginSrcDir, destDir, { recursive: true });
-
-    let enabledViaCli = false;
-    if (opts.enableViaCliIfAvailable !== false) {
-      const result = spawnSync("hermes", ["plugins", "enable", "nlm-memory"], {
-        encoding: "utf8",
-        timeout: 10_000,
-      });
-      enabledViaCli = result.status === 0;
-    }
-    return { destDir, copied: true, alreadyPresent, enabledViaCli, dryRun: false };
-  }
-
-  return { destDir, copied: false, alreadyPresent, enabledViaCli: false, dryRun: true };
-}
-
-export function disconnectHermesAgent(opts?: { dryRun?: boolean }): DisconnectHermesAgentReport {
-  const destDir = hermesAgentPluginDir();
-  const present = existsSync(destDir);
-
-  if (!opts?.dryRun && present) {
-    rmSync(destDir, { recursive: true, force: true });
-    return { destDir, removed: true, dryRun: false };
-  }
-
-  return { destDir, removed: false, dryRun: opts?.dryRun ?? false };
-}

package/src/install/hermes.ts

@@ -1,78 +0,0 @@
-/**
- * `nlm connect hermes` / `nlm disconnect hermes` — writes the nlm-memory
- * MCP server entry into ~/.hermes/config.yaml.
- *
- * Uses yaml's Document API (parseDocument / doc.setIn / doc.toString) to
- * preserve any comments the user has written in their config file. Round-
- * tripping through parse+stringify would silently destroy comments.
- */
-
-import { existsSync, mkdirSync, readFileSync, writeFileSync } from "node:fs";
-import { homedir } from "node:os";
-import { dirname, join } from "node:path";
-import { Document as YamlDocument, parseDocument as parseYamlDocument } from "yaml";
-
-export interface ConnectHermesOptions {
-  readonly nlmBinPath: string;
-  readonly nodeExecPath: string;
-  readonly dryRun?: boolean;
-}
-
-export interface ConnectHermesReport {
-  readonly configPath: string;
-  readonly alreadyPresent: boolean;
-  readonly written: boolean;
-  readonly dryRun: boolean;
-}
-
-export interface DisconnectHermesReport {
-  readonly configPath: string;
-  readonly removed: boolean;
-  readonly dryRun: boolean;
-}
-
-export function hermesConfigPath(): string {
-  return process.env["NLM_HERMES_CONFIG"] ?? join(homedir(), ".hermes", "config.yaml");
-}
-
-function readDocument(path: string): YamlDocument {
-  if (!existsSync(path)) return new YamlDocument();
-  try {
-    return parseYamlDocument(readFileSync(path, "utf8"));
-  } catch {
-    throw new Error(`${path} is not valid YAML. Fix or remove it, then re-run \`nlm connect hermes\`.`);
-  }
-}
-
-export function connectHermes(opts: ConnectHermesOptions): ConnectHermesReport {
-  const configPath = hermesConfigPath();
-  const doc = readDocument(configPath);
-  const alreadyPresent = doc.getIn(["mcp_servers", "nlm-memory"]) !== undefined;
-
-  if (!opts.dryRun) {
-    doc.setIn(["mcp_servers", "nlm-memory"], {
-      command: opts.nodeExecPath,
-      args: [opts.nlmBinPath, "mcp"],
-    });
-    mkdirSync(dirname(configPath), { recursive: true });
-    writeFileSync(configPath, doc.toString(), "utf8");
-  }
-
-  return { configPath, alreadyPresent, written: !opts.dryRun, dryRun: opts.dryRun ?? false };
-}
-
-export function disconnectHermes(opts?: { dryRun?: boolean }): DisconnectHermesReport {
-  const configPath = hermesConfigPath();
-  const doc = readDocument(configPath);
-
-  if (doc.getIn(["mcp_servers", "nlm-memory"]) === undefined) {
-    return { configPath, removed: false, dryRun: opts?.dryRun ?? false };
-  }
-
-  if (!opts?.dryRun) {
-    doc.deleteIn(["mcp_servers", "nlm-memory"]);
-    writeFileSync(configPath, doc.toString(), "utf8");
-  }
-
-  return { configPath, removed: true, dryRun: opts?.dryRun ?? false };
-}