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

package/src/domain/usecases/NotifySilentLiveSessionsUseCase.ts

@@ -1,489 +0,0 @@
-import { LiveSessionActivitySnapshot } from '../entities/LiveSessionActivitySnapshot';
-import { InteractiveLiveSession } from '../entities/InteractiveLiveSession';
-import { LiveSessionProcessSnapshotProvider } from './adapter-interfaces/LiveSessionProcessSnapshotProvider';
-import { InteractiveLiveSessionTranscriptResolver } from './adapter-interfaces/InteractiveLiveSessionTranscriptResolver';
-import { OwnerCallStatusProvider } from './adapter-interfaces/OwnerCallStatusProvider';
-import { RefusalTailStatusProvider } from './adapter-interfaces/RefusalTailStatusProvider';
-import { SessionOutputActivityRepository } from './adapter-interfaces/SessionOutputActivityRepository';
-import { SessionSubAgentActivityRepository } from './adapter-interfaces/SessionSubAgentActivityRepository';
-import { SilentSessionMessageComposer } from './adapter-interfaces/SilentSessionMessageComposer';
-import { SilentSessionNotificationRepository } from './adapter-interfaces/SilentSessionNotificationRepository';
-import { SilentSessionCandidateStateRepository } from './adapter-interfaces/SilentSessionCandidateStateRepository';
-import { SilentSessionHubTaskStatusCacheRepository } from './adapter-interfaces/SilentSessionHubTaskStatusCacheRepository';
-import { Sleeper } from './adapter-interfaces/Sleeper';
-import { IssueRepository } from './adapter-interfaces/IssueRepository';
-import { ResolveInteractiveLiveSessionsUseCase } from './ResolveInteractiveLiveSessionsUseCase';
-
-export const DEFAULT_MAIN_SILENT_THRESHOLD_SECONDS = 10 * 60;
-// Retained only for backward compatibility of the configuration surface
-// (TDPM_SILENT_UNANSWERED_OWNER_CALL_GRACE_SECONDS). The value is no longer
-// consulted: an unanswered owner call suppresses the main-stall reminder
-// unconditionally (treated as an infinite grace). See composeCandidate.
-export const DEFAULT_UNANSWERED_OWNER_CALL_GRACE_SECONDS = 60 * 60;
-export const DEFAULT_SUBAGENT_SILENT_THRESHOLD_SECONDS = 5 * 60;
-export const DEFAULT_SUBAGENT_RUNNING_THRESHOLD_SECONDS = 15 * 60;
-export const DEFAULT_NOTIFICATION_STAGGER_SECONDS = 25;
-export const DEFAULT_CANDIDATE_DEBOUNCE_RECENCY_WINDOW_SECONDS = 15 * 60;
-export const DEFAULT_HUB_TASK_STATUS_CACHE_TTL_SECONDS = 5 * 60;
-
-const GITHUB_ISSUE_OR_PULL_URL_PATTERN =
-  /^https:\/\/github\.com\/([^/]+)\/([^/]+)\/(?:issues|pull)\/(\d+)$/;
-
-const GITHUB_TMUX_SESSION_NAME_PATTERN =
-  /^https_\/\/github_com\/([^/]+)\/([^/]+)\/(?:issues|pull)\/(\d+)$/;
-
-export const parseHubTaskIssueUrlFromSessionName = (
-  sessionName: string,
-): string | null => {
-  if (GITHUB_ISSUE_OR_PULL_URL_PATTERN.test(sessionName)) {
-    return sessionName;
-  }
-  const tmuxMatch = GITHUB_TMUX_SESSION_NAME_PATTERN.exec(sessionName);
-  if (tmuxMatch === null) {
-    return null;
-  }
-  const [, owner, repo, number] = tmuxMatch;
-  const target = sessionName.includes('/pull/') ? 'pull' : 'issues';
-  return `https://github.com/${owner}/${repo}/${target}/${number}`;
-};
-
-const GITHUB_ISSUE_OR_PULL_REQUEST_SESSION_NAME_PATTERN =
-  /^https(:\/\/|_\/\/)github(\.com|_com)\/[^/]+\/[^/]+\/(issues|pull)\/\d+$/;
-
-export const isGitHubIssueOrPullRequestSessionName = (
-  sessionName: string,
-): boolean =>
-  GITHUB_ISSUE_OR_PULL_REQUEST_SESSION_NAME_PATTERN.test(sessionName);
-
-export type HubTaskStatusResolver = Pick<IssueRepository, 'getIssueByUrl'>;
-
-type NotifyCandidate = {
-  sessionName: string;
-  message: string;
-  sectionLabels: string[];
-};
-
-export class NotifySilentLiveSessionsUseCase {
-  private readonly resolveInteractiveLiveSessions =
-    new ResolveInteractiveLiveSessionsUseCase();
-
-  constructor(
-    private readonly liveSessionProcessSnapshotProvider: LiveSessionProcessSnapshotProvider,
-    private readonly interactiveLiveSessionTranscriptResolver: InteractiveLiveSessionTranscriptResolver,
-    private readonly sessionOutputActivityRepository: SessionOutputActivityRepository,
-    private readonly subAgentActivityRepository: SessionSubAgentActivityRepository,
-    private readonly ownerCallStatusProvider: OwnerCallStatusProvider,
-    private readonly notificationRepository: SilentSessionNotificationRepository,
-    private readonly candidateStateRepository: SilentSessionCandidateStateRepository,
-    private readonly messageComposer: SilentSessionMessageComposer,
-    private readonly sleeper: Sleeper,
-    private readonly hubTaskStatusResolver: HubTaskStatusResolver | null = null,
-    private readonly hubTaskStatusCacheRepository: SilentSessionHubTaskStatusCacheRepository | null = null,
-    private readonly refusalTailStatusProvider: RefusalTailStatusProvider | null = null,
-  ) {}
-
-  run = async (params: {
-    mainSilentThresholdSeconds: number;
-    unansweredOwnerCallGraceSeconds: number;
-    subAgentSilentThresholdSeconds: number;
-    subAgentRunningThresholdSeconds: number;
-    staggerSeconds: number;
-    candidateDebounceRecencyWindowSeconds: number;
-    activeHubTaskStatus: string | null;
-    hubTaskStatusCacheTtlSeconds: number;
-    now: Date;
-  }): Promise<void> => {
-    const snapshot =
-      await this.liveSessionProcessSnapshotProvider.getSnapshot();
-    const allInteractiveSessions =
-      this.resolveInteractiveLiveSessions.resolve(snapshot);
-    // Resolve the on-disk Claude transcript for every interactive session
-    // before any name-based narrowing. A resolvable transcript is the proof
-    // that a session is a live Claude agent session (its transcript, keyed by
-    // the session id, is actively present on disk), independent of how the tmux
-    // session is named.
-    const transcriptPathBySessionName =
-      this.interactiveLiveSessionTranscriptResolver.resolveTranscriptPaths(
-        allInteractiveSessions,
-      );
-    // Monitor a session when it is either named after a github.com issue or
-    // pull-request URL (the hub-task sessions) or has a resolvable Claude agent
-    // transcript (role-named resident leader / PM agent sessions such as app,
-    // tdpm-cli, secretary, and the per-project *pm sessions). Gating the
-    // broadened inclusion on the resolvable transcript rather than on merely
-    // being a tmux session keeps the fail-direction safe: a genuine non-agent
-    // interactive session — a login shell, or a viewer such as sso_login or a
-    // tdpm viewer — exposes no Claude transcript on disk, so it is still
-    // excluded and never reminded.
-    const interactiveSessions = allInteractiveSessions.filter(
-      (session) =>
-        isGitHubIssueOrPullRequestSessionName(session.sessionName) ||
-        transcriptPathBySessionName.has(session.sessionName),
-    );
-    const skippedNonAgentSessionCount =
-      allInteractiveSessions.length - interactiveSessions.length;
-    if (skippedNonAgentSessionCount > 0) {
-      console.log(
-        `Silent live session notification: ignoring ${skippedNonAgentSessionCount} interactive session(s) that are neither named after a github.com issue or pull-request URL nor have a resolvable Claude agent transcript.`,
-      );
-    }
-
-    // A session whose most recent assistant turn is a model refusal is
-    // excluded from ALL reminder candidates (main-stall and sub-agent
-    // branches alike): each reminder delivery re-sends the full session
-    // context to the API and is guaranteed to produce another refusal, so
-    // reminding such a session only burns tokens. The gate is state-based
-    // (no time windows) and self-clears once a non-refusal assistant turn
-    // appears after the refusal.
-    const refusalTailedSessionNames =
-      this.refusalTailStatusProvider === null
-        ? new Set<string>()
-        : await this.refusalTailStatusProvider.listRefusalTailedSessionNames(
-            transcriptPathBySessionName,
-          );
-    const monitoredSessions = interactiveSessions.filter((session) => {
-      if (!refusalTailedSessionNames.has(session.sessionName)) {
-        return true;
-      }
-      console.log(
-        `Skipping ${session.sessionName}: last assistant turn was a model refusal; suppressing reminders until a non-refusal turn appears.`,
-      );
-      return false;
-    });
-
-    const snapshots = await this.collectSnapshots(
-      monitoredSessions,
-      transcriptPathBySessionName,
-      params.now,
-    );
-
-    const candidates: NotifyCandidate[] = [];
-    for (const sessionSnapshot of snapshots) {
-      const candidate = this.composeCandidate(sessionSnapshot, params);
-      if (candidate !== null) {
-        candidates.push(candidate);
-      }
-    }
-    candidates.sort((left, right) =>
-      left.sessionName < right.sessionName
-        ? -1
-        : left.sessionName > right.sessionName
-          ? 1
-          : 0,
-    );
-
-    const previousCandidateSessionNames =
-      await this.candidateStateRepository.loadRecentCandidateSessionNames({
-        now: params.now,
-        recencyWindowSeconds: params.candidateDebounceRecencyWindowSeconds,
-      });
-    await this.candidateStateRepository.saveCandidateSessionNames({
-      sessionNames: candidates.map((candidate) => candidate.sessionName),
-      now: params.now,
-    });
-
-    const debouncedCandidates = candidates.filter((candidate) =>
-      previousCandidateSessionNames.has(candidate.sessionName),
-    );
-    const suppressedFirstCycleCount =
-      candidates.length - debouncedCandidates.length;
-
-    console.log(
-      `Silent live session notification: ${debouncedCandidates.length} debounced candidate(s) of ${candidates.length} current candidate(s) across ${interactiveSessions.length} interactive session(s); ${suppressedFirstCycleCount} first-cycle candidate(s) deferred until they persist into the next cycle.`,
-    );
-
-    let sentCount = 0;
-    for (const candidate of debouncedCandidates) {
-      if (
-        !(await this.isHubTaskActive(
-          candidate.sessionName,
-          params.activeHubTaskStatus,
-          params.hubTaskStatusCacheTtlSeconds,
-          params.now,
-        ))
-      ) {
-        continue;
-      }
-      if (sentCount > 0) {
-        await this.sleeper.sleep(params.staggerSeconds * 1000);
-      }
-      await this.notificationRepository.sendSelfCheckNotification(
-        candidate.sessionName,
-        candidate.message,
-      );
-      sentCount += 1;
-      // One line per send, grep-stable on the `Notified ` prefix: the
-      // ISO-8601 UTC timestamp disambiguates concurrent schedule runs and
-      // the section list records what the message actually contained.
-      console.log(
-        `Notified ${candidate.sessionName} at=${params.now.toISOString()} sections=[${candidate.sectionLabels.join(',')}]`,
-      );
-    }
-  };
-
-  private isHubTaskActive = async (
-    sessionName: string,
-    activeHubTaskStatus: string | null,
-    hubTaskStatusCacheTtlSeconds: number,
-    now: Date,
-  ): Promise<boolean> => {
-    if (activeHubTaskStatus === null || this.hubTaskStatusResolver === null) {
-      return true;
-    }
-    const hubTaskIssueUrl = parseHubTaskIssueUrlFromSessionName(sessionName);
-    if (hubTaskIssueUrl === null) {
-      return true;
-    }
-
-    const cachedEntry =
-      this.hubTaskStatusCacheRepository === null
-        ? null
-        : await this.hubTaskStatusCacheRepository.loadHubTaskStatus({
-            url: hubTaskIssueUrl,
-          });
-    const nowEpochSeconds = Math.floor(now.getTime() / 1000);
-    if (cachedEntry !== null) {
-      const cacheAgeSeconds =
-        nowEpochSeconds - cachedEntry.recordedEpochSeconds;
-      if (cacheAgeSeconds <= hubTaskStatusCacheTtlSeconds) {
-        const active = this.isResolvedStatusActive(
-          cachedEntry.state,
-          cachedEntry.status,
-          activeHubTaskStatus,
-        );
-        if (!active) {
-          console.log(
-            `Skipping ${sessionName}: hub task ${hubTaskIssueUrl} is no longer active per cached status (state "${cachedEntry.state}", status "${cachedEntry.status ?? 'null'}", active status "${activeHubTaskStatus}").`,
-          );
-        }
-        return active;
-      }
-    }
-
-    const resolution = await this.tryResolveAndCacheHubTask(
-      hubTaskIssueUrl,
-      activeHubTaskStatus,
-      sessionName,
-      now,
-    );
-    if (resolution.resolved) {
-      return resolution.active;
-    }
-
-    if (cachedEntry !== null) {
-      const active = this.isResolvedStatusActive(
-        cachedEntry.state,
-        cachedEntry.status,
-        activeHubTaskStatus,
-      );
-      console.warn(
-        `Hub task ${hubTaskIssueUrl} for session ${sessionName} could not be resolved (${resolution.reason}); falling back to expired cached status (state "${cachedEntry.state}", status "${cachedEntry.status ?? 'null'}"), so the notification is ${active ? 'sent' : 'suppressed'}.`,
-      );
-      return active;
-    }
-
-    console.warn(
-      `Hub task ${hubTaskIssueUrl} for session ${sessionName} is not resolvable and has no cached status (${resolution.reason}); sending notification (fail-open).`,
-    );
-    return true;
-  };
-
-  private tryResolveAndCacheHubTask = async (
-    hubTaskIssueUrl: string,
-    activeHubTaskStatus: string,
-    sessionName: string,
-    now: Date,
-  ): Promise<
-    { resolved: true; active: boolean } | { resolved: false; reason: string }
-  > => {
-    if (this.hubTaskStatusResolver === null) {
-      return { resolved: false, reason: 'resolver is not configured' };
-    }
-    let issue: Awaited<ReturnType<HubTaskStatusResolver['getIssueByUrl']>>;
-    try {
-      issue = await this.hubTaskStatusResolver.getIssueByUrl(hubTaskIssueUrl);
-    } catch (error) {
-      return {
-        resolved: false,
-        reason: error instanceof Error ? error.message : String(error),
-      };
-    }
-    if (issue === null) {
-      return { resolved: false, reason: 'resolver returned no tracked task' };
-    }
-    if (this.hubTaskStatusCacheRepository !== null) {
-      await this.hubTaskStatusCacheRepository.saveHubTaskStatus({
-        url: hubTaskIssueUrl,
-        state: issue.state,
-        status: issue.status,
-        now,
-      });
-    }
-    const active = this.isResolvedStatusActive(
-      issue.state,
-      issue.status,
-      activeHubTaskStatus,
-    );
-    if (!active) {
-      console.log(
-        `Skipping ${sessionName}: hub task ${hubTaskIssueUrl} is no longer active (state "${issue.state}", status "${issue.status ?? 'null'}", active status "${activeHubTaskStatus}").`,
-      );
-    }
-    return { resolved: true, active };
-  };
-
-  private isResolvedStatusActive = (
-    state: 'OPEN' | 'CLOSED' | 'MERGED',
-    status: string | null,
-    activeHubTaskStatus: string,
-  ): boolean => state === 'OPEN' && status === activeHubTaskStatus;
-
-  private collectSnapshots = async (
-    interactiveSessions: InteractiveLiveSession[],
-    transcriptPathBySessionName: Map<string, string>,
-    now: Date,
-  ): Promise<LiveSessionActivitySnapshot[]> => {
-    const sessionNames = interactiveSessions.map(
-      (session) => session.sessionName,
-    );
-
-    const activities =
-      await this.sessionOutputActivityRepository.listSessionOutputActivities(
-        transcriptPathBySessionName,
-      );
-    const lastOutputBySessionName = new Map<string, number>();
-    for (const activity of activities) {
-      lastOutputBySessionName.set(
-        activity.sessionName,
-        activity.lastOutputEpochSeconds,
-      );
-    }
-
-    const subAgentsBySessionName =
-      await this.subAgentActivityRepository.listSubAgentActivitiesBySessionName(
-        sessionNames,
-        transcriptPathBySessionName,
-      );
-
-    const unansweredOwnerCallEpochSecondsBySessionName =
-      await this.ownerCallStatusProvider.listUnansweredOwnerCallEpochSecondsBySessionName(
-        transcriptPathBySessionName,
-      );
-
-    const nowEpochSeconds = Math.floor(now.getTime() / 1000);
-    return sessionNames.map((sessionName) => {
-      const lastOutputEpochSeconds = lastOutputBySessionName.get(sessionName);
-      const mainSilentSeconds =
-        lastOutputEpochSeconds === undefined
-          ? null
-          : nowEpochSeconds - lastOutputEpochSeconds;
-      const unansweredOwnerCallEpochSeconds =
-        unansweredOwnerCallEpochSecondsBySessionName.get(sessionName);
-      return {
-        sessionName,
-        mainSilentSeconds,
-        subAgents: subAgentsBySessionName.get(sessionName) ?? [],
-        unansweredOwnerCallAgeSeconds:
-          unansweredOwnerCallEpochSeconds === undefined
-            ? null
-            : nowEpochSeconds - unansweredOwnerCallEpochSeconds,
-      };
-    });
-  };
-
-  private composeCandidate = (
-    snapshot: LiveSessionActivitySnapshot,
-    thresholds: {
-      mainSilentThresholdSeconds: number;
-      unansweredOwnerCallGraceSeconds: number;
-      subAgentSilentThresholdSeconds: number;
-      subAgentRunningThresholdSeconds: number;
-      now: Date;
-    },
-  ): NotifyCandidate | null => {
-    const sections: string[] = [];
-    const sectionLabels: string[] = [];
-
-    const mainSilentSeconds = snapshot.mainSilentSeconds;
-    const unansweredOwnerCallAgeSeconds =
-      snapshot.unansweredOwnerCallAgeSeconds;
-    // Owner-defined rule: whenever the latest owner call is newer than the
-    // latest owner reply (i.e. the call is unanswered), the session is
-    // waiting on the owner and MUST NOT receive a main-stall reminder —
-    // unconditionally, with no age or grace expiry. The persistent unread
-    // indicator in the owner's app covers the missed-call case, so a
-    // time-based re-fire is unnecessary. `unansweredOwnerCallGraceSeconds`
-    // is retained in the parameters only for backward compatibility of the
-    // call signature and is intentionally ignored (treated as infinite).
-    const suppressedByUnansweredOwnerCall =
-      unansweredOwnerCallAgeSeconds !== null;
-    // The main-stall reminder is driven purely by silence: a session that has
-    // produced no assistant output for longer than the threshold is reminded
-    // regardless of whether its transcript tail is an in-progress tool_use. A
-    // session that merely looks busy (mid-tool-call) can in fact be stuck, so
-    // its apparent busyness MUST NOT suppress the reminder; the reminder queues
-    // cleanly into the session even when it is mid-turn. The only main-stall
-    // suppression is an unanswered owner call, which is a real wait on the owner.
-    const mainTriggered =
-      mainSilentSeconds !== null &&
-      mainSilentSeconds >= thresholds.mainSilentThresholdSeconds &&
-      !suppressedByUnansweredOwnerCall;
-    if (mainTriggered) {
-      sections.push(
-        this.messageComposer.composeMainStalledSection(mainSilentSeconds),
-      );
-      sectionLabels.push('main-stalled');
-    }
-
-    const idleSubAgents = snapshot.subAgents.filter(
-      (subAgent) =>
-        !subAgent.finishedResultUnconsumed &&
-        !subAgent.waitingOnExternalProcess &&
-        subAgent.silentSeconds >= thresholds.subAgentSilentThresholdSeconds,
-    );
-    const longRunningSubAgents = snapshot.subAgents.filter(
-      (subAgent) =>
-        !subAgent.finishedResultUnconsumed &&
-        subAgent.runningSeconds >= thresholds.subAgentRunningThresholdSeconds,
-    );
-    if (idleSubAgents.length > 0 || longRunningSubAgents.length > 0) {
-      sections.push(
-        this.messageComposer.composeSubAgentSection({
-          idleSubAgents,
-          longRunningSubAgents,
-        }),
-      );
-      for (const subAgent of idleSubAgents) {
-        sectionLabels.push(`sub-agent-idle:${subAgent.label}`);
-      }
-      for (const subAgent of longRunningSubAgents) {
-        sectionLabels.push(`sub-agent-long-running:${subAgent.label}`);
-      }
-    }
-
-    const unconsumedResultSubAgents = snapshot.subAgents.filter(
-      (subAgent) =>
-        subAgent.finishedResultUnconsumed &&
-        subAgent.silentSeconds >= thresholds.subAgentSilentThresholdSeconds,
-    );
-    if (unconsumedResultSubAgents.length > 0) {
-      sections.push(
-        this.messageComposer.composeSubAgentUnconsumedResultSection(
-          unconsumedResultSubAgents,
-        ),
-      );
-      for (const subAgent of unconsumedResultSubAgents) {
-        sectionLabels.push(`sub-agent-result-unconsumed:${subAgent.label}`);
-      }
-    }
-
-    if (sections.length === 0) {
-      return null;
-    }
-    return {
-      sessionName: snapshot.sessionName,
-      message: sections.join('\n\n'),
-      sectionLabels,
-    };
-  };
-}