@aistoragedepot/mcp 0.3.0 → 0.4.0

package/README.md

@@ -52,19 +52,27 @@ Your stored content shows up where you actually work:
 ## Slash-commands (`pull`)
 
 Not every client turns MCP prompts into `/` slash-commands (some only use the tools). To get
-your prompts as **native** slash-commands that always work, sync them to command files:
+your library as **native** slash-commands that always work, sync it to command files:
 
 ```bash
 npx -y @aistoragedepot/mcp pull --token=aisd_your_token
 ```
 
 This writes `~/.claude/commands/aistoragedepot/*.md` — start a new chat and type
-**`/aistoragedepot`** to see them. Re-run any time; it only clears and rewrites its own
-namespaced folder.
+**`/aistoragedepot`** to see them. **Skills** run as-is. **Prompts** take input: their
+`[fields]` show as an argument hint, and whatever you type after the command fills them —
+
+```
+/aistoragedepot:cold-outreach Jane at Globex, about our payments API
+```
+
+— with Claude asking for any value you left out. Re-run any time; it only clears and
+rewrites its own namespaced folder.
 
 | Option | Default | |
 | --- | --- | --- |
 | `--token=…` | `AISD_TOKEN` env | Your API token. |
+| `--types=…` | `skill,prompt` | Which item types become commands (e.g. `skill` for skills only). |
 | `--workspaces="A,B"` | your personal library | Which workspaces to pull (names, or `all`). |
 | `--project` | off | Write to `./.claude/commands/…` (this repo) instead of your home dir. |
 | `--dir=<path>` | `~/.claude/commands/aistoragedepot` | Write somewhere else entirely. |

package/index.mjs

@@ -23,11 +23,21 @@ import {
 
 const BASE_URL = (process.env.AISD_BASE_URL || "https://www.aistoragedepot.com").replace(/\/+$/, "");
 
-// CLI mode: `aistoragedepot-mcp pull` syncs prompts to native slash-command files, then exits.
+// CLI mode: `aistoragedepot-mcp pull` syncs the library to native slash-command files, then
+// exits. The brief settle delay lets undici finish closing its sockets (Connection: close)
+// before process.exit — a hard exit mid-teardown trips a libuv assert on Windows (Node 24)
+// that prints a scary-but-harmless message after the sync succeeds.
 if (process.argv[2] === "pull") {
   const { pull } = await import("./pull.mjs");
-  await pull({ BASE_URL, argv: process.argv.slice(3) });
-  process.exit(0);
+  let code = 0;
+  try {
+    await pull({ BASE_URL, argv: process.argv.slice(3) });
+  } catch (err) {
+    console.error(err?.message || err);
+    code = 1;
+  }
+  await new Promise((r) => setTimeout(r, 50));
+  process.exit(code);
 }
 
 const TOKEN = process.env.AISD_TOKEN;
@@ -131,7 +141,7 @@ function fillBody(body, placeholders, args) {
 }
 
 const server = new Server(
-  { name: "aistoragedepot", version: "0.3.0" },
+  { name: "aistoragedepot", version: "0.4.0" },
   { capabilities: { resources: {}, prompts: {}, tools: {} } },
 );
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aistoragedepot/mcp",
-  "version": "0.3.0",
+  "version": "0.4.0",
   "description": "Model Context Protocol server for your AIStorageDepot library — exposes your prompts, rules, docs, and skills to MCP-aware AI clients (Claude, Cursor, Cline, …).",
   "type": "module",
   "bin": {

package/pull.mjs

@@ -2,9 +2,11 @@
 //
 // MCP prompts-as-slash-commands are supported inconsistently across clients, so this writes
 // real command files that any Claude Code client picks up: ~/.claude/commands/aistoragedepot/*.md
-// (a /aistoragedepot:<name> command each). Defaults to the **skills** in your personal library
-// (self-contained instructions that run cleanly as commands); widen with --workspaces / include
-// prompts with --types. Re-run any time — it clears and rewrites only its own namespaced folder.
+// (a /aistoragedepot:<name> command each). Defaults to the **skills and prompts** in your
+// personal library. Skills run as-is; prompts become commands that take input — their [fields]
+// are declared as an argument-hint and filled from whatever you type after the command
+// ($ARGUMENTS), with Claude asking for anything missing. Narrow with --types / widen with
+// --workspaces. Re-run any time — it clears and rewrites only its own namespaced folder.
 import { mkdir, writeFile, rm } from "node:fs/promises";
 import { homedir } from "node:os";
 import { join, resolve } from "node:path";
@@ -12,6 +14,30 @@ import { join, resolve } from "node:path";
 const slug = (s) =>
   String(s).toLowerCase().replace(/[^a-z0-9]+/g, "-").replace(/^-+|-+$/g, "").slice(0, 60) || "item";
 const typeOf = (it) => (it.type?.slug || it.type?.name || "").toLowerCase();
+const isPromptItem = (it) => it.type?.format === "prompt" || typeOf(it) === "prompt";
+
+// Prompts become commands that take input: the [fields] in the body are declared as an
+// argument-hint (shown in the / picker) and filled from whatever the user types after the
+// command ($ARGUMENTS); Claude asks for any value that's missing.
+const FIELD_RE = /\[([A-Za-z0-9 ]+)\]/g;
+const yq = (s) => String(s).replace(/"/g, "'").replace(/\s+/g, " ").trim(); // YAML-safe one-liner
+
+function promptCommand(it) {
+  const body = String(it.body || "").trim();
+  const fields = [...new Set([...body.matchAll(FIELD_RE)].map((m) => m[1]))];
+  const head = ["---", `description: "${yq(`Prompt — ${it.title} (AIStorageDepot)`)}"`];
+  if (fields.length) head.push(`argument-hint: "${yq(fields.map((f) => `[${f}]`).join(" "))}"`);
+  head.push("---");
+  const tail = fields.length
+    ? [
+        "",
+        "Input: $ARGUMENTS",
+        "",
+        `Fill the bracketed ${fields.length === 1 ? "field" : "fields"} in the prompt above from the input — ask me for any value that's missing — then carry out the prompt.`,
+      ]
+    : ["", "$ARGUMENTS"]; // no fields: whatever the user types just flows in after the prompt
+  return [...head, "", body, ...tail, ""].join("\n");
+}
 
 function parseArgs(argv) {
   const o = {};
@@ -32,18 +58,22 @@ export async function pull({ BASE_URL, argv }) {
     process.exit(1);
   }
   const api = async (p) => {
-    const r = await fetch(BASE_URL + p, { headers: { Authorization: `Bearer ${token}`, Accept: "application/json" } });
+    // Connection: close — keep-alive sockets left open at exit trip a libuv assert on
+    // Windows (Node 24), printing a scary (but harmless) message after the sync finishes.
+    const r = await fetch(BASE_URL + p, {
+      headers: { Authorization: `Bearer ${token}`, Accept: "application/json", Connection: "close" },
+    });
     if (r.status === 401) throw new Error("Token rejected (401) — check it in Settings → API tokens.");
     if (!r.ok) throw new Error(`${p} → HTTP ${r.status}`);
     return r.json();
   };
 
-  // Which item types to pull. Default: skills only. --types=skill,prompt | all.
-  const TYPES = String(o.types || "skill").toLowerCase().split(",").map((s) => s.trim()).filter(Boolean);
+  // Which item types to pull. Default: skills + prompts. --types=skill | prompt | all.
+  const TYPES = String(o.types || "skill,prompt").toLowerCase().split(",").map((s) => s.trim()).filter(Boolean);
   const wantType = (it) => {
     const t = typeOf(it);
     const isSkill = t === "skill";
-    const isPrompt = it.type?.format === "prompt" || t === "prompt";
+    const isPrompt = isPromptItem(it);
     if (TYPES.includes("all")) return isSkill || isPrompt;
     return (TYPES.includes("skill") && isSkill) || (TYPES.includes("prompt") && isPrompt) || TYPES.includes(t);
   };
@@ -72,19 +102,29 @@ export async function pull({ BASE_URL, argv }) {
   const used = new Set();
   let n = 0;
   for (const it of items) {
+    // Clean names by default; on a title collision, try the type as a tiebreaker
+    // (code-review vs code-review-prompt) before falling back to an id suffix.
     let name = slug(it.title);
+    if (used.has(name)) {
+      const alt = slug(`${it.title} ${typeOf(it) || "item"}`);
+      name = used.has(alt) ? `${slug(it.title)}-${it.id.slice(-4)}` : alt;
+    }
     while (used.has(name)) name = `${name}-${it.id.slice(-4)}`;
     used.add(name);
     const body = it.body || "";
-    // Skills already carry their own YAML frontmatter (name/description) — keep it as-is; for anything
-    // without frontmatter (e.g. prompts), add a description so it shows nicely in the / menu.
-    const content = /^---\r?\n/.test(body)
-      ? body.endsWith("\n") ? body : `${body}\n`
-      : `---\ndescription: "${`${it.title} (AIStorageDepot)`.replace(/"/g, "'").replace(/\s+/g, " ").trim()}"\n---\n\n${body}\n`;
+    // Prompts get the argument treatment. Skills already carry their own YAML frontmatter
+    // (name/description) — keep them as-is; anything else without frontmatter gets a
+    // description so it shows nicely in the / menu.
+    const content = isPromptItem(it)
+      ? promptCommand(it)
+      : /^---\r?\n/.test(body)
+        ? body.endsWith("\n") ? body : `${body}\n`
+        : `---\ndescription: "${yq(`${it.title} (AIStorageDepot)`)}"\n---\n\n${body}\n`;
     await writeFile(join(dir, `${name}.md`), content, "utf8");
     n++;
   }
 
   console.log(`✓ Synced ${n} item${n === 1 ? "" : "s"} (${TYPES.join(", ")}) → ${dir}`);
   console.log(`  Start a new chat and type /aistoragedepot to use them.`);
+  console.log(`  Skills run as-is; prompts take input — type values right after the command.`);
 }