@fyeeme/pi-hooks 1.0.0 → 1.0.1

package/CHANGELOG.md

@@ -7,6 +7,13 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [1.0.1] - 2025-07-15
+
+### Fixed
+
+- Process hang when hook command ignores SIGTERM: added SIGKILL escalation after 5s grace window
+- Race condition between `close` and `error` events: added `finish()` guard to prevent double-resolve
+
 ## [1.0.0] - 2025-05-31
 
 ### Added

package/index.ts

@@ -110,27 +110,44 @@ async function runCommand(command: string, cwd: string, stdinJson: unknown): Pro
 		});
 
 		let stdout = "";
+		let settled = false;
+		const finish = (result: HookOutput | null) => {
+			if (settled) return;
+			settled = true;
+			clearTimeout(sigtermTimer);
+			clearTimeout(sigkillTimer);
+			resolve(result);
+		};
+
 		proc.stdout?.on("data", (chunk: Buffer) => {
 			stdout += chunk.toString();
 		});
 
-		const timer = setTimeout(() => proc.kill("SIGTERM"), 10_000);
+		let sigkillTimer: NodeJS.Timeout | undefined;
+
+		const sigtermTimer = setTimeout(() => {
+			try { proc.kill("SIGTERM"); } catch { /* already dead */ }
+			// escalate to SIGKILL after a 5s grace window
+			sigkillTimer = setTimeout(() => {
+				try { proc.kill("SIGKILL"); } catch { /* already dead */ }
+				// force-resolve: a process that survives SIGKILL is unrecoverable
+				finish(null);
+			}, 5_000);
+		}, 10_000);
 
 		proc.on("close", (code) => {
-			clearTimeout(timer);
 			if (code !== 0) console.error(`[hooks] exited ${code}: ${command}`);
 			try {
 				const trimmed = stdout.trim();
-				resolve(trimmed ? (JSON.parse(trimmed) as HookOutput) : null);
+				finish(trimmed ? (JSON.parse(trimmed) as HookOutput) : null);
 			} catch {
-				resolve(null);
+				finish(null);
 			}
 		});
 
 		proc.on("error", (err) => {
-			clearTimeout(timer);
 			console.error(`[hooks] spawn error: ${command}: ${err}`);
-			resolve(null);
+			finish(null);
 		});
 
 		try {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fyeeme/pi-hooks",
-  "version": "1.0.0",
+  "version": "1.0.1",
   "description": "Claude Code-compatible hooks runner for pi. Reads .pi/hooks.json and maps SessionStart, PreToolUse, and Stop events to pi lifecycle events.",
   "type": "module",
   "license": "MIT",