@tiens.nguyen/gonext-cli 1.0.367 → 1.0.369

package/gonext-repl.mjs

@@ -21,7 +21,7 @@
  * interrupted investigation actually resumes it). /reset clears it.
  *
  * Requires the worker daemon to be running (it claims the job).
- * Commands: /exit  /model  /reset  /revert [runId]  /help   Ctrl-C stops following the current run.
+ * Commands: /exit  /model  /reset  /manual-edit [runId]  /help   Ctrl-C stops following the current run.
  */
 import { readFile, mkdir, writeFile, unlink } from "node:fs/promises";
 import { existsSync, statSync, readFileSync } from "node:fs";
@@ -286,7 +286,7 @@ if (argv.includes("--help") || argv.includes("-h")) {
       "  -v, --verbose   show the agent's diagnostic logs ([gonext-agent] …) and python stderr\n" +
       "  -h, --help      this help\n\n" +
       "Auth comes from ~/.gonext/worker.env; models/budgets/RAG come from your web Settings.\n" +
-      "Inside the REPL: /exit · /model · /reset · /revert [runId] · /help — Ctrl-C stops following a turn.\n" +
+      "Inside the REPL: /exit · /model · /reset · /manual-edit [runId] · /help — Ctrl-C stops following a turn.\n" +
       "Questions are enqueued like web questions and executed by the RUNNING\n" +
       "gonext-cli daemon — its terminal shows the full [gonext-agent] logs.\n" +
       "Conversation history is saved per-folder in ~/.gonext/sessions and resumed next time\n" +
@@ -360,7 +360,7 @@ const COMMANDS = [
   { name: "/test-auto", desc: "toggle auto-test (verify & fix until it passes) for this folder" },
   { name: "/rag-local", desc: "toggle storing this folder's RAG knowledge base locally (vs cloud/S3)" },
   { name: "/server", desc: "pick a deployment server (host/user) for this session" },
-  { name: "/revert", usage: "/revert [runId]", desc: "undo the agent's file edits (latest run)" },
+  { name: "/manual-edit", aliases: ["/revert"], usage: "/manual-edit [runId]", desc: "undo the agent's file edits so you can edit them yourself (latest run)" },
   { name: "/reset", aliases: ["/new"], desc: "forget this folder's saved conversation and start fresh" },
   { name: "/output-token", desc: "show the agent code-model OUTPUT-token total for this workspace" },
   { name: "/reset-output-token-global", desc: "reset the GLOBAL (you + coding server) output-token total to 0" },
@@ -641,6 +641,72 @@ if (process.stdout.isTTY) {
     }
   });
 }
+// ---------- edit-mode bottom bar (display-only + Shift+Tab to cycle) ----------
+// A split rule + a mode indicator pinned BELOW the ">> " input line — the bar is the last
+// thing on screen and the user types ABOVE it: "accept edits on" (the agent applies file
+// edits — today's behaviour) vs "manually edit". DISPLAY-ONLY for now — it tracks the mode
+// so the bar reflects it and Shift+Tab cycles it; wiring it to actually gate the agent's
+// edits is a deliberate follow-up.
+let editMode = "accept"; // "accept" | "manual"
+const modeLineText = () =>
+  white("✎ " + (editMode === "accept" ? "accept edits on" : "manually edit")) +
+  dim("   (shift+tab to switch)");
+const FOOTER_ROWS = 2; // the rule row + the mode row
+// True only while the MAIN loop's ">> " prompt is waiting for input. The bar belongs to that
+// prompt, so it must never appear under an ask()/askSecret() question or during a turn.
+let footerActive = false;
+// Physical rows the prompt + typed line occupies (it wraps once past the terminal width) —
+// the bar goes under the LAST of them, not under the caret's row.
+const inputRows = () => {
+  const width = Math.max(20, process.stdout.columns || 80);
+  return Math.floor((PROMPT_COLS + rl.line.length) / width) + 1;
+};
+const cursorPos = () =>
+  (rl.getCursorPos ? rl.getCursorPos() : null) ?? { rows: 0, cols: PROMPT_COLS };
+// Draw the bar below the input line and put the caret back exactly where readline left it.
+// Relative moves only (same reason as the slash overlay): the block must survive the terminal
+// scrolling — and writing past the last screen row scrolls the input line up TOO, so the
+// matching up-move still lands on it.
+function drawFooter() {
+  if (!footerActive || !process.stdout.isTTY || slashOverlayActive) return;
+  const width = Math.max(20, process.stdout.columns || 80);
+  const pos = cursorPos();
+  const down = Math.max(0, inputRows() - 1 - pos.rows); // caret row → last input row
+  process.stdout.write(
+    (down ? `\x1b[${down}B` : "") +
+      "\r\n\x1b[2K" + dim("─".repeat(width)) +
+      // CLIPPED: the mode line is ~44 columns, so on a narrow terminal it would WRAP onto a
+      // third row — and ESC[nA counts PHYSICAL rows, so the caret would come back one row
+      // short and the next keystroke would type over the rule (the #109 failure mode).
+      "\r\n\x1b[2K" + clipVisible(GUTTER + modeLineText(), width - 1) +
+      "\x1b[J" +                                  // nothing stale below the bar
+      `\x1b[${FOOTER_ROWS + down}A\r` +           // back to the caret's row
+      (pos.cols > 0 ? `\x1b[${pos.cols}C` : "")
+  );
+}
+// Wipe the bar (and anything under it) WITHOUT touching the input line — called the instant
+// Enter is pressed so the command's output starts on clean rows instead of printing over the
+// rule and orphaning the mode line.
+function eraseFooter() {
+  const was = footerActive;
+  footerActive = false;
+  if (!was || !process.stdout.isTTY || slashOverlayActive) return;
+  const pos = cursorPos();
+  const down = Math.max(0, inputRows() - 1 - pos.rows) + 1; // first footer row
+  process.stdout.write(
+    `\x1b[${down}B\r\x1b[J\x1b[${down}A\r` + (pos.cols > 0 ? `\x1b[${pos.cols}C` : "")
+  );
+}
+// Shift+Tab redraw: the bar is below the caret, so a plain redraw in place is enough.
+const redrawModeLine = () => drawFooter();
+// Enter must tear the bar down BEFORE readline echoes its newline (which would land the
+// cursor on the rule row and leave the mode line orphaned below the output). Our normal
+// keypress listener runs AFTER readline's, so this one is prepended to run first.
+if (process.stdin.isTTY) {
+  process.stdin.prependListener("keypress", (_ch, key) => {
+    if (key && (key.name === "return" || key.name === "enter")) eraseFooter();
+  });
+}
 if (process.stdin.isTTY) {
   process.stdin.on("keypress", (_ch, key) => {
     // An arrow-key list picker (/server) is up → ↑/↓ move the highlight, Enter chooses.
@@ -665,6 +731,15 @@ if (process.stdin.isTTY) {
       rl.historyIndex = -1;
       return;
     }
+    // Shift+Tab cycles the edit mode shown in the bottom bar (display-only). Skip while the
+    // slash overlay owns the screen so we don't fight its redraw.
+    if (key && key.name === "tab" && key.shift) {
+      if (!slashOverlayActive) {
+        editMode = editMode === "accept" ? "manual" : "accept";
+        redrawModeLine();
+      }
+      return;
+    }
     // Enter is handled by rl.on("line") — don't redraw the hint on the way out.
     if (key && (key.name === "return" || key.name === "enter")) return;
     // ↑/↓ while the menu is up = move the highlight IN PLACE. readline drawing is
@@ -1591,6 +1666,10 @@ rl._refreshLine = () => {
   if (following || listPickerActive || slashOverlayActive) return;
   if (secretInput) return; // don't redraw the typed key (task #127)
   if (_origRefreshLine) _origRefreshLine();
+  // That refresh ends with clearScreenDown — i.e. it just ERASED the bottom bar sitting
+  // under the input line. Redraw it here so every readline redraw (typing, backspace,
+  // history recall, rl.prompt(), a resize) repaints the bar as one atomic step.
+  drawFooter();
 };
 
 const sleep = (ms) => new Promise((r) => setTimeout(r, ms));
@@ -2525,14 +2604,13 @@ async function main() {
         bold(cyan("GONEXT AGENT")) + dim(`  v${REPL_VERSION}`),
         lbl("model") + (p.agentModelId || probe?.modelKey || "?"),
         lbl("coder") + (p.codingModelId || dim("—")),
-        lbl("RAG") +
-          (p.ragEnabled ? (sessionRagMode === "cloud" ? "cloud" : "local") : dim("off")) +
-          dim("   auto-test ") + (sessionTestAuto ? "on" : dim("off")),
+        lbl("RAG") + (p.ragEnabled ? (sessionRagMode === "cloud" ? "cloud" : "local") : dim("off")),
+        sessionTestAuto ? green("auto-test on") : dim("auto-test off"),
         lbl("folder") + ws + dim(wsSync ? ` (${wsSync.run ? "run" : "read-only"}${wsSync.enabled ? " · sync" : ""})` : ""),
       ];
       if (selectedServer)
         statusRows.push(lbl("deploy") + `${selectedServer.name} ${dim(`(${selectedServer.user}@${selectedServer.host})`)}`);
-      statusRows.push(dim(`api ${apiBase.replace(/^https?:\/\//, "")} · key ${workerKey.slice(0, 8)}…`));
+      statusRows.push(dim(`key ${workerKey.slice(0, 8)}…`));
       console.log(renderBanner(statusRows));
     }
     // Task #127, Phase 3: if the base chat model server isn't answering, offer to set it up
@@ -2638,11 +2716,16 @@ async function main() {
     } else {
       // Ctrl+C at a prompt with the slash overlay open = dismiss the overlay first (#109).
       if (slashOverlayActive) closeSlashOverlay();
+      // Wipe the bottom bar before printing below the input line, or the note would land on
+      // the rule row and strand the mode line under it.
+      eraseFooter();
       process.stdout.write("\n" + GUTTER + dim("(/exit to quit)") + "\n");
       // Discard anything half-typed (Ctrl-C at a prompt means "never mind") and
-      // redraw through readline so its cursor tracking stays right.
+      // redraw through readline so its cursor tracking stays right — the refresh repaints
+      // the bar under the fresh prompt.
       rl.line = "";
       rl.cursor = 0;
+      footerActive = process.stdout.isTTY;
       rl.prompt();
     }
   };
@@ -2655,16 +2738,30 @@ async function main() {
   let lastCancelledTask = "";
 
   for (;;) {
+    // The bottom bar (split rule + edit-mode indicator) is pinned BELOW the input line, at
+    // the very bottom of the screen. Arming it here is all it takes: rl.prompt() refreshes
+    // the line, and our _refreshLine hook paints the bar underneath (and repaints it on
+    // every keystroke that erases it). Enter tears it down again.
+    footerActive = process.stdout.isTTY;
     rl.prompt();
-    // Absorb blank Enter-presses SILENTLY under the same prompt — without this, lines
-    // typed while the agent was busy on a long turn (queued, not yet read) all drain at
-    // once the moment the turn ends, each re-printing ">> " with no newline between them
-    // (e.g. ">> >> >> "). Only re-print the prompt for a genuine stdin close or input.
+    // Absorb blank Enter-presses SILENTLY — they never reach the agent. Lines typed while
+    // it was busy on a long turn (queued, not yet read) drain here all at once the moment
+    // the turn ends; a queued line carries no Enter keypress, so its re-prompt overwrites
+    // in place (\r + refresh) rather than stacking ">> >> >> " on one row.
     let lineRaw;
     for (;;) {
       lineRaw = await nextLine();
       if (lineRaw === null || lineRaw.trim()) break;
+      // A bare Enter: its keypress already tore the bar down, and readline left the caret on
+      // a fresh row with no prompt on it. Re-print the prompt so that row isn't blank — the
+      // refresh repaints the bar under it. (Queued lines drain with no keypress at all, so
+      // this re-prompt just overwrites in place and can't stack ">> " on one row.)
+      if (process.stdout.isTTY) {
+        footerActive = true;
+        rl.prompt();
+      }
     }
+    footerActive = false; // a line arrived from the queue (no Enter keypress) → disarm
     if (lineRaw === null) break; // stdin closed
     const line = lineRaw.trim();
     if (line === "/exit" || line === "/quit") break;
@@ -2751,7 +2848,7 @@ async function main() {
       }
       continue;
     }
-    if (line.startsWith("/revert")) {
+    if (line.startsWith("/manual-edit") || line.startsWith("/revert")) {
       const runId = line.split(/\s+/)[1] ?? "";
       await new Promise((res) => {
         const p = spawn(

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tiens.nguyen/gonext-cli",
-  "version": "1.0.367",
+  "version": "1.0.369",
   "description": "GoNext CLI — the gonext terminal plus the local worker that runs agent / OCR / PDF / embedding jobs on your Mac (Ollama / MLX / OpenAI-compatible).",
   "type": "module",
   "license": "MIT",