github-issue-tower-defence-management 1.122.3 → 1.122.5

package/src/adapter/repositories/NoUnansweredOwnerCallStatusProvider.test.ts

@@ -3,12 +3,13 @@ import { NoUnansweredOwnerCallStatusProvider } from './NoUnansweredOwnerCallStat
 describe('NoUnansweredOwnerCallStatusProvider', () => {
   it('always reports no session as having an unanswered owner call', async () => {
     const provider = new NoUnansweredOwnerCallStatusProvider();
-    const result = await provider.listSessionNamesWithUnansweredOwnerCall(
-      new Map([
-        ['session-a', '/transcripts/a.jsonl'],
-        ['session-b', '/transcripts/b.jsonl'],
-      ]),
-    );
+    const result =
+      await provider.listUnansweredOwnerCallEpochSecondsBySessionName(
+        new Map([
+          ['session-a', '/transcripts/a.jsonl'],
+          ['session-b', '/transcripts/b.jsonl'],
+        ]),
+      );
     expect(result.size).toBe(0);
   });
 });

package/src/adapter/repositories/NoUnansweredOwnerCallStatusProvider.ts

@@ -1,9 +1,9 @@
 import { OwnerCallStatusProvider } from '../../domain/usecases/adapter-interfaces/OwnerCallStatusProvider';
 
 export class NoUnansweredOwnerCallStatusProvider implements OwnerCallStatusProvider {
-  listSessionNamesWithUnansweredOwnerCall = (
+  listUnansweredOwnerCallEpochSecondsBySessionName = (
     _transcriptPathBySessionName: Map<string, string>,
-  ): Promise<Set<string>> => {
-    return Promise.resolve(new Set<string>());
+  ): Promise<Map<string, number>> => {
+    return Promise.resolve(new Map<string, number>());
   };
 }

package/src/adapter/repositories/ProcessListSessionSubAgentActivityRepository.test.ts

@@ -54,7 +54,12 @@ describe('ProcessListSessionSubAgentActivityRepository', () => {
     );
     const result = await repository.listSubAgentActivitiesBySessionName(['s1']);
     expect(result.get('s1')).toEqual([
-      { label: 'task-a', silentSeconds: 180, runningSeconds: 1200 },
+      {
+        label: 'task-a',
+        silentSeconds: 180,
+        runningSeconds: 1200,
+        waitingOnExternalProcess: false,
+      },
     ]);
   });
 
@@ -78,7 +83,12 @@ describe('ProcessListSessionSubAgentActivityRepository', () => {
     );
     const result = await repository.listSubAgentActivitiesBySessionName(['s1']);
     expect(result.get('s1')).toEqual([
-      { label: 's1', silentSeconds: 0, runningSeconds: 30 },
+      {
+        label: 's1',
+        silentSeconds: 0,
+        runningSeconds: 30,
+        waitingOnExternalProcess: false,
+      },
     ]);
   });
 

package/src/adapter/repositories/ProcessListSessionSubAgentActivityRepository.ts

@@ -43,6 +43,7 @@ export class ProcessListSessionSubAgentActivityRepository implements SessionSubA
         label,
         silentSeconds: this.silentSecondsResolver.resolveSilentSeconds(label),
         runningSeconds: process.elapsedSeconds,
+        waitingOnExternalProcess: false,
       };
       const existing = result.get(sessionName);
       if (existing === undefined) {

package/src/adapter/repositories/TranscriptOwnerCallStatusProvider.test.ts

@@ -129,13 +129,48 @@ describe('TranscriptOwnerCallStatusProvider', () => {
     ]);
     const provider = new TranscriptOwnerCallStatusProvider('<<OWNER_CALL>>');
 
-    const result = await provider.listSessionNamesWithUnansweredOwnerCall(
-      new Map([[sessionName, transcriptPath]]),
-    );
+    const result =
+      await provider.listUnansweredOwnerCallEpochSecondsBySessionName(
+        new Map([[sessionName, transcriptPath]]),
+      );
 
     expect(result.has(sessionName)).toBe(true);
   });
 
+  it('exposes the epoch seconds of the unanswered owner call so its age can be computed', async () => {
+    const transcriptPath = writeTranscript('workbench.jsonl', [
+      ownerReply('2026-06-27T10:00:00.000Z'),
+      assistantWithMarker('2026-06-27T10:05:00.000Z'),
+    ]);
+    const provider = new TranscriptOwnerCallStatusProvider('<<OWNER_CALL>>');
+
+    const result =
+      await provider.listUnansweredOwnerCallEpochSecondsBySessionName(
+        new Map([[sessionName, transcriptPath]]),
+      );
+
+    expect(result.get(sessionName)).toBe(
+      Math.floor(Date.parse('2026-06-27T10:05:00.000Z') / 1000),
+    );
+  });
+
+  it('exposes the epoch seconds of the latest owner call when several calls are unanswered', async () => {
+    const transcriptPath = writeTranscript('workbench.jsonl', [
+      assistantWithMarker('2026-06-27T09:00:00.000Z'),
+      assistantWithMarker('2026-06-27T10:05:00.000Z'),
+    ]);
+    const provider = new TranscriptOwnerCallStatusProvider('<<OWNER_CALL>>');
+
+    const result =
+      await provider.listUnansweredOwnerCallEpochSecondsBySessionName(
+        new Map([[sessionName, transcriptPath]]),
+      );
+
+    expect(result.get(sessionName)).toBe(
+      Math.floor(Date.parse('2026-06-27T10:05:00.000Z') / 1000),
+    );
+  });
+
   it('does not report a session as waiting when a later owner reply followed the call', async () => {
     const transcriptPath = writeTranscript('workbench.jsonl', [
       assistantWithMarker('2026-06-27T10:00:00.000Z'),
@@ -143,9 +178,10 @@ describe('TranscriptOwnerCallStatusProvider', () => {
     ]);
     const provider = new TranscriptOwnerCallStatusProvider('<<OWNER_CALL>>');
 
-    const result = await provider.listSessionNamesWithUnansweredOwnerCall(
-      new Map([[sessionName, transcriptPath]]),
-    );
+    const result =
+      await provider.listUnansweredOwnerCallEpochSecondsBySessionName(
+        new Map([[sessionName, transcriptPath]]),
+      );
 
     expect(result.has(sessionName)).toBe(false);
   });
@@ -157,9 +193,10 @@ describe('TranscriptOwnerCallStatusProvider', () => {
     ]);
     const provider = new TranscriptOwnerCallStatusProvider('<<OWNER_CALL>>');
 
-    const result = await provider.listSessionNamesWithUnansweredOwnerCall(
-      new Map([[sessionName, transcriptPath]]),
-    );
+    const result =
+      await provider.listUnansweredOwnerCallEpochSecondsBySessionName(
+        new Map([[sessionName, transcriptPath]]),
+      );
 
     expect(result.has(sessionName)).toBe(false);
   });
@@ -171,9 +208,10 @@ describe('TranscriptOwnerCallStatusProvider', () => {
     ]);
     const provider = new TranscriptOwnerCallStatusProvider('<<OWNER_CALL>>');
 
-    const result = await provider.listSessionNamesWithUnansweredOwnerCall(
-      new Map([[sessionName, transcriptPath]]),
-    );
+    const result =
+      await provider.listUnansweredOwnerCallEpochSecondsBySessionName(
+        new Map([[sessionName, transcriptPath]]),
+      );
 
     expect(result.has(sessionName)).toBe(true);
   });
@@ -185,9 +223,10 @@ describe('TranscriptOwnerCallStatusProvider', () => {
     ]);
     const provider = new TranscriptOwnerCallStatusProvider('<<OWNER_CALL>>');
 
-    const result = await provider.listSessionNamesWithUnansweredOwnerCall(
-      new Map([[sessionName, transcriptPath]]),
-    );
+    const result =
+      await provider.listUnansweredOwnerCallEpochSecondsBySessionName(
+        new Map([[sessionName, transcriptPath]]),
+      );
 
     expect(result.has(sessionName)).toBe(true);
   });
@@ -199,9 +238,10 @@ describe('TranscriptOwnerCallStatusProvider', () => {
     ]);
     const provider = new TranscriptOwnerCallStatusProvider('<<OWNER_CALL>>');
 
-    const result = await provider.listSessionNamesWithUnansweredOwnerCall(
-      new Map([[sessionName, transcriptPath]]),
-    );
+    const result =
+      await provider.listUnansweredOwnerCallEpochSecondsBySessionName(
+        new Map([[sessionName, transcriptPath]]),
+      );
 
     expect(result.has(sessionName)).toBe(true);
   });
@@ -214,9 +254,10 @@ describe('TranscriptOwnerCallStatusProvider', () => {
     ]);
     const provider = new TranscriptOwnerCallStatusProvider('<<OWNER_CALL>>');
 
-    const result = await provider.listSessionNamesWithUnansweredOwnerCall(
-      new Map([[sessionName, transcriptPath]]),
-    );
+    const result =
+      await provider.listUnansweredOwnerCallEpochSecondsBySessionName(
+        new Map([[sessionName, transcriptPath]]),
+      );
 
     expect(result.has(sessionName)).toBe(false);
   });
@@ -228,9 +269,10 @@ describe('TranscriptOwnerCallStatusProvider', () => {
     ]);
     const provider = new TranscriptOwnerCallStatusProvider('<<OWNER_CALL>>');
 
-    const result = await provider.listSessionNamesWithUnansweredOwnerCall(
-      new Map([[sessionName, transcriptPath]]),
-    );
+    const result =
+      await provider.listUnansweredOwnerCallEpochSecondsBySessionName(
+        new Map([[sessionName, transcriptPath]]),
+      );
 
     expect(result.has(sessionName)).toBe(true);
   });
@@ -242,9 +284,10 @@ describe('TranscriptOwnerCallStatusProvider', () => {
     ]);
     const provider = new TranscriptOwnerCallStatusProvider('<<OWNER_CALL>>');
 
-    const result = await provider.listSessionNamesWithUnansweredOwnerCall(
-      new Map([[sessionName, transcriptPath]]),
-    );
+    const result =
+      await provider.listUnansweredOwnerCallEpochSecondsBySessionName(
+        new Map([[sessionName, transcriptPath]]),
+      );
 
     expect(result.has(sessionName)).toBe(true);
   });
@@ -256,9 +299,10 @@ describe('TranscriptOwnerCallStatusProvider', () => {
     ]);
     const provider = new TranscriptOwnerCallStatusProvider('<<OWNER_CALL>>');
 
-    const result = await provider.listSessionNamesWithUnansweredOwnerCall(
-      new Map([[sessionName, transcriptPath]]),
-    );
+    const result =
+      await provider.listUnansweredOwnerCallEpochSecondsBySessionName(
+        new Map([[sessionName, transcriptPath]]),
+      );
 
     expect(result.has(sessionName)).toBe(false);
   });
@@ -271,9 +315,10 @@ describe('TranscriptOwnerCallStatusProvider', () => {
     ]);
     const provider = new TranscriptOwnerCallStatusProvider('<<OWNER_CALL>>');
 
-    const result = await provider.listSessionNamesWithUnansweredOwnerCall(
-      new Map([[sessionName, transcriptPath]]),
-    );
+    const result =
+      await provider.listUnansweredOwnerCallEpochSecondsBySessionName(
+        new Map([[sessionName, transcriptPath]]),
+      );
 
     expect(result.has(sessionName)).toBe(false);
   });
@@ -285,23 +330,25 @@ describe('TranscriptOwnerCallStatusProvider', () => {
     ]);
     const provider = new TranscriptOwnerCallStatusProvider('<<OWNER_CALL>>');
 
-    const result = await provider.listSessionNamesWithUnansweredOwnerCall(
-      new Map([[sessionName, transcriptPath]]),
-    );
+    const result =
+      await provider.listUnansweredOwnerCallEpochSecondsBySessionName(
+        new Map([[sessionName, transcriptPath]]),
+      );
 
     expect(result.has(sessionName)).toBe(false);
   });
 
-  it('returns an empty set when the marker is null', async () => {
+  it('returns an empty map when the marker is null', async () => {
     const transcriptPath = writeTranscript('workbench.jsonl', [
       ownerReply('2026-06-27T10:00:00.000Z'),
       assistantWithMarker('2026-06-27T10:05:00.000Z'),
     ]);
     const provider = new TranscriptOwnerCallStatusProvider(null);
 
-    const result = await provider.listSessionNamesWithUnansweredOwnerCall(
-      new Map([[sessionName, transcriptPath]]),
-    );
+    const result =
+      await provider.listUnansweredOwnerCallEpochSecondsBySessionName(
+        new Map([[sessionName, transcriptPath]]),
+      );
 
     expect(result.size).toBe(0);
   });
@@ -309,9 +356,10 @@ describe('TranscriptOwnerCallStatusProvider', () => {
   it('ignores a session whose transcript file is missing', async () => {
     const provider = new TranscriptOwnerCallStatusProvider('<<OWNER_CALL>>');
 
-    const result = await provider.listSessionNamesWithUnansweredOwnerCall(
-      new Map([[sessionName, path.join(rootDirectory, 'missing.jsonl')]]),
-    );
+    const result =
+      await provider.listUnansweredOwnerCallEpochSecondsBySessionName(
+        new Map([[sessionName, path.join(rootDirectory, 'missing.jsonl')]]),
+      );
 
     expect(result.size).toBe(0);
   });
@@ -328,9 +376,10 @@ describe('TranscriptOwnerCallStatusProvider', () => {
     );
     const provider = new TranscriptOwnerCallStatusProvider('<<OWNER_CALL>>');
 
-    const result = await provider.listSessionNamesWithUnansweredOwnerCall(
-      new Map([[sessionName, filePath]]),
-    );
+    const result =
+      await provider.listUnansweredOwnerCallEpochSecondsBySessionName(
+        new Map([[sessionName, filePath]]),
+      );
 
     expect(result.has(sessionName)).toBe(true);
   });
@@ -348,9 +397,10 @@ describe('TranscriptOwnerCallStatusProvider', () => {
     ]);
     const provider = new TranscriptOwnerCallStatusProvider('<call-to-user>');
 
-    const result = await provider.listSessionNamesWithUnansweredOwnerCall(
-      new Map([[sessionName, transcriptPath]]),
-    );
+    const result =
+      await provider.listUnansweredOwnerCallEpochSecondsBySessionName(
+        new Map([[sessionName, transcriptPath]]),
+      );
 
     expect(result.has(sessionName)).toBe(true);
   });

package/src/adapter/repositories/TranscriptOwnerCallStatusProvider.ts

@@ -91,31 +91,41 @@ const hasOwnerTextReply = (content: unknown): boolean => {
 export class TranscriptOwnerCallStatusProvider implements OwnerCallStatusProvider {
   constructor(private readonly ownerCallMarker: string | null) {}
 
-  listSessionNamesWithUnansweredOwnerCall = async (
+  listUnansweredOwnerCallEpochSecondsBySessionName = async (
     transcriptPathBySessionName: Map<string, string>,
-  ): Promise<Set<string>> => {
-    const waiting = new Set<string>();
+  ): Promise<Map<string, number>> => {
+    const unansweredOwnerCallEpochSecondsBySessionName = new Map<
+      string,
+      number
+    >();
     if (this.ownerCallMarker === null || this.ownerCallMarker.length === 0) {
-      return waiting;
+      return unansweredOwnerCallEpochSecondsBySessionName;
     }
     const marker = this.ownerCallMarker;
     for (const [sessionName, transcriptPath] of transcriptPathBySessionName) {
-      if (this.isWaitingForOwnerReply(transcriptPath, marker)) {
-        waiting.add(sessionName);
+      const unansweredOwnerCallEpochMs = this.findUnansweredOwnerCallEpochMs(
+        transcriptPath,
+        marker,
+      );
+      if (unansweredOwnerCallEpochMs !== null) {
+        unansweredOwnerCallEpochSecondsBySessionName.set(
+          sessionName,
+          Math.floor(unansweredOwnerCallEpochMs / 1000),
+        );
       }
     }
-    return waiting;
+    return unansweredOwnerCallEpochSecondsBySessionName;
   };
 
-  private isWaitingForOwnerReply = (
+  private findUnansweredOwnerCallEpochMs = (
     transcriptPath: string,
     marker: string,
-  ): boolean => {
+  ): number | null => {
     let content: string;
     try {
       content = fs.readFileSync(transcriptPath, 'utf8');
     } catch {
-      return false;
+      return null;
     }
     let lastOwnerCallEpochMs: number | null = null;
     let lastOwnerReplyEpochMs: number | null = null;
@@ -155,11 +165,11 @@ export class TranscriptOwnerCallStatusProvider implements OwnerCallStatusProvide
       }
     }
     if (lastOwnerCallEpochMs === null) {
-      return false;
+      return null;
     }
-    return (
-      lastOwnerReplyEpochMs === null ||
+    return lastOwnerReplyEpochMs === null ||
       lastOwnerCallEpochMs > lastOwnerReplyEpochMs
-    );
+      ? lastOwnerCallEpochMs
+      : null;
   };
 }