@gotgenes/pi-subagents 10.0.0 → 10.1.0

package/src/observation/notification.ts

@@ -1,6 +1,6 @@
 import { debugLog } from "#src/debug";
 import { getLifetimeTotal, getSessionContextPercent } from "#src/lifecycle/usage";
-import type { AgentRecord } from "#src/types";
+import type { Agent } from "#src/types";
 import type { AgentActivityTracker } from "#src/ui/agent-activity-tracker";
 
 /** Details attached to custom notification messages for visual rendering. */
@@ -42,7 +42,7 @@ export function getStatusLabel(status: string, error?: string): string {
 }
 
 /** Format a structured task notification matching Claude Code's <task-notification> XML. */
-export function formatTaskNotification(record: AgentRecord, resultMaxLen: number): string {
+export function formatTaskNotification(record: Agent, resultMaxLen: number): string {
   const status = getStatusLabel(record.status, record.error);
   const durationMs = record.completedAt ? record.completedAt - record.startedAt : 0;
   const totalTokens = getLifetimeTotal(record.lifetimeUsage);
@@ -75,7 +75,7 @@ export function formatTaskNotification(record: AgentRecord, resultMaxLen: number
 
 /** Build notification details for the custom message renderer. */
 export function buildNotificationDetails(
-  record: AgentRecord,
+  record: Agent,
   resultMaxLen: number,
   activity?: AgentActivityTracker,
 ): NotificationDetails {
@@ -100,8 +100,8 @@ export function buildNotificationDetails(
   };
 }
 
-/** Build event data for lifecycle events from an AgentRecord. */
-export function buildEventData(record: AgentRecord) {
+/** Build event data for lifecycle events from an Agent. */
+export function buildEventData(record: Agent) {
   const durationMs = record.completedAt ? record.completedAt - record.startedAt : Date.now() - record.startedAt;
   const u = record.lifetimeUsage;
   const total = getLifetimeTotal(u);
@@ -126,7 +126,7 @@ export function buildEventData(record: AgentRecord) {
 
 export interface NotificationSystem {
   cancelNudge: (key: string) => void;
-  sendCompletion: (record: AgentRecord) => void;
+  sendCompletion: (record: Agent) => void;
   cleanupCompleted: (id: string) => void;
   dispose: () => void;
 }
@@ -154,7 +154,7 @@ export class NotificationManager implements NotificationSystem {
     }
   }
 
-  sendCompletion(record: AgentRecord): void {
+  sendCompletion(record: Agent): void {
     this.agentActivity.delete(record.id);
     this.markFinished(record.id);
     this.scheduleNudge(record.id, () => this.emitIndividualNudge(record));
@@ -187,7 +187,7 @@ export class NotificationManager implements NotificationSystem {
     );
   }
 
-  private emitIndividualNudge(record: AgentRecord): void {
+  private emitIndividualNudge(record: Agent): void {
     if (record.notification?.resultConsumed) return;
 
     const notification = formatTaskNotification(record, 500);

package/src/observation/record-observer.ts

@@ -1,16 +1,16 @@
 /**
- * record-observer.ts — Subscribes to session events and updates AgentRecord stats.
+ * record-observer.ts — Subscribes to session events and updates Agent stats.
  *
  * Replaces the scattered callback-wrapping logic in AgentManager's startAgent()
  * and resume() with a single direct subscription.
  */
 
+import type { Agent } from "#src/lifecycle/agent";
 import type { CompactionInfo } from "#src/lifecycle/agent-manager";
-import type { AgentRecord } from "#src/lifecycle/agent-record";
 import type { SubscribableSession } from "#src/types";
 
-export interface RecordObserverOptions {
-  onCompact?: (record: AgentRecord, info: CompactionInfo) => void;
+export interface AgentObserverOptions {
+  onCompact?: (record: Agent, info: CompactionInfo) => void;
 }
 
 /**
@@ -23,10 +23,10 @@ export interface RecordObserverOptions {
  *
  * @returns An unsubscribe function.
  */
-export function subscribeRecordObserver(
+export function subscribeAgentObserver(
   session: SubscribableSession,
-  record: AgentRecord,
-  options?: RecordObserverOptions,
+  record: Agent,
+  options?: AgentObserverOptions,
 ): () => void {
   return session.subscribe((event) => {
     if (event.type === "tool_execution_end") {

package/src/service/service-adapter.ts

@@ -8,17 +8,16 @@
 import type { ParentSnapshot } from "#src/lifecycle/parent-snapshot";
 import type { SpawnOptions, SubagentRecord, SubagentsService } from "#src/service/service";
 import type { ModelRegistry } from "#src/session/model-resolver";
-import type { AgentRecord, SessionContext } from "#src/types";
+import type { Agent, SessionContext } from "#src/types";
 
 /** Narrow interface for the AgentManager — avoids coupling to the concrete class. */
 export interface AgentManagerLike {
   spawn(snapshot: ParentSnapshot, type: string, prompt: string, options: unknown): string;
-  getRecord(id: string): AgentRecord | undefined;
-  listAgents(): AgentRecord[];
+  getRecord(id: string): Agent | undefined;
+  listAgents(): Agent[];
   abort(id: string): boolean;
   waitForAll(): Promise<void>;
   hasRunning(): boolean;
-  queueSteer(id: string, message: string): boolean;
 }
 
 /**
@@ -93,8 +92,9 @@ export class SubagentsServiceAdapter implements SubagentsService {
     }
     const session = record.session;
     if (!session) {
-      // Session not ready yet — queue via manager for delivery once initialized
-      return this.manager.queueSteer(id, message);
+      // Session not ready yet — buffer on the agent for delivery once initialized
+      record.queueSteer(message);
+      return true;
     }
     await session.steer(message);
     return true;
@@ -110,10 +110,10 @@ export class SubagentsServiceAdapter implements SubagentsService {
 }
 
 /**
- * Convert an internal AgentRecord to a serializable SubagentRecord.
+ * Convert an internal Agent to a serializable SubagentRecord.
  * Uses an explicit allowlist — new fields must be opted in.
  */
-export function toSubagentRecord(record: AgentRecord): SubagentRecord {
+export function toSubagentRecord(record: Agent): SubagentRecord {
   const out: SubagentRecord = {
     id: record.id,
     type: record.type,

package/src/session/session-config.ts

@@ -18,19 +18,6 @@ import type { AgentPromptConfig, SubagentType, ThinkingLevel } from "#src/types"
 
 // ── Public interfaces ────────────────────────────────────────────────────────
 
-/**
- * Tool filtering configuration — consumed by `filterActiveTools` in `agent-runner.ts`.
- *
- * 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[];
-  /** Resolved extensions setting: true = inherit all, false = none. */
-  extensions: boolean;
-}
-
 /**
  * IO collaborators injected into `assembleSessionConfig`.
  *
@@ -98,8 +85,10 @@ export interface SessionConfig {
   effectiveCwd: string;
   /** Fully-assembled system prompt string (ready for `systemPromptOverride`). */
   systemPrompt: string;
-  /** Tool filtering cluster — tool allowlist, denylist, and extensions setting. */
-  toolFilter: ToolFilterConfig;
+  /** Built-in tool name allowlist for this agent type. */
+  toolNames: string[];
+  /** Resolved extensions setting: true = inherit all, false = none. */
+  extensions: boolean;
   /**
    * Resolved model instance (undefined → use parent model as passed to SDK).
    * Opaque handle — the assembler passes it through without inspection.
@@ -222,7 +211,8 @@ export function assembleSessionConfig(
   return {
     effectiveCwd,
     systemPrompt,
-    toolFilter: { toolNames, extensions },
+    toolNames,
+    extensions,
     model,
     thinkingLevel,
     noSkills,

package/src/tools/agent-tool.ts

@@ -11,7 +11,7 @@ import { runForeground } from "#src/tools/foreground-runner";
 import { buildDetails, buildTypeListText, textResult } from "#src/tools/helpers";
 import { renderAgentResult } from "#src/tools/result-renderer";
 import { type ModelInfo, resolveSpawnConfig } from "#src/tools/spawn-config";
-import type { AgentRecord } from "#src/types";
+import type { Agent } from "#src/types";
 import { AgentActivityTracker } from "#src/ui/agent-activity-tracker";
 import { type UICtx } from "#src/ui/agent-widget";
 import { type AgentDetails, getDisplayName } from "#src/ui/display";
@@ -33,9 +33,9 @@ export interface AgentActivityAccess {
 /** Narrow manager interface — only the methods the Agent tool calls. */
 export interface AgentToolManager {
 	spawn: (snapshot: ParentSnapshot, type: string, prompt: string, opts: AgentSpawnConfig) => string;
-	spawnAndWait: (snapshot: ParentSnapshot, type: string, prompt: string, opts: Omit<AgentSpawnConfig, "isBackground">) => Promise<AgentRecord>;
-	resume: (id: string, prompt: string, signal: AbortSignal) => Promise<AgentRecord | undefined>;
-	getRecord: (id: string) => AgentRecord | undefined;
+	spawnAndWait: (snapshot: ParentSnapshot, type: string, prompt: string, opts: Omit<AgentSpawnConfig, "isBackground">) => Promise<Agent>;
+	resume: (id: string, prompt: string, signal: AbortSignal) => Promise<Agent | undefined>;
+	getRecord: (id: string) => Agent | undefined;
 }
 
 /** Narrow runtime interface — the Agent tool's slice of SubagentRuntime. */

package/src/tools/background-spawner.ts

@@ -3,14 +3,14 @@ import type { ParentSnapshot } from "#src/lifecycle/parent-snapshot";
 import type { AgentActivityAccess } from "#src/tools/agent-tool";
 import { textResult } from "#src/tools/helpers";
 import type { ResolvedSpawnConfig } from "#src/tools/spawn-config";
-import type { AgentRecord } from "#src/types";
+import type { Agent } from "#src/types";
 import { AgentActivityTracker } from "#src/ui/agent-activity-tracker";
 import { subscribeUIObserver } from "#src/ui/ui-observer";
 
 /** Narrow manager interface for the background spawner. */
 export interface BackgroundManagerDeps {
   spawn(snapshot: ParentSnapshot, type: string, prompt: string, opts: AgentSpawnConfig): string;
-  getRecord(id: string): AgentRecord | undefined;
+  getRecord(id: string): Agent | undefined;
 }
 
 /** Narrow widget interface for the background spawner. */

package/src/tools/foreground-runner.ts

@@ -9,7 +9,7 @@ import {
   textResult,
 } from "#src/tools/helpers";
 import type { ResolvedSpawnConfig } from "#src/tools/spawn-config";
-import type { AgentRecord } from "#src/types";
+import type { Agent } from "#src/types";
 import { AgentActivityTracker } from "#src/ui/agent-activity-tracker";
 import {
   type AgentDetails,
@@ -26,7 +26,7 @@ export interface ForegroundManagerDeps {
     type: string,
     prompt: string,
     opts: Omit<AgentSpawnConfig, "isBackground">,
-  ): Promise<AgentRecord>;
+  ): Promise<Agent>;
 }
 
 /** Narrow widget interface for the foreground runner. */
@@ -62,7 +62,7 @@ export async function runForeground(
 
   const fgState = new AgentActivityTracker(execution.effectiveMaxTurns);
   let unsubUI: (() => void) | undefined;
-  let recordRef: AgentRecord | undefined;
+  let recordRef: Agent | undefined;
 
   const streamUpdate = () => {
     const toolUses = recordRef?.toolUses ?? 0;
@@ -92,7 +92,7 @@ export async function runForeground(
 
   streamUpdate();
 
-  let record: AgentRecord;
+  let record: Agent;
   try {
     record = await manager.spawnAndWait(
       params.snapshot,

package/src/tools/get-result-tool.ts

@@ -4,13 +4,13 @@ import type { AgentConfigLookup } from "#src/config/agent-types";
 import { getAgentConversation } from "#src/lifecycle/agent-runner";
 import { getSessionContextPercent } from "#src/lifecycle/usage";
 import { formatLifetimeTokens, textResult } from "#src/tools/helpers";
-import type { AgentRecord } from "#src/types";
+import type { Agent } from "#src/types";
 import { formatDuration, getDisplayName } from "#src/ui/display";
 
 // ---- Deps interfaces ----
 
 export interface GetResultToolManager {
-	getRecord(id: string): AgentRecord | undefined;
+	getRecord(id: string): Agent | undefined;
 }
 
 export interface GetResultToolNotifications {

package/src/tools/steer-tool.ts

@@ -2,13 +2,12 @@ import { defineTool } from "@earendil-works/pi-coding-agent";
 import { Type } from "@sinclair/typebox";
 import { getSessionContextPercent } from "#src/lifecycle/usage";
 import { formatLifetimeTokens, textResult } from "#src/tools/helpers";
-import type { AgentRecord } from "#src/types";
+import type { Agent } from "#src/types";
 
 // ---- Deps interfaces ----
 
 export interface SteerToolManager {
-	getRecord(id: string): AgentRecord | undefined;
-	queueSteer(id: string, message: string): boolean;
+	getRecord(id: string): Agent | undefined;
 }
 
 export interface SteerToolEvents {
@@ -43,8 +42,8 @@ export class SteerTool {
 		}
 		const session = record.session;
 		if (!session) {
-			// Session not ready yet — queue via manager for delivery once initialized
-			this.manager.queueSteer(record.id, params.message);
+			// Session not ready yet — buffer on the agent for delivery once initialized
+			record.queueSteer(params.message);
 			this.events.emit("subagents:steered", { id: record.id, message: params.message });
 			return textResult(
 				`Steering message queued for agent ${record.id}. It will be delivered once the session initializes.`,

package/src/types.ts

@@ -7,7 +7,7 @@ import type { AgentSessionEvent } from "@earendil-works/pi-coding-agent";
 import type { ModelRegistry } from "#src/session/model-resolver";
 
 
-export { AgentRecord } from "#src/lifecycle/agent-record";
+export { Agent } from "#src/lifecycle/agent";
 export type { AgentSessionEvent, ThinkingLevel };
 
 /**

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

@@ -9,7 +9,7 @@ import { join } from "node:path";
 
 import { BUILTIN_TOOL_NAMES } from "#src/config/agent-types";
 import type { ParentSnapshot } from "#src/lifecycle/parent-snapshot";
-import type { AgentRecord } from "#src/types";
+import type { Agent } from "#src/types";
 import type { AgentFileOps } from "#src/ui/agent-file-ops";
 import { writeAgentFile } from "#src/ui/agent-file-writer";
 import type { MenuUI } from "#src/ui/agent-menu";
@@ -23,7 +23,7 @@ export interface WizardManager {
     type: string,
     prompt: string,
     opts: { description: string; maxTurns: number },
-  ) => Promise<AgentRecord>;
+  ) => Promise<Agent>;
 }
 
 /** Narrow registry interface for reloading after creation. */

package/src/ui/agent-menu.ts

@@ -4,7 +4,7 @@ import { AgentTypeRegistry } from "#src/config/agent-types";
 import type { ParentSnapshot } from "#src/lifecycle/parent-snapshot";
 import { type ModelRegistry, resolveModel } from "#src/session/model-resolver";
 import { getModelLabelFromConfig } from "#src/tools/helpers";
-import type { AgentConfig, AgentRecord } from "#src/types";
+import type { Agent, AgentConfig } from "#src/types";
 import type { AgentActivityTracker } from "#src/ui/agent-activity-tracker";
 import { AgentConfigEditor } from "#src/ui/agent-config-editor";
 import { AgentCreationWizard } from "#src/ui/agent-creation-wizard";
@@ -15,15 +15,15 @@ import { formatDuration, getDisplayName } from "#src/ui/display";
 
 /** Narrow manager interface for menu operations. */
 export interface AgentMenuManager {
-  listAgents: () => AgentRecord[];
-  getRecord: (id: string) => AgentRecord | undefined;
+  listAgents: () => Agent[];
+  getRecord: (id: string) => Agent | undefined;
   /** Used by generate wizard to spawn an agent that writes the .md file. */
   spawnAndWait: (
     parentSnapshot: ParentSnapshot,
     type: string,
     prompt: string,
     opts: { description: string; maxTurns: number },
-  ) => Promise<AgentRecord>;
+  ) => Promise<Agent>;
 }
 
 /** Narrow settings interface required by the agent menu. */
@@ -251,7 +251,7 @@ export class AgentsMenuHandler {
     await this.showRunningAgents(ui);
   }
 
-  private async viewAgentConversation(ui: MenuUI, record: AgentRecord): Promise<void> {
+  private async viewAgentConversation(ui: MenuUI, record: Agent): Promise<void> {
     const session = record.session;
     if (!session) {
       ui.notify(

package/src/ui/conversation-viewer.ts

@@ -9,7 +9,7 @@ import type { AgentSession } from "@earendil-works/pi-coding-agent";
 import { type Component, matchesKey, type TUI, truncateToWidth, visibleWidth } from "@earendil-works/pi-tui";
 import type { AgentConfigLookup } from "#src/config/agent-types";
 import { getLifetimeTotal, getSessionContextPercent } from "#src/lifecycle/usage";
-import type { AgentRecord } from "#src/types";
+import type { Agent } from "#src/types";
 import type { AgentActivityTracker } from "#src/ui/agent-activity-tracker";
 import { buildInvocationTags, formatDuration, formatSessionTokens, getDisplayName, getPromptModeLabel, type Theme } from "#src/ui/display";
 import { formatMessage, formatStreamingIndicator } from "#src/ui/message-formatters";
@@ -25,7 +25,7 @@ export const VIEWPORT_HEIGHT_PCT = 70;
 export interface ConversationViewerOptions {
   tui: TUI;
   session: AgentSession;
-  record: AgentRecord;
+  record: Agent;
   activity: AgentActivityTracker | undefined;
   theme: Theme;
   done: (result: undefined) => void;
@@ -42,7 +42,7 @@ export class ConversationViewer implements Component {
 
   private tui: TUI;
   private session: AgentSession;
-  private record: AgentRecord;
+  private record: Agent;
   private activity: AgentActivityTracker | undefined;
   private theme: Theme;
   private done: (result: undefined) => void;