github-issue-tower-defence-management 1.115.0 → 1.115.2

package/src/adapter/entry-points/handlers/HandleScheduledEventUseCaseHandler.ts

@@ -65,15 +65,6 @@ import {
 
 const DEFAULT_DASHBOARD_DATA_DIR: string | null = null;
 
-// The silent-session notification inspects the single, global set of live tmux
-// sessions, but the scheduled handler runs once per project. If its cooldown
-// were stored in the per-project cache (`./tmp/cache/<projectName>`), the same
-// session would get an independent cooldown record per project and be notified
-// once per enabled project within a single cooldown window. The cooldown must
-// therefore live in a single project-independent cache scope so it is shared
-// across every per-project pass.
-const SILENT_SESSION_NOTIFICATION_CACHE_PATH = './tmp/cache/shared';
-
 const readSilentSeconds = (
   configValue: number | undefined,
   envValue: string | undefined,
@@ -125,7 +116,6 @@ export class HandleScheduledEventUseCaseHandler {
       mainSilentThresholdSeconds?: number;
       subAgentSilentThresholdSeconds?: number;
       subAgentRunningThresholdSeconds?: number;
-      silentNotificationCooldownSeconds?: number;
       silentNotificationStaggerSeconds?: number;
       activeHubTaskStatus?: string;
       silentMainStalledMessage?: string;
@@ -591,15 +581,9 @@ export class HandleScheduledEventUseCaseHandler {
           mergedInput.subAgentTranscriptRootDirectory ??
           process.env.TDPM_SUBAGENT_TRANSCRIPT_ROOT_DIRECTORY ??
           null;
-        const silentSessionNotificationCacheRepository =
-          new LocalStorageCacheRepository(
-            localStorageRepository,
-            SILENT_SESSION_NOTIFICATION_CACHE_PATH,
-          );
         await notifySilentTmuxSessions({
           enabled: silentNotificationEnabled,
           localCommandRunner: nodeLocalCommandRunner,
-          cacheRepository: silentSessionNotificationCacheRepository,
           ownerCallMarker,
           subAgentOutputRootDirectory,
           subAgentProcessMatchPattern,
@@ -619,11 +603,6 @@ export class HandleScheduledEventUseCaseHandler {
             process.env.TDPM_SUBAGENT_RUNNING_THRESHOLD_SECONDS,
             DEFAULT_NOTIFY_SILENT_TMUX_SESSIONS_PARAMS.subAgentRunningThresholdSeconds,
           ),
-          cooldownSeconds: readSilentSeconds(
-            mergedInput.silentNotificationCooldownSeconds,
-            process.env.TDPM_SILENT_NOTIFICATION_COOLDOWN_SECONDS,
-            DEFAULT_NOTIFY_SILENT_TMUX_SESSIONS_PARAMS.cooldownSeconds,
-          ),
           staggerSeconds: readSilentSeconds(
             mergedInput.silentNotificationStaggerSeconds,
             process.env.TDPM_SILENT_NOTIFICATION_STAGGER_SECONDS,

package/src/adapter/entry-points/handlers/notifySilentTmuxSessions.test.ts

@@ -3,8 +3,6 @@ import * as os from 'os';
 import * as path from 'path';
 import { LocalCommandRunner } from '../../../domain/usecases/adapter-interfaces/LocalCommandRunner';
 import { ProcessEnvironReader } from '../../../domain/usecases/adapter-interfaces/ProcessEnvironReader';
-import { LocalStorageCacheRepository } from '../../repositories/LocalStorageCacheRepository';
-import { LocalStorageRepository } from '../../repositories/LocalStorageRepository';
 import { SilentSessionMessageTemplates } from '../../repositories/ConfigurableSilentSessionMessageComposer';
 import { Issue } from '../../../domain/entities/Issue';
 import { SILENT_SESSION_REMINDER_SENTINEL } from '../../../domain/usecases/silentSessionReminderSentinel';
@@ -38,18 +36,15 @@ const createMockRunner = (): Mocked<LocalCommandRunner> => ({
 
 describe('notifySilentTmuxSessions', () => {
   let configDir: string;
-  let cacheDirectory: string;
 
   beforeEach(() => {
     jest.spyOn(console, 'log').mockImplementation(() => undefined);
     configDir = fs.mkdtempSync(path.join(os.tmpdir(), 'silent-config-'));
-    cacheDirectory = fs.mkdtempSync(path.join(os.tmpdir(), 'silent-cache-'));
   });
 
   afterEach(() => {
     jest.restoreAllMocks();
     fs.rmSync(configDir, { force: true, recursive: true });
-    fs.rmSync(cacheDirectory, { force: true, recursive: true });
   });
 
   const writeTranscript = (lines: object[]): void => {
@@ -78,17 +73,6 @@ describe('notifySilentTmuxSessions', () => {
     ]);
   };
 
-  const makeCacheRepository = (): LocalStorageCacheRepository =>
-    new LocalStorageCacheRepository(
-      new LocalStorageRepository(),
-      cacheDirectory,
-    );
-
-  const makeCacheRepositoryAt = (
-    basePath: string,
-  ): LocalStorageCacheRepository =>
-    new LocalStorageCacheRepository(new LocalStorageRepository(), basePath);
-
   const makeEnvironReader = (): ProcessEnvironReader => ({
     readEnviron: (pid: number) =>
       pid === CLAUDE_PID
@@ -126,7 +110,6 @@ describe('notifySilentTmuxSessions', () => {
     enabled: true,
     localCommandRunner: runner,
     processEnvironReader: makeEnvironReader(),
-    cacheRepository: makeCacheRepository(),
     ownerCallMarker: null,
     subAgentOutputRootDirectory: null,
     subAgentProcessMatchPattern: null,
@@ -219,54 +202,27 @@ describe('notifySilentTmuxSessions', () => {
     expect(sendCall).toBeUndefined();
   });
 
-  it('does not re-notify the same silent session on the next cycle within cooldown', async () => {
+  it('re-notifies the same silent session on the next cycle with no cooldown suppression', async () => {
     silentAssistantTranscript();
-    const cacheRepository = makeCacheRepository();
     const firstRunner = liveSessionRunner();
 
-    await notifySilentTmuxSessions({
-      ...baseParams(firstRunner),
-      cacheRepository,
-    });
+    await notifySilentTmuxSessions(baseParams(firstRunner));
 
-    const secondRunner = liveSessionRunner();
-    await notifySilentTmuxSessions({
-      ...baseParams(secondRunner),
-      cacheRepository,
-      now: new Date(NOW.getTime() + 60 * 1000),
-    });
-
-    const secondSendCall = secondRunner.runCommand.mock.calls.find(
+    const firstSendCall = firstRunner.runCommand.mock.calls.find(
       (call) => call[0] === 'tmux' && call[1][0] === 'send-keys',
     );
-    expect(secondSendCall).toBeUndefined();
-  });
-
-  it('shares the cooldown across project passes when a single project-independent cache scope is used', async () => {
-    silentAssistantTranscript();
-    // The handler runs once per project over the same global set of tmux
-    // sessions. When the silent-session cooldown lives in one shared cache
-    // scope, a second per-project pass for the same session within the cooldown
-    // window must NOT re-send, even though it is a different project pass.
-    const sharedCacheBasePath = path.join(cacheDirectory, 'shared');
-    const firstRunner = liveSessionRunner();
-
-    await notifySilentTmuxSessions({
-      ...baseParams(firstRunner),
-      cacheRepository: makeCacheRepositoryAt(sharedCacheBasePath),
-    });
+    expect(firstSendCall?.[1][2]).toBe(SESSION_NAME);
 
     const secondRunner = liveSessionRunner();
     await notifySilentTmuxSessions({
       ...baseParams(secondRunner),
-      cacheRepository: makeCacheRepositoryAt(sharedCacheBasePath),
       now: new Date(NOW.getTime() + 60 * 1000),
     });
 
     const secondSendCall = secondRunner.runCommand.mock.calls.find(
       (call) => call[0] === 'tmux' && call[1][0] === 'send-keys',
     );
-    expect(secondSendCall).toBeUndefined();
+    expect(secondSendCall?.[1][2]).toBe(SESSION_NAME);
   });
 
   const HUB_TASK_SESSION_NAME =
@@ -385,32 +341,4 @@ describe('notifySilentTmuxSessions', () => {
     );
     expect(sendCall?.[1][2]).toBe(HUB_TASK_SESSION_NAME);
   });
-
-  it('re-sends within the cooldown window when each project pass uses its own cache scope', async () => {
-    silentAssistantTranscript();
-    // This reproduces the per-project cooldown defect: when each project pass
-    // stores its cooldown under its own project-scoped cache path, the same
-    // session is notified once per project within the cooldown window.
-    const firstRunner = liveSessionRunner();
-    await notifySilentTmuxSessions({
-      ...baseParams(firstRunner),
-      cacheRepository: makeCacheRepositoryAt(
-        path.join(cacheDirectory, 'umino'),
-      ),
-    });
-
-    const secondRunner = liveSessionRunner();
-    await notifySilentTmuxSessions({
-      ...baseParams(secondRunner),
-      cacheRepository: makeCacheRepositoryAt(
-        path.join(cacheDirectory, 'xmile'),
-      ),
-      now: new Date(NOW.getTime() + 60 * 1000),
-    });
-
-    const secondSendCall = secondRunner.runCommand.mock.calls.find(
-      (call) => call[0] === 'tmux' && call[1][0] === 'send-keys',
-    );
-    expect(secondSendCall?.[1][2]).toBe(SESSION_NAME);
-  });
 });

package/src/adapter/entry-points/handlers/notifySilentTmuxSessions.ts

@@ -8,7 +8,6 @@ import {
   DEFAULT_MAIN_SILENT_THRESHOLD_SECONDS,
   DEFAULT_SUBAGENT_SILENT_THRESHOLD_SECONDS,
   DEFAULT_SUBAGENT_RUNNING_THRESHOLD_SECONDS,
-  DEFAULT_NOTIFICATION_COOLDOWN_SECONDS,
   DEFAULT_NOTIFICATION_STAGGER_SECONDS,
 } from '../../../domain/usecases/NotifySilentLiveSessionsUseCase';
 import { DefaultSilentSessionMessageComposer } from '../../../domain/usecases/DefaultSilentSessionMessageComposer';
@@ -17,7 +16,6 @@ import { ProcFsProcessEnvironReader } from '../../repositories/ProcFsProcessEnvi
 import { FileSystemInteractiveLiveSessionTranscriptResolver } from '../../repositories/FileSystemInteractiveLiveSessionTranscriptResolver';
 import { FileSystemSessionOutputActivityRepository } from '../../repositories/FileSystemSessionOutputActivityRepository';
 import { TmuxSilentSessionNotificationRepository } from '../../repositories/TmuxSilentSessionNotificationRepository';
-import { LocalStorageCacheRepository } from '../../repositories/LocalStorageCacheRepository';
 import { NoUnansweredOwnerCallStatusProvider } from '../../repositories/NoUnansweredOwnerCallStatusProvider';
 import { TranscriptOwnerCallStatusProvider } from '../../repositories/TranscriptOwnerCallStatusProvider';
 import { ProcessListSessionSubAgentActivityRepository } from '../../repositories/ProcessListSessionSubAgentActivityRepository';
@@ -35,7 +33,6 @@ export type NotifySilentTmuxSessionsParams = {
   enabled: boolean;
   localCommandRunner: LocalCommandRunner;
   processEnvironReader?: ProcessEnvironReader;
-  cacheRepository: Pick<LocalStorageCacheRepository, 'getLatest' | 'set'>;
   ownerCallMarker: string | null;
   subAgentOutputRootDirectory: string | null;
   subAgentProcessMatchPattern: string | null;
@@ -43,7 +40,6 @@ export type NotifySilentTmuxSessionsParams = {
   mainSilentThresholdSeconds: number;
   subAgentSilentThresholdSeconds: number;
   subAgentRunningThresholdSeconds: number;
-  cooldownSeconds: number;
   staggerSeconds: number;
   activeHubTaskStatus: string | null;
   hubTaskStatusResolver: HubTaskStatusResolver | null;
@@ -92,7 +88,6 @@ export const notifySilentTmuxSessions = async (
     enabled,
     localCommandRunner,
     processEnvironReader,
-    cacheRepository,
     ownerCallMarker,
     subAgentOutputRootDirectory,
     subAgentProcessMatchPattern,
@@ -100,7 +95,6 @@ export const notifySilentTmuxSessions = async (
     mainSilentThresholdSeconds,
     subAgentSilentThresholdSeconds,
     subAgentRunningThresholdSeconds,
-    cooldownSeconds,
     staggerSeconds,
     activeHubTaskStatus,
     hubTaskStatusResolver,
@@ -132,10 +126,7 @@ export const notifySilentTmuxSessions = async (
       now,
     ),
     createOwnerCallStatusProvider(ownerCallMarker),
-    new TmuxSilentSessionNotificationRepository(
-      localCommandRunner,
-      cacheRepository,
-    ),
+    new TmuxSilentSessionNotificationRepository(localCommandRunner),
     messageComposer,
     new RealSleeper(),
     hubTaskStatusResolver,
@@ -144,7 +135,6 @@ export const notifySilentTmuxSessions = async (
     mainSilentThresholdSeconds,
     subAgentSilentThresholdSeconds,
     subAgentRunningThresholdSeconds,
-    cooldownSeconds,
     staggerSeconds,
     activeHubTaskStatus,
     now,
@@ -155,6 +145,5 @@ export const DEFAULT_NOTIFY_SILENT_TMUX_SESSIONS_PARAMS = {
   mainSilentThresholdSeconds: DEFAULT_MAIN_SILENT_THRESHOLD_SECONDS,
   subAgentSilentThresholdSeconds: DEFAULT_SUBAGENT_SILENT_THRESHOLD_SECONDS,
   subAgentRunningThresholdSeconds: DEFAULT_SUBAGENT_RUNNING_THRESHOLD_SECONDS,
-  cooldownSeconds: DEFAULT_NOTIFICATION_COOLDOWN_SECONDS,
   staggerSeconds: DEFAULT_NOTIFICATION_STAGGER_SECONDS,
 } as const;

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

@@ -1,5 +1,4 @@
 import { LocalCommandRunner } from '../../domain/usecases/adapter-interfaces/LocalCommandRunner';
-import { LocalStorageCacheRepository } from './LocalStorageCacheRepository';
 import { TmuxSilentSessionNotificationRepository } from './TmuxSilentSessionNotificationRepository';
 
 type Mocked<T> = jest.Mocked<T> & jest.MockedObject<T>;
@@ -12,22 +11,11 @@ const createMockRunner = (): Mocked<LocalCommandRunner> => ({
   }),
 });
 
-const createMockCacheRepository = (): Mocked<
-  Pick<LocalStorageCacheRepository, 'getLatest' | 'set'>
-> => ({
-  getLatest: jest.fn().mockResolvedValue(null),
-  set: jest.fn().mockResolvedValue(undefined),
-});
-
 describe('TmuxSilentSessionNotificationRepository', () => {
   describe('sendSelfCheckNotification', () => {
     it('sends the message literally then submits it with Enter', async () => {
       const runner = createMockRunner();
-      const cache = createMockCacheRepository();
-      const repository = new TmuxSilentSessionNotificationRepository(
-        runner,
-        cache,
-      );
+      const repository = new TmuxSilentSessionNotificationRepository(runner);
 
       await repository.sendSelfCheckNotification(
         'https_//github_com/owner/repo/issues/9',
@@ -57,11 +45,7 @@ describe('TmuxSilentSessionNotificationRepository', () => {
         stderr: "can't find session",
         exitCode: 1,
       });
-      const cache = createMockCacheRepository();
-      const repository = new TmuxSilentSessionNotificationRepository(
-        runner,
-        cache,
-      );
+      const repository = new TmuxSilentSessionNotificationRepository(runner);
 
       await expect(
         repository.sendSelfCheckNotification('missing_session', 'message'),
@@ -70,84 +54,4 @@ describe('TmuxSilentSessionNotificationRepository', () => {
       );
     });
   });
-
-  describe('getLastNotifiedEpochSeconds', () => {
-    it('returns the cached epoch seconds for a session', async () => {
-      const runner = createMockRunner();
-      const cache = createMockCacheRepository();
-      cache.getLatest.mockResolvedValue({
-        value: { epochSeconds: 1700000000 },
-        timestamp: new Date(),
-      });
-      const repository = new TmuxSilentSessionNotificationRepository(
-        runner,
-        cache,
-      );
-
-      const result = await repository.getLastNotifiedEpochSeconds(
-        'https_//github_com/owner/repo/issues/9',
-      );
-
-      expect(result).toBe(1700000000);
-      expect(cache.getLatest).toHaveBeenCalledWith(
-        'silent-session-notification/https___github_com_owner_repo_issues_9',
-      );
-    });
-
-    it('returns null when no cache entry exists', async () => {
-      const runner = createMockRunner();
-      const cache = createMockCacheRepository();
-      cache.getLatest.mockResolvedValue(null);
-      const repository = new TmuxSilentSessionNotificationRepository(
-        runner,
-        cache,
-      );
-
-      const result = await repository.getLastNotifiedEpochSeconds(
-        'https_//github_com/owner/repo/issues/9',
-      );
-
-      expect(result).toBeNull();
-    });
-
-    it('returns null when the cached value has no numeric epoch', async () => {
-      const runner = createMockRunner();
-      const cache = createMockCacheRepository();
-      cache.getLatest.mockResolvedValue({
-        value: {},
-        timestamp: new Date(),
-      });
-      const repository = new TmuxSilentSessionNotificationRepository(
-        runner,
-        cache,
-      );
-
-      const result = await repository.getLastNotifiedEpochSeconds(
-        'https_//github_com/owner/repo/issues/9',
-      );
-
-      expect(result).toBeNull();
-    });
-  });
-
-  describe('setLastNotifiedEpochSeconds', () => {
-    it('persists the epoch seconds keyed by the sanitized session name', async () => {
-      const runner = createMockRunner();
-      const cache = createMockCacheRepository();
-      const repository = new TmuxSilentSessionNotificationRepository(
-        runner,
-        cache,
-      );
-
-      await repository.setLastNotifiedEpochSeconds(
-        'https_//github_com/owner/repo/issues/9',
-        1700000000,
-      );
-
-      expect(cache.set).toHaveBeenCalledWith(
-        'silent-session-notification/https___github_com_owner_repo_issues_9',
-        { epochSeconds: 1700000000 },
-      );
-    });
-  });
 });

package/src/adapter/repositories/TmuxSilentSessionNotificationRepository.ts

@@ -1,49 +1,8 @@
 import { SilentSessionNotificationRepository } from '../../domain/usecases/adapter-interfaces/SilentSessionNotificationRepository';
 import { LocalCommandRunner } from '../../domain/usecases/adapter-interfaces/LocalCommandRunner';
-import { LocalStorageCacheRepository } from './LocalStorageCacheRepository';
-
-const CACHE_KEY_PREFIX = 'silent-session-notification';
-
-const readEpochSeconds = (value: object): number | null => {
-  if (!('epochSeconds' in value)) {
-    return null;
-  }
-  const candidate = value.epochSeconds;
-  if (typeof candidate !== 'number') {
-    return null;
-  }
-  return candidate;
-};
 
 export class TmuxSilentSessionNotificationRepository implements SilentSessionNotificationRepository {
-  constructor(
-    private readonly localCommandRunner: LocalCommandRunner,
-    private readonly cacheRepository: Pick<
-      LocalStorageCacheRepository,
-      'getLatest' | 'set'
-    >,
-  ) {}
-
-  getLastNotifiedEpochSeconds = async (
-    sessionName: string,
-  ): Promise<number | null> => {
-    const cached = await this.cacheRepository.getLatest(
-      this.toCacheKey(sessionName),
-    );
-    if (cached === null) {
-      return null;
-    }
-    return readEpochSeconds(cached.value);
-  };
-
-  setLastNotifiedEpochSeconds = async (
-    sessionName: string,
-    epochSeconds: number,
-  ): Promise<void> => {
-    await this.cacheRepository.set(this.toCacheKey(sessionName), {
-      epochSeconds,
-    });
-  };
+  constructor(private readonly localCommandRunner: LocalCommandRunner) {}
 
   sendSelfCheckNotification = async (
     sessionName: string,
@@ -77,7 +36,4 @@ export class TmuxSilentSessionNotificationRepository implements SilentSessionNot
       );
     }
   };
-
-  private toCacheKey = (sessionName: string): string =>
-    `${CACHE_KEY_PREFIX}/${sessionName.replace(/\//g, '_')}`;
 }

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

@@ -5,7 +5,6 @@ import {
   DEFAULT_MAIN_SILENT_THRESHOLD_SECONDS,
   DEFAULT_SUBAGENT_SILENT_THRESHOLD_SECONDS,
   DEFAULT_SUBAGENT_RUNNING_THRESHOLD_SECONDS,
-  DEFAULT_NOTIFICATION_COOLDOWN_SECONDS,
   DEFAULT_NOTIFICATION_STAGGER_SECONDS,
 } from './NotifySilentLiveSessionsUseCase';
 import { Issue } from '../entities/Issue';
@@ -46,7 +45,6 @@ describe('NotifySilentLiveSessionsUseCase', () => {
     mainSilentThresholdSeconds: number;
     subAgentSilentThresholdSeconds: number;
     subAgentRunningThresholdSeconds: number;
-    cooldownSeconds: number;
     staggerSeconds: number;
     activeHubTaskStatus: string | null;
     now: Date;
@@ -54,7 +52,6 @@ describe('NotifySilentLiveSessionsUseCase', () => {
     mainSilentThresholdSeconds: DEFAULT_MAIN_SILENT_THRESHOLD_SECONDS,
     subAgentSilentThresholdSeconds: DEFAULT_SUBAGENT_SILENT_THRESHOLD_SECONDS,
     subAgentRunningThresholdSeconds: DEFAULT_SUBAGENT_RUNNING_THRESHOLD_SECONDS,
-    cooldownSeconds: DEFAULT_NOTIFICATION_COOLDOWN_SECONDS,
     staggerSeconds: DEFAULT_NOTIFICATION_STAGGER_SECONDS,
     activeHubTaskStatus: null,
     now,
@@ -127,8 +124,6 @@ describe('NotifySilentLiveSessionsUseCase', () => {
         .mockResolvedValue(new Set<string>()),
     };
     mockNotificationRepository = {
-      getLastNotifiedEpochSeconds: jest.fn().mockResolvedValue(null),
-      setLastNotifiedEpochSeconds: jest.fn().mockResolvedValue(undefined),
       sendSelfCheckNotification: jest.fn().mockResolvedValue(undefined),
     };
     mockMessageComposer = {
@@ -210,7 +205,6 @@ describe('NotifySilentLiveSessionsUseCase', () => {
     expect(DEFAULT_MAIN_SILENT_THRESHOLD_SECONDS).toBe(600);
     expect(DEFAULT_SUBAGENT_SILENT_THRESHOLD_SECONDS).toBe(300);
     expect(DEFAULT_SUBAGENT_RUNNING_THRESHOLD_SECONDS).toBe(900);
-    expect(DEFAULT_NOTIFICATION_COOLDOWN_SECONDS).toBe(30 * 60);
     expect(DEFAULT_NOTIFICATION_STAGGER_SECONDS).toBe(25);
   });
 
@@ -379,7 +373,7 @@ describe('NotifySilentLiveSessionsUseCase', () => {
     ).not.toHaveBeenCalled();
   });
 
-  it('does not re-notify a session within the cooldown window', async () => {
+  it('re-notifies a session on every consecutive cycle while it remains a valid silent target', async () => {
     setupLiveInteractiveSession('workbench');
     mockSessionOutputActivityRepository.listSessionOutputActivities.mockResolvedValue(
       [
@@ -390,15 +384,44 @@ describe('NotifySilentLiveSessionsUseCase', () => {
         },
       ],
     );
-    mockNotificationRepository.getLastNotifiedEpochSeconds.mockResolvedValue(
-      nowEpochSeconds - DEFAULT_NOTIFICATION_COOLDOWN_SECONDS + 1,
+
+    await useCase.run(runParams());
+    await useCase.run(runParams());
+
+    expect(
+      mockNotificationRepository.sendSelfCheckNotification,
+    ).toHaveBeenCalledTimes(2);
+    expect(
+      mockNotificationRepository.sendSelfCheckNotification,
+    ).toHaveBeenNthCalledWith(1, 'workbench', MAIN_STALLED_SECTION);
+    expect(
+      mockNotificationRepository.sendSelfCheckNotification,
+    ).toHaveBeenNthCalledWith(2, 'workbench', MAIN_STALLED_SECTION);
+  });
+
+  it('notifies on every repeated cycle at the same instant without reading any cooldown state', async () => {
+    setupLiveInteractiveSession('workbench');
+    mockSessionOutputActivityRepository.listSessionOutputActivities.mockResolvedValue(
+      [
+        {
+          sessionName: 'workbench',
+          lastOutputEpochSeconds:
+            nowEpochSeconds - DEFAULT_MAIN_SILENT_THRESHOLD_SECONDS,
+        },
+      ],
     );
 
+    await useCase.run(runParams());
+    await useCase.run(runParams());
     await useCase.run(runParams());
 
     expect(
       mockNotificationRepository.sendSelfCheckNotification,
-    ).not.toHaveBeenCalled();
+    ).toHaveBeenCalledTimes(3);
+    expect(Object.keys(mockNotificationRepository)).toEqual([
+      'sendSelfCheckNotification',
+    ]);
+    expect(runParams()).not.toHaveProperty('cooldownSeconds');
   });
 
   it('sends to multiple sessions sequentially with a stagger delay between sends', async () => {

package/src/domain/usecases/NotifySilentLiveSessionsUseCase.ts

@@ -14,7 +14,6 @@ import { ResolveInteractiveLiveSessionsUseCase } from './ResolveInteractiveLiveS
 export const DEFAULT_MAIN_SILENT_THRESHOLD_SECONDS = 10 * 60;
 export const DEFAULT_SUBAGENT_SILENT_THRESHOLD_SECONDS = 5 * 60;
 export const DEFAULT_SUBAGENT_RUNNING_THRESHOLD_SECONDS = 15 * 60;
-export const DEFAULT_NOTIFICATION_COOLDOWN_SECONDS = 30 * 60;
 export const DEFAULT_NOTIFICATION_STAGGER_SECONDS = 25;
 
 const GITHUB_ISSUE_URL_PATTERN =
@@ -53,7 +52,6 @@ export class NotifySilentLiveSessionsUseCase {
     mainSilentThresholdSeconds: number;
     subAgentSilentThresholdSeconds: number;
     subAgentRunningThresholdSeconds: number;
-    cooldownSeconds: number;
     staggerSeconds: number;
     activeHubTaskStatus: string | null;
     now: Date;
@@ -92,24 +90,8 @@ export class NotifySilentLiveSessionsUseCase {
       `Silent live session notification: ${candidates.length} candidate(s) of ${interactiveSessions.length} interactive session(s).`,
     );
 
-    const nowEpochSeconds = Math.floor(params.now.getTime() / 1000);
     let sentCount = 0;
     for (const candidate of candidates) {
-      const lastNotifiedEpochSeconds =
-        await this.notificationRepository.getLastNotifiedEpochSeconds(
-          candidate.sessionName,
-        );
-      if (
-        lastNotifiedEpochSeconds !== null &&
-        nowEpochSeconds - lastNotifiedEpochSeconds < params.cooldownSeconds
-      ) {
-        console.log(
-          `Skipping ${candidate.sessionName}: notified ${
-            nowEpochSeconds - lastNotifiedEpochSeconds
-          } seconds ago, within cooldown ${params.cooldownSeconds} seconds.`,
-        );
-        continue;
-      }
       if (
         !(await this.isHubTaskActive(
           candidate.sessionName,
@@ -125,10 +107,6 @@ export class NotifySilentLiveSessionsUseCase {
         candidate.sessionName,
         candidate.message,
       );
-      await this.notificationRepository.setLastNotifiedEpochSeconds(
-        candidate.sessionName,
-        nowEpochSeconds,
-      );
       sentCount += 1;
       console.log(`Notified ${candidate.sessionName}.`);
     }

package/src/domain/usecases/adapter-interfaces/SilentSessionNotificationRepository.ts

@@ -1,9 +1,4 @@
 export interface SilentSessionNotificationRepository {
-  getLastNotifiedEpochSeconds: (sessionName: string) => Promise<number | null>;
-  setLastNotifiedEpochSeconds: (
-    sessionName: string,
-    epochSeconds: number,
-  ) => Promise<void>;
   sendSelfCheckNotification: (
     sessionName: string,
     message: string,

package/types/adapter/entry-points/handlers/HandleScheduledEventUseCaseHandler.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"HandleScheduledEventUseCaseHandler.d.ts","sourceRoot":"","sources":["../../../../src/adapter/entry-points/handlers/HandleScheduledEventUseCaseHandler.ts"],"names":[],"mappings":"AAkCA,OAAO,EAAE,KAAK,EAAE,MAAM,gCAAgC,CAAC;AACvD,OAAO,EAAE,OAAO,EAAE,MAAM,kCAAkC,CAAC;AA0D3D,qBAAa,kCAAkC;IAC7C,MAAM,GACJ,gBAAgB,MAAM,EACtB,UAAU,OAAO,KAChB,OAAO,CAAC;QACT,OAAO,EAAE,OAAO,CAAC;QACjB,MAAM,EAAE,KAAK,EAAE,CAAC;QAChB,SAAS,EAAE,OAAO,CAAC;QACnB,eAAe,EAAE,IAAI,EAAE,CAAC;KACzB,GAAG,IAAI,CAAC,CA+iBP;CACH"}
+{"version":3,"file":"HandleScheduledEventUseCaseHandler.d.ts","sourceRoot":"","sources":["../../../../src/adapter/entry-points/handlers/HandleScheduledEventUseCaseHandler.ts"],"names":[],"mappings":"AAkCA,OAAO,EAAE,KAAK,EAAE,MAAM,gCAAgC,CAAC;AACvD,OAAO,EAAE,OAAO,EAAE,MAAM,kCAAkC,CAAC;AAiD3D,qBAAa,kCAAkC;IAC7C,MAAM,GACJ,gBAAgB,MAAM,EACtB,UAAU,OAAO,KAChB,OAAO,CAAC;QACT,OAAO,EAAE,OAAO,CAAC;QACjB,MAAM,EAAE,KAAK,EAAE,CAAC;QAChB,SAAS,EAAE,OAAO,CAAC;QACnB,eAAe,EAAE,IAAI,EAAE,CAAC;KACzB,GAAG,IAAI,CAAC,CAmiBP;CACH"}

package/types/adapter/entry-points/handlers/notifySilentTmuxSessions.d.ts

@@ -1,13 +1,11 @@
 import { LocalCommandRunner } from '../../../domain/usecases/adapter-interfaces/LocalCommandRunner';
 import { ProcessEnvironReader } from '../../../domain/usecases/adapter-interfaces/ProcessEnvironReader';
 import { HubTaskStatusResolver } from '../../../domain/usecases/NotifySilentLiveSessionsUseCase';
-import { LocalStorageCacheRepository } from '../../repositories/LocalStorageCacheRepository';
 import { SilentSessionMessageTemplates } from '../../repositories/ConfigurableSilentSessionMessageComposer';
 export type NotifySilentTmuxSessionsParams = {
     enabled: boolean;
     localCommandRunner: LocalCommandRunner;
     processEnvironReader?: ProcessEnvironReader;
-    cacheRepository: Pick<LocalStorageCacheRepository, 'getLatest' | 'set'>;
     ownerCallMarker: string | null;
     subAgentOutputRootDirectory: string | null;
     subAgentProcessMatchPattern: string | null;
@@ -15,7 +13,6 @@ export type NotifySilentTmuxSessionsParams = {
     mainSilentThresholdSeconds: number;
     subAgentSilentThresholdSeconds: number;
     subAgentRunningThresholdSeconds: number;
-    cooldownSeconds: number;
     staggerSeconds: number;
     activeHubTaskStatus: string | null;
     hubTaskStatusResolver: HubTaskStatusResolver | null;
@@ -27,7 +24,6 @@ export declare const DEFAULT_NOTIFY_SILENT_TMUX_SESSIONS_PARAMS: {
     readonly mainSilentThresholdSeconds: number;
     readonly subAgentSilentThresholdSeconds: number;
     readonly subAgentRunningThresholdSeconds: number;
-    readonly cooldownSeconds: number;
     readonly staggerSeconds: 25;
 };
 //# sourceMappingURL=notifySilentTmuxSessions.d.ts.map

package/types/adapter/entry-points/handlers/notifySilentTmuxSessions.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"notifySilentTmuxSessions.d.ts","sourceRoot":"","sources":["../../../../src/adapter/entry-points/handlers/notifySilentTmuxSessions.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,kBAAkB,EAAE,MAAM,gEAAgE,CAAC;AAGpG,OAAO,EAAE,oBAAoB,EAAE,MAAM,kEAAkE,CAAC;AACxG,OAAO,EAEL,qBAAqB,EAMtB,MAAM,0DAA0D,CAAC;AAOlE,OAAO,EAAE,2BAA2B,EAAE,MAAM,gDAAgD,CAAC;AAQ7F,OAAO,EAEL,6BAA6B,EAC9B,MAAM,6DAA6D,CAAC;AAGrE,MAAM,MAAM,8BAA8B,GAAG;IAC3C,OAAO,EAAE,OAAO,CAAC;IACjB,kBAAkB,EAAE,kBAAkB,CAAC;IACvC,oBAAoB,CAAC,EAAE,oBAAoB,CAAC;IAC5C,eAAe,EAAE,IAAI,CAAC,2BAA2B,EAAE,WAAW,GAAG,KAAK,CAAC,CAAC;IACxE,eAAe,EAAE,MAAM,GAAG,IAAI,CAAC;IAC/B,2BAA2B,EAAE,MAAM,GAAG,IAAI,CAAC;IAC3C,2BAA2B,EAAE,MAAM,GAAG,IAAI,CAAC;IAC3C,+BAA+B,EAAE,MAAM,GAAG,IAAI,CAAC;IAC/C,0BAA0B,EAAE,MAAM,CAAC;IACnC,8BAA8B,EAAE,MAAM,CAAC;IACvC,+BAA+B,EAAE,MAAM,CAAC;IACxC,eAAe,EAAE,MAAM,CAAC;IACxB,cAAc,EAAE,MAAM,CAAC;IACvB,mBAAmB,EAAE,MAAM,GAAG,IAAI,CAAC;IACnC,qBAAqB,EAAE,qBAAqB,GAAG,IAAI,CAAC;IACpD,gBAAgB,EAAE,6BAA6B,CAAC;IAChD,GAAG,EAAE,IAAI,CAAC;CACX,CAAC;AAoCF,eAAO,MAAM,wBAAwB,GACnC,QAAQ,8BAA8B,KACrC,OAAO,CAAC,IAAI,CA8Dd,CAAC;AAEF,eAAO,MAAM,0CAA0C;;;;;;CAM7C,CAAC"}
+{"version":3,"file":"notifySilentTmuxSessions.d.ts","sourceRoot":"","sources":["../../../../src/adapter/entry-points/handlers/notifySilentTmuxSessions.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,kBAAkB,EAAE,MAAM,gEAAgE,CAAC;AAGpG,OAAO,EAAE,oBAAoB,EAAE,MAAM,kEAAkE,CAAC;AACxG,OAAO,EAEL,qBAAqB,EAKtB,MAAM,0DAA0D,CAAC;AAclE,OAAO,EAEL,6BAA6B,EAC9B,MAAM,6DAA6D,CAAC;AAGrE,MAAM,MAAM,8BAA8B,GAAG;IAC3C,OAAO,EAAE,OAAO,CAAC;IACjB,kBAAkB,EAAE,kBAAkB,CAAC;IACvC,oBAAoB,CAAC,EAAE,oBAAoB,CAAC;IAC5C,eAAe,EAAE,MAAM,GAAG,IAAI,CAAC;IAC/B,2BAA2B,EAAE,MAAM,GAAG,IAAI,CAAC;IAC3C,2BAA2B,EAAE,MAAM,GAAG,IAAI,CAAC;IAC3C,+BAA+B,EAAE,MAAM,GAAG,IAAI,CAAC;IAC/C,0BAA0B,EAAE,MAAM,CAAC;IACnC,8BAA8B,EAAE,MAAM,CAAC;IACvC,+BAA+B,EAAE,MAAM,CAAC;IACxC,cAAc,EAAE,MAAM,CAAC;IACvB,mBAAmB,EAAE,MAAM,GAAG,IAAI,CAAC;IACnC,qBAAqB,EAAE,qBAAqB,GAAG,IAAI,CAAC;IACpD,gBAAgB,EAAE,6BAA6B,CAAC;IAChD,GAAG,EAAE,IAAI,CAAC;CACX,CAAC;AAoCF,eAAO,MAAM,wBAAwB,GACnC,QAAQ,8BAA8B,KACrC,OAAO,CAAC,IAAI,CAwDd,CAAC;AAEF,eAAO,MAAM,0CAA0C;;;;;CAK7C,CAAC"}

package/types/adapter/repositories/TmuxSilentSessionNotificationRepository.d.ts

@@ -1,13 +1,8 @@
 import { SilentSessionNotificationRepository } from '../../domain/usecases/adapter-interfaces/SilentSessionNotificationRepository';
 import { LocalCommandRunner } from '../../domain/usecases/adapter-interfaces/LocalCommandRunner';
-import { LocalStorageCacheRepository } from './LocalStorageCacheRepository';
 export declare class TmuxSilentSessionNotificationRepository implements SilentSessionNotificationRepository {
     private readonly localCommandRunner;
-    private readonly cacheRepository;
-    constructor(localCommandRunner: LocalCommandRunner, cacheRepository: Pick<LocalStorageCacheRepository, 'getLatest' | 'set'>);
-    getLastNotifiedEpochSeconds: (sessionName: string) => Promise<number | null>;
-    setLastNotifiedEpochSeconds: (sessionName: string, epochSeconds: number) => Promise<void>;
+    constructor(localCommandRunner: LocalCommandRunner);
     sendSelfCheckNotification: (sessionName: string, message: string) => Promise<void>;
-    private toCacheKey;
 }
 //# sourceMappingURL=TmuxSilentSessionNotificationRepository.d.ts.map