@hcflabs/pi-subagent 28.2.3 → 28.3.0

package/README.md

@@ -43,7 +43,7 @@ Commands, prompts, and UI are for you. Tools and the packaged skill are for Main
 | `subagent_abort` | tool | Terminate an isolated request without discarding retained work. Retained candidates and integration work require release or proof of absence. |
 | `git_read` | tool | Let Role-declared children inspect bounded local Git evidence without Bash, tests, network access, or writes. Not available to Main. |
 | `pi-subagent` | skill | Guide Main through authorization, delegation, checks, integration, and recovery. |
-| Direct status widget (`D`) | ui | Show direct workflows as a tree under `Main` with references, names, Role badges, status, routes, elapsed time, measured task tokens, cost, and tool activity. In fullscreen mode, a click on a working row focuses that worker's verified Herdr pane. |
+| Direct status widget (`D`) | ui | Show direct workflows as a tree under `Main` with references, names, Role badges, status, routes, elapsed time, measured task tokens, cost, and tool activity. In fullscreen mode, a click on a working or finished worker row focuses that worker's verified Herdr pane. |
 | Isolated status widget (`I`) | ui | Show recorded task state, Role/model-class badges, and retained workspaces in the TUI. Readiness does not mean promotion. |
 
 `delegate_task`, `subagent_resume`, `subagent_stage`, `subagent_integrate`, and `subagent_abort` are model-only. Main must make these decisions visibly; `codemode` scripts can call only `subagent_status`.

package/docs/orchestration.md

@@ -200,7 +200,7 @@ Main · 2 working · 1 complete
 └─ ✓ D8.1 [R] complete · Review patch · …
 ```
 
-In Pi fullscreen mode, click a working worker row to focus its Herdr pane. Before the extension moves focus, Herdr must confirm that the pane still runs the exact launched agent. If the check fails, Pi shows a warning and focus does not change. The header, group rows, finished rows, the overflow row, and rows with a truncated reference do nothing when you click them. Regular mode leaves the mouse to the terminal. Use the Herdr tab label or `/subagent` to find a worker from the keyboard. A click moves focus only. It gives no ownership or control authority.
+In Pi fullscreen mode, click a worker row to focus its Herdr pane. This applies to working and finished workers, because a finished worker keeps its Herdr tab. A finished row stays in the widget until your next input. Before the extension moves focus, Herdr must confirm that the pane still runs the exact launched agent. If the check fails, for example because the tab was closed, Pi shows a warning and focus does not change. The header, group rows, the overflow row, and rows with a truncated reference do nothing when you click them. Regular mode leaves the mouse to the terminal. Use the Herdr tab label or `/subagent` to find a worker from the keyboard. A click moves focus only. It gives no ownership or control authority.
 
 The isolated widget shows the recorded task state and any retained workspace. It does not show reliable live model, thinking, or token readings from Herdr. Attention shows before ordinary work, and overflow uses `+N more · /subagent`. An aborted request with visible tasks has its own `request aborted` row. This row does not change the recorded statuses of the tasks.
 

package/extensions/subagent.ts

@@ -117,14 +117,15 @@ function statusCounts(items: readonly WidgetItem[]): string {
 	return STATUS_COUNT_ORDER.flatMap((status) => counts[status] ? [`${counts[status]} ${statusLabel(status)}`] : []).join(" · ");
 }
 
-/** One rendered widget line. `focus` names a live worker item that a click may focus. */
+/** One rendered widget line. `focus` names a worker item that a click may focus. */
 type WidgetRow = { text: string; focus?: string };
 type TreeLine = { top: number; child?: { last: boolean }; item?: [string, WidgetItem]; group?: WidgetItem[] };
 
 /**
  * Render the direct widget as a tree under a Main header.
  * A workflow with one launched worker is a leaf row. A workflow with more workers gets a group row.
- * `focusable` tells which item IDs have a live handle. Only fully visible references become click targets.
+ * `focusable` tells which item IDs have a launched handle. Working and finished worker rows are click targets.
+ * Only fully visible references become click targets.
  */
 function renderWidgetRows(
 	items: ReadonlyArray<[string, WidgetItem]>,
@@ -168,7 +169,7 @@ function renderWidgetRows(
 		const text = truncateToWidth(`${branch}${renderWorkerRow(item, room, now, spinnerIndex, theme)}`, width);
 		// The glyph, a space, and the full reference must be visible before a click can name this worker.
 		const referenceVisible = visibleWidth(connector) + 2 + visibleWidth(item.ref) <= width;
-		rows.push({ text, ...(item.status === "working" && referenceVisible && focusable(id) ? { focus: id } : {}) });
+		rows.push({ text, ...(referenceVisible && focusable(id) ? { focus: id } : {}) });
 	}
 	if (hidden.length) rows.push({ text: truncateToWidth(`${theme.fg("dim", "└─ ")}${theme.fg("muted", `… ${hidden.length} more · ${statusCounts(hidden)}`)}`, width) });
 	return rows;
@@ -242,6 +243,8 @@ export default function subagentExtension(pi: ExtensionAPI): void {
 	});
 	const widgetItems = new Map<string, WidgetItem>();
 	const tokenHandles = new Map<string, DirectHandle>();
+	// A finished worker keeps its Herdr tab. Its row keeps a focus handle until the row leaves the widget.
+	const focusHandles = new Map<string, Pick<DirectHandle, "focus">>();
 	const samplingTokens = new Set<string>();
 	const configStore = createSubagentConfigStore();
 	const configReporter = registerConfigReporting(pi, [configStore]);
@@ -520,9 +523,9 @@ export default function subagentExtension(pi: ExtensionAPI): void {
 	let focusInFlight = false;
 	// A click is a request only. The handle checks the exact Herdr agent identity before it moves focus.
 	const focusWorker = (id: string) => {
-		const handle = tokenHandles.get(id);
+		const handle = focusHandles.get(id);
 		const item = widgetItems.get(id);
-		if (!handle || item?.status !== "working" || focusInFlight) return;
+		if (!handle || !item || focusInFlight) return;
 		const epoch = sessionEpoch;
 		const ctx = latestCtx;
 		focusInFlight = true;
@@ -543,7 +546,7 @@ export default function subagentExtension(pi: ExtensionAPI): void {
 			return {
 				invalidate() {},
 				render(width) {
-					const rows = renderWidgetRows([...widgetItems], width, Date.now(), spinnerIndex, theme, (id) => tokenHandles.has(id));
+					const rows = renderWidgetRows([...widgetItems], width, Date.now(), spinnerIndex, theme, (id) => focusHandles.has(id));
 					renderedWidth = width;
 					hits = rows.flatMap(({ text, focus }, row) => focus ? [{ row, to: visibleWidth(text), id: focus }] : []);
 					return rows.map(({ text }) => text);
@@ -574,6 +577,7 @@ export default function subagentExtension(pi: ExtensionAPI): void {
 			for (const [oldestId, item] of widgetItems) {
 				if (item.status === "working" || item.taskId === taskId) continue;
 				widgetItems.delete(oldestId);
+				focusHandles.delete(oldestId);
 				if (widgetItems.size < MAX_WIDGET_ITEMS) break;
 			}
 		}
@@ -659,6 +663,7 @@ export default function subagentExtension(pi: ExtensionAPI): void {
 		stopWidgetTimer();
 		widgetItems.clear();
 		tokenHandles.clear();
+		focusHandles.clear();
 		samplingTokens.clear();
 		activeTui = undefined;
 		widgetInstalled = false;
@@ -679,7 +684,7 @@ export default function subagentExtension(pi: ExtensionAPI): void {
 	pi.on("input", (event) => {
 		if (event.source === "extension") return;
 		for (const [id, item] of widgetItems) {
-			if (item.status !== "working") widgetItems.delete(id);
+			if (item.status !== "working") { widgetItems.delete(id); focusHandles.delete(id); }
 		}
 		requestWidgetRender();
 	});
@@ -981,7 +986,7 @@ export default function subagentExtension(pi: ExtensionAPI): void {
 				await transient.cleanup();
 				handles.push(handle);
 				handleByEntry.set(entry.id, handle);
-				if (widgetItems.has(entry.id)) tokenHandles.set(entry.id, handle);
+				if (widgetItems.has(entry.id)) { tokenHandles.set(entry.id, handle); focusHandles.set(entry.id, handle); }
 				return handle;
 			};
 			// The first tab is verified before returning a handle; subsequent independent

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hcflabs/pi-subagent",
-  "version": "28.2.3",
+  "version": "28.3.0",
   "description": "Delegate bounded direct work or durable checked isolated task graphs to Pi roles.",
   "keywords": [
     "pi-package",