@hackerrank/astra-cli 0.1.33 → 0.1.34

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hackerrank/astra-cli",
-  "version": "0.1.33",
+  "version": "0.1.34",
   "description": "Minimal zero-dependency AI coding agent for the HackerRank AI Gateway.",
   "type": "module",
   "bin": {

package/src/repl.js

@@ -301,6 +301,7 @@ export class Screen {
     this._busyResume = false;
     this._prevScrollBottom = 0;
     this._prevFooterTop = 0;
+    this._renderedRows = new Map();
     this._inPaste = false;
     this.promptLabel = PROMPT_PREFIX;
     this._onData = this._onData.bind(this);
@@ -343,6 +344,7 @@ export class Screen {
     this.out.off("resize", this._onResize);
     if (this.inp.isTTY) this.inp.setRawMode(false);
     this.out.write("\x1b[?2004l"); // disable bracketed paste mode
+    this.out.write("\x1b[?25h");   // ensure the cursor is restored on exit
     this.out.write("\x1b[r");                    // reset scroll region
     this.out.write(`\x1b[${this.rows};1H\n`);    // move below footer
   }
@@ -369,61 +371,63 @@ export class Screen {
 
   /** Unified redraw of footer divider, multi-line prompt, and status lines without overwriting. */
   redraw() {
-    if (this._prevScrollBottom !== this.scrollBottom) {
+    const scrollChanged = this._prevScrollBottom !== this.scrollBottom;
+    if (scrollChanged) {
       this.out.write(`\x1b[1;${this.scrollBottom}r`);
       this._prevScrollBottom = this.scrollBottom;
+      // Prompt wrapping can move the footer. Clear the old pinned region once;
+      // ordinary input and spinner frames use the row diff below instead.
+      if (this._prevFooterTop && this._prevFooterTop !== this.footerTop) {
+        for (let row = this._prevFooterTop; row <= this.rows; row++) {
+          this.out.write(`\x1b[${row};1H\x1b[2K`);
+        }
+      }
+      this._prevFooterTop = this.footerTop;
+      // A resize or prompt-height change moves pinned rows. Their old
+      // coordinates are no longer meaningful, so repaint them once.
+      this._renderedRows.clear();
     }
 
-    this.out.write("\x1b[s"); // save cursor
-
-    // Clear only from footerTop down to terminal bottom
-    for (let r = this.footerTop; r <= this.rows; r++) {
-      this.out.write(`\x1b[${r};1H\x1b[2K`);
-    }
-
-    // Draw top divider rule at footerTop
-    this.out.write(`\x1b[${this.footerTop};1H`);
-    this.out.write(this.footer[0] || "");
+    const rows = new Map();
+    const put = (row, text) => {
+      if (row >= 1 && row <= this.rows) rows.set(row, text || "");
+    };
 
-    // The loader has its own permanently reserved row. This prevents a busy
-    // transition from moving the input or status rows and causing terminal flicker.
+    put(this.footerTop, this.footer[0]);
     if (this._busy) {
-      this.out.write(`\x1b[${this.loaderRow};1H`);
       const frame = SPINNER_FRAMES[this._busyFrame % SPINNER_FRAMES.length];
-      this.out.write(C.cyan(frame + " ") + C.dim(this._busyLabel + "…"));
-    }
-
-    // Draw the input below the reserved loader row.
-    if (!this._busy) {
+      put(this.loaderRow, C.cyan(frame + " ") + C.dim(this._busyLabel + "…"));
+    } else {
       const visual = computeVisualLines(this.buf, this.promptLabel || PROMPT_PREFIX, this.cols);
-      const visibleLines = visual.slice(-this.promptLines);
-      for (let i = 0; i < visibleLines.length; i++) {
-        const row = this.promptRow + i;
-        this.out.write(`\x1b[${row};1H`);
-        this.out.write(visibleLines[i].text);
+      for (const [i, line] of visual.slice(-this.promptLines).entries()) {
+        put(this.promptRow + i, line.text);
       }
     }
-
-    // Draw status lines below the prompt
     for (let i = 1; i < FOOTER_LINES; i++) {
-      const row = this.statusStartRow + i - 1;
-      if (row <= this.rows) {
-        this.out.write(`\x1b[${row};1H`);
-        this.out.write(this.footer[i] || "");
+      put(this.statusStartRow + i - 1, this.footer[i]);
+    }
+
+    // Update only changed rows. Clearing a row before immediately redrawing it
+    // caused a visible blank frame on every keystroke and spinner tick.
+    const changed = new Set([...this._renderedRows.keys(), ...rows.keys()]);
+    let output = "";
+    for (const row of [...changed].sort((a, b) => a - b)) {
+      const next = rows.get(row) || "";
+      if (this._renderedRows.get(row) !== next) {
+        output += `\x1b[${row};1H\x1b[2K${next}`;
       }
     }
+    this._renderedRows = rows;
 
-    // Place cursor at the end of input
     if (!this._busy) {
       const visual = computeVisualLines(this.buf, this.promptLabel || PROMPT_PREFIX, this.cols);
       const visibleLines = visual.slice(-this.promptLines);
       const lastLine = visibleLines[visibleLines.length - 1] || { visibleLength: 0 };
-      const cursorRow = this.promptRow + (visibleLines.length - 1);
+      const cursorRow = this.promptRow + visibleLines.length - 1;
       const cursorCol = Math.min(this.cols, lastLine.visibleLength + 1);
-      this.out.write(`\x1b[${cursorRow};${cursorCol}H`);
-    } else {
-      this.out.write("\x1b[u"); // restore cursor
+      output += `\x1b[${cursorRow};${cursorCol}H`;
     }
+    if (output) this.out.write(output);
   }
 
   drawFooter() {
@@ -437,6 +441,7 @@ export class Screen {
   /** Start the minimal loading state on the prompt row. */
   startBusy(label = "working") {
     this._busy = true;
+    this.out.write("\x1b[?25l"); // spinner updates must not move a visible cursor
     this._busyLabel = label;
     this._busyFrame = 0;
     this.redraw();
@@ -453,6 +458,7 @@ export class Screen {
     if (this._busyTimer) { clearInterval(this._busyTimer); this._busyTimer = null; }
     this._busy = false;
     this._busyResume = false;
+    this.out.write("\x1b[?25h");
     this.redraw();
   }