@youngjurry/pi-agents 0.7.2 → 0.7.4

package/CHANGELOG.md

@@ -1,5 +1,18 @@
 # Changelog
 
+## 0.7.4 - 2026-09-05
+
+- Match Pi's progressive Skill disclosure for sub-agent Roles instead of eagerly injecting complete `SKILL.md` files.
+- Let an omitted `skills` field inherit all discoverable Skills, an explicit list expose only those Skills, and `skills: []` expose none.
+- Preserve explicit unknown-Skill validation and report inherited Role skills as `*` through on-demand Role discovery.
+
+## 0.7.3 - 2026-09-05
+
+- Fold the full `wait_agent` status tree by default and reuse Pi's `Ctrl+O` expansion state to reveal it on demand.
+- Keep provider-facing wait results limited to newly queued mailbox notices while avoiding overwhelming TUI output.
+- Add optional Role `skills` frontmatter that injects selected complete `SKILL.md` instructions into child system prompts.
+- Report configured Role skills through on-demand Role discovery and fail clearly when a selected Skill is unavailable.
+
 ## 0.7.2 - 2026-09-02
 
 - Keep execution slots reserved until an `AgentSession` is fully settled and safe to evict.

package/README.md

@@ -125,6 +125,7 @@ Role format:
 name: reviewer
 description: Review code without editing
 tools: read, grep, find, ls, bash
+skills: [document]
 model: openai/gpt-5.4
 thinking: high
 nickname_candidates: [Ada, Grace]
@@ -133,6 +134,8 @@ nickname_candidates: [Ada, Grace]
 Review carefully and return findings with exact paths.
 ```
 
+`skills` optionally filters Pi's normal progressive Skill disclosure. Omit the field to inherit all discoverable Skills, specify names such as `skills: [document]` to expose only those Skill names/descriptions/paths, or use `skills: []` to expose none. The child still reads a matching `SKILL.md` on demand rather than placing complete Skill instructions in every prompt. As in Pi's main Agent, the catalog is shown only when `read` or `bash` is active. Unknown explicitly selected Skill names fail clearly instead of being silently ignored.
+
 ## Model configuration
 
 Global sub-agent settings live outside the installed package so updates cannot overwrite them:
@@ -177,6 +180,7 @@ The settings file is optional, but spawning requires a model from either the tas
 - Referenced legacy flat child files are migrated when their main session is resumed
 - Parents receive a compact completion notice instead of the full answer; use `list_agents(view="results")` or read the result file on demand
 - Notices to a busy agent are queued safely: `wait_agent` returns them in its own result, and any leftovers are delivered right after a successful recipient turn
+- `wait_agent` sends only newly queued mailbox notices to the model; its full status tree is folded in the TUI by default and can be toggled with `Ctrl+O`
 - Failed notice delivery is re-queued instead of silently discarded
 - Notices pending when a turn is aborted or errors are deferred to the next explicit turn without restarting the interrupted agent
 - The extension never inserts messages between an assistant tool call and its tool result, keeping session history protocol-valid for strict gateways

package/control.ts

@@ -23,7 +23,7 @@ import {
 	rootAgentInstructions,
 	sanitizeForkMessages,
 } from "./context.ts";
-import { discoverRoles, resolveRole } from "./roles.ts";
+import { discoverRoles, resolveRole, selectRoleSkills } from "./roles.ts";
 import {
 	DEFAULT_CHILD_THINKING_LEVEL,
 	DEFAULT_MAX_CONCURRENT_SUBAGENTS,
@@ -587,7 +587,12 @@ export class AgentControl {
 		await this.transcriptToolDefinitionsPromise;
 	}
 
-	private async createLoader(cwd: string, settingsManager: SettingsManager, instructions: string): Promise<DefaultResourceLoader> {
+	private async createLoader(
+		cwd: string,
+		settingsManager: SettingsManager,
+		instructions: string,
+		assignedSkillNames?: readonly string[],
+	): Promise<DefaultResourceLoader> {
 		const selfPath = path.resolve(this.selfExtensionPath);
 		const loader = new DefaultResourceLoader({
 			cwd,
@@ -597,7 +602,14 @@ export class AgentControl {
 				...base,
 				extensions: base.extensions.filter((extension) => path.resolve(extension.resolvedPath) !== selfPath),
 			}),
-			systemPromptOverride: (base) => `${base || "You are a coding agent."}\n\n${instructions}`,
+			...(assignedSkillNames === undefined ? {} : {
+				skillsOverride: (base: ReturnType<DefaultResourceLoader["getSkills"]>) => ({
+					...base,
+					skills: selectRoleSkills(assignedSkillNames, base.skills),
+				}),
+			}),
+			systemPromptOverride: (base) => [base || "You are a coding agent.", instructions]
+				.filter((part) => part.trim()).join("\n\n"),
 		});
 		await loader.reload();
 		this.captureTranscriptToolDefinitions(loader);
@@ -1141,6 +1153,7 @@ export class AgentControl {
 			model: role.model || settings.defaultModel,
 			thinkingLevel: role.thinkingLevel ?? settings.defaultThinkingLevel ?? DEFAULT_CHILD_THINKING_LEVEL,
 			tools: role.tools,
+			skills: role.skills ?? ["*"],
 			source: role.source,
 		}));
 	}
@@ -1275,7 +1288,12 @@ export class AgentControl {
 		const forkContext = this.forkContextFromSessionManager(sessionManager);
 		const role = resolveRole(this.root.cwd, this.root.ctx.isProjectTrusted(), record.role);
 		const settingsManager = SettingsManager.create(this.root.cwd, getAgentDir());
-		const loader = await this.createLoader(this.root.cwd, settingsManager, this.childInstructions(record, role.systemPrompt));
+		const loader = await this.createLoader(
+			this.root.cwd,
+			settingsManager,
+			this.childInstructions(record, role.systemPrompt),
+			role.skills,
+		);
 		const runtime = await this.getModelRuntime(this.root.ctx);
 		const model = runtime.getModel(record.modelProvider, record.modelId) || this.root.model;
 		if (!model) throw new Error(`model ${record.modelProvider}/${record.modelId} is unavailable`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@youngjurry/pi-agents",
-  "version": "0.7.2",
+  "version": "0.7.4",
   "description": "Persistent in-process Codex-style multi-agent collaboration for Pi",
   "author": "youngjurry",
   "type": "module",

package/roles.ts

@@ -1,7 +1,7 @@
 import * as fs from "node:fs";
 import * as path from "node:path";
 import type { ThinkingLevel } from "@earendil-works/pi-agent-core";
-import { CONFIG_DIR_NAME, getAgentDir, parseFrontmatter } from "@earendil-works/pi-coding-agent";
+import { CONFIG_DIR_NAME, getAgentDir, parseFrontmatter, type Skill } from "@earendil-works/pi-coding-agent";
 import type { AgentRole } from "./types.ts";
 
 const BUILTIN_ROLES: AgentRole[] = [
@@ -31,6 +31,7 @@ type RoleFrontmatter = {
 	name?: unknown;
 	description?: unknown;
 	tools?: unknown;
+	skills?: unknown;
 	model?: unknown;
 	thinking?: unknown;
 	nickname_candidates?: unknown;
@@ -44,6 +45,11 @@ function stringList(value: unknown): string[] | undefined {
 	return result.length > 0 ? result : undefined;
 }
 
+function optionalStringList(value: unknown): string[] | undefined {
+	if (value === undefined) return undefined;
+	return stringList(value) ?? [];
+}
+
 function loadDirectory(directory: string, source: "user" | "project"): AgentRole[] {
 	if (!fs.existsSync(directory)) return [];
 	let entries: fs.Dirent[];
@@ -67,6 +73,7 @@ function loadDirectory(directory: string, source: "user" | "project"): AgentRole
 				description: frontmatter.description.trim(),
 				systemPrompt: body.trim(),
 				tools: stringList(frontmatter.tools),
+				skills: optionalStringList(frontmatter.skills),
 				model: typeof frontmatter.model === "string" ? frontmatter.model.trim() : undefined,
 				thinkingLevel: thinking,
 				nicknameCandidates: stringList(frontmatter.nickname_candidates),
@@ -114,3 +121,13 @@ export function resolveRole(cwd: string, projectTrusted: boolean, name?: string)
 	}
 	return role;
 }
+
+export function selectRoleSkills(requestedNames: readonly string[], discoveredSkills: readonly Skill[]): Skill[] {
+	const skillsByName = new Map(discoveredSkills.map((skill) => [skill.name, skill]));
+	const missing = requestedNames.filter((name) => !skillsByName.has(name));
+	if (missing.length > 0) {
+		const available = [...skillsByName.keys()].sort().join(", ") || "none";
+		throw new Error(`Role references unknown skill(s): ${missing.join(", ")}. Available skills: ${available}`);
+	}
+	return [...new Set(requestedNames)].map((name) => skillsByName.get(name)!);
+}

package/tools.ts

@@ -71,7 +71,14 @@ function renderCollaborationResult(
 	}
 	const icon = data.timedOut ? theme.fg("warning", "◷") : theme.fg("success", "✓");
 	const lines = [`${icon} ${theme.fg("toolTitle", data.tool)}`];
-	for (const agent of data.targets) lines.push(`  ${theme.fg("accent", compactStatus(agent))}`);
+	if (data.tool === "wait_agent" && data.targets.length > 0 && !options.expanded) {
+		const counts = new Map<string, number>();
+		for (const agent of data.targets) counts.set(agent.status, (counts.get(agent.status) ?? 0) + 1);
+		const summary = [...counts.entries()].map(([status, count]) => `${count} ${status}`).join(" · ");
+		lines.push(`  ${theme.fg("muted", `${data.targets.length} agents hidden · ${summary} · Ctrl+O to expand`)}`);
+	} else {
+		for (const agent of data.targets) lines.push(`  ${theme.fg("accent", compactStatus(agent))}`);
+	}
 	if (data.roles?.length) {
 		lines.push(`  ${theme.fg("muted", "Roles:")} ${data.roles.map((role) => role.name).join(", ")}`);
 		if (options.expanded) {
@@ -81,6 +88,7 @@ function renderCollaborationResult(
 					role.model ? `model: ${role.model}` : undefined,
 					role.thinkingLevel ? `thinking: ${role.thinkingLevel}` : undefined,
 					`tools: ${role.tools?.join(", ") || "default set"}`,
+					`skills: ${role.skills.length > 0 ? role.skills.join(", ") : "none"}`,
 				].filter(Boolean).join(" · ");
 				lines.push(`    ${theme.fg("accent", role.name)} — ${role.description}`);
 				lines.push(`      ${theme.fg("dim", configuration)}`);

package/types.ts

@@ -146,6 +146,7 @@ export interface AgentRoleView {
 	model?: string;
 	thinkingLevel?: ThinkingLevel;
 	tools?: string[];
+	skills: string[];
 	source: "builtin" | "user" | "project";
 }
 
@@ -154,6 +155,7 @@ export interface AgentRole {
 	description: string;
 	systemPrompt: string;
 	tools?: string[];
+	skills?: string[];
 	model?: string;
 	thinkingLevel?: ThinkingLevel;
 	nicknameCandidates?: string[];