@adhdev/daemon-core 0.6.47 → 0.6.49

package/src/providers/status-monitor.ts

@@ -37,6 +37,9 @@ export class StatusMonitor {
     private config: MonitorConfig;
     private lastAlertTime = new Map<string, number>();
     private generatingStartTimes = new Map<string, number>();
+    private longGeneratingAlerted = new Map<string, boolean>();
+    private lastProgressFingerprint = new Map<string, string>();
+    private lastProgressChangeAt = new Map<string, number>();
 
     constructor(config?: Partial<MonitorConfig>) {
         this.config = { ...DEFAULT_MONITOR_CONFIG, ...config };
@@ -56,7 +59,7 @@ export class StatusMonitor {
   * Check status transition → return notification event array.
   * Called from each onTick() or detectStatusTransition().
   */
-    check(agentKey: string, status: string, now: number): MonitorEvent[] {
+    check(agentKey: string, status: string, now: number, progressFingerprint?: string): MonitorEvent[] {
         const events: MonitorEvent[] = [];
 
  // 1. Approval waiting notification
@@ -75,18 +78,31 @@ export class StatusMonitor {
         if (status === 'generating' || status === 'streaming') {
             if (!this.generatingStartTimes.has(agentKey)) {
                 this.generatingStartTimes.set(agentKey, now);
+                this.longGeneratingAlerted.set(agentKey, false);
+                const initialFingerprint = progressFingerprint ?? '';
+                this.lastProgressFingerprint.set(agentKey, initialFingerprint);
+                this.lastProgressChangeAt.set(agentKey, now);
+            }
+            const currentFingerprint = progressFingerprint ?? '';
+            const previousFingerprint = this.lastProgressFingerprint.get(agentKey);
+            if (previousFingerprint !== currentFingerprint) {
+                this.lastProgressFingerprint.set(agentKey, currentFingerprint);
+                this.lastProgressChangeAt.set(agentKey, now);
+                this.longGeneratingAlerted.set(agentKey, false);
             }
             if (this.config.longGeneratingAlert) {
-                const startedAt = this.generatingStartTimes.get(agentKey)!;
-                const elapsedSec = Math.round((now - startedAt) / 1000);
-                if (elapsedSec > this.config.longGeneratingThresholdSec) {
+                const progressChangedAt = this.lastProgressChangeAt.get(agentKey) || this.generatingStartTimes.get(agentKey)!;
+                const elapsedSec = Math.round((now - progressChangedAt) / 1000);
+                const alreadyAlerted = this.longGeneratingAlerted.get(agentKey) === true;
+                if (elapsedSec > this.config.longGeneratingThresholdSec && !alreadyAlerted) {
                     if (this.shouldAlert(agentKey + ':long_gen', now)) {
+                        this.longGeneratingAlerted.set(agentKey, true);
                         events.push({
                             type: 'monitor:long_generating',
                             agentKey,
                             elapsedSec,
                             timestamp: now,
-                            message: `${agentKey} has been generating for ${Math.round(elapsedSec / 60)}min`,
+                            message: `${agentKey} output appears stuck for ${Math.round(elapsedSec / 60)}min`,
                         });
                     }
                 }
@@ -94,6 +110,9 @@ export class StatusMonitor {
         } else {
  // Reset timer when switching to non-generating status
             this.generatingStartTimes.delete(agentKey);
+            this.longGeneratingAlerted.delete(agentKey);
+            this.lastProgressFingerprint.delete(agentKey);
+            this.lastProgressChangeAt.delete(agentKey);
         }
 
         return events;
@@ -113,12 +132,18 @@ export class StatusMonitor {
     reset(agentKey?: string): void {
         if (agentKey) {
             this.generatingStartTimes.delete(agentKey);
+            this.longGeneratingAlerted.delete(agentKey);
+            this.lastProgressFingerprint.delete(agentKey);
+            this.lastProgressChangeAt.delete(agentKey);
  // Delete all cooldowns for this key
             for (const k of this.lastAlertTime.keys()) {
                 if (k.startsWith(agentKey)) this.lastAlertTime.delete(k);
             }
         } else {
             this.generatingStartTimes.clear();
+            this.longGeneratingAlerted.clear();
+            this.lastProgressFingerprint.clear();
+            this.lastProgressChangeAt.clear();
             this.lastAlertTime.clear();
         }
     }