@aliou/pi-processes 0.2.2 → 0.3.1

package/hooks/process-end.ts

@@ -37,11 +37,13 @@ export function setupProcessEndHook(pi: ExtensionAPI, manager: ProcessManager) {
 
     const info: ProcessInfo = event.info;
 
-    // Check notification preferences
-    const shouldNotify =
-      (info.status === "killed" && info.notifyOnKill) ||
-      (info.status === "exited" && info.success && info.notifyOnSuccess) ||
-      (info.status === "exited" && !info.success && info.notifyOnFailure);
+    // Determine if the agent should get a turn to react to this process ending.
+    // When true, the agent receives the message in its context and can respond
+    // (e.g. check results, fix code, restart the process).
+    const triggerAgentTurn =
+      (info.status === "killed" && info.alertOnKill) ||
+      (info.status === "exited" && info.success && info.alertOnSuccess) ||
+      (info.status === "exited" && !info.success && info.alertOnFailure);
 
     const runtime = formatRuntime(info.startTime, info.endTime);
 
@@ -65,27 +67,26 @@ export function setupProcessEndHook(pi: ExtensionAPI, manager: ProcessManager) {
       latestContext.ui.notify(message, level);
     }
 
-    // Only send message to agent if notification preferences allow
-    if (shouldNotify) {
-      const details: ProcessUpdateDetails = {
-        processId: info.id,
-        processName: info.name,
-        command: info.command,
-        status: info.status as "exited" | "killed",
-        exitCode: info.exitCode,
-        success: info.success ?? false,
-        runtime,
-      };
+    // Always send the message so it appears in the conversation history.
+    // Only trigger an agent turn when the notification preferences say so.
+    const details: ProcessUpdateDetails = {
+      processId: info.id,
+      processName: info.name,
+      command: info.command,
+      status: info.status as "exited" | "killed",
+      exitCode: info.exitCode,
+      success: info.success ?? false,
+      runtime,
+    };
 
-      pi.sendMessage(
-        {
-          customType: MESSAGE_TYPE_PROCESS_UPDATE,
-          content: message,
-          display: true,
-          details,
-        },
-        { triggerTurn: false },
-      );
-    }
+    pi.sendMessage(
+      {
+        customType: MESSAGE_TYPE_PROCESS_UPDATE,
+        content: message,
+        display: true,
+        details,
+      },
+      { triggerTurn: triggerAgentTurn },
+    );
   });
 }

package/manager.ts

@@ -148,9 +148,9 @@ export class ProcessManager {
       success: null,
       stdoutFile,
       stderrFile,
-      notifyOnSuccess: options?.notifyOnSuccess ?? false,
-      notifyOnFailure: options?.notifyOnFailure ?? true,
-      notifyOnKill: options?.notifyOnKill ?? false,
+      alertOnSuccess: options?.alertOnSuccess ?? false,
+      alertOnFailure: options?.alertOnFailure ?? true,
+      alertOnKill: options?.alertOnKill ?? false,
       process: child,
       lastSignalSent: null,
     };
@@ -306,9 +306,9 @@ export class ProcessManager {
           success: false,
           stdoutFile: "",
           stderrFile: "",
-          notifyOnSuccess: false,
-          notifyOnFailure: true,
-          notifyOnKill: false,
+          alertOnSuccess: false,
+          alertOnFailure: true,
+          alertOnKill: false,
         },
         reason: "not_found",
       };
@@ -317,7 +317,7 @@ export class ProcessManager {
     const signal = opts?.signal ?? "SIGTERM";
     const timeoutMs = opts?.timeoutMs ?? 3000;
 
-    managed.notifyOnKill = false;
+    managed.alertOnKill = false;
 
     if (!LIVE_STATUSES.has(managed.status)) {
       return { ok: true, info: this.toProcessInfo(managed) };
@@ -468,9 +468,9 @@ export class ProcessManager {
       success: managed.success,
       stdoutFile: managed.stdoutFile,
       stderrFile: managed.stderrFile,
-      notifyOnSuccess: managed.notifyOnSuccess,
-      notifyOnFailure: managed.notifyOnFailure,
-      notifyOnKill: managed.notifyOnKill,
+      alertOnSuccess: managed.alertOnSuccess,
+      alertOnFailure: managed.alertOnFailure,
+      alertOnKill: managed.alertOnKill,
     };
   }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aliou/pi-processes",
-  "version": "0.2.2",
+  "version": "0.3.1",
   "type": "module",
   "private": false,
   "keywords": [
@@ -32,8 +32,8 @@
     "@sinclair/typebox": "^0.34.41"
   },
   "peerDependencies": {
-    "@mariozechner/pi-ai": "0.50.0",
-    "@mariozechner/pi-coding-agent": "0.50.0",
-    "@mariozechner/pi-tui": "0.50.0"
+    "@mariozechner/pi-ai": "0.51.0",
+    "@mariozechner/pi-coding-agent": "0.51.0",
+    "@mariozechner/pi-tui": "0.51.0"
   }
 }

package/tools/actions/index.ts

@@ -13,9 +13,9 @@ interface ActionParams {
   command?: string;
   name?: string;
   id?: string;
-  notifyOnSuccess?: boolean;
-  notifyOnFailure?: boolean;
-  notifyOnKill?: boolean;
+  alertOnSuccess?: boolean;
+  alertOnFailure?: boolean;
+  alertOnKill?: boolean;
 }
 
 export async function executeAction(

package/tools/actions/start.ts

@@ -5,9 +5,9 @@ import type { ProcessManager } from "../../manager";
 interface StartParams {
   name?: string;
   command?: string;
-  notifyOnSuccess?: boolean;
-  notifyOnFailure?: boolean;
-  notifyOnKill?: boolean;
+  alertOnSuccess?: boolean;
+  alertOnFailure?: boolean;
+  alertOnKill?: boolean;
 }
 
 export function executeStart(
@@ -37,9 +37,9 @@ export function executeStart(
   }
 
   const proc = manager.start(params.name, params.command, ctx.cwd, {
-    notifyOnSuccess: params.notifyOnSuccess,
-    notifyOnFailure: params.notifyOnFailure,
-    notifyOnKill: params.notifyOnKill,
+    alertOnSuccess: params.alertOnSuccess,
+    alertOnFailure: params.alertOnFailure,
+    alertOnKill: params.alertOnKill,
   });
 
   const message = `Started "${proc.name}" (${proc.id}, PID: ${proc.pid})\nLogs: ${proc.stdoutFile}`;

package/tools/index.ts

@@ -35,22 +35,22 @@ const ProcessesParams = Type.Object({
         "Process ID or name to match (required for output/kill/logs). Can be proc_N or friendly name.",
     }),
   ),
-  notifyOnSuccess: Type.Optional(
+  alertOnSuccess: Type.Optional(
     Type.Boolean({
       description:
-        "Notify when process completes successfully (default: false). Use for builds/tests where you need confirmation.",
+        "Get a turn to react when process completes successfully (default: false). Use for builds/tests where you need confirmation.",
     }),
   ),
-  notifyOnFailure: Type.Optional(
+  alertOnFailure: Type.Optional(
     Type.Boolean({
       description:
-        "Notify when process fails/crashes (default: true). Use to be alerted of unexpected failures.",
+        "Get a turn to react when process fails/crashes (default: true). Use to be alerted of unexpected failures.",
     }),
   ),
-  notifyOnKill: Type.Optional(
+  alertOnKill: Type.Optional(
     Type.Boolean({
       description:
-        "Notify when process is killed by external signal (default: false). Note: killing via tool never notifies.",
+        "Get a turn to react when process is killed by external signal (default: false). Note: killing via tool never triggers a turn.",
     }),
   ),
 });
@@ -63,9 +63,9 @@ export function setupProcessesTools(pi: ExtensionAPI, manager: ProcessManager) {
     label: "Processes",
     description: `Manage background processes. Actions:
 - start: Run command in background (requires 'name' and 'command')
-  - notifyOnSuccess (default: false): Get notified when process completes successfully
-  - notifyOnFailure (default: true): Get notified when process crashes/fails
-  - notifyOnKill (default: false): Get notified if killed by external signal (killing via tool never notifies)
+  - alertOnSuccess (default: false): Get a turn to react when process completes successfully
+  - alertOnFailure (default: true): Get a turn to react when process crashes/fails
+  - alertOnKill (default: false): Get a turn to react if killed by external signal (killing via tool never triggers a turn)
 - list: Show all managed processes with their IDs and names
 - output: Get recent stdout/stderr (requires 'id' - can be proc_N or name match)
 - logs: Get log file paths to inspect with read tool (requires 'id')
@@ -74,11 +74,11 @@ export function setupProcessesTools(pi: ExtensionAPI, manager: ProcessManager) {
 
 Important: You DON'T need to poll or wait for processes. Notifications arrive automatically based on your preferences. Start processes and continue with other work - you'll be informed if something requires attention.
 
-Note: User always sees notifications in UI. Notification preferences only control whether YOU (the agent) are informed.`,
+Note: User always sees process updates in the UI. The notify flags control whether YOU (the agent) get a turn to react (e.g. check results, fix code, restart).`,
 
     parameters: ProcessesParams,
 
-    async execute(_toolCallId, params, _onUpdate, ctx) {
+    async execute(_toolCallId, params, _signal, _onUpdate, ctx) {
       return executeAction(params, manager, ctx);
     },