claude-remote-approver 0.5.0 → 0.5.1

package/bin/cli.mjs

@@ -12,6 +12,7 @@ import fs from "node:fs";
 import path from "node:path";
 import os from "node:os";
 import qrcode from "qrcode-terminal";
+import { ASK } from "../src/hook.mjs";
 
 // ---------------------------------------------------------------------------
 // main
@@ -87,8 +88,8 @@ export async function main(args, deps) {
       try {
         input = JSON.parse(deps.stdin);
       } catch {
-        const ask = { hookSpecificOutput: { hookEventName: "PermissionRequest", decision: { behavior: "ask" } } };
-        deps.stdout.write(JSON.stringify(ask) + "\n");
+        deps.stderr.write("[claude-remote-approver] Invalid hook input. Falling back to CLI.\n");
+        deps.stdout.write(JSON.stringify(ASK) + "\n");
         break;
       }
 
@@ -96,8 +97,8 @@ export async function main(args, deps) {
       try {
         result = await deps.processHook(input, deps);
       } catch {
-        const ask = { hookSpecificOutput: { hookEventName: "PermissionRequest", decision: { behavior: "ask" } } };
-        deps.stdout.write(JSON.stringify(ask) + "\n");
+        deps.stderr.write("[claude-remote-approver] Hook processing failed. Falling back to CLI.\n");
+        deps.stdout.write(JSON.stringify(ASK) + "\n");
         break;
       }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-remote-approver",
-  "version": "0.5.0",
+  "version": "0.5.1",
   "description": "Approve or deny Claude Code permission prompts remotely from your phone via ntfy.sh",
   "type": "module",
   "bin": {

package/src/hook.mjs

@@ -3,7 +3,7 @@
 import crypto from "node:crypto";
 import { DEFAULT_CONFIG } from "./config.mjs";
 
-const ASK = Object.freeze({ hookSpecificOutput: Object.freeze({ hookEventName: "PermissionRequest", decision: Object.freeze({ behavior: "ask" }) }) });
+export const ASK = Object.freeze({ hookSpecificOutput: Object.freeze({ hookEventName: "PermissionRequest", decision: Object.freeze({ behavior: "ask" }) }) });
 const DENY = Object.freeze({ hookSpecificOutput: Object.freeze({ hookEventName: "PermissionRequest", decision: Object.freeze({ behavior: "deny" }) }) });
 const MAX_RETRIES = 3;
 
@@ -44,7 +44,7 @@ export async function sendWithRetry(sendFn, params) {
       return await sendFn(params);
     } catch (err) {
       if (i === MAX_RETRIES - 1) {
-        console.error("sendNotification failed:", err);
+        console.error(`[claude-remote-approver] Notification failed after ${MAX_RETRIES} attempts:`, err.message, "— Falling back to CLI.");
         return null;
       }
     }
@@ -139,13 +139,14 @@ export async function processAskUserQuestion(input, deps) {
         timeout: config.timeout * 1000,
       });
     } catch (err) {
-      console.error("waitForResponse failed:", err);
+      console.error("[claude-remote-approver] Response listener failed:", err.message, "— Falling back to CLI.");
       return ASK;
     }
 
     if (response.answer) {
       answers[q.question] = response.answer;
     } else {
+      console.error("[claude-remote-approver] No answer received. Falling back to CLI.");
       return ASK;
     }
   }
@@ -211,11 +212,18 @@ export async function processHook(input, { loadConfig, sendNotification, waitFor
       timeout,
     });
   } catch (err) {
-    console.error("waitForResponse failed:", err);
+    console.error("[claude-remote-approver] Response listener failed:", err.message, "— Falling back to CLI.");
     return ASK;
   }
 
-  if (response.timeout || response.error) return ASK;
+  if (response.timeout) {
+    console.error("[claude-remote-approver] Timed out waiting for response. Falling back to CLI.");
+    return ASK;
+  }
+  if (response.error) {
+    console.error("[claude-remote-approver] Response error:", response.error.message, "— Falling back to CLI.");
+    return ASK;
+  }
   if (response.approved === false) return DENY;
   return { hookSpecificOutput: { hookEventName: "PermissionRequest", decision: { behavior: "allow" } } };
 }

package/src/ntfy.mjs

@@ -91,7 +91,7 @@ export async function waitForResponse({ server, topic, requestId, timeout }) {
     if (err?.name === "AbortError") {
       return { timeout: true };
     }
-    console.error("[claude-remote-approver] waitForResponse error:", err);
+    console.error("[claude-remote-approver] waitForResponse error:", err.message ?? err);
     return { error: err };
   }
 }