@proagentstore/cli 0.4.2 → 0.4.3

package/dist/browser-runner/runner.js

@@ -680,6 +680,45 @@ export class LocalRunner {
         const head = (cut >= 0 ? text.slice(0, cut) : text).replace(/```[a-z]*\n?|```/g, " ").replace(/\s+/g, " ").trim();
         return head.slice(0, 400);
     }
+    /**
+     * After a WRITE action, read the field's REAL value back via the standard
+     * browser_evaluate tool (evaluates on the element by ref — no reinvented DOM
+     * traversal). Masked/intl fields transform input (e.g. "0404453580" → "+61404453580"),
+     * so without this signal the brain can't tell the value "took" and oscillates
+     * between formats. Returns the prompt's expected "…now reads…" / "REJECTED…" line.
+     * Never throws — best-effort feedback.
+     */
+    async readBackField(mcp, action) {
+        if (!action.ref)
+            return "";
+        const fn = "el => ({ v: el.value !== undefined ? String(el.value) : (el.getAttribute('aria-checked') ?? el.textContent ?? '').trim(), " +
+            "bad: el.getAttribute('aria-invalid') === 'true' || !!(el.validity && !el.validity.valid), " +
+            "msg: el.validationMessage || '' })";
+        const res = await mcp.callTool("browser_evaluate", { element: action.name || "field", target: action.ref, function: fn }).catch(() => null);
+        if (!res || res.isError)
+            return "";
+        const txt = mcp.textOf(res);
+        const i = txt.indexOf("### Result");
+        if (i < 0)
+            return "";
+        const after = txt.slice(i + "### Result".length).trim();
+        const end = after.indexOf("\n###");
+        const block = (end >= 0 ? after.slice(0, end) : after).trim();
+        let parsed;
+        try {
+            parsed = JSON.parse(block);
+        }
+        catch {
+            return "";
+        }
+        if (typeof parsed?.v !== "string")
+            return "";
+        const name = action.name || "field";
+        const shown = parsed.v.length > 80 ? `${parsed.v.slice(0, 80)}…` : parsed.v;
+        if (parsed.bad)
+            return `⚠ "${name}" REJECTED: now reads "${shown}"${parsed.msg ? ` — ${parsed.msg.slice(0, 80)}` : ""}`;
+        return `"${name}" now reads "${shown}"`;
+    }
     /** The snapshot ref the brain must target the element by (standard-tool `target`). */
     refOf(action) {
         const ref = (action.ref || "").trim();
@@ -817,12 +856,21 @@ export class LocalRunner {
         if (action.action === "click" || action.action === "navigate" || action.action === "key") {
             await active.waitForLoadState("networkidle", { timeout: 3_500 }).catch(() => undefined);
         }
+        // For a WRITE, read the field's real value back (masked/intl fields transform
+        // input, e.g. "0404453580" → "+61404453580") so the brain knows it took and
+        // stops oscillating between formats — the signal inspectField used to give.
+        let feedback = this.conciseFeedback(text);
+        if (action.action === "type" || action.action === "select" || action.action === "check") {
+            const rb = await this.readBackField(mcp, action);
+            if (rb)
+                feedback = rb;
+        }
         return {
             ok: true,
             url: active.url(),
             title: await active.title().catch(() => ""),
             challenge: await detectHumanChallenge(active),
-            feedback: this.conciseFeedback(text) || undefined,
+            feedback: feedback || undefined,
         };
     }
     // ── Agent-driven application lifecycle (called by the remote Workflow brain) ──

package/package.json

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