runsignal 0.1.5 → 0.1.6

package/bin/runsignal.js

@@ -3,6 +3,7 @@
 import {Command} from 'commander';
 import {spawn, execSync} from 'node:child_process';
 import {promises as fs} from 'node:fs';
+import {existsSync} from 'node:fs';
 import os from 'node:os';
 import path from 'node:path';
 import {
@@ -122,43 +123,44 @@ function buildHookBlock(shellKind) {
     return `${HOOK_START}
 function RunSignal-Resolve-NativeCommand {
   param([string]$Name)
-  $cmd = Get-Command $Name -CommandType Application -ErrorAction SilentlyContinue | Select-Object -First 1
-  if (-not $cmd) { return $null }
-  return $cmd.Source
+  $candidates = @(Get-Command $Name -CommandType Application -ErrorAction SilentlyContinue)
+  if (-not $candidates -or $candidates.Count -eq 0) { return $null }
+  $preferred = $candidates | Where-Object { $_.Source -match '\\.exe$' } | Select-Object -First 1
+  if ($preferred) { return $preferred.Source }
+  $launcher = $candidates | Where-Object { $_.Source -match '\\.(cmd|bat)$' } | Select-Object -First 1
+  if ($launcher) { return $launcher.Source }
+  $secondary = $candidates | Where-Object { $_.Source -match '\\.[A-Za-z0-9]+$' } | Select-Object -First 1
+  if ($secondary) { return $secondary.Source }
+  return $candidates[0].Source
 }
 function claude {
   $native = RunSignal-Resolve-NativeCommand "claude"
   if (-not $native) {
-    $home = [Environment]::GetFolderPath("UserProfile")
-    $fallback = Join-Path $home ".local\\bin\\claude.exe"
+    $userHome = [Environment]::GetFolderPath("UserProfile")
+    $fallback = Join-Path $userHome ".local\\bin\\claude.exe"
     if (Test-Path $fallback) { $native = $fallback }
   }
   if (-not $native) { Write-Error "RunSignal hook: native claude executable not found. Add your native installer path to PATH (example: %USERPROFILE%\\.local\\bin)."; return }
-  if ($args.Count -eq 0) { & $native; return }
   runsignal wrap -- $native @args
 }
 function codex {
   $native = RunSignal-Resolve-NativeCommand "codex"
   if (-not $native) { Write-Error "RunSignal hook: native codex executable not found."; return }
-  if ($args.Count -eq 0) { & $native; return }
   runsignal wrap -- $native @args
 }
 function cursor {
   $native = RunSignal-Resolve-NativeCommand "cursor"
   if (-not $native) { Write-Error "RunSignal hook: native cursor executable not found."; return }
-  if ($args.Count -eq 0) { & $native; return }
   runsignal wrap -- $native @args
 }
 function gemini {
   $native = RunSignal-Resolve-NativeCommand "gemini"
   if (-not $native) { Write-Error "RunSignal hook: native gemini executable not found."; return }
-  if ($args.Count -eq 0) { & $native; return }
   runsignal wrap -- $native @args
 }
 function aider {
   $native = RunSignal-Resolve-NativeCommand "aider"
   if (-not $native) { Write-Error "RunSignal hook: native aider executable not found."; return }
-  if ($args.Count -eq 0) { & $native; return }
   runsignal wrap -- $native @args
 }
 ${HOOK_END}
@@ -183,35 +185,30 @@ claude() {
   local native
   native="$(runsignal_resolve_native claude)"
   if [ -z "$native" ]; then echo "RunSignal hook: native claude executable not found." >&2; return 127; fi
-  if [ "$#" -eq 0 ]; then "$native"; return $?; fi
   runsignal wrap -- "$native" "$@"
 }
 codex() {
   local native
   native="$(runsignal_resolve_native codex)"
   if [ -z "$native" ]; then echo "RunSignal hook: native codex executable not found." >&2; return 127; fi
-  if [ "$#" -eq 0 ]; then "$native"; return $?; fi
   runsignal wrap -- "$native" "$@"
 }
 cursor() {
   local native
   native="$(runsignal_resolve_native cursor)"
   if [ -z "$native" ]; then echo "RunSignal hook: native cursor executable not found." >&2; return 127; fi
-  if [ "$#" -eq 0 ]; then "$native"; return $?; fi
   runsignal wrap -- "$native" "$@"
 }
 gemini() {
   local native
   native="$(runsignal_resolve_native gemini)"
   if [ -z "$native" ]; then echo "RunSignal hook: native gemini executable not found." >&2; return 127; fi
-  if [ "$#" -eq 0 ]; then "$native"; return $?; fi
   runsignal wrap -- "$native" "$@"
 }
 aider() {
   local native
   native="$(runsignal_resolve_native aider)"
   if [ -z "$native" ]; then echo "RunSignal hook: native aider executable not found." >&2; return 127; fi
-  if [ "$#" -eq 0 ]; then "$native"; return $?; fi
   runsignal wrap -- "$native" "$@"
 }
 ${HOOK_END}
@@ -435,6 +432,35 @@ function quoteForShell(arg) {
   return `"${escaped}"`;
 }
 
+function stripWrappingQuotes(value) {
+  const raw = String(value || '').trim();
+  if ((raw.startsWith('"') && raw.endsWith('"')) || (raw.startsWith("'") && raw.endsWith("'"))) {
+    return raw.slice(1, -1);
+  }
+  return raw;
+}
+
+function isWindowsScriptLauncher(filePath) {
+  if (process.platform !== 'win32') return false;
+  const ext = path.extname(String(filePath || '')).toLowerCase();
+  return ext === '.cmd' || ext === '.bat' || ext === '.ps1';
+}
+
+function normalizeWindowsExecutablePath(filePath) {
+  const raw = String(filePath || '');
+  if (process.platform !== 'win32') return raw;
+  if (!raw) return raw;
+  const ext = path.extname(raw).toLowerCase();
+  if (ext) return raw;
+  const exe = `${raw}.exe`;
+  if (existsSync(exe)) return exe;
+  const cmd = `${raw}.cmd`;
+  if (existsSync(cmd)) return cmd;
+  const bat = `${raw}.bat`;
+  if (existsSync(bat)) return bat;
+  return raw;
+}
+
 async function runLogin(options) {
   const config = await readConfig();
   const baseUrl = String(options.appUrl || config.app_url || DEFAULT_APP_URL).replace(/\/$/, '');
@@ -593,13 +619,71 @@ async function runWrap(commandParts) {
     commandParts.length === 1
       ? String(commandParts[0])
       : commandParts.map((part) => quoteForShell(part)).join(' ');
+  const singleCommand = String(commandParts[0] || '');
+  const singleExecutable = normalizeWindowsExecutablePath(stripWrappingQuotes(singleCommand));
   const interactivePassthrough =
-    commandParts.length === 1 && !/[\s"'`]/.test(String(commandParts[0]));
-  const child = spawn(commandString, {
-    stdio: interactivePassthrough ? 'inherit' : ['inherit', 'pipe', 'pipe'],
-    shell: true,
-    env: process.env
-  });
+    commandParts.length === 1 && (existsSync(singleExecutable) || !/\s/.test(singleExecutable));
+  const child = interactivePassthrough
+    ? isWindowsScriptLauncher(singleExecutable)
+      ? spawn(singleExecutable, [], {
+          stdio: 'inherit',
+          shell: true,
+          env: process.env
+        })
+      : spawn(singleExecutable, [], {
+          stdio: 'inherit',
+          shell: false,
+          env: process.env
+        })
+    : spawn(commandString, {
+        stdio: ['inherit', 'pipe', 'pipe'],
+        shell: true,
+        env: process.env
+      });
+
+  let heartbeatInterval = null;
+  if (interactivePassthrough) {
+    const startedAt = Date.now();
+    try {
+      await sendEvent({
+        apiFetch,
+        baseUrl,
+        machineApiKey,
+        event: {
+          message: `Session started: ${singleExecutable}`,
+          event_type: 'session_started',
+          repo,
+          branch,
+          severity: 'info',
+          context,
+          agent: agentLabel
+        }
+      });
+    } catch {
+      // non-blocking
+    }
+
+    heartbeatInterval = setInterval(() => {
+      const elapsedMin = Math.max(1, Math.floor((Date.now() - startedAt) / 60_000));
+      sendEvent({
+        apiFetch,
+        baseUrl,
+        machineApiKey,
+        event: {
+          message: `Session running (${elapsedMin} min): ${singleExecutable}`,
+          event_type: 'session_running',
+          repo,
+          branch,
+          severity: 'info',
+          context,
+          agent: agentLabel
+        }
+      }).catch(() => {
+        // non-blocking
+      });
+    }, 120_000);
+    heartbeatInterval.unref?.();
+  }
 
   let sawApproval = false;
   let sawQuestion = false;
@@ -699,8 +783,12 @@ async function runWrap(commandParts) {
   }
 
   const exitCode = await new Promise((resolve) => {
+    child.on('error', () => resolve(1));
     child.on('close', (code) => resolve(code ?? 1));
   });
+  if (heartbeatInterval) {
+    clearInterval(heartbeatInterval);
+  }
   await Promise.allSettled(Array.from(pendingTasks));
 
   const completionMessage =
@@ -774,7 +862,9 @@ function normalizeEventType(value) {
     raw === 'task_completed' ||
     raw === 'error' ||
     raw === 'quota_warning' ||
-    raw === 'question'
+    raw === 'question' ||
+    raw === 'session_started' ||
+    raw === 'session_running'
   ) {
     return raw;
   }
@@ -944,7 +1034,7 @@ async function runSetup(options) {
 
 const program = new Command();
 
-program.name('runsignal').description('RunSignal CLI').version('0.1.5');
+program.name('runsignal').description('RunSignal CLI').version('0.1.6');
 
 program
   .command('setup')

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "runsignal",
-   "version": "0.1.5",
+   "version": "0.1.6",
   "description": "RunSignal CLI for wrapping AI agent runs and handling approvals",
   "type": "module",
   "license": "MIT",