tokens-for-good 0.4.1 → 0.4.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "tokens-for-good",
-  "version": "0.4.1",
+  "version": "0.4.2",
   "type": "module",
   "description": "Donate your spare AI tokens to research nonprofits for Fierce Philanthropy",
   "bin": {

package/src/cli.js

@@ -1,4 +1,4 @@
-#!/usr/bin/env node
+#!/usr/bin/env node
 
 // CLI entry point for tokens-for-good.
 // Usage:

package/src/init.js

@@ -7,6 +7,7 @@ import { readFileSync, writeFileSync, existsSync, mkdirSync } from 'fs';
 import { join, dirname } from 'path';
 import { homedir } from 'os';
 import { fileURLToPath } from 'url';
+import { spawnSync } from 'child_process';
 import { detectPlatform } from './platform.js';
 import { loadState, saveState } from './state.js';
 
@@ -238,15 +239,44 @@ function writeSessionStartHook() {
       matcher: '',
       hooks: [{
         type: 'command',
-        command: IS_WINDOWS
-          ? 'cmd /c npx -y tokens-for-good session-start-hook'
-          : 'npx -y tokens-for-good session-start-hook',
+        command: hookCommand(),
       }],
     });
   }
   writeJson(abs, cfg);
 }
 
+// Claude Code runs SessionStart hooks under Git Bash on Windows with a
+// stripped PATH that typically does not include C:\Program Files\nodejs,
+// so a bare `npx` lookup fails silently. Resolve the absolute npx path at
+// init time (when the user's full PATH is available) and bake it into the
+// hook command so it works regardless of Claude Code's hook-runner PATH.
+function hookCommand() {
+  if (!IS_WINDOWS) return 'npx -y tokens-for-good session-start-hook';
+
+  const npxPath = resolveWindowsNpxPath();
+  // Bash accepts double-quoted paths with spaces; escape backslashes for JSON.
+  return `"${npxPath}" -y tokens-for-good session-start-hook`;
+}
+
+function resolveWindowsNpxPath() {
+  // First try `where npx.cmd` — most reliable when PATH is correct.
+  try {
+    const r = spawnSync('where', ['npx.cmd'], { encoding: 'utf-8' });
+    if (r.status === 0) {
+      const first = r.stdout.trim().split(/\r?\n/)[0];
+      if (first && existsSync(first)) return first;
+    }
+  } catch { /* fall through */ }
+
+  // Fallback: npx.cmd usually sits alongside node.exe.
+  const alongside = join(dirname(process.execPath), 'npx.cmd');
+  if (existsSync(alongside)) return alongside;
+
+  // Last-resort guess — user's hook may need manual edit if this is wrong.
+  return 'C:\\Program Files\\nodejs\\npx.cmd';
+}
+
 function writeSkillFile(name) {
   const src = join(PKG_ROOT, 'skills', `${name}.md`);
   const dst = join(homedir(), '.claude', 'skills', name, 'SKILL.md');