@gotgenes/pi-subagents 7.8.0 → 8.0.0

package/src/lifecycle/agent-runner.ts

@@ -22,7 +22,7 @@ import type { ShellExec, SubagentType, ThinkingLevel } from "#src/types";
 const EXCLUDED_TOOL_NAMES = ["Agent", "get_subagent_result", "steer_subagent"];
 
 /**
- * Filter the session's active tool names according to extension/denylist rules.
+ * Filter the session's active tool names according to extension rules.
  *
  * Run twice - once before `bindExtensions` (filters built-in tools) and once after
  * (filters extension-registered tools, which only join the active set during
@@ -36,16 +36,13 @@ function filterActiveTools(
   activeTools: string[],
   config: ToolFilterConfig,
 ): string[] {
-  const { toolNames, extensions, disallowedSet } = config;
+  const { toolNames, extensions } = config;
   if (extensions === false) {
-    // Extensions disabled: only apply the denylist to built-in tools.
-    if (!disallowedSet) return activeTools;
-    return activeTools.filter((t) => !disallowedSet.has(t));
+    return activeTools;
   }
   const builtinToolNameSet = new Set(toolNames);
   return activeTools.filter((t) => {
     if (EXCLUDED_TOOL_NAMES.includes(t)) return false;
-    if (disallowedSet?.has(t)) return false;
     if (builtinToolNameSet.has(t)) return true;
     if (Array.isArray(extensions)) {
       return extensions.some((ext) => t.startsWith(ext) || t.includes(ext));
@@ -341,9 +338,9 @@ export async function runAgent(
   });
 
   // Filter active tools: remove our own tools to prevent nesting,
-  // apply extension allowlist if specified, and apply disallowedTools denylist.
+  // apply extension allowlist if specified.
   // First pass - over built-in tools, before bindExtensions registers extension tools.
-  if (cfg.toolFilter.extensions !== false || cfg.toolFilter.disallowedSet) {
+  if (cfg.toolFilter.extensions !== false) {
     const filtered = filterActiveTools(session.getActiveToolNames(), cfg.toolFilter);
     session.setActiveToolsByName(filtered);
   }
@@ -367,9 +364,8 @@ export async function runAgent(
   // Extension-registered tools (added during bindExtensions) are not in the
   // session's active set when the first filter pass runs above. Without this
   // re-filter, the `extensions: string[]` allowlist branch never matches any
-  // extension tools and `extensions: true` lets non-allowlisted denylist
-  // entries slip in. Run the same filter against the post-bind active set.
-  if (cfg.toolFilter.extensions !== false || cfg.toolFilter.disallowedSet) {
+  // extension tools. Run the same filter against the post-bind active set.
+  if (cfg.toolFilter.extensions !== false) {
     const refiltered = filterActiveTools(session.getActiveToolNames(), cfg.toolFilter);
     session.setActiveToolsByName(refiltered);
   }

package/src/session/session-config.ts

@@ -21,14 +21,12 @@ import type { AgentPromptConfig, SubagentType, ThinkingLevel } from "#src/types"
 /**
  * Tool filtering configuration — consumed by `filterActiveTools` in `agent-runner.ts`.
  *
- * Groups the three fields that travel together through the assembly→runner boundary:
- * the built-in tool allowlist, the optional denylist, and the extensions setting.
+ * Groups the two fields that travel together through the assembly→runner boundary:
+ * the built-in tool allowlist and the extensions setting.
  */
 export interface ToolFilterConfig {
   /** Built-in tool name allowlist for this agent type. */
   toolNames: string[];
-  /** Disallowed tool set from agentConfig. undefined when empty. */
-  disallowedSet: Set<string> | undefined;
   /** Resolved extensions setting: false | true | string[] allowlist. */
   extensions: boolean | string[];
 }
@@ -210,11 +208,6 @@ export function assembleSessionConfig(
   // tell the resource loader not to load them again.
   const noSkills = skills === false || Array.isArray(skills);
 
-  // Disallowed tools set (for filterActiveTools in runAgent)
-  const disallowedSet = agentConfig.disallowedTools
-    ? new Set(agentConfig.disallowedTools)
-    : undefined;
-
   // Model resolution: explicit option > config model string > parent model
   const model =
     options.model ??
@@ -229,7 +222,7 @@ export function assembleSessionConfig(
   return {
     effectiveCwd,
     systemPrompt,
-    toolFilter: { toolNames, disallowedSet, extensions },
+    toolFilter: { toolNames, extensions },
     model,
     thinkingLevel,
     noSkills,

package/src/types.ts

@@ -42,8 +42,6 @@ export interface AgentPromptConfig {
 /** Unified agent configuration — used for both default and user-defined agents. */
 export interface AgentConfig extends AgentIdentity, AgentPromptConfig {
   builtinToolNames?: string[];
-  /** Tool denylist — these tools are removed even if `builtinToolNames` or extensions include them. */
-  disallowedTools?: string[];
   /** true = inherit all, string[] = only listed, false = none */
   extensions: true | string[] | false;
   /** true = inherit all, string[] = only listed, false = none */

package/src/ui/agent-config-editor.ts

@@ -53,8 +53,6 @@ export function buildEjectContent(cfg: AgentConfig): string {
   if (cfg.skills === false) fmFields.push("skills: false");
   else if (Array.isArray(cfg.skills))
     fmFields.push(`skills: ${cfg.skills.join(", ")}`);
-  if (cfg.disallowedTools?.length)
-    fmFields.push(`disallowed_tools: ${cfg.disallowedTools.join(", ")}`);
   if (cfg.inheritContext) fmFields.push("inherit_context: true");
   if (cfg.runInBackground) fmFields.push("run_in_background: true");
   if (cfg.isolated) fmFields.push("isolated: true");

package/src/ui/agent-creation-wizard.ts

@@ -106,7 +106,6 @@ max_turns: <optional max agentic turns. 0 or omit for unlimited (default)>
 prompt_mode: <"replace" (body IS the full system prompt) or "append" (body is appended to default prompt). Default: replace>
 extensions: <true (inherit all MCP/extension tools), false (none), or comma-separated names. Default: true>
 skills: <true (inherit all), false (none), or comma-separated skill names to preload into prompt. Default: true>
-disallowed_tools: <comma-separated tool names to block, even if otherwise available. Omit for none>
 inherit_context: <true to fork parent conversation into agent so it sees chat history. Default: false>
 run_in_background: <true to run in background by default. Default: false>
 isolated: <true for no extension/MCP tools, only built-in tools. Default: false>