@retasc/cli 1.6.1 → 1.6.2

package/dist/commands/bind.js

@@ -11,7 +11,13 @@ function isInteractive() {
 async function ask(question) {
     const rl = createInterface({ input: stdin, output: stdout });
     try {
-        return (await rl.question(question)).trim();
+        // readline's question() NEVER settles when stdin hits EOF (Ctrl-D, or a
+        // closed pipe) — it silently drops the callback rather than resolving ""
+        // or throwing, so a bare `await` here wedges the CLI with no output and no
+        // exit. Race the close event so callers fail fast instead (RTSC-269).
+        const closed = new Promise((_, reject) => rl.once("close", () => reject(new Error("input closed"))));
+        closed.catch(() => { }); // the normal path closes rl too; that loser rejection is expected
+        return (await Promise.race([rl.question(question), closed])).trim();
     }
     finally {
         rl.close();
@@ -25,18 +31,37 @@ async function confirm(question, assumeYes) {
     const a = (await ask(`${question} [y/N] `)).toLowerCase();
     return a === "y" || a === "yes";
 }
-/** Pick from a list interactively, or return undefined to fall through to create. */
-async function pick(label, items, render) {
+/**
+ * Pick from a list interactively. Returns the chosen item, or undefined when the
+ * user explicitly picks the trailing "create new" option.
+ *
+ * Invalid input re-prompts rather than falling through: undefined used to mean
+ * BOTH "chose create-new" and "typed garbage", so a typo like `1)` silently
+ * created a new org/project instead of selecting option 1 (RTSC-269).
+ *
+ * The answer must be plain digits. Number() alone accepts "0x4"/"4e0"/"+4"/
+ * "4.0", which would route those straight to create-new — the same
+ * non-canonical-input-hits-the-destructive-branch bug in a new coat.
+ */
+export async function pick(label, items, render, askFn = ask) {
     if (!items.length)
         return undefined;
     console.log(`\n${label}:`);
     items.forEach((it, i) => console.log(`  ${i + 1}) ${render(it)}`));
-    console.log(`  ${items.length + 1}) + create new`);
-    const a = await ask("Choose a number: ");
-    const n = Number(a);
-    if (Number.isInteger(n) && n >= 1 && n <= items.length)
-        return items[n - 1];
-    return undefined; // create-new (or invalid → treated as create)
+    const createNew = items.length + 1;
+    console.log(`  ${createNew}) + create new`);
+    for (let attempt = 0; attempt < 3; attempt++) {
+        const a = await askFn("Choose a number: ");
+        if (/^\d+$/.test(a)) {
+            const n = Number(a);
+            if (n >= 1 && n <= items.length)
+                return items[n - 1];
+            if (n === createNew)
+                return undefined;
+        }
+        console.log(`Please enter a number between 1 and ${createNew}.`);
+    }
+    throw new Error("no valid choice — aborting");
 }
 export async function bindAction(opts) {
     const cfg = loadConfig();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@retasc/cli",
-  "version": "1.6.1",
+  "version": "1.6.2",
   "description": "Retasc CLI — sign in with GitHub, create projects, mint agent API keys, and wire your agent to the Retasc MCP server in one command.",
   "type": "module",
   "bin": {