github-issue-tower-defence-management 2.19.1 → 2.19.2

package/src/domain/usecases/DefaultSilentSessionMessageComposer.test.ts

@@ -1,533 +0,0 @@
-import { DefaultSilentSessionMessageComposer } from './DefaultSilentSessionMessageComposer';
-import { SILENT_SESSION_REMINDER_SENTINEL } from './silentSessionReminderSentinel';
-
-describe('DefaultSilentSessionMessageComposer', () => {
-  const composer = new DefaultSilentSessionMessageComposer();
-
-  it('embeds the reminder sentinel in the main-stalled section', () => {
-    const section = composer.composeMainStalledSection(600);
-    expect(section).toContain(SILENT_SESSION_REMINDER_SENTINEL);
-  });
-
-  it('embeds the reminder sentinel in the sub-agent section', () => {
-    const subAgent = {
-      label: 'sub-process-1',
-      silentSeconds: 360,
-      runningSeconds: 1200,
-      waitingOnExternalProcess: false,
-      finishedResultUnconsumed: false,
-    };
-    const section = composer.composeSubAgentSection({
-      idleSubAgents: [subAgent],
-      longRunningSubAgents: [subAgent],
-    });
-    expect(section).toContain(SILENT_SESSION_REMINDER_SENTINEL);
-  });
-
-  it('embeds the reminder sentinel in the unconsumed-result section', () => {
-    const section = composer.composeSubAgentUnconsumedResultSection([
-      {
-        label: 'agent-aaabbbbcccc30001',
-        silentSeconds: 5220,
-        runningSeconds: 5400,
-        waitingOnExternalProcess: false,
-        finishedResultUnconsumed: true,
-      },
-    ]);
-    expect(section).toContain(SILENT_SESSION_REMINDER_SENTINEL);
-  });
-
-  it('names every finished sub-agent and the whole minutes its result has stayed unconsumed', () => {
-    const section = composer.composeSubAgentUnconsumedResultSection([
-      {
-        label: 'agent-aaabbbbcccc30001',
-        silentSeconds: 5220,
-        runningSeconds: 5400,
-        waitingOnExternalProcess: false,
-        finishedResultUnconsumed: true,
-      },
-      {
-        label: 'agent-aaabbbbcccc30002',
-        silentSeconds: 359,
-        runningSeconds: 900,
-        waitingOnExternalProcess: false,
-        finishedResultUnconsumed: true,
-      },
-    ]);
-    expect(section).toContain('This is an automated status check.');
-    expect(section).toContain(
-      'agent-aaabbbbcccc30001: result unconsumed for 87m',
-    );
-    expect(section).toContain(
-      'agent-aaabbbbcccc30002: result unconsumed for 5m',
-    );
-  });
-
-  it('renders the main-stalled message as a neutral automated status notice with the silent minutes substituted', () => {
-    const section = composer.composeMainStalledSection(600);
-    expect(section).toContain('This is an automated status check.');
-    expect(section).toContain('No output has been observed for 10 minutes.');
-    expect(section).toContain(
-      'If you are waiting on an automated process that runs without the owner',
-    );
-    expect(section).toContain('Otherwise please continue with the next step');
-    expect(section).toContain('1.');
-    expect(section).toContain('2.');
-    expect(section).toContain('3.');
-    expect(section).toContain('4.');
-  });
-
-  it('limits the no-action wait to an automated process that runs without the owner', () => {
-    const section = composer.composeMainStalledSection(600);
-    expect(section).toContain(
-      'If you are waiting on an automated process that runs without the owner - a continuous-integration run, an external API, or another process - no action is needed beyond logging one line explaining the wait.',
-    );
-    expect(section).not.toContain(
-      'If you are waiting on an external process, no action is needed',
-    );
-  });
-
-  it('asks for an owner-call when the session is waiting on something only the owner can supply', () => {
-    const section = composer.composeMainStalledSection(600);
-    expect(section).toContain(
-      'If you are waiting on something only the owner can supply - a decision, an approval, a credential, or an answer - please raise an owner-call now',
-    );
-    expect(section).toContain(
-      'that wait is resolved only after the owner has been notified',
-    );
-  });
-
-  it('states each of the four checkpoints exactly once', () => {
-    const section = composer.composeMainStalledSection(600);
-    const occurrences = (needle: string): number =>
-      section.split(needle).length - 1;
-    expect(occurrences('Keep the session task list current')).toBe(1);
-    expect(occurrences('Run independent pieces of work in parallel')).toBe(1);
-    expect(
-      occurrences(
-        'Keep a monitor in place that notices when a sub-agent has produced no output for about 5 minutes.',
-      ),
-    ).toBe(1);
-    expect(
-      occurrences(
-        'Resume the assigned work now by taking its next concrete step with a tool call',
-      ),
-    ).toBe(1);
-  });
-
-  it('requests a remaining-minutes estimate in the next output', () => {
-    const section = composer.composeMainStalledSection(600);
-    expect(section).toContain(
-      'an estimate of the remaining minutes to finish all tasks',
-    );
-  });
-
-  it('floors the silent seconds to whole minutes', () => {
-    const section = composer.composeMainStalledSection(659);
-    expect(section).toContain('No output has been observed for 10 minutes.');
-  });
-
-  it('solicits no owner-call anywhere in the main-stalled section', () => {
-    const section = composer.composeMainStalledSection(600);
-    const solicitingPhrases = [
-      'share it through a new owner-call',
-      'The owner is notified only when an owner-call is raised.',
-      'in the format documented for this session',
-      're-raise',
-      'This reminder is delivered only to sessions that have no registered unanswered owner-call.',
-      'that call was not registered',
-      'the owner has not been notified',
-      "a completion still awaits the owner's acknowledgment",
-    ];
-    for (const phrase of solicitingPhrases) {
-      expect(section).not.toContain(phrase);
-    }
-  });
-
-  it('instructs the main-stalled session to resume the assigned work with a concrete next step', () => {
-    const section = composer.composeMainStalledSection(600);
-    expect(section).toContain(
-      'Resume the assigned work now by taking its next concrete step with a tool call',
-    );
-    expect(section).toContain(
-      'report the result of that step in your next output',
-    );
-  });
-
-  it('states in the main-stalled section that a period without output is not by itself a reason to contact the owner', () => {
-    const section = composer.composeMainStalledSection(600);
-    expect(section).toContain(
-      'A period without output means the assigned work is not progressing, so it is not by itself a reason to contact the owner',
-    );
-    expect(section).toContain(
-      'please resume the work rather than sending a message about the absence of progress',
-    );
-  });
-
-  it('contains no marker-tag example, tag name, or angle bracket in the format guidance', () => {
-    const section = composer.composeMainStalledSection(600);
-    expect(section).not.toContain('marker tag');
-    expect(section).not.toContain('opening and closing pair');
-    expect(section).not.toContain('<');
-    expect(section).not.toContain('>');
-  });
-
-  it('omits the self-diagnosis guidance from the stale-owner-call section', () => {
-    const section = composer.composeMainStalledWithStaleOwnerCallSection(
-      600,
-      3600,
-    );
-    expect(section).not.toContain(
-      'This reminder is delivered only to sessions that have no registered unanswered owner-call.',
-    );
-  });
-
-  it('omits the self-diagnosis guidance from the sub-agent sections', () => {
-    const subAgent = {
-      label: 'sub-process-1',
-      silentSeconds: 360,
-      runningSeconds: 1200,
-      waitingOnExternalProcess: false,
-      finishedResultUnconsumed: false,
-    };
-    const section = composer.composeSubAgentSection({
-      idleSubAgents: [subAgent],
-      longRunningSubAgents: [subAgent],
-    });
-    expect(section).not.toContain(
-      'This reminder is delivered only to sessions that have no registered unanswered owner-call.',
-    );
-  });
-
-  it('embeds the reminder sentinel in the stale-owner-call main-stalled section', () => {
-    const section = composer.composeMainStalledWithStaleOwnerCallSection(
-      600,
-      3600,
-    );
-    expect(section).toContain(SILENT_SESSION_REMINDER_SENTINEL);
-  });
-
-  it('renders the stale-owner-call section with the silent and owner-call-age minutes substituted', () => {
-    const section = composer.composeMainStalledWithStaleOwnerCallSection(
-      659,
-      3659,
-    );
-    expect(section).toContain(
-      'No output has been observed for 10 minutes, and the owner call raised 60 minutes ago has not yet been acknowledged by the owner.',
-    );
-  });
-
-  it('instructs unconditionally to re-raise the content as a fresh self-contained owner call', () => {
-    const section = composer.composeMainStalledWithStaleOwnerCallSection(
-      600,
-      3600,
-    );
-    expect(section).toContain(
-      'Please re-raise its content as a fresh, self-contained owner call.',
-    );
-  });
-
-  it('frames every unanswered owner call kind as awaiting acknowledgment', () => {
-    const section = composer.composeMainStalledWithStaleOwnerCallSection(
-      600,
-      3600,
-    );
-    expect(section).toContain('acknowledg');
-    expect(section).toContain(
-      'a completion report, a question, or a decision request',
-    );
-    expect(section).toContain("still awaiting the owner's acknowledgment");
-  });
-
-  it('contains no not-blocked escape phrasing and no conditional re-raise phrasing', () => {
-    const section = composer.composeMainStalledWithStaleOwnerCallSection(
-      600,
-      3600,
-    );
-    expect(section).not.toContain('no longer blocked');
-    expect(section).not.toContain('continue with the next step');
-    expect(section).not.toContain('If you are still waiting');
-    expect(section).not.toContain('If you are');
-  });
-
-  it('requests a remaining-minutes estimate in the stale-owner-call section', () => {
-    const section = composer.composeMainStalledWithStaleOwnerCallSection(
-      600,
-      3600,
-    );
-    expect(section).toContain(
-      'an estimate of the remaining minutes to finish all tasks',
-    );
-  });
-
-  it('states the owner-call format guidance exactly once in the stale-owner-call section', () => {
-    const section = composer.composeMainStalledWithStaleOwnerCallSection(
-      600,
-      3600,
-    );
-    const formatOccurrences =
-      section.split('in the format documented for this session').length - 1;
-    expect(formatOccurrences).toBe(1);
-  });
-
-  it('contains no marker-tag example, tag name, or angle bracket in the stale-owner-call section', () => {
-    const section = composer.composeMainStalledWithStaleOwnerCallSection(
-      600,
-      3600,
-    );
-    expect(section).not.toContain('marker tag');
-    expect(section).not.toContain('opening and closing pair');
-    expect(section).not.toContain('<');
-    expect(section).not.toContain('>');
-  });
-
-  it('composes the stale-owner-call section distinctly from the plain main-stalled section', () => {
-    const plainSection = composer.composeMainStalledSection(600);
-    const staleSection = composer.composeMainStalledWithStaleOwnerCallSection(
-      600,
-      3600,
-    );
-    expect(staleSection).not.toBe(plainSection);
-    expect(plainSection).not.toContain('has not yet been acknowledged');
-  });
-
-  it('emits a distinct idle message for a sub-agent that is only output-idle', () => {
-    const section = composer.composeSubAgentSection({
-      idleSubAgents: [
-        {
-          label: 'sub-process-idle',
-          silentSeconds: 360,
-          runningSeconds: 60,
-          waitingOnExternalProcess: false,
-          finishedResultUnconsumed: false,
-        },
-      ],
-      longRunningSubAgents: [],
-    });
-    expect(section).toContain('sub-process-idle');
-    expect(section).toContain('no output for 6m');
-    expect(section).toContain('please restart it, hand it off, or replace it');
-    expect(section).toContain('waiting on an external dependency');
-    expect(section).not.toContain('running longer than a task usually takes');
-  });
-
-  it('frames the idle message as an automated status check with the system-measured duration', () => {
-    const section = composer.composeSubAgentSection({
-      idleSubAgents: [
-        {
-          label: 'sub-process-idle',
-          silentSeconds: 360,
-          runningSeconds: 60,
-          waitingOnExternalProcess: false,
-          finishedResultUnconsumed: false,
-        },
-      ],
-      longRunningSubAgents: [],
-    });
-    expect(section).toContain('This is an automated status check.');
-    expect(section).toContain(
-      'produced no output for about the minutes shown, measured from their last tool activity',
-    );
-    expect(section).toContain('no output for 6m');
-  });
-
-  it('tells the agent that logging one line suffices for a legitimate external wait in the idle message', () => {
-    const section = composer.composeSubAgentSection({
-      idleSubAgents: [
-        {
-          label: 'sub-process-idle',
-          silentSeconds: 360,
-          runningSeconds: 60,
-          waitingOnExternalProcess: false,
-          finishedResultUnconsumed: false,
-        },
-      ],
-      longRunningSubAgents: [],
-    });
-    expect(section).toContain(
-      'a continuous-integration run, an external API, or another process',
-    );
-    expect(section).toContain(
-      'no action is needed — please log one line noting the wait',
-    );
-  });
-
-  it('emits a distinct long-running message for a sub-agent that has only run too long', () => {
-    const section = composer.composeSubAgentSection({
-      idleSubAgents: [],
-      longRunningSubAgents: [
-        {
-          label: 'sub-process-long',
-          silentSeconds: 30,
-          runningSeconds: 1200,
-          waitingOnExternalProcess: false,
-          finishedResultUnconsumed: false,
-        },
-      ],
-    });
-    expect(section).toContain('sub-process-long');
-    expect(section).toContain('running for 20m');
-    expect(section).toContain('running longer than a task usually takes');
-    expect(section).toContain(
-      'breaking the task down, restarting it, handing it off, or replacing it',
-    );
-    expect(section).toContain(
-      'If it is progressing normally, no action is needed.',
-    );
-  });
-
-  it('does not foreground the short idle time in the long-running message', () => {
-    const section = composer.composeSubAgentSection({
-      idleSubAgents: [],
-      longRunningSubAgents: [
-        {
-          label: 'sub-process-long',
-          silentSeconds: 30,
-          runningSeconds: 1200,
-          waitingOnExternalProcess: false,
-          finishedResultUnconsumed: false,
-        },
-      ],
-    });
-    expect(section).not.toContain('no output for');
-    expect(section).not.toContain('silent for');
-  });
-
-  it('emits both distinct messages for a sub-agent matching both conditions, kept separate', () => {
-    const subAgent = {
-      label: 'sub-process-both',
-      silentSeconds: 360,
-      runningSeconds: 1200,
-      waitingOnExternalProcess: false,
-      finishedResultUnconsumed: false,
-    };
-    const section = composer.composeSubAgentSection({
-      idleSubAgents: [subAgent],
-      longRunningSubAgents: [subAgent],
-    });
-    expect(section).toContain('no output for 6m');
-    expect(section).toContain('running for 20m');
-    expect(section).toContain('running longer than a task usually takes');
-    const sentinelOccurrences =
-      section.split(SILENT_SESSION_REMINDER_SENTINEL).length - 1;
-    expect(sentinelOccurrences).toBe(2);
-  });
-
-  it('groups each sub-agent under the condition it matched', () => {
-    const section = composer.composeSubAgentSection({
-      idleSubAgents: [
-        {
-          label: 'idle-only',
-          silentSeconds: 360,
-          runningSeconds: 60,
-          waitingOnExternalProcess: false,
-          finishedResultUnconsumed: false,
-        },
-      ],
-      longRunningSubAgents: [
-        {
-          label: 'long-only',
-          silentSeconds: 10,
-          runningSeconds: 960,
-          waitingOnExternalProcess: false,
-          finishedResultUnconsumed: false,
-        },
-      ],
-    });
-    const [idleSection, longRunningSection] = section.split('\n\n');
-    expect(idleSection).toContain('idle-only');
-    expect(idleSection).not.toContain('long-only');
-    expect(longRunningSection).toContain('long-only');
-    expect(longRunningSection).not.toContain('idle-only');
-  });
-
-  it('composes default texts free of pressure- and prohibition-styled vocabulary that trips model safety classifiers', () => {
-    const flaggedPatterns = [
-      /unacceptable/i,
-      /\bnever\b/i,
-      /\b[Dd]o NOT\b/,
-      /\b[Yy]ou MUST\b/,
-      /\bwastes?\b/i,
-      /silently stalls/i,
-      /\bstall(s|ed|ing)?\b/i,
-      /raise one now/i,
-      /fire the owner-call now/i,
-      /do not wait passively/i,
-      /is prohibited/i,
-    ];
-    const subAgent = {
-      label: 'sub-process-1',
-      silentSeconds: 360,
-      runningSeconds: 1200,
-      waitingOnExternalProcess: false,
-      finishedResultUnconsumed: false,
-    };
-    const composedDefaultTexts = [
-      composer.composeMainStalledSection(600),
-      composer.composeMainStalledWithStaleOwnerCallSection(600, 3600),
-      composer.composeSubAgentSection({
-        idleSubAgents: [subAgent],
-        longRunningSubAgents: [subAgent],
-      }),
-      composer.composeSubAgentUnconsumedResultSection([subAgent]),
-    ];
-    for (const text of composedDefaultTexts) {
-      for (const pattern of flaggedPatterns) {
-        expect(text).not.toMatch(pattern);
-      }
-    }
-  });
-
-  it('composes default texts free of angle-bracket tag examples and emoji characters', () => {
-    const emojiPattern =
-      /[\u{1F000}-\u{1FAFF}\u{2190}-\u{21FF}\u{2300}-\u{23FF}\u{2600}-\u{27BF}\u{2B00}-\u{2BFF}]/u;
-    const subAgent = {
-      label: 'sub-process-1',
-      silentSeconds: 360,
-      runningSeconds: 1200,
-      waitingOnExternalProcess: false,
-      finishedResultUnconsumed: false,
-    };
-    const composedDefaultTexts = [
-      composer.composeMainStalledSection(600),
-      composer.composeMainStalledWithStaleOwnerCallSection(600, 3600),
-      composer.composeSubAgentSection({
-        idleSubAgents: [subAgent],
-        longRunningSubAgents: [subAgent],
-      }),
-      composer.composeSubAgentUnconsumedResultSection([subAgent]),
-    ];
-    for (const text of composedDefaultTexts) {
-      expect(text).not.toContain('<');
-      expect(text).not.toContain('>');
-      expect(text).not.toMatch(emojiPattern);
-      expect(text).not.toContain('\u{FE0F}');
-      expect(text).not.toContain('\u{200D}');
-      // The bracketed reminder sentinel remains allowed.
-      expect(text).toContain(SILENT_SESSION_REMINDER_SENTINEL);
-    }
-  });
-
-  it('does not contain any host-specific or internal identifiers', () => {
-    const mainSection = composer.composeMainStalledSection(600);
-    const subAgent = {
-      label: 'sub-process-1',
-      silentSeconds: 360,
-      runningSeconds: 1200,
-      waitingOnExternalProcess: false,
-      finishedResultUnconsumed: false,
-    };
-    const subSection = composer.composeSubAgentSection({
-      idleSubAgents: [subAgent],
-      longRunningSubAgents: [subAgent],
-    });
-    const unconsumedResultSection =
-      composer.composeSubAgentUnconsumedResultSection([subAgent]);
-    const combined =
-      `${mainSection}\n${subSection}\n${unconsumedResultSection}`.toLowerCase();
-    expect(combined).not.toContain('claude');
-    expect(combined).not.toContain('take ownership');
-    expect(combined).not.toContain('/home/');
-    expect(combined).not.toContain('.jsonl');
-  });
-});

package/src/domain/usecases/DefaultSilentSessionMessageComposer.ts

@@ -1,122 +0,0 @@
-import { SubAgentActivity } from '../entities/LiveSessionActivitySnapshot';
-import {
-  SilentSessionMessageComposer,
-  SubAgentStallSections,
-} from './adapter-interfaces/SilentSessionMessageComposer';
-import { SILENT_SESSION_REMINDER_SENTINEL } from './silentSessionReminderSentinel';
-
-const formatMinutes = (seconds: number): string => {
-  const minutes = Math.floor(seconds / 60);
-  return `${minutes}m`;
-};
-
-const composeIdleSubAgentSection = (
-  idleSubAgents: SubAgentActivity[],
-): string => {
-  const lines = idleSubAgents.map(
-    (subAgent) =>
-      `- ${subAgent.label}: no output for ${formatMinutes(
-        subAgent.silentSeconds,
-      )}`,
-  );
-  return [
-    `${SILENT_SESSION_REMINDER_SENTINEL} This is an automated status check. The following sub-process(es) have produced no output for about the minutes shown, measured from their last tool activity:`,
-    ...lines,
-    'For each one: if it is waiting on an external dependency (a continuous-integration run, an external API, or another process), no action is needed — please log one line noting the wait. If it appears stuck, please restart it, hand it off, or replace it.',
-  ].join('\n');
-};
-
-const composeLongRunningSubAgentSection = (
-  longRunningSubAgents: SubAgentActivity[],
-): string => {
-  const lines = longRunningSubAgents.map(
-    (subAgent) =>
-      `- ${subAgent.label}: running for ${formatMinutes(
-        subAgent.runningSeconds,
-      )}`,
-  );
-  return [
-    `${SILENT_SESSION_REMINDER_SENTINEL} This is an automated status check. The following sub-process(es) have been running longer than a task usually takes:`,
-    ...lines,
-    'If one is not making progress toward completion, consider breaking the task down, restarting it, handing it off, or replacing it. If it is progressing normally, no action is needed.',
-  ].join('\n');
-};
-
-const composeUnconsumedResultSubAgentSection = (
-  unconsumedResultSubAgents: SubAgentActivity[],
-): string => {
-  const lines = unconsumedResultSubAgents.map(
-    (subAgent) =>
-      `- ${subAgent.label}: result unconsumed for ${formatMinutes(
-        subAgent.silentSeconds,
-      )}`,
-  );
-  return [
-    `${SILENT_SESSION_REMINDER_SENTINEL} This is an automated status check. The following sub-process(es) have finished and their results have been waiting unread for about the minutes shown:`,
-    ...lines,
-    'For each one: please read its result, act on it, and then mark the sub-process as finished so it is no longer tracked as running. If you have already acted on the result, marking it finished is all that remains.',
-  ].join('\n');
-};
-
-export const composeOwnerCallFormatGuidance = (): string => {
-  return 'Please share it through a new owner-call in the format documented for this session, written to be self-contained so the owner can understand the situation from that single message.';
-};
-
-const composeMainStalledMessage = (mainSilentSeconds: number): string => {
-  const minutes = Math.floor(mainSilentSeconds / 60);
-  return [
-    `${SILENT_SESSION_REMINDER_SENTINEL} This is an automated status check. No output has been observed for ${minutes} minutes. If you are waiting on an automated process that runs without the owner - a continuous-integration run, an external API, or another process - no action is needed beyond logging one line explaining the wait. If you are waiting on something only the owner can supply - a decision, an approval, a credential, or an answer - please raise an owner-call now, because that wait is resolved only after the owner has been notified. Otherwise please continue with the next step, with these points in mind:`,
-    `1. Keep the session task list current, marking finished items as done.`,
-    `2. Run independent pieces of work in parallel across sub-agents.`,
-    `3. Keep a monitor in place that notices when a sub-agent has produced no output for about 5 minutes.`,
-    `4. Resume the assigned work now by taking its next concrete step with a tool call, and report the result of that step in your next output. A period without output means the assigned work is not progressing, so it is not by itself a reason to contact the owner; please resume the work rather than sending a message about the absence of progress.`,
-    `Please also include in your next output an estimate of the remaining minutes to finish all tasks.`,
-  ].join('\n');
-};
-
-const composeMainStalledWithStaleOwnerCallMessage = (
-  mainSilentSeconds: number,
-  unansweredOwnerCallAgeSeconds: number,
-): string => {
-  const silentMinutes = Math.floor(mainSilentSeconds / 60);
-  const ownerCallAgeMinutes = Math.floor(unansweredOwnerCallAgeSeconds / 60);
-  return [
-    `${SILENT_SESSION_REMINDER_SENTINEL} This is an automated status check. No output has been observed for ${silentMinutes} minutes, and the owner call raised ${ownerCallAgeMinutes} minutes ago has not yet been acknowledged by the owner.`,
-    `An owner call without an acknowledgment — whether it carried a completion report, a question, or a decision request — is still awaiting the owner's acknowledgment, and the earlier message may have been missed. Please re-raise its content as a fresh, self-contained owner call. ${composeOwnerCallFormatGuidance()}`,
-    `Please also include in your next output an estimate of the remaining minutes to finish all tasks.`,
-  ].join('\n');
-};
-
-export class DefaultSilentSessionMessageComposer implements SilentSessionMessageComposer {
-  composeMainStalledSection = (mainSilentSeconds: number): string => {
-    return composeMainStalledMessage(mainSilentSeconds);
-  };
-
-  composeMainStalledWithStaleOwnerCallSection = (
-    mainSilentSeconds: number,
-    unansweredOwnerCallAgeSeconds: number,
-  ): string => {
-    return composeMainStalledWithStaleOwnerCallMessage(
-      mainSilentSeconds,
-      unansweredOwnerCallAgeSeconds,
-    );
-  };
-
-  composeSubAgentSection = (stallSections: SubAgentStallSections): string => {
-    const sections: string[] = [];
-    if (stallSections.idleSubAgents.length > 0) {
-      sections.push(composeIdleSubAgentSection(stallSections.idleSubAgents));
-    }
-    if (stallSections.longRunningSubAgents.length > 0) {
-      sections.push(
-        composeLongRunningSubAgentSection(stallSections.longRunningSubAgents),
-      );
-    }
-    return sections.join('\n\n');
-  };
-
-  composeSubAgentUnconsumedResultSection = (
-    unconsumedResultSubAgents: SubAgentActivity[],
-  ): string =>
-    composeUnconsumedResultSubAgentSection(unconsumedResultSubAgents);
-}