claude-remote-approver 0.3.6 → 0.4.1

package/README.md

@@ -51,7 +51,7 @@ npm install -g claude-remote-approver
 claude-remote-approver setup
 ```
 
-Setup prints a QR code. Scan it with the ntfy app to subscribe, and you are done.
+Setup prints a QR code. Scan it with the ntfy app to subscribe, then **start a new Claude Code session**. The hook is loaded at session startup, so any session that was already running before installation will not have the hook active.
 
 ## Installation
 

package/package.json

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

package/src/ntfy.mjs

@@ -69,6 +69,7 @@ export async function waitForResponse({ server, topic, requestId, timeout }) {
             if (parsed.requestId === requestId) {
               clearTimeout(timer);
               controller.signal.removeEventListener('abort', onAbort);
+              controller.abort();
               return { approved: parsed.approved };
             }
           } catch {
@@ -91,6 +92,345 @@ export async function waitForResponse({ server, topic, requestId, timeout }) {
   }
 }
 
+/**
+ * Strip markdown formatting from text, returning plain text.
+ *
+ * @param {string} text - Markdown text to strip
+ * @returns {string} Plain text with markdown removed
+ */
+export function stripMarkdown(text) {
+  // Input guard
+  if (text.length > MAX_INPUT) {
+    text = text.slice(0, MAX_INPUT);
+  }
+
+  // Order matters: fenced code blocks must be first to prevent processing markdown inside them.
+  let result = text
+    .replace(/```[\s\S]*?(?:```|$)/g, '')                // Fenced code blocks
+    .replace(/^[ \t]*(?:(?:-[ \t]*){3,}|(?:\*[ \t]*){3,}|(?:_[ \t]*){3,})$/gm, '') // Horizontal rules (before list markers)
+    .replace(/^#{1,6}\s+/gm, '')                          // Headers
+    .replace(/^(?:>[ \t]?)+/gm, '')                       // Block quotes
+    .replace(/^[ \t]*[-*+] /gm, '')                       // Unordered list markers with indent
+    .replace(/^[ \t]*\d+\. /gm, '');                      // Ordered list markers with indent
+
+  result = stripInline(result);
+
+  return result.replace(/\n{2,}/g, '\n').trim();
+}
+
+// ---------------------------------------------------------------------------
+// Constants and helpers
+// ---------------------------------------------------------------------------
+
+const MAX_INPUT = 10000;
+
+/**
+ * Count consecutive runs of character ch starting at pos.
+ *
+ * @param {string} text
+ * @param {number} pos
+ * @param {string} ch
+ * @returns {number}
+ */
+function countRun(text, pos, ch) {
+  let count = 0;
+  while (pos + count < text.length && text[pos + count] === ch) count++;
+  return count;
+}
+
+const RE_ALPHANUMERIC = /[a-zA-Z0-9]/;
+const RE_WHITESPACE = /\s/;
+const RE_ASCII_PUNCTUATION = /[!"#$%&'()*+,\-./:;<=>?@[\\\]^_`{|}~]/;
+
+/**
+ * @param {string} ch
+ * @returns {boolean}
+ */
+function isAlphanumeric(ch) { return RE_ALPHANUMERIC.test(ch); }
+
+/**
+ * @param {string} ch
+ * @returns {boolean}
+ */
+function isWhitespace(ch) { return RE_WHITESPACE.test(ch); }
+
+/**
+ * @param {string} ch
+ * @returns {boolean}
+ */
+function isAsciiPunctuation(ch) { return RE_ASCII_PUNCTUATION.test(ch); }
+
+/**
+ * Precompute matched bracket/paren pairs using a stack in O(n).
+ * Skips backslash-escaped characters so that \[ \] \( \) don't create false pairs.
+ * Returns a Map from opening index to closing index.
+ *
+ * @param {string} str
+ * @param {string} open
+ * @param {string} close
+ * @returns {Map<number, number>}
+ */
+function precomputePairs(str, open, close) {
+  const pairs = new Map();
+  const stack = [];
+  let i = 0;
+  while (i < str.length) {
+    if (str[i] === '\\' && i + 1 < str.length && isAsciiPunctuation(str[i + 1])) {
+      i += 2;
+      continue;
+    }
+    // Skip code spans — brackets inside are literal
+    if (str[i] === '`') {
+      const tickCount = countRun(str, i, '`');
+      const closeIdx = findBacktickCloser(str, tickCount, i + tickCount);
+      if (closeIdx !== -1) {
+        i = closeIdx + tickCount;
+        continue;
+      }
+      i += tickCount;
+      continue;
+    }
+    if (str[i] === open) stack.push(i);
+    else if (str[i] === close && stack.length > 0) {
+      pairs.set(stack.pop(), i);
+    }
+    i++;
+  }
+  return pairs;
+}
+
+/**
+ * Find exactly tickCount consecutive backticks (not more, not less).
+ * CommonMark: backslash inside code spans is literal, so no escape skipping.
+ *
+ * @param {string} text
+ * @param {number} tickCount
+ * @param {number} start
+ * @returns {number}
+ */
+function findBacktickCloser(text, tickCount, start) {
+  let i = start;
+  while (i < text.length) {
+    if (text[i] === '`') {
+      const run = countRun(text, i, '`');
+      if (run === tickCount) return i;
+      i += run;
+      continue;
+    }
+    i++;
+  }
+  return -1;
+}
+
+/**
+ * Find a closing ~~ for strikethrough, skipping backslash-escaped characters.
+ *
+ * @param {string} text
+ * @param {number} start
+ * @returns {number}
+ */
+function findStrikethroughCloser(text, start) {
+  let i = start;
+  while (i < text.length - 1) {
+    if (text[i] === '\\' && isAsciiPunctuation(text[i + 1])) {
+      i += 2;
+      continue;
+    }
+    if (text[i] === '~' && text[i + 1] === '~') return i;
+    i++;
+  }
+  return -1;
+}
+
+/**
+ * Find a closer for emphasis marker ch with at least markerLen consecutive chars.
+ * Skips \* and \_ (escaped markers).
+ * Closer condition: run >= markerLen AND preceding char is not whitespace.
+ *
+ * @param {string} text
+ * @param {string} ch
+ * @param {number} markerLen
+ * @param {number} start
+ * @returns {number}
+ */
+function findEmphasisCloser(text, ch, markerLen, start) {
+  let i = start;
+  while (i < text.length) {
+    if (text[i] === '\\' && i + 1 < text.length && isAsciiPunctuation(text[i + 1])) {
+      i += 2;
+      continue;
+    }
+    if (text[i] === ch) {
+      const run = countRun(text, i, ch);
+      if (run >= markerLen && i > 0 && !isWhitespace(text[i - 1])) return i;
+      i += run;
+      continue;
+    }
+    i++;
+  }
+  return -1;
+}
+
+/**
+ * Handle emphasis markers (* or _).
+ * Returns { output, nextPos } on success, or null if the run should be treated as literal.
+ *
+ * @param {string} text
+ * @param {number} pos
+ * @returns {{ output: string, nextPos: number } | null}
+ */
+function handleEmphasis(text, pos) {
+  const ch = text[pos];
+
+  // Count run length
+  const runLen = countRun(text, pos, ch);
+
+  // Opener condition:
+  //   - prevChar is NOT alphanumeric (or start of string)
+  //   - char after the run is NOT whitespace and not end of string
+  const prevChar = pos > 0 ? text[pos - 1] : '';
+  const afterIdx = pos + runLen;
+  const afterChar = afterIdx < text.length ? text[afterIdx] : '';
+
+  const isOpener = !isAlphanumeric(prevChar) && afterChar !== '' && !isWhitespace(afterChar);
+
+  if (!isOpener) return null;
+
+  // Try matching closest, longest-first (min(runLen, 3) down to 1)
+  const maxMarker = Math.min(runLen, 3);
+  // NOTE: O(n²) worst case when many openers lack closers (k openers × O(n) scan).
+  // Bounded by MAX_INPUT=10000; measured ~163ms worst case. Acceptable for notification text.
+  for (let markerLen = maxMarker; markerLen >= 1; markerLen--) {
+    let searchFrom = pos + runLen;
+    while (true) {
+      const idx = findEmphasisCloser(text, ch, markerLen, searchFrom);
+      if (idx === -1) break; // no closer found for this markerLen
+      const content = text.slice(pos + markerLen, idx);
+      if (content.length === 0) {
+        // Empty emphasis — skip this closer and keep searching
+        searchFrom = idx + countRun(text, idx, ch);
+        continue;
+      }
+      return { output: stripInline(content), nextPos: idx + markerLen };
+    }
+  }
+
+  return null;
+}
+
+/**
+ * Strip inline markdown formatting by scanning character by character.
+ *
+ * @param {string} text
+ * @returns {string}
+ */
+function stripInline(text) {
+  // NOTE: Recursive calls for emphasis/strikethrough/link content.
+  // Depth bounded by nesting level (shallow in real-world markdown).
+  const bracketPairs = precomputePairs(text, '[', ']');
+  const parenPairs = precomputePairs(text, '(', ')');
+
+  let out = '';
+  let i = 0;
+
+  while (i < text.length) {
+    const ch = text[i];
+
+    if (ch === '\\' && i + 1 < text.length && isAsciiPunctuation(text[i + 1])) {
+      out += text[i + 1];
+      i += 2;
+      continue;
+    }
+
+    if (ch === '`') {
+      const tickCount = countRun(text, i, '`');
+      const closeIdx = findBacktickCloser(text, tickCount, i + tickCount);
+      if (closeIdx !== -1) {
+        out += text.slice(i + tickCount, closeIdx);
+        i = closeIdx + tickCount;
+        continue;
+      }
+      // Unclosed backtick(s) — output as literal
+      out += text.slice(i, i + tickCount);
+      i += tickCount;
+      continue;
+    }
+
+    if (ch === '!' && i + 1 < text.length && text[i + 1] === '[') {
+      const closeBracket = bracketPairs.get(i + 1);
+      if (closeBracket !== undefined && closeBracket + 1 < text.length && text[closeBracket + 1] === '(') {
+        const closeParen = parenPairs.get(closeBracket + 1);
+        if (closeParen !== undefined) {
+          const altText = text.slice(i + 2, closeBracket);
+          out += stripInline(altText);
+          i = closeParen + 1;
+          continue;
+        }
+      }
+      // Not a valid image — output ! as literal
+      out += ch;
+      i++;
+      continue;
+    }
+
+    if (ch === '[') {
+      const closeBracket = bracketPairs.get(i);
+      if (closeBracket !== undefined && closeBracket + 1 < text.length && text[closeBracket + 1] === '(') {
+        const closeParen = parenPairs.get(closeBracket + 1);
+        if (closeParen !== undefined) {
+          const linkText = text.slice(i + 1, closeBracket);
+          out += stripInline(linkText);
+          i = closeParen + 1;
+          continue;
+        }
+      }
+      // Not a valid link — output as literal
+      out += ch;
+      i++;
+      continue;
+    }
+
+    if (ch === '~' && i + 1 < text.length && text[i + 1] === '~') {
+      const searchStart = i + 2;
+      const closeIdx = findStrikethroughCloser(text, searchStart);
+      if (closeIdx !== -1) {
+        const content = text.slice(searchStart, closeIdx);
+        if (content.length === 0) {
+          out += '~~';
+          i += 2;
+          continue;
+        }
+        out += stripInline(content);
+        i = closeIdx + 2;
+        continue;
+      }
+      // No closer — output ~~ as literal
+      out += '~~';
+      i += 2;
+      continue;
+    }
+
+    if (ch === '*' || ch === '_') {
+      const result = handleEmphasis(text, i);
+      if (result !== null) {
+        out += result.output;
+        i = result.nextPos;
+        continue;
+      }
+      // Not an opener or no closer — output entire run as literal
+      const runLen = countRun(text, i, ch);
+      out += text.slice(i, i + runLen);
+      i += runLen;
+      continue;
+    }
+
+    out += ch;
+    i++;
+  }
+
+  return out;
+}
+
 /**
  * Format tool information for display in the notification.
  *
@@ -98,6 +438,21 @@ export async function waitForResponse({ server, topic, requestId, timeout }) {
  * @returns {{ title: string, message: string }}
  */
 export function formatToolInfo({ hook_event_name, tool_name, tool_input }) {
+  // Plan approval detection
+  if (tool_name === 'ExitPlanMode' && typeof tool_input?.plan === 'string') {
+    const PLAN_MESSAGE_MAX_LENGTH = 300;
+    const title = 'Claude Code: Plan Review';
+    if (!tool_input.plan.trim()) {
+      return { title, message: '(empty plan)' };
+    }
+    const raw = tool_input.plan;
+    const plain = stripMarkdown(raw);
+    const message = plain
+      ? (plain.length > PLAN_MESSAGE_MAX_LENGTH ? plain.slice(0, PLAN_MESSAGE_MAX_LENGTH) + '...' : plain)
+      : '(empty plan)';
+    return { title, message };
+  }
+
   const title = `Claude Code: ${tool_name}`;
   let message;