premanmcp 0.16.2 → 0.16.3

package/bin/integrations.js

@@ -522,10 +522,31 @@ async function askStep(question, { assumeYes, canGoBack, defaultYes = true }) {
   if (assumeYes) return fallback;
   const shown = defaultYes ? "Y/n" : "y/N";
   const hint = canGoBack ? `[${shown}/b]` : `[${shown}]`;
-  const answer = (await promptText(`${question} ${hint}: `)).trim().toLowerCase();
-  if (canGoBack && (answer === "b" || answer === "back")) return "back";
-  if (answer === "") return fallback;
-  return answer === "y" || answer === "yes" ? "yes" : "no";
+  // Anything that was not yes, no or a usable back used to fall through to no.
+  // On the repository question that is the most expensive misreading in the
+  // walk: "b" typed at the one step with nothing behind it, or a plain "sure",
+  // disconnected the repo and explained itself in a line that had already
+  // scrolled past by the time the next question drew.
+  //
+  // Bounded, and empty still takes the default, so a closed stdin ends the
+  // question rather than starting a spin.
+  for (let attempt = 0; attempt < 3; attempt += 1) {
+    const answer = (await promptText(`${question} ${hint}: `)).trim().toLowerCase();
+    if (answer === "") return fallback;
+    if (answer === "y" || answer === "yes") return "yes";
+    if (answer === "n" || answer === "no") return "no";
+    if (answer === "b" || answer === "back") {
+      if (canGoBack) return "back";
+      // Offered nowhere, so this is somebody guessing. Say why it did nothing
+      // rather than silently spending their answer on the opposite of it.
+      process.stdout.write("  Nothing to go back to yet.\n");
+      continue;
+    }
+    process.stdout.write(
+      `  Please answer y or n${canGoBack ? ", or b to go back" : ""}.\n`
+    );
+  }
+  return fallback;
 }
 
 /**

package/bin/shared.js

@@ -294,9 +294,28 @@ export function openUrl(url) {
 }
 
 export async function promptText(question) {
+  // A stream that has already ended will never deliver a line, and readline does
+  // not emit 'close' for an 'end' that happened before the interface existed --
+  // so the second prompt after stdin ran out resolved nothing, kept nothing
+  // alive, and the process simply exited mid-walk with a zero status.
+  if (process.stdin.readableEnded || process.stdin.destroyed) {
+    // Still shown, and still terminated, so the transcript reads the way it
+    // would have if somebody had pressed Enter.
+    process.stdout.write(`${question}\n`);
+    return "";
+  }
   const rl = createInterface({ input: process.stdin, output: process.stdout });
   try {
-    return (await rl.question(question)).trim();
+    // An exhausted or closed stdin never answers, and rl.question waits for an
+    // answer that cannot come. That is how a run given fewer answers than it had
+    // questions -- `onboard < /dev/null`, a script whose here-doc ran out, a
+    // pipe whose writer exited -- stopped dead on a prompt instead of finishing
+    // on its defaults. Empty is what every caller already treats as "no answer".
+    const answer = await Promise.race([
+      rl.question(question),
+      new Promise((resolve) => rl.once("close", () => resolve(""))),
+    ]);
+    return answer.trim();
   } finally {
     rl.close();
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "premanmcp",
-  "version": "0.16.2",
+  "version": "0.16.3",
   "description": "Turn APIs into agent-callable MCP tools with auth, testing, and audit logs",
   "type": "module",
   "bin": {