@tt-a1i/openpi 0.4.0 → 0.6.0

package/extensions/subagents/navigation.ts

@@ -19,6 +19,11 @@ export interface SubagentStripEntry {
   counts: ActivityCounts;
 }
 
+/** Changes only when the strip needs an immediate lifecycle repaint. */
+export function subagentStripEntryKey(entry: SubagentStripEntry | undefined) {
+  return entry ? `${entry.snapshot.id}:${entry.snapshot.status}` : undefined;
+}
+
 function cleanLine(value: string) {
   return sanitizeTerminalText(value).replace(/\s+/g, " ").trim();
 }
@@ -71,9 +76,11 @@ function statusGlyph(snapshot: SubagentSnapshot, theme: Theme, now: number) {
   return theme.fg("error", "✗");
 }
 
+const SUBAGENT_STRIP_INTERVAL_MS = 500;
+
 /** One-line subagent manager entry with the same affordance as Workflow. */
 export class SubagentStripWidget {
-  private readonly timer: ReturnType<typeof setInterval>;
+  private timer: ReturnType<typeof setInterval> | undefined;
   private readonly tui: TUI;
   private readonly theme: Theme;
   private readonly strip: BelowEditorStripState;
@@ -89,17 +96,39 @@ export class SubagentStripWidget {
     this.theme = theme;
     this.strip = strip;
     this.getEntry = getEntry;
-    this.timer = setInterval(() => this.tui.requestRender(), 500);
-    this.timer.unref?.();
+    this.syncSpinner();
   }
 
   dispose() {
-    clearInterval(this.timer);
+    if (this.timer) clearInterval(this.timer);
+    this.timer = undefined;
   }
 
-  invalidate() {}
+  invalidate() {
+    this.syncSpinner();
+  }
+
+  private syncSpinner() {
+    if (this.getEntry()?.snapshot.status === "running") {
+      if (this.timer) return;
+      const timer = setInterval(() => {
+        if (this.timer !== timer) return;
+        if (this.getEntry()?.snapshot.status !== "running") {
+          clearInterval(timer);
+          this.timer = undefined;
+        }
+        this.tui.requestRender();
+      }, SUBAGENT_STRIP_INTERVAL_MS);
+      this.timer = timer;
+      timer.unref?.();
+      return;
+    }
+    if (this.timer) clearInterval(this.timer);
+    this.timer = undefined;
+  }
 
   render(width: number) {
+    this.syncSpinner();
     const entry = this.getEntry();
     if (!entry || width <= 0) return [];
     const { snapshot, counts } = entry;

package/extensions/subagents/src/backend.ts

@@ -5,11 +5,11 @@
  * on it rather than on the pi session directly.
  *
  * - pi (./backends/pi.ts): in-process `createAgentSession()` via the pi SDK.
- * - stub (./backends/stub.ts): scripted sessions, test-only.
  */
 
 import type { Effect, Scope, Stream } from "effect";
 import { Context } from "effect";
+import type { AgentToolRenderer } from "../../shared/agent-tool-renderer.ts";
 import type {
   BackendName,
   SendError,
@@ -19,6 +19,13 @@ import type {
   SubagentMeta,
 } from "./domain.ts";
 
+export interface SubagentCleanupReceipt {
+  /** True when the session/worktree state could not be proven quiescent. */
+  readonly uncertain: boolean;
+  /** Bounded operator-facing explanation of the cleanup result. */
+  readonly message: string;
+}
+
 /**
  * A live subagent session. The manager is the single consumer of `events`;
  * it folds them into the `SubagentSnapshot` everything else reads.
@@ -42,6 +49,10 @@ export interface SubagentSession {
    * this with a timeout and fall back to closing the session scope.
    */
   readonly interrupt: Effect.Effect<void>;
+  /** Ephemeral Pi-native tool projection for the operator-facing child page. */
+  readonly toolRenderer?: AgentToolRenderer;
+  /** Cleanup receipt populated when scope teardown cannot prove safety. */
+  readonly cleanupReceipt?: () => SubagentCleanupReceipt | undefined;
 }
 
 export interface SubagentBackend {