@fitchmultz/pi-subagents 0.46.0 → 0.46.2

package/src/agents/agents.ts

@@ -40,7 +40,7 @@ import {
 } from "./builtin-overrides.ts";
 import { loadConfiguredPackageAgents } from "./package-agents.ts";
 import { parseChain, parseJsonChain } from "./chain-serializer.ts";
-import { mergeAgentsForScope } from "./agent-selection.ts";
+import { mergeAgents } from "./agent-selection.ts";
 import { errorMessage } from "./config-values.ts";
 export { buildRuntimeName, frontmatterNameForConfig, parsePackageName } from "./identity.ts";
 
@@ -124,17 +124,26 @@ function loadChainsFromDir(
   }
   return { chains, diagnostics };
 }
-function discoverChains(
-  userDir: string,
-  projectDirs: readonly string[],
-  scope: AgentScope,
-): { chains: ChainConfig[]; chainDiagnostics: ChainDiscoveryDiagnostic[] } {
-  const projectChains = (scope === "user" ? [] : projectDirs).map((dir) =>
+export function discoverChains(
+  cwd: string,
+  options: AgentDiscoveryOptions = {},
+  scope: AgentScope = "both",
+): {
+  chains: ChainConfig[];
+  chainDiagnostics: ChainDiscoveryDiagnostic[];
+  userChainDir: string;
+  projectChainDir: string | null;
+} {
+  const userChainDir = path.join(getAgentDir(), "chains");
+  const project = resolveNearestProjectChainDirs(cwd, options);
+  const projectChains = (scope === "user" ? [] : project.readDirs).map((dir) =>
     loadChainsFromDir(dir, "project"),
   );
   const userChains =
-    scope === "project" ? { chains: [], diagnostics: [] } : loadChainsFromDir(userDir, "user");
+    scope === "project" ? { chains: [], diagnostics: [] } : loadChainsFromDir(userChainDir, "user");
   return {
+    userChainDir,
+    projectChainDir: project.preferredDir,
     chains: [...userChains.chains, ...projectChains.flatMap((entry) => entry.chains)],
     chainDiagnostics: [
       ...userChains.diagnostics,
@@ -160,7 +169,7 @@ export function discoverAgents(
   const paths = profilePaths(cwd, options);
   const profiles = discoverLocalProfiles(paths, scope);
   const packages = loadConfiguredPackageAgents(cwd, scope, options);
-  const agents = mergeAgentsForScope(scope, profiles.user, profiles.project, [
+  const agents = mergeAgents(profiles.user, profiles.project, [
     ...profiles.builtin,
     ...packages.agents,
   ]).filter((agent) => agent.disabled !== true);
@@ -186,10 +195,8 @@ export function discoverAgentsAll(
   projectSettingsPath: string | null;
 } {
   const paths = profilePaths(cwd, options);
-  const userChainDir = path.join(getAgentDir(), "chains");
-  const projectChainPaths = resolveNearestProjectChainDirs(cwd, options);
   const profiles = discoverLocalProfiles(paths, scope);
-  const chains = discoverChains(userChainDir, projectChainPaths.readDirs, scope);
+  const chains = discoverChains(cwd, options, scope);
   const packages = loadConfiguredPackageAgents(cwd, scope, options);
   return {
     ...profiles,
@@ -201,8 +208,6 @@ export function discoverAgentsAll(
     ],
     userDir: paths.userDirs.at(0) ?? path.join(getAgentDir(), "agents"),
     projectDir: paths.projectDir,
-    userChainDir,
-    projectChainDir: projectChainPaths.preferredDir,
     userSettingsPath: paths.userSettingsPath,
     projectSettingsPath: paths.projectSettingsPath,
   };

package/src/extension/index.ts

@@ -1,18 +1,25 @@
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
-import { SUBAGENT_CHILD_ENV } from "../runs/shared/pi-args.ts";
-import { ensureTempRoot } from "../shared/temp-root.ts";
+// Keep deferred imports on Pi's host-peer mapping path in production-only installs.
+// oxlint-disable-next-line import/no-unassigned-import
+import "@earendil-works/pi-coding-agent";
+import { SUBAGENT_CHILD_ENV } from "../runs/shared/child-route-env.ts";
 import { errorMessage } from "../shared/unknown.ts";
 import { loadConfig } from "./config.ts";
-import { ParentSubagentRuntime } from "./parent-runtime.ts";
-import { cleanupStaleRuntime } from "./runtime-reload.ts";
 
 export { loadConfig } from "./config.ts";
 
-export default function registerSubagentExtension(pi: ExtensionAPI): void {
+export default async function registerSubagentExtension(pi: ExtensionAPI): Promise<void> {
   if (process.env[SUBAGENT_CHILD_ENV] === "1") {
     // Explicit fanout-child owns nested delegation; globally discovered parent code remains inert.
     return;
   }
+  // Children discover this entry too; keep the parent execution/history/UI graph behind the guard.
+  const [{ ensureTempRoot }, { ParentSubagentRuntime }, { cleanupStaleRuntime }] =
+    await Promise.all([
+      import("../shared/temp-root.ts"),
+      import("./parent-runtime.ts"),
+      import("./runtime-reload.ts"),
+    ]);
   try {
     ensureTempRoot();
   } catch (error) {

package/src/extension/result-snapshot.ts

@@ -1,32 +1,6 @@
-import type {
-  AgentProgress,
-  Details,
-  SingleResult,
-  SubagentExecutionResult,
-} from "../shared/types.ts";
+import type { SubagentExecutionResult } from "../shared/types.ts";
 import type { ReadonlyInput } from "../shared/types/inputs.ts";
 
-/** A new result owns its progress buffers; native messages and accounting stay shared snapshots. */
-export function copyProgress(progress: ReadonlyInput<AgentProgress>): AgentProgress {
-  return {
-    ...progress,
-    recentTools: progress.recentTools.map((tool) => ({ ...tool })),
-    recentOutput: [...progress.recentOutput],
-  };
-}
-
-export function copyResult(result: ReadonlyInput<SingleResult>): SingleResult {
-  return { ...result, progress: result.progress ? copyProgress(result.progress) : undefined };
-}
-
-export function copyDetails(details: ReadonlyInput<Details>): Details {
-  return {
-    ...details,
-    results: details.results.map(copyResult),
-    progress: details.progress?.map(copyProgress),
-  };
-}
-
 /** Preserve the native result envelope while giving the receiving tool its own content array. */
 export function copyExecutionResult(
   result: ReadonlyInput<SubagentExecutionResult>,

package/src/history/ingest.ts

@@ -9,7 +9,7 @@ import { AnswerTexts } from "./answer-texts.ts";
 import { object } from "./values.ts";
 import { identity, stamp, openSource, requireSource } from "./source-file.ts";
 export { identity, stamp } from "./source-file.ts";
-import { number, text } from "./rows.ts";
+import { number, text, type Row } from "./rows.ts";
 import { stageEntry, publishTools, type IngestRecord } from "./entry-ingestion.ts";
 export { previewProjection, previewLimits, compactEntry } from "./preview.ts";
 
@@ -173,7 +173,6 @@ export class SourceIngest {
       });
       return;
     }
-    this.stagedEntry = stageEntry(this.store, this.source, record);
     this.textCursor = 0;
     this.phase = "text";
   }
@@ -235,64 +234,73 @@ export class SourceIngest {
   }
   private indexText(): void {
     const record = this.record;
-    const rowid = this.stagedEntry;
-    if (!record || rowid === undefined) {
-      throw new HistoryIndexError("CORRUPT", "Missing staged entry.");
+    if (!record) {
+      throw new HistoryIndexError("CORRUPT", "Missing ingestion record.");
     }
     const chunks = this.store.all(
-      "SELECT rowid,* FROM pending_text WHERE rowid>? ORDER BY rowid LIMIT 32",
+      "SELECT rowid,* FROM pending_text WHERE rowid>? ORDER BY rowid LIMIT 33",
       this.textCursor,
     );
-    if (chunks.length > 0) {
-      this.store.transaction(() => {
-        for (const chunk of chunks) {
-          if (!allowed(record.value, text(chunk, "field"))) {
-            continue;
-          }
-          const body = safeText(text(chunk, "text"), 1_048_576);
-          const id = this.store.run(
-            "INSERT INTO documents(entry_rowid,field,text_start,text_end,preview) VALUES (?,?,?,?,?)",
-            rowid,
-            text(chunk, "field"),
-            number(chunk, "position"),
-            number(chunk, "end_position"),
-            safeText(body, 512),
-          ).lastInsertRowid;
-          this.store.run("INSERT INTO corpus(rowid,text) VALUES (?,?)", id, body);
-        }
-      });
-      const last = chunks.at(-1);
-      if (last) {
-        this.textCursor = number(last, "rowid");
+    const complete = chunks.length <= 32;
+    const batch = chunks.slice(0, 32);
+    let rowid: number | undefined;
+    const write = () => {
+      rowid = this.stagedEntry ?? stageEntry(this.store, this.source, record);
+      this.indexChunks(rowid, record, batch);
+      if (complete) {
+        this.publishEntry(rowid, record);
       }
-      return;
+    };
+    if (complete) {
+      this.published(record.end, write);
+      this.stagedEntry = undefined;
+      this.phase = "read";
+    } else {
+      this.store.transaction(write);
+      this.stagedEntry = rowid;
+      this.textCursor = number(batch[31], "rowid");
     }
+  }
+  private indexChunks(rowid: number, record: IngestRecord, chunks: readonly Row[]): void {
+    for (const chunk of chunks) {
+      if (!allowed(record.value, text(chunk, "field"))) {
+        continue;
+      }
+      const body = safeText(text(chunk, "text"), 1_048_576);
+      const id = this.store.run(
+        "INSERT INTO documents(entry_rowid,field,text_start,text_end,preview) VALUES (?,?,?,?,?)",
+        rowid,
+        text(chunk, "field"),
+        number(chunk, "position"),
+        number(chunk, "end_position"),
+        safeText(body, 512),
+      ).lastInsertRowid;
+      this.store.run("INSERT INTO corpus(rowid,text) VALUES (?,?)", id, body);
+    }
+  }
+  private publishEntry(rowid: number, record: IngestRecord): void {
     const value = record.value;
     const answers = this.answerTexts.fingerprints(
       value,
       typeof value.id === "string" ? value.id : `legacy-${record.start}`,
     );
-    this.published(record.end, () => {
-      if (
-        this.store.run(
-          "UPDATE entries SET published=1 WHERE rowid=? AND source_id=? AND generation=? AND published=0",
-          rowid,
-          this.id,
-          this.source.generation,
-        ).changes !== 1
-      ) {
-        throw new HistoryIndexError(
-          "SOURCE_CHANGED",
-          "Staged entry disappeared before publication; reconcile before advancing the source.",
-        );
-      }
-      for (const [digest, item] of answers) {
-        this.store.run("INSERT OR IGNORE INTO answers VALUES (?,?,?)", rowid, digest, item);
-      }
-      publishTools(this.store, this.source, rowid, value);
-    });
-    this.stagedEntry = undefined;
-    this.phase = "read";
+    if (
+      this.store.run(
+        "UPDATE entries SET published=1 WHERE rowid=? AND source_id=? AND generation=? AND published=0",
+        rowid,
+        this.id,
+        this.source.generation,
+      ).changes !== 1
+    ) {
+      throw new HistoryIndexError(
+        "SOURCE_CHANGED",
+        "Staged entry disappeared before publication; reconcile before advancing the source.",
+      );
+    }
+    for (const [digest, item] of answers) {
+      this.store.run("INSERT OR IGNORE INTO answers VALUES (?,?,?)", rowid, digest, item);
+    }
+    publishTools(this.store, this.source, rowid, value);
   }
   /** Each pump reads <=64 KiB or indexes <=32 chunks, then returns to the IPC event loop. */
   step(): boolean {

package/src/history/prune.ts

@@ -30,7 +30,7 @@ export function pruneHistory(
   return store.transaction(() => {
     let removed = previousSources === undefined && pruneRuns(store, runs);
     for (const source of store.all(
-      `SELECT id,path FROM sources WHERE id NOT IN (SELECT source_id FROM children WHERE source_id IS NOT NULL)${previousSources ? ` AND id IN (${previousSources.map(() => "?").join(",")})` : ""}`,
+      `SELECT id,path FROM sources WHERE NOT EXISTS (SELECT 1 FROM children WHERE children.source_id=sources.id)${previousSources ? ` AND id IN (${previousSources.map(() => "?").join(",")})` : ""}`,
       ...(previousSources ?? []),
     )) {
       const id = text(source, "id");

package/src/history/run-projection.ts

@@ -147,10 +147,8 @@ export function projectRun(
   foreground: Readonly<ReadonlyMap<string, ReadonlyForegroundResumeRun>>,
 ): HistoryRunRow {
   const state: OwnedRunReadState = { ownedRuns: runs, foregroundRuns: foreground };
-  const questions = listRunQuestions(getRunMetadataDir(run.runId)).filter(
-    (question) =>
-      question.ownerSessionId === ownerSessionId &&
-      ["awaiting_input", "answer_pending"].includes(question.state),
+  const questions = listRunQuestions(getRunMetadataDir(run.runId), ownerSessionId).filter(
+    (question) => ["awaiting_input", "answer_pending"].includes(question.state),
   );
   return {
     ...ownedRunView(run, state, {

package/src/history/search-queries.ts

@@ -93,7 +93,8 @@ function plan(
     clauses.push("c.agent=?");
     params.push(input.agent);
   }
-  // Scope physical records once before aggregation; bm25 still uses the whole corpus.
+  // Materialized IDs reject common out-of-scope postings without per-hit entry lookups.
+  // Each source is scoped once; bm25 still uses the whole corpus.
   const eligible =
     hasText(input.runId) || hasText(input.agent)
       ? `eligible AS MATERIALIZED (SELECT e.rowid FROM entries e JOIN sources s ON s.id=e.source_id WHERE e.published=1 AND e.generation=s.generation AND e.source_id IN (SELECT c.source_id FROM children c WHERE ${clauses.slice(4).join(" AND ")})),`
@@ -166,6 +167,12 @@ export class SearchQueries {
     limit: number,
   ): readonly Row[] {
     // Match every token across a record's visible windows; a quoted phrase cannot span fields.
+    // Relevance benefits from hit-driven attribution. Newest keeps the native join order:
+    // forcing hit order regresses newest sorting across many shared attempts.
+    const joined =
+      score === "matched.score"
+        ? "FROM matched CROSS JOIN entries e ON e.rowid=matched.entry_rowid CROSS JOIN sources s ON s.id=e.source_id CROSS JOIN children c ON c.source_id=s.id JOIN excerpts ON excerpts.entry_rowid=matched.entry_rowid AND excerpts.choice=1 JOIN documents d ON d.id=excerpts.id"
+        : "FROM matched JOIN excerpts ON excerpts.entry_rowid=matched.entry_rowid AND excerpts.choice=1 JOIN documents d ON d.id=excerpts.id JOIN entries e ON e.rowid=matched.entry_rowid JOIN sources s ON s.id=e.source_id JOIN children c ON c.source_id=s.id";
     // Ranking/filters precede LIMIT. The parent owns the hard deadline for a costly FTS scan.
     return this.scope.store.all(
       `WITH ${filter.eligible} hits AS MATERIALIZED (
@@ -173,7 +180,7 @@ export class SearchQueries {
     ), terms AS (SELECT entry_rowid,term,MIN(score) AS score FROM hits GROUP BY entry_rowid,term),
     matched AS (SELECT entry_rowid,SUM(score) AS score FROM terms GROUP BY entry_rowid HAVING COUNT(*)=${expression.length}),
     excerpts AS (SELECT id,entry_rowid,ROW_NUMBER() OVER (PARTITION BY entry_rowid ORDER BY score,id) AS choice FROM hits)
-    SELECT d.*,${score} AS score,e.*,d.id AS document_id,s.path,s.session_id,c.run_id,c.child_index,c.agent,d.preview AS document_preview FROM matched JOIN excerpts ON excerpts.entry_rowid=matched.entry_rowid AND excerpts.choice=1 JOIN documents d ON d.id=excerpts.id JOIN entries e ON e.rowid=matched.entry_rowid JOIN sources s ON s.id=e.source_id JOIN children c ON c.source_id=s.id LEFT JOIN entries terminal ON terminal.source_id=s.id AND terminal.generation=s.generation AND terminal.id=c.terminal_entry_id AND terminal.published=1 WHERE ${filter.clauses.join(" AND ")} ORDER BY score,d.id,c.run_id,c.child_index LIMIT ?`,
+    SELECT d.*,${score} AS score,e.*,d.id AS document_id,s.path,s.session_id,c.run_id,c.child_index,c.agent,d.preview AS document_preview ${joined} LEFT JOIN entries terminal ON terminal.source_id=s.id AND terminal.generation=s.generation AND terminal.id=c.terminal_entry_id AND terminal.published=1 WHERE ${filter.clauses.join(" AND ")} ORDER BY score,d.id,c.run_id,c.child_index LIMIT ?`,
       ...filter.eligibleParams,
       ...filter.params,
       limit + 1,

package/src/history/store.ts

@@ -1,7 +1,7 @@
 import * as path from "node:path";
 import { openGeneration } from "./database-generation.ts";
 import { OutputDigests } from "./output-digests.ts";
-import type { DatabaseSync } from "node:sqlite";
+import type { DatabaseSync, StatementSync } from "node:sqlite";
 import { historyTransaction, openHistoryDatabase } from "./database.ts";
 import { hash, safeText, hasText } from "./text.ts";
 export { hash, exactTextDigest, safeText, integer, pageLimit } from "./text.ts";
@@ -45,6 +45,7 @@ export class HistoryStore {
     this.counters[operation] += amount;
   }
   private ingestLock?: DatabaseSync;
+  private readonly statements = new Map<string, StatementSync>();
   private readonly outputDigests = new OutputDigests();
   constructor(agentDir: string, owner: string) {
     this.owner = owner;
@@ -72,12 +73,11 @@ export class HistoryStore {
     return row;
   }
   get(sql: string, ...params: readonly IndexInput[]): Row | undefined {
-    const row = this.db.prepare(sql).get(...params);
+    const row = this.statement(sql).get(...params);
     return row === undefined ? undefined : indexRow(row);
   }
   all(sql: string, ...params: readonly IndexInput[]): readonly Row[] {
-    return this.db
-      .prepare(sql)
+    return this.statement(sql)
       .all(...params)
       .map(indexRow);
   }
@@ -85,7 +85,22 @@ export class HistoryStore {
     sql: string,
     ...params: readonly IndexInput[]
   ): ReturnType<ReturnType<DatabaseSync["prepare"]>["run"]> {
-    return this.db.prepare(sql).run(...params);
+    return this.statement(sql).run(...params);
+  }
+  private statement(sql: string): StatementSync {
+    let statement = this.statements.get(sql);
+    if (!statement) {
+      statement = this.db.prepare(sql);
+      // Dynamic browse/filter shapes cannot grow this connection's cache indefinitely.
+      if (this.statements.size >= 128) {
+        const oldest = this.statements.keys().next().value;
+        if (oldest !== undefined) {
+          this.statements.delete(oldest);
+        }
+      }
+      this.statements.set(sql, statement);
+    }
+    return statement;
   }
   deleteDocuments(sourceId: string, unpublishedOnly = false): void {
     const where = `entry_rowid IN (SELECT rowid FROM entries WHERE source_id=?${unpublishedOnly ? " AND published=0" : ""})`;
@@ -279,6 +294,7 @@ export class HistoryStore {
     }
   }
   close(): void {
+    this.statements.clear();
     this.ingestLock?.close();
     this.db.close();
   }

package/src/history/text-chunks.ts

@@ -171,9 +171,14 @@ export class TextChunks {
       this.fields.clear();
     }
     if (this.staged.length > 0) {
-      const insert = store.db.prepare("INSERT INTO pending_text VALUES (?,?,?,?)");
       for (const item of this.staged) {
-        insert.run(item.field, item.start, item.end, item.text);
+        store.run(
+          "INSERT INTO pending_text VALUES (?,?,?,?)",
+          item.field,
+          item.start,
+          item.end,
+          item.text,
+        );
       }
       this.staged = [];
     }

package/src/history/transcript-queries.ts

@@ -58,7 +58,7 @@ export class TranscriptQueries {
     boundary: Boundary,
     ids: readonly string[],
   ): readonly (readonly [string, boolean])[] {
-    const delivered: (readonly [string, boolean])[] = [];
+    const humans: (readonly [string, number])[] = [];
     const where = boundary.clauses.join(" AND ");
     for (const id of new Set(ids)) {
       const human = this.scope.store.get(
@@ -67,17 +67,23 @@ export class TranscriptQueries {
         id,
       );
       if (human) {
-        delivered.push([
-          id,
-          this.scope.store.get(
-            `SELECT rowid FROM entries WHERE ${where} AND start>? AND assistant_text=1 LIMIT 1`,
-            ...boundary.params,
-            number(human, "start"),
-          ) !== undefined,
-        ]);
+        humans.push([id, number(human, "start")]);
       }
     }
-    return delivered;
+    if (humans.length === 0) {
+      return [];
+    }
+    // One tail lookup answers every ID, without scanning before the earliest requested human.
+    const assistant = this.scope.store.get(
+      `SELECT start FROM entries WHERE ${where} AND start>? AND assistant_text=1 ORDER BY start DESC LIMIT 1`,
+      ...boundary.params,
+      Math.min(...humans.map(([, start]) => start)),
+    );
+    const assistantStart = assistant ? number(assistant, "start") : undefined;
+    return humans.map(([id, start]): readonly [string, boolean] => [
+      id,
+      assistantStart !== undefined && start < assistantStart,
+    ]);
   }
   private readMetadata(
     boundary: Boundary,

package/src/intercom/result-intercom.ts

@@ -1,5 +1,4 @@
 import type { ReadonlyInput } from "../shared/types/inputs.ts";
-import { copyDetails, copyResult } from "../extension/result-snapshot.ts";
 import { compactNestedResultChildren, formatNestedResultLines } from "./nested-result-tree.ts";
 export {
   compactNestedResultChildren,
@@ -13,7 +12,6 @@ import * as fs from "node:fs";
 import { formatRunAction } from "../shared/status-format.ts";
 import { SUBAGENT_CHILD_ENV, SUBAGENT_FANOUT_CHILD_ENV } from "../runs/shared/pi-args.ts";
 import type {
-  Details,
   SingleResult,
   SubagentResultIntercomChild,
   SubagentResultIntercomPayload,
@@ -308,104 +306,3 @@ export function buildSubagentResultIntercomPayload(
   };
   return { ...payload, message: formatSubagentResultIntercomMessage(payload) };
 }
-
-function stripSingleResultOutputs(result: ReadonlyInput<SingleResult>): SingleResult {
-  return {
-    ...copyResult(result),
-    messages: undefined,
-    finalOutput: undefined,
-    truncation: undefined,
-  };
-}
-
-export function stripDetailsOutputsForIntercomReceipt(
-  details: ReadonlyInput<Details>,
-  delivery?: Details["intercomDelivery"],
-): Details {
-  return {
-    ...copyDetails(details),
-    results: details.results.map(stripSingleResultOutputs),
-    ...(delivery ? { intercomDelivery: delivery } : {}),
-  };
-}
-
-function compactReceiptSummary(summary: string): string {
-  const withoutOutput = summary.split(/\n\nOutput:\n|\sOutput:\s/)[0] ?? summary;
-  const normalized = withoutOutput.replace(/\s+/g, " ").trim();
-  return normalized.length > 240 ? `${normalized.slice(0, 239)}…` : normalized;
-}
-
-function formatChildReferences(
-  children: readonly SubagentResultIntercomChild[],
-  field: "artifactPath" | "intercomTarget" | "sessionPath",
-  heading: string,
-): string[] {
-  const lines: string[] = [];
-  for (const child of children) {
-    const value = child[field];
-    if (typeof value === "string") {
-      lines.push(`- ${child.agent} [${child.status}]: ${value}`);
-    }
-  }
-  return lines.length > 0 ? [heading, ...lines] : [];
-}
-
-function formatReceiptReferences(payload: SubagentResultIntercomPayload): string[] {
-  const lines: string[] = [];
-  if (hasText(payload.resultPath)) {
-    lines.push(`Saved result (acceptance details when configured): ${payload.resultPath}`);
-  }
-  for (const child of payload.children) {
-    if (hasText(child.metadataPath)) {
-      lines.push(
-        `Result metadata (${child.agent}; acceptance details when configured): ${child.metadataPath}${fs.existsSync(child.metadataPath) ? "" : " (missing)"}`,
-      );
-    }
-  }
-  lines.push(
-    ...formatChildReferences(payload.children, "artifactPath", "Artifacts:"),
-    ...formatChildReferences(
-      payload.children,
-      "intercomTarget",
-      "Run intercom targets (may be inactive after completion):",
-    ),
-    ...formatChildReferences(payload.children, "sessionPath", "Sessions:"),
-  );
-  const nonCompleted = payload.children.filter((child) => child.status !== "completed");
-  if (nonCompleted.length > 0) {
-    lines.push("Non-completed children:");
-    for (const child of nonCompleted) {
-      lines.push(`- ${child.agent} [${child.status}]: ${compactReceiptSummary(child.summary)}`);
-    }
-  }
-  return lines;
-}
-
-export function formatSubagentResultReceipt(input: {
-  readonly mode: SubagentRunMode;
-  readonly runId: string;
-  readonly payload: SubagentResultIntercomPayload;
-}): string {
-  const counts = countStatuses(input.payload.children);
-  const modeLabel = {
-    single: "single subagent result",
-    parallel: "parallel subagent results",
-    chain: "chain subagent results",
-  }[input.mode];
-  const lines = [
-    `Delivered ${modeLabel} via intercom.`,
-    "Delivery: succeeded",
-    `Run: ${input.runId}`,
-    `Child outcome: ${input.payload.status}`,
-    `Children: ${formatStatusCounts(counts)}`,
-    ...(hasText(input.payload.error)
-      ? [
-          `Workflow ${input.payload.status === "paused" ? "paused" : "error"}: ${input.payload.error}`,
-        ]
-      : []),
-  ];
-
-  lines.push(...formatReceiptReferences(input.payload));
-  lines.push("Full grouped output was sent over intercom.");
-  return lines.join("\n");
-}