@pushary/agent-hooks 0.87.1 → 0.87.2

package/dist/bin/pushary-codex-bridge.js

@@ -36,6 +36,8 @@ import { existsSync } from "fs";
 import { spawn } from "child_process";
 import { basename } from "path";
 var MAX_RPC_LINE = 1024 * 1024;
+var STDERR_TAIL_LIMIT = 2e3;
+var FAILURE_MESSAGE_MAX = 240;
 var MAX_PENDING_RPC = 8;
 var MAX_TRANSCRIPT_TEXT = 32 * 1024;
 var MAX_MESSAGE_BUFFERS = 16;
@@ -46,20 +48,34 @@ var JsonRpcClient = class {
   nextId = 1;
   buffer = "";
   closed = false;
+  stderrTail = "";
+  failure = null;
   constructor(command, cwd, timeoutMs) {
     const options = { cwd, env: process.env, stdio: ["pipe", "pipe", "pipe"] };
     this.child = process.platform === "win32" ? spawnClaude(command.command, command.args ?? [], options) : spawn(command.command, command.args ?? [], options);
     this.child.stdout?.setEncoding("utf8");
     this.child.stdout?.on("data", (chunk) => this.read(String(chunk)));
-    this.child.stderr?.resume();
+    this.child.stderr?.setEncoding("utf8");
+    this.child.stderr?.on("data", (chunk) => this.keepStderrTail(String(chunk)));
     this.child.once("error", (error) => this.fail(error));
-    this.child.once("exit", () => this.fail(new Error("Codex app-server exited")));
+    this.child.once("exit", (code, signal) => this.fail(new Error(this.exitMessage(code, signal))));
     this.timeoutMs = timeoutMs;
   }
+  keepStderrTail(chunk) {
+    this.stderrTail = (this.stderrTail + chunk).slice(-STDERR_TAIL_LIMIT);
+  }
+  exitMessage(code, signal) {
+    const how = signal ? `signal ${signal}` : `code ${code ?? "unknown"}`;
+    const tail = this.stderrTail.trim();
+    return tail ? `Codex app-server exited (${how}): ${tail}` : `Codex app-server exited (${how})`;
+  }
   timeoutMs;
   get isClosed() {
     return this.closed;
   }
+  get closedReason() {
+    return this.failure?.message ?? "Codex app-server exited";
+  }
   onNotification(listener) {
     this.notifications.add(listener);
     return () => this.notifications.delete(listener);
@@ -142,6 +158,7 @@ var JsonRpcClient = class {
   fail(error) {
     if (this.closed) return;
     this.closed = true;
+    this.failure = error;
     this.rejectPending(error);
     if (this.child.exitCode == null) this.child.kill();
   }
@@ -281,7 +298,7 @@ var runCodexBridge = async (options) => {
       await relay?.flush();
       await liveSessionSleep(options.pollIntervalMs ?? 2e3, options.signal);
     }
-    if (rpc.isClosed && !options.signal.aborted) throw new Error("Codex app-server exited");
+    if (rpc.isClosed && !options.signal.aborted) throw new Error(rpc.closedReason);
     if (activeTurnId) {
       await rpc.request("turn/interrupt", { threadId, turnId: activeTurnId }).catch(() => null);
     }
@@ -289,8 +306,8 @@ var runCodexBridge = async (options) => {
     await report("session_closed", "Session stopped").catch(() => null);
     return "stopped";
   } catch (error) {
+    const message = error instanceof Error ? error.message : String(error);
     if (!linked && options.claimId) {
-      const message = error instanceof Error ? error.message : String(error);
       for (let attempt = 0; attempt < 3; attempt++) {
         const acknowledged = await postLiveSession(options, "/api/agent/spawn/ack", {
           spawnId: options.spawnId,
@@ -299,7 +316,7 @@ var runCodexBridge = async (options) => {
           state: "failed",
           failure: {
             code: "bridge_failed",
-            message: message.slice(0, 240),
+            message: message.slice(0, FAILURE_MESSAGE_MAX),
             retryable: false
           }
         }).catch(() => null);
@@ -307,7 +324,7 @@ var runCodexBridge = async (options) => {
         await liveSessionSleep(200 * (attempt + 1), options.signal);
       }
     }
-    await report("error", "Live bridge failed").catch(() => null);
+    await report("error", `Live bridge failed: ${message}`.slice(0, FAILURE_MESSAGE_MAX)).catch(() => null);
     return "failed";
   } finally {
     stopHeartbeat();

package/dist/bin/pushary-connect.js

@@ -6,7 +6,7 @@ import {
   confirmAppConnection,
   connectDevice,
   printConnectInstructions
-} from "../chunk-A3PFIWWN.js";
+} from "../chunk-3LKYBFWA.js";
 import "../chunk-3EGEA4KH.js";
 import {
   checkApiKey

package/dist/bin/pushary-doctor.js

@@ -60,7 +60,7 @@ import {
   fetchChannels,
   reachVerdict,
   verifyRoundTrip
-} from "../chunk-A3PFIWWN.js";
+} from "../chunk-3LKYBFWA.js";
 import "../chunk-3EGEA4KH.js";
 import {
   checkApiKey,

package/dist/bin/pushary-setup.js

@@ -81,7 +81,7 @@ import {
   printConnectInstructions,
   setHumanStream,
   verifyRoundTrip
-} from "../chunk-A3PFIWWN.js";
+} from "../chunk-3LKYBFWA.js";
 import "../chunk-3EGEA4KH.js";
 import {
   checkApiKey,
@@ -1865,7 +1865,8 @@ var main = async () => {
     // `false` is allowed to hold back "Setup complete.".
     roundTripAnswered: roundTrip === null ? null : roundTrip.kind === "answered" ? true : roundTrip.kind === "unanswered" ? false : null
   });
-  const daemon = keyCheck?.kind === "ok" && completed.length > 0 ? await ensureDaemonRunning() : null;
+  const keyIsTrusted = keyFromPairing || keyCheck?.kind === "ok";
+  const daemon = keyIsTrusted && completed.length > 0 ? await ensureDaemonRunning() : null;
   console.log(`  ${closing.ok ? green(bold(closing.headline)) : yellow(bold(closing.headline))}`);
   if (failed.length > 0) {
     console.log();

package/dist/bin/pushary-status.js

@@ -8,7 +8,7 @@ import {
   describeReach,
   fetchChannels,
   reachVerdict
-} from "../chunk-A3PFIWWN.js";
+} from "../chunk-3LKYBFWA.js";
 import "../chunk-3EGEA4KH.js";
 import {
   checkApiKey,

package/dist/{chunk-A3PFIWWN.js → chunk-3LKYBFWA.js}

@@ -72,6 +72,15 @@ var describeAppRepair = (verdict) => {
       ];
   }
 };
+var describeJustPairedRepair = (verdict) => {
+  if (verdict.kind === "nobody") {
+    return [
+      "The app asks for the notification permission after it grants the pairing.",
+      "Open the Pushary app and allow notifications. That is the whole remaining step."
+    ];
+  }
+  return describeAppRepair(verdict) ?? [];
+};
 
 // src/setup/verify.ts
 var plural2 = (count, one) => `${count} ${one}${count === 1 ? "" : "s"}`;
@@ -586,14 +595,55 @@ var describeExistingConnection = (channels) => {
   }
   return "Phone already connected";
 };
+var APP_REGISTER_TIMEOUT_MS = 45e3;
+var appRegisterTimeoutMs = () => {
+  const raw = Number(process.env.PUSHARY_APP_REGISTER_TIMEOUT_MS);
+  return Number.isFinite(raw) && raw >= 0 ? raw : APP_REGISTER_TIMEOUT_MS;
+};
+var ownedApp = (status) => status.yours?.appDevices ?? status.appDevices;
+var waitForAppRegistration = async (apiKey) => {
+  const deadline = Date.now() + appRegisterTimeoutMs();
+  if (Date.now() >= deadline) return { kind: "timeout" };
+  const stop = startSpinner(() => {
+    const left = Math.max(0, Math.ceil((deadline - Date.now()) / 1e3));
+    return `Waiting for your phone to register for notifications ${dim(`(${left}s)`)}`;
+  });
+  try {
+    while (Date.now() < deadline) {
+      await sleep(CONNECT_POLL_INTERVAL_MS);
+      const probe = await fetchChannels(apiKey);
+      if (probe.kind === "unauthorized") {
+        stop(yellow("!"), "The server stopped accepting this key");
+        return { kind: "unauthorized" };
+      }
+      if (probe.kind === "locked") {
+        stop(yellow("!"), "This workspace has no active plan");
+        return { kind: "locked" };
+      }
+      if (probe.kind !== "ok") continue;
+      if (!probe.status.precise || ownedApp(probe.status) > 0) {
+        stop(check, "Phone registered for notifications");
+        return { kind: "connected", channels: probe.status };
+      }
+    }
+  } catch {
+  }
+  stop(yellow("!"), "This phone has not registered for notifications yet");
+  return { kind: "timeout" };
+};
 var confirmAppConnection = async (apiKey) => {
   const probe = await fetchChannels(apiKey);
   if (probe.kind !== "ok") return describeProbeFailure(probe);
-  const channels = probe.status;
-  const ownedApp = channels.yours?.appDevices ?? channels.appDevices;
-  if (channels.precise && ownedApp === 0) {
-    const repair = describeAppRepair(reachVerdict(channels)) ?? [];
-    console.log(`  ${yellow("!")} No device of yours is registered for notifications yet.`);
+  let channels = probe.status;
+  if (channels.precise && ownedApp(channels) === 0) {
+    const settled = await waitForAppRegistration(apiKey);
+    if (settled.kind === "unauthorized") return describeProbeFailure({ kind: "unauthorized" });
+    if (settled.kind === "locked") return describeProbeFailure({ kind: "locked" });
+    if (settled.kind === "connected") channels = settled.channels;
+  }
+  if (channels.precise && ownedApp(channels) === 0) {
+    const repair = describeJustPairedRepair(reachVerdict(channels));
+    console.log(`  ${yellow("!")} Paired, but this phone has not registered for notifications yet.`);
     for (const line of repair) console.log(`  ${dim(line)}`);
     console.log(`  ${dim("Then run")} ${cyan("pushary doctor")}${dim(".")}`);
     return phoneFailure("app_not_registered", repair);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pushary/agent-hooks",
-  "version": "0.87.1",
+  "version": "0.87.2",
   "description": "Permission hooks for AI coding agents: route tool approvals through Pushary push notifications",
   "keywords": [
     "pushary",