bgrun 3.9.0 → 3.10.0

package/dashboard/app/api/stop/[name]/route.ts

@@ -1,8 +1,11 @@
 /**
  * POST /api/stop/:name — Stop a running process
+ * 
+ * Kills the registered PID, then kills anything remaining on the port.
+ * Sets PID to 0 to prevent reconciliation from hijacking unrelated processes.
  */
-import { getProcess } from '../../../../../src/db';
-import { isProcessRunning, terminateProcess } from '../../../../../src/platform';
+import { getProcess, updateProcessPid } from '../../../../../src/db';
+import { isProcessRunning, terminateProcess, getProcessPorts, killProcessOnPort } from '../../../../../src/platform';
 import { measure } from 'measure-fn';
 
 export async function POST(req: Request, { params }: { params: { name: string } }) {
@@ -15,9 +18,24 @@ export async function POST(req: Request, { params }: { params: { name: string }
 
     const running = await isProcessRunning(proc.pid);
     if (!running) {
-        return Response.json({ error: 'Process not running' }, { status: 404 });
+        // Already dead — mark PID as 0 to prevent reconciliation
+        updateProcessPid(name, 0);
+        return Response.json({ success: true, already_stopped: true });
     }
 
+    // Detect ports BEFORE killing so we can clean them up
+    const ports = await getProcessPorts(proc.pid);
+
     await measure(`Stop "${name}" (PID ${proc.pid})`, () => terminateProcess(proc.pid));
+
+    // Also kill anything still on the ports
+    for (const port of ports) {
+        await killProcessOnPort(port);
+    }
+
+    // Mark PID as 0 — prevents reconcileProcessPids from re-attaching
+    // a random matching process as this one
+    updateProcessPid(name, 0);
+
     return Response.json({ success: true });
 }

package/dist/index.js

@@ -25,6 +25,8 @@ function getHomeDir() {
   return os.homedir();
 }
 async function isProcessRunning(pid, command) {
+  if (pid <= 0)
+    return false;
   return plat.measure(`PID ${pid} alive?`, async () => {
     try {
       if (command && (command.includes("docker run") || command.includes("docker-compose up") || command.includes("docker compose up"))) {
@@ -1041,6 +1043,7 @@ async function handleStop(name) {
   for (const port of ports) {
     await killProcessOnPort(port);
   }
+  updateProcessPid(name, 0);
   announce(`Process '${name}' has been stopped (kept in registry).`, "Process Stopped");
 }
 async function handleDeleteAll() {

package/package.json

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

package/src/commands/cleanup.ts

@@ -1,5 +1,5 @@
 
-import { getProcess, removeProcessByName, removeProcess, getAllProcesses, removeAllProcesses } from "../db";
+import { getProcess, removeProcessByName, removeProcess, getAllProcesses, removeAllProcesses, updateProcessPid } from "../db";
 import { isProcessRunning, terminateProcess, getProcessPorts, killProcessOnPort, waitForPortFree } from "../platform";
 import { announce, error } from "../logger";
 import * as fs from "fs";
@@ -82,6 +82,10 @@ export async function handleStop(name: string) {
         await killProcessOnPort(port);
     }
 
+    // Mark PID as 0 — prevents reconcileProcessPids from re-attaching
+    // a random matching process as this one
+    updateProcessPid(name, 0);
+
     announce(`Process '${name}' has been stopped (kept in registry).`, "Process Stopped");
 }
 

package/src/platform.ts

@@ -28,6 +28,9 @@ export function getHomeDir(): string {
  * For Docker containers, checks container status instead of PID
  */
 export async function isProcessRunning(pid: number, command?: string): Promise<boolean> {
+  // PID 0 means intentionally stopped — never alive
+  if (pid <= 0) return false;
+
   return plat.measure(`PID ${pid} alive?`, async () => {
     try {
       // Docker container detection
@@ -318,7 +321,9 @@ export async function reconcileProcessPids(
 ): Promise<Map<string, number>> {
   return await plat.measure('Reconcile PIDs', async () => {
     const result = new Map<string, number>();
-    const needsReconciliation = processes.filter(p => deadPids.has(p.pid));
+    // Skip processes with PID=0 — these were intentionally stopped
+    // and should NOT be reconciled to avoid hijacking unrelated processes
+    const needsReconciliation = processes.filter(p => deadPids.has(p.pid) && p.pid > 0);
     if (needsReconciliation.length === 0) return result;
 
     try {