@youngjurry/pi-agents 0.7.3 → 0.7.4

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # 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.

package/README.md

@@ -134,7 +134,7 @@ nickname_candidates: [Ada, Grace]
 Review carefully and return findings with exact paths.
 ```
 
-`skills` is optional. Each named Skill must be discoverable by Pi. Its complete `SKILL.md` is loaded into that Role's child system prompt, including for tightly restricted Roles that do not expose `read` or `bash`. Relative references remain rooted at the Skill's base directory. Unknown Skill names fail explicitly instead of silently weakening the Role. Roles without `skills` retain Pi's normal progressive disclosure: the Skill catalog appears only when `read` or `bash` is active, and the child decides whether to load a matching Skill.
+`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
 

package/control.ts

@@ -23,7 +23,7 @@ import {
 	rootAgentInstructions,
 	sanitizeForkMessages,
 } from "./context.ts";
-import { discoverRoles, formatAssignedSkills, resolveRole } from "./roles.ts";
+import { discoverRoles, resolveRole, selectRoleSkills } from "./roles.ts";
 import {
 	DEFAULT_CHILD_THINKING_LEVEL,
 	DEFAULT_MAX_CONCURRENT_SUBAGENTS,
@@ -594,8 +594,7 @@ export class AgentControl {
 		assignedSkillNames?: readonly string[],
 	): Promise<DefaultResourceLoader> {
 		const selfPath = path.resolve(this.selfExtensionPath);
-		let loader: DefaultResourceLoader;
-		loader = new DefaultResourceLoader({
+		const loader = new DefaultResourceLoader({
 			cwd,
 			agentDir: getAgentDir(),
 			settingsManager,
@@ -603,11 +602,14 @@ export class AgentControl {
 				...base,
 				extensions: base.extensions.filter((extension) => path.resolve(extension.resolvedPath) !== selfPath),
 			}),
-			systemPromptOverride: (base) => [
-				base || "You are a coding agent.",
-				instructions,
-				formatAssignedSkills(assignedSkillNames, loader.getSkills().skills),
-			].filter((part) => part.trim()).join("\n\n"),
+			...(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);
@@ -1151,7 +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,
+			skills: role.skills ?? ["*"],
 			source: role.source,
 		}));
 	}

package/package.json

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

package/roles.ts

@@ -45,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[];
@@ -68,7 +73,7 @@ function loadDirectory(directory: string, source: "user" | "project"): AgentRole
 				description: frontmatter.description.trim(),
 				systemPrompt: body.trim(),
 				tools: stringList(frontmatter.tools),
-				skills: stringList(frontmatter.skills),
+				skills: optionalStringList(frontmatter.skills),
 				model: typeof frontmatter.model === "string" ? frontmatter.model.trim() : undefined,
 				thinkingLevel: thinking,
 				nicknameCandidates: stringList(frontmatter.nickname_candidates),
@@ -117,18 +122,12 @@ export function resolveRole(cwd: string, projectTrusted: boolean, name?: string)
 	return role;
 }
 
-export function formatAssignedSkills(requestedNames: readonly string[] | undefined, discoveredSkills: readonly Skill[]): string {
-	if (!requestedNames?.length) return "";
+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}`);
 	}
-	const sections = requestedNames.map((name) => {
-		const skill = skillsByName.get(name)!;
-		const instructions = fs.readFileSync(skill.filePath, "utf8").trim();
-		return `<skill>\nName: ${skill.name}\nLocation: ${skill.filePath}\nBase directory: ${skill.baseDir}\n\n${instructions}\n</skill>`;
-	});
-	return `<agent_skills>\nThe following Role-selected skills are fully loaded. Follow them when completing the task. Resolve relative paths against each skill's base directory.\n\n${sections.join("\n\n")}\n</agent_skills>`;
+	return [...new Set(requestedNames)].map((name) => skillsByName.get(name)!);
 }

package/tools.ts

@@ -88,7 +88,7 @@ function renderCollaborationResult(
 					role.model ? `model: ${role.model}` : undefined,
 					role.thinkingLevel ? `thinking: ${role.thinkingLevel}` : undefined,
 					`tools: ${role.tools?.join(", ") || "default set"}`,
-					role.skills?.length ? `skills: ${role.skills.join(", ")}` : undefined,
+					`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,7 +146,7 @@ export interface AgentRoleView {
 	model?: string;
 	thinkingLevel?: ThinkingLevel;
 	tools?: string[];
-	skills?: string[];
+	skills: string[];
 	source: "builtin" | "user" | "project";
 }