Fix notification: use file-existence check instead of process kill, bump delay to 5s

chook.mjs

···
 
 // --- Pending notification ---
 
-const NOTIFY_DELAY_SECS = 3;
+const NOTIFY_DELAY_SECS = 5;
 
 function pendingFile(sessionId) {
   return `/tmp/chook-pending-${sessionId}`;
···
 
 function scheduleNotification(sessionId, toolName, detail) {
   const short = detail.length > 80 ? detail.slice(0, 77) + "..." : detail;
-  const msg = short.replace(/"/g, '\\"');
-  // Spawn a detached shell that sleeps then notifies
-  const child = spawn(
-    "sh",
-    [
-      "-c",
-      `sleep ${NOTIFY_DELAY_SECS} && osascript -e 'display notification "${msg}" with title "chook: ${toolName} needs approval"'`,
-    ],
-    { detached: true, stdio: "ignore" },
-  );
+  const msg = short.replace(/'/g, "'\\''");
+  const pf = pendingFile(sessionId);
+  // The script checks if the pending file still exists before notifying.
+  // PostToolUse deletes it to cancel.
+  const script = `sleep ${NOTIFY_DELAY_SECS}; if [ -f '${pf}' ]; then osascript -e 'display notification "${msg}" with title "chook: ${toolName} needs approval"'; rm -f '${pf}'; fi`;
+  const child = spawn("sh", ["-c", script], {
+    detached: true,
+    stdio: "ignore",
+  });
   child.unref();
-  // Write PID so PostToolUse can cancel it
   try {
-    writeFileSync(pendingFile(sessionId), String(child.pid));
+    writeFileSync(pf, String(child.pid));
   } catch {}
 }
 
 function cancelNotification(sessionId) {
-  const f = pendingFile(sessionId);
+  // Just delete the file — the sleeping script checks for it before notifying
   try {
-    const pid = parseInt(readFileSync(f, "utf-8").trim());
-    // Kill the process group (negative PID) to get the sleep + osascript
-    try { process.kill(-pid); } catch {}
-    try { process.kill(pid); } catch {}
-    unlinkSync(f);
+    unlinkSync(pendingFile(sessionId));
   } catch {}
 }