@proagentstore/cli 0.4.110 → 0.4.112

package/dist/index.js

@@ -1107,7 +1107,7 @@ import { Command as Command7 } from "commander";
 // src/commands/runner/process.ts
 import { spawn as spawn2 } from "child_process";
 import { existsSync as existsSync6 } from "fs";
-import { resolve as resolve4 } from "path";
+import { resolve as resolve4, win32 } from "path";
 import { createServer as createServer2 } from "net";
 import { fileURLToPath as fileURLToPath2 } from "url";
 import { createRequire } from "module";
@@ -1156,6 +1156,9 @@ function runsFromSource() {
 function bundledRunnerPath() {
   return fileURLToPath2(new URL("./browser-runner/index.js", import.meta.url));
 }
+function spawnNeedsShell(command, platform = process.platform) {
+  return platform === "win32" && !win32.isAbsolute(command);
+}
 function runnerSpawnSpec(opts) {
   const root = findWorkspaceRoot();
   const localPackage = resolve4(root, "packages", "browser-runner", "src", "index.ts");
@@ -1172,7 +1175,7 @@ function runnerSpawnSpec(opts) {
     command = process.execPath;
     args = [bundledPackage, ...runnerArgs];
   }
-  return { command, args, cwd };
+  return { command, args, cwd, shell: spawnNeedsShell(command) };
 }
 function startRunnerForeground(opts) {
   const spec = runnerSpawnSpec(opts);
@@ -1180,7 +1183,7 @@ function startRunnerForeground(opts) {
     const child = spawn2(spec.command, spec.args, {
       cwd: spec.cwd,
       stdio: "inherit",
-      shell: process.platform === "win32"
+      shell: spec.shell
     });
     child.on("error", reject);
     child.on("exit", (code) => {
@@ -1636,7 +1639,8 @@ function parseRecovery(raw) {
   const r = raw;
   if (!r || typeof r.target !== "string" || r.target.length > 80 || typeof r.at !== "string" || !Array.isArray(r.attachments)) return void 0;
   if (r.attachments.length > 200 || !r.attachments.every((a) => typeof a === "string" && a.length > 0 && a.length <= 128)) return void 0;
-  return { target: r.target, attachments: [...r.attachments], at: r.at };
+  const verifiedAt = typeof r.verifiedAt === "string" && r.verifiedAt ? r.verifiedAt : void 0;
+  return { target: r.target, attachments: [...r.attachments], at: r.at, ...verifiedAt ? { verifiedAt } : {} };
 }
 var statuses = /* @__PURE__ */ new Set([
   "enabled",
@@ -1785,6 +1789,7 @@ async function boundedWork(work, bound = {}) {
 function updateVerdict(attempt, facts, attachedNow = []) {
   if (!isVersion(facts.current) || !isVersion(facts.latest)) return { kind: "unverified", reason: `the running (${facts.current}) or published (${facts.latest}) version could not be read` };
   const cycle = attempt.recovery;
+  if (cycle?.verifiedAt) return { kind: "completed" };
   if (!cycle) {
     if (attempt.status === "running" || attempt.status === "restarting") return { kind: "unknown", reason: "an update attempt was recorded by an older runner without the record needed to verify it" };
     return { kind: "current" };
@@ -1936,8 +1941,11 @@ var AutomaticUpdateController = class {
     const plan = this.deps.plan(facts);
     this.decisionAt = this.now();
     this.blockerClass = plan.action === "wait" ? blockerClassFor(plan.waitingFor) : null;
-    if (plan.action === "up-to-date") {
-      const verdict = updateVerdict(this.deps.policy().value, facts, this.deps.attachments?.() ?? []);
+    const policy = this.deps.policy().value;
+    const open = policy.recovery && !policy.recovery.verifiedAt ? policy.recovery : void 0;
+    const reached = !!open && isVersion(open.target) && isVersion(facts.current) && !olderThan(facts.current, open.target);
+    if (plan.action === "up-to-date" || reached) {
+      const verdict = updateVerdict(policy, facts, this.deps.attachments?.() ?? []);
       if (verdict.kind === "unverified") {
         this.failures++;
         this.deps.status("failure", { reason: `Not verified: ${verdict.reason}.` });
@@ -1946,7 +1954,8 @@ var AutomaticUpdateController = class {
         this.deps.status("enabled", { latestVersion: facts.latest ?? void 0, reason: `Not verified: ${verdict.reason}.` });
       } else {
         this.failures = 0;
-        this.deps.status(verdict.kind === "verified" ? "verified-success" : "enabled", { latestVersion: facts.latest ?? void 0, reason: void 0, recovery: void 0 });
+        const resolved = verdict.kind === "verified" && policy.recovery ? { recovery: { ...policy.recovery, verifiedAt: new Date(this.now()).toISOString() } } : {};
+        this.deps.status(verdict.kind === "current" ? "enabled" : "verified-success", { latestVersion: facts.latest ?? void 0, reason: void 0, ...resolved });
       }
       this.schedule();
       return;
@@ -1961,14 +1970,17 @@ var AutomaticUpdateController = class {
       this.defer();
       return;
     }
-    this.deps.status("running", { latestVersion: plan.latest, reason: void 0, recovery: { target: plan.latest, attachments: [...this.deps.attachments?.() ?? []], at: new Date(this.now()).toISOString() } });
-    const restarted = await this.install(plan, true);
+    const target = open && isVersion(open.target) ? open.target : plan.latest;
+    const attempt = { ...plan, latest: target };
+    const recovery = open ?? { target, attachments: [...this.deps.attachments?.() ?? []], at: new Date(this.now()).toISOString() };
+    this.deps.status("running", { latestVersion: target, reason: void 0, recovery });
+    const restarted = await this.install(attempt, true);
     if (restarted) {
-      this.deps.status("restarting", { latestVersion: plan.latest, reason: void 0 });
+      this.deps.status("restarting", { latestVersion: target, reason: void 0 });
       return;
     }
     this.blockerClass = "idle_wait";
-    this.defer(plan.latest);
+    this.defer(target);
   }
   defer(latest) {
     if (this.waiting) return;
@@ -2788,7 +2800,7 @@ function createRunnerCommand() {
     const runner = spawn4(spec.command, spec.args, {
       cwd: spec.cwd,
       stdio: ["ignore", "pipe", "pipe"],
-      shell: process.platform === "win32",
+      shell: spec.shell,
       // #896: the runtime answers `/health` with the lock's identity and stops for a
       // request quoting its nonce — so a contender can prove who is on that port, and a
       // replace is an authorisation rather than a signal aimed at a name pattern.

package/package.json

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