@tmustier/pi-agent-teams 0.5.0 → 0.5.2

package/CHANGELOG.md

@@ -1,5 +1,23 @@
 # Changelog
 
+## 0.5.2
+
+### Fixes
+
+- **Pi 0.62 metadata compatibility** — updated tool metadata wiring for recent Pi releases so teams tools continue to render the right prompt snippets/guidelines and stay compatible with current core APIs.
+- **Non-interactive exit hang** — leader polling timers now call `unref()` so print/json child sessions can exit cleanly instead of hanging after the agent finishes. This fixes subagent and other nested Pi flows that load the teams extension in the background.
+
+## 0.5.1
+
+### Features
+
+- **Automatic startup GC** — on session start, silently removes stale team directories older than 24h (fire-and-forget, never blocks). Reuses the existing `gcStaleTeamDirs()` with age + state checks. (Thanks **@RensTillmann** — #8, #30)
+- **Exit cleanup of empty team dirs** — on session shutdown, deletes the session's own team directory if it has no tasks in any namespace, no active teammates (RPC or manual), and no attach claim from another session. (Thanks **@RensTillmann** — #8, #30)
+
+### Fixes
+
+- Added `excludeTeamIds` parameter to `gcStaleTeamDirs()` to prevent startup GC from removing the current session's team (important for resumed sessions older than 24h).
+
 ## 0.5.0
 
 ### Features

package/extensions/teams/cleanup.ts

@@ -79,13 +79,15 @@ export async function gcStaleTeamDirs(opts: {
 	maxAgeMs: number;
 	repoCwd?: string;
 	dryRun?: boolean;
+	/** Team IDs to skip (e.g. the current session's team). */
+	excludeTeamIds?: ReadonlySet<string>;
 }): Promise<{
 	scanned: number;
 	removed: string[];
 	skipped: Array<{ teamId: string; reason: string }>;
 	warnings: string[];
 }> {
-	const { teamsRootDir, maxAgeMs, repoCwd, dryRun } = opts;
+	const { teamsRootDir, maxAgeMs, repoCwd, dryRun, excludeTeamIds } = opts;
 	const teamsRootAbs = path.resolve(teamsRootDir);
 	const removed: string[] = [];
 	const skipped: Array<{ teamId: string; reason: string }> = [];
@@ -103,6 +105,10 @@ export async function gcStaleTeamDirs(opts: {
 	const now = Date.now();
 
 	for (const teamId of teamEntries) {
+		if (excludeTeamIds?.has(teamId)) {
+			skipped.push({ teamId, reason: "excluded" });
+			continue;
+		}
 		const teamDir = path.join(teamsRootAbs, teamId);
 		let stat: fs.Stats;
 		try {

package/extensions/teams/leader-teams-tool.ts

@@ -191,6 +191,11 @@ export function registerTeamsTool(opts: {
 			"Optional overrides: model='<provider>/<modelId>' and thinking (off|minimal|low|medium|high|xhigh).",
 			"For governance, the user can run /team delegate on (leader restricted to coordination) or /team spawn <name> plan (worker needs plan approval).",
 		].join(" "),
+		promptSnippet: "Delegate work across teammates, inspect member status, message workers, and manage team lifecycle/tasks.",
+		promptGuidelines: [
+			"Use this tool when the user wants parallel agent work, worker coordination, or team lifecycle/task management.",
+			"Prefer member_status before interrupting or reassigning active teammates when the current state is unclear.",
+		],
 		parameters: TeamsToolParamsSchema,
 
 		async execute(_toolCallId, params: TeamsToolParamsType, signal, _onUpdate, ctx): Promise<AgentToolResult<unknown>> {

package/extensions/teams/leader.ts

@@ -9,8 +9,9 @@ import { TEAM_MAILBOX_NS, taskAssignmentPayload } from "./protocol.js";
 import { createTask, listTasks, unassignTasksForAgent, updateTask, type TeamTask } from "./task-store.js";
 import { TeammateRpc } from "./teammate-rpc.js";
 import { ensureTeamConfig, loadTeamConfig, setMemberStatus, upsertMember, type TeamConfig } from "./team-config.js";
-import { getTeamDir } from "./paths.js";
-import { heartbeatTeamAttachClaim, releaseTeamAttachClaim } from "./team-attach-claim.js";
+import { getTeamDir, getTeamsRootDir } from "./paths.js";
+import { assessAttachClaimFreshness, heartbeatTeamAttachClaim, readTeamAttachClaim, releaseTeamAttachClaim } from "./team-attach-claim.js";
+import { cleanupTeamDir, gcStaleTeamDirs } from "./cleanup.js";
 import { ensureWorktreeCwd, cleanupWorktrees } from "./worktree.js";
 import { ActivityTracker, TranscriptTracker } from "./activity-tracker.js";
 import { openInteractiveWidget } from "./teams-panel.js";
@@ -106,6 +107,29 @@ async function createSessionForTeammate(
 	}
 }
 
+/** Check if a team dir has any task files across all task-list namespaces. */
+async function teamDirHasAnyTasks(teamDir: string): Promise<boolean> {
+	const tasksDir = path.join(teamDir, "tasks");
+	let taskListDirs: string[];
+	try {
+		taskListDirs = await fs.promises.readdir(tasksDir);
+	} catch {
+		return false;
+	}
+	for (const listDir of taskListDirs) {
+		const listPath = path.join(tasksDir, listDir);
+		try {
+			const stat = await fs.promises.stat(listPath);
+			if (!stat.isDirectory()) continue;
+			const files = await fs.promises.readdir(listPath);
+			if (files.some((f) => f.endsWith(".json") && !f.startsWith("."))) return true;
+		} catch {
+			continue;
+		}
+	}
+	return false;
+}
+
 // Message parsers are shared with the worker implementation.
 export function runLeader(pi: ExtensionAPI): void {
 	const teammates = new Map<string, TeammateRpc>();
@@ -701,6 +725,15 @@ export function runLeader(pi: ExtensionAPI): void {
 			style,
 		});
 
+		// Startup GC: silently remove stale team directories from previous sessions (24h age floor).
+		void gcStaleTeamDirs({
+			teamsRootDir: getTeamsRootDir(),
+			maxAgeMs: 24 * 60 * 60 * 1000,
+			excludeTeamIds: new Set([currentTeamId]),
+		}).catch(() => {
+			// Best-effort; never block the session.
+		});
+
 		await refreshTasks();
 		renderWidget();
 
@@ -717,6 +750,8 @@ export function runLeader(pi: ExtensionAPI): void {
 				refreshInFlight = false;
 			}
 		}, 1000);
+		// Don't keep non-interactive/child pi processes alive just because leader polling exists.
+		refreshTimer.unref?.();
 
 		inboxTimer = setInterval(async () => {
 			if (isStopping) return;
@@ -728,6 +763,7 @@ export function runLeader(pi: ExtensionAPI): void {
 				inboxInFlight = false;
 			}
 		}, 700);
+		inboxTimer.unref?.();
 	});
 
 	pi.on("session_switch", async (_event, ctx) => {
@@ -787,6 +823,7 @@ export function runLeader(pi: ExtensionAPI): void {
 				refreshInFlight = false;
 			}
 		}, 1000);
+		refreshTimer.unref?.();
 
 		inboxTimer = setInterval(async () => {
 			if (isStopping) return;
@@ -798,12 +835,14 @@ export function runLeader(pi: ExtensionAPI): void {
 				inboxInFlight = false;
 			}
 		}, 700);
+		inboxTimer.unref?.();
 	});
 
 	pi.on("session_shutdown", async () => {
 		if (!currentCtx) return;
 		await releaseActiveAttachClaim(currentCtx);
 		stopLoops();
+		const hadTeammates = teammates.size > 0;
 		const strings = getTeamsStrings(style);
 		await stopAllTeammates(currentCtx, `The ${strings.teamNoun} is over`);
 
@@ -817,6 +856,30 @@ export function runLeader(pi: ExtensionAPI): void {
 			} catch {
 				// Best-effort — don't block shutdown.
 			}
+
+			// Exit cleanup: delete own team directory if it's empty.
+			// Conservative: only if no RPC teammates were active, no online workers in
+			// config (manual/tmux), no tasks in ANY namespace, and no fresh attach claim.
+			// (Dirs with completed tasks are left for the 24h startup GC — intentionally
+			// asymmetric for safety.)
+			if (!hadTeammates) {
+				try {
+					const claim = await readTeamAttachClaim(teamDir);
+					const claimIsLive = claim !== null && !assessAttachClaimFreshness(claim).isStale;
+					if (claimIsLive) {
+						// Another session is using this team — don't delete.
+					} else {
+						// Also check config for online non-lead members (manual/tmux workers).
+						const cfg = await loadTeamConfig(teamDir);
+						const hasOnlineWorkers = cfg?.members.some((m) => m.role !== "lead" && m.status === "online") ?? false;
+						if (!hasOnlineWorkers && !(await teamDirHasAnyTasks(teamDir))) {
+							await cleanupTeamDir(getTeamsRootDir(), teamDir);
+						}
+					}
+				} catch {
+					// Best-effort; never block shutdown.
+				}
+			}
 		}
 	});
 

package/extensions/teams/worker.ts

@@ -143,6 +143,11 @@ export function runWorker(pi: ExtensionAPI): void {
 		name: "team_message",
 		label: "Team Message",
 		description: "Send a message to a comrade. Use this to coordinate with peers on related tasks. Set urgent=true to interrupt their active turn (use sparingly — only for time-sensitive coordination).",
+		promptSnippet: "Send a coordination message to another teammate, optionally as an urgent interruption.",
+		promptGuidelines: [
+			"Use this tool for teammate-to-teammate coordination instead of overloading task status fields with freeform messages.",
+			"Set urgent=true only when the recipient must be interrupted before finishing their current turn.",
+		],
 		parameters: TeamMessageToolParamsSchema,
 		async execute(
 			_toolCallId,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tmustier/pi-agent-teams",
-  "version": "0.5.0",
+  "version": "0.5.2",
   "description": "Claude Code agent teams style workflow for Pi.",
   "license": "MIT",
   "author": "Thomas Mustier",
@@ -39,10 +39,10 @@
   },
   "devDependencies": {
     "@eslint/js": "^9.39.2",
-    "@mariozechner/pi-agent-core": "^0.52.8",
-    "@mariozechner/pi-ai": "^0.52.8",
-    "@mariozechner/pi-coding-agent": "^0.52.8",
-    "@mariozechner/pi-tui": "^0.52.8",
+    "@mariozechner/pi-agent-core": "^0.62.0",
+    "@mariozechner/pi-ai": "^0.62.0",
+    "@mariozechner/pi-coding-agent": "^0.62.0",
+    "@mariozechner/pi-tui": "^0.62.0",
     "@sinclair/typebox": "^0.34.48",
     "eslint": "^9.39.2",
     "tsx": "^4.20.5",

package/scripts/smoke-test.mts

@@ -554,14 +554,18 @@ console.log("\n7. Pi extension loading");
 		return typeof c === "string" ? c : undefined;
 	})();
 
+	const versionOutput = `${res.stdout ?? ""}${res.stderr ?? ""}`.trim();
+
 	if (errCode === "ENOENT") {
 		console.log("  (skipped) pi CLI not found on PATH");
 	} else if (errCode === "ETIMEDOUT") {
 		console.log("  (skipped) pi --version timed out");
 	} else if (res.status !== 0) {
 		console.log("  (skipped) pi --version returned non-zero exit code");
+	} else if (versionOutput.length === 0) {
+		console.log("  (skipped) pi --version produced no output");
 	} else {
-		assert((res.stdout ?? "").trim().length > 0, "pi --version works");
+		assert(versionOutput.length > 0, "pi --version works");
 	}
 }