pi-subagents-j0k3r 1.3.0 → 1.3.1

package/CHANGELOG.md

@@ -1,5 +1,19 @@
 # Changelog
 
+## 1.3.1 - 2026-07-13
+
+### Fixed
+- Fixed `/subagents` live rendering so `toolcall_delta` argument JSON is no longer displayed as assistant text before native tool cards.
+- Preserved streamed thinking output while routing only `text_delta` events into assistant text, matching Pi's main-thread rendering behavior.
+
+## 1.3.0 - 2026-07-12
+
+### Changed
+- Modularized extension composition, tools, renderers, UI, runner, and model-profile code into cohesive modules.
+- Split each registered subagent tool into a dedicated file and separated complex rendering responsibilities.
+- Reorganized monolithic tests into 24 domain-focused files with 206 passing scenarios.
+- Preserved root exports, historical deep imports, runtime behavior, tool contracts, package contents, and privacy boundaries.
+
 ## 1.2.1 - 2026-07-12
 
 ### Fixed

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-subagents-j0k3r",
-  "version": "1.3.0",
+  "version": "1.3.1",
   "description": "Installable Pi package that adds markdown-defined subagents, delegated task tools, history, and model profiles.",
   "type": "module",
   "license": "MIT",

package/src/manager.ts

@@ -175,8 +175,19 @@ export class SubagentManager {
     private onTerminalBackgroundTask?: (task: SubagentTask) => void,
   ) {}
 
-  listAgents(cwd: string) {
-    return loadSubagents(cwd).map((a) => ({ name: a.name, description: a.description, filePath: a.filePath, tools: a.tools, model: a.model, effort: a.effort }));
+  listAgents(cwd: string, ctx: any = {}) {
+    const config = readSubagentsConfig(cwd);
+    return loadSubagents(cwd).map((definition) => {
+      const profile = resolveEffectiveSubagentProfile({ agentName: definition.name, definition, config, ctx });
+      return {
+        name: definition.name,
+        description: definition.description,
+        filePath: definition.filePath,
+        tools: definition.tools,
+        model: profile.model.value,
+        effort: profile.effort.value,
+      };
+    });
   }
 
   listTasks(cwd?: string) {

package/src/render/tools/subagent-list-agents.ts

@@ -0,0 +1,45 @@
+import type { ModelRef, ThinkingEffort } from '../types.js';
+import { textComponent } from './components.js';
+
+export type ListedSubagent = {
+  name: string;
+  model?: ModelRef;
+  effort?: ThinkingEffort;
+  tools: string[];
+};
+
+function modelLabel(model?: ModelRef): string {
+  return model ? `${model.provider}/${model.id}` : 'default/current';
+}
+
+function summary(agent: ListedSubagent): string {
+  return `${agent.name} · model: ${modelLabel(agent.model)} · effort: ${agent.effort ?? 'default/current'}`;
+}
+
+export function formatSubagentList(agents: ListedSubagent[], includeTools: boolean): string {
+  if (!agents.length) return 'No subagents available.';
+  return agents.map((agent) => (
+    includeTools ? `${summary(agent)} · tools: ${agent.tools.join(', ') || 'none'}` : summary(agent)
+  )).join('\n');
+}
+
+export function renderSubagentListResult(result: any, expanded: boolean, theme: any) {
+  const agents: ListedSubagent[] = Array.isArray(result?.details?.agents) ? result.details.agents : [];
+  if (!agents.length) return textComponent('No subagents available.');
+  const dim = (text: string) => theme?.fg?.('dim', text) ?? text;
+
+  if (expanded) {
+    return textComponent(agents.flatMap((agent) => [
+      summary(agent),
+      dim(`  tools: ${agent.tools.join(', ') || 'none'}`),
+    ]).join('\n'));
+  }
+
+  const visible = agents.slice(0, 5);
+  const hidden = agents.length - visible.length;
+  return textComponent([
+    ...visible.map(summary),
+    hidden > 0 ? dim(`… ${hidden} more agents hidden`) : undefined,
+    dim('ctrl+o to expand'),
+  ].filter((line): line is string => Boolean(line)).join('\n'));
+}

package/src/render/types.ts

@@ -1 +1 @@
-export type { SubagentTask } from '../types.js';
+export type { ModelRef, SubagentTask, ThinkingEffort } from '../types.js';

package/src/runner/event-processing.ts

@@ -64,11 +64,9 @@ function formatToolCall(name: string, args: any): string {
 
 function eventTranscript(event: any): string {
   const messageEvent = event?.assistantMessageEvent;
-  const delta = messageEvent?.type !== 'thinking_delta' && typeof messageEvent?.delta === 'string'
+  const delta = messageEvent?.type === 'text_delta' && typeof messageEvent.delta === 'string'
     ? messageEvent.delta
-    : messageEvent?.type === 'text_delta' && typeof messageEvent.delta === 'string'
-      ? messageEvent.delta
-      : undefined;
+    : undefined;
   if (event?.type === 'message_update' && typeof delta === 'string') return sanitizeInteractionTransportText(delta);
 
   if (event?.type === 'tool_execution_start') {
@@ -188,11 +186,9 @@ export async function promptWithInactivity(
       }
     }
     const messageEvent = event?.assistantMessageEvent;
-    const delta = messageEvent?.type !== 'thinking_delta' && typeof messageEvent?.delta === 'string'
+    const delta = messageEvent?.type === 'text_delta' && typeof messageEvent.delta === 'string'
       ? messageEvent.delta
-      : messageEvent?.type === 'text_delta' && typeof messageEvent.delta === 'string'
-        ? messageEvent.delta
-        : undefined;
+      : undefined;
     if (event?.type === 'message_end' && event.message?.role === 'assistant') usage = addUsage(usage, event.message.usage);
     if (event?.type === 'message_update' && typeof delta === 'string') {
       output += sanitizeInteractionTransportText(delta);

package/src/runner/snapshot-builder.ts

@@ -103,12 +103,8 @@ export class ThreadSnapshotBuilder {
   update(event: any): void {
     const now = new Date().toISOString();
     const messageEvent = event?.assistantMessageEvent;
-    const textDelta = event?.type === 'message_update' && messageEvent?.type !== 'thinking_delta'
-      ? typeof messageEvent?.delta === 'string'
-        ? messageEvent.delta
-        : messageEvent?.type === 'text_delta' && typeof messageEvent.delta === 'string'
-          ? messageEvent.delta
-          : undefined
+    const textDelta = event?.type === 'message_update' && messageEvent?.type === 'text_delta' && typeof messageEvent.delta === 'string'
+      ? messageEvent.delta
       : undefined;
     const thinkingDelta = event?.type === 'message_update' && messageEvent?.type === 'thinking_delta' && typeof messageEvent.delta === 'string'
       ? messageEvent.delta

package/src/tools/subagent-list-agents.ts

@@ -1,5 +1,6 @@
 import { Type } from 'typebox';
 import type { SubagentManager } from '../manager.js';
+import { formatSubagentList, renderSubagentListResult } from '../render/tools/subagent-list-agents.js';
 import { ok, fail } from './tool-response.js';
 
 export function createSubagentListAgentsTool(manager: SubagentManager) {
@@ -10,7 +11,15 @@ export function createSubagentListAgentsTool(manager: SubagentManager) {
     promptSnippet: 'List available subagents loaded from global/project agents and subagents markdown directories.',
     parameters: Type.Object({}),
     async execute(_id: string, _params: any, _signal: any, _onUpdate: any, ctx: any) {
-      try { const agents = manager.listAgents(ctx?.cwd ?? process.cwd()); return ok(`Found ${agents.length} subagent(s).`, { agents }); } catch (e) { return fail(e); }
+      try {
+        const agents = manager.listAgents(ctx?.cwd ?? process.cwd(), ctx);
+        return ok(formatSubagentList(agents, true), { agents });
+      } catch (e) {
+        return fail(e);
+      }
+    },
+    renderResult(result: any, { expanded }: any, theme: any) {
+      return renderSubagentListResult(result, Boolean(expanded), theme);
     },
   };
 }