@pushary/agent-hooks 1.9.49 → 1.9.50

package/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## 1.9.50
+
+- Setup no longer reports a problem when it tried to install phone-start on its own and the background service did not start. The run closes on what the approval check found, and the closing notes say phone-start is not running. `--control on` still fails the run when phone-start cannot be made ready.
+
 ## 1.9.49
 
 - A program at a nested path under a system folder, such as `/opt/homebrew/bin/tools/cat` or `/usr/local/bin/x/zsh`, is no longer treated as the system tool of the same name. It asks a person where it used to run as a read, and `pushary codex` no longer reads a script out of a shell at such a path. Programs directly inside `/bin`, `/usr/bin`, `/usr/local/bin` and `/opt/homebrew/bin` are judged as before. The `pushary codex` half needs the matching server.

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

@@ -9,7 +9,7 @@ import {
 import "../chunk-BXK6ZJVJ.js";
 import {
   resolveProviderBinary
-} from "../chunk-OWQHE2VV.js";
+} from "../chunk-GBKSCDEK.js";
 import "../chunk-RIVNNBEK.js";
 import {
   withSpawnProcessOwnership

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

@@ -16,7 +16,7 @@ import {
 import "../chunk-BXK6ZJVJ.js";
 import {
   resolveProviderBinary
-} from "../chunk-OWQHE2VV.js";
+} from "../chunk-GBKSCDEK.js";
 import "../chunk-RIVNNBEK.js";
 import "../chunk-FPKQGWAF.js";
 import "../chunk-XNZB274I.js";

package/dist/bin/pushary-daemon.js

@@ -5,7 +5,7 @@ import {
 import {
   HEARTBEAT_INTERVAL_MS,
   sendHeartbeat
-} from "../chunk-QYBL4EHD.js";
+} from "../chunk-GTGXQ7FJ.js";
 import "../chunk-KIO2MZQC.js";
 import {
   writeHuman
@@ -59,7 +59,7 @@ import {
   launchableProviders,
   resolveControlPolicy,
   trackProviderInventory
-} from "../chunk-OWQHE2VV.js";
+} from "../chunk-GBKSCDEK.js";
 import "../chunk-RIVNNBEK.js";
 import {
   SESSION_REAP_SILENCE_MS,

package/dist/bin/pushary-doctor.js

@@ -26,7 +26,7 @@ import "../chunk-3EGEA4KH.js";
 import {
   readMachineReadiness,
   readManagedControlEvidence
-} from "../chunk-QYBL4EHD.js";
+} from "../chunk-GTGXQ7FJ.js";
 import {
   assessCodexHookTrust,
   readCodexTrustForPusharyHooks
@@ -148,7 +148,7 @@ import {
   expectedLaunchCapabilities,
   hasSpawnProvider,
   resolveControlPolicy
-} from "../chunk-OWQHE2VV.js";
+} from "../chunk-GBKSCDEK.js";
 import "../chunk-RIVNNBEK.js";
 import "../chunk-XHKBHWLX.js";
 import "../chunk-6ICPEJY5.js";

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

@@ -12,7 +12,7 @@ import {
 } from "../chunk-BXK6ZJVJ.js";
 import {
   resolveProviderBinary
-} from "../chunk-OWQHE2VV.js";
+} from "../chunk-GBKSCDEK.js";
 import "../chunk-RIVNNBEK.js";
 import {
   SESSION_IDLE_TIMEOUT_MS,

package/dist/bin/pushary-setup.js

@@ -44,7 +44,7 @@ import "../chunk-3EGEA4KH.js";
 import {
   readManagedControlEvidence,
   waitForMachineReadiness
-} from "../chunk-QYBL4EHD.js";
+} from "../chunk-GTGXQ7FJ.js";
 import {
   ensureCodexTrustsPusharyHooks
 } from "../chunk-KIO2MZQC.js";
@@ -220,8 +220,9 @@ import {
   hasSpawnProvider,
   missingControlPrerequisite,
   nodeCodexInstallProbe,
+  phoneStartIsRequired,
   resolveControlPolicy
-} from "../chunk-OWQHE2VV.js";
+} from "../chunk-GBKSCDEK.js";
 import {
   detectClaudeVersion
 } from "../chunk-RIVNNBEK.js";
@@ -693,6 +694,15 @@ var COMMAND_SURFACE = [
 ];
 var SURFACE_COLUMN = Math.max(...COMMAND_SURFACE.map((entry) => entry.command.length)) + 2;
 var renderCommandSurface = () => COMMAND_SURFACE.map((entry) => `${entry.command.padEnd(SURFACE_COLUMN)}${entry.summary}`);
+var PHONE_START_VERIFIED_LINE = "Phone-start and steering: local and server checks passed just now. Recheck after sleep or network changes.";
+var PHONE_START_NOT_RUNNING_LINE = "Phone-start and steering: not running on this computer. Approvals do not depend on it. Run pushary doctor to see what is missing.";
+var PHONE_START_UNVERIFIED_LINE = "Phone-start and steering: unverified. Run pushary doctor to check the provider, background service and server route.";
+var phoneStartStatusLine = (facts) => {
+  if (facts.managedControlReady === true) return PHONE_START_VERIFIED_LINE;
+  if (facts.daemonReady === false) return PHONE_START_NOT_RUNNING_LINE;
+  return PHONE_START_UNVERIFIED_LINE;
+};
+var phoneStartFailedTheRun = (facts) => facts.daemonReady === false && facts.phoneStartRequired !== false;
 var PHONE_START_UNVERIFIED_STEPS = [
   "Agent configuration and managed control are separate. Phone-start and steering could not be verified.",
   "pushary daemon install   install or repair the background service",
@@ -712,7 +722,7 @@ var WHAT_PHONE_START_NEEDS = {
 };
 var phoneStartProblemSteps = (missing) => missing ? [PHONE_START_CANNOT_RUN_LINE, ...WHAT_PHONE_START_NEEDS[missing], STOP_REQUIRING_PHONE_START_STEP] : PHONE_START_UNVERIFIED_STEPS;
 var nextSteps = (facts, reachable) => {
-  const lines = ["Delivery follows the current approval mode, recipient rules and presence. pushary mode status shows the workspace setting.", facts.managedControlReady === true ? "Phone-start and steering: local and server checks passed just now. Recheck after sleep or network changes." : "Phone-start and steering: unverified. Run pushary doctor to check the provider, background service and server route."];
+  const lines = ["Delivery follows the current approval mode, recipient rules and presence. pushary mode status shows the workspace setting.", phoneStartStatusLine(facts)];
   lines.push("Before stepping away, run pushary wait to see how long phone approvals wait and what happens if you do not answer.");
   if (facts.staleEnvironmentKey) {
     lines.push("Open a new terminal or unset PUSHARY_API_KEY, then run pushary doctor before starting an agent.");
@@ -767,7 +777,7 @@ var describeClosing = (facts) => {
       reason: "stale-environment-key"
     };
   }
-  if (facts.daemonReady === false) {
+  if (phoneStartFailedTheRun(facts)) {
     return {
       ok: false,
       headline: "Setup finished with 1 problem.",
@@ -3084,6 +3094,7 @@ var main = async () => {
     reachesYou,
     managedControlReady: !controlLifecycleFailure && managedControl.ready,
     daemonReady: controlLifecycleFailure ? false : daemonReady,
+    phoneStartRequired: phoneStartIsRequired({ mode: controlMode, policy: controlPolicy, lifecycleFailed: controlLifecycleFailure !== null }),
     phoneStartMissingPrerequisite: controlMissingPrerequisite,
     transcriptSync,
     // null when the check did not run, so it changes nothing. Only an outright

package/dist/bin/pushary-status.js

@@ -8,7 +8,7 @@ import "../chunk-3EGEA4KH.js";
 import {
   describeLastSeen,
   listMachines
-} from "../chunk-QYBL4EHD.js";
+} from "../chunk-GTGXQ7FJ.js";
 import "../chunk-QCGNGEGE.js";
 import {
   CONNECTED_THROUGH_MAC_APP,
@@ -39,7 +39,7 @@ import {
 import {
   exitOnHelpFlag
 } from "../chunk-BOVSUH4N.js";
-import "../chunk-OWQHE2VV.js";
+import "../chunk-GBKSCDEK.js";
 import "../chunk-RIVNNBEK.js";
 import "../chunk-XHKBHWLX.js";
 import "../chunk-6ICPEJY5.js";

package/dist/bin/pushary-upgrade.js

@@ -22,7 +22,7 @@ import {
 } from "../chunk-HXRH7ZCK.js";
 import {
   waitForMachineReadiness
-} from "../chunk-QYBL4EHD.js";
+} from "../chunk-GTGXQ7FJ.js";
 import "../chunk-KIO2MZQC.js";
 import {
   readMacAppAgentKey
@@ -68,7 +68,7 @@ import {
   expectedLaunchCapabilities,
   hasSpawnProvider,
   resolveControlPolicy
-} from "../chunk-OWQHE2VV.js";
+} from "../chunk-GBKSCDEK.js";
 import "../chunk-RIVNNBEK.js";
 import "../chunk-XHKBHWLX.js";
 import "../chunk-6ICPEJY5.js";

package/dist/{chunk-OWQHE2VV.js → chunk-GBKSCDEK.js}

@@ -123,6 +123,7 @@ var missingControlPrerequisite = (input) => {
   if (!input.managerAvailable) return "service-manager";
   return null;
 };
+var phoneStartIsRequired = (input) => input.mode === "on" || input.policy === "legacy" || input.lifecycleFailed;
 var preferGlobalProviderBinary = (platform) => platform === "win32";
 var resolveProviderBinary = (provider, platform = process.platform) => preferGlobalProviderBinary(platform) ? resolveGlobalBinary(provider) ?? resolvePathBinary(provider, platform) : resolvePathBinary(provider, platform) ?? resolveGlobalBinary(provider);
 var codexBinaryIsBroken = (binary) => codexInstallHealthAt(binary).kind === "binary-missing";
@@ -195,6 +196,7 @@ export {
   hasSpawnProvider,
   resolveControlPolicy,
   missingControlPrerequisite,
+  phoneStartIsRequired,
   resolveProviderBinary,
   codexBinaryIsBroken,
   detectProviderLiveCapabilities,

package/dist/{chunk-QYBL4EHD.js → chunk-GTGXQ7FJ.js}

@@ -14,7 +14,7 @@ import {
   expectedLaunchCapabilities,
   hasSpawnProvider,
   resolveControlPolicy
-} from "./chunk-OWQHE2VV.js";
+} from "./chunk-GBKSCDEK.js";
 import {
   getMachineId
 } from "./chunk-LY3OJFBW.js";

package/package.json

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