@pushary/agent-hooks 1.5.1 → 1.6.0

package/dist/{chunk-JKBDGZFL.js → chunk-OP7HFIWE.js}

@@ -1,21 +1,21 @@
 import {
   detectClaudeVersion
-} from "./chunk-HUAOMX4U.js";
+} from "./chunk-NDS27FXU.js";
 import {
   ensureCodexTrustsPusharyHooks
-} from "./chunk-6EEANS74.js";
+} from "./chunk-XQOT4CCK.js";
 import {
   addClaudeMcpServer,
   addPusharyHooks,
   addPusharyToolPermissions
-} from "./chunk-4ERLRNOK.js";
+} from "./chunk-HWG7X3VI.js";
 import {
   claudeUnprovenHolds,
   claudeWired,
   codexUnprovenHolds,
   codexWired,
   geminiWired
-} from "./chunk-Z4VGIBJ7.js";
+} from "./chunk-MOQOJOMD.js";
 import {
   CLAUDE_HOOK_BINARY,
   CLAUDE_HOOK_EVENTS,
@@ -27,7 +27,7 @@ import {
   needsReregistration,
   registrationSignatures,
   supportedClaudeEvents
-} from "./chunk-2AUIEZYS.js";
+} from "./chunk-BK5F3FGE.js";
 import {
   CODEX_HOOK_BINARY,
   addCodexHookTrust,
@@ -47,7 +47,7 @@ import {
   registrationHolderOf,
   resolveGlobalBinary,
   vscodePluginMcp
-} from "./chunk-6IU45EHE.js";
+} from "./chunk-F2OPOBRO.js";
 import {
   readJsonSafe,
   writeFileAtomic,
@@ -126,6 +126,7 @@ var reapplyClaudeSettings = (apiKey, paths = {}) => {
   const claudeJsonWritable = claudeJson != null;
   let hooks = false;
   let mcp = false;
+  let failed = false;
   try {
     if (!settingsWritable) throw new Error("settings.json could not be parsed");
     const s = settings;
@@ -135,6 +136,7 @@ var reapplyClaudeSettings = (apiKey, paths = {}) => {
     writeJson(settingsPath, s);
     hooks = true;
   } catch {
+    failed = true;
   }
   if (apiKey) {
     try {
@@ -144,9 +146,10 @@ var reapplyClaudeSettings = (apiKey, paths = {}) => {
       writeJson(claudeJsonPath, cj);
       mcp = true;
     } catch {
+      failed = true;
     }
   }
-  return { configured: true, reapplied: hooks || mcp, hooks, mcp };
+  return { configured: true, reapplied: hooks || mcp, hooks, mcp, failed };
 };
 var reapplyClaudeForVersion = (apiKey, paths = {}) => {
   const idle = { verdict: "current", rewrote: false, missing: [] };
@@ -215,6 +218,7 @@ var reapplyCodexSettings = (apiKey, paths = {}) => {
   const command = guardBinary((paths.resolveBinary ?? resolveGlobalBinary)(CODEX_HOOK_BINARY) ?? CODEX_HOOK_BINARY);
   let hooks = false;
   let mcp = false;
+  let failed = hooksConfig === null || config === null;
   const vacated = codexVacatedIndexes(hooksConfig ?? void 0);
   if (hooksConfig != null) {
     try {
@@ -222,6 +226,7 @@ var reapplyCodexSettings = (apiKey, paths = {}) => {
       writeJson(hooksPath, hooksConfig);
       hooks = true;
     } catch {
+      failed = true;
     }
   }
   if (config != null) {
@@ -232,15 +237,18 @@ var reapplyCodexSettings = (apiKey, paths = {}) => {
       }
       if (apiKey) {
         addCodexMcpServer(config, apiKey);
-        mcp = true;
       }
       if (hooks || apiKey) {
         writeFileAtomic(configPath, stringifyTOML(config), modeToPreserve(configPath), "utf-8");
+        mcp = Boolean(apiKey);
       }
     } catch {
+      failed = true;
+      hooks = false;
     }
   }
-  return { configured: true, reapplied: hooks || mcp, hooks, mcp };
+  if (config === null) hooks = false;
+  return { configured: true, reapplied: hooks || mcp, hooks, mcp, failed };
 };
 var repairCodexHolds = (paths = {}) => {
   const idle = { configured: false, reapplied: false, hooks: false, mcp: false };
@@ -265,13 +273,14 @@ var reapplyGeminiSettings = (apiKey, paths = {}) => {
   let mcp = false;
   try {
     addGeminiHooks(settings, guardBinary(resolveGlobalBinary(GEMINI_HOOK_BINARY) ?? GEMINI_HOOK_BINARY));
-    hooks = true;
     if (apiKey) {
       addGeminiMcpServer(settings, apiKey);
-      mcp = true;
     }
     writeJson(settingsPath, settings);
+    hooks = true;
+    mcp = Boolean(apiKey);
   } catch {
+    return { configured: true, reapplied: false, hooks: false, mcp: false, failed: true };
   }
   return { configured: true, reapplied: hooks || mcp, hooks, mcp };
 };
@@ -286,7 +295,7 @@ var reapplyPluginMcp = (mcpPath, apiKey) => {
     writeJson(mcpPath, mcpConfig);
     return { configured: true, reapplied: true, hooks: false, mcp: true };
   } catch {
-    return { configured: true, reapplied: false, hooks: false, mcp: false };
+    return { configured: true, reapplied: false, hooks: false, mcp: false, failed: true };
   }
 };
 var reapplyCursorPlugin = (apiKey, paths = {}) => reapplyPluginMcp(
@@ -308,6 +317,7 @@ var reapplyAllAgents = (apiKey) => {
       const result = runner.run();
       if (result.configured) done.push({ id: runner.id, label: runner.label, result });
     } catch {
+      done.push({ id: runner.id, label: runner.label, result: { configured: false, reapplied: false, hooks: false, mcp: false, failed: true } });
     }
   }
   return done;

package/dist/{chunk-5UQ5ATIE.js → chunk-OR3A3RLH.js}

@@ -1,6 +1,6 @@
 import {
   guardBinary
-} from "./chunk-6IU45EHE.js";
+} from "./chunk-F2OPOBRO.js";
 
 // src/bell/install.ts
 import { join } from "path";

package/dist/{chunk-XWEONSLV.js → chunk-PCVVCS4A.js}

@@ -1,7 +1,7 @@
 import {
   describeKeyCheck,
   keyCheckFromResponse
-} from "./chunk-43LJSM4F.js";
+} from "./chunk-CSW6A2EW.js";
 import {
   createIo
 } from "./chunk-5RUIFDTP.js";

package/dist/{chunk-KJRALILH.js → chunk-PRZXYILJ.js}

@@ -15,11 +15,11 @@ import {
 } from "./chunk-HETOKNIL.js";
 import {
   reportedInstallSource
-} from "./chunk-HU5LSYL2.js";
+} from "./chunk-VMNI33F5.js";
 import {
   PAIR_TTL_SECONDS,
   isAnswerSource
-} from "./chunk-T4QY64OR.js";
+} from "./chunk-YFD7CRIZ.js";
 import {
   NPX_LATEST_COMMAND
 } from "./chunk-EA3IELHP.js";

package/dist/{chunk-MU3WU2FO.js → chunk-RBPJO4KG.js}

@@ -1,6 +1,6 @@
 import {
   autoUpdateState
-} from "./chunk-6IU45EHE.js";
+} from "./chunk-F2OPOBRO.js";
 import {
   readJsonSafe,
   writeJsonAtomic

package/dist/{chunk-TIWURSLS.js → chunk-SFNRAEEW.js}

@@ -1,6 +1,6 @@
 import {
   redactSecrets
-} from "./chunk-T4QY64OR.js";
+} from "./chunk-YFD7CRIZ.js";
 
 // src/crypto.ts
 import nacl from "tweetnacl";

package/dist/{chunk-USQDVBEV.js → chunk-TDKE3V6W.js}

@@ -1,6 +1,6 @@
 import {
   resolveInstallSourceFor
-} from "./chunk-HU5LSYL2.js";
+} from "./chunk-VMNI33F5.js";
 import {
   backupFile,
   readJsonSafe,

package/dist/{chunk-HU5LSYL2.js → chunk-VMNI33F5.js}

@@ -3,7 +3,7 @@ import {
 } from "./chunk-OSJEMXSR.js";
 import {
   normalizeInstallSource
-} from "./chunk-T4QY64OR.js";
+} from "./chunk-YFD7CRIZ.js";
 
 // src/install-source.ts
 import { readFileSync } from "fs";

package/dist/{chunk-UYDTFCHW.js → chunk-VZANUMIF.js}

@@ -1,6 +1,6 @@
 import {
   PRETOOLUSE_GATED_TOOLS
-} from "./chunk-T4QY64OR.js";
+} from "./chunk-YFD7CRIZ.js";
 
 // src/claude-tools.ts
 var POSTTOOLUSE_MIRROR_ONLY_TOOLS = ["TodoWrite", "Task", "WebFetch", "WebSearch"];

package/dist/{chunk-3ZSGKLSO.js → chunk-XGL5JC6U.js}

@@ -1,6 +1,6 @@
 import {
   SETUP_FLAGS
-} from "./chunk-TFKBFQCE.js";
+} from "./chunk-A5URWZTP.js";
 import {
   parseFlags
 } from "./chunk-GMXKITVA.js";
@@ -14,7 +14,7 @@ import {
   CLAUDE_CONNECTOR_URL,
   COWORK_CONNECTOR_LIMIT,
   COWORK_STANDING_INSTRUCTIONS
-} from "./chunk-T4QY64OR.js";
+} from "./chunk-YFD7CRIZ.js";
 
 // src/cli/cowork.ts
 import { readFileSync } from "fs";

package/dist/{chunk-6EEANS74.js → chunk-XQOT4CCK.js}

@@ -6,7 +6,7 @@ import {
   codexHooksJsonIn,
   isPusharyCodexCommand,
   resolvePathBinary
-} from "./chunk-6IU45EHE.js";
+} from "./chunk-F2OPOBRO.js";
 import {
   getPackageVersion
 } from "./chunk-HWJW3CPO.js";
@@ -37,7 +37,8 @@ var CODEX_INSTALLATION_ID_FILE = "installation_id";
 var CODEX_NOT_ASKED = {
   notInteractive: "Pushary only starts Codex when you run setup, upgrade or doctor yourself at a terminal",
   noCodexOnPath: "there is no codex command on PATH",
-  neverOpened: "Codex has not been opened on this machine yet"
+  neverOpened: "Codex has not been opened on this machine yet",
+  insideAnApp: "the codex command resolves inside an app bundle, which Pushary does not run"
 };
 var UNRECOGNISED_LISTING = "Codex answered hooks/list in a shape Pushary does not recognise";
 var isRecord = (value) => typeof value === "object" && value !== null && !Array.isArray(value);
@@ -115,6 +116,9 @@ var mayAskCodex = (conditions) => {
   if (!conditions.interactive) return { kind: "skipped", reason: CODEX_NOT_ASKED.notInteractive };
   if (!conditions.codexOnPath) return { kind: "skipped", reason: CODEX_NOT_ASKED.noCodexOnPath };
   if (!conditions.codexOpenedBefore) return { kind: "skipped", reason: CODEX_NOT_ASKED.neverOpened };
+  if (conditions.codexOnPath.split(/[\\/]/).some((part) => part.toLowerCase().endsWith(".app"))) {
+    return { kind: "skipped", reason: CODEX_NOT_ASKED.insideAnApp };
+  }
   return { kind: "ask", binary: conditions.codexOnPath };
 };
 var canonicalPath = (path) => {
@@ -233,9 +237,10 @@ var runCodexAppServer = (launch, conversation) => {
 };
 var launchOf = (options) => {
   const home = options.codexHome ?? codexHome();
+  const binary = options.codexOnPath === void 0 ? resolvePathBinary("codex") ?? null : options.codexOnPath;
   const decision = mayAskCodex({
     interactive: options.interactive ?? process.stdout.isTTY === true,
-    codexOnPath: options.codexOnPath === void 0 ? resolvePathBinary("codex") ?? null : options.codexOnPath,
+    codexOnPath: binary ? canonicalPath(binary) : null,
     codexOpenedBefore: existsSync(join(home, CODEX_INSTALLATION_ID_FILE))
   });
   if (decision.kind === "skipped") return decision;