@tritard/waterbrother 0.6.0 → 0.6.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tritard/waterbrother",
-  "version": "0.6.0",
+  "version": "0.6.2",
   "description": "Waterbrother: Grok-powered coding CLI with local tools, sessions, operator modes, and approval controls",
   "type": "module",
   "bin": {

package/src/prompt.js

@@ -8,10 +8,15 @@ export async function promptLine(label, { input = process.stdin, output = proces
     let buf = "";
     let resolved = false;
     const shouldPauseOnCleanup = typeof input.pause === "function";
+    const isTTY = input.isTTY && typeof input.setRawMode === "function";
+
     function cleanup() {
       resolved = true;
       input.removeListener("data", onData);
       if (signal) signal.removeEventListener("abort", onAbort);
+      if (isTTY) {
+        try { input.setRawMode(false); } catch {}
+      }
       if (shouldPauseOnCleanup) {
         try {
           input.pause();
@@ -25,10 +30,27 @@ export async function promptLine(label, { input = process.stdin, output = proces
         if (ch === "\r") continue;
         if (ch === "\n") {
           cleanup();
+          output.write("\n");
           resolve(buf);
           return;
         }
+        if (ch === "\u0003") {
+          cleanup();
+          reject(new DOMException("The operation was aborted.", "AbortError"));
+          return;
+        }
+        // Backspace
+        if (ch === "\u007f" || ch === "\b") {
+          if (buf.length > 0) {
+            buf = buf.slice(0, -1);
+            output.write("\b \b");
+          }
+          continue;
+        }
+        // Skip non-printable control chars
+        if (ch.charCodeAt(0) < 32) continue;
         buf += ch;
+        output.write(ch);
       }
     }
     function onAbort() {
@@ -36,6 +58,15 @@ export async function promptLine(label, { input = process.stdin, output = proces
       cleanup();
       reject(signal.reason || new DOMException("The operation was aborted.", "AbortError"));
     }
+
+    // Ensure clean stdin state before listening
+    if (isTTY) {
+      try { input.setRawMode(false); } catch {}
+    }
+    input.pause();
+    if (isTTY) {
+      input.setRawMode(true);
+    }
     input.on("data", onData);
     input.resume();
     if (signal) signal.addEventListener("abort", onAbort, { once: true });

package/src/router.js

@@ -22,14 +22,15 @@ function looksLikeWorkRequest(text) {
 function inferPassFromText(text) {
   const l = lower(text);
   if (!l) return null;
-  if (/^(ship it|accept|merge it|looks good ship it)\b/.test(l)) return { intent: "accept", confidence: "high" };
-  if (/^(fix these|fix that|address those|redo|rebuild|try again|patch it|build it|go ahead|go$|implement it)\b/.test(l)) return { intent: "build", confidence: "high" };
-  if (/^(check it|what'?s wrong|what is wrong|review it|challenge( harder)?|find bugs|poke holes|stress test it)\b/.test(l)) return { intent: "challenge", confidence: "high" };
-  if (/^(what am i not thinking of|what else|what if|surprise me|invent|give me alternatives|think wider|out of the box)\b/.test(l)) return { intent: "invent", confidence: "high" };
-  if (/^(should we|which should|which option|what should we|think deeper|tell me more about|compare |tradeoff|jwt or |sessions or |oauth|oauth2)\b/.test(l)) return { intent: "decide", confidence: "medium" };
+  // Decision detail must be checked before generic "decide" — "tell me more about 2" is detail, not decide
   if (/^[1-9]\d*$/.test(l)) return { intent: "choose", confidence: "high", optionIndex: Number.parseInt(l, 10) };
   const moreMatch = l.match(/^(tell me more about|more on|details on|expand)\s+(\d+)\b/);
   if (moreMatch) return { intent: "decision-detail", confidence: "high", optionIndex: Number.parseInt(moreMatch[2], 10) };
+  if (/^(ship it|accept|merge it|looks good ship it)\b/.test(l)) return { intent: "accept", confidence: "high" };
+  if (/^(rebuild|try again|patch it|build it|go ahead|implement it)\b/.test(l)) return { intent: "build", confidence: "high" };
+  if (/^(check it|what'?s wrong|what is wrong|review it|challenge( harder)?|find bugs|poke holes|stress test it)\b/.test(l)) return { intent: "challenge", confidence: "high" };
+  if (/^(what am i not thinking of|what else|what if|surprise me|invent|give me alternatives|think wider|out of the box)\b/.test(l)) return { intent: "invent", confidence: "high" };
+  if (/^(should we|which should|which option|what should we|think deeper|compare |tradeoff|jwt or |sessions or |oauth|oauth2)\b/.test(l)) return { intent: "decide", confidence: "medium" };
   if (/^ignore\b/.test(l)) return { intent: "ignore", confidence: "medium" };
   return null;
 }
@@ -62,8 +63,13 @@ export function routeNaturalInput(text, { task = null } = {}) {
         return { kind: "choose-recommended-and-build", confidence: "high", raw };
       }
     }
+    if (task.state === "build-ready") {
+      if (/^(go|go ahead|do it|build it)\b/i.test(raw)) {
+        return { kind: "build", confidence: "high", raw };
+      }
+    }
     if (task.state === "review-ready") {
-      if (/^(fix these|fix them|address those|redo)\b/i.test(raw)) {
+      if (/^(fix these|fix them|fix that|address those|redo)\b/i.test(raw)) {
         return { kind: "fix-review-findings", confidence: "high", raw };
       }
       if (/^(think deeper|re-think|rethink|step back)\b/i.test(raw)) {