@proagentstore/cli 0.3.3 → 0.3.5

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

@@ -194,3 +194,79 @@ export async function performBrowserAction(page, action, clickRobustly, resumeFi
         }
     }
 }
+/**
+ * After a WRITE action (type/select/check), read back what the field actually
+ * holds now + any validation error rendered near it. This is the semantic
+ * feedback the brain otherwise lacks: `fill()` succeeding doesn't mean the value
+ * "took" (masked/intl widgets concatenate; validators reject a format). Returns a
+ * short human string for the action log, or "" when there's nothing notable.
+ * Never throws.
+ */
+export async function inspectField(page, action) {
+    if (action.action !== "type" && action.action !== "select" && action.action !== "check")
+        return "";
+    if (!action.name)
+        return "";
+    try {
+        const role = action.role;
+        let loc = role ? page.getByRole(role, { name: action.name }) : page.getByText(action.name, { exact: false });
+        loc = typeof action.nth === "number" ? loc.nth(action.nth) : loc.first();
+        const el = await loc.elementHandle({ timeout: 1_500 }).catch(() => null);
+        if (!el)
+            return "";
+        const info = await el
+            .evaluate((node) => {
+            const e = node;
+            const raw = typeof e.value === "string" ? e.value : (e.getAttribute("aria-checked") ?? e.textContent ?? "");
+            const value = String(raw).trim().slice(0, 120);
+            const invalid = e.getAttribute("aria-invalid") === "true" || (typeof e.matches === "function" && e.matches(":invalid"));
+            let err = "";
+            const rx = /invalid|required|must|valid|error|format|enter a|please/i;
+            const describedby = e.getAttribute("aria-describedby");
+            if (describedby) {
+                for (const id of describedby.split(/\s+/)) {
+                    const d = document.getElementById(id);
+                    const t = (d?.textContent || "").trim();
+                    if (t && rx.test(t)) {
+                        err = t;
+                        break;
+                    }
+                }
+            }
+            if (!err) {
+                const scope = e.closest("[class*=field], [class*=form-group], [class*=form-item], fieldset") || e.parentElement;
+                const cand = scope?.querySelector('[role="alert"], [class*=error i], [class*=invalid i], [class*=danger i], [class*=help-block i]');
+                const t = (cand?.textContent || "").trim();
+                if (t && rx.test(t) && t.length < 180)
+                    err = t;
+            }
+            return { value, invalid, err: err.replace(/\s+/g, " ").slice(0, 180) };
+        })
+            .catch(() => null);
+        if (!info)
+            return "";
+        const typed = String(action.text ?? "").trim();
+        const stuck = !!typed && info.value === typed; // our value actually took
+        const parts = [];
+        // Report the field's real value when it differs from what we sent, or when the
+        // field is authoritatively invalid.
+        if (info.value && (info.invalid || !stuck)) {
+            parts.push(`"${action.name}" now reads "${info.value}"${typed && info.value !== typed ? ` (you sent "${typed}")` : ""}`);
+        }
+        // Flag REJECTED ONLY on an authoritative per-field signal (aria-invalid / :invalid)
+        // or when our value clearly did NOT take. A nearby error while the value stuck is
+        // often stale (widgets show a "format" error mid-type that clears on blur/submit)
+        // or belongs to another field — reporting it as REJECTED sent the brain into a
+        // false-retry loop on a value that was actually accepted.
+        if (info.invalid || (typed && !stuck)) {
+            if (info.err)
+                parts.unshift(`⚠ "${action.name}" REJECTED: "${info.err}"`);
+            else if (info.invalid)
+                parts.unshift(`⚠ "${action.name}" is marked invalid`);
+        }
+        return parts.join("; ");
+    }
+    catch {
+        return "";
+    }
+}

package/dist/browser-runner/runner.js

@@ -5,7 +5,7 @@ import { basename, dirname, join, resolve } from "node:path";
 import { pathToFileURL } from "node:url";
 import { captureScreenshotDataUrl, challengeSolved, detectHumanChallenge } from "./challenge.js";
 import { resolveRealChromeProfileDir, seedProfileCopy } from "./browser-profile.js";
-import { performBrowserAction } from "./browser-actions.js";
+import { inspectField, performBrowserAction } from "./browser-actions.js";
 import { HumanHandoffError, RunnerInputError } from "./errors.js";
 import { RunnerStore } from "./store.js";
 import { CodingRuntime } from "./coding/runtime.js";
@@ -664,11 +664,15 @@ export class LocalRunner {
         if (action.action === "click" || action.action === "navigate" || action.action === "key") {
             await active.waitForLoadState("networkidle", { timeout: 3_500 }).catch(() => undefined);
         }
+        // Read back what a write action actually did (real field value + any validation
+        // error) so the brain gets semantic feedback, not just "Playwright didn't throw".
+        const feedback = await inspectField(active, action).catch(() => "");
         return {
             ok: true,
             url: active.url(),
             title: await active.title().catch(() => ""),
             challenge: await detectHumanChallenge(active),
+            feedback: feedback || undefined,
         };
     }
     /**

package/package.json

@@ -1,46 +1,46 @@
 {
-	"name": "@proagentstore/cli",
-	"version": "0.3.3",
-	"description": "CLI for creating, publishing, and running ProAgentStore agents",
-	"license": "MIT",
-	"type": "module",
-	"bin": {
-		"pags": "dist/index.js"
-	},
-	"homepage": "https://github.com/ProAgentStore/platform#readme",
-	"repository": {
-		"type": "git",
-		"url": "git+https://github.com/ProAgentStore/platform.git",
-		"directory": "packages/cli"
-	},
-	"keywords": [
-		"proagentstore",
-		"pags",
-		"cli",
-		"agents",
-		"mcp",
-		"browser-runner"
-	],
-	"publishConfig": {
-		"access": "public"
-	},
-	"scripts": {
-		"build": "pnpm --filter @proagentstore/browser-runner build && tsup src/index.ts --format esm --dts && node scripts/copy-browser-runner.mjs",
-		"dev": "tsx src/index.ts",
-		"typecheck": "tsc --noEmit"
-	},
-	"dependencies": {
-		"chalk": "^5.6.2",
-		"commander": "^13.0.0",
-		"playwright": "^1.60.0"
-	},
-	"devDependencies": {
-		"@types/node": "^22.0.0",
-		"tsup": "^8.4.0",
-		"tsx": "^4.21.0",
-		"typescript": "^5.7.0"
-	},
-	"files": [
-		"dist"
-	]
+  "name": "@proagentstore/cli",
+  "version": "0.3.5",
+  "description": "CLI for creating, publishing, and running ProAgentStore agents",
+  "license": "MIT",
+  "type": "module",
+  "bin": {
+    "pags": "dist/index.js"
+  },
+  "homepage": "https://github.com/ProAgentStore/platform#readme",
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/ProAgentStore/platform.git",
+    "directory": "packages/cli"
+  },
+  "keywords": [
+    "proagentstore",
+    "pags",
+    "cli",
+    "agents",
+    "mcp",
+    "browser-runner"
+  ],
+  "publishConfig": {
+    "access": "public"
+  },
+  "scripts": {
+    "build": "pnpm --filter @proagentstore/browser-runner build && tsup src/index.ts --format esm --dts && node scripts/copy-browser-runner.mjs",
+    "dev": "tsx src/index.ts",
+    "typecheck": "tsc --noEmit"
+  },
+  "dependencies": {
+    "chalk": "^5.6.2",
+    "commander": "^13.0.0",
+    "playwright": "^1.60.0"
+  },
+  "devDependencies": {
+    "@types/node": "^22.0.0",
+    "tsup": "^8.4.0",
+    "tsx": "^4.21.0",
+    "typescript": "^5.7.0"
+  },
+  "files": [
+    "dist"
+  ]
 }