@aliou/pi-processes 0.10.0 → 0.10.1

package/extensions/processes/config/migrations/002-stamp-v0-10-0-config-version.ts

@@ -17,5 +17,5 @@ export const configVersionStampMigration: Migration<ProcessConfig> = {
   shouldRun: needsConfigVersionStamp,
   run: (config) => stampConfigVersion(config),
   message:
-    "Updated pi-processes settings for v0.10.0. This release rewrites the package into separate process, logs, and dock extensions while preserving your existing settings. Release notes will be published at https://github.com/aliou/pi-processes/releases/tag/v0.10.0.",
+    "Updated pi-processes settings for v0.10.0. This release rewrites the package into separate process, logs, and dock extensions while preserving your existing settings. Release notes are available at https://github.com/aliou/pi-processes/releases/tag/v0.10.0.",
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aliou/pi-processes",
-  "version": "0.10.0",
+  "version": "0.10.1",
   "license": "MIT",
   "type": "module",
   "private": false,
@@ -68,12 +68,6 @@
     "typescript": "^5.9.3",
     "vitest": "^4.1.5"
   },
-  "pnpm": {
-    "overrides": {
-      "@earendil-works/pi-ai": "$@earendil-works/pi-coding-agent",
-      "@earendil-works/pi-tui": "$@earendil-works/pi-coding-agent"
-    }
-  },
   "scripts": {
     "typecheck": "tsc --noEmit",
     "lint": "biome check",
@@ -88,7 +82,7 @@
     "test:e2e": "vitest run --config vitest.e2e.config.ts",
     "release": "pnpm changeset publish"
   },
-  "packageManager": "pnpm@10.26.1",
+  "packageManager": "pnpm@11.5.1",
   "peerDependenciesMeta": {
     "@earendil-works/pi-ai": {
       "optional": true

package/src/manager/process-runtime-controller.ts

@@ -83,6 +83,11 @@ export class ProcessRuntimeController {
     this.registry.add(managed);
 
     if (!child.pid) {
+      // No pid means the process never started. The async spawn `error`
+      // event may not have been delivered yet (Node emits it on a later
+      // turn of the event loop, and it cannot fire while start() runs),
+      // so attach a handler that finalizes the record with the real
+      // reason when it arrives.
       this.logs.appendErrorLine(managed.stderrFile, "Spawn error: missing pid");
       managed.exitCode = -1;
       managed.success = false;
@@ -91,6 +96,14 @@ export class ProcessRuntimeController {
       managed.endTime = Date.now();
       this.releaseRuntimeHandles(managed);
       this.transition(managed, "exited");
+      child.on("error", (err) => {
+        this.logs.appendErrorLine(
+          managed.stderrFile,
+          `Process error: ${err.message}`,
+        );
+        managed.endReason = "spawn_error";
+        managed.errorMessage = err.message;
+      });
       return managed;
     }
 
@@ -361,24 +374,31 @@ export class ProcessRuntimeController {
     });
 
     child.on("error", (err) => {
-      this.logs.appendErrorLine(
-        managed.stderrFile,
-        `Process error: ${err.message}`,
-      );
-
-      if (!managed.endTime) {
-        this.releaseRuntimeHandles(managed);
-        managed.exitCode = -1;
-        managed.success = false;
-        managed.endReason = "spawn_error";
-        managed.errorMessage = err.message;
-        managed.endTime = Date.now();
-        this.output.flush(managed);
-        this.transition(managed, "exited");
-      }
+      this.handleSpawnError(managed, err);
     });
   }
 
+  private handleSpawnError(
+    managed: ManagedProcessRecord,
+    err: NodeJS.ErrnoException,
+  ): void {
+    this.logs.appendErrorLine(
+      managed.stderrFile,
+      `Process error: ${err.message}`,
+    );
+
+    if (managed.endTime) return;
+
+    this.releaseRuntimeHandles(managed);
+    managed.exitCode = -1;
+    managed.success = false;
+    managed.endReason = "spawn_error";
+    managed.errorMessage = err.message;
+    managed.endTime = Date.now();
+    this.output.flush(managed);
+    this.transition(managed, "exited");
+  }
+
   private ensureWatcherRunning(): void {
     if (this.watcher) return;
     if (!this.registry.hasAliveishProcesses()) return;

package/src/utils/process-group.ts

@@ -3,6 +3,7 @@
  * Uses signal 0 to test existence without actually sending a signal.
  */
 export function isProcessGroupAlive(pgid: number): boolean {
+  if (!Number.isInteger(pgid) || pgid <= 0) return false;
   try {
     process.kill(-pgid, 0);
     return true;
@@ -18,5 +19,8 @@ export function isProcessGroupAlive(pgid: number): boolean {
  * Negative PID targets the process group.
  */
 export function killProcessGroup(pgid: number, signal: NodeJS.Signals): void {
+  if (!Number.isInteger(pgid) || pgid <= 0) {
+    throw new RangeError("Process group ID must be a positive integer");
+  }
   process.kill(-pgid, signal);
 }