@proagentstore/cli 0.1.21 → 0.1.23

package/dist/browser-runner/browser-actions.js

@@ -7,7 +7,7 @@ import { RunnerInputError } from "./errors.js";
  * matches, hidden checkboxes, and hidden file inputs. Pure over `page`; the
  * runner passes its clickRobustly helper. The caller handles post-action settle.
  */
-export async function performBrowserAction(page, action, clickRobustly) {
+export async function performBrowserAction(page, action, clickRobustly, resumeFile) {
     const locate = () => {
         const role = action.role;
         let loc = role
@@ -135,9 +135,15 @@ export async function performBrowserAction(page, action, clickRobustly) {
             // Always attach via Playwright — never a native dialog. Handles both a
             // direct <input type=file> (set files on it) and a styled "Upload"
             // button that opens a native chooser (intercept the filechooser).
-            const file = resolve(String(action.file ?? ""));
-            if (!file || !existsSync(file))
-                throw new RunnerInputError("upload requires an existing local file path");
+            // The LLM doesn't know the runner's local path, so prefer the résumé the
+            // runner already resolved (downloaded from the platform); fall back to an
+            // explicit action.file only if it's a real local file.
+            const explicit = action.file ? resolve(String(action.file)) : "";
+            const file = resumeFile && existsSync(resumeFile)
+                ? resumeFile
+                : explicit && existsSync(explicit) ? explicit : "";
+            if (!file)
+                throw new RunnerInputError("no résumé available to upload — upload one in the console (Knowledge → Résumé)");
             let done = false;
             // 1. Label-associated input (some forms).
             if (action.name) {

package/dist/browser-runner/runner.js

@@ -1,5 +1,5 @@
 import { spawnSync } from "node:child_process";
-import { existsSync, mkdirSync, rmSync } from "node:fs";
+import { existsSync, mkdirSync, rmSync, writeFileSync } from "node:fs";
 import { createRequire } from "node:module";
 import { basename, dirname, join, resolve } from "node:path";
 import { pathToFileURL } from "node:url";
@@ -393,6 +393,37 @@ export class LocalRunner {
             chooser.setFiles(filePath).catch(() => undefined);
         });
     }
+    resumeFileCache = new Map();
+    /**
+     * Resolve a résumé reference to a LOCAL file path. A remote runner gets a
+     * signed URL (the file lives in the platform's R2) — download it once and
+     * cache it. A same-machine run may still pass a local path.
+     */
+    async resolveResumeFile(ref) {
+        if (/^https?:\/\//i.test(ref)) {
+            const cached = this.resumeFileCache.get(ref);
+            if (cached && existsSync(cached))
+                return cached;
+            try {
+                const res = await fetch(ref);
+                if (!res.ok)
+                    return null;
+                const buf = Buffer.from(await res.arrayBuffer());
+                const disp = res.headers.get("content-disposition") || "";
+                const name = (disp.match(/filename="?([^"]+)"?/i)?.[1] || "resume.pdf").replace(/[^\w.\-]/g, "_");
+                const dir = join(this.config.dataDir, "uploads");
+                mkdirSync(dir, { recursive: true });
+                const dest = join(dir, name);
+                writeFileSync(dest, buf);
+                this.resumeFileCache.set(ref, dest);
+                return dest;
+            }
+            catch {
+                return null;
+            }
+        }
+        return existsSync(resolve(ref)) ? resolve(ref) : null;
+    }
     /** End a takeover session (human finished or gave up). */
     async endTakeover(taskId) {
         const session = this.takeovers.get(taskId);
@@ -472,9 +503,14 @@ export class LocalRunner {
             headless: this.config.headless,
             acceptDownloads: true,
             downloadsPath,
+            // viewport:null + --start-maximized → the page uses the FULL window size
+            // instead of Playwright's cramped 1280x720 default. Without this a
+            // reCAPTCHA challenge popup near the edge overflows and gets cut off the
+            // right in the takeover view, so it can't be solved.
+            viewport: null,
             // Light anti-detection: drop the most obvious automation tell so fewer
             // CAPTCHAs trigger in the first place (the human still solves the rest).
-            args: ["--disable-blink-features=AutomationControlled"],
+            args: ["--disable-blink-features=AutomationControlled", "--start-maximized", "--window-size=1512,982"],
         };
         // Prefer the real Chrome build (better TLS/fingerprint → fewer CAPTCHAs);
         // fall back to bundled Chromium if Chrome isn't installed. Disable with
@@ -583,14 +619,18 @@ export class LocalRunner {
     async browserAct(action, resumePath) {
         const page = await this.getActivePage();
         // Arm résumé auto-attach so a file chooser never blocks the flow (see method).
-        if (resumePath && existsSync(resolve(resumePath))) {
-            this.applyResumePath = resolve(resumePath);
+        // resumePath may be a signed URL (remote runner) or a local path — resolve to
+        // a real local file either way.
+        if (resumePath) {
+            const local = await this.resolveResumeFile(resumePath);
+            if (local)
+                this.applyResumePath = local;
         }
         if (this.applyResumePath)
             this.armFileAutoAttach(page, this.applyResumePath);
         // The selector-free action switch lives in browser-actions.ts (kept this
         // file from ballooning); the post-action settle + result stay here.
-        await performBrowserAction(page, action, (p, loc) => this.clickRobustly(p, loc));
+        await performBrowserAction(page, action, (p, loc) => this.clickRobustly(p, loc), this.applyResumePath);
         // A click may trigger SPA navigation or open a new tab/popup. Give it a beat
         // to settle and follow the (possibly new) active page, so the next snapshot
         // reflects the new state — not the element the brain just clicked.

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@proagentstore/cli",
-	"version": "0.1.21",
+	"version": "0.1.23",
 	"description": "CLI for creating, publishing, and running ProAgentStore agents",
 	"license": "MIT",
 	"type": "module",