@yugabytedb/meko-mcp 1.0.5 → 1.0.6

package/bin/create-meko-setup.mjs

@@ -374,6 +374,8 @@ async function main() {
     // Step 1 — connection type. Local Meko only makes sense for users who
     // have the server repo cloned locally (docker-compose stack on :8000);
     // npm/npx users have no way to run it, so hide the option for them.
+    // When only Cloud is available, skip the prompt entirely — asking to
+    // pick between one option is pointless UX.
     const connectionOptions = [
       { label: 'Cloud Meko', description: 'Hosted at mekodev.com (requires URL + API key)' },
     ];
@@ -383,7 +385,10 @@ async function main() {
         description: 'Local stack at localhost:8000',
       });
     }
-    const connectionType = await askChoice('Connection type:', connectionOptions);
+    const connectionType =
+      connectionOptions.length === 1
+        ? connectionOptions[0].label
+        : await askChoice('Connection type:', connectionOptions);
 
     if (connectionType === 'Cloud Meko') {
       // Step 2 — MCP URL

package/lib/migrate/orchestrator.mjs

@@ -283,11 +283,17 @@ export async function runMigration(opts) {
       const [, memRes] = await Promise.all([addMsgPromise, memPromise]);
       messagesSent++;
       if (!opts.noExtract) {
-        if (memRes && memRes.ok) {
+        if (memRes && memRes.ok && !isErrorText(memRes.text)) {
           memoryPairsSent++;
           prog.counts.memoryPairsSent++;
         } else if (memRes) {
           prog.counts.memoryErrors++;
+          log.warn("memory_add_failed", {
+            agentId: ref.agentId,
+            sessionUuid: ref.sessionUuid,
+            userUuid: ex.user_uuid,
+            detail: errorDetail(memRes),
+          });
         }
       }
       lastUuid = ex.user_uuid;
@@ -361,7 +367,7 @@ export async function runMigration(opts) {
       metadata: JSON.stringify(item.metadata),
     }, { fireAndForget: true });
 
-    if (res.ok) {
+    if (res.ok && !isErrorText(res.text)) {
       state.markMemory(st, item.agentId, item.hash, {
         memoryId: extractId(res.text) || "",
         name: item.name,
@@ -370,7 +376,12 @@ export async function runMigration(opts) {
       prog.counts.memoriesFromMD++;
     } else {
       prog.counts.memoryErrors++;
-      log.warn("memory_md_failed", { agentId: item.agentId, name: item.name, error: res.error?.message });
+      log.warn("memory_md_failed", {
+        agentId: item.agentId,
+        name: item.name,
+        error: res.error?.message,
+        detail: errorDetail(res),
+      });
     }
     if (++opsSinceFlush >= FLUSH_EVERY) flushState();
     writeProgress();
@@ -441,3 +452,34 @@ function extractId(text) {
     return null;
   }
 }
+
+/**
+ * The MCP transport returns `{ok:true}` whenever the HTTP call itself
+ * succeeded, even if the tool's body encoded a failure. memory_add in
+ * older server builds returned plain "Error adding memory: ..." strings
+ * on exception; newer builds return JSON `{"error":"memory_add_failed",...}`.
+ * Either way, the text content is the authoritative success signal.
+ */
+function isErrorText(text) {
+  if (!text) return false;
+  const trimmed = text.trim();
+  if (trimmed.startsWith("Error adding memory")) return true;
+  if (trimmed.startsWith("{")) {
+    try {
+      const obj = JSON.parse(trimmed);
+      if (obj && typeof obj === "object" && obj.error) return true;
+    } catch { /* not JSON; fall through */ }
+  }
+  return false;
+}
+
+function errorDetail(res) {
+  if (!res) return null;
+  if (res.error?.message) return res.error.message;
+  if (!res.text) return null;
+  try {
+    const obj = JSON.parse(res.text);
+    if (obj?.error) return obj.detail || obj.error;
+  } catch { /* not JSON */ }
+  return res.text.slice(0, 200);
+}

package/lib/prompt.mjs

@@ -164,16 +164,21 @@ export async function askText(questionText, defaultValue) {
  * @returns {Promise<string>}
  */
 export async function askSecret(questionText) {
+  // Fully close the singleton readline BEFORE printing the prompt or
+  // entering raw mode. pause() alone was insufficient — readline left
+  // stdin in a state where the terminal echo would still leak keystrokes
+  // before our raw-mode handler wrote its `*` mask. close() forcefully
+  // releases stdin back to the OS, which lets setRawMode actually
+  // suppress echo. getRL() will recreate the interface on the next
+  // askText/askYesNo call, so downstream prompts still work.
+  if (rl) {
+    rl.close();
+    rl = null;
+  }
+
   // Print the prompt without a trailing newline.
   process.stdout.write(`  ${questionText} `);
 
-  // Pause the singleton readline so its data listener doesn't echo each
-  // keystroke in parallel with our manual "*" write. Without this, users
-  // see `d*j*f*k*...` (raw char + mask interleaved) and backspace fights
-  // with readline over cursor position.
-  const pausedRL = rl;
-  if (pausedRL) pausedRL.pause();
-
   return new Promise((resolve, reject) => {
     let secret = "";
     const stdin = process.stdin;
@@ -185,7 +190,6 @@ export async function askSecret(questionText) {
       const fallbackRL = createInterface({ input: stdin, output: process.stdout });
       fallbackRL.question("", (answer) => {
         fallbackRL.close();
-        if (pausedRL) pausedRL.resume();
         resolve(answer.trim());
       });
       return;
@@ -200,8 +204,8 @@ export async function askSecret(questionText) {
     function cleanup() {
       stdin.setRawMode(wasRaw);
       stdin.removeListener("data", onData);
+      stdin.pause();
       process.stdout.write("\n");
-      if (pausedRL) pausedRL.resume();
     }
 
     function onData(ch) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yugabytedb/meko-mcp",
-  "version": "1.0.5",
+  "version": "1.0.6",
   "description": "Single-line installer for Meko MCP Server across Claude and Cursor clients",
   "type": "module",
   "bin": {