@tiens.nguyen/gonext-local-worker 1.0.187 → 1.0.188

package/gonext-repl.mjs

@@ -53,6 +53,11 @@ const isCodeCloseLine = (line) => line.trim() === "</code>";
 // "unzip_file(...) | → Unzipped 256 files…") duplicates what's already visible in the
 // gonext-local-worker daemon's own terminal — swallow it here to keep the REPL terse.
 const isToolStepSummary = (line) => / \| → /.test(line);
+// The router's internal play-by-play ("Routing your request…", "→ Agent mode (needs
+// tools)", "→ Chat reply", "Choosing a tool…") is implementation detail, not something a
+// user needs — also visible in the daemon's own terminal. Swallowed; see routingAnnounced.
+const isRoutingLine = (line) =>
+  /^(Routing your request…|Choosing a tool…|→ Agent mode.*|→ Chat reply)$/.test(line.trim());
 
 // Playful words for the local "thinking…" ticker, reused verbatim from the worker's
 // thinking_words.txt sibling (falls back to a tiny built-in list if it's missing).
@@ -106,7 +111,17 @@ if (!workerKey || !apiBase) {
 // awaits (workspace prompt fetch, payload probe) are queued instead of lost — vital
 // for piped/scripted input, harmless for interactive TTYs. Both ask() and the main
 // loop consume from this queue (never rl.question, to avoid double-capture).
-const rl = createInterface({ input: process.stdin, output: process.stdout });
+//
+// terminal: false — we draw our OWN prompt (cyan ">> ") and colorize output ourselves;
+// readline's default terminal:true (auto-enabled whenever output is a TTY) makes it ALSO
+// track cursor position and redraw the input line on every keystroke using ITS OWN
+// (empty, since we never call rl.setPrompt/rl.prompt) prompt — that tracking goes stale
+// the moment we write anything readline doesn't know about (which is everything we print
+// between prompts), and its next redraw emits a stray leading "[" (a malformed/partial
+// cursor-position escape) right before typed input, e.g. "[>> hi". With terminal:false,
+// readline stops managing the line/cursor itself; the OS's own canonical-mode terminal
+// still echoes keystrokes normally, and 'line' events fire exactly the same.
+const rl = createInterface({ input: process.stdin, output: process.stdout, terminal: false });
 const _lineQueue = [];
 let _lineWaiter = null;
 let _stdinClosed = false;
@@ -256,6 +271,7 @@ async function runAgentTurn(history) {
   let statusShown = false; // a transient status line is currently on screen
   let warnedPending = false;
   let jobRunning = false; // only tick "thinking…" once the worker has claimed the job
+  let routingAnnounced = false; // one friendly line replaces the whole routing play-by-play
 
   // --- Live "thinking…" ticker ---------------------------------------------------
   // Between visible tokens (prompt-eval / awaiting the first token) we show ONE in-place
@@ -326,6 +342,7 @@ async function runAgentTurn(history) {
   // them prints in blue (not dim, so it stands out from "Thought:" prose); blank lines
   // keep spacing; everything else commits dim to scrollback.
   let inCode = false;
+  let lastWasBlank = true; // suppresses a leading blank line and collapses repeats
   const consume = (chunk) => {
     carry += stripThinkTags(chunk);
     let nl;
@@ -334,6 +351,19 @@ async function runAgentTurn(history) {
       carry = carry.slice(nl + 1);
       if (isHeartbeatLine(line) || isFenceLine(line)) continue; // swallow, no trace
       if (isToolStepSummary(line)) { lastContentAt = Date.now(); continue; } // swallow
+      if (isRoutingLine(line)) {
+        // Replace the whole routing play-by-play with ONE friendly line, shown once
+        // per turn — the raw labels are implementation detail (visible in the daemon's
+        // own terminal if you need them).
+        if (!routingAnnounced) {
+          routingAnnounced = true;
+          onContent();
+          process.stdout.write(dim(`${pickWord()}…`) + "\n");
+          lastWasBlank = false;
+        }
+        lastContentAt = Date.now();
+        continue;
+      }
       if (!inCode && isCodeOpenLine(line)) {
         stampIfThinking(); // hold/refresh the stamp; the code body commits it below
         inCode = true;
@@ -344,14 +374,21 @@ async function runAgentTurn(history) {
         if (line.trim() === "") { lastContentAt = Date.now(); continue; } // skip blank lines in code
         onContent();
         process.stdout.write(blue(line) + "\n");
+        lastWasBlank = false;
         continue;
       }
       if (line.trim() === "") {
-        if (!thinking && !statusShown) process.stdout.write("\n");
+        // Several sources (swallowed routing lines, swallowed step summaries) each leave
+        // their own blank-line companion behind — collapse consecutive blanks into one.
+        if (!thinking && !statusShown && !lastWasBlank) {
+          process.stdout.write("\n");
+          lastWasBlank = true;
+        }
         continue;
       }
       onContent();
       process.stdout.write(dim(line) + "\n");
+      lastWasBlank = false;
     }
     // A non-empty remainder is a partial CONTENT line (heartbeats always arrive whole with
     // a newline), so tokens are actively flowing — keep the ticker asleep.
@@ -430,7 +467,10 @@ async function main() {
   console.log(dim("Ask about this repo, or /help. Ctrl-C aborts a running turn.\n"));
 
   const history = [];
-  rl.on("SIGINT", () => {
+  // With terminal:false, Ctrl-C is no longer intercepted by readline's raw-mode byte
+  // scanning (rl.on("SIGINT", …) would never fire) — it now delivers a real OS SIGINT to
+  // the process instead, which we catch directly here. Same behavior as before.
+  process.on("SIGINT", () => {
     if (following) {
       followAborted = true;
       process.stdout.write(red("\n^C "));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tiens.nguyen/gonext-local-worker",
-  "version": "1.0.187",
+  "version": "1.0.188",
   "description": "Polls GoNext cloud API for async local LLM jobs and runs them against Ollama/OpenAI-compatible servers on this Mac",
   "type": "module",
   "license": "MIT",