@pushary/agent-hooks 0.5.0 → 0.5.1

package/dist/bin/pushary-clean.js

@@ -98,11 +98,11 @@ var main = async () => {
       const cleaned = [];
       let skipping = false;
       for (const line of lines) {
-        if (line.trim() === "[mcp_servers.pushary]") {
+        if (/^\[mcp_servers\.pushary(?:\.|]$)/.test(line.trim())) {
           skipping = true;
           continue;
         }
-        if (skipping && (line.startsWith("[") || line.trim() === "")) {
+        if (skipping && line.startsWith("[") && !/^\[mcp_servers\.pushary/.test(line.trim())) {
           skipping = false;
         }
         if (skipping) continue;

package/dist/bin/pushary-doctor.js

@@ -68,6 +68,23 @@ var main = async () => {
   } else {
     check(false, "Claude Code: settings.json", "not found");
   }
+  const codexConfigPath = join(homedir(), ".codex", "config.toml");
+  if (existsSync(codexConfigPath)) {
+    const codexConfig = readFileSync(codexConfigPath, "utf-8");
+    const hasPusharyMcp = codexConfig.includes("[mcp_servers.pushary]");
+    check(hasPusharyMcp, "Codex: MCP server configured");
+    if (hasPusharyMcp) {
+      const hasAutoApprove = codexConfig.includes('default_tools_approval_mode = "approve"');
+      check(hasAutoApprove, "Codex: tools auto-allowed", hasAutoApprove ? 'default_tools_approval_mode = "approve"' : "missing \u2014 MCP calls will prompt for approval");
+      const hasPerToolOverrides = /\[mcp_servers\.pushary\.tools\./.test(codexConfig);
+      if (hasPerToolOverrides) {
+        console.log(`  ${warn} Codex: per-tool approval overrides detected ${dim("(redundant with default_tools_approval_mode)")}`);
+      }
+    }
+    check(codexConfig.includes("pushary-codex"), "Codex: notify handler configured");
+    const codexSkillPath = join(homedir(), ".codex", "skills", "pushary", "SKILL.md");
+    check(existsSync(codexSkillPath), "Codex: skill installed");
+  }
   check(existsSync(SKILL_PATH), "Skill installed", existsSync(SKILL_PATH) ? SKILL_PATH : "not found");
   let globalVersion = "";
   try {

package/dist/bin/pushary-setup.js

@@ -146,29 +146,86 @@ var setupClaudeCode = async (apiKey) => {
   console.log(`  ${dim("\u2022")} Hooks: route permission approvals through push notifications`);
   console.log(`  ${dim("\u2022")} Auto-allowed tools: no permission prompts for Pushary MCP calls`);
 };
+var findPython310Plus = () => {
+  const candidates = ["python3.13", "python3.12", "python3.11", "python3.10", "python3", "python"];
+  for (const py of candidates) {
+    try {
+      const version = execSync(`${py} --version 2>&1`, { encoding: "utf-8", stdio: "pipe" }).trim();
+      const match = version.match(/Python (\d+)\.(\d+)/);
+      if (match && (Number(match[1]) > 3 || Number(match[1]) === 3 && Number(match[2]) >= 10)) {
+        return py;
+      }
+    } catch {
+    }
+  }
+  return null;
+};
+var installPythonPlugin = (pythonBin) => {
+  execSync(`${pythonBin} -m pip install --upgrade hermes-plugin-pushary`, { stdio: "pipe" });
+};
 var setupHermes = async (_apiKey) => {
   console.log(`
   ${bold("Setting up Hermes Agent")}
 `);
-  let pipInstalled = false;
+  const whichCmd = process.platform === "win32" ? "where" : "which";
+  const hasHermes = (() => {
+    try {
+      execSync(`${whichCmd} hermes`, { stdio: "ignore" });
+      return true;
+    } catch {
+      return false;
+    }
+  })();
+  if (!hasHermes) {
+    console.log(`  ${yellow("!")} Hermes CLI not found. Skipping.`);
+    console.log(`  ${dim("Install Hermes and re-run setup to configure.")}`);
+    return;
+  }
   await spinner("Installing hermes-plugin-pushary", async () => {
+    try {
+      execSync("uv pip install hermes-plugin-pushary", { stdio: "pipe" });
+      return;
+    } catch {
+    }
+    let python = findPython310Plus();
+    if (!python) {
+      if (process.platform === "darwin") {
+        try {
+          execSync("which brew", { stdio: "ignore" });
+          execSync("brew install python@3.12", { stdio: "pipe" });
+          python = findPython310Plus();
+        } catch {
+        }
+      } else if (process.platform === "linux") {
+        for (const [check2, install] of [
+          ["which apt-get", "sudo apt-get update -qq && sudo apt-get install -y -qq python3 python3-pip"],
+          ["which dnf", "sudo dnf install -y -q python3 python3-pip"],
+          ["which yum", "sudo yum install -y -q python3 python3-pip"],
+          ["which pacman", "sudo pacman -S --noconfirm python python-pip"]
+        ]) {
+          try {
+            execSync(check2, { stdio: "ignore" });
+            execSync(install, { stdio: "pipe" });
+            python = findPython310Plus();
+            if (python) break;
+          } catch {
+          }
+        }
+      }
+    }
+    if (python) {
+      installPythonPlugin(python);
+      return;
+    }
     for (const pip of ["pip3", "pip"]) {
       try {
         execSync(`${pip} install hermes-plugin-pushary`, { stdio: "pipe" });
-        pipInstalled = true;
         return;
-      } catch (err) {
-        const msg = err instanceof Error ? err.stderr?.toString() ?? "" : "";
-        if (msg.includes("No matching distribution")) {
-          throw new Error("requires Python 3.10+ (your pip uses an older version)");
-        }
+      } catch {
       }
     }
-    throw new Error("pip not found");
+    throw new Error("Python 3.10+ not found and could not be installed");
   });
-  if (!pipInstalled) {
-    console.log(`  ${dim("  Install Python 3.10+ and re-run setup to fix.")}`);
-  }
   await spinner("Enabling plugin", async () => {
     execSync("hermes plugins enable pushary", { stdio: "ignore" });
   });
@@ -181,9 +238,10 @@ var setupCodex = async (_apiKey) => {
   console.log(`
   ${bold("Setting up Codex")}
 `);
+  const whichCmd = process.platform === "win32" ? "where" : "which";
   const hasCodex = (() => {
     try {
-      execSync("which codex", { stdio: "ignore" });
+      execSync(`${whichCmd} codex`, { stdio: "ignore" });
       return true;
     } catch {
       return false;
@@ -195,6 +253,7 @@ var setupCodex = async (_apiKey) => {
     return;
   }
   await installGlobally();
+  const codexConfig = join(homedir(), ".codex", "config.toml");
   await spinner("Adding Pushary MCP server to Codex", async () => {
     try {
       execSync(
@@ -204,7 +263,42 @@ var setupCodex = async (_apiKey) => {
     } catch {
     }
   });
-  const codexConfig = join(homedir(), ".codex", "config.toml");
+  await spinner("Auto-allowing all Pushary tools", async () => {
+    let config = "";
+    try {
+      config = readFileSync(codexConfig, "utf-8");
+    } catch {
+    }
+    const lines = config.split("\n");
+    const cleaned = [];
+    let skippingToolSection = false;
+    for (const line of lines) {
+      if (/^\[mcp_servers\.pushary\.tools\./.test(line.trim())) {
+        skippingToolSection = true;
+        continue;
+      }
+      if (skippingToolSection) {
+        if (line.startsWith("[") || line.trim() === "") {
+          skippingToolSection = false;
+          if (line.startsWith("[")) {
+            cleaned.push(line);
+            continue;
+          }
+        } else {
+          continue;
+        }
+      }
+      cleaned.push(line);
+    }
+    config = cleaned.join("\n");
+    if (!config.includes("default_tools_approval_mode")) {
+      config = config.replace(
+        /(\[mcp_servers\.pushary\]\n)/,
+        '$1default_tools_approval_mode = "approve"\n'
+      );
+    }
+    writeFileSync(codexConfig, config, "utf-8");
+  });
   await spinner("Adding notify handler for Codex events", async () => {
     const globalPrefix = execSync("npm prefix -g", { encoding: "utf-8" }).trim();
     const pusharyCodexPath = join(globalPrefix, "bin", "pushary-codex");
@@ -225,6 +319,7 @@ notify = ["${pusharyCodexPath}"]
   console.log();
   console.log(`  ${dim("What this configured:")}`);
   console.log(`  ${dim("\u2022")} MCP server: Codex can send notifications and ask questions`);
+  console.log(`  ${dim("\u2022")} Auto-allowed tools: no permission prompts for Pushary MCP calls`);
   console.log(`  ${dim("\u2022")} Notify handler: captures turn completions and approval requests`);
 };
 var setupCursor = async (apiKey) => {
@@ -324,7 +419,7 @@ var main = async () => {
     message: "Which agents do you use? " + dim("(space = toggle, enter = confirm)"),
     choices: [
       { name: `Claude Code  ${dim("MCP + hooks + auto-allowed tools")}`, value: "claude_code" },
-      { name: `Codex        ${dim("MCP server via codex mcp add")}`, value: "codex" },
+      { name: `Codex        ${dim("MCP + notify handler + auto-allowed tools")}`, value: "codex" },
       { name: `Hermes       ${dim("native plugin + auto-error notifications")}`, value: "hermes" },
       { name: `Cursor       ${dim("MCP server")}`, value: "cursor" }
     ]
@@ -334,8 +429,19 @@ var main = async () => {
     console.log(`
   ${dim("No agents selected. API key saved.")}`);
   } else {
+    const failed = [];
     for (const agent of agents) {
-      await AGENT_SETUP[agent](trimmedKey);
+      try {
+        await AGENT_SETUP[agent](trimmedKey);
+      } catch (err) {
+        failed.push(agent.replace("_", " "));
+        console.log(`  ${yellow("!")} ${agent.replace("_", " ")} setup failed: ${formatError(err)}`);
+        console.log(`  ${dim("Other agents will continue. Re-run setup to retry.")}`);
+      }
+    }
+    if (failed.length > 0) {
+      console.log();
+      console.log(`  ${yellow("!")} Failed: ${failed.join(", ")} ${dim("(others completed successfully)")}`);
     }
   }
   const sendTest = await confirm({ message: "Send a test notification?", default: true });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pushary/agent-hooks",
-  "version": "0.5.0",
+  "version": "0.5.1",
   "description": "Permission hooks for AI coding agents: route tool approvals through Pushary push notifications",
   "author": "Pushary <business@pushary.com>",
   "homepage": "https://pushary.com",