pi-better-harness 0.21.0 → 0.22.0

package/README.md

@@ -64,14 +64,20 @@ need interactive confirmation. Full policy: [pi-better-sandbox](https://github.c
 
 ## Minimal Tool Output
 
-The bundled harness also provides `/tool-output minimal`, `/tool-output normal`,
-and `/tool-output` (toggle). Minimal mode folds each tool call into a single-line
-header, including running calls. Long headers are truncated to the terminal
-width; result bodies, images, boxes, and tool spacers are hidden. Built-in,
-extension, and MCP calls are included. Execution, sandboxing, and agent-facing
-payloads are unchanged. Ctrl+O reveals the original tool blocks; Ctrl+O again
-folds them. `/tool-output normal` restores ordinary rendering. Error result
-bodies are also hidden in minimal mode and remain available when expanded.
+The bundled harness provides **Tool output: Normal / Minimal** in
+`/harness-settings`. `/tool-output minimal`, `/tool-output normal`, and
+`/tool-output` (toggle) remain compatibility shortcuts to the same preference.
+Minimal mode folds each tool call into a single-line header, including running
+calls. Muted tool names and quieter inline command/path context distinguish
+activity from conversation text; running and failed calls retain visible state
+markers. Long headers are truncated to the terminal width; result bodies,
+images, boxes, and tool spacers are hidden. Built-in, extension, and MCP calls
+are included. Execution, sandboxing, and agent-facing payloads are unchanged.
+In newer Pi fullscreen mode, click a folded row to expand its original details,
+then click the expanded call/result to collapse it. Regular terminals keep mouse
+input for terminal selection and scrollback; Ctrl+O expands tools in all modes.
+`/tool-output normal` restores ordinary rendering. Error result bodies are also
+hidden in minimal mode and remain available when expanded.
 
 Normal mode is the default. The preference is saved in the current session,
 including resume/reload. This is a version-sensitive internal TUI adapter,
@@ -94,7 +100,9 @@ packages without duplicating their configuration. `/sandbox`, `/subagents
 settings`, and `/goal settings` remain available in standalone installations.
 Packages without a settings screen are not listed. Pi's `/settings` is unchanged.
 
-The hub also owns **Completions while busy**, shared by Subagents and Background
+The hub includes **Tool output** when the bundled renderer extension is loaded.
+Changes apply immediately to the current session and do not save a global
+default. The hub also owns **Completions while busy**, shared by Subagents and Background
 Tasks. Choose **Wait until idle** (the default) or **Steer active run**. Changes
 apply immediately and autosave to the current session branch, including across
 reloads. Press **Ctrl+S** to save the current choice as your default for future

package/extensions/minimal-output/hook.ts

@@ -4,12 +4,15 @@ import { createRequire } from "node:module";
 import { dirname, join } from "node:path";
 import { pathToFileURL } from "node:url";
 import { stripVTControlCharacters } from "node:util";
+import type { Theme } from "@earendil-works/pi-coding-agent";
 
 interface ToolComponent {
   toolName: string;
   args: unknown;
   callRendererComponent?: Component;
   expanded: boolean;
+  isPartial?: boolean;
+  result?: { isError?: boolean };
   showImages: boolean;
   resultRendererComponent?: unknown;
   ui: { requestRender(): void };
@@ -19,7 +22,10 @@ interface ToolComponent {
 }
 
 type Method = (this: ToolComponent, ...args: unknown[]) => unknown;
+interface ToolMouseEvent { type: string; button: string; y: number; shift?: boolean; alt?: boolean; ctrl?: boolean }
+type MouseHandler = (this: ToolComponent, event: ToolMouseEvent) => unknown;
 interface ToolPrototype {
+  handleMouse?: MouseHandler;
   updateDisplay: Method;
   getResultRenderer: Method;
   getTextOutput: Method;
@@ -42,7 +48,7 @@ export interface MinimalOutputHook {
 const HOOK = Symbol.for("pi-better-harness.minimal-output-hook");
 const emptyResult = () => ({ render: () => [] as string[], invalidate() {} });
 
-function compactCall(component: ToolComponent, width: number): string[] {
+function compactCall(component: ToolComponent, width: number, theme?: Theme): string[] {
   if (width <= 0) return [];
   // Render the call alone at a wider width so terminal wrapping becomes truncation.
   const header = component.callRendererComponent?.render(Math.max(4096, width + 1))
@@ -50,10 +56,24 @@ function compactCall(component: ToolComponent, width: number): string[] {
     .find((line) => line.length > 0);
   let text = header;
   if (!text) {
-    const args = JSON.stringify(component.args);
-    text = `${component.toolName}${args && args !== "{}" ? ` ${args}` : ""}`;
+    const args = component.args as Record<string, unknown> | undefined;
+    const hint = args?.command ?? args?.path ?? args?.pattern;
+    const detail = typeof hint === "string" ? hint.replace(/\r?\n/g, " ") : JSON.stringify(component.args);
+    text = `${component.toolName}${detail && detail !== "{}" ? ` ${detail}` : ""}`;
   }
-  return [truncateToWidth(text, width)];
+  const failed = component.result?.isError === true;
+  const running = !component.result || component.isPartial;
+  const detail = text.startsWith(`${component.toolName} `) ? text.slice(component.toolName.length).trimStart()
+    : text === component.toolName ? "" : text;
+  const label = component.toolName === "bash" ? "Shell"
+    : component.toolName.replace(/_/g, " ").replace(/^./, letter => letter.toUpperCase());
+  const marker = failed ? "!" : running ? "~" : ">";
+  const suffix = failed ? " (failed)" : running ? " (running)" : "";
+  const title = `${marker} ${label}`;
+  const styled = theme
+    ? `${theme.fg(failed ? "error" : "muted", title)}${detail ? `  ${theme.fg("dim", detail)}` : ""}${theme.fg(failed ? "error" : "muted", suffix)}`
+    : `${title}${detail ? `  ${detail}` : ""}${suffix}`;
+  return [truncateToWidth(styled, width)];
 }
 
 function hostRequire(): NodeRequire {
@@ -104,7 +124,7 @@ function sdkEntry(require: ReturnType<typeof createRequire>): string {
 }
 
 /** Internal TUI adapter: no tools are replaced and no result content is changed. */
-export function installMinimalOutputHook(prototype: ToolPrototype): MinimalOutputHook {
+export function installMinimalOutputHook(prototype: ToolPrototype, getTheme?: () => Theme): MinimalOutputHook {
   for (const name of ["updateDisplay", "getResultRenderer", "getTextOutput", "setExpanded", "invalidate", "render"] as const) {
     if (typeof prototype[name] !== "function") {
       throw new Error(`Pi's tool display API is incompatible: missing ${name}. Normal output remains enabled.`);
@@ -117,6 +137,7 @@ export function installMinimalOutputHook(prototype: ToolPrototype): MinimalOutpu
       getResultRenderer: prototype.getResultRenderer,
       getTextOutput: prototype.getTextOutput,
       render: prototype.render,
+      ...(typeof prototype.handleMouse === "function" ? { handleMouse: prototype.handleMouse } : {}),
     };
     const seen = new WeakSet<ToolComponent>();
     const components = new Set<WeakRef<ToolComponent>>();
@@ -133,10 +154,11 @@ export function installMinimalOutputHook(prototype: ToolPrototype): MinimalOutpu
         }
       },
       restore() {
-        for (const name of Object.keys(originals) as Array<keyof typeof originals>) {
+        for (const name of ["updateDisplay", "getResultRenderer", "getTextOutput", "render"] as const) {
           // Do not remove another extension's subsequently installed wrapper.
           if (prototype[name] === wrappers[name]) Object.assign(prototype, { [name]: originals[name] });
         }
+        if (originals.handleMouse && prototype.handleMouse === mouseWrapper) prototype.handleMouse = originals.handleMouse;
         delete prototype[HOOK];
         components.clear();
       },
@@ -164,9 +186,19 @@ export function installMinimalOutputHook(prototype: ToolPrototype): MinimalOutpu
         return current.enabled && !this.expanded ? "" : originals.getTextOutput.apply(this, args);
       },
       render(this: ToolComponent, width: number): string[] {
-        return current.enabled && !this.expanded ? compactCall(this, width) : originals.render.call(this, width);
+        return current.enabled && !this.expanded ? compactCall(this, width, getTheme?.()) : originals.render.call(this, width);
       },
     };
+    const mouseWrapper: MouseHandler = function(event) {
+      if (current.enabled && !this.expanded) {
+        if (event.type !== "click" || event.button !== "left" || event.y !== 0 || event.shift || event.alt || event.ctrl) return undefined;
+        this.setExpanded(true);
+        this.ui.requestRender();
+        return { handled: true, render: true };
+      }
+      return originals.handleMouse?.call(this, event);
+    };
+    if (originals.handleMouse) prototype.handleMouse = mouseWrapper;
     Object.assign(prototype, wrappers);
     prototype[HOOK] = state;
   }

package/extensions/minimal-output/index.ts

@@ -1,5 +1,6 @@
 import type { ExtensionAPI, ExtensionContext } from "@earendil-works/pi-coding-agent";
 import { installMinimalOutputHook, loadToolPrototype, type MinimalOutputHook } from "./hook.ts";
+import type { SettingsControl } from "../settings/registry.ts";
 
 const ENTRY = "pi-better-harness-tool-output";
 
@@ -24,7 +25,7 @@ export default function minimalOutputExtension(pi: ExtensionAPI): void {
     }
     if (hook) return true;
     try {
-      hook = installMinimalOutputHook(await loadToolPrototype());
+      hook = installMinimalOutputHook(await loadToolPrototype(), () => ctx.ui.theme);
       return true;
     } catch (error) {
       ctx.ui.notify(`Minimal tool output is unavailable: ${error instanceof Error ? error.message : String(error)}`, "warning");
@@ -32,13 +33,13 @@ export default function minimalOutputExtension(pi: ExtensionAPI): void {
     }
   }
 
-  pi.on("session_start", async (_event, ctx) => {
+  async function restore(ctx: ExtensionContext): Promise<void> {
     if (ctx.mode !== "tui") return;
     enabled = false;
     for (const entry of ctx.sessionManager.getBranch()) {
       if (entry.type === "custom" && entry.customType === ENTRY) {
-        const data = entry.data as { enabled?: unknown } | undefined;
-        if (typeof data?.enabled === "boolean") enabled = data.enabled;
+        const data = entry.data as { version?: unknown; enabled?: unknown } | undefined;
+        if (data?.version === 1 && typeof data.enabled === "boolean") enabled = data.enabled;
       }
     }
     // Observe transcript components from startup, so the toggle also redraws existing rows.
@@ -47,9 +48,35 @@ export default function minimalOutputExtension(pi: ExtensionAPI): void {
       hook!.setEnabled(enabled);
     } else enabled = false;
     status(ctx);
-  });
+  }
+
+  async function change(value: string, ctx: ExtensionContext): Promise<void> {
+    if (value !== "Normal" && value !== "Minimal") throw new Error("Invalid tool output mode.");
+    if (!await ensureHook(ctx)) throw new Error("Tool output settings are unavailable in this Pi runtime.");
+    const data = { version: 1, enabled: value === "Minimal" };
+    try { pi.appendEntry(ENTRY, data); }
+    catch (error) { data.version = 0; data.enabled = enabled; throw error; }
+    enabled = data.enabled;
+    if (enabled) collapseTools(ctx);
+    hook!.setEnabled(enabled);
+    status(ctx);
+  }
+
+  const setting: SettingsControl = {
+    id: "tool-output", label: "Tool output", values: ["Normal", "Minimal"],
+    get: () => enabled ? "Minimal" : "Normal", change,
+  };
+  const register = () => pi.events.emit("harness-settings:register", setting);
+  let stopSettings: (() => void) | undefined;
+  const subscribe = () => { stopSettings ??= pi.events.on("harness-settings:request", register); };
+  subscribe();
+  register();
+  pi.on("session_start", async (_event, ctx) => { subscribe(); await restore(ctx); register(); });
+  pi.on("session_tree", async (_event, ctx) => { await restore(ctx); });
 
   pi.on("session_shutdown", () => {
+    stopSettings?.();
+    stopSettings = undefined;
     hook?.dispose();
     hook = undefined;
     enabled = false;
@@ -71,12 +98,8 @@ export default function minimalOutputExtension(pi: ExtensionAPI): void {
         ctx.ui.notify("Usage: /tool-output [minimal|normal]", "warning");
         return;
       }
-      if (!await ensureHook(ctx)) return;
-      enabled = mode ? mode === "minimal" : !enabled;
-      if (enabled) collapseTools(ctx);
-      hook!.setEnabled(enabled);
-      pi.appendEntry(ENTRY, { version: 1, enabled });
-      status(ctx);
+      try { await change((mode ? mode === "minimal" : !enabled) ? "Minimal" : "Normal", ctx); }
+      catch (error) { ctx.ui.notify(error instanceof Error ? error.message : String(error), "warning"); return; }
       ctx.ui.notify(enabled
         ? "Minimal tool output on. Compact call headers remain visible. Ctrl+O reveals results."
         : "Normal tool output restored.", "info");

package/extensions/settings/index.ts

@@ -30,7 +30,7 @@ export default function harnessSettingsExtension(pi: ExtensionAPI): void {
             get: readSettings,
             change: mode => changeCallbackSetting(pi, ctx, mode),
             save: () => saveCallbackDefault(readSettings().mode),
-          });
+          }, registry.controls());
         } catch (error) {
           ctx.ui.notify(`Settings unavailable: ${error instanceof Error ? error.message : String(error)}`, "error");
           return;

package/extensions/settings/page.ts

@@ -1,6 +1,6 @@
 import type { Theme } from "@earendil-works/pi-coding-agent";
 import { Container, SettingsList, Text, getKeybindings, setKeybindings, KeybindingsManager, TUI_KEYBINDINGS, matchesKey, type SettingItem } from "@earendil-works/pi-tui";
-import type { SettingsLink } from "./registry.ts";
+import type { SettingsControl, SettingsLink } from "./registry.ts";
 
 export interface CallbackSettingsControl {
   get(): { mode: "hold" | "steer"; source: "session" | "default" };
@@ -11,7 +11,7 @@ export interface CallbackSettingsControl {
 const CALLBACK_SETTING_ID = "callbacks:while-busy";
 const callbackLabel = (mode: "hold" | "steer") => mode === "steer" ? "Steer active run" : "Wait until idle";
 
-export async function chooseHarnessSetting(ctx: any, links: SettingsLink[], selected?: string, callbacks?: CallbackSettingsControl): Promise<string | undefined> {
+export async function chooseHarnessSetting(ctx: any, links: SettingsLink[], selected?: string, callbacks?: CallbackSettingsControl, controls: SettingsControl[] = []): Promise<string | undefined> {
   return ctx.ui.custom((tui: any, theme: Theme, _kb: unknown, done: (id?: string) => void) => {
     let callbackMode = callbacks?.get().mode ?? "hold";
     const callbackSetting: SettingItem | undefined = callbacks ? {
@@ -23,9 +23,11 @@ export async function chooseHarnessSetting(ctx: any, links: SettingsLink[], sele
     } : undefined;
     const items: SettingItem[] = [
       ...(callbackSetting ? [callbackSetting] : []),
+      ...controls.map(control => ({ id: `setting:${control.id}`, label: control.label, currentValue: control.get(), description: "Session setting", values: control.values })),
       ...links.map(link => ({ id: `link:${link.id}`, label: link.label, currentValue: link.command, description: "Open package settings", values: [link.command] })),
     ];
     const feedback = new Text("", 0, 0);
+    let changing = false;
     const report = (message: string, error = false) => {
       feedback.setText(theme.fg(error ? "error" : "muted", message));
       tui.requestRender?.();
@@ -35,6 +37,20 @@ export async function chooseHarnessSetting(ctx: any, links: SettingsLink[], sele
       value: (text, active) => active ? theme.inverse(text) : theme.fg("muted", text),
       description: text => theme.fg("muted", text), cursor: "> ", hint: text => theme.fg("dim", text),
     }, (id, value) => {
+      const control = controls.find(item => `setting:${item.id}` === id);
+      if (control) {
+        const previous = control.get();
+        changing = true;
+        report("Saving session setting...");
+        Promise.resolve().then(() => control.change(value, ctx)).then(() => {
+          list.updateValue(id, control.get());
+          report("Session setting saved.");
+        }).catch(error => {
+          list.updateValue(id, previous);
+          report(error instanceof Error ? error.message : String(error), true);
+        }).finally(() => { changing = false; tui.requestRender?.(); });
+        return;
+      }
       if (id !== CALLBACK_SETTING_ID || !callbacks || !callbackSetting) { done(id); return; }
       try {
         const nextMode = value === "Steer active run" ? "steer" : "hold";
@@ -67,6 +83,7 @@ export async function chooseHarnessSetting(ctx: any, links: SettingsLink[], sele
       render: (width: number) => page.render(width),
       invalidate: () => page.invalidate(),
       handleInput(data: string) {
+        if (changing) return;
         if (callbacks && matchesKey(data, "ctrl+s")) {
           try { callbacks.save(); report("Callback default saved."); }
           catch (error) { report(error instanceof Error ? error.message : String(error), true); }

package/extensions/settings/registry.ts

@@ -7,21 +7,38 @@ export interface SettingsLink {
   open(ctx: ExtensionCommandContext): Promise<void> | void;
 }
 
+export interface SettingsControl {
+  id: string;
+  label: string;
+  values: string[];
+  get(): string;
+  change(value: string, ctx: ExtensionCommandContext): Promise<void> | void;
+}
+
 export function createSettingsRegistry(pi: ExtensionAPI) {
   const links = new Map<string, SettingsLink>();
+  const controls = new Map<string, SettingsControl>();
   const conflicts = new Set<string>();
   const stop = pi.events.on("harness-settings:register", (candidate: unknown) => {
-    const link = candidate as Partial<SettingsLink> | null;
-    if (!link || !/^[a-z][a-z0-9-]*$/.test(link.id ?? "") || typeof link.label !== "string" ||
-      typeof link.command !== "string" || !link.command.startsWith("/") || typeof link.open !== "function") return;
+    const link = candidate as Partial<SettingsLink & SettingsControl> | null;
+    if (!link || !/^[a-z][a-z0-9-]*$/.test(link.id ?? "") || typeof link.label !== "string") return;
+    const control = Array.isArray(link.values) && link.values.length > 1 && link.values.every(value => typeof value === "string")
+      && typeof link.get === "function" && typeof link.change === "function";
+    if (!control && (typeof link.command !== "string" || !link.command.startsWith("/") || typeof link.open !== "function")) return;
     if (conflicts.has(link.id!)) return;
     const prior = links.get(link.id!);
-    if (prior && prior.open !== link.open) { links.delete(link.id!); conflicts.add(link.id!); return; }
-    links.set(link.id!, link as SettingsLink);
+    const priorControl = controls.get(link.id!);
+    if ((prior && (control || prior.open !== link.open)) ||
+      (priorControl && (!control || priorControl.change !== link.change || priorControl.get !== link.get))) {
+      links.delete(link.id!); controls.delete(link.id!); conflicts.add(link.id!); return;
+    }
+    if (control) controls.set(link.id!, link as SettingsControl);
+    else links.set(link.id!, link as SettingsLink);
   });
   return {
-    refresh() { links.clear(); conflicts.clear(); pi.events.emit("harness-settings:request", undefined); },
+    refresh() { links.clear(); controls.clear(); conflicts.clear(); pi.events.emit("harness-settings:request", undefined); },
     list() { return [...links.values()].sort((a, b) => a.label.localeCompare(b.label)); },
-    dispose() { stop(); links.clear(); conflicts.clear(); },
+    controls() { return [...controls.values()].sort((a, b) => a.label.localeCompare(b.label)); },
+    dispose() { stop(); links.clear(); controls.clear(); conflicts.clear(); },
   };
-}
+}

package/node_modules/pi-better-plan/README.md

@@ -45,13 +45,15 @@ For any workflow using the shared task-plan contract (including `resolve-issues`
   "event": "unit-validated",
   "revision": 12,
   "changes": [
-    { "id": "1201", "set": { "stage": "done", "status": "succeeded", "worker": null, "headSha": "abc123" } },
+    { "id": "1201", "set": { "stage": "done", "status": "in-flight", "worker": null, "headSha": "abc123" } },
     { "id": "C1", "set": { "status": "combining" } }
   ],
   "profiling": { "outcome": "succeeded", "wallMs": 540000 }
 } }
 ```
 
+Each issue row shows where it is between code and delivery: `●` while it is being built, `◐ implemented` once `stage` is `done` with a `headSha`, `◑ review passed` while `reviewedHead` equals `headSha`, and `✓` only at `status: succeeded`, when the change has landed on the target branch by a merged PR or a direct push. Only `✓` rows count as complete.
+
 `changes` addresses units, components, and fleet stages by id (a change without an id sets run-level fields); all changes in one call are one transition. For a scope change, a change with `target` `unit` or `component` and an `add` row appends a new row; existing rows cannot be removed or renamed. The tool checks ids, dependencies (including cycles), worker slots, and status/stage values against the shared issue-resolution contract (a fleet status of `n/a` is saved as `not-applicable`), refuses a stale `revision`, appends an optional `decision`, then saves the plan atomically with `planRevision + 1` and a new `updatedAt`, and appends one profiling event with the same revision to the log the run already uses (`profiling/run.jsonl` or `profiling.jsonl`). A rejected update changes nothing. The event is appended before the plan is renamed into place, so a crash between the two can leave the log one revision ahead; the next update notices, reuses that revision, and marks its event with `logAheadRevision`.
 
 The binding records the active workflow's name, which is shown in the plan view. A different workflow cannot read or update that binding; a new invocation or ownership release invalidates it. The skill name is not an allowlist: explicit binding opts a workflow into the shared schema and transition rules, while path confinement to `.resolve-issues/rush/<run-id>/task-plan.json`, run identity, size limits, and revision checks still apply. Workflows using other schemas or storage locations retain their own planning and must not bind incompatible state.

package/node_modules/pi-better-plan/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-better-plan",
-  "version": "0.5.4",
+  "version": "0.6.0",
   "description": "Structured execution plans with persistent progress for Pi.",
   "license": "MIT",
   "type": "module",

package/node_modules/pi-better-plan/src/plan-presentation.ts

@@ -5,7 +5,9 @@ export interface PlanRenderTheme {
   fg(color: string, value: string): string;
 }
 
-export type DisplayStatus = "completed" | "in_progress" | "pending" | "blocked" | "failed" | "skipped" | "unknown";
+// `implemented` and `reviewed` are in-progress states a workflow row can show between
+// "code done" and "delivered"; native plans never use them.
+export type DisplayStatus = "completed" | "in_progress" | "implemented" | "reviewed" | "pending" | "blocked" | "failed" | "skipped" | "unknown";
 export interface PlanRow {
   label: string;
   title: string;
@@ -24,6 +26,8 @@ export function statusStyle(status: DisplayStatus): { glyph: string; color: stri
   switch (status) {
     case "completed": return { glyph: "✓", color: "success", label: "" };
     case "in_progress": return { glyph: "●", color: "accent", label: "active" };
+    case "implemented": return { glyph: "◐", color: "accent", label: "implemented" };
+    case "reviewed": return { glyph: "◑", color: "success", label: "review passed" };
     case "blocked": return { glyph: "!", color: "warning", label: "blocked" };
     case "failed": return { glyph: "×", color: "error", label: "failed" };
     case "skipped": return { glyph: "—", color: "dim", label: "skipped" };
@@ -45,7 +49,8 @@ export function renderPlanPresentation(
   const count = (status: DisplayStatus) => plan.rows.filter((row) => row.status === status).length;
   const summary = [`${count("completed")}/${plan.rows.length} complete`];
   for (const [status, label] of [
-    ["in_progress", "in progress"], ["blocked", "blocked"], ["failed", "failed"],
+    ["in_progress", "in progress"], ["implemented", "implemented"], ["reviewed", "review passed"],
+    ["blocked", "blocked"], ["failed", "failed"],
     ["skipped", "skipped"], ["unknown", "unknown"],
   ] as const) {
     if (count(status)) summary.push(`${count(status)} ${label}`);
@@ -67,7 +72,7 @@ export function renderPlanPresentation(
       theme.fg(style.color, style.glyph) + " " + theme.fg("dim", label) +
       " ".repeat(Math.max(0, labelWidth - visibleWidth(label)) + 2);
     const titleColor = row.status === "blocked" ? "warning" : row.status === "failed" ? "error" :
-      row.status === "in_progress" ? "text" : "muted";
+      row.status === "in_progress" || row.status === "implemented" || row.status === "reviewed" ? "text" : "muted";
     const badge = planText(row.statusLabel ?? style.label);
     const available = size - visibleWidth(prefix);
     // At narrow widths use an inline status so it survives title truncation.

package/node_modules/pi-better-plan/src/workflow-plan.ts

@@ -16,6 +16,9 @@ interface RushUnit {
   dependsOn: string[];
   worker?: string | number | undefined;
   note?: string | undefined;
+  headSha?: string | undefined;
+  reviewedHead?: string | undefined;
+  delivery?: string | undefined;
 }
 
 export interface RushPlan {
@@ -78,6 +81,9 @@ export function projectRushPlan(data: unknown, runId: string): RushPlan {
       dependsOn: Array.isArray(unit.dependsOn) ? unit.dependsOn.filter((id): id is string => typeof id === "string") : [],
       worker: typeof unit.worker === "string" || typeof unit.worker === "number" ? unit.worker : undefined,
       note: typeof unit.note === "string" ? unit.note : undefined,
+      headSha: typeof unit.headSha === "string" && unit.headSha ? unit.headSha : undefined,
+      reviewedHead: typeof unit.reviewedHead === "string" && unit.reviewedHead ? unit.reviewedHead : undefined,
+      delivery: typeof unit.delivery === "string" && unit.delivery ? unit.delivery : undefined,
     };
   });
   const fleet: RushPlan["fleet"] = {};
@@ -133,6 +139,15 @@ function workflowStatus(status: string): DisplayStatus {
   }
 }
 
+// A unit is delivered (completed) only at `status: succeeded`, whether its change landed by a
+// merged PR or a direct push. Between code done and delivery, a unit with a HEAD shows
+// whether that HEAD passed review; without one it shows plain in-progress, as before.
+function unitStatus(unit: RushUnit): DisplayStatus {
+  const status = workflowStatus(unit.status);
+  if (status !== "in_progress" || unit.stage !== "done" || !unit.headSha) return status;
+  return unit.reviewedHead === unit.headSha ? "reviewed" : "implemented";
+}
+
 function presentRushPlan(plan: RushPlan, owner: string): PlanPresentation {
   // The warehouse canary only applies to data work; hide it otherwise.
   const stages = ["explore", "implement", "review", "ci", ...(plan.warehouseCanaryRequired ? ["canary"] : [])];
@@ -147,8 +162,8 @@ function presentRushPlan(plan: RushPlan, owner: string): PlanPresentation {
   return {
     metadata,
     rows: plan.issues.map((unit) => {
-      const status = workflowStatus(unit.status);
-      const details = [`${unit.stage} · ${unit.status}${unit.worker !== undefined ? ` · worker ${unit.worker}` : ""}`];
+      const status = unitStatus(unit);
+      const details = [`${unit.stage} · ${unit.status}${unit.delivery ? ` · ${unit.delivery}` : ""}${unit.worker !== undefined ? ` · worker ${unit.worker}` : ""}`];
       if (unit.dependsOn.length) details.push(`after: ${unit.dependsOn.map((id) => `#${id}`).join(", ")}`);
       if (unit.note) details.push(unit.note);
       return {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-better-harness",
-  "version": "0.21.0",
+  "version": "0.22.0",
   "description": "Pi extension bundle for a write sandbox, subagents, background tasks, SSH, goals, and structured plans.",
   "license": "MIT",
   "type": "module",
@@ -55,7 +55,7 @@
   "dependencies": {
     "pi-better-background-tasks": "0.7.0",
     "pi-better-goal": "0.8.1",
-    "pi-better-plan": "0.5.4",
+    "pi-better-plan": "0.6.0",
     "pi-better-sandbox": "0.11.0",
     "pi-better-ssh": "0.1.2",
     "pi-better-subagents": "0.15.0",