roforge-cli 0.3.0 → 0.3.1

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.1",
+  "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).
@@ -112,7 +120,7 @@ const DEFAULTS = {
 
 export function loadFileConfig() {
   try {
-    return JSON.parse(fs.readFileSync(CONFIG_FILE, "utf8"));
+    return JSON.parse(fs.readFileSync(configFile(), "utf8"));
   } catch {
     return {};
   }
@@ -121,8 +129,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 +142,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/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/tui.js

@@ -1,11 +1,25 @@
 // 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.
+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";
 
-const VERSION = "0.2.0";
+const VERSION = (() => {
+  try {
+    return createRequire(import.meta.url)("../../package.json").version;
+  } catch {
+    return "dev";
+  }
+})();
+
+// 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 } = {}) {
@@ -132,6 +146,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 +218,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: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`);