@agent-commons/cli 0.1.16 → 0.1.18

package/dist/bin.js

@@ -106,7 +106,7 @@ var sym = {
   bullet: import_chalk.default.dim("\u2022"),
   dot: import_chalk.default.dim("\xB7")
 };
-function banner(version = "0.1.16") {
+function banner(version = "0.1.18") {
   const line = import_chalk.default.cyan("  \u2500".padEnd(2) + "\u2500".repeat(44));
   console.log("");
   console.log(line);
@@ -515,7 +515,7 @@ ${sym.ok} Agent created`);
       process.exit(1);
     }
   });
-  const autonomy = cmd.command("autonomy").description("Manage agent heartbeat / autonomy");
+  const autonomy = cmd.command("autonomy").description("Manage agent heartbeat");
   autonomy.command("status").description("Show autonomy status for an agent").requiredOption("--agent <agentId>", "Agent ID").option("--json", "Output as JSON").action(async (opts) => {
     const client = makeClient();
     const spinner = spin("Fetching autonomy status\u2026");
@@ -525,7 +525,7 @@ ${sym.ok} Agent created`);
       const s = res.data;
       if (opts.json) return jsonOut(s);
       console.log(`
-${c.bold("Autonomy Status")}`);
+${c.bold("Heartbeat Status")}`);
       detail([
         ["Enabled", s.enabled ? c.bold("yes") : "no"],
         ["Interval", s.intervalSec ? `${s.intervalSec}s` : "n/a"],
@@ -539,7 +539,7 @@ ${c.bold("Autonomy Status")}`);
       process.exit(1);
     }
   });
-  autonomy.command("enable").description("Enable autonomous heartbeat for an agent").requiredOption("--agent <agentId>", "Agent ID").option("--interval <seconds>", "Heartbeat interval in seconds (min 30)", "300").action(async (opts) => {
+  autonomy.command("enable").description("Enable heartbeat for an agent").requiredOption("--agent <agentId>", "Agent ID").option("--interval <seconds>", "Heartbeat interval in seconds (min 30)", "300").action(async (opts) => {
     const client = makeClient();
     const spinner = spin("Enabling autonomy\u2026");
     try {
@@ -557,7 +557,7 @@ ${sym.ok} Autonomy enabled for agent ${c.id(opts.agent)}`);
       process.exit(1);
     }
   });
-  autonomy.command("disable").description("Disable autonomous heartbeat for an agent").requiredOption("--agent <agentId>", "Agent ID").action(async (opts) => {
+  autonomy.command("disable").description("Disable heartbeat for an agent").requiredOption("--agent <agentId>", "Agent ID").action(async (opts) => {
     const client = makeClient();
     const spinner = spin("Disabling autonomy\u2026");
     try {
@@ -571,7 +571,7 @@ ${sym.ok} Autonomy disabled for agent ${c.id(opts.agent)}`);
       process.exit(1);
     }
   });
-  autonomy.command("trigger").description("Trigger a single heartbeat beat immediately").requiredOption("--agent <agentId>", "Agent ID").action(async (opts) => {
+  autonomy.command("trigger").description("Trigger a single heartbeat immediately").requiredOption("--agent <agentId>", "Agent ID").action(async (opts) => {
     const client = makeClient();
     const spinner = spin("Triggering heartbeat\u2026");
     try {
@@ -1184,79 +1184,14 @@ ${sym.fail} ${c.error(event.message ?? event.type)}`);
 
 // src/commands/run.ts
 var import_commander7 = require("commander");
-function runCommand() {
-  return new import_commander7.Command("run").description("Send a single prompt to an agent and stream the response").argument("<prompt>", "Prompt text to send").option("--agent <agentId>", "Agent ID").option("--session <sessionId>", "Session ID").option("--no-stream", "Disable streaming (wait for full response)").option("--json", "Output raw event stream as JSON lines").action(async (prompt2, opts) => {
-    const cfg = loadConfig();
-    const agentId = opts.agent ?? cfg.defaultAgentId;
-    if (!agentId) {
-      console.error(c.error("Specify --agent <agentId> or set defaultAgentId with `agc config set defaultAgentId <id>`"));
-      process.exit(1);
-    }
-    const params = {
-      agentId,
-      sessionId: opts.session,
-      messages: [{ role: "user", content: prompt2 }],
-      ...cfg.initiator && { initiatorId: cfg.initiator }
-    };
-    if (opts.noStream) {
-      const spinner = spin("Running\u2026");
-      try {
-        const client = makeClient();
-        const result = await client.run.once(params);
-        spinner.stop();
-        if (opts.json) return jsonOut(result);
-        const text = result?.content ?? result?.text ?? result?.message ?? JSON.stringify(result);
-        console.log(text);
-      } catch (err) {
-        spinner.stop();
-        printError(err);
-        process.exit(1);
-      }
-      return;
-    }
-    try {
-      const client = makeClient();
-      let hasOutput = false;
-      for await (const event of client.agents.stream(params)) {
-        if (opts.json) {
-          console.log(JSON.stringify(event));
-          continue;
-        }
-        if (event.type === "token") {
-          process.stdout.write(event.content ?? "");
-          hasOutput = true;
-        } else if (event.type === "final") {
-          if (hasOutput) process.stdout.write("\n");
-          const e = event;
-          if (e.content && !hasOutput) console.log(e.content);
-          break;
-        } else if (event.type === "error") {
-          if (hasOutput) process.stdout.write("\n");
-          console.error(`
-${sym.fail} ${c.error(event.message ?? "Error")}`);
-          process.exit(1);
-        }
-      }
-      if (hasOutput && !opts.json) process.stdout.write("\n");
-    } catch (err) {
-      printError(err);
-      process.exit(1);
-    }
-  });
-}
-
-// src/commands/chat.ts
-var import_commander8 = require("commander");
 var readline3 = __toESM(require("readline"));
-var import_fs4 = require("fs");
-var import_path4 = require("path");
-var import_os3 = require("os");
 
 // src/local-tools.ts
 var import_fs3 = require("fs");
 var import_path3 = require("path");
 var import_child_process2 = require("child_process");
 var readline2 = __toESM(require("readline"));
+var pdfParse = require("pdf-parse/lib/pdf-parse.js");
 var managedProcesses = /* @__PURE__ */ new Map();
 function capBuffer(existing, chunk, maxBytes) {
   const joined = existing + chunk;
@@ -1309,7 +1244,7 @@ function readFileForContext(rootDir, filePath) {
     return `[error reading file: ${err?.message}]`;
   }
 }
-function buildLocalToolsManifest(rootDir, snapshot, fileContextBlocks = []) {
+function buildLocalToolsManifest(rootDir, snapshot, fileContextBlocks = [], autoApprove = false) {
   const fileSection = fileContextBlocks.length ? `
 ### File contents included in this turn
 
@@ -1335,7 +1270,8 @@ ${fileSection}
 2. **Always show the actual output** returned by the tool in your response. Never say "I listed the files" without showing them. Report exactly what the tool returns.
 3. **Never fabricate results.** Wait for the real tool output before responding.
 4. **Sensitive paths are blocked** (.ssh, .gnupg, .aws, .env, credentials). Attempting to access them will return an error.
-5. **cli_write_file and cli_run_command require the user to confirm** before executing \u2014 you will see the result after they approve.
+5. ${autoApprove ? "**cli_write_file and cli_run_command execute immediately** \u2014 auto-approve is active, no user confirmation is required." : "**cli_write_file and cli_run_command require the user to confirm** before executing \u2014 you will see the result after they approve."}
+6. **Git commits must carry the agc co-author trailer.** Always include \`--trailer "Co-Authored-By: <AgentName> (agc) <agc-agent@users.noreply.github.com>"\` when running \`git commit\`. The CLI injects this automatically \u2014 do not omit it or pass \`--no-trailer\`.
 
 ### Available CLI tools
 
@@ -1407,6 +1343,13 @@ function extractToolCall(text) {
   }
   return null;
 }
+function injectAgcTrailer(command, args, agentId, agentName) {
+  if (command !== "git") return args;
+  if (!args.some((a) => a === "commit")) return args;
+  if (args.some((a) => a.includes("Co-Authored-By: agc"))) return args;
+  const identity = agentName ? `${agentName} (agc)` : agentId ? `agc/${agentId}` : "agc agent";
+  return [...args, "--trailer", `Co-Authored-By: ${identity} <agc-agent@users.noreply.github.com>`];
+}
 function safePath(root, userPath) {
   const abs = (0, import_path3.resolve)(root, userPath);
   const rel = (0, import_path3.relative)(root, abs);
@@ -1433,6 +1376,7 @@ function assertNotSensitive(abs) {
   }
 }
 async function confirm(message, config, permissionKey) {
+  if (config.autoApprove) return true;
   const cached = config.permissions.get(permissionKey);
   if (cached === "allow") return true;
   if (cached === "deny") return false;
@@ -1505,9 +1449,24 @@ function extractViaCommand(cmd, cmdArgs) {
   });
 }
 async function extractPdfText(abs) {
+  try {
+    const buffer = (0, import_fs3.readFileSync)(abs);
+    const data = await pdfParse(buffer);
+    const text2 = data.text?.trim();
+    if (text2) {
+      const MAX_CHARS = 15e4;
+      if (text2.length > MAX_CHARS) {
+        return text2.slice(0, MAX_CHARS) + `
+
+[\u2026truncated \u2014 showing first ${MAX_CHARS.toLocaleString()} characters of ${text2.length.toLocaleString()} total]`;
+      }
+      return text2;
+    }
+  } catch {
+  }
   const text = await extractViaCommand("pdftotext", [abs, "-"]);
   if (text) return text;
-  return `[Cannot extract PDF text: pdftotext not found. Install with: brew install poppler]`;
+  return `[Cannot extract PDF text: the file may be scanned/image-only or password-protected]`;
 }
 async function extractOfficeText(abs, ext) {
   const text = await extractViaCommand("textutil", ["-stdout", "-cat", "txt", abs]);
@@ -1522,13 +1481,19 @@ async function toolReadFile(args, cfg) {
   if (!(0, import_fs3.existsSync)(abs)) throw new Error(`File not found: ${userPath}`);
   const stat = (0, import_fs3.statSync)(abs);
   if (stat.isDirectory()) throw new Error(`"${userPath}" is a directory, not a file`);
-  if (stat.size > 5e5) throw new Error(`File too large to read (${Math.round(stat.size / 1024)} KB). Max 500 KB.`);
   const ext = (0, import_path3.extname)(abs).toLowerCase();
-  if (PDF_EXTS.has(ext)) return extractPdfText(abs);
-  if (OFFICE_EXTS.has(ext)) return extractOfficeText(abs, ext);
+  if (PDF_EXTS.has(ext)) {
+    if (stat.size > 5e7) throw new Error(`PDF too large to read (${Math.round(stat.size / 1e6)} MB). Max 50 MB.`);
+    return extractPdfText(abs);
+  }
+  if (OFFICE_EXTS.has(ext)) {
+    if (stat.size > 2e7) throw new Error(`Document too large to read (${Math.round(stat.size / 1e6)} MB). Max 20 MB.`);
+    return extractOfficeText(abs, ext);
+  }
   if (UNREADABLE_BINARY_EXTS.has(ext)) {
-    throw new Error(`Cannot read binary file "${userPath}" (${ext} format). Only text, PDF, and Word documents are supported.`);
+    throw new Error(`Cannot read binary file "${userPath}" (${ext} format). Only text, PDF, and Office documents are supported.`);
   }
+  if (stat.size > 5e5) throw new Error(`File too large to read (${Math.round(stat.size / 1024)} KB). Max 500 KB.`);
   return (0, import_fs3.readFileSync)(abs, "utf8");
 }
 async function toolWriteFile(args, cfg) {
@@ -1590,7 +1555,8 @@ async function toolRunCommand(args, cfg) {
   if (!command || typeof command !== "string") throw new Error('run_command requires a "command" string');
   if (!Array.isArray(cmdArgs)) throw new Error('"args" must be an array of strings');
   const workDir = cwd ? safePath(cfg.rootDir, cwd) : cfg.rootDir;
-  const preview = [command, ...cmdArgs].join(" ");
+  const injectedArgs = injectAgcTrailer(command, cmdArgs, cfg.agentId, cfg.agentName);
+  const preview = [command, ...injectedArgs].join(" ");
   const timeoutMs = Math.min((typeof timeout_seconds === "number" ? timeout_seconds : 120) * 1e3, 3e5);
   const ok = await confirm(
     `Agent wants to run: \x1B[1m${preview}\x1B[0m
@@ -1601,7 +1567,7 @@ async function toolRunCommand(args, cfg) {
   if (!ok) return "User denied command execution.";
   if (interactive) {
     return new Promise((resolve2) => {
-      const child = (0, import_child_process2.spawn)(command, cmdArgs.map(String), { cwd: workDir, stdio: "inherit" });
+      const child = (0, import_child_process2.spawn)(command, injectedArgs.map(String), { cwd: workDir, stdio: "inherit" });
       const timer = setTimeout(() => {
         child.kill();
         resolve2(`(command timed out after ${timeoutMs / 1e3}s)`);
@@ -1617,7 +1583,7 @@ async function toolRunCommand(args, cfg) {
     });
   }
   return new Promise((resolve2) => {
-    (0, import_child_process2.execFile)(command, cmdArgs.map(String), { cwd: workDir, timeout: timeoutMs, maxBuffer: 1024 * 1024 }, (err, stdout, stderr) => {
+    (0, import_child_process2.execFile)(command, injectedArgs.map(String), { cwd: workDir, timeout: timeoutMs, maxBuffer: 1024 * 1024 }, (err, stdout, stderr) => {
       const out = [stdout, stderr].filter(Boolean).join("\n--- stderr ---\n");
       if (err && !out) return resolve2(`Error: ${err.message}`);
       resolve2(out || "(no output)");
@@ -1792,7 +1758,190 @@ async function runLocalTool(call, cfg) {
   return result;
 }
 
+// src/commands/run.ts
+function runCommand() {
+  return new import_commander7.Command("run").description("Send a single prompt to an agent and stream the response").argument("<prompt>", "Prompt text to send").option("--agent <agentId>", "Agent ID").option("--session <sessionId>", "Resume an existing session by ID").option("--new-session", "Create a new session and print its ID for future use").option("--local", "Enable local file system access (with permission prompts)").option("-y, --yes", "Enable local file system access and auto-approve all operations").option("--no-stream", "Disable streaming (wait for full response)").option("--json", "Output raw event stream as JSON lines").action(async (prompt2, opts) => {
+    const cfg = loadConfig();
+    const agentId = opts.agent ?? cfg.defaultAgentId;
+    if (!agentId) {
+      console.error(c.error("Specify --agent <agentId> or set defaultAgentId with `agc config set defaultAgentId <id>`"));
+      process.exit(1);
+    }
+    if (opts.session && opts.newSession) {
+      console.error(c.error("Cannot use --session and --new-session together."));
+      process.exit(1);
+    }
+    const client = makeClient();
+    let sessionId = opts.session;
+    if (opts.session) {
+      const spinner = spin("Loading session\u2026");
+      try {
+        await client.sessions.get(opts.session);
+        spinner.stop();
+      } catch {
+        spinner.stop();
+        console.error(c.error(`Session "${opts.session}" not found.`));
+        process.exit(1);
+      }
+    }
+    if (opts.newSession) {
+      const spinner = spin("Creating session\u2026");
+      try {
+        const res = await client.sessions.create({
+          agentId,
+          initiator: cfg.initiator ?? "",
+          title: `agc run ${(/* @__PURE__ */ new Date()).toISOString().slice(0, 16)}`,
+          source: "cli"
+        });
+        const session = res?.data ?? res;
+        sessionId = session.sessionId;
+        spinner.stop();
+      } catch (err) {
+        spinner.stop();
+        printError(err);
+        process.exit(1);
+      }
+    }
+    const localEnabled = opts.yes || opts.local;
+    const autoApprove = !!opts.yes;
+    let localToolsCfg = null;
+    let cliContext;
+    if (localEnabled) {
+      const rootDir = process.cwd();
+      localToolsCfg = {
+        rootDir,
+        sessionId: sessionId ?? "run",
+        appendLog: () => {
+        },
+        permissions: /* @__PURE__ */ new Map(),
+        agentId,
+        autoApprove
+      };
+      const snapshot = buildDirSnapshot(rootDir, 2);
+      cliContext = buildLocalToolsManifest(rootDir, snapshot, [], autoApprove);
+    }
+    if (!opts.json) {
+      const rows = [];
+      if (sessionId) {
+        const label = opts.newSession ? `${c.id(sessionId)}${c.dim(" (new)")}` : `${c.id(sessionId)}${c.dim(" (resumed)")}`;
+        rows.push(["Session", label]);
+      }
+      if (localEnabled) {
+        rows.push(["Local tools", autoApprove ? c.warn("enabled  (auto-approve on)") : c.success("enabled")]);
+      }
+      if (rows.length) {
+        detail(rows);
+        console.log();
+      }
+    }
+    const params = {
+      agentId,
+      sessionId,
+      messages: [{ role: "user", content: prompt2 }],
+      ...cfg.initiator && { initiatorId: cfg.initiator },
+      ...cliContext && { cliContext }
+    };
+    if (opts.noStream) {
+      const spinner = spin("Running\u2026");
+      try {
+        const result = await client.run.once(params);
+        spinner.stop();
+        if (opts.json) return jsonOut(result);
+        const text = result?.content ?? result?.text ?? result?.message ?? JSON.stringify(result);
+        console.log(text);
+        if (sessionId) console.log(c.dim(`
+Session: ${sessionId}  (resume with: agc run --session ${sessionId} "<prompt>")`));
+      } catch (err) {
+        spinner.stop();
+        printError(err);
+        process.exit(1);
+      }
+      return;
+    }
+    try {
+      let hasOutput = false;
+      let toolStartMs = 0;
+      let lastToolName = "";
+      for await (const event of client.agents.stream(params)) {
+        if (opts.json) {
+          console.log(JSON.stringify(event));
+          continue;
+        }
+        if (event.type === "token") {
+          process.stdout.write(event.content ?? "");
+          hasOutput = true;
+        } else if (event.type === "cli_tool_request" && localToolsCfg) {
+          const { requestId, tool: toolName, args } = event;
+          const displayName = String(toolName).replace("cli_", "");
+          if (hasOutput) {
+            process.stdout.write("\n");
+            hasOutput = false;
+          }
+          process.stdout.write(`  ${c.dim("\u2500")} ${c.bold(displayName)}`);
+          const startMs = Date.now();
+          let result;
+          let toolOk = true;
+          try {
+            result = await runLocalTool({ tool: displayName, args: args ?? {} }, localToolsCfg);
+          } catch (err) {
+            result = `Error: ${err?.message ?? String(err)}`;
+            toolOk = false;
+          }
+          const elapsed = ((Date.now() - startMs) / 1e3).toFixed(1);
+          readline3.cursorTo(process.stdout, 0);
+          readline3.clearLine(process.stdout, 0);
+          process.stdout.write(`  ${c.dim("\u2500")} ${c.bold(displayName)}  ${toolOk ? sym.ok : sym.fail}  ${c.dim("(" + elapsed + "s)")}
+`);
+          try {
+            await fetch(`${cfg.apiUrl}/v1/agents/cli-tool-result`, {
+              method: "POST",
+              headers: { "Content-Type": "application/json", "Authorization": `Bearer ${cfg.apiKey}` },
+              body: JSON.stringify({ requestId, result })
+            });
+          } catch {
+          }
+        } else if (event.type === "toolStart") {
+          lastToolName = event.toolName ?? "";
+          toolStartMs = Date.now();
+          if (hasOutput) {
+            process.stdout.write("\n");
+            hasOutput = false;
+          }
+          process.stdout.write(`  ${c.dim("\u2500")} ${c.bold(lastToolName)}`);
+        } else if (event.type === "toolEnd") {
+          const elapsed = ((Date.now() - toolStartMs) / 1e3).toFixed(1);
+          readline3.cursorTo(process.stdout, 0);
+          readline3.clearLine(process.stdout, 0);
+          process.stdout.write(`  ${c.dim("\u2500")} ${c.bold(lastToolName)}  ${sym.ok}  ${c.dim("(" + elapsed + "s)")}
+`);
+        } else if (event.type === "final") {
+          if (hasOutput) process.stdout.write("\n");
+          const e = event;
+          if (e.content && !hasOutput) console.log(e.content);
+          if (sessionId) console.log(c.dim(`
+Session: ${sessionId}  (resume with: agc run --session ${sessionId} "<prompt>")`));
+          break;
+        } else if (event.type === "error") {
+          if (hasOutput) process.stdout.write("\n");
+          console.error(`
+${sym.fail} ${c.error(event.message ?? "Error")}`);
+          process.exit(1);
+        }
+      }
+      if (hasOutput && !opts.json) process.stdout.write("\n");
+    } catch (err) {
+      printError(err);
+      process.exit(1);
+    }
+  });
+}
+
 // src/commands/chat.ts
+var import_commander8 = require("commander");
+var readline4 = __toESM(require("readline"));
+var import_fs4 = require("fs");
+var import_path4 = require("path");
+var import_os3 = require("os");
 var SESSIONS_DIR = (0, import_path4.join)((0, import_os3.homedir)(), ".agc", "sessions");
 function ensureSessionsDir() {
   if (!(0, import_fs4.existsSync)(SESSIONS_DIR)) (0, import_fs4.mkdirSync)(SESSIONS_DIR, { recursive: true });
@@ -1929,7 +2078,7 @@ ${c.bold("Agent Commons Chat")}`);
       });
     }
     console.log(c.dim("\nType your message and press Enter. Type /help for commands.\n"));
-    const rl = readline3.createInterface({
+    const rl = readline4.createInterface({
       input: process.stdin,
       output: process.stdout,
       terminal: true,
@@ -2018,12 +2167,13 @@ ${content}
         messages: [{ role: "user", content: userMessage }],
         ...cliContext && { cliContext }
       };
-      process.stdout.write(c.primary("agent") + c.dim(" \u203A "));
       if (opts.noStream) {
-        const spinner = spin("");
+        process.stdout.write(c.primary("agent") + c.dim(" \u203A "));
+        const spinner = spin("thinking\u2026");
         try {
           const result = await client.run.once(params);
           spinner.stop();
+          process.stdout.write(c.primary("agent") + c.dim(" \u203A "));
           const text = extractText(result);
           console.log(text);
           appendSessionLog(sessionId, {
@@ -2041,29 +2191,57 @@ ${sym.fail} ${c.error(err.message ?? String(err))}`);
         try {
           let hasOutput = false;
           let agentContent = "";
+          let toolStartMs = 0;
+          let lastToolName = "";
+          const thinkingSpinner = spin("thinking\u2026");
           for await (const event of client.agents.stream(params)) {
             if (event.type === "token") {
+              if (thinkingSpinner.isSpinning) {
+                thinkingSpinner.stop();
+                process.stdout.write(c.primary("agent") + c.dim(" \u203A "));
+              }
               const tok = event.content ?? "";
               process.stdout.write(tok);
               agentContent += tok;
               hasOutput = true;
             } else if (event.type === "cli_tool_request" && localToolsCfg) {
+              if (thinkingSpinner.isSpinning) thinkingSpinner.stop();
               const { requestId, tool: toolName, args } = event;
               const displayName = String(toolName).replace("cli_", "");
+              const argStr = toolArgSummary(displayName, args ?? {});
+              const isWaiting = displayName === "wait_for_process";
               if (hasOutput) {
                 process.stdout.write("\n");
                 hasOutput = false;
               }
-              process.stdout.write(c.dim(`  [local] ${displayName}\u2026`));
+              process.stdout.write(`  ${c.dim("\u2500")} ${c.bold(displayName)}${argStr ? "  " + c.dim(argStr) : ""}`);
+              const startMs = Date.now();
+              let elapsedSec = 0;
+              let elapsedInterval = null;
+              if (isWaiting) {
+                elapsedInterval = setInterval(() => {
+                  elapsedSec++;
+                  readline4.cursorTo(process.stdout, 0);
+                  process.stdout.write(`  ${c.dim("\u2500")} ${c.bold(displayName)}${argStr ? "  " + c.dim(argStr) : ""}  ${c.dim(elapsedSec + "s\u2026")}`);
+                }, 1e3);
+              }
               let result;
+              let toolOk = true;
               try {
-                const localToolName = String(toolName).replace("cli_", "");
-                result = await runLocalTool({ tool: localToolName, args: args ?? {} }, localToolsCfg);
-                process.stdout.write(c.dim(" \u2713\n"));
+                result = await runLocalTool({ tool: displayName, args: args ?? {} }, localToolsCfg);
               } catch (err) {
                 result = `Error: ${err?.message ?? String(err)}`;
-                process.stdout.write(c.dim(" \u2717\n"));
+                toolOk = false;
               }
+              if (elapsedInterval) clearInterval(elapsedInterval);
+              const elapsed = ((Date.now() - startMs) / 1e3).toFixed(1);
+              const preview = toolOk ? toolResultPreview(displayName, result) : "";
+              readline4.cursorTo(process.stdout, 0);
+              readline4.clearLine(process.stdout, 0);
+              const statusIcon = toolOk ? sym.ok : sym.fail;
+              const previewPart = preview ? `  ${c.dim(preview)}` : "";
+              process.stdout.write(`  ${c.dim("\u2500")} ${c.bold(displayName)}${argStr ? "  " + c.dim(argStr) : ""}  ${statusIcon}${previewPart}  ${c.dim("(" + elapsed + "s)")}
+`);
               appendSessionLog(sessionId, {
                 type: "local_tool_result",
                 tool: toolName,
@@ -2083,14 +2261,20 @@ ${sym.fail} ${c.error(err.message ?? String(err))}`);
                 console.error(c.warn(`
   [local] Failed to submit tool result: ${postErr?.message}`));
               }
-            } else if (event.type === "ping") {
+            } else if (event.type === "keepalive") {
             } else if (event.type === "toolStart") {
-              const name = event.toolName ?? "";
+              if (thinkingSpinner.isSpinning) thinkingSpinner.stop();
+              lastToolName = event.toolName ?? "";
+              toolStartMs = Date.now();
               if (hasOutput) process.stdout.write("\n");
-              process.stdout.write(c.dim(`  [tool] ${name}\u2026`));
+              process.stdout.write(`  ${c.dim("\u2500")} ${c.bold(lastToolName)}`);
               hasOutput = false;
             } else if (event.type === "toolEnd") {
-              process.stdout.write(c.dim(" done\n"));
+              const elapsed = ((Date.now() - toolStartMs) / 1e3).toFixed(1);
+              readline4.cursorTo(process.stdout, 0);
+              readline4.clearLine(process.stdout, 0);
+              process.stdout.write(`  ${c.dim("\u2500")} ${c.bold(lastToolName)}  ${sym.ok}  ${c.dim("(" + elapsed + "s)")}
+`);
               process.stdout.write(c.primary("agent") + c.dim(" \u203A "));
               hasOutput = false;
             } else if (event.type === "final") {
@@ -2127,12 +2311,14 @@ ${sym.fail} ${c.error(err.message ?? String(err))}`);
               }
               break;
             } else if (event.type === "error") {
+              if (thinkingSpinner.isSpinning) thinkingSpinner.stop();
               if (hasOutput) process.stdout.write("\n");
               console.error(`
 ${sym.fail} ${c.error(event.message ?? "Stream error")}`);
               break;
             }
           }
+          if (thinkingSpinner.isSpinning) thinkingSpinner.stop();
           process.stdout.write("\n");
           if (localToolsCfg && agentContent) {
             await handleLocalToolLoop(agentContent, localToolsCfg, client, agentId, sessionId, appendSessionLog);
@@ -2143,6 +2329,8 @@ ${sym.fail} ${c.error(event.message ?? "Stream error")}`);
         }
       }
       console.log();
+      readline4.cursorTo(process.stdout, 0);
+      readline4.clearLine(process.stdout, 0);
       rl.resume();
       rl.prompt();
     });
@@ -2166,16 +2354,25 @@ async function handleLocalToolLoop(agentText, cfg, client, agentId, sessionId, a
   }
   const toolCall = extractToolCall(agentText);
   if (!toolCall) return;
-  process.stdout.write(c.dim(`
-  [local] ${toolCall.tool}`));
+  const argStr = toolArgSummary(toolCall.tool, toolCall.args ?? {});
+  process.stdout.write(`
+  ${c.dim("\u2500")} ${c.bold(toolCall.tool)}${argStr ? "  " + c.dim(argStr) : ""}`);
+  const startMs = Date.now();
   let result;
+  let toolOk = true;
   try {
     result = await runLocalTool(toolCall, cfg);
-    process.stdout.write(c.dim(" \u2713\n"));
   } catch (err) {
     result = `Error: ${err?.message ?? String(err)}`;
-    process.stdout.write(c.dim(" \u2717\n"));
+    toolOk = false;
   }
+  const elapsed = ((Date.now() - startMs) / 1e3).toFixed(1);
+  const preview = toolOk ? toolResultPreview(toolCall.tool, result) : "";
+  readline4.cursorTo(process.stdout, 0);
+  readline4.clearLine(process.stdout, 0);
+  const previewPart = preview ? `  ${c.dim(preview)}` : "";
+  process.stdout.write(`  ${c.dim("\u2500")} ${c.bold(toolCall.tool)}${argStr ? "  " + c.dim(argStr) : ""}  ${toolOk ? sym.ok : sym.fail}${previewPart}  ${c.dim("(" + elapsed + "s)")}
+`);
   const resultMsg = `[Tool result: ${toolCall.tool}]
 \`\`\`
 ${result}
@@ -2190,6 +2387,8 @@ ${result}
   process.stdout.write(c.primary("agent") + c.dim(" \u203A "));
   let followContent = "";
   try {
+    let loopToolName = "";
+    let loopToolStartMs = 0;
     for await (const evt of client.agents.stream({
       agentId,
       sessionId,
@@ -2200,11 +2399,16 @@ ${result}
         process.stdout.write(tok);
         followContent += tok;
       } else if (evt.type === "toolStart") {
-        const name = evt.toolName ?? "";
+        loopToolName = evt.toolName ?? "";
+        loopToolStartMs = Date.now();
         if (followContent) process.stdout.write("\n");
-        process.stdout.write(c.dim(`  [tool] ${name}\u2026`));
+        process.stdout.write(`  ${c.dim("\u2500")} ${c.bold(loopToolName)}`);
       } else if (evt.type === "toolEnd") {
-        process.stdout.write(c.dim(" done\n"));
+        const elapsed2 = ((Date.now() - loopToolStartMs) / 1e3).toFixed(1);
+        readline4.cursorTo(process.stdout, 0);
+        readline4.clearLine(process.stdout, 0);
+        process.stdout.write(`  ${c.dim("\u2500")} ${c.bold(loopToolName)}  ${sym.ok}  ${c.dim("(" + elapsed2 + "s)")}
+`);
         process.stdout.write(c.primary("agent") + c.dim(" \u203A "));
       } else if (evt.type === "final") {
         const txt = extractText(evt?.payload);
@@ -2228,6 +2432,80 @@ ${sym.fail} ${c.error(evt.message ?? "Stream error")}`);
   }
   await handleLocalToolLoop(followContent, cfg, client, agentId, sessionId, appendLog, depth + 1);
 }
+function truncate(s, max) {
+  const str = String(s ?? "");
+  return str.length <= max ? str : str.slice(0, max - 1) + "\u2026";
+}
+function toolArgSummary(toolName, args) {
+  switch (toolName) {
+    case "read_file":
+      return truncate(args.path ?? "", 60);
+    case "write_file":
+      return truncate(args.path ?? "", 60);
+    case "delete_file":
+      return truncate(args.path ?? "", 60);
+    case "list_directory":
+      return truncate(args.path ?? ".", 60);
+    case "run_command":
+      return truncate(args.command ?? "", 60);
+    case "start_process":
+      return truncate(args.command ?? "", 60);
+    case "wait_for_process":
+      return truncate(args.process_id ?? "", 20);
+    case "process_status":
+      return truncate(args.process_id ?? "", 20);
+    case "kill_process":
+      return truncate(args.process_id ?? "", 20);
+    case "list_processes":
+      return "";
+    case "search_files": {
+      const parts = [args.pattern, args.query].filter(Boolean);
+      return truncate(parts.join(" "), 60);
+    }
+    default: {
+      const first = args.path ?? args.query ?? args.command ?? args.pattern ?? "";
+      return truncate(String(first), 60);
+    }
+  }
+}
+function toolResultPreview(toolName, result) {
+  if (!result || result.startsWith("Error:")) return "";
+  switch (toolName) {
+    case "read_file": {
+      const lines = result.split("\n").length;
+      return `${lines} lines`;
+    }
+    case "write_file":
+      return "written";
+    case "delete_file":
+      return "deleted";
+    case "list_directory": {
+      const count = result.split("\n").filter(Boolean).length;
+      return `${count} entries`;
+    }
+    case "run_command": {
+      const first = result.split("\n").find((l) => l.trim());
+      return first ? truncate(first.trim(), 50) : "done";
+    }
+    case "start_process": {
+      const match = result.match(/process[_\s-]?id[:\s]+([a-zA-Z0-9_-]+)/i) ?? result.match(/"id"[:\s]+"([^"]+)"/);
+      return match ? `pid ${match[1]}` : "started";
+    }
+    case "wait_for_process": {
+      if (/done|complete|exit/i.test(result)) return "done";
+      if (/running/i.test(result)) return "still running";
+      return truncate(result.split("\n")[0]?.trim() ?? "", 40);
+    }
+    case "search_files": {
+      const count = result.split("\n").filter(Boolean).length;
+      return `${count} match${count === 1 ? "" : "es"}`;
+    }
+    default: {
+      const first = result.split("\n").find((l) => l.trim());
+      return first ? truncate(first.trim(), 50) : "";
+    }
+  }
+}
 function extractText(payload) {
   if (!payload) return "";
   if (typeof payload === "string") return payload;
@@ -2766,8 +3044,8 @@ function skillsCommand() {
   });
   cmd.command("delete <slug>").description("Permanently delete a skill").option("--yes", "Skip confirmation prompt").option("--json", "Output result as JSON").action(async (slug, opts) => {
     if (!opts.yes) {
-      const readline4 = await import("readline");
-      const rl = readline4.createInterface({ input: process.stdin, output: process.stdout });
+      const readline5 = await import("readline");
+      const rl = readline5.createInterface({ input: process.stdin, output: process.stdout });
       const answer = await new Promise(
         (resolve2) => rl.question(c.warn(`Delete skill "${slug}"? This cannot be undone. [y/N] `), resolve2)
       );
@@ -3523,7 +3801,7 @@ async function pickAgentInteractively(action) {
   return agentId;
 }
 var program = new import_commander16.Command();
-program.name("agc").description("Agent Commons CLI \u2014 interact with the Agent Commons platform").version("0.1.16", "-v, --version").action(async () => {
+program.name("agc").description("Agent Commons CLI \u2014 interact with the Agent Commons platform").version("0.1.18", "-v, --version").action(async () => {
   await interactiveMenu();
 });
 program.addCommand(loginCommand());

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agent-commons/cli",
-  "version": "0.1.16",
+  "version": "0.1.18",
   "description": "Agent Commons CLI — chat, run, and manage agents from your terminal",
   "license": "MIT",
   "bin": {
@@ -11,15 +11,17 @@
     "dist"
   ],
   "dependencies": {
-    "commander": "^12.1.0",
+    "@types/pdf-parse": "^1.1.5",
     "chalk": "^5.3.0",
+    "commander": "^12.1.0",
     "ora": "^8.1.1",
-    "@agent-commons/sdk": "0.1.12"
+    "pdf-parse": "^1.1.1",
+    "@agent-commons/sdk": "0.1.13"
   },
   "devDependencies": {
+    "@types/node": "^22.10.2",
     "tsup": "^8.3.5",
-    "typescript": "^5.7.2",
-    "@types/node": "^22.10.2"
+    "typescript": "^5.7.2"
   },
   "engines": {
     "node": ">=18"