roforge-cli 0.3.0 → 0.3.2

package/README.md

@@ -14,15 +14,13 @@ roforge analyze <file...>   official Luau analyzer
 ## Install
 
 ```bash
-# GitHub Packages (published — any GitHub token with read:packages works)
-npm config set @hacvilke:registry https://npm.pkg.github.com
-echo "//npm.pkg.github.com/:_authToken=YOUR_GITHUB_TOKEN" >> ~/.npmrc
-npm i -g @hacvilke/roforge-cli
-
-# …or clone the repo and run `node cli/bin/roforge.js`
+npm i -g roforge-cli
 roforge
 ```
 
+Also on **GitHub Packages** as `@hacvilke/roforge-cli` (any GitHub token with
+`read:packages` works), or clone the repo and run `node cli/bin/roforge.js`.
+
 Requires Node ≥ 18.17. No other dependencies.
 
 ## Free tiers (no card needed)

package/bin/roforge.js

@@ -84,12 +84,26 @@ async function main() {
       const promptKey = async (label) => {
         process.stderr.write(`Paste ${label} (input hidden): `);
         let val = "";
-        if (process.stdin.isTTY) {
-          process.stdin.setRawMode(true);
+        // raw mode hides input; legacy Windows consoles may not support it —
+        // fall back to a visible paste instead of crashing
+        let raw = false;
+        if (process.stdin.isTTY && typeof process.stdin.setRawMode === "function") {
+          try {
+            process.stdin.setRawMode(true);
+            raw = true;
+          } catch {
+            raw = false;
+          }
+        }
+        if (raw) {
           for await (const chunk of process.stdin) {
             for (const ch of String(chunk)) {
               if (ch === "\r" || ch === "\n" || ch === "\x04") {
-                process.stdin.setRawMode(false);
+                try {
+                  process.stdin.setRawMode(false);
+                } catch {
+                  /* ignore */
+                }
                 console.error("");
                 return val;
               }
@@ -98,6 +112,7 @@ async function main() {
             }
           }
         } else {
+          process.stderr.write(dim("(raw input unsupported — paste the key and press Enter)\n"));
           for await (const chunk of process.stdin) val += String(chunk);
           return val.trim();
         }
@@ -336,7 +351,14 @@ async function oneShot(cfg, prompt) {
     },
   });
   await session.init();
-  const out = await session.send(prompt);
+  let out;
+  try {
+    out = await session.send(prompt);
+  } catch (e) {
+    console.error(red(`error: ${e.message || e}`) + "\n");
+    bridge.stop();
+    process.exit(1);
+  }
   process.stdout.write("\n");
   bridge.stop();
   process.exit(out.ok ? 0 : 1);

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "roforge-cli",
-  "version": "0.3.0",
-  "description": "RoForge — Claude-Code-style local AI agent for Roblox Studio. BYOK, zero backend, zero dependencies.",
+  "version": "0.3.2",
+  "description": "RoForge \u2014 Claude-Code-style local AI agent for Roblox Studio. BYOK, zero backend, zero dependencies.",
   "type": "module",
   "bin": {
     "roforge": "bin/roforge.js"

package/src/config.js

@@ -8,8 +8,16 @@ import os from "node:os";
 import path from "node:path";
 import { randomToken } from "./util.js";
 
-export const CONFIG_DIR = process.env.ROFORGE_CONFIG_DIR || path.join(os.homedir(), ".roforge");
-export const CONFIG_FILE = path.join(CONFIG_DIR, "config.json");
+// Resolved lazily so ROFORGE_CONFIG_DIR changes (tests, late env) are honored.
+export function configDir() {
+  return process.env.ROFORGE_CONFIG_DIR || path.join(os.homedir(), ".roforge");
+}
+export function configFile() {
+  return path.join(configDir(), "config.json");
+}
+// (static snapshots — display only; read/write go through configFile())
+export const CONFIG_DIR = configDir();
+export const CONFIG_FILE = configFile();
 
 // Provider registry. "auto" (default) picks the first configured provider,
 // free-tier providers first (gemini → groq → openrouter → anthropic → openai).
@@ -38,7 +46,10 @@ export const PROVIDERS = {
     env: "OPENROUTER_API_KEY",
     baseField: "openrouterBaseUrl",
     defaultModel: "qwen/qwen3-coder",
-    freeModel: "qwen/qwen3-coder:free", // :free models: req/day limit, no billing
+    // Free tier rotates — this is the live free model verified 2026-09-13
+    // via the public OpenRouter models API + a real request. When it dies,
+    // check https://openrouter.ai/models?max_price=0 and update here.
+    freeModel: "nvidia/nemotron-3-super-120b-a12b:free",
     hasFreeTier: true,
   },
   anthropic: {
@@ -106,13 +117,14 @@ const DEFAULTS = {
     "gpt-4o": { input: 2.5, output: 10 },
     "gemini-2.5-flash": { input: 0, output: 0 },
     "llama-3.3-70b-versatile": { input: 0, output: 0 },
-    "qwen/qwen3-coder:free": { input: 0, output: 0 },
+    "nvidia/nemotron-3-super-120b-a12b:free": { input: 0, output: 0 },
+    "nvidia/nemotron-3-ultra-550b-a55b:free": { input: 0, output: 0 },
   },
 };
 
 export function loadFileConfig() {
   try {
-    return JSON.parse(fs.readFileSync(CONFIG_FILE, "utf8"));
+    return JSON.parse(fs.readFileSync(configFile(), "utf8"));
   } catch {
     return {};
   }
@@ -121,8 +133,8 @@ export function loadFileConfig() {
 export function saveFileConfig(patch) {
   const current = loadFileConfig();
   const next = deepMerge(current, patch);
-  fs.mkdirSync(CONFIG_DIR, { recursive: true });
-  fs.writeFileSync(CONFIG_FILE, JSON.stringify(next, null, 2));
+  fs.mkdirSync(configDir(), { recursive: true });
+  fs.writeFileSync(configFile(), JSON.stringify(next, null, 2));
   return next;
 }
 
@@ -134,11 +146,39 @@ function deepMerge(a, b) {
   return out;
 }
 
+// Lenient import: env-var-style keys (GEMINI_API_KEY, OPENROUTER_API_KEY, …)
+// accepted from ANYWHERE in the config file (top-level or nested, e.g. under
+// "bridge"), mapped onto the canonical key fields.
+const ENV_STYLE_KEY_FIELDS = {
+  GEMINI_API_KEY: "geminiKey",
+  GROQ_API_KEY: "groqKey",
+  OPENROUTER_API_KEY: "openrouterKey",
+  ANTHROPIC_API_KEY: "anthropicKey",
+  OPENAI_API_KEY: "openaiKey",
+};
+function collectEnvStyleKeys(node, out) {
+  if (!node || typeof node !== "object") return out;
+  for (const [k, v] of Object.entries(node)) {
+    if (typeof v === "string" && v && ENV_STYLE_KEY_FIELDS[k] && !out[ENV_STYLE_KEY_FIELDS[k]]) {
+      out[ENV_STYLE_KEY_FIELDS[k]] = v;
+    } else if (v && typeof v === "object") {
+      collectEnvStyleKeys(v, out);
+    }
+  }
+  return out;
+}
+
 // Resolved, runtime config (env overrides applied, defaults filled).
 export function resolveConfig() {
   const file = loadFileConfig();
   const cfg = deepMerge(DEFAULTS, file);
 
+  // env-var-style keys anywhere in the file (e.g. {"bridge": {"OPENROUTER_API_KEY": "…"}})
+  const envStyle = collectEnvStyleKeys(file, {});
+  for (const [field, val] of Object.entries(envStyle)) {
+    if (!cfg[field]) cfg[field] = val;
+  }
+
   cfg.geminiKey = process.env.GEMINI_API_KEY || cfg.geminiKey || "";
   cfg.groqKey = process.env.GROQ_API_KEY || cfg.groqKey || "";
   cfg.openrouterKey = process.env.OPENROUTER_API_KEY || cfg.openrouterKey || "";

package/src/providers/openai.js

@@ -3,6 +3,34 @@
 import { createSSE } from "../util.js";
 import { ProviderError } from "./anthropic.js";
 
+// fetch with a per-attempt timeout and ONE automatic retry on network-level
+// failures (UND_ERR_CONNECT_TIMEOUT etc.) — user-initiated aborts are never
+// retried.
+export async function fetchWithRetry(url, opts, { timeoutMs = 60000, retries = 1 } = {}) {
+  let lastErr;
+  for (let attempt = 0; attempt <= retries; attempt++) {
+    if (opts.signal?.aborted) throw lastErr || new ProviderError("aborted");
+    const controller = new AbortController();
+    const timer = setTimeout(() => controller.abort(new Error("timeout")), timeoutMs);
+    const onAbort = () => controller.abort(opts.signal.reason);
+    if (opts.signal) opts.signal.addEventListener("abort", onAbort, { once: true });
+    try {
+      return await fetch(url, { ...opts, signal: controller.signal });
+    } catch (e) {
+      lastErr = e;
+      if (opts.signal?.aborted) break; // user Ctrl+C — don't retry
+      const sig = `${e.cause?.code || ""} ${e.name} ${e.message}`;
+      const isNetwork = /UND_ERR|ECONN|ETIMEDOUT|EAI_AGAIN|EPIPE|EHOSTUNREACH|ENOTFOUND|timeout|aborted/i.test(sig);
+      if (!isNetwork || attempt === retries) break;
+      await new Promise((r) => setTimeout(r, 1200 * (attempt + 1)));
+    } finally {
+      clearTimeout(timer);
+      if (opts.signal) opts.signal.removeEventListener("abort", onAbort);
+    }
+  }
+  throw lastErr;
+}
+
 export async function chatStream(cfg, params, events = {}) {
   const key = cfg.openaiKey;
   if (!key) throw new ProviderError("No OpenAI API key. Run `roforge login` or set OPENAI_API_KEY.");
@@ -17,7 +45,7 @@ export async function chatStream(cfg, params, events = {}) {
 
   let res;
   try {
-    res = await fetch(`${cfg.openaiBaseUrl}/v1/chat/completions`, {
+    res = await fetchWithRetry(`${cfg.openaiBaseUrl}/v1/chat/completions`, {
       method: "POST",
       headers: {
         Authorization: `Bearer ${key}`,
@@ -27,10 +55,20 @@ export async function chatStream(cfg, params, events = {}) {
       signal: params.signal,
     });
   } catch (e) {
-    throw new ProviderError(`network error calling OpenAI: ${e.cause?.code || e.message}`);
+    throw new ProviderError(
+      `network error calling OpenAI: ${e.cause?.code || e.message} (retried once — if it persists, check your connection)`
+    );
   }
   if (!res.ok) {
     const text = await res.text().catch(() => "");
+    // OpenRouter retires free slugs; the 404 body names the paid replacement
+    if (res.status === 404 && /unavailable for free/i.test(text)) {
+      const m = text.match(/use this slug instead:\s*([^\s",}]+)/);
+      throw new ProviderError(
+        `${params.model} was retired from the free tier. Paid slug: ${m ? m[1] : "(see error)"} — ` +
+          "or pick a live free model: https://openrouter.ai/models?max_price=0 (then /model openrouter:<slug>)"
+      );
+    }
     throw new ProviderError(`OpenAI HTTP ${res.status}: ${text.slice(0, 400)}`);
   }
 

package/src/session.js

@@ -32,6 +32,9 @@ export class Session {
     return effectiveProvider(this.cfg) || this.cfg.provider || "auto";
   }
   get provider() {
+    // "auto" with no configured keys → null (caller shows a friendly
+    // "no API key" error instead of silently hitting Anthropic).
+    if (this.providerName === "auto" && !effectiveProvider(this.cfg)) return null;
     return PROVIDER_MODULES[this.providerName] || Anthropic;
   }
   get model() {
@@ -80,6 +83,15 @@ Current state:
   }
 
   async send(userText) {
+    if (!this.provider) {
+      const prov = this.cfg.provider && this.cfg.provider !== "auto" ? this.cfg.provider : "auto";
+      throw new Error(
+        `No API key found for ${prov === "auto" ? "any provider" : prov}. ` +
+          "Run `roforge login --provider <gemini|groq|openrouter|anthropic|openai>` " +
+          "(free keys: aistudio.google.com, console.groq.com, openrouter.ai) or set the " +
+          "matching *_API_KEY env var / key in your config."
+      );
+    }
     this.turns++;
     this.aborted = false;
     this._controller = new AbortController();

package/src/tui/frame.js

@@ -0,0 +1,264 @@
+// LiveRegion — a flicker-free live region at the bottom of the append-scroll
+// output, sized to preserve the terminal scrollback above it (the core trick
+// behind Claude Code's fluid TUI, adapted to an append-scroll layout).
+//
+// Model & cursor invariant:
+//   • Committed history scrolls normally above the region (plain append).
+//   • The region owns the currently-streaming content: every completed
+//     content line is appended (terminal scrolls, nothing is ever lost), and
+//     one status line always sits on the last line, rewritten in place.
+//   • The cursor is ALWAYS at the end of the status line.
+//   • Adding a line = overwrite the status line with the new content line,
+//     then write the status on the fresh line below (single write, no
+//     intermediate clear → no flicker).
+//   • The last (partially streamed) content line is rewritten in place as it
+//     grows: up one line, rewrite, back down, rewrite the status.
+//   • Committing = release(): the region's lines simply become history; the
+//     cursor drops to a fresh line below for the next turn.
+//
+// Long lines are pre-wrapped to the terminal width (segment-aware) so the
+// terminal never auto-wraps and breaks the cursor arithmetic.
+// A terminal resize erases the region; the next update re-renders it.
+
+import { COLUMNS } from "./ansi.js";
+
+const SGR_RESET = "\x1b[0m";
+function sgrFor(attr) {
+  if (!attr) return SGR_RESET;
+  const p = [];
+  if (attr & 1) p.push("1"); // bold
+  if (attr & 2) p.push("2"); // dim
+  if (attr & 4) p.push("31"); // red
+  if (attr & 8) p.push("32"); // green
+  if (attr & 16) p.push("33"); // yellow
+  if (attr & 32) p.push("36"); // cyan
+  if (attr & 64) p.push("35"); // magenta
+  if (attr & 128) p.push("34"); // blue
+  return "\x1b[" + p.join(";") + "m";
+}
+
+export const ATTR = {
+  PLAIN: 0,
+  BOLD: 1,
+  DIM: 2,
+  RED: 4,
+  GREEN: 8,
+  YELLOW: 16,
+  CYAN: 32,
+  MAGENTA: 64,
+  BLUE: 128,
+};
+
+/**
+ * Word-aware wrap for styled lines.
+ * @param {Array<{text: string, attr?: number}>} segments
+ * @param {number} width
+ * @returns {Array<Array<{text: string, attr?: number}>>} wrapped lines
+ */
+export function wrapSegments(segments, width) {
+  width = Math.max(2, width);
+  const lines = [[]];
+  let col = 0;
+  let pendingSpace = 0; // width of whitespace awaiting a word (dropped on wrap)
+  const lastLine = () => lines[lines.length - 1];
+  for (const seg of segments || []) {
+    const text = String(seg.text ?? "");
+    const attr = seg.attr ?? 0;
+    // pieces alternate: word, whitespace, word, ...
+    for (const piece of text.split(/(\s+)/)) {
+      if (!piece) continue;
+      if (/^\s+$/.test(piece)) {
+        // remember; only emitted when a following word lands on this line
+        pendingSpace = piece.length;
+        continue;
+      }
+      let w = piece;
+      while (w.length) {
+        // a pending space only counts when the word isn't starting a fresh line
+        const spaceNeeded = col > 0 && pendingSpace ? pendingSpace : 0;
+        const room = width - col - spaceNeeded;
+        if (w.length <= room) {
+          // whole word fits
+          if (spaceNeeded) {
+            lastLine().push({ text: " ".repeat(spaceNeeded), attr });
+            col += spaceNeeded;
+          }
+          pendingSpace = 0;
+          lastLine().push({ text: w, attr });
+          col += w.length;
+          w = "";
+        } else if (w.length > width) {
+          // unbreakable: longer than a full line — hard-break it
+          if (col > 0 || spaceNeeded) {
+            lines.push([]);
+            col = 0;
+          }
+          pendingSpace = 0;
+          lastLine().push({ text: w.slice(0, width - col), attr });
+          w = w.slice(width - col);
+          lines.push([]);
+          col = 0;
+        } else {
+          // breakable word that doesn't fit the remainder — wrap it whole
+          lines.push([]);
+          col = 0;
+          pendingSpace = 0;
+        }
+      }
+    }
+  }
+  // drop a trailing line that is empty or whitespace-only
+  if (lines.length > 1) {
+    const tail = lines[lines.length - 1];
+    if (!tail.length || tail.every((s) => /^\s*$/.test(s.text))) lines.pop();
+  }
+  return lines.length ? lines : [[]];
+}
+
+function renderLine(segments) {
+  let out = "";
+  let state = 0;
+  for (const seg of segments || []) {
+    const a = seg.attr ?? 0;
+    if (a !== state) {
+      out += sgrFor(a);
+      state = a;
+    }
+    out += String(seg.text ?? "");
+  }
+  if (state) out += SGR_RESET;
+  return out;
+}
+
+export class LiveRegion {
+  /**
+   * @param {(s: string) => void} emit
+   * @param {{ maxRows?: number, enabled?: boolean }} opts
+   *   maxRows is accepted for API compatibility; the append model keeps every
+   *   completed line live (terminal scrollback is the cap), so it is a no-op.
+   */
+  constructor(emit, { maxRows = 6, enabled = true } = {}) {
+    this.emit = emit;
+    this.maxRows = maxRows;
+    this.enabled = enabled && Boolean(process.stdout.isTTY);
+    this.active = false;
+    this._rendered = false;
+    this._shown = 0; // completed content display lines already written
+    this._lastContent = null;
+  }
+
+  begin(_opts) {
+    this.active = true;
+    this._rendered = false;
+    this._shown = 0;
+    this._lastContent = null;
+  }
+
+  get isActive() {
+    return this.active;
+  }
+
+  /**
+   * Update the region.
+   * @param {Array<Array<{text, attr?}>>} contentLines styled logical lines —
+   *   the full content so far, where the LAST line is the partially streamed
+   *   line (grows across calls). Earlier lines are complete.
+   * @param {Array<{text, attr?}>|null} status styled status line (last row)
+   */
+  update(contentLines, status) {
+    if (!this.active) return;
+    this._lastContent = contentLines;
+    const cols = COLUMNS();
+    // wrap every logical line into display lines
+    const display = [];
+    for (const line of contentLines || []) display.push(...wrapSegments(line, cols));
+    const statusLine = status ? wrapSegments(status, cols - 1).slice(0, 1)[0] || [] : [];
+    const C = display.length;
+
+    if (!this._rendered) {
+      // first render: append everything from the current cursor position.
+      // The first line is written at the cursor (col 0 of a fresh line, or
+      // right after a sameLine header) — no leading newline.
+      const rows = [...display, ...(statusLine.length ? [statusLine] : [])];
+      let out = "";
+      rows.forEach((r, i) => {
+        if (i > 0) out += "\n";
+        out += renderLine(r) + "\x1b[K";
+      });
+      if (out) this.emit(out);
+      this._rendered = true;
+      this._shown = C;
+      this._hadStatus = statusLine.length > 0;
+      return;
+    }
+
+    const newLines = display.slice(this._shown);
+    const lastLine = C > 0 ? display[C - 1] : null;
+    let out = "";
+    if (newLines.length === 0 && C === 0 && !statusLine.length) {
+      // nothing to show and nothing stale to clear — but a previously
+      // rendered status line must be blanked
+      if (this._hadStatus) {
+        out = "\x1b[1G\x1b[K";
+        this._hadStatus = false;
+      }
+      if (out) this.emit(out);
+      return;
+    }
+    if (newLines.length > 0) {
+      // New completed line(s): the cursor sits at the end of the status line.
+      // Overwrite it with the first new line, push the rest below, then write
+      // the status on the fresh bottom line. One write, no flicker.
+      out += "\x1b[1G";
+      for (const nl of newLines) out += renderLine(nl) + "\x1b[K\n";
+      if (statusLine.length) out += renderLine(statusLine) + "\x1b[K";
+      this._shown = C;
+      this._hadStatus = statusLine.length > 0;
+    } else if (C > 0 || statusLine.length) {
+      // No new lines: the last content line may have grown (or only the
+      // status changed). Rewrite the last content line in place, then the
+      // status.
+      if (C > 0) {
+        out += "\x1b[1A\x1b[1G"; // up to the last content line
+        out += renderLine(lastLine) + "\x1b[K"; // rewrite it (clear stale tail)
+        out += "\n"; // back down to the status line
+      } else {
+        out += "\x1b[1G";
+      }
+      out += renderLine(statusLine) + "\x1b[K"; // clears stale status text too
+      this._hadStatus = statusLine.length > 0;
+    }
+    if (out) this.emit(out);
+  }
+
+  /** Final update + release; region lines become committed history. */
+  end(status) {
+    if (!this.active) return;
+    if (status) this.update(this._lastContent || [], status);
+    this.release();
+  }
+
+  /** Release: drop the cursor to a fresh line below; lines become history. */
+  release() {
+    if (this.active && this._rendered) this.emit("\n");
+    this.active = false;
+    this._rendered = false;
+    this._shown = 0;
+  }
+
+  /** Erase the region on terminal resize (stale width would be wrong). */
+  eraseOnResize() {
+    if (!this.active || !this._rendered) return;
+    const h = this._shown + 1; // content lines + status
+    const out = ["\x1b[" + Math.max(0, h - 1) + "A", "\x1b[1G"];
+    for (let i = 0; i < h; i++) {
+      out.push("\x1b[2K");
+      if (i < h - 1) out.push("\n");
+    }
+    // return to the region top so the next update re-renders in place
+    out.push("\x1b[" + Math.max(0, h - 1) + "A");
+    this.emit(out.join(""));
+    this._rendered = false;
+    this._shown = 0;
+  }
+}

package/src/tui/markdown.js

@@ -3,7 +3,23 @@
 // code-fence state across deltas, and resets cleanly per assistant segment.
 // Deliberately small: headings, bullets, numbered lists, bold, inline code,
 // fences, blockquotes, rules. Everything else passes through untouched.
-import { bold, dim, cyan, gray } from "./ansi.js";
+//
+// Two output modes:
+//   push()/finish()    — ANSI strings (piped / non-TTY path, legacy rendering)
+//   pushLines() & co   — styled segment lines (Array<{text, attr}>) for the
+//                        LiveRegion, which wraps them to terminal width
+//                        itself while preserving per-segment style.
+import { bold, dim, cyan } from "./ansi.js";
+import { ATTR } from "./frame.js";
+
+const A = ATTR;
+
+// attr → string-mode ANSI helper (respects NO_COLOR / --no-color at call time)
+const STRING_STYLE = {
+  [A.BOLD]: bold,
+  [A.DIM]: dim,
+  [A.CYAN]: cyan,
+};
 
 export class MarkdownStream {
   constructor() {
@@ -11,57 +27,129 @@ export class MarkdownStream {
     this.inFence = false;
   }
 
-  // Consume a chunk; returns the renderable output for completed lines.
+  // --- string mode (piped output) -------------------------------------------
+
+  // Consume a chunk; returns the rendered output for completed lines.
   push(delta) {
+    const lines = this.pushLines(delta);
+    if (!lines.length) return "";
+    return lines.map((segs) => segsToAnsi(segs)).join("\n") + "\n";
+  }
+
+  // Flush the incomplete trailing line (at end of an assistant segment).
+  finish() {
+    const segs = this.finishLines();
+    return segs ? segsToAnsi(segs) + "\n" : "";
+  }
+
+  // --- segment mode (LiveRegion) --------------------------------------------
+
+  // Consume a chunk; returns styled lines for every line that COMPLETED in
+  // this delta (each line = Array<{text, attr}>). The partial trailing line
+  // stays buffered; use partialLines() to render it.
+  pushLines(delta) {
     this.buf += String(delta ?? "");
-    let out = "";
+    const out = [];
     let idx;
     while ((idx = this.buf.indexOf("\n")) !== -1) {
       const line = this.buf.slice(0, idx);
       this.buf = this.buf.slice(idx + 1);
-      out += this._renderLine(line) + "\n";
+      out.push(this._renderLineSegs(line));
     }
     return out;
   }
 
-  // Flush the incomplete trailing line (at end of an assistant segment).
-  finish() {
-    if (!this.buf.length) return "";
+  // Styled segments for the currently buffered (incomplete) line, or null.
+  partialLines() {
+    if (!this.buf.length) return null;
+    return this._renderLineSegs(this.buf);
+  }
+
+  // Flush the incomplete trailing line as segments (or null).
+  finishLines() {
+    if (!this.buf.length) return null;
     const line = this.buf;
     this.buf = "";
-    return this._renderLine(line) + "\n";
+    return this._renderLineSegs(line);
   }
 
-  _renderLine(line) {
+  // --- shared core ------------------------------------------------------------
+
+  _renderLineSegs(line) {
     if (/^\s*(```|~~~)/.test(line)) {
       this.inFence = !this.inFence;
-      return dim("  " + line.trim());
+      return [{ text: "  " + line.trim(), attr: A.DIM }];
     }
     if (this.inFence) {
-      return dim("  " + line);
+      return [{ text: "  " + line, attr: A.DIM }];
     }
-    return this._renderPlain(line);
+    return this._renderPlainSegs(line);
+  }
+
+  _pushSeg(line, text, attr) {
+    if (!text) return;
+    const last = line[line.length - 1];
+    if (last && last.attr === attr) last.text += text;
+    else line.push({ text, attr: attr || 0 });
   }
 
-  _renderPlain(line) {
+  _renderPlainSegs(line) {
     const h = line.match(/^(#{1,4})\s+(.*)$/);
-    if (h) return bold(h[1] + " " + this._inline(h[2]));
+    if (h) {
+      const out = [{ text: h[1] + " ", attr: A.BOLD }];
+      for (const seg of this._inlineSegs(h[2])) {
+        // heading text is bold by default; code inside keeps its own style
+        this._pushSeg(out, seg.text, seg.attr === A.CYAN ? seg.attr : A.BOLD);
+      }
+      return out;
+    }
     if (/^\s*([-*+])\s+/.test(line)) {
-      return cyan("•  ") + this._inline(line.replace(/^\s*[-*+]\s+/, ""));
+      const out = [{ text: "•  ", attr: A.CYAN }];
+      for (const seg of this._inlineSegs(line.replace(/^\s*[-*+]\s+/, ""))) this._pushSeg(out, seg.text, seg.attr);
+      return out;
     }
     const num = line.match(/^\s*(\d+)[.)]\s+(.*)$/);
-    if (num) return gray(num[1] + ".") + " " + this._inline(num[2]);
+    if (num) {
+      const out = [{ text: num[1] + ". ", attr: A.DIM }];
+      for (const seg of this._inlineSegs(num[2])) this._pushSeg(out, seg.text, seg.attr);
+      return out;
+    }
     if (/^\s*>\s?/.test(line)) {
-      return dim("│ ") + dim(this._inline(line.replace(/^\s*>\s?/, "")));
+      const out = [{ text: "│ ", attr: A.DIM }];
+      for (const seg of this._inlineSegs(line.replace(/^\s*>\s?/, ""))) this._pushSeg(out, seg.text, A.DIM);
+      return out;
     }
-    if (/^\s*([-*_])\1{2,}\s*$/.test(line)) return dim("────────────────────────────────");
-    if (!line.trim()) return "";
-    return this._inline(line);
+    if (/^\s*([-*_])\1{2,}\s*$/.test(line)) {
+      return [{ text: "────────────────────────────────", attr: A.DIM }];
+    }
+    if (!line.trim()) return [];
+    return this._inlineSegs(line);
+  }
+
+  // Split inline text into plain / code / bold segments.
+  _inlineSegs(s) {
+    const out = [];
+    // tokenize: `code`, **bold**, and plain runs
+    const re = /(`[^`]+`|\*\*[^*]+\*\*)/g;
+    let last = 0;
+    let m;
+    while ((m = re.exec(s)) !== null) {
+      if (m.index > last) this._pushSeg(out, s.slice(last, m.index), A.PLAIN);
+      const tok = m[0];
+      if (tok.startsWith("`")) this._pushSeg(out, tok.slice(1, -1), A.CYAN);
+      else this._pushSeg(out, tok.slice(2, -2), A.BOLD);
+      last = m.index + tok.length;
+    }
+    if (last < s.length) this._pushSeg(out, s.slice(last), A.PLAIN);
+    return out;
   }
+}
 
-  _inline(s) {
-    s = s.replace(/`([^`]+)`/g, (_, c) => cyan(c));
-    s = s.replace(/\*\*([^*]+)\*\*/g, (_, b) => bold(b));
-    return s;
+export function segsToAnsi(segs) {
+  let out = "";
+  for (const seg of segs || []) {
+    const style = STRING_STYLE[seg.attr] || ((t) => t);
+    out += style(seg.text);
   }
+  return out;
 }

package/src/tui/tui.js

@@ -1,14 +1,35 @@
 // RoForge TUI — Claude-Code-style interactive terminal session.
-// Append-style rendering (terminal scrollback preserved), single status line
-// with a spinner, approval prompts, slash commands, input history.
+// Append-style rendering (terminal scrollback preserved).
+//
+// Two render paths, picked once at startup:
+//   • LIVE (stdout is a TTY): the streaming block (markdown + status line) is
+//     a LiveRegion — rewritten in place every frame, zero flicker, history
+//     committed above it. See frame.js.
+//   • LEGACY (piped output / tests): plain append + \r-spinner, exactly the
+//     pre-LiveRegion behavior.
+import { createRequire } from "node:module";
 import { bold, dim, red, green, yellow, cyan, magenta, gray, wrap, SPINNER_FRAMES, CLEAR_LINE } from "./ansi.js";
 import { parseModelRef, PROVIDERS } from "../config.js";
-import { MarkdownStream } from "./markdown.js";
+import { MarkdownStream, segsToAnsi } from "./markdown.js";
+import { LiveRegion, ATTR } from "./frame.js";
+
+const VERSION = (() => {
+  try {
+    return createRequire(import.meta.url)("../../package.json").version;
+  } catch {
+    return "dev";
+  }
+})();
 
-const VERSION = "0.2.0";
+// Known model names (for a soft /model warning; anything else is allowed).
+const KNOWN_MODELS = new Set();
+for (const p of Object.values(PROVIDERS)) {
+  if (p.defaultModel) KNOWN_MODELS.add(p.defaultModel);
+  if (p.freeModel) KNOWN_MODELS.add(p.freeModel);
+}
 
 export class TUI {
-  constructor(session, { out = process.stdout, err = process.stderr } = {}) {
+  constructor(session, { out = process.stdout, err = process.stderr, live } = {}) {
     this.session = session;
     this.out = out;
     this.err = err;
@@ -24,7 +45,16 @@ export class TUI {
     this.spinnerVisible = false;
     this.ctrlCTime = 0;
     this.running = false;
+
+    // LiveRegion (live path). `live` overrides detection (tests).
+    this.live = new LiveRegion((s) => this.out.write(s), { maxRows: 6 });
+    this._liveOK =
+      live === undefined ? Boolean(process.stdout.isTTY) && this.out === process.stdout : live;
     this._md = null; // active MarkdownStream for the current assistant segment
+    this._segLines = []; // completed styled lines for the current segment
+    this._statusLabel = "thinking…";
+    this._costStatus = null; // final cost line for this turn (plain text)
+    this._lastLiveSig = null;
     this._toolOutputs = []; // recent tool outputs, expandable via /out
     this._toolOutSeq = 0;
   }
@@ -71,6 +101,8 @@ export class TUI {
         // Ctrl+C
         if (this.busy) {
           this.session.abort();
+          this._stopSpinner();
+          this._liveCommit();
           this.out.write("\r\n" + yellow("aborted — type a new message or /exit\n"));
           continue;
         }
@@ -132,6 +164,17 @@ export class TUI {
       this._slash(line);
       return;
     }
+    // Shell commands typed into the TUI go to the model and fail
+    // confusingly — catch the common ones and point at the terminal.
+    if (/^(roforge|npm|node|npx|git)\b(\s|$)/.test(line)) {
+      this.out.write(
+        yellow(
+          "that's a shell command, not a chat message — /exit first, then run it in your terminal " +
+            "(e.g. `roforge login`). In the TUI use /help for commands.\n"
+        )
+      );
+      return;
+    }
     this.history.push(line);
     this.historyIndex = -1;
     this._runTurn(line);
@@ -193,6 +236,11 @@ export class TUI {
           }
           const free = PROVIDERS[this.session.providerName]?.hasFreeTier ? dim(" · free tier") : "";
           this.out.write(`model → ${this.session.cfg._activeModel} (${this.session.providerName}${free})\n`);
+          if (!ref && !KNOWN_MODELS.has(arg)) {
+            this.out.write(
+              dim(`  (unrecognized model name — double-check the spelling, or pin explicitly: /model provider:model, e.g. /model gemini:2.5-flash)\n`)
+            );
+          }
         } else {
           const free = PROVIDERS[this.session.providerName]?.hasFreeTier ? dim(" · free tier") : "";
           this.out.write(`current: ${this.session.model} (${this.session.providerName}${free})\n`);
@@ -269,12 +317,18 @@ export class TUI {
   async _runTurn(text) {
     this.out.write(dim("you> ") + text + "\n");
     this.busy = true;
-    this._md = null;
+    this._resetSegment();
+    this._costStatus = null;
+    this._lastLiveSig = null;
     try {
       await this.session.send(text);
     } catch (e) {
+      this._stopSpinner();
+      this._liveCommit();
       this.out.write(red(`error: ${e.message || e}`) + "\n");
     }
+    // commit the live block with the final cost line as its status
+    this._liveCommit(this._costStatus ? [{ text: this._costStatus, attr: ATTR.DIM }] : null);
     this.busy = false;
     this._printPrompt();
   }
@@ -284,17 +338,92 @@ export class TUI {
     this.out.write("> ");
   }
 
+  // ---------------- live-region plumbing ----------------
+
+  // Region content for the current assistant segment: completed lines + the
+  // partial line, with the magenta "RoForge> " header on the first
+  // non-empty line (leading blank lines from the model stay blank).
+  _regionLines() {
+    const lines = [...this._segLines];
+    const partial = this._md ? this._md.partialLines() : null;
+    if (partial) lines.push(partial);
+    const first = lines.findIndex((l) => l && l.some((s) => s.text));
+    if (first === -1) return [];
+    lines[first] = [{ text: "RoForge> ", attr: ATTR.MAGENTA }, ...lines[first]];
+    return lines;
+  }
+
+  _statusSegs() {
+    if (this._costStatus) return [{ text: this._costStatus, attr: ATTR.DIM }];
+    const frame = SPINNER_FRAMES[this.spinnerFrame];
+    return [{ text: frame + " " + this._statusLabel, attr: ATTR.DIM }];
+  }
+
+  // Redraw the live region (no-op when nothing changed since the last frame).
+  _liveRefresh() {
+    if (!this._liveOK || !this.live.isActive) return;
+    const lines = this._regionLines();
+    const status = this._statusSegs();
+    const last = lines.length ? lines[lines.length - 1] : null;
+    const sig =
+      lines.length +
+      ":" +
+      (last ? last.map((s) => s.text).join("") : "") +
+      "|" +
+      status.map((s) => s.text).join("");
+    if (sig === this._lastLiveSig) return;
+    this._lastLiveSig = sig;
+    this.live.update(lines, status);
+  }
+
+  // Commit the live block to history. statusSegs = final status line, or null
+  // to commit with a blank one (acts as a separator).
+  _liveCommit(statusSegs = null) {
+    if (!this._liveOK || !this.live.isActive) return;
+    this._finalizeMd();
+    this.live.update(this._regionLines(), statusSegs || []);
+    this.live.release();
+    this._lastLiveSig = null;
+  }
+
+  _finalizeMd() {
+    if (this._md) {
+      const tail = this._md.finishLines();
+      if (tail) this._segLines.push(tail);
+      this._md = null;
+    }
+  }
+
+  // Start a fresh assistant segment (clears streamed content + markdown).
+  // The "running tool" region and gaps between segments show no content, so
+  // the stale block never re-renders in a new region.
+  _resetSegment() {
+    this._md = null;
+    this._segLines = [];
+  }
+
+  // ---------------- spinner ----------------
+
   _startSpinner(label = "thinking…") {
-    if (!process.stdout.isTTY) return;
     this._stopSpinner();
-    this.spinnerVisible = true;
-    this.out.write(label);
-    this.spinnerTimer = setInterval(() => {
-      this.spinnerFrame = (this.spinnerFrame + 1) % SPINNER_FRAMES.length;
-      const len = label.length + 3;
-      this.out.write("\r" + CLEAR_LINE + this.spinnerFrame + " " + label.slice(0, Math.max(0, len - 2)));
-    }, 90);
-    this.spinnerTimer.unref && this.spinnerTimer.unref();
+    this._statusLabel = label;
+    if (this._liveOK) {
+      this.spinnerTimer = setInterval(() => {
+        this.spinnerFrame = (this.spinnerFrame + 1) % SPINNER_FRAMES.length;
+        if (this.live.isActive) this._liveRefresh();
+      }, 90);
+      this.spinnerTimer.unref && this.spinnerTimer.unref();
+      this._liveRefresh();
+    } else if (process.stdout.isTTY) {
+      this.spinnerVisible = true;
+      this.out.write(label);
+      this.spinnerTimer = setInterval(() => {
+        this.spinnerFrame = (this.spinnerFrame + 1) % SPINNER_FRAMES.length;
+        const len = label.length + 3;
+        this.out.write("\r" + CLEAR_LINE + this.spinnerFrame + " " + label.slice(0, Math.max(0, len - 2)));
+      }, 90);
+      this.spinnerTimer.unref && this.spinnerTimer.unref();
+    }
   }
 
   _stopSpinner() {
@@ -310,32 +439,42 @@ export class TUI {
 
   // ---------------- ui event sink (Session) ----------------
 
-  _flushMd() {
-    if (this._md) {
-      const tail = this._md.finish();
-      if (tail) this.out.write(tail);
-      this._md = null;
-    }
-  }
-
   onText(delta) {
     this._stopSpinner();
-    if (!this._assistantHeaderShown) {
-      this.out.write(magenta("RoForge> ") );
-      this._assistantHeaderShown = true;
-    }
     if (!this._md) this._md = new MarkdownStream();
-    const rendered = this._md.push(delta);
-    if (rendered) this.out.write(rendered);
+    const newLines = this._md.pushLines(delta);
+    for (const l of newLines) this._segLines.push(l);
+    if (this._liveOK) {
+      if (!this.live.isActive) this.live.begin();
+      this._liveRefresh();
+    } else {
+      if (!this._assistantHeaderShown) {
+        this.out.write(magenta("RoForge> "));
+        this._assistantHeaderShown = true;
+      }
+      if (newLines.length) this.out.write(newLines.map(segsToAnsi).join("\n") + "\n");
+    }
   }
 
   onAssistantDone() {
-    this._flushMd();
+    if (this._md) {
+      const tail = this._md.finishLines();
+      this._md = null;
+      if (tail) {
+        if (this._liveOK) {
+          this._segLines.push(tail);
+          this._liveRefresh();
+        } else {
+          this.out.write(segsToAnsi(tail) + "\n");
+        }
+      }
+    }
   }
 
   onToolStart(tool, args) {
     this._stopSpinner();
-    this._flushMd();
+    this._liveCommit(); // assistant block → history (blank separator)
+    this._resetSegment(); // running region shows status only, no content
     let argsStr;
     try {
       argsStr = JSON.stringify(args || {});
@@ -347,7 +486,8 @@ export class TUI {
     this._toolOutputs.push({ id: this._toolOutSeq, tool: tool.name, args: argsStr, full: "" });
     if (this._toolOutputs.length > 30) this._toolOutputs.shift();
     this.out.write(dim(`  ⚙ [${this._toolOutSeq}] ${tool.name}(${argsStr})`) + "\n");
-    this._startSpinner(dim("running " + tool.name + "…"));
+    if (this._liveOK) this.live.begin();
+    this._startSpinner("running " + tool.name + "…");
   }
 
   onToolEnd(tool, args, result) {
@@ -355,6 +495,8 @@ export class TUI {
     const r = String(result || "");
     const last = this._toolOutputs[this._toolOutputs.length - 1];
     if (last && last.tool === tool.name) last.full = r;
+    this._liveCommit(); // "running…" block → history
+    this._resetSegment(); // next assistant text starts a fresh segment
     const first = r.split("\n")[0].slice(0, 120);
     const more = (r.length > 120 || r.includes("\n")) && last ? dim(` (more: /out ${last.id})`) : "";
     if (r.startsWith("ERROR")) {
@@ -362,27 +504,45 @@ export class TUI {
     } else {
       this.out.write(dim(`    ↳ ${first}`) + more + "\n");
     }
+    if (this._liveOK) this.live.begin();
     this._startSpinner("thinking…");
   }
 
   onInfo(msg) {
     this._stopSpinner();
+    this._liveCommit();
     this.out.write(gray(msg) + "\n");
   }
 
   onWarn(msg) {
     this._stopSpinner();
+    this._liveCommit();
     this.out.write(red(msg) + "\n");
   }
 
   onStatus(msg) {
-    // Only the per-turn cost footer is printed here; the spinner covers
-    // "thinking…" and tool progress is shown on its own lines.
-    if (String(msg).includes("tok")) this.out.write("\n" + gray(msg) + "\n");
+    const m = String(msg);
+    if (m.includes("tok")) {
+      // per-turn cost footer — becomes the live block's final status line
+      this._costStatus = m;
+      if (this._liveOK && this.live.isActive) {
+        this._stopSpinner();
+        this._liveRefresh();
+      } else if (!this._liveOK) {
+        this.out.write("\n" + gray(m) + "\n");
+      }
+      return;
+    }
+    // "thinking… (step n/total)" / "done" → live status label
+    if (this._liveOK && this.live.isActive) {
+      this._statusLabel = m === "done" ? "finishing…" : m;
+      this._liveRefresh();
+    }
   }
 
   async promptApproval(name, args) {
     this._stopSpinner();
+    this._liveCommit();
     let target = "";
     try {
       target = JSON.stringify(args || {});
@@ -391,7 +551,9 @@ export class TUI {
     }
     if (target.length > 100) target = target.slice(0, 97) + "…";
     this.out.write(yellow(`  ✋ approve ${name}(${target})? `) + dim("[y]es / [n]o / [a]lways "));
-    return await this._readChar();
+    const ans = await this._readChar();
+    this.out.write("\n"); // next output starts on a fresh line
+    return ans;
   }
 
   _readChar() {
@@ -446,6 +608,15 @@ export class TUI {
     if (process.stdin.isTTY) {
       this._setRaw(true);
       process.stdin.on("data", (c) => this._onData(c));
+      if (this._liveOK) {
+        this._onResize = () => {
+          if (this.live.isActive) {
+            this.live.eraseOnResize();
+            this._lastLiveSig = null;
+          }
+        };
+        process.stdout.on("resize", this._onResize);
+      }
       this._printPrompt();
       return true;
     }
@@ -457,6 +628,11 @@ export class TUI {
   stop() {
     this.running = false;
     this._stopSpinner();
+    this._liveCommit();
+    if (this._onResize) {
+      process.stdout.removeListener("resize", this._onResize);
+      this._onResize = null;
+    }
     this._setRaw(false);
     process.stdin.removeAllListeners("data");
   }