@proagentstore/cli 0.4.79 → 0.4.80

package/dist/browser-runner/local-browser/bridge.js

@@ -115,6 +115,8 @@ export class BrowserBridge {
     findings = [];
     sourceFailures = [];
     summary = null;
+    /** Tool calls that reached the bridge. Zero at exit means the CLI never used the browser (#944). */
+    calls = 0;
     constructor(browser, host) {
         this.browser = browser;
         this.host = host;
@@ -128,6 +130,7 @@ export class BrowserBridge {
         ];
     }
     async callTool(name, args = {}) {
+        this.calls++;
         if (name === "record_finding")
             return this.recordFinding(args);
         if (name === "report_source_failure")

package/dist/browser-runner/local-browser/engine.js

@@ -143,6 +143,17 @@ export function signInHelp(engine) {
         ? "The Claude Code CLI on this machine is not signed in. Run `claude`, then /login, on that machine and start the run again."
         : "The Codex CLI on this machine is not signed in. Run `codex login` on that machine and start the run again.";
 }
+/**
+ * Why a run that never touched the browser bridge failed, and the step that fixes it (#944). The usual
+ * cause is the CLI refusing the bridge's tools (an approval policy, #952) or not loading the server.
+ */
+export function bridgeUnused(engine, said) {
+    const blocked = /approval|not (allowed|permitted)|permission|denied/i.test(said);
+    const cause = blocked
+        ? `The ${engine === "claude" ? "Claude Code" : "Codex"} CLI was refused the PAGS browser tools.`
+        : `The ${engine === "claude" ? "Claude Code" : "Codex"} CLI finished without using the PAGS browser tools.`;
+    return `${cause} No page was opened, so this run found nothing. Update the CLI on that machine (npm i -g @proagentstore/cli), restart \`pags up\`, and start the run again.`;
+}
 /** The CLI's own closing text, from its structured output — the summary when it never called finish_research. */
 export function finalText(engine, lines) {
     for (let i = lines.length - 1; i >= 0; i--) {

package/dist/browser-runner/local-browser/runtime.js

@@ -27,7 +27,7 @@ import { RunnerInputError } from "../errors.js";
 import { BrowserBridge, domainWithin } from "./bridge.js";
 import { BRIDGE_ENV } from "./bridge-stdio.js";
 import { LOCAL_BROWSER_AUTH_MODES, LOCAL_BROWSER_ENGINES, LOCAL_BROWSER_TASK_TYPE, redactDetail, redactText, secretEnvValues, } from "./contract.js";
-import { buildEngineSpec, finalText, missingLogin, observedEngineAuth, researchPrompt, signInHelp } from "./engine.js";
+import { bridgeUnused, buildEngineSpec, finalText, missingLogin, observedEngineAuth, researchPrompt, signInHelp } from "./engine.js";
 const MAX_EVENTS = 2000;
 const OUTPUT_LINES = 400;
 const SAFE_ID = /^[A-Za-z0-9_.:-]{1,100}$/;
@@ -387,6 +387,11 @@ export class LocalBrowserRuntime {
             const tail = run.output.slice(-5).join("\n").slice(-800);
             return { outcome: "failed", summary, error: `The ${e.engine} CLI exited with code ${code}${tail ? `: ${tail}` : ""}` };
         }
+        // A CLI that exits cleanly having never called the bridge did no research, whatever its prose
+        // says. Live run aac758dc (#952) ended `completed` with "the browser bridge required approval":
+        // no page opened, and the run looked like a success with nothing found.
+        if (!bridge?.calls)
+            return { outcome: "failed", summary, error: bridgeUnused(e.engine, summary) };
         return { outcome: "completed", summary };
     }
     end(run, r) {

package/package.json

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