claude-remote-approver 0.1.0 → 0.3.0

package/README.md

@@ -51,7 +51,7 @@ npm install -g claude-remote-approver
 claude-remote-approver setup
 ```
 
-Setup prints a topic name. Subscribe to that topic in the ntfy app, and you are done.
+Setup prints a QR code. Scan it with the ntfy app to subscribe, and you are done.
 
 ## Installation
 
@@ -73,7 +73,7 @@ This command does three things:
 2. **Creates a config file** at `~/.claude-remote-approver.json` with your topic and default settings. The file is created with permission `0600` (owner read/write only).
 3. **Registers the hook** in Claude Code's `~/.claude/settings.json` under `hooks.PermissionRequest`. If a previous hook entry from this tool exists, it is replaced.
 
-After running setup, open the ntfy app on your phone and subscribe to the topic printed in the terminal.
+After running setup, scan the QR code displayed in the terminal with the ntfy app on your phone. You can also manually subscribe using the URL printed below the QR code.
 
 ## Usage
 
@@ -84,6 +84,14 @@ Configure the tool and register the hook with Claude Code.
 ```bash
 claude-remote-approver setup
 # Setup complete. Topic: cra-<hex>
+#
+# Scan this QR code in the ntfy app to subscribe:
+#
+# ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄
+# █ ▄▄▄▄▄ █ █ ▄▄▄▄▄ █
+# ...
+#
+# Subscribe URL: https://ntfy.sh/cra-<hex>
 ```
 
 ### `test`
@@ -108,6 +116,39 @@ claude-remote-approver status
 # Timeout: 120s
 ```
 
+### `enable`
+
+Re-enable the hook after it has been disabled.
+
+```bash
+claude-remote-approver enable
+# Hook enabled.
+```
+
+This reads the existing configuration and re-registers the hook in Claude Code's settings. You must have run `setup` at least once before using this command.
+
+### `disable`
+
+Temporarily disable the hook without removing your configuration.
+
+```bash
+claude-remote-approver disable
+# Hook disabled. Run 'claude-remote-approver enable' to re-enable.
+```
+
+Your topic and settings are preserved. Use `enable` to re-activate.
+
+### `uninstall`
+
+Remove the hook and delete all configuration.
+
+```bash
+claude-remote-approver uninstall
+# Uninstalled. Hook removed and configuration deleted.
+```
+
+This removes the hook entry from Claude Code's settings.json and deletes `~/.claude-remote-approver.json`. To use the tool again, run `setup`.
+
 ### `hook`
 
 Internal command. Claude Code calls this automatically via the registered hook. It reads a JSON payload from stdin and writes a decision to stdout. You do not need to run this manually.

package/bin/cli.mjs

@@ -3,12 +3,15 @@
 /**
  * CLI entry point for claude-remote-approver.
  *
- * Subcommands: setup | test | status | hook
+ * Subcommands: setup | test | status | enable | disable | uninstall | hook
  * All I/O goes through the injected `deps` object so the module is fully testable.
  */
 
 import { fileURLToPath } from "node:url";
-import { realpathSync } from "node:fs";
+import fs from "node:fs";
+import path from "node:path";
+import os from "node:os";
+import qrcode from "qrcode-terminal";
 
 // ---------------------------------------------------------------------------
 // main
@@ -16,11 +19,11 @@ import { realpathSync } from "node:fs";
 
 export async function main(args, deps) {
   if (args.includes("--help") || args.includes("-h")) {
-    deps.stdout.write("Usage: claude-remote-approver <command>\n\nCommands:\n  setup   Set up remote approval\n  test    Send a test notification\n  status  Show current configuration\n  hook    Process a Claude Code hook (internal)\n");
+    deps.stdout.write("Usage: claude-remote-approver <command>\n\nCommands:\n  setup       Set up remote approval\n  test        Send a test notification\n  status      Show current configuration\n  enable      Re-enable the hook\n  disable     Temporarily disable the hook\n  uninstall   Remove hook and delete configuration\n  hook        Process a Claude Code hook (internal)\n");
     return;
   }
   if (args.includes("--version") || args.includes("-v")) {
-    deps.stdout.write("0.1.0\n");
+    deps.stdout.write(`${deps.version}\n`);
     return;
   }
 
@@ -29,7 +32,23 @@ export async function main(args, deps) {
   switch (command) {
     case "setup": {
       const result = await deps.runSetup(deps);
-      deps.stdout.write(`Setup complete. Topic: ${result.topic}\n`);
+      deps.stdout.write(`Setup complete. Topic: ${result.topic}\n\n`);
+
+      try {
+        const host = new URL(result.ntfyServer).host;
+        const ntfyUrl = `ntfy://${host}/${result.topic}`;
+        const httpsUrl = `${result.ntfyServer.replace(/\/+$/, "")}/${result.topic}`;
+
+        deps.stdout.write("Scan this QR code in the ntfy app to subscribe:\n\n");
+        // qrcode-terminal invokes the callback synchronously
+        deps.generateQR(ntfyUrl, { small: true }, (qrString) => {
+          deps.stdout.write(qrString + "\n\n");
+          deps.stdout.write(`Subscribe URL: ${httpsUrl}\n`);
+        });
+      } catch {
+        deps.stderr.write(`Warning: Invalid ntfyServer URL in config: ${result.ntfyServer}\n`);
+        deps.stdout.write(`Subscribe to topic "${result.topic}" in the ntfy app.\n`);
+      }
       break;
     }
 
@@ -86,9 +105,56 @@ export async function main(args, deps) {
       break;
     }
 
+    case "uninstall": {
+      try {
+        deps.unregisterHook(deps.settingsPath);
+      } catch (err) {
+        deps.stderr.write(`Error: Failed to remove hook: ${err.message}\n`);
+        break;
+      }
+      try {
+        deps.unlinkSync(deps.configPath);
+      } catch (err) {
+        if (err.code !== "ENOENT") {
+          deps.stderr.write(`Error: Failed to delete config: ${err.message}\n`);
+          break;
+        }
+      }
+      deps.stdout.write("Uninstalled. Hook removed and configuration deleted.\n");
+      break;
+    }
+
+    case "disable": {
+      try {
+        deps.unregisterHook(deps.settingsPath);
+      } catch (err) {
+        deps.stderr.write(`Error: Failed to disable hook: ${err.message}\n`);
+        break;
+      }
+      deps.stdout.write("Hook disabled. Run 'claude-remote-approver enable' to re-enable.\n");
+      break;
+    }
+
+    case "enable": {
+      const config = deps.loadConfig();
+      if (!config.topic) {
+        deps.stderr.write("Error: No topic configured. Run 'claude-remote-approver setup' first.\n");
+        deps.exit(1);
+        break;
+      }
+      try {
+        deps.registerHook(deps.settingsPath, deps.getHookCommand());
+      } catch (err) {
+        deps.stderr.write(`Error: Failed to enable hook: ${err.message}\n`);
+        break;
+      }
+      deps.stdout.write("Hook enabled.\n");
+      break;
+    }
+
     default: {
       deps.stderr.write(
-        "Usage: claude-remote-approver <command>\n\nCommands:\n  setup   Configure topic and register hook\n  test    Send a test notification\n  status  Show current configuration\n  hook    Process a Claude Code hook (reads JSON from stdin)\n",
+        "Usage: claude-remote-approver <command>\n\nCommands:\n  setup       Set up remote approval\n  test        Send a test notification\n  status      Show current configuration\n  enable      Re-enable the hook\n  disable     Temporarily disable the hook\n  uninstall   Remove hook and delete configuration\n  hook        Process a Claude Code hook (internal)\n",
       );
       deps.exit(1);
       break;
@@ -105,13 +171,15 @@ const isMain =
   process.argv[1] &&
   (() => {
     try {
-      return realpathSync(fileURLToPath(import.meta.url)) === realpathSync(process.argv[1]);
+      return fs.realpathSync(fileURLToPath(import.meta.url)) === fs.realpathSync(process.argv[1]);
     } catch {
       return false;
     }
   })();
 
 if (isMain) {
+  const pkg = JSON.parse(fs.readFileSync(new URL("../package.json", import.meta.url), "utf-8"));
+
   const { loadConfig, saveConfig, generateTopic } = await import(
     "../src/config.mjs"
   );
@@ -119,7 +187,7 @@ if (isMain) {
     "../src/ntfy.mjs"
   );
   const { processHook } = await import("../src/hook.mjs");
-  const { runSetup } = await import("../src/setup.mjs");
+  const { runSetup, registerHook, getHookCommand, unregisterHook } = await import("../src/setup.mjs");
 
   const args = process.argv.slice(2);
 
@@ -141,6 +209,14 @@ if (isMain) {
     formatToolInfo,
     processHook,
     runSetup,
+    registerHook,
+    getHookCommand,
+    unregisterHook,
+    version: pkg.version,
+    generateQR: (text, opts, cb) => qrcode.generate(text, opts, cb),
+    unlinkSync: fs.unlinkSync,
+    configPath: (await import("../src/config.mjs")).CONFIG_PATH,
+    settingsPath: path.join(os.homedir(), ".claude", "settings.json"),
     stdout: process.stdout,
     stderr: process.stderr,
     stdin: stdinData,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-remote-approver",
-  "version": "0.1.0",
+  "version": "0.3.0",
   "description": "Approve or deny Claude Code permission prompts remotely from your phone via ntfy.sh",
   "type": "module",
   "bin": {
@@ -13,7 +13,7 @@
     "README.md"
   ],
   "scripts": {
-    "test": "node --test test/**/*.test.mjs"
+    "test": "node --test test/*.test.mjs"
   },
   "keywords": [
     "claude",
@@ -32,5 +32,8 @@
   "repository": {
     "type": "git",
     "url": "git+https://github.com/eguchiyuuichi/claude-remote-approver.git"
+  },
+  "dependencies": {
+    "qrcode-terminal": "0.12.0"
   }
 }

package/src/setup.mjs

@@ -1,11 +1,21 @@
 import fs from "node:fs";
 import path from "node:path";
+import { fileURLToPath } from "node:url";
+
+/**
+ * Returns true if the entry belongs to claude-remote-approver.
+ */
+function isCraEntry(entry) {
+  if (entry.hooks?.some((h) => h.command?.includes("claude-remote-approver"))) return true;
+  if (entry.command?.includes("claude-remote-approver")) return true;
+  return false;
+}
 
 /**
  * Returns the hook command string: `node <absolute_path_to_src/hook.mjs>`
  */
 export function getHookCommand() {
-  const hookPath = path.resolve(import.meta.dirname, "hook.mjs");
+  const hookPath = path.resolve(path.dirname(fileURLToPath(import.meta.url)), "hook.mjs");
   return `node "${hookPath}"`;
 }
 
@@ -33,11 +43,9 @@ export function registerHook(settingsPath, hookCommand) {
     settings.hooks.PermissionRequest = [];
   }
 
-  const existingIndex = settings.hooks.PermissionRequest.findIndex(
-    (h) => h.command && h.command.includes("claude-remote-approver")
-  );
+  const existingIndex = settings.hooks.PermissionRequest.findIndex(isCraEntry);
 
-  const hookEntry = { type: "command", command: hookCommand };
+  const hookEntry = { hooks: [{ type: "command", command: hookCommand }] };
 
   if (existingIndex >= 0) {
     settings.hooks.PermissionRequest[existingIndex] = hookEntry;
@@ -48,6 +56,39 @@ export function registerHook(settingsPath, hookCommand) {
   fs.writeFileSync(settingsPath, JSON.stringify(settings, null, 2));
 }
 
+/**
+ * Removes the claude-remote-approver hook entry from Claude's settings.json.
+ * If the file does not exist, does nothing.
+ */
+export function unregisterHook(settingsPath) {
+  let settings;
+  try {
+    settings = JSON.parse(fs.readFileSync(settingsPath, "utf-8"));
+  } catch (err) {
+    if (err.code === "ENOENT") return;
+    throw err;
+  }
+
+  if (!settings.hooks?.PermissionRequest) return;
+
+  const original = settings.hooks.PermissionRequest;
+  const filtered = original.filter((entry) => !isCraEntry(entry));
+
+  if (filtered.length === original.length) return;
+
+  if (filtered.length === 0) {
+    delete settings.hooks.PermissionRequest;
+  } else {
+    settings.hooks.PermissionRequest = filtered;
+  }
+
+  if (Object.keys(settings.hooks).length === 0) {
+    delete settings.hooks;
+  }
+
+  fs.writeFileSync(settingsPath, JSON.stringify(settings, null, 2));
+}
+
 /**
  * Runs the full setup flow:
  * 1. Generate a topic
@@ -71,5 +112,5 @@ export async function runSetup({
   const hookCommand = getHookCommand();
   registerHook(settingsPath, hookCommand);
 
-  return { topic, configPath, settingsPath };
+  return { topic, ntfyServer: config.ntfyServer, configPath, settingsPath };
 }