portal-agent-cli 3.0.0 → 3.0.2

package/lib/tools/index.mjs

@@ -36,18 +36,18 @@ export const SAFE_TOOLS = new Set([
 ]);
 
 export const CATALOG = [
-  "- read_file(path, start_line?, end_line?)             read file with line numbers",
-  "- write_file(path, content)                            write or overwrite a whole file",
-  "- edit_file(path, old_string, new_string, replace_all?)  exact find and replace",
-  "- append_file(path, content)                           add text to the end of a file",
-  "- undo_file(path)                                      restore the last backup",
-  "- list_files(path?)                                    list a directory with sizes",
-  "- find_files(pattern)                                  glob search, e.g. **/*.ts",
-  "- grep_search(pattern, path?, case_sensitive?)         regex inside files",
-  "- delete_file(path)                                    move a file to .portal-trash",
-  "- move_file(from, to)                                  rename or relocate a file",
-  "- shell(command)                                       run a shell command",
-  "- git_status()                                         show git status",
-  "- git_diff(path?, staged?)                             show uncommitted changes",
-  "- git_log(count?)                                      recent commits"
+  "read_file(path, start_line?, end_line?)",
+  "write_file(path, content)",
+  "edit_file(path, old_string, new_string, replace_all?)",
+  "append_file(path, content)",
+  "undo_file(path)",
+  "list_files(path?)",
+  "find_files(pattern)",
+  "grep_search(pattern, path?, case_sensitive?)",
+  "delete_file(path)",
+  "move_file(from, to)",
+  "shell(command)",
+  "git_status()",
+  "git_diff(path?, staged?)",
+  "git_log(count?)"
 ].join("\n");

package/lib/tools/list.mjs

@@ -7,8 +7,7 @@ export async function list_files(args) {
   const entries = await readdir(abs, { withFileTypes: true });
 
   if (!entries.length) {
-    return "EMPTY. This directory has no files or subdirectories. " +
-      "Do not call list_files again on this path.";
+    return "EMPTY. Do not call list_files again on this path.";
   }
 
   const rows = [];
@@ -19,9 +18,7 @@ export async function list_files(args) {
       try {
         const st = await stat(full);
         size = formatSize(st.size);
-      } catch (err) {
-        size = "";
-      }
+      } catch (err) {}
     }
     const tag = e.isDirectory() ? "[dir] " : "      ";
     rows.push(tag + e.name.padEnd(40) + " " + size);

package/lib/tools/read.mjs

@@ -8,16 +8,14 @@ export async function read_file(args) {
   const text = await readFile(abs, "utf8");
 
   if (text.length > MAX_BYTES) {
-    return text.slice(0, MAX_BYTES) + "\n[truncated at " + MAX_BYTES + " bytes]";
+    return text.slice(0, MAX_BYTES) + "\n[truncated]";
   }
 
   const lines = text.split("\n");
   const start = Math.max(1, Number(args.start_line) || 1);
   const end = Math.min(lines.length, Number(args.end_line) || lines.length);
 
-  if (start > end) {
-    return "invalid range: start_line " + start + " is after end_line " + end;
-  }
+  if (start > end) return "invalid range";
 
   const width = String(end).length;
   const out = [];
@@ -26,7 +24,7 @@ export async function read_file(args) {
   }
 
   const header = start === 1 && end === lines.length
-    ? lines.length + " lines total"
+    ? lines.length + " lines"
     : "lines " + start + "-" + end + " of " + lines.length;
 
   return "[" + header + "]\n" + out.join("\n");

package/lib/tools/shell.mjs

@@ -4,9 +4,26 @@ import { getWorkspace } from "./path.mjs";
 const MAX_OUTPUT = 65000;
 const TIMEOUT_MS = 60000;
 
+function normalizeWinCommand(cmd) {
+  if (process.platform !== "win32") return cmd;
+  return cmd.replace(/(\S+)/g, function (token) {
+    if (token.indexOf("://") >= 0) return token;
+    if (/^\/[a-zA-Z](\s|$)/.test(token)) return token;
+    if (/^[A-Za-z]:\//.test(token) || (token.indexOf("/") > 0 && token.indexOf("://") < 0)) {
+      const leading = token.startsWith('"') ? '"' : "";
+      const trailing = token.endsWith('"') ? '"' : "";
+      const body = token.slice(leading.length, token.length - trailing.length);
+      return leading + body.split("/").join("\\") + trailing;
+    }
+    return token;
+  });
+}
+
 export function shell(args) {
-  const cmd = String(args.command || "");
-  if (!cmd) return Promise.resolve("shell requires a command");
+  const raw = String(args.command || "");
+  if (!raw) return Promise.resolve("shell requires a command");
+
+  const cmd = normalizeWinCommand(raw);
 
   return new Promise(function (done) {
     const ws = getWorkspace();
@@ -26,7 +43,7 @@ export function shell(args) {
 
     const timer = setTimeout(function () {
       child.kill();
-      done("timed out after " + (TIMEOUT_MS / 1000) + "s\n" + stderr);
+      done("timed out\n" + stderr);
     }, TIMEOUT_MS);
 
     child.on("error", function (err) {
@@ -38,6 +55,7 @@ export function shell(args) {
       clearTimeout(timer);
       const both = [stdout.trimEnd(), stderr.trimEnd()].filter(Boolean).join("\n");
       let s = "exit " + (code === null ? "null" : code) + "\n" + (both || "(no output)");
+      if (cmd !== raw) s += "\n[normalized: " + cmd + "]";
       if (s.length > MAX_OUTPUT) s = s.slice(0, MAX_OUTPUT) + "\n[truncated]";
       done(s);
     });

package/lib/tools/walk.mjs

@@ -13,8 +13,6 @@ export function setIgnore(names) {
   ignore = new Set(names || []);
 }
 
-// Depth-first file walker. Yields full absolute paths. Skips ignored dirs
-// and hidden files (except .env.example which is often relevant).
 export async function* walk(dir) {
   let items;
   try {

package/lib/tools/write.mjs

@@ -2,40 +2,29 @@ import { writeFile, mkdir, readFile } from "node:fs/promises";
 import { dirname } from "node:path";
 import { safePath } from "./path.mjs";
 
-// Write a whole file. Creates parent directories. If the file already
-// exists, saves a backup first.
 export async function write_file(args) {
   const abs = safePath(args.path);
   const content = args.content || "";
   await mkdir(dirname(abs), { recursive: true });
 
-  // Backup existing file
   try {
     const existing = await readFile(abs, "utf8");
     if (existing !== content) {
-      const backup = abs + ".portal-bak";
-      await writeFile(backup, existing, "utf8");
+      await writeFile(abs + ".portal-bak", existing, "utf8");
     }
-  } catch (e) {
-    // new file, no backup needed
-  }
+  } catch (e) {}
 
   await writeFile(abs, content, "utf8");
   const lines = content.length === 0 ? 0 : content.split("\n").length;
   return "wrote " + content.length + " bytes (" + lines + " lines) to " + args.path;
 }
 
-// Append to end of a file, creating it if missing.
 export async function append_file(args) {
   const abs = safePath(args.path);
   await mkdir(dirname(abs), { recursive: true });
 
   let existing = "";
-  try {
-    existing = await readFile(abs, "utf8");
-  } catch (e) {
-    existing = "";
-  }
+  try { existing = await readFile(abs, "utf8"); } catch (e) { existing = ""; }
 
   const content = args.content || "";
   const sep = existing.length > 0 && !existing.endsWith("\n") ? "\n" : "";

package/lib/turn.mjs

@@ -16,8 +16,7 @@ export async function runTurn(page, provider, prompt, config, session) {
 
   const seen = new Map();
   let next = sys + "\n\nTask: " + prompt;
-  let approvedAlways = new Set(session.approvedAlways || []);
-  const newTools = [];
+  const approvedAlways = new Set(session.approvedAlways || []);
 
   for (let step = 0; step < config.maxSteps; step++) {
     status("waiting for " + provider.label + "...");
@@ -44,75 +43,57 @@ export async function runTurn(page, provider, prompt, config, session) {
 
       if (n >= REPEAT_STOP) {
         process.stdout.write(theme.error("x loop detected: " + c.tool +
-          " called " + n + " times with same args. stopping.") + "\n");
+          " repeated. stopping.") + "\n");
         stopped = true;
         break;
       }
 
       if (n === REPEAT_WARN) {
-        process.stdout.write(theme.warn("! " + c.tool + " repeated " + n +
-          " times. warning the model.") + "\n");
+        process.stdout.write(theme.warn("! " + c.tool + " repeated.") + "\n");
       }
 
       logTool(c.tool, c.args);
 
-      // Approval gate for sensitive tools
-      const needsApproval = !config.yolo &&
+      const needs = !config.yolo &&
         !SAFE_TOOLS.has(c.tool) &&
         !approvedAlways.has(c.tool);
 
-      if (needsApproval) {
-        const decision = await askApproval(c.tool, c.args);
-        if (decision === "no") {
+      if (needs) {
+        const d = await askApproval(c.tool, c.args);
+        if (d === "no") {
           const msg = "user denied " + c.tool;
           toolResult(msg, true);
           results.push("Tool: " + c.tool + "\nStatus: denied\n" + msg);
           continue;
         }
-        if (decision === "always") {
-          approvedAlways.add(c.tool);
-          newTools.push(c.tool);
-        }
+        if (d === "always") approvedAlways.add(c.tool);
       }
 
       let out;
-      let isError = false;
+      let isErr = false;
       try {
         const fn = TOOLS[c.tool];
-        if (!fn) {
-          out = "unknown tool: " + c.tool;
-          isError = true;
-        } else {
-          out = await fn(c.args || {});
-          if (typeof out === "string" && out.indexOf("failed") === 0) isError = true;
-        }
+        if (!fn) { out = "unknown tool: " + c.tool; isErr = true; }
+        else out = await fn(c.args || {});
       } catch (e) {
         out = "error: " + (e.message || e);
-        isError = true;
+        isErr = true;
       }
 
-      toolResult(out, isError);
+      toolResult(out, isErr);
       results.push("Tool: " + c.tool + "\nResult:\n" + out);
     }
 
     if (stopped) {
       session.turns.push({ role: "user", text: prompt });
-      session.turns.push({ role: "assistant", text: "(stopped: loop detected)" });
+      session.turns.push({ role: "assistant", text: "(stopped: loop)" });
       session.updatedAt = Date.now();
       return;
     }
 
-    const footer = [
-      "Results above.",
-      "Do not repeat any call with the same arguments.",
-      "If the task is complete, reply with plain prose and no tool calls."
-    ].join("\n");
-
-    next = "Tool results:\n\n" + results.join("\n\n") + "\n\n" + footer;
+    next = "Tool results:\n\n" + results.join("\n\n") +
+      "\n\nContinue. Do not repeat any call.";
   }
 
-  process.stdout.write(theme.warn("max steps reached for this turn.") + "\n");
-  session.turns.push({ role: "user", text: prompt });
-  session.turns.push({ role: "assistant", text: "(max steps reached)" });
-  session.updatedAt = Date.now();
+  process.stdout.write(theme.warn("max steps reached.") + "\n");
 }

package/lib/wait.mjs

@@ -13,9 +13,7 @@ export async function findFirstVisible(page, selectors, ms) {
       const loc = page.locator(sel).last();
       try {
         if (await loc.isVisible({ timeout: 200 })) return loc;
-      } catch (e) {
-        // try next selector
-      }
+      } catch (e) {}
     }
     await sleep(200);
   }
@@ -24,29 +22,26 @@ export async function findFirstVisible(page, selectors, ms) {
 
 export async function waitForComposer(page, provider, ms) {
   const end = Date.now() + ms;
-  let lastHint = Date.now();
+  let hint = Date.now();
   while (Date.now() < end) {
     for (const sel of provider.input) {
       const loc = page.locator(sel).last();
       let ok = false;
       try {
         ok = await loc.isVisible({ timeout: 200 });
-      } catch (e) {
-        ok = false;
-      }
+      } catch (e) {}
       if (ok) return loc;
     }
-    if (Date.now() - lastHint > 15000) {
+    if (Date.now() - hint > 15000) {
       const left = Math.round((end - Date.now()) / 1000);
       process.stdout.write("  waiting for login (" + left + "s left)\n");
-      lastHint = Date.now();
+      hint = Date.now();
     }
     await sleep(1000);
   }
   throw new Error(
-    "Timed out waiting for the composer.\n" +
-    "Log in to the provider in the Chrome window and try again, " +
-    "or raise --login-timeout."
+    "Timed out waiting for the composer. Log in to the provider " +
+    "and try again, or raise --login-timeout."
   );
 }
 
@@ -55,9 +50,7 @@ export async function countMatching(page, selectors) {
     try {
       const n = await page.locator(sel).count();
       if (n > 0) return n;
-    } catch (e) {
-      // try next
-    }
+    } catch (e) {}
   }
   return 0;
 }

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "portal-agent-cli",
-  "version": "3.0.0",
-  "description": "Drive web AI products from the terminal with local tools. Multi-provider, session persistence, approval gate, undo.",
+  "version": "3.0.2",
+  "description": "Drive web AI products from the terminal with local tools.",
   "type": "module",
   "bin": {
     "portal-agent": "portal.mjs"
@@ -29,9 +29,7 @@
     "ai",
     "agent",
     "deepseek",
-    "chatgpt",
-    "gemini",
-    "playwright"
+    "chatgpt"
   ],
   "license": "MIT"
 }

package/portal.mjs

@@ -1,10 +1,4 @@
 #!/usr/bin/env node
-//
-// portal-agent -- drive web AI products from the terminal with local tools.
-//
-// Entry point. Loads config, parses arguments, and dispatches to either the
-// interactive REPL or the one-shot exec mode.
-//
 import { parseArgs } from "./lib/args.mjs";
 import { loadConfig } from "./lib/config.mjs";
 import { runRepl } from "./lib/repl.mjs";
@@ -14,23 +8,16 @@ import { theme } from "./lib/theme.mjs";
 async function main() {
   const opts = parseArgs();
   const config = await loadConfig(opts);
-
   if (config.verbose) {
     process.stdout.write(theme.dim("config: " + JSON.stringify(config, null, 2)) + "\n");
   }
-
-  if (opts.exec) {
-    await runExec(config);
-  } else {
-    await runRepl(config);
-  }
+  if (opts.exec) await runExec(config);
+  else await runRepl(config);
 }
 
 main().catch(function (e) {
   const msg = e && e.message ? e.message : String(e);
   process.stderr.write(theme.error("fatal: " + msg) + "\n");
-  if (process.env.PORTAL_DEBUG) {
-    process.stderr.write(e.stack + "\n");
-  }
+  if (process.env.PORTAL_DEBUG) process.stderr.write(e.stack + "\n");
   process.exit(1);
 });