claude-flow 2.0.0-alpha.60 → 2.0.0-alpha.62

package/src/coordination/metrics.ts

@@ -1,4 +1,3 @@
-import { getErrorMessage } from '../utils/error-handler.js';
 /**
  * Metrics and monitoring for coordination performance
  */
@@ -9,7 +8,7 @@ import { SystemEvents } from '../utils/types.js';
 
 export interface CoordinationMetrics {
   timestamp: Date;
-  
+
   // Task metrics
   taskMetrics: {
     totalTasks: number;
@@ -22,7 +21,7 @@ export interface CoordinationMetrics {
     tasksByPriority: Record<string, number>;
     tasksByType: Record<string, number>;
   };
-  
+
   // Agent metrics
   agentMetrics: {
     totalAgents: number;
@@ -33,7 +32,7 @@ export interface CoordinationMetrics {
     avgTasksPerAgent: number;
     agentsByType: Record<string, number>;
   };
-  
+
   // Resource metrics
   resourceMetrics: {
     totalResources: number;
@@ -44,7 +43,7 @@ export interface CoordinationMetrics {
     lockContention: number; // waiting requests
     deadlockCount: number;
   };
-  
+
   // Coordination metrics
   coordinationMetrics: {
     messagesSent: number;
@@ -55,7 +54,7 @@ export interface CoordinationMetrics {
     workStealingEvents: number;
     circuitBreakerTrips: number;
   };
-  
+
   // Performance metrics
   performanceMetrics: {
     coordinationLatency: number; // avg ms
@@ -82,7 +81,7 @@ export class CoordinationMetricsCollector {
   private messageStartTimes = new Map<string, Date>();
   private lockStartTimes = new Map<string, Date>();
   private collectionInterval?: number;
-  
+
   // Counters
   private counters = {
     totalTasks: 0,
@@ -98,7 +97,7 @@ export class CoordinationMetricsCollector {
     deadlockCount: 0,
     errors: 0,
   };
-  
+
   // Gauges
   private gauges = {
     activeTasks: 0,
@@ -109,7 +108,7 @@ export class CoordinationMetricsCollector {
     freeResources: 0,
     lockContention: 0,
   };
-  
+
   // Histograms (for calculating averages)
   private histograms = {
     taskDurations: [] as number[],
@@ -132,7 +131,7 @@ export class CoordinationMetricsCollector {
    */
   start(): void {
     this.logger.info('Starting coordination metrics collection');
-    
+
     this.collectionInterval = setInterval(() => {
       this.collectMetrics();
     }, this.collectionIntervalMs);
@@ -146,7 +145,7 @@ export class CoordinationMetricsCollector {
       clearInterval(this.collectionInterval);
       delete this.collectionInterval;
     }
-    
+
     this.logger.info('Stopped coordination metrics collection');
   }
 
@@ -159,13 +158,13 @@ export class CoordinationMetricsCollector {
       metric,
       value,
     };
-    
+
     if (tags !== undefined) {
       sample.tags = tags;
     }
-    
+
     this.samples.push(sample);
-    
+
     // Keep only last 10000 samples to prevent memory bloat
     if (this.samples.length > 10000) {
       this.samples = this.samples.slice(-5000);
@@ -178,15 +177,15 @@ export class CoordinationMetricsCollector {
   getCurrentMetrics(): CoordinationMetrics {
     const now = new Date();
     const minuteAgo = new Date(now.getTime() - 60000);
-    
+
     // Calculate throughput (items per minute)
-    const recentSamples = this.samples.filter(s => s.timestamp >= minuteAgo);
-    const taskCompletions = recentSamples.filter(s => s.metric === 'task.completed').length;
-    const errorCount = recentSamples.filter(s => s.metric === 'error').length;
-    
+    const recentSamples = this.samples.filter((s) => s.timestamp >= minuteAgo);
+    const taskCompletions = recentSamples.filter((s) => s.metric === 'task.completed').length;
+    const errorCount = recentSamples.filter((s) => s.metric === 'error').length;
+
     return {
       timestamp: now,
-      
+
       taskMetrics: {
         totalTasks: this.counters.totalTasks,
         activeTasks: this.gauges.activeTasks,
@@ -198,7 +197,7 @@ export class CoordinationMetricsCollector {
         tasksByPriority: this.getTasksByPriority(),
         tasksByType: this.getTasksByType(),
       },
-      
+
       agentMetrics: {
         totalAgents: this.gauges.activeAgents + this.gauges.idleAgents,
         activeAgents: this.gauges.activeAgents,
@@ -208,7 +207,7 @@ export class CoordinationMetricsCollector {
         avgTasksPerAgent: this.calculateAvgTasksPerAgent(),
         agentsByType: this.getAgentsByType(),
       },
-      
+
       resourceMetrics: {
         totalResources: this.gauges.lockedResources + this.gauges.freeResources,
         lockedResources: this.gauges.lockedResources,
@@ -218,7 +217,7 @@ export class CoordinationMetricsCollector {
         lockContention: this.gauges.lockContention,
         deadlockCount: this.counters.deadlockCount,
       },
-      
+
       coordinationMetrics: {
         messagesSent: this.counters.messagesSent,
         messagesReceived: this.counters.messagesReceived,
@@ -228,7 +227,7 @@ export class CoordinationMetricsCollector {
         workStealingEvents: this.counters.workStealingEvents,
         circuitBreakerTrips: this.counters.circuitBreakerTrips,
       },
-      
+
       performanceMetrics: {
         coordinationLatency: this.average(this.histograms.coordinationLatencies),
         schedulingLatency: this.average(this.histograms.schedulingLatencies),
@@ -244,28 +243,26 @@ export class CoordinationMetricsCollector {
    */
   getMetricHistory(metric: string, since?: Date): MetricsSample[] {
     const cutoff = since || new Date(Date.now() - 3600000); // 1 hour ago
-    
-    return this.samples.filter(s => 
-      s.metric === metric && s.timestamp >= cutoff
-    );
+
+    return this.samples.filter((s) => s.metric === metric && s.timestamp >= cutoff);
   }
 
   /**
    * Get top metrics by value
    */
   getTopMetrics(limit = 10): Array<{ metric: string; value: number; timestamp: Date }> {
-    const recent = this.samples.filter(s => 
-      s.timestamp >= new Date(Date.now() - 300000) // 5 minutes
+    const recent = this.samples.filter(
+      (s) => s.timestamp >= new Date(Date.now() - 300000), // 5 minutes
     );
-    
+
     const byMetric = new Map<string, number>();
     const timestamps = new Map<string, Date>();
-    
+
     for (const sample of recent) {
       byMetric.set(sample.metric, (byMetric.get(sample.metric) || 0) + sample.value);
       timestamps.set(sample.metric, sample.timestamp);
     }
-    
+
     return Array.from(byMetric.entries())
       .sort((a, b) => b[1] - a[1])
       .slice(0, limit)
@@ -295,14 +292,14 @@ export class CoordinationMetricsCollector {
     this.eventBus.on(SystemEvents.TASK_COMPLETED, (data: any) => {
       this.counters.completedTasks++;
       this.gauges.activeTasks = Math.max(0, this.gauges.activeTasks - 1);
-      
+
       const startTime = this.taskStartTimes.get(data.taskId);
       if (startTime) {
         const duration = new Date().getTime() - startTime.getTime();
         this.histograms.taskDurations.push(duration);
         this.taskStartTimes.delete(data.taskId);
       }
-      
+
       this.recordMetric('task.completed', 1);
     });
 
@@ -354,14 +351,14 @@ export class CoordinationMetricsCollector {
     this.eventBus.on(SystemEvents.RESOURCE_RELEASED, (data: any) => {
       this.gauges.freeResources++;
       this.gauges.lockedResources = Math.max(0, this.gauges.lockedResources - 1);
-      
+
       const startTime = this.lockStartTimes.get(data.resourceId);
       if (startTime) {
         const duration = new Date().getTime() - startTime.getTime();
         this.histograms.lockDurations.push(duration);
         this.lockStartTimes.delete(data.resourceId);
       }
-      
+
       this.recordMetric('resource.released', 1);
     });
 
@@ -380,14 +377,14 @@ export class CoordinationMetricsCollector {
 
     this.eventBus.on(SystemEvents.MESSAGE_RECEIVED, (data: any) => {
       this.counters.messagesReceived++;
-      
+
       const startTime = this.messageStartTimes.get(data.message.id);
       if (startTime) {
         const duration = new Date().getTime() - startTime.getTime();
         this.histograms.messageDurations.push(duration);
         this.messageStartTimes.delete(data.message.id);
       }
-      
+
       this.recordMetric('message.received', 1);
     });
 
@@ -428,10 +425,10 @@ export class CoordinationMetricsCollector {
    */
   private collectMetrics(): void {
     const metrics = this.getCurrentMetrics();
-    
+
     // Emit metrics event
     this.eventBus.emit('metrics:coordination', metrics);
-    
+
     // Log summary
     this.logger.debug('Coordination metrics collected', {
       activeTasks: metrics.taskMetrics.activeTasks,
@@ -455,13 +452,13 @@ export class CoordinationMetricsCollector {
   private getTasksByPriority(): Record<string, number> {
     const priorities = ['low', 'medium', 'high', 'critical'];
     const result: Record<string, number> = {};
-    
+
     for (const priority of priorities) {
-      result[priority] = this.samples.filter(s => 
-        s.metric === 'task.created' && s.tags?.priority === priority
+      result[priority] = this.samples.filter(
+        (s) => s.metric === 'task.created' && s.tags?.priority === priority,
       ).length;
     }
-    
+
     return result;
   }
 
@@ -470,20 +467,20 @@ export class CoordinationMetricsCollector {
    */
   private getTasksByType(): Record<string, number> {
     const types = new Set<string>();
-    
+
     for (const sample of this.samples) {
       if (sample.metric === 'task.created' && sample.tags?.type) {
         types.add(sample.tags.type);
       }
     }
-    
+
     const result: Record<string, number> = {};
     for (const type of types) {
-      result[type] = this.samples.filter(s => 
-        s.metric === 'task.created' && s.tags?.type === type
+      result[type] = this.samples.filter(
+        (s) => s.metric === 'task.created' && s.tags?.type === type,
       ).length;
     }
-    
+
     return result;
   }
 
@@ -492,20 +489,20 @@ export class CoordinationMetricsCollector {
    */
   private getAgentsByType(): Record<string, number> {
     const types = new Set<string>();
-    
+
     for (const sample of this.samples) {
       if (sample.metric === 'agent.spawned' && sample.tags?.type) {
         types.add(sample.tags.type);
       }
     }
-    
+
     const result: Record<string, number> = {};
     for (const type of types) {
-      result[type] = this.samples.filter(s => 
-        s.metric === 'agent.spawned' && s.tags?.type === type
+      result[type] = this.samples.filter(
+        (s) => s.metric === 'agent.spawned' && s.tags?.type === type,
       ).length;
     }
-    
+
     return result;
   }
 
@@ -565,22 +562,22 @@ export class CoordinationMetricsCollector {
     this.taskStartTimes.clear();
     this.messageStartTimes.clear();
     this.lockStartTimes.clear();
-    
+
     // Reset counters
     for (const key in this.counters) {
       (this.counters as any)[key] = 0;
     }
-    
+
     // Reset gauges
     for (const key in this.gauges) {
       (this.gauges as any)[key] = 0;
     }
-    
+
     // Clear histograms
     for (const key in this.histograms) {
       (this.histograms as any)[key] = [];
     }
-    
+
     this.logger.info('Coordination metrics cleared');
   }
-}
+}

package/src/coordination/resources.ts

@@ -1,4 +1,3 @@
-import { getErrorMessage } from '../utils/error-handler.js';
 /**
  * Resource manager for preventing conflicts and deadlocks
  */
@@ -33,19 +32,19 @@ export class ResourceManager {
 
   async initialize(): Promise<void> {
     this.logger.info('Initializing resource manager');
-    
+
     // Set up periodic cleanup
     setInterval(() => this.cleanup(), 30000); // Every 30 seconds
   }
 
   async shutdown(): Promise<void> {
     this.logger.info('Shutting down resource manager');
-    
+
     // Release all locks
     for (const [resourceId, agentId] of this.locks) {
       await this.release(resourceId, agentId);
     }
-    
+
     this.resources.clear();
     this.locks.clear();
     this.waitQueue.clear();
@@ -92,7 +91,7 @@ export class ResourceManager {
 
     const queue = this.waitQueue.get(resourceId)!;
     queue.push(request);
-    
+
     // Sort by priority and timestamp
     queue.sort((a, b) => {
       if (a.priority !== b.priority) {
@@ -101,7 +100,7 @@ export class ResourceManager {
       return a.timestamp.getTime() - b.timestamp.getTime(); // Earlier first
     });
 
-    this.logger.info('Agent added to resource wait queue', { 
+    this.logger.info('Agent added to resource wait queue', {
       resourceId,
       agentId,
       queueLength: queue.length,
@@ -120,7 +119,7 @@ export class ResourceManager {
       }
 
       // Check if our request is still in queue
-      const ourRequest = queue.find(req => req.agentId === agentId);
+      const ourRequest = queue.find((req) => req.agentId === agentId);
       if (!ourRequest) {
         // Request was removed (possibly by cleanup)
         throw new ResourceLockError('Resource request cancelled');
@@ -130,15 +129,16 @@ export class ResourceManager {
     }
 
     // Timeout - remove from queue
-    const index = queue.findIndex(req => req.agentId === agentId);
+    const index = queue.findIndex((req) => req.agentId === agentId);
     if (index !== -1) {
       queue.splice(index, 1);
     }
 
-    throw new ResourceLockError(
-      `Resource acquisition timeout for ${resourceId}`,
-      { resourceId, agentId, timeout: this.config.resourceTimeout },
-    );
+    throw new ResourceLockError(`Resource acquisition timeout for ${resourceId}`, {
+      resourceId,
+      agentId,
+      timeout: this.config.resourceTimeout,
+    });
   }
 
   async release(resourceId: string, agentId: string): Promise<void> {
@@ -146,7 +146,7 @@ export class ResourceManager {
 
     const currentLock = this.locks.get(resourceId);
     if (currentLock !== agentId) {
-      this.logger.warn('Attempted to release unowned resource', { 
+      this.logger.warn('Attempted to release unowned resource', {
         resourceId,
         agentId,
         currentLock,
@@ -161,7 +161,7 @@ export class ResourceManager {
     const queue = this.waitQueue.get(resourceId);
     if (queue && queue.length > 0) {
       const nextRequest = queue.shift()!;
-      
+
       // Grant lock to next in queue
       await this.lockResource(resourceId, nextRequest.agentId);
     }
@@ -173,14 +173,12 @@ export class ResourceManager {
       return;
     }
 
-    this.logger.info('Releasing all resources for agent', { 
+    this.logger.info('Releasing all resources for agent', {
       agentId,
       resourceCount: resources.size,
     });
 
-    const promises = Array.from(resources).map(
-      resourceId => this.release(resourceId, agentId),
-    );
+    const promises = Array.from(resources).map((resourceId) => this.release(resourceId, agentId));
 
     await Promise.all(promises);
     this.agentResources.delete(agentId);
@@ -192,22 +190,19 @@ export class ResourceManager {
 
   getWaitingRequests(): Map<string, string[]> {
     const waiting = new Map<string, string[]>();
-    
+
     for (const [resourceId, queue] of this.waitQueue) {
       if (queue.length > 0) {
-        waiting.set(
-          queue[0].agentId,
-          [...(waiting.get(queue[0].agentId) || []), resourceId],
-        );
+        waiting.set(queue[0].agentId, [...(waiting.get(queue[0].agentId) || []), resourceId]);
       }
     }
-    
+
     return waiting;
   }
 
-  async getHealthStatus(): Promise<{ 
-    healthy: boolean; 
-    error?: string; 
+  async getHealthStatus(): Promise<{
+    healthy: boolean;
+    error?: string;
     metrics?: Record<string, number>;
   }> {
     const totalResources = this.resources.size;
@@ -217,7 +212,7 @@ export class ResourceManager {
 
     for (const queue of this.waitQueue.values()) {
       totalWaiting += queue.length;
-      queue.forEach(req => waitingAgents.add(req.agentId));
+      queue.forEach((req) => waitingAgents.add(req.agentId));
     }
 
     return {
@@ -234,13 +229,13 @@ export class ResourceManager {
 
   private async lockResource(resourceId: string, agentId: string): Promise<void> {
     const resource = this.resources.get(resourceId)!;
-    
+
     resource.locked = true;
     resource.lockedBy = agentId;
     resource.lockedAt = new Date();
-    
+
     this.locks.set(resourceId, agentId);
-    
+
     // Track agent resources
     if (!this.agentResources.has(agentId)) {
       this.agentResources.set(agentId, new Set());
@@ -262,9 +257,9 @@ export class ResourceManager {
     resource.locked = false;
     delete resource.lockedBy;
     delete resource.lockedAt;
-    
+
     this.locks.delete(resourceId);
-    
+
     // Remove from agent resources
     this.agentResources.get(agentId)?.delete(resourceId);
 
@@ -284,10 +279,10 @@ export class ResourceManager {
 
     // Clean up stale wait requests
     for (const [resourceId, queue] of this.waitQueue) {
-      const filtered = queue.filter(req => {
+      const filtered = queue.filter((req) => {
         const age = now - req.timestamp.getTime();
         if (age > this.config.resourceTimeout) {
-          this.logger.warn('Removing stale resource request', { 
+          this.logger.warn('Removing stale resource request', {
             resourceId,
             agentId: req.agentId,
             age,
@@ -310,7 +305,7 @@ export class ResourceManager {
       if (resource?.lockedAt) {
         const lockAge = now - resource.lockedAt.getTime();
         if (lockAge > this.config.resourceTimeout * 2) {
-          this.logger.warn('Force releasing stale lock', { 
+          this.logger.warn('Force releasing stale lock', {
             resourceId,
             agentId,
             lockAge,
@@ -320,4 +315,4 @@ export class ResourceManager {
       }
     }
   }
-}
+}