@proagentstore/cli 0.4.7 → 0.4.9

package/dist/browser-runner/runner.js

@@ -41,6 +41,10 @@ export class LocalRunner {
      */
     activePage = null;
     fileAttachArmed = new WeakSet();
+    // When the explicit `upload` action is driving a chooser itself (browser_file_upload),
+    // suppress the global auto-attach handler so the résumé isn't attached TWICE (the click
+    // that opens the chooser would otherwise trigger both auto-attach AND browser_file_upload).
+    suppressAutoAttach = false;
     applyResumePath = null;
     /**
      * The INDUSTRY-STANDARD `@playwright/mcp` server, attached over CDP to the same
@@ -411,6 +415,9 @@ export class LocalRunner {
             return;
         this.fileAttachArmed.add(page);
         page.on("filechooser", (chooser) => {
+            // Skip when an explicit upload action is already attaching (avoids double-attach).
+            if (this.suppressAutoAttach)
+                return;
             chooser.setFiles(filePath).catch(() => undefined);
         });
     }
@@ -761,13 +768,32 @@ export class LocalRunner {
             case "check":
                 return mcp.callTool("browser_click", { element: label, target: this.refOf(a) });
             case "upload": {
-                // Standard two-step: clicking the upload control opens the file chooser,
-                // which puts the @playwright/mcp server into its file-chooser modal state;
-                // browser_file_upload then resolves that state with the résumé path.
-                await mcp.callTool("browser_click", { element: label, target: this.refOf(a) });
                 if (!this.applyResumePath)
                     throw new RunnerInputError("no résumé file available to upload");
-                return mcp.callTool("browser_file_upload", { paths: [this.applyResumePath] });
+                // Standard two-step FIRST (respects the brain's target): clicking the upload
+                // control opens the file chooser → browser_file_upload attaches the résumé.
+                // Suppress the global auto-attach handler for the duration so the résumé isn't
+                // attached twice (the click below can itself open a chooser the handler grabs).
+                this.suppressAutoAttach = true;
+                try {
+                    const ref = (a.ref || "").trim();
+                    if (ref)
+                        await mcp.callTool("browser_click", { element: label, target: ref }).catch(() => undefined);
+                    const res = await mcp.callTool("browser_file_upload", { paths: [this.applyResumePath] });
+                    if (!res.isError || !/modal state|file chooser/i.test(mcp.textOf(res)))
+                        return res;
+                    // The target was a LABEL / DROP-ZONE, not the chooser trigger, so no file
+                    // chooser opened. Fall back to setting the résumé directly on the page's
+                    // file input — robust across hidden inputs, drop-zones and custom upload
+                    // widgets (the semantically-correct upload field the brain clicks is often
+                    // a container, not the button that fires the chooser).
+                    const page = await this.getActivePage();
+                    await page.locator('input[type="file"]').first().setInputFiles(this.applyResumePath, { timeout: 8_000 });
+                    return { content: [{ type: "text", text: "résumé attached directly to the file input" }] };
+                }
+                finally {
+                    this.suppressAutoAttach = false;
+                }
             }
             case "select":
                 return mcp.callTool("browser_select_option", { element: label, target: this.refOf(a), values: [a.text ?? ""] });

package/package.json

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