portal-agent-cli 3.0.3 → 3.0.5

package/lib/tools/git.mjs

@@ -1,52 +1,4 @@
 import { spawn } from "node:child_process";
 import { getWorkspace } from "./path.mjs";
 
-function run(args) {
-  return new Promise(function (done) {
-    const child = spawn("git", args, {
-      cwd: getWorkspace(),
-      shell: false,
-      windowsHide: true
-    });
-    let out = "";
-    let err = "";
-    if (child.stdout) child.stdout.on("data", function (d) { out += String(d); });
-    if (child.stderr) child.stderr.on("data", function (d) { err += String(d); });
-    const timer = setTimeout(function () {
-      child.kill();
-      done({ code: -1, out: out, err: "timed out" });
-    }, 15000);
-    child.on("error", function (e) {
-      clearTimeout(timer);
-      done({ code: -1, out: "", err: e.message });
-    });
-    child.on("close", function (code) {
-      clearTimeout(timer);
-      done({ code: code, out: out, err: err });
-    });
-  });
-}
-
-export async function git_status() {
-  const r = await run(["status", "--short", "--branch"]);
-  if (r.code !== 0) return "not a git repository";
-  return r.out.trim() || "working tree clean";
-}
-
-export async function git_diff(args) {
-  const cmd = ["diff"];
-  if (args && args.staged) cmd.push("--cached");
-  if (args && args.path) cmd.push("--", args.path);
-  const r = await run(cmd);
-  if (r.code !== 0) return r.err || "git diff failed";
-  const out = r.out.trim();
-  if (!out) return "no changes";
-  return out.length > 30000 ? out.slice(0, 30000) + "\n[truncated]" : out;
-}
-
-export async function git_log(args) {
-  const n = (args && args.count) || 10;
-  const r = await run(["log", "--oneline", "-" + n]);
-  if (r.code !== 0) return "not a git repository";
-  return r.out.trim() || "no commits";
-}
+function run(args

package/lib/tools/grep.mjs

@@ -1,41 +1 @@
-import { readFile, stat } from "node:fs/promises";
-import { relative } from "node:path";
-import { walk } from "./walk.mjs";
-import { safePath, toPosix, getWorkspace } from "./path.mjs";
-
-export async function grep_search(args) {
-  const pattern = String(args.pattern || "");
-  if (!pattern) return "grep_search requires a pattern";
-
-  let re;
-  try {
-    re = new RegExp(pattern, args.case_sensitive ? "" : "i");
-  } catch (e) {
-    return "bad regex: " + e.message;
-  }
-
-  const ws = getWorkspace();
-  const startPath = args.path ? safePath(args.path) : ws;
-  const hits = [];
-
-  for await (const f of walk(startPath)) {
-    if (hits.length >= 80) break;
-    let info;
-    try { info = await stat(f); } catch (e) { continue; }
-    if (info.size > 512000) continue;
-
-    let text;
-    try { text = await readFile(f, "utf8"); } catch (e) { continue; }
-    const lines = text.split("\n");
-
-    for (let i = 0; i < lines.length && hits.length < 80; i++) {
-      if (re.test(lines[i])) {
-        const rel = toPosix(relative(ws, f));
-        hits.push(rel + ":" + (i + 1) + ":" + lines[i].trim().slice(0, 200));
-      }
-    }
-  }
-
-  if (!hits.length) return "no matches";
-  return hits.length + " matches:\n" + hits.join("\n");
-}
+import { readFile, stat } from "node:fs/p

package/lib/tools/index.mjs

@@ -1,53 +1,3 @@
 import { read_file } from "./read.mjs";
 import { write_file, append_file } from "./write.mjs";
-import { edit_file, undo_file } from "./edit.mjs";
-import { list_files } from "./list.mjs";
-import { find_files } from "./find.mjs";
-import { grep_search } from "./grep.mjs";
-import { delete_file, move_file } from "./delete.mjs";
-import { shell } from "./shell.mjs";
-import { git_status, git_diff, git_log } from "./git.mjs";
-
-export const TOOLS = {
-  read_file: read_file,
-  write_file: write_file,
-  edit_file: edit_file,
-  append_file: append_file,
-  undo_file: undo_file,
-  list_files: list_files,
-  find_files: find_files,
-  grep_search: grep_search,
-  delete_file: delete_file,
-  move_file: move_file,
-  shell: shell,
-  git_status: git_status,
-  git_diff: git_diff,
-  git_log: git_log
-};
-
-export const SAFE_TOOLS = new Set([
-  "read_file",
-  "list_files",
-  "find_files",
-  "grep_search",
-  "git_status",
-  "git_diff",
-  "git_log"
-]);
-
-export const CATALOG = [
-  "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");
+import { edit_file, undo_file } from "./edit.m

package/lib/tools/list.mjs

@@ -1,35 +1,2 @@
 import { readdir, stat } from "node:fs/promises";
-import { join } from "node:path";
-import { safePath } from "./path.mjs";
-
-export async function list_files(args) {
-  const abs = safePath(args.path || ".");
-  const entries = await readdir(abs, { withFileTypes: true });
-
-  if (!entries.length) {
-    return "EMPTY. Do not call list_files again on this path.";
-  }
-
-  const rows = [];
-  for (const e of entries) {
-    const full = join(abs, e.name);
-    let size = "";
-    if (e.isFile()) {
-      try {
-        const st = await stat(full);
-        size = formatSize(st.size);
-      } catch (err) {}
-    }
-    const tag = e.isDirectory() ? "[dir] " : "      ";
-    rows.push(tag + e.name.padEnd(40) + " " + size);
-  }
-
-  rows.sort();
-  return rows.join("\n");
-}
-
-function formatSize(bytes) {
-  if (bytes < 1024) return bytes + " B";
-  if (bytes < 1024 * 1024) return Math.round(bytes / 1024) + " KB";
-  return (bytes / 1024 / 1024).toFixed(1) + " MB";
-}
+import { join } from

package/lib/tools/path.mjs

@@ -2,21 +2,4 @@ import { resolve, relative, sep } from "node:path";
 
 let ws = process.cwd();
 
-export function setWorkspace(p) { ws = resolve(p); }
-export function getWorkspace() { return ws; }
-
-export function safePath(p) {
-  if (!p || typeof p !== "string") {
-    throw new Error("path must be a non-empty string");
-  }
-  const abs = resolve(ws, p);
-  const rel = relative(ws, abs);
-  if (rel === ".." || rel.slice(0, 3) === "..\\" || rel.slice(0, 3) === "../") {
-    throw new Error("path escapes workspace: " + p);
-  }
-  return abs;
-}
-
-export function toPosix(p) {
-  return p.split(sep).join("/");
-}
+export function setWorkspace(p) { ws = resolve(p

package/lib/tools/read.mjs

@@ -1,31 +1 @@
-import { readFile } from "node:fs/promises";
-import { safePath } from "./path.mjs";
-
-const MAX_BYTES = 500000;
-
-export async function read_file(args) {
-  const abs = safePath(args.path);
-  const text = await readFile(abs, "utf8");
-
-  if (text.length > MAX_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";
-
-  const width = String(end).length;
-  const out = [];
-  for (let i = start - 1; i < end; i++) {
-    out.push(String(i + 1).padStart(width) + "| " + lines[i]);
-  }
-
-  const header = start === 1 && end === lines.length
-    ? lines.length + " lines"
-    : "lines " + start + "-" + end + " of " + lines.length;
-
-  return "[" + header + "]\n" + out.join("\n");
-}
+import {

package/lib/tools/shell.mjs

@@ -2,62 +2,4 @@ import { spawn } from "node:child_process";
 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 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();
-    const isWin = process.platform === "win32";
-
-    const child = spawn(cmd, {
-      cwd: ws,
-      shell: isWin ? true : "/bin/sh",
-      windowsHide: true
-    });
-
-    let stdout = "";
-    let stderr = "";
-
-    if (child.stdout) child.stdout.on("data", function (d) { stdout += String(d); });
-    if (child.stderr) child.stderr.on("data", function (d) { stderr += String(d); });
-
-    const timer = setTimeout(function () {
-      child.kill();
-      done("timed out\n" + stderr);
-    }, TIMEOUT_MS);
-
-    child.on("error", function (err) {
-      clearTimeout(timer);
-      done("spawn error: " + err.message);
-    });
-
-    child.on("close", function (code) {
-      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);
-    });
-  });
-}
+const TIMEO

package/lib/tools/walk.mjs

@@ -1,30 +1,4 @@
 import { readdir } from "node:fs/promises";
 import { join } from "node:path";
 
-const DEFAULT_IGNORE = new Set([
-  "node_modules", ".git", ".hg", ".svn", "dist", "build", "out",
-  ".next", ".nuxt", ".turbo", ".cache", "target", "__pycache__",
-  ".venv", "venv", "env", "coverage", ".idea", ".vscode", ".DS_Store"
-]);
-
-let ignore = DEFAULT_IGNORE;
-
-export function setIgnore(names) {
-  ignore = new Set(names || []);
-}
-
-export async function* walk(dir) {
-  let items;
-  try {
-    items = await readdir(dir, { withFileTypes: true });
-  } catch (e) {
-    return;
-  }
-  for (const it of items) {
-    if (ignore.has(it.name)) continue;
-    if (it.name.startsWith(".") && it.name !== ".env.example") continue;
-    const full = join(dir, it.name);
-    if (it.isDirectory()) yield* walk(full);
-    else if (it.isFile()) yield full;
-  }
-}
+const DEFAULT_IGNORE = new

package/lib/tools/write.mjs

@@ -1,35 +1,2 @@
 import { writeFile, mkdir, readFile } from "node:fs/promises";
-import { dirname } from "node:path";
-import { safePath } from "./path.mjs";
-
-export async function write_file(args) {
-  const abs = safePath(args.path);
-  const content = args.content || "";
-  await mkdir(dirname(abs), { recursive: true });
-
-  try {
-    const existing = await readFile(abs, "utf8");
-    if (existing !== content) {
-      await writeFile(abs + ".portal-bak", existing, "utf8");
-    }
-  } 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;
-}
-
-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 = ""; }
-
-  const content = args.content || "";
-  const sep = existing.length > 0 && !existing.endsWith("\n") ? "\n" : "";
-  const add = content.endsWith("\n") ? content : content + "\n";
-
-  await writeFile(abs, existing + sep + add, "utf8");
-  return "appended " + content.length + " bytes to " + args.path;
-}
+import { dirname } from "node:path

package/lib/turn.mjs

@@ -1,99 +1,2 @@
 import { send } from "./send.mjs";
-import { parseCalls, stripCalls } from "./parse.mjs";
-import { TOOLS, SAFE_TOOLS } from "./tools/index.mjs";
-import { getWorkspace } from "./tools/path.mjs";
-import { buildSystem } from "./system.mjs";
-import { askApproval } from "./safety.mjs";
-import { reply, tool as logTool, toolResult, status } from "./log.mjs";
-import { theme } from "./theme.mjs";
-
-const REPEAT_WARN = 2;
-const REPEAT_STOP = 3;
-
-export async function runTurn(page, provider, prompt, config, session) {
-  const instructions = config.instructionsText || null;
-  const sys = buildSystem(getWorkspace(), instructions);
-
-  const seen = new Map();
-  let next = sys + "\n\nTask: " + prompt;
-  const approvedAlways = new Set(session.approvedAlways || []);
-
-  for (let step = 0; step < config.maxSteps; step++) {
-    status("waiting for " + provider.label + "...");
-    const r = await send(page, provider, next, config.timeout);
-    const calls = parseCalls(r);
-
-    if (!calls.length) {
-      const prose = stripCalls(r);
-      reply(prose);
-      session.turns.push({ role: "user", text: prompt });
-      session.turns.push({ role: "assistant", text: prose });
-      session.approvedAlways = Array.from(approvedAlways);
-      session.updatedAt = Date.now();
-      return;
-    }
-
-    const results = [];
-    let stopped = false;
-
-    for (const c of calls) {
-      const sig = c.tool + ":" + JSON.stringify(c.args || {});
-      const n = (seen.get(sig) || 0) + 1;
-      seen.set(sig, n);
-
-      if (n >= REPEAT_STOP) {
-        process.stdout.write(theme.error("x loop detected: " + c.tool +
-          " repeated. stopping.") + "\n");
-        stopped = true;
-        break;
-      }
-
-      if (n === REPEAT_WARN) {
-        process.stdout.write(theme.warn("! " + c.tool + " repeated.") + "\n");
-      }
-
-      logTool(c.tool, c.args);
-
-      const needs = !config.yolo &&
-        !SAFE_TOOLS.has(c.tool) &&
-        !approvedAlways.has(c.tool);
-
-      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 (d === "always") approvedAlways.add(c.tool);
-      }
-
-      let out;
-      let isErr = false;
-      try {
-        const fn = TOOLS[c.tool];
-        if (!fn) { out = "unknown tool: " + c.tool; isErr = true; }
-        else out = await fn(c.args || {});
-      } catch (e) {
-        out = "error: " + (e.message || e);
-        isErr = true;
-      }
-
-      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)" });
-      session.updatedAt = Date.now();
-      return;
-    }
-
-    next = "Tool results:\n\n" + results.join("\n\n") +
-      "\n\nContinue. Do not repeat any call.";
-  }
-
-  process.stdout.write(theme.warn("max steps reached.") + "\n");
-}
+import { parseCalls, stripCalls } from "./parse.mjs

package/lib/wait.mjs

@@ -27,9 +27,7 @@ export async function waitForComposer(page, provider, ms) {
     for (const sel of provider.input) {
       const loc = page.locator(sel).last();
       let ok = false;
-      try {
-        ok = await loc.isVisible({ timeout: 200 });
-      } catch (e) {}
+      try { ok = await loc.isVisible({ timeout: 200 }); } catch (e) {}
       if (ok) return loc;
     }
     if (Date.now() - hint > 15000) {
@@ -39,10 +37,7 @@ export async function waitForComposer(page, provider, ms) {
     }
     await sleep(1000);
   }
-  throw new Error(
-    "Timed out waiting for the composer. Log in to the provider " +
-    "and try again, or raise --login-timeout."
-  );
+  throw new Error("Timed out waiting for composer.");
 }
 
 export async function countMatching(page, selectors) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "portal-agent-cli",
-  "version": "3.0.3",
+  "version": "3.0.5",
   "description": "Drive web AI products from the terminal with local tools.",
   "type": "module",
   "bin": {
@@ -16,20 +16,8 @@
   "engines": {
     "node": ">=20"
   },
-  "scripts": {
-    "start": "node portal.mjs"
-  },
   "dependencies": {
     "playwright": "^1.48.0"
   },
-  "keywords": [
-    "browser",
-    "automation",
-    "cli",
-    "ai",
-    "agent",
-    "deepseek",
-    "chatgpt"
-  ],
   "license": "MIT"
 }

package/portal.mjs

@@ -8,9 +8,6 @@ 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);
 }
@@ -18,6 +15,5 @@ async function main() {
 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");
   process.exit(1);
 });