@arhen/pi-core-subagent 1.0.8 → 1.1.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@arhen/pi-core-subagent",
-  "version": "1.0.8",
+  "version": "1.1.0",
   "type": "module",
   "description": "pi extension: fast in-process subagents with single/parallel/chain, background runs, intercom and agent-to-agent mailbox. Leader defines agents inline.",
   "license": "MIT",

package/src/index.ts

@@ -18,7 +18,6 @@ import { StringEnum, type Api, type AssistantMessage, type Model } from "@earend
 import { Text, truncateToWidth } from "@earendil-works/pi-tui";
 import type { Component, TUI } from "@earendil-works/pi-tui";
 import { Type } from "typebox";
-import { lookupAgent } from "./agents.ts";
 import { CHILD_TALK_TOOLS, createChildTools, createWatchdog, type ChildHandlers } from "./child.ts";
 import { createMailbox, type Mailbox } from "./mailbox.ts";
 
@@ -251,6 +250,10 @@ function compactLines(run: RunSnapshot): string[] {
  *   └─ ✓ reviewer · 6 tools · 44s
  * Static icons (no animation); latest activity + tool count + runtime per agent.
  */
+const WIDGET_MAX_RUNS = 3;
+const WIDGET_MAX_TASKS_PER_RUN = 4;
+const WIDGET_MAX_LINES = 10;
+
 class SubagentsWidget implements Component {
 	constructor(
 		private readonly getRuns: () => RunSnapshot[],
@@ -265,17 +268,18 @@ class SubagentsWidget implements Component {
 		const runs = this.getRuns();
 		if (runs.length === 0) return [];
 		const lines: string[] = [];
-		runs.forEach((run, ri) => {
-			if (run.tasks.length === 0) return;
-			if (ri > 0) lines.push("");
+		let renderedRuns = 0;
+		for (const run of runs.slice(0, WIDGET_MAX_RUNS)) {
+			if (run.tasks.length === 0) continue;
+			if (lines.length > 0) lines.push("");
 			const done = run.tasks.filter((t) => TERMINAL.includes(t.status)).length;
 			const active = !TERMINAL.includes(run.status);
 			const head = active ? "accent" : "dim";
 			lines.push(truncateToWidth(`${this.theme.fg(head, active ? "●" : "○")} ${this.theme.fg(head, `Subagents (${done}/${run.tasks.length})`)}`, width, "…"));
 			const allDone = TERMINAL.includes(run.status);
-			const visible = run.tasks.slice(0, MAX_TASKS);
+			const visible = run.tasks.slice(0, WIDGET_MAX_TASKS_PER_RUN);
 			visible.forEach((task, i) => {
-				const last = i === visible.length - 1 && run.tasks.length <= MAX_TASKS;
+				const last = i === visible.length - 1 && run.tasks.length <= WIDGET_MAX_TASKS_PER_RUN;
 				const conn = this.theme.fg("dim", last ? "└─" : "├─");
 				const activity =
 					!TERMINAL.includes(task.status) && task.lastActivity
@@ -287,8 +291,12 @@ class SubagentsWidget implements Component {
 					: `${statusIcon(task.status)} ${task.agent} · ${activity}${taskStatsWithUsage(task)} · ${taskTimer(task)}`;
 				lines.push(truncateToWidth(`${conn} ${line}`, width, "…"));
 			});
-			if (run.tasks.length > MAX_TASKS) lines.push(`${this.theme.fg("dim", "└─")} ${this.theme.fg("dim", `+${run.tasks.length - MAX_TASKS} more`)}`);
-		});
+			if (run.tasks.length > WIDGET_MAX_TASKS_PER_RUN) lines.push(`${this.theme.fg("dim", "└─")} ${this.theme.fg("dim", `+${run.tasks.length - WIDGET_MAX_TASKS_PER_RUN} more tasks`)}`);
+			renderedRuns += 1;
+			if (lines.length >= WIDGET_MAX_LINES) break; // keep the editor visible
+		}
+		const hiddenRuns = runs.length - renderedRuns;
+		if (hiddenRuns > 0) lines.push(`${this.theme.fg("dim", "└─")} ${this.theme.fg("dim", `+${hiddenRuns} more run${hiddenRuns > 1 ? "s" : ""}`)}`);
 		return lines;
 	}
 }
@@ -622,17 +630,16 @@ class SubagentManager {
 
 		// Inline params win; otherwise fall back to an existing agent file
 		// (~/.agents, .pi/agents, user dir). Never creates files.
-		const fileAgent = input.prompt?.trim() ? undefined : lookupAgent(task.agent, task.cwd);
-		const prompt = input.prompt?.trim() || fileAgent?.prompt;
-		const thinking = input.thinking ?? fileAgent?.thinking;
-		const baseTools = input.tools ?? (input.write ? WRITE_TOOLS : fileAgent?.tools ?? READONLY_TOOLS);
+		const prompt = input.prompt?.trim();
+		const thinking = input.thinking;
+		const baseTools = input.tools ?? (input.write ? WRITE_TOOLS : READONLY_TOOLS);
 		const tools = [...baseTools, ...(run.allowIntercom ? CHILD_TALK_TOOLS : [])];
 
 		// Model + thinking resolve against the pi model registry; a bad request
 		// fails the TASK with a helpful message, not the whole run.
 		let model: Model<Api> | undefined;
 		try {
-			model = resolveChildModel(ctx, input.model ?? fileAgent?.model);
+			model = resolveChildModel(ctx, input.model);
 			validateThinking(model, thinking);
 		} catch (err) {
 			this.updateTask(run, task, {
@@ -646,7 +653,7 @@ class SubagentManager {
 		this.updateTask(run, task, {
 			status: "starting",
 			startedAt: Date.now(),
-			model: input.model ?? fileAgent?.model,
+			model: input.model,
 			thinking,
 			tools,
 		}, ctx, onUpdate);
@@ -1000,7 +1007,7 @@ let eventSeq = 0;
 
 const TaskItem = Type.Object({
 	id: Type.Optional(Type.String({ description: "Optional stable task id" })),
-	agent: Type.String({ minLength: 1, description: "Agent name. Invent it (inline prompt below), or reuse a name from .agents/, .pi/agents/, or ~/.pi/agent/agents/ to inherit its prompt and toolset. Never creates files." }),
+	agent: Type.String({ minLength: 1, description: "Agent name you invent. Always define the agent inline: prompt (system prompt) + toolset (write: true for write access). Never create agent files." }),
 	task: Type.String({ minLength: 1, description: "Task for this agent" }),
 	prompt: Type.Optional(Type.String({ description: "System prompt defining this agent's behavior. Optional — a minimal default is used." })),
 	write: Type.Optional(Type.Boolean({ description: "true = write toolset (read, bash, edit, write); default false = read-only (read, grep, find, ls)" })),

package/src/agents.ts

@@ -1,70 +0,0 @@
-/**
- * Optional agent-file lookup — fallback for inline definitions.
- *
- * The leader defines agents inline (prompt + toolset). If a task has no inline
- * prompt, we LOOK UP a file by agent name — never create one:
- *   <cwd>/.agents/<name>.md        (nearest project dir first)
- *   <cwd>/.pi/agents/<name>.md
- *   ~/.pi/agent/agents/<name>.md   (user)
- * First match wins; explicit inline params always override file contents.
- */
-
-import * as fs from "node:fs";
-import * as path from "node:path";
-import { getAgentDir, parseFrontmatter } from "@earendil-works/pi-coding-agent";
-
-export interface FileAgent {
-	prompt: string;
-	tools?: string[];
-	model?: string;
-	thinking?: string;
-}
-
-function projectAgentDirs(cwd: string): string[] {
-	const out: string[] = [];
-	let dir = cwd;
-	for (;;) {
-		for (const sub of [".agents", path.join(".pi", "agents")]) {
-			const candidate = path.join(dir, sub);
-			try {
-				if (fs.statSync(candidate).isDirectory()) out.push(candidate);
-			} catch {
-				/* missing dir is fine */
-			}
-		}
-		const parent = path.dirname(dir);
-		if (parent === dir) break;
-		dir = parent;
-	}
-	return out;
-}
-
-export function lookupAgent(name: string, cwd: string): FileAgent | undefined {
-	const dirs = [...projectAgentDirs(cwd), path.join(getAgentDir(), "agents")];
-	for (const dir of dirs) {
-		const filePath = path.join(dir, `${name}.md`);
-		let content: string;
-		try {
-			content = fs.readFileSync(filePath, "utf-8");
-		} catch {
-			continue;
-		}
-		try {
-			const { frontmatter, body } = parseFrontmatter<Record<string, string>>(content);
-			if (!frontmatter.name || frontmatter.name !== name) continue;
-			const tools = frontmatter.tools
-				?.split(",")
-				.map((t) => t.trim())
-				.filter(Boolean);
-			return {
-				prompt: body,
-				tools: tools && tools.length > 0 ? tools : undefined,
-				model: frontmatter.model,
-				thinking: frontmatter.thinking,
-			};
-		} catch {
-			continue;
-		}
-	}
-	return undefined;
-}