@pi-unipi/subagents 0.2.4 → 0.2.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-unipi/subagents",
-  "version": "0.2.4",
+  "version": "0.2.5",
   "description": "Subagents for UniPi — parallel execution, file locking, workflow integration",
   "type": "module",
   "main": "dist/index.js",

package/src/__tests__/badge-generation.test.ts

@@ -205,6 +205,7 @@ describe("Badge generation — event bus (CRITICAL FIX)", () => {
     const validLifecycleEvents = [
       "session_start", "session_shutdown", "input",
       "tool_call", "tool_execution_start",
+      "agent_end", "before_agent_start",
     ];
 
     // Check that pi.on() is only used with lifecycle events
@@ -228,11 +229,26 @@ describe("Badge generation — event bus (CRITICAL FIX)", () => {
 // ─── Test: Event flow ──────────────────────────────────────────────
 
 describe("Badge generation — event flow", () => {
-  it("utility emits BADGE_GENERATE_REQUEST on first input", () => {
+  it("utility emits BADGE_GENERATE_REQUEST after agent responds (deferred from input)", () => {
     const src = readSource("packages/utility/src/index.ts");
 
+    // BADGE_GENERATE_REQUEST should be emitted in agent_end handler, not input
     assert.ok(src.includes("BADGE_GENERATE_REQUEST"));
     assert.ok(src.includes('source: "input-hook"'));
+
+    // input handler should NOT emit BADGE_GENERATE_REQUEST directly
+    const inputBlock = src.match(/pi\.on\("input"[\s\S]*?(?=pi\.on\(|$)/)?.[0] ?? "";
+    assert.ok(
+      !inputBlock.includes("BADGE_GENERATE_REQUEST"),
+      "input handler should NOT emit BADGE_GENERATE_REQUEST — deferred to agent_end",
+    );
+
+    // agent_end handler should emit BADGE_GENERATE_REQUEST
+    const agentEndBlock = src.match(/pi\.on\("agent_end"[\s\S]*?(?=pi\.on\(|$)/)?.[0] ?? "";
+    assert.ok(
+      agentEndBlock.includes("BADGE_GENERATE_REQUEST"),
+      "agent_end handler should emit BADGE_GENERATE_REQUEST with full conversation context",
+    );
   });
 
   it("BADGE_GENERATE_REQUEST event is defined in core", () => {

package/src/agent-runner.ts

@@ -172,12 +172,16 @@ export async function runAgent(
   let toolNames = getToolNamesForType(type, agentConfig);
 
   // Create resource loader
+  // Respect agentConfig.extensions/skills flags: if explicitly false, skip loading.
+  // This prevents explore/work agents from loading all parent extensions.
   const agentDir = getAgentDir();
+  const skipExtensions = options.isolated || agentConfig?.extensions === false;
+  const skipSkills = options.isolated || agentConfig?.skills === false;
   const loader = new DefaultResourceLoader({
     cwd: effectiveCwd,
     agentDir,
-    noExtensions: options.isolated,
-    noSkills: options.isolated,
+    noExtensions: skipExtensions,
+    noSkills: skipSkills,
     noPromptTemplates: true,
     noThemes: true,
     noContextFiles: true,
@@ -213,15 +217,19 @@ export async function runAgent(
   });
   session.setActiveToolsByName(activeTools);
 
-  // Bind extensions
-  await session.bindExtensions({
-    onError: (err) => {
-      options.onToolActivity?.({
-        type: "end",
-        toolName: `extension-error:${err.extensionPath}`,
-      });
-    },
-  });
+  // Bind extensions — only if extensions were loaded.
+  // Skipping for agents with extensions: false avoids firing session_start
+  // on an empty extension set, preventing unnecessary MODULE_READY cascade.
+  if (!skipExtensions) {
+    await session.bindExtensions({
+      onError: (err) => {
+        options.onToolActivity?.({
+          type: "end",
+          toolName: `extension-error:${err.extensionPath}`,
+        });
+      },
+    });
+  }
 
   options.onSessionCreated?.(session);