@pushary/agent-hooks 0.78.0 → 0.80.0

package/dist/bin/pushary-setup.js

@@ -1,17 +1,12 @@
 #!/usr/bin/env node
-import {
-  runBrowserLogin
-} from "../chunk-Z3Z4VK5F.js";
-import {
-  addClaudeMcpServer,
-  addPusharyHooks,
-  addPusharyToolPermissions
-} from "../chunk-3CBPY2G5.js";
 import {
   claudeWired,
   codexWired,
   geminiWired
-} from "../chunk-46UEJXQO.js";
+} from "../chunk-UBUPETEQ.js";
+import {
+  runBrowserLogin
+} from "../chunk-Z3Z4VK5F.js";
 import {
   describeDetection,
   detectAllAgents,
@@ -23,12 +18,17 @@ import {
   shortenHome,
   vscodeSettingsTargets,
   writeInstructionBlock
-} from "../chunk-IZA7SWXX.js";
+} from "../chunk-WYVKPLVT.js";
 import {
   GEMINI_HOOK_BINARY,
   addGeminiHooks,
   addGeminiMcpServer
-} from "../chunk-CC2K2SST.js";
+} from "../chunk-7DYFAM32.js";
+import {
+  addClaudeMcpServer,
+  addPusharyHooks,
+  addPusharyToolPermissions
+} from "../chunk-R6AYBATA.js";
 import {
   addCodexHookTrust,
   addCodexHooks,
@@ -46,15 +46,24 @@ import {
   geminiSettings,
   pusharyConfigFile,
   vscodePluginDir
-} from "../chunk-QCYN5VXB.js";
+} from "../chunk-Q4QULGAV.js";
+import {
+  reportEvent
+} from "../chunk-DUYM5SHH.js";
 import {
   confirmAppConnection,
   connectDevice,
   connectViaAppPairing,
+  describeRoundTrip,
   printConnectInstructions,
-  setHumanStream
-} from "../chunk-MGFZXUUR.js";
+  setHumanStream,
+  verifyRoundTrip
+} from "../chunk-BOMF4C2Q.js";
 import "../chunk-3EGEA4KH.js";
+import {
+  checkApiKey,
+  describeKeyCheck
+} from "../chunk-EBP4YG75.js";
 import {
   KEY_FILE_MODE,
   exportLine,
@@ -62,7 +71,7 @@ import {
   readConfigFileKey,
   readKeySource,
   writeKey
-} from "../chunk-NHKWU7MN.js";
+} from "../chunk-E6I3DBJW.js";
 import {
   backupFile,
   readJsonSafe,
@@ -70,16 +79,21 @@ import {
   writeJsonAtomic
 } from "../chunk-6MTNS63X.js";
 import {
-  checkApiKey,
-  describeKeyCheck
-} from "../chunk-DLBW37U6.js";
+  parseFlags
+} from "../chunk-GMXKITVA.js";
 import {
   installClaudeAlias,
   resolveShellRc
 } from "../chunk-BC3VCZ3E.js";
+import "../chunk-BSZYIAZL.js";
+import "../chunk-SAF6HGAA.js";
+import "../chunk-PPD3HHNJ.js";
 import {
-  parseFlags
-} from "../chunk-GMXKITVA.js";
+  UsageError
+} from "../chunk-7QLSKOSU.js";
+import {
+  isValidApiKey
+} from "../chunk-T6BNEQ2A.js";
 import {
   installGlobally,
   npmErrorMessage,
@@ -87,23 +101,12 @@ import {
   resolveGlobalBinDir,
   resolveGlobalBinary
 } from "../chunk-MUEW424A.js";
+import "../chunk-RN3NOEJF.js";
 import {
   getPackageVersion,
   isNewerVersion,
   renderCommandHelp
-} from "../chunk-7OYGFJYZ.js";
-import {
-  reportEvent
-} from "../chunk-HPDLDQC5.js";
-import "../chunk-BSZYIAZL.js";
-import "../chunk-SAF6HGAA.js";
-import {
-  UsageError
-} from "../chunk-7QLSKOSU.js";
-import {
-  isValidApiKey
-} from "../chunk-TJJBF73A.js";
-import "../chunk-RN3NOEJF.js";
+} from "../chunk-ZUKH2NPJ.js";
 import "../chunk-2UMNXADU.js";
 import {
   EXIT
@@ -166,6 +169,14 @@ var describeClosing = (facts) => {
       exitCode: EXIT.PROBLEMS_FOUND
     };
   }
+  if (facts.roundTripAnswered === true) {
+    return {
+      ok: true,
+      headline: "Setup complete, and proven.",
+      next: [...nextSteps(facts, true), "", `Approvals: ${ACTIVITY_URL}`],
+      exitCode: EXIT.OK
+    };
+  }
   if (facts.reachChecked && !facts.reachesYou) {
     return {
       ok: false,
@@ -178,6 +189,22 @@ var describeClosing = (facts) => {
       exitCode: EXIT.NO_DEVICE
     };
   }
+  if (facts.roundTripAnswered === false) {
+    return {
+      ok: false,
+      headline: "Agents configured. Nobody answered the test question.",
+      next: [
+        "The push reached a device and no answer came back, which is the failure that usually",
+        "goes unnoticed: approvals will keep arriving and keep going unanswered.",
+        "",
+        "Check the notification opened something you could tap, then:",
+        "pushary doctor --roundtrip   send another one",
+        "",
+        "If you simply were not looking, re-run setup or pass --verify push to skip this check."
+      ],
+      exitCode: EXIT.PROBLEMS_FOUND
+    };
+  }
   if (!facts.reachChecked) {
     return {
       ok: true,
@@ -307,6 +334,20 @@ var installAgentSkill = async (request, effects) => {
 };
 
 // src/setup/telemetry.ts
+var roundTripOutcome = (outcome) => {
+  switch (outcome?.kind) {
+    case "answered":
+      return "answered";
+    case "notify-only":
+      return "delivered";
+    case "unanswered":
+      return "delivered";
+    case "no-recipients":
+      return "unreachable";
+    default:
+      return null;
+  }
+};
 var currentStep = "start";
 var markStep = (step) => {
   currentStep = step;
@@ -373,9 +414,20 @@ var parseConnectMode = (raw) => {
   if (CONNECT_MODES.includes(value)) return value;
   throw new UsageError(`--connect must be one of: ${CONNECT_MODES.join(", ")} (got ${raw})`);
 };
+var VERIFY_MODES = ["roundtrip", "push", "none"];
+var parseVerifyMode = (raw) => {
+  const value = raw.trim().toLowerCase();
+  if (VERIFY_MODES.includes(value)) return value;
+  throw new UsageError(`--verify must be one of: ${VERIFY_MODES.join(", ")} (got ${raw})`);
+};
+var effectiveVerify = (options, context) => {
+  if (options.verifyExplicit) return options.verify;
+  if (options.verify === "roundtrip") return "roundtrip";
+  return context.paired ? "roundtrip" : options.verify;
+};
 var SETUP_FLAGS = {
   bools: ["key-stdin", "skip-phone", "yes", "y", "json", "dry-run", "help", "h"],
-  values: ["key", "connect", "agents"]
+  values: ["key", "connect", "agents", "verify"]
 };
 var isAgentChoice = (value) => AGENT_CHOICES.includes(value);
 var parseAgentSelection = (raw, yes) => {
@@ -412,6 +464,9 @@ var parseSetupOptions = (input2) => {
   const agents = parseAgentSelection(agentsFlag, yes);
   const connectRaw = flags.values.get("connect");
   const connectMode = connectRaw === void 0 ? "app" : parseConnectMode(connectRaw);
+  const verifyRaw = flags.values.get("verify");
+  const nonInteractive = yes || agentsFlag !== void 0 || flags.bools.has("json") || !input2.stdinIsTty;
+  const verify = verifyRaw !== void 0 ? parseVerifyMode(verifyRaw) : nonInteractive ? "push" : "roundtrip";
   const key = flags.values.get("key")?.trim() || null;
   const keyStdin = flags.bools.has("key-stdin");
   if (key && keyStdin) {
@@ -437,13 +492,15 @@ var parseSetupOptions = (input2) => {
     // cannot see it is waiting for, and whatever is reading stdout waits forever
     // for an object that never arrives. Asking for machine-readable output is
     // itself a statement that no human is watching.
-    nonInteractive: yes || agentsFlag !== void 0 || flags.bools.has("json") || !input2.stdinIsTty,
+    nonInteractive,
     json: flags.bools.has("json"),
     dryRun: flags.bools.has("dry-run"),
     help,
     agents,
     key,
-    keyStdin
+    keyStdin,
+    verify,
+    verifyExplicit: verifyRaw !== void 0
   };
 };
 
@@ -1722,6 +1779,31 @@ var main = async () => {
   } else {
     phone = await connectDevice(trimmedKey, { showCapabilities: !options.connectExplicit });
   }
+  let roundTrip = null;
+  const verifyMode = effectiveVerify(options, { paired: keyFromPairing });
+  if (verifyMode === "roundtrip" && keyCheck?.kind !== "locked" && !skipPhone && // Reaching nobody is already reported by the delivery check above, in copy
+  // that names the fix. Asking a question nobody can receive would repeat it
+  // 55 seconds later and add nothing.
+  //
+  // `unattributable` is the exception and the case this is best at. It means
+  // the push landed on a multi-member site through a channel that carries no
+  // owner, so nothing in the counts can say whether it was THIS user's device.
+  // A tap settles it, which is why describeVerifyOutcome already tells people
+  // to run `doctor --roundtrip` here. Doing it inline saves the round trip
+  // being a thing they have to know to run.
+  (phone?.reachesYou || phone?.outcome?.kind === "unattributable")) {
+    console.log();
+    console.log(`  ${dim("One last check. Answering it is the proof.")}`);
+    await spinner("Waiting for you to tap the notification", async () => {
+      roundTrip = await verifyRoundTrip(trimmedKey);
+    });
+    const report = describeRoundTrip(roundTrip);
+    console.log(`  ${report.ok ? check : yellow("!")} ${report.message}`);
+    for (const line of report.next) console.log(`  ${dim(line)}`);
+  } else if (verifyMode === "roundtrip") {
+    console.log();
+    console.log(`  ${dim("Skipped the round trip: nothing is connected that could answer it.")}`);
+  }
   console.log();
   if (keyCheck?.kind === "locked") {
     const report = describeKeyCheck(keyCheck);
@@ -1765,7 +1847,10 @@ var main = async () => {
     completed,
     failed,
     reachChecked,
-    reachesYou
+    reachesYou,
+    // null when the check did not run, so it changes nothing. Only an outright
+    // `false` is allowed to hold back "Setup complete.".
+    roundTripAnswered: roundTrip === null ? null : roundTrip.kind === "answered" ? true : roundTrip.kind === "unanswered" ? false : null
   });
   console.log(`  ${closing.ok ? green(bold(closing.headline)) : yellow(bold(closing.headline))}`);
   if (failed.length > 0) {
@@ -1785,10 +1870,11 @@ var main = async () => {
     completed,
     skipped,
     failed,
-    // 'answered' is deliberately unreachable here: it means a human tapped, and
-    // setup no longer sends a question. It stays in the type for doctor
-    // --roundtrip, which does.
-    connectOutcome: phone === null ? "skipped" : phone.reachesYou ? "delivered" : "unreachable",
+    // 'answered' is finally reachable. It was in the enum from the start and
+    // nothing produced it, because setup stopped at a delivered notification, so
+    // the one value meaning "a human tapped" was never emitted by the command
+    // whose job is to prove a human can.
+    connectOutcome: roundTripOutcome(roundTrip) ?? (phone === null ? "skipped" : phone.reachesYou ? "delivered" : "unreachable"),
     keyCheck: keyCheck?.kind ?? "skipped",
     exitCode
   });
@@ -1800,6 +1886,10 @@ var main = async () => {
     failed,
     keyCheck: keyCheck?.kind ?? "skipped",
     reach: phone === null ? "skipped" : phone.reachesYou ? "yours" : phone.outcome?.kind ?? phone.detail ?? "unreachable",
+    // Named separately from `reach` so a caller can tell "a push landed" from
+    // "a person answered" without inferring it. 'skipped' means the check did
+    // not run, never that it failed.
+    roundTrip: roundTrip === null ? "skipped" : roundTrip.kind,
     rc: rc.kind,
     rcPath: rc.path,
     exitCode

package/dist/bin/pushary-stats.js

@@ -1,12 +1,12 @@
 #!/usr/bin/env node
-import {
-  exitOnHelpFlag
-} from "../chunk-BBK3TTU2.js";
-import "../chunk-7OYGFJYZ.js";
 import {
   readLedgerSummary
-} from "../chunk-IA5GBKNL.js";
-import "../chunk-TJJBF73A.js";
+} from "../chunk-MHA2WB7S.js";
+import "../chunk-T6BNEQ2A.js";
+import {
+  exitOnHelpFlag
+} from "../chunk-RRZZ7TFJ.js";
+import "../chunk-ZUKH2NPJ.js";
 import {
   getApiKey
 } from "../chunk-2UMNXADU.js";
@@ -40,5 +40,5 @@ if (connected) {
   console.log("A key is configured now, so new moments reach your phone instead of waiting in the terminal.");
 } else {
   console.log("Each of those sat in the terminal until you noticed. With Pushary they land on your lock screen as a yes or no.");
-  console.log("7-day trial: https://pushary.com/sign-up?utm_source=cli&utm_medium=stats");
+  console.log("3-day trial: https://pushary.com/sign-up?utm_source=cli&utm_medium=stats");
 }

package/dist/bin/pushary-status.js

@@ -1,40 +1,41 @@
 #!/usr/bin/env node
-import {
-  describeLastSeen,
-  listMachines
-} from "../chunk-NQTRA7H5.js";
 import {
   describeReach,
   fetchChannels,
   reachVerdict
-} from "../chunk-MGFZXUUR.js";
+} from "../chunk-BOMF4C2Q.js";
 import "../chunk-3EGEA4KH.js";
-import {
-  readKeySource
-} from "../chunk-NHKWU7MN.js";
-import "../chunk-6MTNS63X.js";
 import {
   checkApiKey,
   describeKeyCheck
-} from "../chunk-DLBW37U6.js";
+} from "../chunk-EBP4YG75.js";
 import {
   createIo
 } from "../chunk-TX7KBKT7.js";
-import "../chunk-BC3VCZ3E.js";
+import {
+  readKeySource
+} from "../chunk-E6I3DBJW.js";
+import "../chunk-6MTNS63X.js";
 import {
   rejectUnknownFlags
 } from "../chunk-GMXKITVA.js";
-import {
-  exitOnHelpFlag
-} from "../chunk-BBK3TTU2.js";
-import "../chunk-7OYGFJYZ.js";
+import "../chunk-BC3VCZ3E.js";
 import "../chunk-BSZYIAZL.js";
 import "../chunk-SAF6HGAA.js";
+import "../chunk-PPD3HHNJ.js";
 import {
   UsageError
 } from "../chunk-7QLSKOSU.js";
-import "../chunk-TJJBF73A.js";
+import "../chunk-T6BNEQ2A.js";
+import {
+  describeLastSeen,
+  listMachines
+} from "../chunk-GQLB2GBJ.js";
 import "../chunk-RN3NOEJF.js";
+import {
+  exitOnHelpFlag
+} from "../chunk-RRZZ7TFJ.js";
+import "../chunk-ZUKH2NPJ.js";
 import "../chunk-2UMNXADU.js";
 import {
   EXIT

package/dist/bin/pushary-stop-hook.js

@@ -1,11 +1,12 @@
 #!/usr/bin/env node
 import {
   handleStop
-} from "../chunk-HPDLDQC5.js";
+} from "../chunk-DUYM5SHH.js";
 import "../chunk-BSZYIAZL.js";
 import "../chunk-SAF6HGAA.js";
+import "../chunk-PPD3HHNJ.js";
 import "../chunk-7QLSKOSU.js";
-import "../chunk-TJJBF73A.js";
+import "../chunk-T6BNEQ2A.js";
 import "../chunk-RN3NOEJF.js";
 import "../chunk-2UMNXADU.js";
 import "../chunk-KZERVKTD.js";

package/dist/bin/pushary-stopfailure-hook.js

@@ -1,11 +1,12 @@
 #!/usr/bin/env node
 import {
   handleStopFailure
-} from "../chunk-HPDLDQC5.js";
+} from "../chunk-DUYM5SHH.js";
 import "../chunk-BSZYIAZL.js";
 import "../chunk-SAF6HGAA.js";
+import "../chunk-PPD3HHNJ.js";
 import "../chunk-7QLSKOSU.js";
-import "../chunk-TJJBF73A.js";
+import "../chunk-T6BNEQ2A.js";
 import "../chunk-RN3NOEJF.js";
 import "../chunk-2UMNXADU.js";
 import "../chunk-KZERVKTD.js";

package/dist/bin/pushary-suggestions.js

@@ -5,13 +5,13 @@ import {
 import {
   rejectUnknownFlags
 } from "../chunk-GMXKITVA.js";
-import {
-  exitOnHelpFlag
-} from "../chunk-BBK3TTU2.js";
-import "../chunk-7OYGFJYZ.js";
 import {
   UsageError
 } from "../chunk-7QLSKOSU.js";
+import {
+  exitOnHelpFlag
+} from "../chunk-RRZZ7TFJ.js";
+import "../chunk-ZUKH2NPJ.js";
 import {
   getApiKey,
   getBaseUrl

package/dist/bin/pushary-upgrade.js

@@ -1,15 +1,15 @@
 #!/usr/bin/env node
-import {
-  addClaudeMcpServer,
-  addPusharyHooks,
-  addPusharyToolPermissions
-} from "../chunk-3CBPY2G5.js";
 import {
   GEMINI_HOOK_BINARY,
   addGeminiHooks,
   addGeminiMcpServer,
   hasGeminiHooks
-} from "../chunk-CC2K2SST.js";
+} from "../chunk-7DYFAM32.js";
+import {
+  addClaudeMcpServer,
+  addPusharyHooks,
+  addPusharyToolPermissions
+} from "../chunk-R6AYBATA.js";
 import {
   CODEX_HOOK_BINARY,
   addCodexHookTrust,
@@ -23,10 +23,11 @@ import {
   hasCodexHooks,
   vscodePluginDir,
   vscodePluginMcp
-} from "../chunk-QCYN5VXB.js";
+} from "../chunk-Q4QULGAV.js";
 import {
   readJsonSafe
 } from "../chunk-6MTNS63X.js";
+import "../chunk-T6BNEQ2A.js";
 import {
   execNpm,
   npmErrorMessage,
@@ -36,11 +37,10 @@ import {
 } from "../chunk-MUEW424A.js";
 import {
   exitOnHelpFlag
-} from "../chunk-BBK3TTU2.js";
+} from "../chunk-RRZZ7TFJ.js";
 import {
   isNewerVersion
-} from "../chunk-7OYGFJYZ.js";
-import "../chunk-TJJBF73A.js";
+} from "../chunk-ZUKH2NPJ.js";
 import {
   getApiKey
 } from "../chunk-2UMNXADU.js";

package/dist/bin/pushary-wait.js

@@ -1,26 +1,26 @@
 #!/usr/bin/env node
-import {
-  exitOnFailedResponse
-} from "../chunk-7BBUTWBD.js";
 import {
   describeWaitLadder
 } from "../chunk-IIENZE2J.js";
-import "../chunk-DLBW37U6.js";
+import {
+  exitOnFailedResponse
+} from "../chunk-U2PZKYZM.js";
+import "../chunk-EBP4YG75.js";
 import {
   createIo
 } from "../chunk-TX7KBKT7.js";
 import {
   rejectUnknownFlags
 } from "../chunk-GMXKITVA.js";
-import {
-  exitOnHelpFlag
-} from "../chunk-BBK3TTU2.js";
-import "../chunk-7OYGFJYZ.js";
 import "../chunk-SAF6HGAA.js";
 import {
   UsageError
 } from "../chunk-7QLSKOSU.js";
-import "../chunk-TJJBF73A.js";
+import "../chunk-T6BNEQ2A.js";
+import {
+  exitOnHelpFlag
+} from "../chunk-RRZZ7TFJ.js";
+import "../chunk-ZUKH2NPJ.js";
 import {
   getApiKey,
   getBaseUrl

package/dist/bin/pushary.js

@@ -1,14 +1,14 @@
 #!/usr/bin/env node
+import {
+  installProxyDispatcher
+} from "../chunk-SAF6HGAA.js";
+import "../chunk-7QLSKOSU.js";
 import {
   COMMANDS,
   getPackageVersion,
   isCommandName,
   renderHelp
-} from "../chunk-7OYGFJYZ.js";
-import {
-  installProxyDispatcher
-} from "../chunk-SAF6HGAA.js";
-import "../chunk-7QLSKOSU.js";
+} from "../chunk-ZUKH2NPJ.js";
 import {
   EXIT
 } from "../chunk-KZERVKTD.js";
@@ -22,6 +22,7 @@ var LOADERS = {
   login: () => import("./pushary-login.js"),
   logout: () => import("./pushary-logout.js"),
   connect: () => import("./pushary-connect.js"),
+  bell: () => import("./pushary-bell.js"),
   doctor: () => import("./pushary-doctor.js"),
   clean: () => import("./pushary-clean.js"),
   mode: () => import("./pushary-mode.js"),