@arhen/pi-core-subagent 1.0.9 → 1.1.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@arhen/pi-core-subagent",
-  "version": "1.0.9",
+  "version": "1.1.1",
   "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";
 
@@ -631,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, {
@@ -655,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);
@@ -1009,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)" })),
@@ -1135,7 +1133,19 @@ export default function (pi: ExtensionAPI) {
 		renderCall(args, theme) {
 			const mode = args.chain?.length ? `chain ${args.chain.length}` : args.tasks?.length ? `parallel ${args.tasks.length}` : `single ${args.agent ?? "?"}`;
 			const flags = [args.background ? "bg" : "", args.allowIntercom ? "talk" : ""].filter(Boolean).join(" ");
-			return new Text(`${theme.fg("toolTitle", theme.bold("subagent"))} ${theme.fg("accent", mode)}${flags ? ` ${theme.fg("muted", `[${flags}]`)}` : ""}`, 0, 0);
+			// Params used, dimmed: model, thinking, toolset, per-task write count.
+			const tasks = args.tasks ?? args.chain ?? [];
+			const writeCount = tasks.filter((t: any) => t.write).length;
+			const parts: string[] = [];
+			if (args.model) parts.push(args.model);
+			if (args.thinking) parts.push(args.thinking);
+			if (args.write) parts.push("write");
+			else if (writeCount === 0) parts.push("read-only");
+			if (writeCount > 0) parts.push(`${writeCount} write`);
+			if (args.concurrency) parts.push(`c${args.concurrency}`);
+			if (args.maxRuntimeMs) parts.push(`${Math.round(args.maxRuntimeMs / 60000)}m`);
+			const params = parts.length > 0 ? `\n  ${theme.fg("dim", parts.join(" · "))}` : "";
+			return new Text(`${theme.fg("toolTitle", theme.bold("subagent"))} ${theme.fg("accent", mode)}${flags ? ` ${theme.fg("muted", `[${flags}]`)}` : ""}${params}`, 0, 0);
 		},
 		renderResult(result, { expanded }, theme) {
 			const run = result.details?.run;

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;
-}