-
Notifications
You must be signed in to change notification settings - Fork 7
Closed
Labels
bugSomething isn't workingSomething isn't workingwindowsWindows platform specificWindows platform specific
Description
Problem
electron/process-detector.ts has two Unix-only assumptions that make the entire module non-functional on Windows:
1. ps command does not exist on Windows (line 119)
execFile("ps", ["-eo", "pid,ppid,args"], (err, stdout) => { ... })ps -eo pid,ppid,args is a Unix command. On Windows this throws ENOENT. The entire detectCli() function fails immediately.
2. Path splitting only handles forward slashes (line 26)
const baseName = firstToken.split("/").pop() ?? "";Windows paths like C:\Program Files\nodejs\node.exe are not split on \, so baseName retains the full path string. This causes wrapper detection (WRAPPER_NAMES.has(baseName)) and CLI pattern matching to fail.
Windows alternatives for process enumeration
Get-CimInstance Win32_Process | Select ProcessId,ParentProcessId,CommandLine— exposes all three required fields (PID, PPID, CommandLine)tasklist /fo csv /v— available by default but lacks ParentProcessId and full CommandLinewmic process get ProcessId,ParentProcessId,CommandLine /format:csv— deprecated, removed from newer Windows
Affected functionality
- CLI tool detection (claude, codex, etc.) in terminal tabs
- Process tree traversal for session tracking
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
bugSomething isn't workingSomething isn't workingwindowsWindows platform specificWindows platform specific