@shanepadgett/tau-agent 0.33.1 → 0.35.0

package/extensions/subagent/cmux-dashboard.ts

@@ -106,10 +106,7 @@ function dashboardState(status: SubagentInvocationSnapshot["status"]): string {
 	}
 }
 
-export function formatDashboardMarkdown(snapshots: readonly SubagentInvocationSnapshot[]): string {
-	const ordered = [...snapshots].sort(
-		(a, b) => a.startedAt - b.startedAt || a.invocationId.localeCompare(b.invocationId),
-	);
+function dashboardCounts(ordered: readonly SubagentInvocationSnapshot[]): string {
 	const activeCount = ordered.filter((snapshot) =>
 		["waiting", "starting", "running"].includes(snapshot.status),
 	).length;
@@ -119,7 +116,39 @@ export function formatDashboardMarkdown(snapshots: readonly SubagentInvocationSn
 	const counts = [`${activeCount} active`, `${doneCount} done`];
 	if (failedCount > 0) counts.push(`${failedCount} failed`);
 	if (stoppedCount > 0) counts.push(`${stoppedCount} stopped`);
-	const lines = ["# Subagents", "", counts.join(" · "), ""];
+	return counts.join(" · ");
+}
+
+function dashboardTableRow(details: SubagentInvocationSnapshot): string {
+	const latest = details.actions.at(-1);
+	const currentTool = details.currentActivity?.match(/^\S+/)?.[0];
+	const lastTool = currentTool ?? latest?.tool ?? "";
+	const ctx = typeof details.contextPercent === "number" ? `${details.contextPercent.toFixed(1)}%` : "—";
+	return `| ${tableCell(`${details.displayName} (${details.agent})`, 48)} | ${dashboardState(details.status)} | ${tableCell(lastTool, 32) || "—"} | ${details.toolCalls} | $${details.usage.cost.toFixed(4)} | ${ctx} | ${elapsed(details.durationMs)} |`;
+}
+
+function dashboardInputSection(details: SubagentInvocationSnapshot): string[] {
+	const lines = [
+		`### ${tableCell(details.displayName, 80)}`,
+		"",
+		tableCell(details.agent, 80),
+		"",
+		quote(details.task) || "> _(empty)_",
+	];
+	if (details.files.length > 0) {
+		lines.push("", "**Files**", "");
+		for (const file of details.files) lines.push(`- ${tableCell(file, 300)}`);
+	}
+	if (details.error) lines.push("", "**Error**", "", quote(details.error));
+	lines.push("");
+	return lines;
+}
+
+function formatDashboardMarkdown(snapshots: readonly SubagentInvocationSnapshot[]): string {
+	const ordered = [...snapshots].sort(
+		(a, b) => a.startedAt - b.startedAt || a.invocationId.localeCompare(b.invocationId),
+	);
+	const lines = ["# Subagents", "", dashboardCounts(ordered), ""];
 	if (ordered.length === 0) {
 		lines.push("_No subagents._", "");
 		return lines.join("\n");
@@ -127,32 +156,12 @@ export function formatDashboardMarkdown(snapshots: readonly SubagentInvocationSn
 	lines.push(
 		"| Agent | State | Last tool | Calls | Cost | Ctx | Time |",
 		"| --- | --- | --- | ---: | ---: | ---: | ---: |",
+		...ordered.map(dashboardTableRow),
+		"",
+		"## Inputs",
+		"",
 	);
-	for (const details of ordered) {
-		const latest = details.actions.at(-1);
-		const currentTool = details.currentActivity?.match(/^\S+/)?.[0];
-		const lastTool = currentTool ?? latest?.tool ?? "";
-		const ctx = typeof details.contextPercent === "number" ? `${details.contextPercent.toFixed(1)}%` : "—";
-		lines.push(
-			`| ${tableCell(`${details.displayName} (${details.agent})`, 48)} | ${dashboardState(details.status)} | ${tableCell(lastTool, 32) || "—"} | ${details.toolCalls} | $${details.usage.cost.toFixed(4)} | ${ctx} | ${elapsed(details.durationMs)} |`,
-		);
-	}
-	lines.push("", "## Inputs", "");
-	for (const details of ordered) {
-		lines.push(
-			`### ${tableCell(details.displayName, 80)}`,
-			"",
-			tableCell(details.agent, 80),
-			"",
-			quote(details.task) || "> _(empty)_",
-		);
-		if (details.files.length > 0) {
-			lines.push("", "**Files**", "");
-			for (const file of details.files) lines.push(`- ${tableCell(file, 300)}`);
-		}
-		if (details.error) lines.push("", "**Error**", "", quote(details.error));
-		lines.push("");
-	}
+	for (const details of ordered) lines.push(...dashboardInputSection(details));
 	return lines.join("\n");
 }
 

package/extensions/subagent/index.ts

@@ -204,22 +204,86 @@ export default function subagentExtension(pi: ExtensionAPI): void {
 		if (!pi.getActiveTools().includes("subagent")) return undefined;
 		const discovery = await discoverAgents(ctx.cwd, ctx.isProjectTrusted());
 		warn(discovery, ctx);
-		const lines = (await parentVisibleAgents(ctx, discovery)).map((agent) => `- ${agent.name}: ${agent.description}`);
+		const agents = await parentVisibleAgents(ctx, discovery);
+		const lines = agents.map((agent) => `- ${agent.name}: ${agent.description}`);
+		const scoutGuidance = agents.some((agent) => agent.name === "scout")
+			? "\n\nScout only for substantial multi-hop lookup that would flood parent context. Skip few known-path reads, single declaration lookups, or small digs with most evidence already in hand; use tools directly. When uncertain, dig yourself. Do not send code review, diagnosis, design, or other judgment work to scout."
+			: "";
 		const prompt = `## Subagents
-Use \`subagent\` when an available agent matches a focused part of the task.
+Use \`subagent\` only when an available agent's listed purpose matches the delegated work. Stay inside that purpose. Off-purpose calls are denied — for example, do not send code review to a lookup-only agent.
 
 Available agents for this turn:
-${lines.join("\n")}
+${lines.join("\n") || "none"}
 
 Start a fresh thread with \`agent\` and \`task\`. Continue an existing thread with \`thread\` and \`task\`. Reuse a thread when feedback or follow-up work depends on its prior reads and reasoning. Start fresh for unrelated work or when its context is stale or oversized.
 
 Pass \`files\` when exact relevant files are already known. Tau autoreads current line-numbered snapshots into that child turn before it starts.
 
-Delegate one focused task per call. Children do not inherit parent messages. Include exact absolute reference paths when a child must inspect a repository outside the current working directory.
-
-Scout only for substantial multi-hop lookup that would flood parent context. Skip few known-path reads, single declaration lookups, or small digs with most evidence already in hand; use tools directly. When uncertain, dig yourself.`;
+Delegate one focused task per call. Children do not inherit parent messages. Include exact absolute reference paths when a child must inspect a repository outside the current working directory.${scoutGuidance}`;
 		return { systemPrompt: `${event.systemPrompt}\n\n${prompt}` };
 	});
+	const resolveFreshSubagentDefinition = async (ctx: ExtensionContext, agent: string) => {
+		const discovery = await discoverAgents(ctx.cwd, ctx.isProjectTrusted());
+		warn(discovery, ctx);
+		const definition = discovery.agents.get(agent);
+		const invalid = discovery.invalid.get(agent);
+		if (!definition) {
+			const reason = invalid?.map((item) => item.reason).join("; ") ?? "unknown agent";
+			const names = (await parentVisibleAgents(ctx, discovery)).map((item) => item.name).join(", ") || "none";
+			return {
+				ok: false as const,
+				phase: "discovery" as const,
+				error: `Agent ${agent} discovery failed: ${reason}. Runnable agents: ${names}`,
+			};
+		}
+		const disabled = await disabledAgentNames(ctx);
+		if (disabled.effective.has(definition.name)) {
+			const source = disabled.configured.has(definition.name) ? "in Tau settings" : "for this session";
+			return {
+				ok: false as const,
+				phase: "discovery" as const,
+				error: `Agent ${definition.name} is disabled ${source}.`,
+			};
+		}
+		if (definition.name === "context-sync") {
+			const sync = (await loadTauExtensionSettings(ctx, contextSettings)).sync;
+			if (!sync.enabled) {
+				return { ok: false as const, phase: "discovery" as const, error: "Context sync is disabled in settings." };
+			}
+			if (!sync.automation) {
+				return {
+					ok: false as const,
+					phase: "discovery" as const,
+					error: "Context-sync automation is disabled. Use /context-sync or enable extensions.context.sync.automation.",
+				};
+			}
+		}
+		return { ok: true as const, definition };
+	};
+
+	const disabledContinuationResult = async (
+		ctx: ExtensionContext,
+		threadKey: string | undefined,
+		task: string,
+		parentModel: string,
+		parentThinking: ReturnType<ExtensionAPI["getThinkingLevel"]>,
+	) => {
+		const thread = runtime.listThreads(ctx.cwd).find((item) => item.id === threadKey);
+		if (!thread) return undefined;
+		const disabled = await disabledAgentNames(ctx);
+		if (!disabled.effective.has(thread.definition.name)) return undefined;
+		const source = disabled.configured.has(thread.definition.name) ? "in Tau settings" : "for this session";
+		return failedToolResult(
+			thread.definition.name,
+			task,
+			"discovery",
+			parentModel,
+			parentThinking,
+			`Agent ${thread.definition.name} is disabled ${source}.`,
+			threadKey,
+		);
+	};
+
 	pi.registerTool(
 		defineTool<typeof params, SubagentDetails>({
 			name: "subagent",
@@ -251,24 +315,14 @@ Scout only for substantial multi-hop lookup that would flood parent context. Ski
 				}
 
 				if (continuing) {
-					const thread = runtime.listThreads(ctx.cwd).find((item) => item.id === threadKey);
-					if (thread) {
-						const disabled = await disabledAgentNames(ctx);
-						if (disabled.effective.has(thread.definition.name)) {
-							const source = disabled.configured.has(thread.definition.name)
-								? "in Tau settings"
-								: "for this session";
-							return failedToolResult(
-								thread.definition.name,
-								task,
-								"discovery",
-								parentModel,
-								parentThinking,
-								`Agent ${thread.definition.name} is disabled ${source}.`,
-								threadKey,
-							);
-						}
-					}
+					const disabledContinuation = await disabledContinuationResult(
+						ctx,
+						threadKey,
+						task,
+						parentModel,
+						parentThinking,
+					);
+					if (disabledContinuation) return disabledContinuation;
 				}
 
 				return runtime.execute({
@@ -291,41 +345,7 @@ Scout only for substantial multi-hop lookup that would flood parent context. Ski
 							],
 							details,
 						}),
-					resolveFreshDefinition: async () => {
-						const discovery = await discoverAgents(ctx.cwd, ctx.isProjectTrusted());
-						warn(discovery, ctx);
-						const definition = discovery.agents.get(agent);
-						const invalid = discovery.invalid.get(agent);
-						if (!definition) {
-							const reason = invalid?.map((item) => item.reason).join("; ") ?? "unknown agent";
-							const names =
-								(await parentVisibleAgents(ctx, discovery)).map((item) => item.name).join(", ") || "none";
-							return {
-								ok: false,
-								phase: "discovery",
-								error: `Agent ${agent} discovery failed: ${reason}. Runnable agents: ${names}`,
-							};
-						}
-						const disabled = await disabledAgentNames(ctx);
-						if (disabled.effective.has(definition.name)) {
-							const source = disabled.configured.has(definition.name) ? "in Tau settings" : "for this session";
-							return { ok: false, phase: "discovery", error: `Agent ${definition.name} is disabled ${source}.` };
-						}
-						if (definition.name === "context-sync") {
-							const sync = (await loadTauExtensionSettings(ctx, contextSettings)).sync;
-							if (!sync.enabled) {
-								return { ok: false, phase: "discovery", error: "Context sync is disabled in settings." };
-							}
-							if (!sync.automation) {
-								return {
-									ok: false,
-									phase: "discovery",
-									error: "Context-sync automation is disabled. Use /context-sync or enable extensions.context.sync.automation.",
-								};
-							}
-						}
-						return { ok: true, definition };
-					},
+					resolveFreshDefinition: () => resolveFreshSubagentDefinition(ctx, agent),
 				});
 			},
 			renderCall(args, theme, context) {

package/extensions/subagent/resume.ts

@@ -29,7 +29,7 @@ export function retainSubagentTurn(state: SubagentResumeState, record: RetainedT
 	};
 }
 
-export function selectRetainedTurns(
+function selectRetainedTurns(
 	state: SubagentResumeState,
 	optionalTokenBudget = RESUME_OPTIONAL_HISTORY_TOKENS,
 ): { records: readonly RetainedTurnRecord[]; omitted: number } {