pi-herdr-agents 1.2.1 → 1.2.2

package/CHANGELOG.md

@@ -7,11 +7,18 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 Generated by [`auto-changelog`](https://github.com/CookPete/auto-changelog).
 
-## [v1.2.1](https://github.com/giuseppecrj/pi-herdr-agents/compare/v1.2.0...v1.2.1)
+## [v1.2.2](https://github.com/giuseppecrj/pi-herdr-agents/compare/v1.2.1...v1.2.2)
+
+### Commits
+
+- fix: hide completion tool from auto-exit children [`e4cad6b`](https://github.com/giuseppecrj/pi-herdr-agents/commit/e4cad6bdbe5d1952f5bb838640493c20ab332dbd)
+
+## [v1.2.1](https://github.com/giuseppecrj/pi-herdr-agents/compare/v1.2.0...v1.2.1) - 2026-08-13
 
 ### Commits
 
 - docs: polish package readme and artwork [`908dd76`](https://github.com/giuseppecrj/pi-herdr-agents/commit/908dd76875c91871b8d91113e6d1633d5112d5b3)
+- chore: release v1.2.1 [`df87f12`](https://github.com/giuseppecrj/pi-herdr-agents/commit/df87f126ec5cad4735c441752471aa8bd724f2e9)
 
 ## [v1.2.0](https://github.com/giuseppecrj/pi-herdr-agents/compare/v1.1.0...v1.2.0) - 2026-08-13
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-herdr-agents",
-  "version": "1.2.1",
+  "version": "1.2.2",
   "description": "Asynchronous Pi subagents and approved review workflows in Herdr, with optional isolated Git worktrees",
   "keywords": [
     "pi-package",

package/pi-extension/subagents/index.ts

@@ -117,6 +117,7 @@ import {
 	persistWorktreeResult,
 	runSubagentScript,
 	writeWorktreeManifest,
+	buildSubagentToolAllowlist,
 	type WorktreeHandoff,
 	type WorktreeLaunch,
 } from "./launch.ts";
@@ -1404,32 +1405,6 @@ function updateWidget() {
  * first positional message so that /skill: args land in messages[1..] and arrive
  * as standalone prompts in the child session.
  */
-const SUBAGENT_CONTROL_TOOLS = ["caller_ping", "subagent_done"] as const;
-
-/**
- * Build the child --tools allowlist.
- *
- * Pi 0.70+ applies --tools to built-in, extension, and custom tools. If a
- * subagent definition restricts tools to e.g. "read,bash,write", the child
- * control tools from subagent-done.ts would otherwise be hidden, leaving a
- * manually resumed or user-touched subagent unable to call subagent_done.
- */
-function buildSubagentToolAllowlist(effectiveTools?: string): string | null {
-	const requested = (effectiveTools ?? "")
-		.split(",")
-		.map((tool) => tool.trim())
-		.filter(Boolean);
-
-	if (requested.length === 0) return null;
-
-	const allow = new Set(requested);
-	for (const tool of SUBAGENT_CONTROL_TOOLS) {
-		allow.add(tool);
-	}
-
-	return [...allow].join(",");
-}
-
 function buildPiPromptArgs(params: {
 	effectiveSkills?: string;
 	taskDelivery: "direct" | "artifact";

package/pi-extension/subagents/launch.ts

@@ -28,7 +28,6 @@ import {
 	type HerdrWorktreeSurface,
 } from "./terminal.ts";
 
-const SUBAGENT_CONTROL_TOOLS = ["caller_ping", "subagent_done"] as const;
 const SUBAGENTS_DIR = dirname(fileURLToPath(import.meta.url));
 
 type SubagentSessionMode = "standalone" | "lineage-only" | "fork";
@@ -550,7 +549,10 @@ function buildPiCommand(
 			shellQuote(artifacts.systemPromptFile),
 		);
 	}
-	const toolAllowlist = buildToolAllowlist(request.behavior.tools);
+	const toolAllowlist = buildSubagentToolAllowlist(
+		request.behavior.tools,
+		request.behavior.autoExit,
+	);
 	if (toolAllowlist) parts.push("--tools", shellQuote(toolAllowlist));
 	if (!request.handoff) {
 		for (const prompt of buildPromptArgs(
@@ -579,7 +581,9 @@ function buildPiCommand(
 		env.push(`PI_SUBAGENT_NAME=${shellQuote(request.name)}`);
 		if (request.agent)
 			env.push(`PI_SUBAGENT_AGENT=${shellQuote(request.agent)}`);
-		if (request.behavior.autoExit) env.push("PI_SUBAGENT_AUTO_EXIT=1");
+		env.push(
+			`PI_SUBAGENT_AUTO_EXIT=${request.behavior.autoExit ? "1" : "0"}`,
+		);
 		env.push(`PI_SUBAGENT_SESSION=${shellQuote(artifacts.sessionFile)}`);
 		env.push(`PI_SUBAGENT_ID=${shellQuote(resolved.id)}`);
 		env.push(`PI_SUBAGENT_ACTIVITY_FILE=${shellQuote(artifacts.activityFile)}`);
@@ -678,7 +682,7 @@ async function launchResumedPiSubagent(
 		`PI_SUBAGENT_SESSION=${shellQuote(request.sessionFile)}`,
 		`PI_SUBAGENT_ID=${shellQuote(id)}`,
 		`PI_SUBAGENT_ACTIVITY_FILE=${shellQuote(activityFile)}`,
-		...(autoExit ? ["PI_SUBAGENT_AUTO_EXIT=1"] : []),
+		`PI_SUBAGENT_AUTO_EXIT=${autoExit ? "1" : "0"}`,
 	];
 	const command = [
 		...env,
@@ -722,14 +726,19 @@ async function launchResumedPiSubagent(
 	};
 }
 
-function buildToolAllowlist(tools?: string): string | null {
+export function buildSubagentToolAllowlist(
+	tools?: string,
+	autoExit = false,
+): string | null {
 	const requested = (tools ?? "")
 		.split(",")
 		.map((tool) => tool.trim())
 		.filter(Boolean);
 	if (requested.length === 0) return null;
 	const allow = new Set(requested);
-	for (const tool of SUBAGENT_CONTROL_TOOLS) allow.add(tool);
+	allow.delete("subagent_done");
+	allow.add("caller_ping");
+	if (!autoExit) allow.add("subagent_done");
 	return [...allow].join(",");
 }
 

package/pi-extension/subagents/subagent-done.ts

@@ -1,7 +1,7 @@
 /**
  * Extension loaded into sub-agents.
  * - Shows agent identity + available tools as a styled widget above the editor (toggle with Ctrl+J)
- * - Provides a `subagent_done` tool for autonomous agents to self-terminate
+ * - Provides a `subagent_done` tool for interactive agents to self-terminate
  */
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 import { Box, Text } from "@earendil-works/pi-tui";
@@ -302,6 +302,8 @@ export default function (pi: ExtensionAPI) {
     },
   });
 
+  if (autoExit) return;
+
   pi.registerTool({
     name: "subagent_done",
     label: "Subagent Done",