@pushary/agent-hooks 0.62.0 → 0.62.1

package/CHANGELOG.md

@@ -1,5 +1,33 @@
 # Changelog
 
+## 0.62.1
+
+### `pushary clean --dry-run` was not a dry run
+
+In 0.62.0 the flag ran four real operations while printing "would":
+
+- uninstalled the global `@pushary/agent-hooks` package
+- rewrote your shell profile to remove the `claude` alias
+- stripped the Pushary block from `~/.codex/AGENTS.md`
+- stripped it from `~/.gemini/GEMINI.md`
+
+If you ran `pushary clean --dry-run` on 0.62.0 and the CLI then seemed to
+disappear, that is why. Reinstall with `npm i -g @pushary/agent-hooks`, and check
+your shell profile and those two files against version control if you keep them
+there.
+
+### `pushary upgrade` could destroy `~/.claude.json`
+
+`upgrade` treated an unparseable config as an empty one and wrote it back, so a
+truncated `~/.claude.json` was replaced with nothing but the Pushary MCP entry,
+losing every project, MCP server and permission rule in it. A half-written
+`~/.claude.json` is what an interrupted write leaves behind, so this needed only
+one earlier crash to line up.
+
+A config that cannot be parsed is now left untouched, and the other agent files
+are still refreshed around it.
+
+
 ## 0.62.0
 
 ### `pushary clean` could leave your API key on disk

package/dist/bin/pushary-clean.js

@@ -91,6 +91,19 @@ var guardedExec = (command, options, describe) => {
   if (dryRun) return wouldNote("run    ", describe);
   execSync(command, options);
 };
+var guardedInstructionBlockRemoval = (path) => {
+  if (!dryRun) return removeInstructionBlock(path);
+  if (!existsSync(path)) return false;
+  const hasBlock = readFileSync(path, "utf-8").includes("pushary");
+  if (hasBlock) wouldNote("rewrite", path);
+  return hasBlock;
+};
+var guardedAliasRemoval = () => {
+  if (!dryRun) return removeClaudeAlias();
+  const shells = [".zshrc", ".bashrc", ".bash_profile", ".profile"].map((name) => join(homedir(), name)).filter((path) => existsSync(path) && readFileSync(path, "utf-8").includes("pushary claude"));
+  for (const path of shells) wouldNote("rewrite", path);
+  return shells;
+};
 var writeJson = (path, data) => {
   guardedWrite(path, JSON.stringify(data, null, 2) + "\n");
 };
@@ -219,7 +232,7 @@ var main = async () => {
     console.log(`  ${skip} Codex hooks ${dim("(not found)")}`);
   }
   const codexAgentsMd2 = codexAgentsMd();
-  if (removeInstructionBlock(codexAgentsMd2)) {
+  if (guardedInstructionBlockRemoval(codexAgentsMd2)) {
     console.log(`  ${check} Codex AGENTS.md ${dim("(removed Pushary block)")}`);
   } else {
     console.log(`  ${skip} Codex AGENTS.md ${dim("(no pushary block)")}`);
@@ -244,7 +257,7 @@ var main = async () => {
     console.log(`  ${skip} Gemini CLI settings ${dim("(not found)")}`);
   }
   const geminiMd = join(homedir(), ".gemini", "GEMINI.md");
-  if (removeInstructionBlock(geminiMd)) {
+  if (guardedInstructionBlockRemoval(geminiMd)) {
     console.log(`  ${check} Gemini GEMINI.md ${dim("(removed Pushary block)")}`);
   } else {
     console.log(`  ${skip} Gemini GEMINI.md ${dim("(no pushary block)")}`);
@@ -288,7 +301,7 @@ var main = async () => {
     } catch {
     }
   }
-  const aliasRemovedFrom = removeClaudeAlias();
+  const aliasRemovedFrom = guardedAliasRemoval();
   if (aliasRemovedFrom.length > 0) {
     console.log(`  ${check} claude alias ${dim(`(removed from ${aliasRemovedFrom.map((p) => p.split("/").pop()).join(", ")})`)}`);
   } else {
@@ -314,7 +327,11 @@ var main = async () => {
   } catch {
   }
   try {
-    execNpm("uninstall -g --no-workspaces @pushary/agent-hooks", { stdio: "ignore", timeout: 3e4 });
+    if (dryRun) {
+      wouldNote("run    ", "npm uninstall -g @pushary/agent-hooks");
+    } else {
+      execNpm("uninstall -g --no-workspaces @pushary/agent-hooks", { stdio: "ignore", timeout: 3e4 });
+    }
     console.log(`  ${check} Global package ${dim(did("uninstalled"))}`);
   } catch {
     console.log(`  ${skip} Global package ${dim("(not installed)")}`);

package/dist/bin/pushary-connect.js

@@ -13,7 +13,8 @@ import {
 } from "../chunk-N4DA4CJB.js";
 import {
   readKeySource
-} from "../chunk-247C4RE5.js";
+} from "../chunk-COC6NPWG.js";
+import "../chunk-6MTNS63X.js";
 import {
   checkApiKey
 } from "../chunk-YJ7YZRVV.js";

package/dist/bin/pushary-doctor.js

@@ -35,7 +35,8 @@ import {
 import {
   keyPrefix,
   readKeySource
-} from "../chunk-247C4RE5.js";
+} from "../chunk-COC6NPWG.js";
+import "../chunk-6MTNS63X.js";
 import {
   checkApiKey,
   describeKeyCheck

package/dist/bin/pushary-login.js

@@ -9,7 +9,8 @@ import {
 import {
   readKeySource,
   writeKey
-} from "../chunk-247C4RE5.js";
+} from "../chunk-COC6NPWG.js";
+import "../chunk-6MTNS63X.js";
 import {
   checkApiKey,
   describeKeyCheck

package/dist/bin/pushary-logout.js

@@ -8,7 +8,8 @@ import {
 import {
   clearKey,
   readKeySource
-} from "../chunk-247C4RE5.js";
+} from "../chunk-COC6NPWG.js";
+import "../chunk-6MTNS63X.js";
 import {
   createIo
 } from "../chunk-TX7KBKT7.js";

package/dist/bin/pushary-setup.js

@@ -42,13 +42,15 @@ import {
 } from "../chunk-N4DA4CJB.js";
 import {
   KEY_FILE_MODE,
-  backupFile,
   exportLine,
+  writeKey
+} from "../chunk-COC6NPWG.js";
+import {
+  backupFile,
   readJsonSafe,
   writeFileAtomic,
-  writeJsonAtomic,
-  writeKey
-} from "../chunk-247C4RE5.js";
+  writeJsonAtomic
+} from "../chunk-6MTNS63X.js";
 import {
   checkApiKey,
   describeKeyCheck

package/dist/bin/pushary-status.js

@@ -7,7 +7,8 @@ import {
 import "../chunk-3EGEA4KH.js";
 import {
   readKeySource
-} from "../chunk-247C4RE5.js";
+} from "../chunk-COC6NPWG.js";
+import "../chunk-6MTNS63X.js";
 import {
   checkApiKey,
   describeKeyCheck

package/dist/bin/pushary-upgrade.js

@@ -19,6 +19,9 @@ import {
   codexHooksJson,
   hasCodexHooks
 } from "../chunk-ER657KRJ.js";
+import {
+  readJsonSafe
+} from "../chunk-6MTNS63X.js";
 import {
   execNpm,
   npmErrorMessage,
@@ -48,11 +51,10 @@ import { parse as parseTOML, stringify as stringifyTOML } from "smol-toml";
 var CLAUDE_SETTINGS = join(homedir(), ".claude", "settings.json");
 var CLAUDE_JSON = join(homedir(), ".claude.json");
 var readJson = (filePath) => {
-  try {
-    return JSON.parse(readFileSync(filePath, "utf-8"));
-  } catch {
-    return null;
-  }
+  const result = readJsonSafe(filePath);
+  if (result.kind === "ok") return result.value;
+  if (result.kind === "missing") return {};
+  return null;
 };
 var writeJson = (filePath, data) => {
   const dir = dirname(filePath);
@@ -68,10 +70,13 @@ var reapplyClaudeSettings = (apiKey, paths = {}) => {
   const claudeJson = readJson(claudeJsonPath);
   const configured = settings != null && hasPusharyClaudeHooks(settings) || claudeJson != null && hasPusharyMcp(claudeJson);
   if (!configured) return { reapplied: false, hooks: false, mcp: false };
+  const settingsWritable = settings != null;
+  const claudeJsonWritable = claudeJson != null;
   let hooks = false;
   let mcp = false;
   try {
-    const s = settings ?? {};
+    if (!settingsWritable) throw new Error("settings.json could not be parsed");
+    const s = settings;
     addPusharyToolPermissions(s);
     addPusharyHooks(s, resolveGlobalBinDir("pushary-hook"));
     writeJson(settingsPath, s);
@@ -80,7 +85,8 @@ var reapplyClaudeSettings = (apiKey, paths = {}) => {
   }
   if (apiKey) {
     try {
-      const cj = claudeJson ?? {};
+      if (!claudeJsonWritable) throw new Error(".claude.json could not be parsed");
+      const cj = claudeJson;
       addClaudeMcpServer(cj, apiKey);
       writeJson(claudeJsonPath, cj);
       mcp = true;

package/dist/chunk-6MTNS63X.js

@@ -0,0 +1,55 @@
+// src/cli/fsx.ts
+import { copyFileSync, existsSync, fsyncSync, mkdirSync, openSync, closeSync, readFileSync, renameSync, unlinkSync, writeFileSync } from "fs";
+import { dirname, join } from "path";
+var readJsonSafe = (filePath) => {
+  let raw;
+  try {
+    raw = readFileSync(filePath, "utf-8");
+  } catch (err) {
+    const error = err;
+    if (error.code === "ENOENT") return { kind: "missing" };
+    return { kind: "unreadable", error };
+  }
+  if (raw.trim() === "") return { kind: "ok", value: {} };
+  try {
+    return { kind: "ok", value: JSON.parse(raw) };
+  } catch (err) {
+    return { kind: "unparseable", error: err, raw };
+  }
+};
+var stamp = () => (/* @__PURE__ */ new Date()).toISOString().replace(/[:.]/g, "-");
+var backupFile = (filePath) => {
+  const target = `${filePath}.pushary-backup-${stamp()}`;
+  copyFileSync(filePath, target);
+  return target;
+};
+var writeFileAtomic = (filePath, contents, mode) => {
+  const dir = dirname(filePath);
+  if (!existsSync(dir)) mkdirSync(dir, { recursive: true });
+  const temp = join(dir, `.${process.pid}-${stamp()}.pushary-tmp`);
+  try {
+    writeFileSync(temp, contents, mode === void 0 ? "utf-8" : { encoding: "utf-8", mode });
+    const handle = openSync(temp, "r+");
+    try {
+      fsyncSync(handle);
+    } finally {
+      closeSync(handle);
+    }
+    renameSync(temp, filePath);
+  } catch (err) {
+    try {
+      if (existsSync(temp)) unlinkSync(temp);
+    } catch {
+    }
+    throw err;
+  }
+};
+var writeJsonAtomic = (filePath, data, mode) => writeFileAtomic(filePath, `${JSON.stringify(data, null, 2)}
+`, mode);
+
+export {
+  readJsonSafe,
+  backupFile,
+  writeFileAtomic,
+  writeJsonAtomic
+};

package/dist/{chunk-247C4RE5.js → chunk-COC6NPWG.js}

@@ -1,3 +1,9 @@
+import {
+  backupFile,
+  readJsonSafe,
+  writeFileAtomic,
+  writeJsonAtomic
+} from "./chunk-6MTNS63X.js";
 import {
   resolveShellRc
 } from "./chunk-BC3VCZ3E.js";
@@ -5,59 +11,10 @@ import {
   configFilePath
 } from "./chunk-2UMNXADU.js";
 
-// src/cli/fsx.ts
-import { copyFileSync, existsSync, fsyncSync, mkdirSync, openSync, closeSync, readFileSync, renameSync, unlinkSync, writeFileSync } from "fs";
-import { dirname, join } from "path";
-var readJsonSafe = (filePath) => {
-  let raw;
-  try {
-    raw = readFileSync(filePath, "utf-8");
-  } catch (err) {
-    const error = err;
-    if (error.code === "ENOENT") return { kind: "missing" };
-    return { kind: "unreadable", error };
-  }
-  if (raw.trim() === "") return { kind: "ok", value: {} };
-  try {
-    return { kind: "ok", value: JSON.parse(raw) };
-  } catch (err) {
-    return { kind: "unparseable", error: err, raw };
-  }
-};
-var stamp = () => (/* @__PURE__ */ new Date()).toISOString().replace(/[:.]/g, "-");
-var backupFile = (filePath) => {
-  const target = `${filePath}.pushary-backup-${stamp()}`;
-  copyFileSync(filePath, target);
-  return target;
-};
-var writeFileAtomic = (filePath, contents, mode) => {
-  const dir = dirname(filePath);
-  if (!existsSync(dir)) mkdirSync(dir, { recursive: true });
-  const temp = join(dir, `.${process.pid}-${stamp()}.pushary-tmp`);
-  try {
-    writeFileSync(temp, contents, mode === void 0 ? "utf-8" : { encoding: "utf-8", mode });
-    const handle = openSync(temp, "r+");
-    try {
-      fsyncSync(handle);
-    } finally {
-      closeSync(handle);
-    }
-    renameSync(temp, filePath);
-  } catch (err) {
-    try {
-      if (existsSync(temp)) unlinkSync(temp);
-    } catch {
-    }
-    throw err;
-  }
-};
-var writeJsonAtomic = (filePath, data, mode) => writeFileAtomic(filePath, `${JSON.stringify(data, null, 2)}
-`, mode);
-
 // src/keystore.ts
-import { chmodSync, existsSync as existsSync2, readFileSync as readFileSync2, realpathSync, writeFileSync as writeFileSync2 } from "fs";
+import { chmodSync, existsSync, readFileSync, realpathSync, writeFileSync } from "fs";
 import { homedir } from "os";
-import { join as join2 } from "path";
+import { join } from "path";
 var KEY_FILE_MODE = 384;
 var FALLBACK_RCS = [".zshrc", ".zprofile", ".bashrc", ".bash_profile"];
 var OUR_EXPORT = /^[ \t]*export[ \t]+PUSHARY_API_KEY=(['"]?)(pk_[a-f0-9]+\.[a-f0-9]+)\1[ \t]*$/;
@@ -86,7 +43,7 @@ var rewriteRcExport = (rcPath, apiKey, shell = null) => {
   const target = realPathOf(rcPath);
   let content = "";
   try {
-    content = readFileSync2(target, "utf-8");
+    content = readFileSync(target, "utf-8");
   } catch (err) {
     const error = err;
     if (error.code !== "ENOENT") {
@@ -141,7 +98,7 @@ var rewriteRcExport = (rcPath, apiKey, shell = null) => {
 var resolveRcTarget = () => {
   const resolved = resolveShellRc();
   if (resolved) return { path: resolved.path, shell: resolved.shell };
-  const fallback = FALLBACK_RCS.map((name) => join2(homedir(), name)).find(existsSync2);
+  const fallback = FALLBACK_RCS.map((name) => join(homedir(), name)).find(existsSync);
   return fallback ? { path: fallback, shell: null } : null;
 };
 var writeKey = (apiKey, options = {}) => {
@@ -181,13 +138,13 @@ var clearKey = (options = {}) => {
   if (target) {
     try {
       const real = realpathSync(target.path);
-      const lines = readFileSync2(real, "utf8").split("\n");
+      const lines = readFileSync(real, "utf8").split("\n");
       const next = lines.map((line) => {
         if (!OUR_EXPORT.test(line)) return line;
         rcLinesDisabled += 1;
         return `# ${line.trim()}   # removed by pushary logout`;
       });
-      if (rcLinesDisabled > 0) writeFileSync2(real, next.join("\n"));
+      if (rcLinesDisabled > 0) writeFileSync(real, next.join("\n"));
     } catch {
       rcLinesDisabled = 0;
     }
@@ -223,10 +180,6 @@ var readKeySource = () => {
 };
 
 export {
-  readJsonSafe,
-  backupFile,
-  writeFileAtomic,
-  writeJsonAtomic,
   KEY_FILE_MODE,
   exportLine,
   keyPrefix,

package/package.json

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