pi-subagents-lite 1.4.0 → 1.4.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-subagents-lite",
-   "version": "1.4.0",
+  "version": "1.4.1",
   "description": "Lightweight sub-agents for pi — spawn specialized agents with isolated sessions, tools, and models.",
   "keywords": [
     "pi-package",

package/src/agents/agent-discovery.ts

@@ -130,11 +130,11 @@ function parseFrontmatter(
 /*  parseExtensions                                                    */
 /* ------------------------------------------------------------------ */
 
-/** Split comma-separated string, trim whitespace, and remove empty entries. */
+/** Split comma-separated string, trim whitespace, strip brackets, and remove empty entries. */
 function splitCommaList(value: string): string[] {
   return value
     .split(",")
-    .map((s) => s.trim())
+    .map((s) => s.trim().replace(/^\[|\]$/g, "").trim())
     .filter((s) => s.length > 0);
 }
 

package/src/agents/agent-runner.ts

@@ -528,23 +528,23 @@ export async function runAgent(
   const config = getConfig(type, store.agent.loadSkillsImplicitly, store.agent.loadExtensionsImplicitly);
   const agentConfig = getAgentConfig(type);
 
-  // Warn on mutual exclusion violations
-  const notify = (msg: string) => {
-    if (ctx.ui?.notify) ctx.ui.notify(`[pi-subagents] ${msg}`, "warning");
-    else console.warn(`[pi-subagents] ${msg}`);
-  };
+  // Buffer warnings during setup to avoid inserting custom_message entries
+  // between tool_use and tool_result in the session tree (causes Anthropic 400).
+  // Flushed after runTurnLoop completes.
+  const warnings: string[] = [];
+  const bufferNotify = (msg: string) => { warnings.push(msg); };
   if (agentConfig?.excludeTools && Array.isArray(agentConfig.tools)) {
-    notify(`agent "${type}": both tools and exclude_tools set — tools (whitelist) wins`);
+    bufferNotify(`agent "${type}": both tools and exclude_tools set — tools (whitelist) wins`);
   }
   if (agentConfig?.excludeExtensions && Array.isArray(agentConfig.extensions)) {
-    notify(`agent "${type}": both extensions and exclude_extensions set — extensions (whitelist) wins`);
+    bufferNotify(`agent "${type}": both extensions and exclude_extensions set — extensions (whitelist) wins`);
   }
 
   const effectiveCwd = options.cwd ?? ctx.cwd;
   const env = await detectEnv(options.pi, effectiveCwd);
 
   // Resolve system prompt mode + source prompts + context files
-  const { mode, extras: promptExtras } = resolveSystemPromptSources(ctx, effectiveCwd, notify);
+  const { mode, extras: promptExtras } = resolveSystemPromptSources(ctx, effectiveCwd, bufferNotify);
 
   const systemPrompt = buildPrompt(
     type, agentConfig, config, effectiveCwd, env,
@@ -553,7 +553,7 @@ export async function runAgent(
   const { loader, reloadAndMap } = createResourceLoader(config, agentConfig, effectiveCwd, systemPrompt);
   const { extResult } = await reloadAndMap();
   const session = await createAndConfigureSession(
-    ctx, options, agentConfig, type, effectiveCwd, loader, extResult, notify,
+    ctx, options, agentConfig, type, effectiveCwd, loader, extResult, bufferNotify,
   );
   const { unsubscribe: unsubTurns, getAborted, getTurnLimited } = wireTurnTracking(session, {
     ...options,
@@ -561,5 +561,12 @@ export async function runAgent(
   });
 
   const responseText = await runTurnLoop(session, prompt, options, unsubTurns);
+
+  // Flush buffered warnings now that tool_result is in the session tree.
+  for (const msg of warnings) {
+    if (ctx.ui?.notify) ctx.ui.notify(`[pi-subagents-lite] ${msg}`, "warning");
+    else console.warn(`[pi-subagents-lite] ${msg}`);
+  }
+
   return { responseText, session, aborted: getAborted(), turnLimited: getTurnLimited() };
 }