slidemind 0.1.6 → 0.1.8

package/src/installers/codex.mjs

@@ -85,36 +85,10 @@ function configTomlFile() {
   return path.join(codexRoot(), "config.toml");
 }
 
-// Register the slidemind MCP server. Prefer Codex's own `codex mcp add`, which
-// is TOML-aware and idempotent: it sets `[mcp_servers.slidemind]` by key, so a
-// re-install/upgrade — or a pre-existing entry added manually or by an earlier
-// `codex mcp add` — is REPLACED in place rather than duplicated, while unrelated
-// tables, top-level keys, and comments survive.
-//
-// The previous hand-rolled writer wrapped its block in `# slidemind:begin/end`
-// comment markers and deduped by them; a `[mcp_servers.slidemind]` table that
-// lacked those markers (manual edit, or a Codex-side config rewrite that dropped
-// the comments) was invisible to it, so it appended a SECOND table — a
-// duplicate-key TOML that broke config.toml. Delegating to Codex removes that
-// whole failure mode.
-//
-// `codex mcp` landed around Codex 0.128. If the binary is missing or too old
-// (spawn error / non-zero exit) we fall back to a minimal, non-destructive
-// writer that never duplicates.
-export function registerMcpServer(packageRoot) {
-  const mcpServer = path.join(packageRoot, "src", "mcp", "server.mjs");
-  const viaCli = spawnSync(
-    "codex",
-    ["mcp", "add", NAMESPACE, "--", "node", mcpServer],
-    { stdio: "ignore", shell: process.platform === "win32" },
-  );
-  if (!viaCli.error && viaCli.status === 0) {
-    sweepLegacyMarkers();
-    return "registered via codex mcp";
-  }
-  return fallbackWriteMcpServerBlock(mcpServer);
-}
-
+// slidemind is a CLI steered by the skill — there is no MCP server. We only ever
+// tear down a `[mcp_servers.slidemind]` table an older install registered.
+// Prefer Codex's own `codex mcp remove` (TOML-aware); fall back to a marker strip
+// when the binary is missing or too old.
 export function unregisterMcpServer() {
   const viaCli = spawnSync(
     "codex",
@@ -125,30 +99,6 @@ export function unregisterMcpServer() {
   return fallbackClearMcpServerBlock();
 }
 
-function mcpServerBlock(mcpServer) {
-  return (
-    `[mcp_servers.${NAMESPACE}]\n` +
-    `command = "node"\n` +
-    `args = ["${mcpServer.replace(/\\/g, "\\\\").replace(/"/g, '\\"')}"]\n`
-  );
-}
-
-// Fallback for when `codex mcp` is unavailable. Minimal and non-destructive:
-// never append a second entry — if a `[mcp_servers.slidemind]` already exists
-// (ours or the user's) we leave it untouched; otherwise we append our block.
-export function fallbackWriteMcpServerBlock(mcpServer) {
-  const file = configTomlFile();
-  const existing = fs.existsSync(file) ? fs.readFileSync(file, "utf8") : "";
-  if (existing.includes(`[mcp_servers.${NAMESPACE}]`)) {
-    return "already registered (left as-is)";
-  }
-  const sep =
-    existing.length && !existing.endsWith("\n") ? "\n\n" : existing.length ? "\n" : "";
-  fs.mkdirSync(path.dirname(file), { recursive: true });
-  fs.writeFileSync(file, existing + sep + mcpServerBlock(mcpServer));
-  return "registered (fallback)";
-}
-
 // Fallback uninstall: strip a legacy marker-wrapped block if one is present;
 // otherwise point the user at `codex mcp remove` (we won't guess at the bounds
 // of a marker-less table without Codex's TOML parser).
@@ -165,24 +115,6 @@ export function fallbackClearMcpServerBlock() {
   return "run `codex mcp remove slidemind` to remove the entry";
 }
 
-// After Codex rewrites the `[mcp_servers.slidemind]` table, an upgrader's old
-// slidemind block can leave orphan `# slidemind:begin` / `# slidemind:end`
-// comment lines behind. Drop just those literal lines (valid TOML either way —
-// purely cosmetic).
-function sweepLegacyMarkers() {
-  const file = configTomlFile();
-  if (!fs.existsSync(file)) return;
-  const before = fs.readFileSync(file, "utf8");
-  const after = before
-    .split("\n")
-    .filter(
-      (line) =>
-        line.trim() !== TOML_MARK_BEGIN && line.trim() !== TOML_MARK_END,
-    )
-    .join("\n");
-  if (after !== before) fs.writeFileSync(file, after);
-}
-
 function writeAgentsBlock() {
   const block =
     `${AGENTS_MARK_BEGIN}\n` +
@@ -260,12 +192,17 @@ export async function install({ packageRoot, flags }) {
   writeAgentsBlock();
   ui.step({ status: "ok", label: "AGENTS.md", value: agentsFile() });
 
-  // Register the slidemind MCP server so agents can list/push into open
-  // PowerPoint windows via slidemind_list_open_presentations / slidemind_push_deck.
-  const mcpNote = registerMcpServer(packageRoot);
-  ui.step({ status: "ok", label: "mcp server", value: mcpNote });
+  // slidemind is a CLI now — clean up any [mcp_servers.slidemind] table an older
+  // install registered so re-running init converges to the CLI-only setup.
+  if (
+    fs.existsSync(configTomlFile()) &&
+    fs.readFileSync(configTomlFile(), "utf8").includes(`[mcp_servers.${NAMESPACE}]`)
+  ) {
+    const mcpNote = unregisterMcpServer();
+    ui.step({ status: "remove", label: "mcp server", value: mcpNote });
+  }
 
-  ui.note("restart Codex to pick up the skill + MCP server");
+  ui.note("restart Codex to pick up the skill");
 }
 
 export async function uninstall() {

package/src/installers/opencode.mjs

@@ -71,28 +71,20 @@ function saveOpencodeConfig(cfg) {
   fs.writeFileSync(file, JSON.stringify(cfg, null, 2) + "\n");
 }
 
+// slidemind is a CLI steered by the skill — there is no MCP server. This only
+// tears down any `mcp.slidemind` entry an older install registered, so re-running
+// `init` converges to the CLI-only setup. Returns whether it removed one.
 function unregisterMcpServer() {
   const file = opencodeConfigFile();
-  if (!fs.existsSync(file)) return;
+  if (!fs.existsSync(file)) return false;
   const cfg = loadOpencodeConfig();
   if (cfg.mcp && cfg.mcp[NAMESPACE]) {
     delete cfg.mcp[NAMESPACE];
     if (Object.keys(cfg.mcp).length === 0) delete cfg.mcp;
     saveOpencodeConfig(cfg);
+    return true;
   }
-}
-
-function registerMcpServer(packageRoot) {
-  const mcpServer = path.join(packageRoot, "src", "mcp", "server.mjs");
-  const cfg = loadOpencodeConfig();
-  if (!cfg["$schema"]) cfg["$schema"] = "https://opencode.ai/config.json";
-  cfg.mcp = cfg.mcp || {};
-  cfg.mcp[NAMESPACE] = {
-    type: "local",
-    command: ["node", mcpServer],
-    enabled: true,
-  };
-  saveOpencodeConfig(cfg);
+  return false;
 }
 
 function removeLegacyArtifacts() {
@@ -153,10 +145,10 @@ export async function install({ packageRoot, flags }) {
     );
   }
 
-  // Register the slidemind MCP server so agents can list/push into open
-  // PowerPoint windows via slidemind_list_open_presentations / slidemind_push_deck.
-  registerMcpServer(packageRoot);
-  ui.step({ status: "ok", label: "mcp server", value: opencodeConfigFile() });
+  // slidemind is a CLI now — remove any MCP registration an older install left.
+  if (unregisterMcpServer()) {
+    ui.step({ status: "remove", label: "mcp server", value: "removed legacy registration (slidemind is a CLI)" });
+  }
 }
 
 export async function uninstall() {

package/src/lib/reused-slides.mjs

@@ -1,9 +1,7 @@
 // Shared numeric-prefix + filename logic for folding reused slide-search
 // results into a deck as native one-slide .pptx (the "reuse → fold into a deck"
-// path). Both `slidemind add-slide` (bin/slidemind.mjs) and the
-// `slidemind_add_slide_to_deck` MCP tool (src/mcp/server.mjs) drive the SAME
-// math, so it lives here as one pure, tested unit — the callers keep the bridge
-// round-trip and the fs write.
+// path behind `slidemind add-slide`, bin/slidemind.mjs). Kept here as one pure,
+// tested unit — the caller keeps the bridge round-trip and the fs write.
 //
 // Play order over the CLI is filename order (see src/lib/slide-order.mjs): a
 // reused slide is named `NN-reused-<shortid>.pptx`, zero-padded to width 2, so