@prohost/cli 0.16.10 → 0.16.11

package/CHANGELOG.md

@@ -6,6 +6,15 @@ builds via `prepack`, runs the suite, publishes, and tags `cli-v<version>`.
 
 ## Unreleased
 
+## 0.16.11
+
+- **An end that can't check what is left of a terminal is retried, never said done.** When a retry
+  of `code_thread.end_requested` (after a kill whose tmux answer was lost) couldn't ask `ps` which
+  saved panes still belong to the terminal, it reported the end done with nothing reaped, so a
+  process group that outlived its window could keep running. Now what was just closed is reaped,
+  the saved panes stay in the unfinished `.ended` mark, and the end is asked for again until `ps`
+  answers.
+
 ## 0.16.10
 
 - **A Code thread's terminal can be ended from ProhostAI.** `code_thread.end_requested` closes the

package/dist/codehost/terminals.d.ts

@@ -544,8 +544,8 @@ export declare class CodeTerminals {
      * Of panes an earlier attempt saved, those whose process group is still this terminal's. A group
      * outlives its leader, and a pid is never handed out while a group of that id exists: so one whose
      * leader pid runs no process now is ours if anything is left in it. One whose pid runs a process is
-     * ours only if that process started when the saved one did; one with no saved start, or when `ps`
-     * can't say, is left alone.
+     * ours only if that process started when the saved one did; one with no saved start is left alone.
+     * `undefined` when `ps` couldn't say: nothing is judged, and the caller keeps them for a retry.
      */
     private stillOurs;
     restart(request: RestartRequest, outcome?: RestartOutcome, hooks?: RestartHooks): Promise<boolean>;

package/dist/codehost/terminals.js

@@ -856,7 +856,22 @@ export class CodeTerminals {
         this.records.delete(request.terminalId);
         this.panePids.delete(request.terminalId);
         this.lastInterrupt.delete(request.terminalId);
-        await this.reapGroups(fresh.map((entry) => entry.pid), await this.stillOurs(saved));
+        const ours = await this.stillOurs(saved);
+        if (ours === undefined) {
+            // `ps` couldn't say which saved panes are still this terminal's: what was just closed is reaped,
+            // the rest stays saved in the unfinished mark, and the end is asked for again rather than said done.
+            await this.reapGroups(fresh.map((entry) => entry.pid));
+            try {
+                this.markEnded(request, saved, false);
+            }
+            catch {
+                /* the earlier mark still holds them */
+            }
+            this.log(`    ! code thread ${short}: couldn't check what is left of its terminal; retrying`);
+            outcome.retry = true;
+            return false;
+        }
+        await this.reapGroups(fresh.map((entry) => entry.pid), ours);
         try {
             // Finished and reaped: nothing left for a retry to signal, and a late redelivery is answered ended.
             this.markEnded(request, [], true);
@@ -899,8 +914,8 @@ export class CodeTerminals {
      * Of panes an earlier attempt saved, those whose process group is still this terminal's. A group
      * outlives its leader, and a pid is never handed out while a group of that id exists: so one whose
      * leader pid runs no process now is ours if anything is left in it. One whose pid runs a process is
-     * ours only if that process started when the saved one did; one with no saved start, or when `ps`
-     * can't say, is left alone.
+     * ours only if that process started when the saved one did; one with no saved start is left alone.
+     * `undefined` when `ps` couldn't say: nothing is judged, and the caller keeps them for a retry.
      */
     async stillOurs(saved) {
         if (saved.length === 0)
@@ -910,7 +925,7 @@ export class CodeTerminals {
             starts = await (this.options.paneStarts ?? paneStarts)(saved.map((entry) => entry.pid));
         }
         catch {
-            return [];
+            return undefined;
         }
         return saved
             .filter((entry) => {

package/dist/version.d.ts

@@ -7,6 +7,6 @@
  * `package.json`, so a release bump that forgets this file fails the suite
  * instead of shipping a `User-Agent` that lies about which build is calling.
  */
-export declare const CLI_VERSION = "0.16.10";
+export declare const CLI_VERSION = "0.16.11";
 /** Sent on every HTTP request the CLI makes back into ProhostAI. */
-export declare const USER_AGENT = "prohost-cli/0.16.10";
+export declare const USER_AGENT = "prohost-cli/0.16.11";

package/dist/version.js

@@ -7,6 +7,6 @@
  * `package.json`, so a release bump that forgets this file fails the suite
  * instead of shipping a `User-Agent` that lies about which build is calling.
  */
-export const CLI_VERSION = '0.16.10';
+export const CLI_VERSION = '0.16.11';
 /** Sent on every HTTP request the CLI makes back into ProhostAI. */
 export const USER_AGENT = `prohost-cli/${CLI_VERSION}`;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@prohost/cli",
-  "version": "0.16.10",
+  "version": "0.16.11",
   "description": "Run your own AI agent as a ProhostAI teammate, and stream your account's webhooks to your laptop.",
   "type": "module",
   "license": "MIT",