@henryqw/pi-subagent 1.0.0 → 1.1.0

package/README.md

@@ -36,11 +36,13 @@ Fields:
 | --- | --- | --- |
 | `name` | yes | Role selected by Main |
 | `description` | yes | Tells Main when to use role |
-| `tools` | yes | Exact built-in and extension tool allowlist; use `[]` for none |
+| `tools` | no | Omit for Pi effective `defaultTools`; a non-empty list is an exact built-in and extension tool allowlist; use `[]` for none |
 | `extensions` | no | Absolute/user-home paths or package sources passed to Pi `--extension` |
 | `skills` | no | Effective Pi Skill names loaded for role |
 | Markdown body | yes | Role system instructions |
 
+Omitted `tools` uses Pi's effective `defaultTools` for built-ins. Tools registered by Role extensions stay active without listing their names. A non-empty `tools` list strictly allowlists both built-in and extension tools; `tools: []` sends `--no-tools`.
+
 String lists may also use comma-separated text, matching Pi's example role files. Repository-relative extension paths are rejected: child working directory is delegated project, so relative paths could load untrusted project code. Use absolute paths, `~/...`, or explicit package sources such as `npm:...`.
 
 Skill entries use Pi Skill names, normally Skill directory names, not filesystem paths. At delegation time, package resolves names from Main's effective Pi Skill registry and passes matching files to child. Missing or unavailable Skills produce warning and are skipped; they do not block delegation. This preserves Main's trust and Skill collision decisions.

package/extensions/subagent.ts

@@ -26,7 +26,7 @@ type Config = { models: Partial<Record<ModelClass, ConfiguredModel>> };
 type Role = {
 	name: string;
 	description: string;
-	tools: string[];
+	tools?: string[];
 	extensions: string[];
 	skills: string[];
 	systemPrompt: string;
@@ -161,7 +161,7 @@ export function loadRoles(agentDir = getAgentDir()): Role[] {
 			return {
 				name: cleanText(frontmatter.name, "name", file),
 				description: cleanText(frontmatter.description, "description", file),
-				tools: stringList(frontmatter.tools, "tools", file, true),
+				tools: frontmatter.tools === undefined ? undefined : stringList(frontmatter.tools, "tools", file, true),
 				extensions: extensionList(frontmatter.extensions, file),
 				skills: stringList(frontmatter.skills, "skills", file),
 				systemPrompt: cleanText(parsed.body, "system prompt", file),
@@ -691,8 +691,10 @@ export default function subagentExtension(pi: ExtensionAPI): void {
 				const args = ["--mode", "json", "-p", "--no-session", "--no-extensions", "--no-skills"];
 				for (const extension of role.extensions) args.push("--extension", extension);
 				for (const skill of resolvedSkills.paths) args.push("--skill", skill);
-				if (role.tools.length) args.push("--tools", role.tools.join(","));
-				else args.push("--no-tools");
+				if (role.tools !== undefined) {
+					if (role.tools.length) args.push("--tools", role.tools.join(","));
+					else args.push("--no-tools");
+				}
 				args.push("--model", modelReferenceValue);
 				if (thinkingLevel) args.push("--thinking", thinkingLevel);
 				args.push(ctx.isProjectTrusted() ? "--approve" : "--no-approve");

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@henryqw/pi-subagent",
-  "version": "1.0.0",
+  "version": "1.1.0",
   "description": "Delegate one task to an isolated Pi role with explicit extensions and skills.",
   "keywords": [
     "pi-package",
@@ -24,9 +24,9 @@
     "pack:check": "npm pack --dry-run"
   },
   "peerDependencies": {
-    "@earendil-works/pi-ai": "^0.84.1",
-    "@earendil-works/pi-coding-agent": "^0.84.1",
-    "@earendil-works/pi-tui": "^0.84.1",
+    "@earendil-works/pi-ai": "^0.84.2",
+    "@earendil-works/pi-coding-agent": "^0.84.2",
+    "@earendil-works/pi-tui": "^0.84.2",
     "typebox": "^1.3.7"
   },
   "repository": {