byterover-cli 2.1.2 → 2.1.3

package/dist/oclif/commands/restart.d.ts

@@ -89,6 +89,7 @@ export default class Restart extends Command {
      */
     private static waitForPidToDie;
     protected cleanupAllDaemonFiles(dataDir: string): void;
+    protected exitProcess(code: number): void;
     protected killAllBrvProcesses(dataDir: string): Promise<void>;
     run(): Promise<void>;
     protected startDaemon(serverPath: string): Promise<EnsureDaemonResult>;

package/dist/oclif/commands/restart.js

@@ -95,7 +95,7 @@ All open sessions and background processes are stopped before the fresh start.`;
      * When cwd is resolved: check only absolute patterns (precise, no false positives).
      * When cwd is unavailable: also check relative fallback patterns (./bin/dev.js).
      */
-    static killByMacOsProcScan(patterns, excludePid) {
+    static killByMacOsProcScan(patterns, excludePids) {
         const psResult = spawnSync('ps', ['-A', '-o', 'pid,args'], { encoding: 'utf8' });
         if (!psResult.stdout)
             return;
@@ -106,7 +106,7 @@ All open sessions and background processes are stopped before the fresh start.`;
                 continue;
             const pid = Number.parseInt(match[1], 10);
             const rawCmdline = match[2].trim();
-            if (Number.isNaN(pid) || pid === excludePid)
+            if (Number.isNaN(pid) || excludePids.has(pid))
                 continue;
             // Resolve relative .js path using cwd map to get an absolute path for matching.
             let cmdline = rawCmdline;
@@ -170,7 +170,7 @@ All open sessions and background processes are stopped before the fresh start.`;
      * Works on all Linux distros including Alpine — /proc is a kernel feature,
      * no userspace tools required.
      */
-    static killByProcScan(patterns, excludePid) {
+    static killByProcScan(patterns, excludePids) {
         let entries;
         try {
             entries = readdirSync('/proc');
@@ -180,7 +180,7 @@ All open sessions and background processes are stopped before the fresh start.`;
         }
         for (const entry of entries) {
             const pid = Number.parseInt(entry, 10);
-            if (Number.isNaN(pid) || pid === excludePid)
+            if (Number.isNaN(pid) || excludePids.has(pid))
                 continue;
             try {
                 const args = readFileSync(join('/proc', entry, 'cmdline'), 'utf8')
@@ -235,17 +235,20 @@ All open sessions and background processes are stopped before the fresh start.`;
     static patternKill() {
         const brvBinDir = dirname(process.argv[1]);
         const allPatterns = Restart.buildKillPatterns(brvBinDir, process.argv[1]);
+        // Exclude both the current node process and its parent shell wrapper (install.sh installs
+        // use a shell script that forks node — killing the wrapper garbles terminal output).
+        const excludePids = new Set([process.pid, process.ppid]);
         if (process.platform === 'win32') {
             const whereClause = allPatterns.map((p) => `$_.CommandLine -like '*${p}*'`).join(' -or ');
-            const script = `Get-CimInstance Win32_Process | Where-Object { (${whereClause}) -and $_.ProcessId -ne ${process.pid} } | ForEach-Object { Stop-Process -Id $_.ProcessId -Force -ErrorAction SilentlyContinue }`;
+            const script = `Get-CimInstance Win32_Process | Where-Object { (${whereClause}) -and $_.ProcessId -ne ${process.pid} -and $_.ProcessId -ne ${process.ppid} } | ForEach-Object { Stop-Process -Id $_.ProcessId -Force -ErrorAction SilentlyContinue }`;
             spawnSync('powershell', ['-Command', script], { stdio: 'ignore' });
         }
         else if (process.platform === 'linux') {
-            Restart.killByProcScan(allPatterns, process.pid);
+            Restart.killByProcScan(allPatterns, excludePids);
         }
         else {
             // macOS (and other Unix): ps -A scan with lsof cwd resolution for relative paths
-            Restart.killByMacOsProcScan(allPatterns, process.pid);
+            Restart.killByMacOsProcScan(allPatterns, excludePids);
         }
     }
     static sleep(ms) {
@@ -284,6 +287,10 @@ All open sessions and background processes are stopped before the fresh start.`;
             }
         }
     }
+    exitProcess(code) {
+        // eslint-disable-next-line n/no-process-exit, unicorn/no-process-exit
+        process.exit(code);
+    }
     async killAllBrvProcesses(dataDir) {
         this.log('Stopping processes...');
         // Read PID directly from daemon.json — no health-check filtering.
@@ -315,7 +322,7 @@ All open sessions and background processes are stopped before the fresh start.`;
             const result = await this.startDaemon(serverPath);
             if (result.success) {
                 this.log(`Daemon started (PID ${result.info.pid}, port ${result.info.port})`);
-                return;
+                break;
             }
             const detail = result.spawnError ? ` (${result.spawnError})` : '';
             if (attempt < MAX_ATTEMPTS) {
@@ -326,6 +333,11 @@ All open sessions and background processes are stopped before the fresh start.`;
             }
         }
         /* eslint-enable no-await-in-loop */
+        // Force exit — oclif does not call process.exit() after run() returns,
+        // relying on the event loop to drain. In production, third-party plugin
+        // hooks (e.g. @oclif/plugin-update) can leave open handles that prevent
+        // the process from exiting naturally. mcp.ts uses the same pattern.
+        this.exitProcess(0);
     }
     async startDaemon(serverPath) {
         return ensureDaemonRunning({ serverPath, timeoutMs: DAEMON_START_TIMEOUT_MS });

package/oclif.manifest.json

@@ -997,6 +997,104 @@
         "switch.js"
       ]
     },
+    "space:list": {
+      "aliases": [],
+      "args": {},
+      "description": "List all teams and spaces",
+      "examples": [
+        "<%= config.bin %> space list",
+        "<%= config.bin %> space list --format json"
+      ],
+      "flags": {
+        "format": {
+          "char": "f",
+          "description": "Output format",
+          "name": "format",
+          "default": "text",
+          "hasDynamicHelp": false,
+          "multiple": false,
+          "options": [
+            "text",
+            "json"
+          ],
+          "type": "option"
+        }
+      },
+      "hasDynamicHelp": false,
+      "hiddenAliases": [],
+      "id": "space:list",
+      "pluginAlias": "byterover-cli",
+      "pluginName": "byterover-cli",
+      "pluginType": "core",
+      "strict": true,
+      "enableJsonFlag": false,
+      "isESM": true,
+      "relativePath": [
+        "dist",
+        "oclif",
+        "commands",
+        "space",
+        "list.js"
+      ]
+    },
+    "space:switch": {
+      "aliases": [],
+      "args": {},
+      "description": "Switch to a different space",
+      "examples": [
+        "<%= config.bin %> space switch --team acme --name my-space",
+        "<%= config.bin %> space switch --team acme --name my-space --format json"
+      ],
+      "flags": {
+        "format": {
+          "char": "f",
+          "description": "Output format",
+          "name": "format",
+          "default": "text",
+          "hasDynamicHelp": false,
+          "multiple": false,
+          "options": [
+            "text",
+            "json"
+          ],
+          "type": "option"
+        },
+        "name": {
+          "char": "n",
+          "description": "Name of the space to switch to",
+          "name": "name",
+          "required": true,
+          "hasDynamicHelp": false,
+          "multiple": false,
+          "type": "option"
+        },
+        "team": {
+          "char": "t",
+          "description": "Team name",
+          "name": "team",
+          "required": true,
+          "hasDynamicHelp": false,
+          "multiple": false,
+          "type": "option"
+        }
+      },
+      "hasDynamicHelp": false,
+      "hiddenAliases": [],
+      "id": "space:switch",
+      "pluginAlias": "byterover-cli",
+      "pluginName": "byterover-cli",
+      "pluginType": "core",
+      "strict": true,
+      "enableJsonFlag": false,
+      "isESM": true,
+      "relativePath": [
+        "dist",
+        "oclif",
+        "commands",
+        "space",
+        "switch.js"
+      ]
+    },
     "providers:connect": {
       "aliases": [],
       "args": {
@@ -1237,104 +1335,6 @@
         "switch.js"
       ]
     },
-    "space:list": {
-      "aliases": [],
-      "args": {},
-      "description": "List all teams and spaces",
-      "examples": [
-        "<%= config.bin %> space list",
-        "<%= config.bin %> space list --format json"
-      ],
-      "flags": {
-        "format": {
-          "char": "f",
-          "description": "Output format",
-          "name": "format",
-          "default": "text",
-          "hasDynamicHelp": false,
-          "multiple": false,
-          "options": [
-            "text",
-            "json"
-          ],
-          "type": "option"
-        }
-      },
-      "hasDynamicHelp": false,
-      "hiddenAliases": [],
-      "id": "space:list",
-      "pluginAlias": "byterover-cli",
-      "pluginName": "byterover-cli",
-      "pluginType": "core",
-      "strict": true,
-      "enableJsonFlag": false,
-      "isESM": true,
-      "relativePath": [
-        "dist",
-        "oclif",
-        "commands",
-        "space",
-        "list.js"
-      ]
-    },
-    "space:switch": {
-      "aliases": [],
-      "args": {},
-      "description": "Switch to a different space",
-      "examples": [
-        "<%= config.bin %> space switch --team acme --name my-space",
-        "<%= config.bin %> space switch --team acme --name my-space --format json"
-      ],
-      "flags": {
-        "format": {
-          "char": "f",
-          "description": "Output format",
-          "name": "format",
-          "default": "text",
-          "hasDynamicHelp": false,
-          "multiple": false,
-          "options": [
-            "text",
-            "json"
-          ],
-          "type": "option"
-        },
-        "name": {
-          "char": "n",
-          "description": "Name of the space to switch to",
-          "name": "name",
-          "required": true,
-          "hasDynamicHelp": false,
-          "multiple": false,
-          "type": "option"
-        },
-        "team": {
-          "char": "t",
-          "description": "Team name",
-          "name": "team",
-          "required": true,
-          "hasDynamicHelp": false,
-          "multiple": false,
-          "type": "option"
-        }
-      },
-      "hasDynamicHelp": false,
-      "hiddenAliases": [],
-      "id": "space:switch",
-      "pluginAlias": "byterover-cli",
-      "pluginName": "byterover-cli",
-      "pluginType": "core",
-      "strict": true,
-      "enableJsonFlag": false,
-      "isESM": true,
-      "relativePath": [
-        "dist",
-        "oclif",
-        "commands",
-        "space",
-        "switch.js"
-      ]
-    },
     "hub:registry:add": {
       "aliases": [],
       "args": {
@@ -1534,5 +1534,5 @@
       ]
     }
   },
-  "version": "2.1.2"
+  "version": "2.1.3"
 }

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "byterover-cli",
   "description": "ByteRover's CLI",
-  "version": "2.1.2",
+  "version": "2.1.3",
   "author": "ByteRover",
   "bin": {
     "brv": "./bin/run.js"