ai-lens 0.8.21 → 0.8.22

package/.commithash

@@ -1 +1 @@
-9f6f7a9
+23120cb

package/cli/hooks.js

@@ -106,9 +106,15 @@ export function captureCommand() {
   const capturePath = CAPTURE_PATH.replace(/\\/g, '/');
   const escaped = shellEscape(capturePath);
   // /usr/bin/env doesn't need shell-escaping, but named paths do
-  return nodePath === '/usr/bin/env node'
+  const base = nodePath === '/usr/bin/env node'
     ? `/usr/bin/env node ${escaped}`
     : `${shellEscape(nodePath.replace(/\\/g, '/'))} ${escaped}`;
+  // On Windows, prefix with "& " so the command works in both PowerShell and cmd.exe.
+  // PowerShell treats a quoted path like "node.exe" as a string expression, not a command;
+  // the & (call) operator is required. In cmd.exe, & is a command separator — the empty
+  // first part is a no-op, and the real command runs as the second part.
+  if (process.platform === 'win32') return `& ${base}`;
+  return base;
 }
 
 // ---------------------------------------------------------------------------

package/cli/status.js

@@ -98,8 +98,10 @@ function validateHookCommandPaths(tool) {
   }
 
   // Extract node path (first token). Skip if /usr/bin/env node (node resolved via PATH)
-  if (!command.startsWith('/usr/bin/env node')) {
-    const nodeMatch = command.match(/^["']([^"']+)["']|^(\S+)/);
+  // Strip "& " prefix (PowerShell call operator, added on Windows) before matching.
+  const cmdForNode = command.replace(/^& /, '');
+  if (!cmdForNode.startsWith('/usr/bin/env node')) {
+    const nodeMatch = cmdForNode.match(/^["']([^"']+)["']|^(\S+)/);
     if (nodeMatch) {
       const nodePath = nodeMatch[1] || nodeMatch[2];
       if (nodePath !== 'node' && !existsSync(nodePath)) {
@@ -174,21 +176,72 @@ function checkCaptureRun(installedTools) {
       }
     } catch { /* best effort */ }
 
-    toolResults.push({ name, ok: exitOk, cmd: testCmd, exitDetail, captureNote });
+    // Shell command test: execute the actual hook command string through the shell
+    // to catch path/quoting/slash issues that direct spawn bypasses.
+    // This simulates how Claude Code / Cursor actually invoke hooks on the OS.
+    const shellSessionId = 'status-shell-' + Date.now() + '-' + Math.random().toString(36).slice(2, 7);
+    const shellEvent = JSON.stringify({
+      hook_event_name: 'Stop',
+      session_id: shellSessionId,
+      stop_reason: 'test',
+    });
+    let shellOk = false;
+    let shellDetail = '';
+    try {
+      const shellResult = spawnSync(command, {
+        input: shellEvent,
+        shell: true,
+        encoding: 'utf-8',
+        timeout: 10_000,
+        env: { ...process.env, AI_LENS_PROJECTS: join(homedir(), '.ai-lens-status-check-nonexistent') },
+        windowsHide: true,
+      });
+      if (shellResult.error) throw shellResult.error;
+      shellOk = shellResult.status === 0;
+      shellDetail = shellOk ? 'exit 0' : `Exit code: ${shellResult.status}\nError: ${(shellResult.stderr || '').trim() || '(no stderr)'}`;
+    } catch (err) {
+      shellDetail = `Exit code: N/A\nError: ${err.message}`;
+    }
+
+    let shellCaptureNote = '';
+    try {
+      if (existsSync(CAPTURE_LOG_PATH)) {
+        const logLines = readFileSync(CAPTURE_LOG_PATH, 'utf-8').split(/\r?\n/).filter(Boolean);
+        for (let i = logLines.length - 1; i >= 0; i--) {
+          try {
+            const entry = JSON.parse(logLines[i]);
+            if (entry.session_id === shellSessionId) {
+              shellCaptureNote = entry.reason || entry.msg || 'unknown';
+              break;
+            }
+          } catch { /* skip unparseable lines */ }
+        }
+      }
+    } catch { /* best effort */ }
+
+    toolResults.push({ name, ok: exitOk, cmd: testCmd, exitDetail, captureNote, shellOk, shellCmd: command, shellDetail, shellCaptureNote });
   }
 
-  const allOk = toolResults.every(r => r.ok);
-  const failedTools = toolResults.filter(r => !r.ok).map(r => r.name);
+  const allOk = toolResults.every(r => r.ok && r.shellOk);
+  const failedTools = toolResults.filter(r => !r.ok || !r.shellOk).map(r => r.name);
   const summaryParts = toolResults.map(r => {
-    const status = r.ok ? 'OK' : 'FAILED';
-    return r.captureNote ? `${r.name}: ${status} (${r.captureNote})` : `${r.name}: ${status}`;
+    const directStatus = r.ok ? 'OK' : 'FAILED';
+    const shellStatus = r.shellOk ? 'OK' : 'FAILED';
+    const parts = [`${r.name}: ${directStatus}`];
+    if (!r.shellOk) parts[0] += `, shell: ${shellStatus}`;
+    if (r.captureNote) parts[0] += ` (${r.captureNote})`;
+    return parts[0];
   });
   const summary = allOk
     ? `capture runs OK (${summaryParts.join(', ')})`
     : `capture failed for: ${failedTools.join(', ')}`;
-  const detail = toolResults.map(r =>
-    `[${r.name}]\n  Ran: ${r.cmd}\n  Result: ${r.exitDetail}${r.captureNote ? `\n  Capture log: ${r.captureNote}` : ''}`
-  ).join('\n\n');
+  const detail = toolResults.map(r => {
+    let text = `[${r.name}]\n  Ran: ${r.cmd}\n  Result: ${r.exitDetail}`;
+    if (r.captureNote) text += `\n  Capture log: ${r.captureNote}`;
+    text += `\n  Shell: ${r.shellCmd} < (test event)\n  Shell result: ${r.shellDetail}`;
+    if (r.shellCaptureNote) text += `\n  Shell capture log: ${r.shellCaptureNote}`;
+    return text;
+  }).join('\n\n');
 
   return { ok: allOk, summary, detail };
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ai-lens",
-  "version": "0.8.21",
+  "version": "0.8.22",
   "type": "module",
   "description": "Centralized session analytics for AI coding tools",
   "bin": {