@xynogen/pix-subagent 0.1.1 → 0.1.3

package/README.md

@@ -8,7 +8,7 @@ Pi extension — planner-driven sub-agents with 3 tools, live widget (model alwa
 pi install npm:@xynogen/pix-subagent
 ```
 
-> Also included in [`@xynogen/pix-core`](https://github.com/xynogen/pix-mono/tree/main/packages/pix-core):
+> Also included in [`@xynogen/pix-core`](https://www.npmjs.com/package/@xynogen/pix-core):
 >
 > ```bash
 > pi install npm:@xynogen/pix-core

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@xynogen/pix-subagent",
-	"version": "0.1.1",
+	"version": "0.1.3",
 	"description": "Pi tool — planner-driven sub-agents: spawn, fetch, steer scoped child agents",
 	"type": "module",
 	"main": "src/index.ts",

package/src/index.ts

@@ -86,7 +86,6 @@ export default function registerPixSubagent(pi: ExtensionAPI): void {
 		// onComplete — fire subagent-notification nudge for each finished bg agent
 		(record) => {
 			agentActivity.delete(record.id);
-			widget.markFinished(record.id);
 
 			if (record.resultConsumed) {
 				widget.update();
@@ -191,9 +190,9 @@ export default function registerPixSubagent(pi: ExtensionAPI): void {
 		}
 	});
 
-	pi.on("tool_execution_start", (_event, ctx) => {
-		widget.setUICtx(ctx.ui as Parameters<typeof widget.setUICtx>[0]);
+	pi.on("turn_start", (_event, ctx) => {
 		widget.onTurnStart();
+		widget.setUICtx(ctx.ui as Parameters<typeof widget.setUICtx>[0]);
 		widget.ensureTimer();
 	});
 

package/src/tools.ts

@@ -180,10 +180,10 @@ If the target is already known, use a direct tool — \`read\` for a known path,
 
 ## Usage notes
 - Always include a short (3-5 word) description summarizing what the agent will do (shown in UI).
-- When you launch multiple agents for independent work, send them in a single message with multiple tool uses, with run_in_background: true on each, so they run concurrently.
+- When you launch multiple agents for independent work, send them in a single message with multiple tool uses, so they run concurrently.
 - When the agent is done, it returns a single message. The result is not visible to the user — to show the user, send a text message with a concise summary.
 - Trust but verify: an agent's summary describes what it intended to do, not what it did. When an agent writes or edits code, check the actual changes before reporting work as done.
-- Use run_in_background for work you don't need immediately. You will be notified when it completes — do NOT poll or sleep waiting for it.
+- Use run_in_background: true only when you explicitly do not need to see the output inline (e.g. fire-and-forget background tasks). Default is foreground — the agent streams inline and you see its work as it runs.
 - Use resume with an agent ID to continue a previous agent's work.
 - Use agent_steer to send mid-run messages to a running background agent.
 - Use model to specify a model from the available models list above (provider/id or fuzzy e.g. "haiku").
@@ -296,12 +296,10 @@ export function createAgentTool(
 
 			const stats = buildStats(details, theme);
 
-			// Streaming / running
+			// Streaming / running — live state shown by the ● Agents widget, so the
+			// inline transcript stays empty to avoid stacking one card per agent.
 			if (isPartial || details.status === "running") {
-				const frame = SPINNER[details.spinnerFrame ?? 0];
-				let line = theme.fg("accent", frame) + (stats ? ` ${stats}` : "");
-				line += `\n${theme.fg("dim", `  ⎿  ${details.activity ?? "thinking…"}`)}`;
-				return new Text(line, 0, 0);
+				return new Text("", 0, 0);
 			}
 
 			// Background launched

package/src/ui/widget.ts

@@ -133,8 +133,8 @@ export class AgentWidget {
 	private uiCtx: UICtx | undefined;
 	private widgetFrame = 0;
 	private widgetInterval: ReturnType<typeof setInterval> | undefined;
-	private finishedTurnAge = new Map<string, number>();
-	private static readonly ERROR_LINGER_TURNS = 2;
+	private static readonly FINISHED_LINGER_MS = 5_000;
+	private static readonly ERROR_LINGER_MS = 15_000;
 	private widgetRegistered = false;
 	private tui: unknown = undefined;
 	private lastStatusText: string | undefined;
@@ -154,9 +154,6 @@ export class AgentWidget {
 	}
 
 	onTurnStart() {
-		for (const [id, age] of this.finishedTurnAge) {
-			this.finishedTurnAge.set(id, age + 1);
-		}
 		this.update();
 	}
 
@@ -166,18 +163,14 @@ export class AgentWidget {
 		}
 	}
 
-	private shouldShowFinished(agentId: string, status: string): boolean {
-		const age = this.finishedTurnAge.get(agentId) ?? 0;
-		const maxAge = ERROR_STATUSES.has(status)
-			? AgentWidget.ERROR_LINGER_TURNS
-			: 1;
-		return age < maxAge;
-	}
-
-	markFinished(agentId: string) {
-		if (!this.finishedTurnAge.has(agentId)) {
-			this.finishedTurnAge.set(agentId, 0);
-		}
+	private shouldShowFinished(status: string, completedAt: number): boolean {
+		// Linger a few seconds after finish, then drop. The ✓ … Done line in the
+		// transcript is the permanent record; errors stay longer so failures are
+		// noticed. The 80ms widget timer re-evaluates this continuously.
+		const linger = ERROR_STATUSES.has(status)
+			? AgentWidget.ERROR_LINGER_MS
+			: AgentWidget.FINISHED_LINGER_MS;
+		return Date.now() - completedAt < linger;
 	}
 
 	private renderFinishedLine(
@@ -246,8 +239,8 @@ export class AgentWidget {
 			(a) =>
 				a.status !== "running" &&
 				a.status !== "queued" &&
-				a.completedAt &&
-				this.shouldShowFinished(a.id, a.status),
+				a.completedAt != null &&
+				this.shouldShowFinished(a.status, a.completedAt),
 		);
 
 		if (running.length === 0 && queued.length === 0 && finished.length === 0)
@@ -403,7 +396,10 @@ export class AgentWidget {
 		for (const a of allAgents) {
 			if (a.status === "running") runningCount++;
 			else if (a.status === "queued") queuedCount++;
-			else if (a.completedAt && this.shouldShowFinished(a.id, a.status))
+			else if (
+				a.completedAt != null &&
+				this.shouldShowFinished(a.status, a.completedAt)
+			)
 				hasFinished = true;
 		}
 		const hasActive = runningCount > 0 || queuedCount > 0;
@@ -422,10 +418,6 @@ export class AgentWidget {
 				clearInterval(this.widgetInterval);
 				this.widgetInterval = undefined;
 			}
-			for (const [id] of this.finishedTurnAge) {
-				if (!allAgents.some((a) => a.id === id))
-					this.finishedTurnAge.delete(id);
-			}
 			return;
 		}