claude-tg 1.0.8 → 1.0.10

package/bin/claude-tg.js

@@ -140,6 +140,25 @@ daemon
     }
   });
 
+const HOOKS_LOG_PATH = path.join(CONFIG_DIR, 'hooks.log');
+
+program
+  .command('hooks-log')
+  .description('Show hook execution log (diagnose if hooks are firing)')
+  .action(() => {
+    if (!fs.existsSync(HOOKS_LOG_PATH)) {
+      console.log('No hooks log found. Hooks have not been called yet.');
+      console.log('Make sure Claude Code is running and try using a tool.');
+      return;
+    }
+    const content = fs.readFileSync(HOOKS_LOG_PATH, 'utf8');
+    const lines = content.trim().split('\n');
+    const last = lines.slice(-30);
+    console.log(`--- Last ${last.length} hook log entries ---\n`);
+    console.log(last.join('\n'));
+    console.log(`\nFull log: ${HOOKS_LOG_PATH}`);
+  });
+
 // --- Send commands (stateless, no daemon needed) ---
 
 function createTelegramClient() {
@@ -276,18 +295,27 @@ program
         const settings = JSON.parse(fs.readFileSync(settingsPath, 'utf8'));
         const hooks = settings.hooks || {};
 
-        // Extract script path from hook command (handles optional env prefix)
+        // Extract script path and node binary from hook command
         function extractScriptPath(cmd) {
           const match = cmd.match(/node\s+(.+)$/);
           return match ? match[1].trim() : null;
         }
+        function extractNodeBin(cmd) {
+          const match = cmd.match(/^(?:\S+=\S+\s+)?(\S*node)\s/);
+          return match ? match[1] : 'node';
+        }
 
         const permHooks = hooks.PermissionRequest || [];
         const permCmd = permHooks[0]?.hooks?.[0]?.command || 'NOT FOUND';
         console.log(`  PermissionRequest: ${permCmd}`);
         if (permCmd !== 'NOT FOUND') {
+          const nodeBin = extractNodeBin(permCmd);
           const scriptPath = extractScriptPath(permCmd);
-          if (scriptPath && fs.existsSync(scriptPath)) {
+          if (nodeBin !== 'node' && !fs.existsSync(nodeBin)) {
+            console.log(`    Node binary: NO — ${nodeBin}`);
+            console.log('    Run: claude-tg setup (to fix hook paths)');
+            ok = false;
+          } else if (scriptPath && fs.existsSync(scriptPath)) {
             console.log('    File exists: YES');
           } else {
             console.log(`    File exists: NO — ${scriptPath || permCmd}`);
@@ -299,8 +327,13 @@ program
         const notifCmd = notifHooks[0]?.hooks?.[0]?.command || 'NOT FOUND';
         console.log(`  Notification: ${notifCmd}`);
         if (notifCmd !== 'NOT FOUND') {
+          const nodeBin = extractNodeBin(notifCmd);
           const scriptPath = extractScriptPath(notifCmd);
-          if (scriptPath && fs.existsSync(scriptPath)) {
+          if (nodeBin !== 'node' && !fs.existsSync(nodeBin)) {
+            console.log(`    Node binary: NO — ${nodeBin}`);
+            console.log('    Run: claude-tg setup (to fix hook paths)');
+            ok = false;
+          } else if (scriptPath && fs.existsSync(scriptPath)) {
             console.log('    File exists: YES');
           } else {
             console.log(`    File exists: NO — ${scriptPath || notifCmd}`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-tg",
-  "version": "1.0.8",
+  "version": "1.0.10",
   "description": "Control Claude Code from Telegram — approve permissions, answer questions, reply to idle sessions, send files, all from your phone",
   "bin": {
     "claude-tg": "./bin/claude-tg.js"

package/src/hooks/notification.js

@@ -4,10 +4,21 @@
 // Detects parent TTY, reads hook input from stdin, fires POST to daemon, exits.
 
 const http = require('http');
+const fs = require('fs');
+const path = require('path');
 const { execSync } = require('child_process');
 
 const DAEMON_PORT = parseInt(process.env.CLAUDE_TG_PORT || '7483', 10);
 const DAEMON_HOST = '127.0.0.1';
+const HOOK_LOG = path.join(process.env.HOME, '.claude-telegram-bridge', 'hooks.log');
+
+function hookLog(msg) {
+  try {
+    const dir = path.dirname(HOOK_LOG);
+    if (!fs.existsSync(dir)) fs.mkdirSync(dir, { recursive: true });
+    fs.appendFileSync(HOOK_LOG, `[${new Date().toISOString()}] [notify] ${msg}\n`);
+  } catch {}
+}
 
 function readStdin() {
   return new Promise((resolve, reject) => {
@@ -70,21 +81,27 @@ function postToDaemon(body) {
 }
 
 async function main() {
+  hookLog('Hook called');
   try {
     const input = await readStdin();
     const hookInput = input.hookInput || input;
     const ttyPath = findTty();
+    const notifType = hookInput.notification_type || hookInput.type || 'unknown';
+
+    hookLog(`type=${notifType} session=${hookInput.session_id} tty=${ttyPath} port=${DAEMON_PORT}`);
 
     await postToDaemon({
       session_id: hookInput.session_id,
       cwd: hookInput.cwd,
-      notification_type: hookInput.notification_type || hookInput.type,
+      notification_type: notifType,
       message: hookInput.message,
       transcript_path: hookInput.transcript_path,
       tty_path: ttyPath,
     });
-  } catch {
-    // Daemon unreachable — silently ignore
+
+    hookLog(`Sent to daemon OK`);
+  } catch (err) {
+    hookLog(`ERROR: ${err.message || err}`);
   }
   process.exit(0);
 }

package/src/hooks/permission-request.js

@@ -5,10 +5,21 @@
 // On error/timeout: exits 0 with no output → falls back to local dialog.
 
 const http = require('http');
+const fs = require('fs');
+const path = require('path');
 const { execSync } = require('child_process');
 
 const DAEMON_PORT = parseInt(process.env.CLAUDE_TG_PORT || '7483', 10);
 const DAEMON_HOST = '127.0.0.1';
+const HOOK_LOG = path.join(process.env.HOME, '.claude-telegram-bridge', 'hooks.log');
+
+function hookLog(msg) {
+  try {
+    const dir = path.dirname(HOOK_LOG);
+    if (!fs.existsSync(dir)) fs.mkdirSync(dir, { recursive: true });
+    fs.appendFileSync(HOOK_LOG, `[${new Date().toISOString()}] [perm] ${msg}\n`);
+  } catch {}
+}
 
 function findTty() {
   try {
@@ -71,11 +82,14 @@ function postToDaemon(path, body) {
 }
 
 async function main() {
+  hookLog('Hook called');
   try {
     const input = await readStdin();
-
     const hookInput = input.hookInput || input;
     const ttyPath = findTty();
+
+    hookLog(`tool=${hookInput.tool_name} session=${hookInput.session_id} tty=${ttyPath} port=${DAEMON_PORT}`);
+
     const result = await postToDaemon('/api/permission', {
       session_id: hookInput.session_id,
       cwd: hookInput.cwd,
@@ -87,10 +101,12 @@ async function main() {
     });
 
     if (!result || !result.decision) {
-      // No decision — fall back to local dialog
+      hookLog('No decision from daemon — falling back to local dialog');
       process.exit(0);
     }
 
+    hookLog(`Decision: ${JSON.stringify(result.decision)}`);
+
     const output = {
       hookSpecificOutput: {
         hookEventName: 'PermissionRequest',
@@ -100,8 +116,8 @@ async function main() {
     };
 
     process.stdout.write(JSON.stringify(output));
-  } catch {
-    // Daemon unreachable or error — fall back to local dialog
+  } catch (err) {
+    hookLog(`ERROR: ${err.message || err}`);
     process.exit(0);
   }
 }

package/src/setup.js

@@ -26,13 +26,16 @@ function telegramApiCall(token, method) {
 
 function getHooksConfig(port) {
   const envPrefix = port !== 7483 ? `CLAUDE_TG_PORT=${port} ` : '';
+  // Use absolute path to the node binary that ran setup — ensures hooks work
+  // even if node isn't in PATH for non-interactive shells (nvm, fnm, etc.)
+  const nodeBin = process.execPath;
   return {
     PermissionRequest: [
       {
         hooks: [
           {
             type: 'command',
-            command: `${envPrefix}node ${path.join(HOOKS_DIR, 'permission-request.js')}`,
+            command: `${envPrefix}${nodeBin} ${path.join(HOOKS_DIR, 'permission-request.js')}`,
             timeout: 1800,
             statusMessage: 'Waiting for Telegram approval...',
           },
@@ -41,11 +44,10 @@ function getHooksConfig(port) {
     ],
     Notification: [
       {
-        matcher: 'idle_prompt|elicitation_dialog',
         hooks: [
           {
             type: 'command',
-            command: `${envPrefix}node ${path.join(HOOKS_DIR, 'notification.js')}`,
+            command: `${envPrefix}${nodeBin} ${path.join(HOOKS_DIR, 'notification.js')}`,
             async: true,
           },
         ],