@pushary/agent-hooks 0.62.3 → 0.62.4

package/CHANGELOG.md

@@ -1,5 +1,31 @@
 # Changelog
 
+## 0.62.4
+
+### `pushary clean` crashed partway through on Node 24
+
+`clean` threw `ERR_INVALID_ARG_TYPE: The "options.force" property must be of
+type boolean` and died mid-run, after cleaning some agents and before touching
+the rest. Introduced in 0.62.0 by the `--dry-run` work, which started passing
+`force: undefined` to `rmSync` on every call that omitted the option. Older Node
+ignored that; Node 24 rejects it.
+
+If a clean died on you, re-run it. It is safe to run repeatedly and picks up
+whatever is left.
+
+The three mutation primitives behind `--dry-run` now live in `src/cli/dry-run.ts`
+instead of inside the command file. Both `--dry-run` defects that reached users
+were in those few lines, and both got past tests that could only read the source
+as text, because code in `bin/` runs at import and cannot be called by a test.
+They are ordinary functions with ordinary tests now.
+
+### The authorize page's code input overflowed its card
+
+The eight character slots were a fixed width, so they added up to 488px inside a
+384px card and spilled out of both edges. They now share the row and shrink to
+fit, down to a 320px phone.
+
+
 ## 0.62.3
 
 ### `setup --connect app --key ...` discarded your key

package/dist/bin/pushary-clean.js

@@ -30,12 +30,32 @@ import "../chunk-DQAN3JQP.js";
 import "../chunk-KZERVKTD.js";
 
 // bin/pushary-clean.ts
-import { existsSync, readFileSync, writeFileSync, copyFileSync, rmSync, readdirSync } from "fs";
+import { existsSync, readFileSync, copyFileSync, readdirSync } from "fs";
 import { join } from "path";
 import { homedir, tmpdir } from "os";
-import { execSync } from "child_process";
+import { execSync as execSync2 } from "child_process";
 import { confirm } from "@inquirer/prompts";
 import { parse as parseTOML, stringify as stringifyTOML } from "smol-toml";
+
+// src/cli/dry-run.ts
+import { execSync } from "child_process";
+import { rmSync, writeFileSync } from "fs";
+var createGuards = (dryRun2, note) => ({
+  write: (path, contents) => {
+    if (dryRun2) return note("rewrite", path);
+    writeFileSync(path, contents, "utf-8");
+  },
+  remove: (path, options = {}) => {
+    if (dryRun2) return note("remove ", path);
+    rmSync(path, { recursive: true, force: options.force ?? false });
+  },
+  exec: (command, options, describe) => {
+    if (dryRun2) return note("run    ", describe);
+    execSync(command, options);
+  }
+});
+
+// bin/pushary-clean.ts
 exitOnHelpFlag("clean");
 var dim = (s) => `\x1B[2m${s}\x1B[0m`;
 var bold = (s) => `\x1B[1m${s}\x1B[0m`;
@@ -54,7 +74,7 @@ var PUSHARY_DIR = join(homedir(), ".pushary");
 var SHELL_FILES = [".zshrc", ".zprofile", ".bashrc", ".bash_profile"].map((f) => join(homedir(), f));
 var resolveHermesPython = () => {
   try {
-    const launcher = execSync("command -v hermes", { encoding: "utf-8", stdio: "pipe", timeout: 5e3 }).trim();
+    const launcher = execSync2("command -v hermes", { encoding: "utf-8", stdio: "pipe", timeout: 5e3 }).trim();
     if (launcher) {
       const shebang = readFileSync(launcher, "utf-8").split("\n", 1)[0];
       if (shebang.startsWith("#!")) {
@@ -80,18 +100,10 @@ var did = (past) => dryRun ? `(would ${past === "cleaned" ? "clean" : past === "
 var wouldNote = (action, path) => {
   console.log(`  ${dim("would")} ${action} ${path}`);
 };
-var guardedWrite = (path, contents) => {
-  if (dryRun) return wouldNote("rewrite", path);
-  writeFileSync(path, contents, "utf-8");
-};
-var guardedRemove = (path, options = {}) => {
-  if (dryRun) return wouldNote("remove ", path);
-  rmSync(path, { recursive: true, force: options.force });
-};
-var guardedExec = (command, options, describe) => {
-  if (dryRun) return wouldNote("run    ", describe);
-  execSync(command, options);
-};
+var guards = createGuards(dryRun, wouldNote);
+var guardedWrite = (path, contents) => guards.write(path, contents);
+var guardedRemove = (path, options) => guards.remove(path, options);
+var guardedExec = (command, options, describe) => guards.exec(command, options ?? {}, describe);
 var guardedInstructionBlockRemoval = (path) => {
   if (!dryRun) return removeInstructionBlock(path);
   if (!existsSync(path)) return false;

package/package.json

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