aether-code 0.23.0 → 0.25.0

package/bin/aether-code.js

@@ -9,6 +9,8 @@
 
 import process from "node:process";
 import path from "node:path";
+import os from "node:os";
+import fs from "node:fs";
 import { runAgent } from "../src/agent.js";
 import { runRepl } from "../src/repl.js";
 import { runSetup } from "../src/setup.js";
@@ -26,7 +28,7 @@ import {
 import readline from "node:readline";
 import { c, errorLine, divider, setTerminalTitle } from "../src/render.js";
 
-const VERSION = "0.23.0";
+const VERSION = "0.25.0";
 
 /**
  * Try to start MCP servers from ~/.aether/mcp.json. Returns a started
@@ -132,6 +134,18 @@ function die(msg, code = 1) {
   process.exit(code);
 }
 
+// True for Windows system folders we should never write a user's project into.
+function isSystemDir(p) {
+  const low = p.toLowerCase().replace(/[\\/]+$/, "").replace(/\//g, "\\");
+  return (
+    low.endsWith("\\windows\\system32") ||
+    low.endsWith("\\windows\\syswow64") ||
+    low.endsWith("\\windows") ||
+    low.endsWith("\\system32") ||
+    /\\program files( \(x86\))?$/.test(low)
+  );
+}
+
 async function main() {
   const args = parseArgs(process.argv.slice(2));
   setTerminalTitle("Aether");
@@ -145,7 +159,16 @@ async function main() {
     return;
   }
 
-  const cwd = args.flags.cwd ? path.resolve(args.flags.cwd) : process.cwd();
+  // Where files land. If the user didn't pass --cwd and we were launched from a
+  // Windows system folder (admin cmd opens in C:\WINDOWS\system32), don't dump
+  // their project there — they'd never find it. Work in the Desktop instead.
+  let cwd = args.flags.cwd ? path.resolve(args.flags.cwd) : process.cwd();
+  let redirectedFrom = null;
+  if (!args.flags.cwd && isSystemDir(cwd)) {
+    const desktop = path.join(os.homedir(), "Desktop");
+    const target = fs.existsSync(desktop) ? desktop : os.homedir();
+    if (target.toLowerCase() !== cwd.toLowerCase()) { redirectedFrom = cwd; cwd = target; }
+  }
   // Skip-permissions by DEFAULT (like `claude --dangerously-skip-permissions`):
   // auto-approve writes/shell and don't sandbox file paths, so the agent can
   // just build. Opt back in with --review (y/N before each action) and
@@ -184,6 +207,10 @@ async function main() {
   if (!prompt) {
     if (cwd !== process.cwd()) process.chdir(cwd);
     const mcpManager = await bootMcp();
+    if (redirectedFrom) {
+      console.log(c.gray(`Note: launched from a system folder (${redirectedFrom}).`));
+      console.log(c.gray(`Your files will be saved to ${c.cyan(cwd)} so you can find them. Use ${c.cyan("--cwd <path>")} or ${c.cyan("/cwd")} to change.`));
+    }
     await runRepl({ cwd, autoYes, unsafePaths, maxTurns, mcpManager });
     return;
   }
@@ -198,6 +225,7 @@ async function main() {
   console.log(divider());
   const modeLabel = autoYes && unsafePaths ? " · skip-permissions" : `${autoYes ? " · auto-yes" : " · review"}${unsafePaths ? "" : " · sandboxed"}`;
   console.log(c.magenta(c.bold("aether-code")) + c.gray(` · cwd ${cwd}${modeLabel}`));
+  if (redirectedFrom) console.log(c.gray(`(launched from ${redirectedFrom} — saving to ${cwd} instead)`));
   console.log(c.gray(`task: `) + prompt);
   console.log(divider());
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "aether-code",
-  "version": "0.23.0",
+  "version": "0.25.0",
   "description": "Uncensored AI coding agent for your terminal — Claude Code alternative with MCP support. Reads code, writes files, runs commands. Drives IDA Pro, Roblox Studio, Wireshark, Blender, and any MCP server. No refusal layer.",
   "homepage": "https://trynoguard.com",
   "repository": {

package/src/agent.js

@@ -8,7 +8,7 @@ import { agentTurnStream, AetherError } from "./api.js";
 import { TOOL_DEFINITIONS, executeTool } from "./tools.js";
 import { unnamespaceToolName } from "./mcp.js";
 import { loadAllSkills, selectSkills, renderSkillsBlock } from "./skills.js";
-import { c, divider, turn, toolLabel, toolSummary, makeTokenStripper, errorLine } from "./render.js";
+import { c, divider, turn, toolLabel, toolSummary, makeTokenStripper, errorLine, startSpinner } from "./render.js";
 
 const DEFAULT_MAX_TURNS = 25;
 
@@ -77,6 +77,12 @@ export async function runAgent({
   let totalIn = 0;
   let totalOut = 0;
   let lastBalance = null;
+  // Loop guard: count identical (name+args) tool calls across the whole run so a
+  // confused model can't burn turns re-running the same call (e.g. glob *.md x9).
+  const callCounts = new Map();
+  // Files the run created/edited, shown in a summary at the end so the user
+  // knows exactly what was produced and where to find it.
+  const filesTouched = new Map();
 
   for (let i = 0; i < maxTurns; i++) {
     // No turn header and no leading blank here — each step (assistant text and
@@ -98,6 +104,11 @@ export async function runAgent({
     const turnMessages = buildTurnMessages(messages, allSkills, referencedPaths);
 
     let res;
+    // "Thinking" spinner: shown from request-send until the first token or tool
+    // call arrives, so the wait doesn't look dead. Stopped exactly once.
+    const spinner = startSpinner("thinking");
+    let spinStopped = false;
+    const stopSpin = () => { if (!spinStopped) { spinStopped = true; spinner.stop(); } };
     try {
       res = await agentTurnStream({
         messages: turnMessages,
@@ -107,6 +118,7 @@ export async function runAgent({
           // be split across stream chunks) before display.
           const clean = stripper.push(text);
           if (!clean) return;
+          stopSpin();
           if (!lastWasText) {
             process.stdout.write("\n" + c.cyan("● "));
             lastWasText = true;
@@ -118,6 +130,7 @@ export async function runAgent({
           // call — the clean label is printed at execution time, so no noisy
           // "preparing args" placeholder here.
           if (delta.name && !announced.has(delta.index)) {
+            stopSpin();
             announced.add(delta.index);
             if (lastWasText) process.stdout.write("\n");
             lastWasText = false;
@@ -125,11 +138,13 @@ export async function runAgent({
         },
       });
     } catch (err) {
+      stopSpin();
       if (err instanceof AetherError) {
         return { ok: false, error: err, totalCredits, totalIn, totalOut, balance: lastBalance, messages };
       }
       throw err;
     }
+    stopSpin(); // ensure it's cleared even if the turn produced no output
 
     // Flush any held-back partial token, then close the line.
     const tail = stripper.flush();
@@ -155,6 +170,12 @@ export async function runAgent({
 
     const toolCalls = res.message.tool_calls ?? [];
     if (toolCalls.length === 0) {
+      if (filesTouched.size) {
+        console.log("\n" + c.bold("Files") + c.gray(` in ${cwd}`));
+        for (const [p, action] of filesTouched) {
+          console.log("  " + c.green("●") + " " + p + c.gray(` — ${action}`));
+        }
+      }
       return { ok: true, totalCredits, totalIn, totalOut, turns: i + 1, balance: lastBalance, messages };
     }
 
@@ -167,14 +188,37 @@ export async function runAgent({
       const label = toolLabel(call.function.name, args);
       if (label) console.log("\n" + c.cyan("●") + " " + label);
 
+      // Loop guard — short-circuit a tool call we've already run 3+ times with
+      // the same args, and tell the model to change approach instead of looping.
+      const sig = `${call.function.name}:${call.function.arguments || ""}`;
+      const seen = (callCounts.get(sig) || 0) + 1;
+      callCounts.set(sig, seen);
+      if (seen > 3) {
+        console.log("  " + c.red("└─") + " " + c.gray("skipped (repeated call)"));
+        messages.push({
+          role: "tool",
+          tool_call_id: call.id,
+          content:
+            "STOP: you have already run this exact tool call 3 times and the result will not change. Do NOT call it again. " +
+            "If you were looking for a file that doesn't exist, create it with write_file. Otherwise change approach or finish and summarize.",
+        });
+        continue;
+      }
+
       // Route to MCP if the tool name is namespaced (mcp__server__tool);
       // otherwise execute the built-in tool. unnamespaceToolName returns
       // null for non-MCP names, which is our cheap dispatch test.
+      const slow = call.function.name === "web_search" || call.function.name === "web_fetch";
+      const tspin = slow ? startSpinner(call.function.name === "web_search" ? "searching the web" : "fetching page") : null;
       let result;
-      if (mcpManager && unnamespaceToolName(call.function.name)) {
-        result = await mcpManager.callTool(call.function.name, args);
-      } else {
-        result = await executeTool(call, { cwd, autoYes, unsafePaths });
+      try {
+        if (mcpManager && unnamespaceToolName(call.function.name)) {
+          result = await mcpManager.callTool(call.function.name, args);
+        } else {
+          result = await executeTool(call, { cwd, autoYes, unsafePaths });
+        }
+      } finally {
+        if (tspin) tspin.stop();
       }
 
       // Track paths the model has touched. Skills with path-pattern triggers
@@ -183,6 +227,10 @@ export async function runAgent({
       if (call.function.name === "read_file" || call.function.name === "edit_file" || call.function.name === "write_file") {
         if (typeof args.path === "string") referencedPaths.push(args.path);
       }
+      if (result.ok && typeof args.path === "string" && (call.function.name === "write_file" || call.function.name === "edit_file")) {
+        const rel = path.relative(cwd, path.resolve(cwd, args.path)) || args.path;
+        filesTouched.set(rel, call.function.name === "write_file" ? "created" : "edited");
+      }
       const summary = toolSummary(call.function.name, result);
       if (summary) console.log(summary);
 

package/src/render.js

@@ -63,7 +63,7 @@ export function toolLabel(name, args) {
     case "write_file":   return `${verb("write")}  ${arg(a.path)}`;
     case "edit_file":    return `${verb("edit")}   ${arg(a.path)}`;
     case "list_dir":     return `${verb("list")}   ${arg(a.path)}`;
-    case "glob_files":   return `${verb("glob")}   ${arg(a.pattern)}`;
+    case "glob_files":   return `${verb("find")}   ${arg(a.pattern)}`;
     case "search_files": return `${verb("search")} ${arg(`/${a.pattern}/ in ${a.path ?? "."}`)}`;
     case "run_shell":    return `${verb("run")}    ${arg(a.command)}`;
     case "web_search":   return `${verb("search web")} ${arg(JSON.stringify(a.query ?? ""))}`;
@@ -80,6 +80,33 @@ export function setTerminalTitle(title) {
   if (process.stdout.isTTY) process.stdout.write(`\x1b]0;${title}\x07`);
 }
 
+// Live "working…" spinner. Rewrites a single line in place (\r) with a rotating
+// half-circle (Geometric Shapes — same block as ●, so cmd.exe-safe) + elapsed
+// seconds, so the user sees the agent is alive while the model thinks or a slow
+// tool runs. stop() clears the line and restores the cursor. No-op in non-TTY.
+export function startSpinner(label) {
+  if (!process.stdout.isTTY) return { stop: () => {} };
+  const frames = ["◐", "◓", "◑", "◒"];
+  const t0 = Date.now();
+  let i = 0;
+  process.stdout.write("\x1b[?25l"); // hide cursor
+  const render = () => {
+    const s = Math.floor((Date.now() - t0) / 1000);
+    const tail = s > 0 ? c.gray(`  ${s}s`) : "";
+    process.stdout.write(`\r\x1b[K${c.cyan(frames[i % frames.length])} ${c.gray(label + "…")}${tail}`);
+    i++;
+  };
+  render();
+  const timer = setInterval(render, 120);
+  if (typeof timer.unref === "function") timer.unref(); // don't keep the process alive
+  return {
+    stop: () => {
+      clearInterval(timer);
+      process.stdout.write("\r\x1b[K\x1b[?25h"); // clear line + show cursor
+    },
+  };
+}
+
 // Terse one-line result summary instead of dumping raw JSON / file contents.
 // Tools whose handlers already render rich output (diffs, the shell stream, the
 // plan) just get a check — the detail was already printed.

package/src/repl.js

@@ -16,7 +16,7 @@ import { c, errorLine, box } from "./render.js";
 import { checkForUpdate } from "./update-check.js";
 import { promptBox, EXIT_SIGNAL } from "./box-input.js";
 
-const VERSION = "0.23.0";
+const VERSION = "0.25.0";
 const MODEL_NAME = "Aether Core";
 
 const SHORTCUTS = `

package/src/tools.js

@@ -574,7 +574,12 @@ async function editFile(args, opts) {
     return { ok: false, output: "path, find, replace are required" };
   }
   const abs = resolveSafe(args.path, opts);
-  if (!fs.existsSync(abs)) return { ok: false, output: `File not found: ${args.path}` };
+  if (!fs.existsSync(abs)) {
+    return {
+      ok: false,
+      output: `File not found: ${args.path}. It doesn't exist yet — use write_file to CREATE it (edit_file only modifies existing files). Do not retry edit_file on this path.`,
+    };
+  }
   const oldContent = fs.readFileSync(abs, "utf8");
   const occurrences = oldContent.split(args.find).length - 1;
   if (occurrences === 0) {