bgrun 3.12.2 → 3.12.3

package/dist/index.js

@@ -25,6 +25,7 @@ __export(exports_platform, {
   reconcileProcessPids: () => reconcileProcessPids,
   readFileTail: () => readFileTail,
   psExec: () => psExec,
+  parseUnixListeningPorts: () => parseUnixListeningPorts,
   killProcessOnPort: () => killProcessOnPort,
   isWindows: () => isWindows,
   isProcessRunning: () => isProcessRunning,
@@ -457,6 +458,17 @@ async function getProcessBatchResources(pids) {
     return resourceMap;
   }) ?? new Map;
 }
+function parseUnixListeningPorts(output) {
+  const ports = new Set;
+  for (const line of output.split(`
+`)) {
+    const portMatch = line.match(/:(\d+)\s+\(LISTEN\)/);
+    if (portMatch) {
+      ports.add(parseInt(portMatch[1]));
+    }
+  }
+  return Array.from(ports);
+}
 async function getProcessPorts(pid) {
   try {
     if (isWindows()) {
@@ -473,29 +485,21 @@ async function getProcessPorts(pid) {
     } else {
       try {
         const result2 = await $`ss -tlnp`.nothrow().quiet().text();
-        const ports2 = new Set;
+        const ports = new Set;
         for (const line of result2.split(`
 `)) {
           if (line.includes(`pid=${pid}`)) {
             const portMatch = line.match(/:(\d+)\s/);
             if (portMatch) {
-              ports2.add(parseInt(portMatch[1]));
+              ports.add(parseInt(portMatch[1]));
             }
           }
         }
-        if (ports2.size > 0)
-          return Array.from(ports2);
+        if (ports.size > 0)
+          return Array.from(ports);
       } catch {}
       const result = await $`lsof -Pan -p ${pid} -iTCP -sTCP:LISTEN`.nothrow().quiet().text();
-      const ports = new Set;
-      for (const line of result.split(`
-`)) {
-        const portMatch = line.match(/:(\d+)\s+\(LISTEN\)/);
-        if (portMatch) {
-          ports.add(parseInt(portMatch[1]));
-        }
-      }
-      return Array.from(ports);
+      return parseUnixListeningPorts(result);
     }
   } catch {
     return [];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bgrun",
-  "version": "3.12.2",
+  "version": "3.12.3",
   "description": "bgrun — A lightweight process manager for Bun",
   "type": "module",
   "main": "./src/api.ts",

package/src/bgrun.test.ts

@@ -10,7 +10,7 @@ import { describe, expect, test } from 'bun:test'
 import { parseEnvString, calculateRuntime } from './utils'
 import { stripAnsi, truncateString, truncatePath } from './table'
 import { detectPackageManager, formatDeployToolError } from './deploy'
-import { isProcessRunning } from './platform'
+import { isProcessRunning, parseUnixListeningPorts } from './platform'
 import { mkdirSync, rmSync } from 'fs'
 
 // Use a test-specific database to avoid polluting real data
@@ -141,6 +141,39 @@ describe('isProcessRunning', () => {
     })
 })
 
+describe('parseUnixListeningPorts', () => {
+    test('extracts only LISTEN ports from lsof output', () => {
+        const output = [
+            'COMMAND   PID USER   FD   TYPE DEVICE SIZE/OFF NODE NAME',
+            'bun     12345 root   21u  IPv4 123456      0t0  TCP *:3400 (LISTEN)',
+            'bun     12345 root   22u  IPv4 123457      0t0  TCP 127.0.0.1:9222 (LISTEN)',
+        ].join('\n')
+
+        expect(parseUnixListeningPorts(output)).toEqual([3400, 9222])
+    })
+
+    test('ignores non-LISTEN sockets from broad lsof output', () => {
+        const output = [
+            'COMMAND   PID USER   FD   TYPE DEVICE SIZE/OFF NODE NAME',
+            'bun     12345 root   18u  IPv4 111111      0t0  TCP 127.0.0.1:49440->127.0.0.1:3000 (ESTABLISHED)',
+            'bun     12345 root   19u  IPv4 111112      0t0  TCP 127.0.0.1:49441->127.0.0.1:3737 (ESTABLISHED)',
+            'bun     12345 root   20u  IPv4 111113      0t0  TCP *:3400 (LISTEN)',
+        ].join('\n')
+
+        expect(parseUnixListeningPorts(output)).toEqual([3400])
+    })
+
+    test('returns empty array for no-port worker output', () => {
+        const output = [
+            'COMMAND   PID USER   FD   TYPE DEVICE SIZE/OFF NODE NAME',
+            'bun     12345 root   18u  unix 0xffff      0t0      /tmp/bun.sock',
+            'bun     12345 root   19u  IPv4 111111      0t0  TCP 127.0.0.1:49440->127.0.0.1:3000 (ESTABLISHED)',
+        ].join('\n')
+
+        expect(parseUnixListeningPorts(output)).toEqual([])
+    })
+})
+
 // ─── detectPackageManager ───────────────────────────────
 
 describe('formatDeployToolError', () => {
@@ -217,64 +250,64 @@ describe('detectPackageManager', () => {
         }
     })
 })
-
-// ─── Dependencies ───────────────────────────────────────
-
-describe('addDependency', () => {
-    test('adds a valid dependency', () => {
-        removeAllDependencies('web-server');
-        removeAllDependencies('database');
-        const ok = addDependency('web-server', 'database');
-        expect(ok).toBe(true);
-        expect(getDependencies('web-server')).toContain('database');
-    })
-
-    test('prevents self-dependency', () => {
-        expect(addDependency('api', 'api')).toBe(false);
-    })
-
-    test('prevents duplicate dependency', () => {
-        removeAllDependencies('app');
-        addDependency('app', 'db');
-        expect(addDependency('app', 'db')).toBe(false);
-    })
-
-    test('prevents circular dependency', () => {
-        removeAllDependencies('a');
-        removeAllDependencies('b');
-        removeAllDependencies('c');
-        addDependency('a', 'b');
-        addDependency('b', 'c');
-        // c -> a would create a cycle
-        expect(addDependency('c', 'a')).toBe(false);
-    })
-})
-
-describe('getDependencyGraph', () => {
-    test('returns full graph', () => {
-        removeAllDependencies('svc-a');
-        removeAllDependencies('svc-b');
-        addDependency('svc-a', 'svc-b');
-        const graph = getDependencyGraph();
-        expect(graph['svc-a']).toContain('svc-b');
-    })
-})
-
-describe('getDependents', () => {
-    test('finds processes that depend on a target', () => {
-        removeAllDependencies('frontend');
-        removeAllDependencies('backend');
-        addDependency('frontend', 'backend');
-        expect(getDependents('backend')).toContain('frontend');
-    })
-})
-
-describe('removeDependency', () => {
-    test('removes an existing dependency', () => {
-        removeAllDependencies('x');
-        addDependency('x', 'y');
-        expect(getDependencies('x')).toContain('y');
-        removeDependency('x', 'y');
-        expect(getDependencies('x')).not.toContain('y');
-    })
-})
+
+// ─── Dependencies ───────────────────────────────────────
+
+describe('addDependency', () => {
+    test('adds a valid dependency', () => {
+        removeAllDependencies('web-server');
+        removeAllDependencies('database');
+        const ok = addDependency('web-server', 'database');
+        expect(ok).toBe(true);
+        expect(getDependencies('web-server')).toContain('database');
+    })
+
+    test('prevents self-dependency', () => {
+        expect(addDependency('api', 'api')).toBe(false);
+    })
+
+    test('prevents duplicate dependency', () => {
+        removeAllDependencies('app');
+        addDependency('app', 'db');
+        expect(addDependency('app', 'db')).toBe(false);
+    })
+
+    test('prevents circular dependency', () => {
+        removeAllDependencies('a');
+        removeAllDependencies('b');
+        removeAllDependencies('c');
+        addDependency('a', 'b');
+        addDependency('b', 'c');
+        // c -> a would create a cycle
+        expect(addDependency('c', 'a')).toBe(false);
+    })
+})
+
+describe('getDependencyGraph', () => {
+    test('returns full graph', () => {
+        removeAllDependencies('svc-a');
+        removeAllDependencies('svc-b');
+        addDependency('svc-a', 'svc-b');
+        const graph = getDependencyGraph();
+        expect(graph['svc-a']).toContain('svc-b');
+    })
+})
+
+describe('getDependents', () => {
+    test('finds processes that depend on a target', () => {
+        removeAllDependencies('frontend');
+        removeAllDependencies('backend');
+        addDependency('frontend', 'backend');
+        expect(getDependents('backend')).toContain('frontend');
+    })
+})
+
+describe('removeDependency', () => {
+    test('removes an existing dependency', () => {
+        removeAllDependencies('x');
+        addDependency('x', 'y');
+        expect(getDependencies('x')).toContain('y');
+        removeDependency('x', 'y');
+        expect(getDependencies('x')).not.toContain('y');
+    })
+})

package/src/platform.ts

@@ -608,6 +608,20 @@ export async function getProcessBatchResources(pids: number[]): Promise<Map<numb
   }) ?? new Map();
 }
 
+/**
+ * Parse Unix lsof LISTEN output and return only true listening TCP ports.
+ */
+export function parseUnixListeningPorts(output: string): number[] {
+  const ports = new Set<number>();
+  for (const line of output.split('\n')) {
+    const portMatch = line.match(/:(\d+)\s+\(LISTEN\)/);
+    if (portMatch) {
+      ports.add(parseInt(portMatch[1]));
+    }
+  }
+  return Array.from(ports);
+}
+
 /**
  * Get the TCP ports a process is currently listening on by querying the OS.
  * Returns an array of port numbers (empty if none or process not found).
@@ -643,14 +657,7 @@ export async function getProcessPorts(pid: number): Promise<number[]> {
       } catch { /* ss not available, try lsof */ }
 
       const result = await $`lsof -Pan -p ${pid} -iTCP -sTCP:LISTEN`.nothrow().quiet().text();
-      const ports = new Set<number>();
-      for (const line of result.split('\n')) {
-        const portMatch = line.match(/:(\d+)\s+\(LISTEN\)/);
-        if (portMatch) {
-          ports.add(parseInt(portMatch[1]));
-        }
-      }
-      return Array.from(ports);
+      return parseUnixListeningPorts(result);
     }
   } catch {
     return [];