acuvo-code 0.4.1 → 0.4.2

package/lib/input-box.mjs

@@ -67,7 +67,20 @@ export function visibleWidth(s) {
  * @returns {{lines: string[], cursorColumn: number}} 1-based cursor column
  */
 export function renderBox({ value = '', cursor = 0, columns = 80, prompt = '› ' } = {}) {
-  const width = Math.max(20, Math.min(100, columns - 1));
+  /**
+   * ── ⚠️ FULL WIDTH. THE 100-COLUMN CAP WAS WRONG AND IT LOOKED WRONG ────────
+   *
+   * Roman, from a screenshot: *"ours isn't the entire width."* In a ~200-column
+   * terminal a 100-column box reads as a half-finished element rather than as a
+   * deliberate measure — it is the input, and the input should be as wide as the
+   * place you are typing.
+   *
+   * ⚠️ `columns - 1`, NOT `columns`. A box drawn to the very last column makes
+   * many terminals wrap to the next row the moment the final border character is
+   * written, which pushes everything down by one and breaks the cursor
+   * arithmetic for the rest of the session.
+   */
+  const width = Math.max(20, columns - 1);
   const inner = width - 2;
   const promptWidth = visibleWidth(prompt);
 
@@ -274,7 +287,19 @@ export function readBoxedLine({ input, output, history = [], onInterrupt = null,
       input.off('data', onData);
       input.off('end', onEnd);
       try { input.setRawMode?.(false); } catch { /* not a TTY any more */ }
-      output.write('\n');
+      /**
+       * ── ⚠️⚠️ MOVE BELOW THE BOX BEFORE ANYTHING ELSE WRITES ─────────────────
+       *
+       * Seen in a screenshot: an MCP warning printed straight ON TOP of the
+       * bottom border, which was left visible only where the message was
+       * shorter than the box.
+       *
+       * The cursor rests on the INPUT line — line 2 of 3. A bare `\n` moves it
+       * to line 3, which IS the bottom border, so the very next thing anyone
+       * writes overwrites it. Down one FIRST, then a newline, so the cursor
+       * lands on fresh ground below a box that stays intact.
+       */
+      output.write(`${CSI}1B\n`);
       resolve({ value, reason });
     };