@narumitw/pi-subagents 0.38.0 → 0.40.0

package/README.md

@@ -390,13 +390,16 @@ Built-in agents inherit the active/default Pi model instead of forcing a provide
 
 Open `/subagents`, choose **Advanced settings**, then **Agent tool permissions** in an interactive
 Pi session to edit the tools each subagent may use. The standard bounded multi-select keeps a
-one-save draft: toggles do not write until **Save changes**, Escape discards the draft, and unavailable
-configured tool names remain visible and preserved. These are user settings stored in `~/.pi/agent/pi-subagents.json` and affect future sessions.
+one-save draft: toggles do not write until **Save changes**, Escape leaves the draft without writing,
+and unavailable configured tool names remain visible and preserved. In TUI mode, type to fuzzy-search
+tool names and availability metadata; Save and Discard remain pinned below the matches. These are user
+settings stored in `~/.pi/agent/pi-subagents.json` and affect future sessions.
 
 Compatibility: a valid legacy `pi-subagents-config.json` remains readable with a warning and is never modified automatically; rename it to `pi-subagents.json`. The first subsequent settings save writes the canonical file. If both files exist, the new filename takes precedence.
 
 - Select an agent, then press Enter or Space to toggle tools.
-- Press `S` to save, or Esc to cancel and return to agent selection.
+- Choose **Save changes** to write the draft, choose **Discard draft** to abandon it, or press Esc to
+  return to agent selection without writing.
 - Save the default selection to remove a custom override and use the agent defaults again.
 - Deselect every tool and save to run that agent with no tools.
 

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@narumitw/pi-subagents",
-	"version": "0.38.0",
+	"version": "0.40.0",
 	"description": "Pi extension for delegating work to specialized isolated subagents.",
 	"type": "module",
 	"license": "MIT",
@@ -40,11 +40,11 @@
 		"@earendil-works/pi-tui": "*"
 	},
 	"devDependencies": {
-		"@biomejs/biome": "2.5.5",
-		"@earendil-works/pi-agent-core": "0.82.1",
-		"@earendil-works/pi-ai": "0.82.1",
-		"@earendil-works/pi-coding-agent": "0.82.1",
-		"@earendil-works/pi-tui": "0.82.1",
+		"@biomejs/biome": "2.5.6",
+		"@earendil-works/pi-agent-core": "0.83.0",
+		"@earendil-works/pi-ai": "0.83.0",
+		"@earendil-works/pi-coding-agent": "0.83.0",
+		"@earendil-works/pi-tui": "0.83.0",
 		"typescript": "7.0.2"
 	},
 	"repository": {

package/src/config-ui.ts

@@ -297,6 +297,7 @@ async function showSubagentManager(
 			"tool-draft": () => ({
 				kind: "multiSelect",
 				title: toolDraft ? `${safeTerminalText(toolDraft.agentName)} tools` : "Agent tools",
+				enableSearch: true,
 				lines: toolDraft
 					? [
 							`Source: ${safeTerminalText(toolDraft.agentSource)}`,
@@ -311,6 +312,7 @@ async function showSubagentManager(
 							id: name,
 							label: safeTerminalText(name),
 							description: available ? "Available tool" : "Configured tool is not currently loaded",
+							searchText: available ? "available tool" : "configured unavailable preserved",
 							selected: toolDraft?.selected.has(name) ?? false,
 							disabled: !available,
 							disabledReason: available

package/src/stateful-tool-params.ts

@@ -73,6 +73,7 @@ export type ValidatedMailboxParams =
 
 export function validateManageParams(params: unknown): ValidatedManageParams {
 	const values = parameterRecord(params, "subagent_manage");
+	assertKnownKeys("subagent_manage", values, ["action", "agentId", "includeClosed", "subtree"]);
 	const action = values.action;
 	if (
 		typeof action !== "string" ||
@@ -81,18 +82,32 @@ export function validateManageParams(params: unknown): ValidatedManageParams {
 		throw new Error(`subagent_manage action must be one of: ${MANAGE_ACTIONS.join(", ")}`);
 	}
 	if (action === "list") {
-		assertOnlyActionKeys("subagent_manage", action, values, ["action", "includeClosed"]);
 		assertOptionalBoolean("subagent_manage", action, values, "includeClosed");
-		return values as ValidatedManageParams;
+		return {
+			action,
+			...(values.includeClosed === undefined ? {} : { includeClosed: values.includeClosed }),
+		} as ValidatedManageParams;
 	}
-	assertOnlyActionKeys("subagent_manage", action, values, ["action", "agentId", "subtree"]);
 	assertRequiredString("subagent_manage", action, values, "agentId");
 	assertOptionalBoolean("subagent_manage", action, values, "subtree");
-	return values as ValidatedManageParams;
+	return {
+		action,
+		agentId: values.agentId,
+		...(values.subtree === undefined ? {} : { subtree: values.subtree }),
+	} as ValidatedManageParams;
 }
 
 export function validateMailboxParams(params: unknown): ValidatedMailboxParams {
 	const values = parameterRecord(params, "subagent_mailbox");
+	assertKnownKeys("subagent_mailbox", values, [
+		"action",
+		"agentId",
+		"message",
+		"senderId",
+		"deduplicationKey",
+		"acknowledge",
+		"limit",
+	]);
 	const action = values.action;
 	if (
 		typeof action !== "string" ||
@@ -101,13 +116,6 @@ export function validateMailboxParams(params: unknown): ValidatedMailboxParams {
 		throw new Error(`subagent_mailbox action must be one of: ${MAILBOX_ACTIONS.join(", ")}`);
 	}
 	if (action === "send") {
-		assertOnlyActionKeys("subagent_mailbox", action, values, [
-			"action",
-			"agentId",
-			"message",
-			"senderId",
-			"deduplicationKey",
-		]);
 		assertRequiredString("subagent_mailbox", action, values, "agentId");
 		assertRequiredString("subagent_mailbox", action, values, "message");
 		const message = values.message as string;
@@ -123,14 +131,16 @@ export function validateMailboxParams(params: unknown): ValidatedMailboxParams {
 				'subagent_mailbox action "send" requires deduplicationKey at most 256 characters',
 			);
 		}
-		return values as ValidatedMailboxParams;
+		return {
+			action,
+			agentId: values.agentId,
+			message,
+			...(values.senderId === undefined ? {} : { senderId: values.senderId }),
+			...(values.deduplicationKey === undefined
+				? {}
+				: { deduplicationKey: values.deduplicationKey }),
+		} as ValidatedMailboxParams;
 	}
-	assertOnlyActionKeys("subagent_mailbox", action, values, [
-		"action",
-		"agentId",
-		"acknowledge",
-		"limit",
-	]);
 	assertRequiredString("subagent_mailbox", action, values, "agentId");
 	assertOptionalBoolean("subagent_mailbox", action, values, "acknowledge");
 	if (
@@ -142,7 +152,12 @@ export function validateMailboxParams(params: unknown): ValidatedMailboxParams {
 	) {
 		throw new Error('subagent_mailbox action "read" requires limit between 1 and 20');
 	}
-	return values as ValidatedMailboxParams;
+	return {
+		action,
+		agentId: values.agentId,
+		...(values.acknowledge === undefined ? {} : { acknowledge: values.acknowledge }),
+		...(values.limit === undefined ? {} : { limit: values.limit }),
+	} as ValidatedMailboxParams;
 }
 
 function parameterRecord(params: unknown, toolName: string): Record<string, unknown> {
@@ -152,18 +167,15 @@ function parameterRecord(params: unknown, toolName: string): Record<string, unkn
 	return params as Record<string, unknown>;
 }
 
-function assertOnlyActionKeys(
+function assertKnownKeys(
 	toolName: string,
-	action: string,
 	values: Record<string, unknown>,
 	allowed: readonly string[],
 ): void {
 	const unexpected = Object.keys(values).find(
 		(key) => values[key] !== undefined && !allowed.includes(key),
 	);
-	if (unexpected) {
-		throw new Error(`${toolName} action "${action}" does not accept ${unexpected}`);
-	}
+	if (unexpected) throw new Error(`${toolName} does not accept ${unexpected}`);
 }
 
 function assertRequiredString(