@pokit/tabs-core 0.0.39 → 0.0.42

package/dist/process-manager.d.ts

@@ -43,6 +43,7 @@ export declare class ProcessManager {
     private options;
     private processes;
     private outputBuffers;
+    private expectedStops;
     private flushScheduled;
     private destroyed;
     constructor(items: TabSpec[], options: ProcessManagerOptions);

package/dist/process-manager.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"process-manager.d.ts","sourceRoot":"","sources":["../src/process-manager.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAGH,OAAO,KAAK,EAAE,SAAS,EAAE,UAAU,EAAE,MAAM,YAAY,CAAC;AAExD,eAAO,MAAM,eAAe,KAAK,CAAC;AAMlC;;;GAGG;AACH,MAAM,MAAM,OAAO,GAAG;IACpB,KAAK,EAAE,MAAM,CAAC;IACd,IAAI,EAAE,MAAM,CAAC;CACd,CAAC;AAEF,MAAM,MAAM,uBAAuB,GAAG;IACpC,2DAA2D;IAC3D,cAAc,EAAE,CAAC,KAAK,EAAE,MAAM,EAAE,KAAK,EAAE,MAAM,EAAE,KAAK,IAAI,CAAC;IACzD,2CAA2C;IAC3C,cAAc,EAAE,CAAC,KAAK,EAAE,MAAM,EAAE,MAAM,EAAE,SAAS,EAAE,QAAQ,CAAC,EAAE,MAAM,KAAK,IAAI,CAAC;IAC9E,gDAAgD;IAChD,OAAO,EAAE,CAAC,KAAK,EAAE,MAAM,EAAE,KAAK,EAAE,MAAM,KAAK,IAAI,CAAC;CACjD,CAAC;AAEF,MAAM,MAAM,qBAAqB,GAAG;IAClC,qCAAqC;IACrC,GAAG,EAAE,MAAM,CAAC;IACZ,yCAAyC;IACzC,GAAG,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,GAAG,SAAS,CAAC,CAAC;IACxC,kCAAkC;IAClC,SAAS,EAAE,uBAAuB,CAAC;CACpC,CAAC;AAUF;;;;;;;GAOG;AACH,qBAAa,cAAc;IACzB,OAAO,CAAC,KAAK,CAAY;IACzB,OAAO,CAAC,OAAO,CAAwB;IACvC,OAAO,CAAC,SAAS,CAA+B;IAChD,OAAO,CAAC,aAAa,CAAwC;IAC7D,OAAO,CAAC,cAAc,CAAS;IAC/B,OAAO,CAAC,SAAS,CAAS;gBAEd,KAAK,EAAE,OAAO,EAAE,EAAE,OAAO,EAAE,qBAAqB;IAM5D;;OAEG;IACH,cAAc,IAAI,UAAU,EAAE;IAU9B;;OAEG;IACH,KAAK,IAAI,IAAI;IAQb;;OAEG;IACH,OAAO,CAAC,KAAK,EAAE,MAAM,GAAG,IAAI;IA2B5B;;OAEG;IACH,IAAI,CAAC,KAAK,EAAE,MAAM,GAAG,IAAI;IAUzB;;OAEG;IACH,OAAO,IAAI,IAAI;IAQf;;OAEG;IACH,OAAO,IAAI,IAAI;IAUf,OAAO,CAAC,YAAY;IAgCpB,OAAO,CAAC,YAAY;IAkBpB,OAAO,CAAC,aAAa;IAMrB,OAAO,CAAC,WAAW;CAcpB"}
+{"version":3,"file":"process-manager.d.ts","sourceRoot":"","sources":["../src/process-manager.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAGH,OAAO,KAAK,EAAE,SAAS,EAAE,UAAU,EAAE,MAAM,YAAY,CAAC;AAWxD,eAAO,MAAM,eAAe,KAAK,CAAC;AAMlC;;;GAGG;AACH,MAAM,MAAM,OAAO,GAAG;IACpB,KAAK,EAAE,MAAM,CAAC;IACd,IAAI,EAAE,MAAM,CAAC;CACd,CAAC;AAEF,MAAM,MAAM,uBAAuB,GAAG;IACpC,2DAA2D;IAC3D,cAAc,EAAE,CAAC,KAAK,EAAE,MAAM,EAAE,KAAK,EAAE,MAAM,EAAE,KAAK,IAAI,CAAC;IACzD,2CAA2C;IAC3C,cAAc,EAAE,CAAC,KAAK,EAAE,MAAM,EAAE,MAAM,EAAE,SAAS,EAAE,QAAQ,CAAC,EAAE,MAAM,KAAK,IAAI,CAAC;IAC9E,gDAAgD;IAChD,OAAO,EAAE,CAAC,KAAK,EAAE,MAAM,EAAE,KAAK,EAAE,MAAM,KAAK,IAAI,CAAC;CACjD,CAAC;AAEF,MAAM,MAAM,qBAAqB,GAAG;IAClC,qCAAqC;IACrC,GAAG,EAAE,MAAM,CAAC;IACZ,yCAAyC;IACzC,GAAG,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,GAAG,SAAS,CAAC,CAAC;IACxC,kCAAkC;IAClC,SAAS,EAAE,uBAAuB,CAAC;CACpC,CAAC;AAUF;;;;;;;GAOG;AACH,qBAAa,cAAc;IACzB,OAAO,CAAC,KAAK,CAAY;IACzB,OAAO,CAAC,OAAO,CAAwB;IACvC,OAAO,CAAC,SAAS,CAA+B;IAChD,OAAO,CAAC,aAAa,CAAwC;IAC7D,OAAO,CAAC,aAAa,CAAqB;IAC1C,OAAO,CAAC,cAAc,CAAS;IAC/B,OAAO,CAAC,SAAS,CAAS;gBAEd,KAAK,EAAE,OAAO,EAAE,EAAE,OAAO,EAAE,qBAAqB;IAM5D;;OAEG;IACH,cAAc,IAAI,UAAU,EAAE;IAU9B;;OAEG;IACH,KAAK,IAAI,IAAI;IAQb;;OAEG;IACH,OAAO,CAAC,KAAK,EAAE,MAAM,GAAG,IAAI;IA4B5B;;OAEG;IACH,IAAI,CAAC,KAAK,EAAE,MAAM,GAAG,IAAI;IAWzB;;OAEG;IACH,OAAO,IAAI,IAAI;IASf;;OAEG;IACH,OAAO,IAAI,IAAI;IAUf,OAAO,CAAC,YAAY;IAwCpB,OAAO,CAAC,YAAY;IAkBpB,OAAO,CAAC,aAAa;IAMrB,OAAO,CAAC,WAAW;CAcpB"}

package/dist/process-manager.js

@@ -5,6 +5,16 @@
  * Framework-agnostic - provides callbacks for UI frameworks to consume.
  */
 import { spawn } from 'node:child_process';
+function killProcessTree(proc) {
+    if (proc.killed || proc.pid == null)
+        return;
+    try {
+        process.kill(-proc.pid, 'SIGTERM');
+    }
+    catch {
+        // Process may have already exited
+    }
+}
 export const OUTPUT_BATCH_MS = 16;
 // =============================================================================
 // ProcessManager Class
@@ -22,6 +32,7 @@ export class ProcessManager {
     options;
     processes = [];
     outputBuffers = new Map();
+    expectedStops = new Set();
     flushScheduled = false;
     destroyed = false;
     constructor(items, options) {
@@ -62,8 +73,9 @@ export class ProcessManager {
             return;
         // Kill existing process
         const existingProc = this.processes[index];
-        if (existingProc && !existingProc.killed) {
-            existingProc.kill('SIGTERM');
+        if (existingProc) {
+            this.expectedStops.add(index);
+            killProcessTree(existingProc);
         }
         // Clear output buffer
         this.outputBuffers.delete(index);
@@ -82,8 +94,9 @@ export class ProcessManager {
      */
     kill(index) {
         const proc = this.processes[index];
-        if (proc && !proc.killed) {
-            proc.kill('SIGTERM');
+        if (proc) {
+            this.expectedStops.add(index);
+            killProcessTree(proc);
         }
         this.options.callbacks.onOutputUpdate(index, ['', 'Stopped']);
         this.options.callbacks.onStatusChange(index, 'stopped');
@@ -92,9 +105,10 @@ export class ProcessManager {
      * Kill all processes
      */
     killAll() {
-        for (const proc of this.processes) {
-            if (proc && !proc.killed) {
-                proc.kill('SIGTERM');
+        for (const [index, proc] of this.processes.entries()) {
+            if (proc) {
+                this.expectedStops.add(index);
+                killProcessTree(proc);
             }
         }
     }
@@ -120,6 +134,7 @@ export class ProcessManager {
                 FORCE_COLOR: '1',
             },
             stdio: ['inherit', 'pipe', 'pipe'],
+            detached: true,
         });
         this.processes[index] = proc;
         const handleData = (data) => this.appendOutput(index, data);
@@ -127,6 +142,11 @@ export class ProcessManager {
         proc.stderr?.on('data', handleData);
         proc.on('close', (code) => {
             this.flushOutput();
+            this.processes[index] = null;
+            // Ignore close transitions for intentional stops (kill/restart/destroy).
+            if (this.expectedStops.delete(index)) {
+                return;
+            }
             const status = code === 0 ? 'done' : 'error';
             this.options.callbacks.onStatusChange(index, status, code ?? undefined);
         });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pokit/tabs-core",
-  "version": "0.0.39",
+  "version": "0.0.42",
   "description": "Core tab management utilities for pok CLI applications",
   "keywords": [
     "cli",
@@ -45,7 +45,7 @@
   },
   "peerDependencies": {
     "react": "^18.0.0 || ^19.0.0",
-    "@pokit/core": "0.0.39"
+    "@pokit/core": "0.0.42"
   },
   "peerDependenciesMeta": {
     "react": {
@@ -56,7 +56,7 @@
     "@types/bun": "latest",
     "@types/react": "^19.2.0",
     "react": "^19.2.0",
-    "@pokit/core": "0.0.39"
+    "@pokit/core": "0.0.42"
   },
   "engines": {
     "bun": ">=1.0.0"

package/src/process-manager.ts

@@ -8,6 +8,15 @@
 import { spawn, type ChildProcess } from 'node:child_process';
 import type { TabStatus, TabProcess } from './types.js';
 
+function killProcessTree(proc: ChildProcess): void {
+  if (proc.killed || proc.pid == null) return;
+  try {
+    process.kill(-proc.pid, 'SIGTERM');
+  } catch {
+    // Process may have already exited
+  }
+}
+
 export const OUTPUT_BATCH_MS = 16;
 
 // =============================================================================
@@ -62,6 +71,7 @@ export class ProcessManager {
   private options: ProcessManagerOptions;
   private processes: (ChildProcess | null)[] = [];
   private outputBuffers: Map<number, OutputBuffer> = new Map();
+  private expectedStops = new Set<number>();
   private flushScheduled = false;
   private destroyed = false;
 
@@ -106,8 +116,9 @@ export class ProcessManager {
 
     // Kill existing process
     const existingProc = this.processes[index];
-    if (existingProc && !existingProc.killed) {
-      existingProc.kill('SIGTERM');
+    if (existingProc) {
+      this.expectedStops.add(index);
+      killProcessTree(existingProc);
     }
 
     // Clear output buffer
@@ -130,8 +141,9 @@ export class ProcessManager {
    */
   kill(index: number): void {
     const proc = this.processes[index];
-    if (proc && !proc.killed) {
-      proc.kill('SIGTERM');
+    if (proc) {
+      this.expectedStops.add(index);
+      killProcessTree(proc);
     }
 
     this.options.callbacks.onOutputUpdate(index, ['', 'Stopped']);
@@ -142,9 +154,10 @@ export class ProcessManager {
    * Kill all processes
    */
   killAll(): void {
-    for (const proc of this.processes) {
-      if (proc && !proc.killed) {
-        proc.kill('SIGTERM');
+    for (const [index, proc] of this.processes.entries()) {
+      if (proc) {
+        this.expectedStops.add(index);
+        killProcessTree(proc);
       }
     }
   }
@@ -173,6 +186,7 @@ export class ProcessManager {
         FORCE_COLOR: '1',
       } as NodeJS.ProcessEnv,
       stdio: ['inherit', 'pipe', 'pipe'],
+      detached: true,
     });
 
     this.processes[index] = proc;
@@ -184,6 +198,13 @@ export class ProcessManager {
 
     proc.on('close', (code) => {
       this.flushOutput();
+      this.processes[index] = null;
+
+      // Ignore close transitions for intentional stops (kill/restart/destroy).
+      if (this.expectedStops.delete(index)) {
+        return;
+      }
+
       const status: TabStatus = code === 0 ? 'done' : 'error';
       this.options.callbacks.onStatusChange(index, status, code ?? undefined);
     });