vibecodingmachine-core 2026.3.14-1528 → 2026.6.17-1835

package/src/agents/OperationTypes.js

@@ -1,6 +1,6 @@
 /**
  * Operation Types and Status Enumerations
- * 
+ *
  * Defines types and statuses for logged operations.
  * Follows constitutional requirements: <555 lines, test-first approach.
  */
@@ -12,16 +12,16 @@
 const OperationType = {
   // Agent installation
   INSTALL: 'INSTALL',
-  
+
   // Agent verification
   VERIFY: 'VERIFY',
-  
+
   // Status check
   CHECK_STATUS: 'CHECK_STATUS',
-  
+
   // Agent update
   UPDATE: 'UPDATE',
-  
+
   // Agent removal
   UNINSTALL: 'UNINSTALL'
 };
@@ -33,16 +33,16 @@ const OperationType = {
 const OperationStatus = {
   // Operation initiated
   STARTED: 'STARTED',
-  
+
   // Operation finished successfully
   COMPLETED: 'COMPLETED',
-  
+
   // Operation failed
   FAILED: 'FAILED',
-  
+
   // Operation timed out
   TIMEOUT: 'TIMEOUT',
-  
+
   // Operation was cancelled
   CANCELLED: 'CANCELLED'
 };
@@ -186,7 +186,7 @@ function isValidOperationStatus(status) {
 function getValidFinalStatuses(type) {
   // All operations can end in these states
   const commonFinal = [OperationStatus.COMPLETED, OperationStatus.FAILED, OperationStatus.TIMEOUT];
-  
+
   // Add type-specific final statuses
   switch (type) {
     case OperationType.INSTALL:
@@ -277,11 +277,11 @@ function getRetryDelay(status, attempt = 1) {
   };
 
   const baseDelay = baseDelays[status] || 1000;
-  
+
   // Exponential backoff with jitter
   const exponentialDelay = baseDelay * Math.pow(2, attempt - 1);
   const jitter = Math.random() * 1000; // Add up to 1 second jitter
-  
+
   return Math.min(exponentialDelay + jitter, 60000); // Cap at 1 minute
 }
 

package/src/agents/PerformanceMetricsCollector.js

@@ -1,6 +1,6 @@
 /**
  * Performance Metrics Collector
- * 
+ *
  * Collects and manages performance metrics for agent operations.
  * Follows constitutional requirements: <555 lines, test-first approach.
  */
@@ -18,17 +18,17 @@ class PerformanceMetricsCollector {
     this.enabled = options.enabled !== false; // Default to enabled
     this.maxHistorySize = options.maxHistorySize || 1000;
     this.flushInterval = options.flushInterval || 60000; // 1 minute
-    
+
     // Metrics storage
     this.metrics = {
       operations: new Map(),
       system: new Map(),
       performance: new Map()
     };
-    
+
     // Active operations tracking
     this.activeOperations = new Map();
-    
+
     // Start periodic flush if enabled
     if (this.enabled) {
       this.startPeriodicFlush();
@@ -43,10 +43,10 @@ class PerformanceMetricsCollector {
    */
   startOperation(operationId, context = {}) {
     if (!this.enabled) return this.createNoOpTracker();
-    
+
     const startTime = process.hrtime.bigint();
     const startMemory = this.getMemoryUsage();
-    
+
     const operation = {
       id: operationId,
       type: context.type || 'UNKNOWN',
@@ -58,16 +58,16 @@ class PerformanceMetricsCollector {
       phases: [],
       checkpoints: []
     };
-    
+
     this.activeOperations.set(operationId, operation);
-    
+
     // Record initial system metrics
     this.recordSystemMetrics('operation_start', {
       operationId,
       type: operation.type,
       agentId: operation.agentId
     });
-    
+
     if (this.logger) {
       this.logger.debug('Started operation tracking', {
         operationId,
@@ -75,7 +75,7 @@ class PerformanceMetricsCollector {
         agentId: operation.agentId
       });
     }
-    
+
     return {
       addPhase: (phaseName, details = {}) => this.addOperationPhase(operationId, phaseName, details),
       addCheckpoint: (checkpointName, details = {}) => this.addOperationCheckpoint(operationId, checkpointName, details),
@@ -93,19 +93,19 @@ class PerformanceMetricsCollector {
    */
   addOperationPhase(operationId, phaseName, details = {}) {
     if (!this.enabled) return;
-    
+
     const operation = this.activeOperations.get(operationId);
     if (!operation) return;
-    
+
     const phase = {
       name: phaseName,
       startTime: Date.now(),
       startMemory: this.getMemoryUsage(),
       details
     };
-    
+
     operation.phases.push(phase);
-    
+
     if (this.logger) {
       this.logger.debug(`Added operation phase: ${phaseName}`, {
         operationId,
@@ -123,19 +123,19 @@ class PerformanceMetricsCollector {
    */
   addOperationCheckpoint(operationId, checkpointName, details = {}) {
     if (!this.enabled) return;
-    
+
     const operation = this.activeOperations.get(operationId);
     if (!operation) return;
-    
+
     const checkpoint = {
       name: checkpointName,
       timestamp: Date.now(),
       memory: this.getMemoryUsage(),
       details
     };
-    
+
     operation.checkpoints.push(checkpoint);
-    
+
     if (this.logger) {
       this.logger.debug(`Added operation checkpoint: ${checkpointName}`, {
         operationId,
@@ -152,20 +152,20 @@ class PerformanceMetricsCollector {
    */
   completeOperation(operationId, result = {}) {
     if (!this.enabled) return;
-    
+
     const operation = this.activeOperations.get(operationId);
     if (!operation) return;
-    
+
     const endTime = process.hrtime.bigint();
     const endMemory = this.getMemoryUsage();
-    
+
     operation.endTime = Number(endTime / 1000000n);
     operation.endMemory = endMemory;
     operation.duration = operation.endTime - operation.startTime;
     operation.memoryDelta = endMemory - operation.startMemory;
     operation.status = 'COMPLETED';
     operation.result = result;
-    
+
     // Complete final phase
     if (operation.phases.length > 0) {
       const lastPhase = operation.phases[operation.phases.length - 1];
@@ -175,13 +175,13 @@ class PerformanceMetricsCollector {
         lastPhase.memoryDelta = endMemory - lastPhase.startMemory;
       }
     }
-    
+
     // Store in metrics
     this.storeOperationMetrics(operation);
-    
+
     // Remove from active operations
     this.activeOperations.delete(operationId);
-    
+
     // Record completion metrics
     this.recordSystemMetrics('operation_complete', {
       operationId,
@@ -190,7 +190,7 @@ class PerformanceMetricsCollector {
       memoryDelta: operation.memoryDelta,
       success: true
     });
-    
+
     if (this.logger) {
       this.logger.info('Completed operation tracking', {
         operationId,
@@ -208,26 +208,26 @@ class PerformanceMetricsCollector {
    */
   failOperation(operationId, error) {
     if (!this.enabled) return;
-    
+
     const operation = this.activeOperations.get(operationId);
     if (!operation) return;
-    
+
     const endTime = process.hrtime.bigint();
     const endMemory = this.getMemoryUsage();
-    
+
     operation.endTime = Number(endTime / 1000000n);
     operation.endMemory = endMemory;
     operation.duration = operation.endTime - operation.startTime;
     operation.memoryDelta = endMemory - operation.startMemory;
     operation.status = 'FAILED';
     operation.error = error.message;
-    
+
     // Store in metrics
     this.storeOperationMetrics(operation);
-    
+
     // Remove from active operations
     this.activeOperations.delete(operationId);
-    
+
     // Record failure metrics
     this.recordSystemMetrics('operation_fail', {
       operationId,
@@ -236,7 +236,7 @@ class PerformanceMetricsCollector {
       error: error.message,
       success: false
     });
-    
+
     if (this.logger) {
       this.logger.warn('Failed operation tracking', {
         operationId,
@@ -254,12 +254,12 @@ class PerformanceMetricsCollector {
   storeOperationMetrics(operation) {
     const typeMetrics = this.metrics.operations.get(operation.type) || [];
     typeMetrics.push(operation);
-    
+
     // Limit history size
     if (typeMetrics.length > this.maxHistorySize) {
       typeMetrics.shift();
     }
-    
+
     this.metrics.operations.set(operation.type, typeMetrics);
   }
 
@@ -270,11 +270,11 @@ class PerformanceMetricsCollector {
    */
   recordSystemMetrics(event, data = {}) {
     if (!this.enabled) return;
-    
+
     const timestamp = Date.now();
     const memory = this.getMemoryUsage();
     const cpu = this.getCpuUsage();
-    
+
     const metric = {
       timestamp,
       event,
@@ -282,15 +282,15 @@ class PerformanceMetricsCollector {
       cpu,
       data
     };
-    
+
     const systemMetrics = this.metrics.system.get(event) || [];
     systemMetrics.push(metric);
-    
+
     // Limit history size
     if (systemMetrics.length > this.maxHistorySize) {
       systemMetrics.shift();
     }
-    
+
     this.metrics.system.set(event, systemMetrics);
   }
 
@@ -302,10 +302,10 @@ class PerformanceMetricsCollector {
   getOperationMetrics(operationId) {
     const operation = this.activeOperations.get(operationId);
     if (!operation) return null;
-    
+
     const currentMemory = this.getMemoryUsage();
     const currentTime = Date.now();
-    
+
     return {
       id: operation.id,
       type: operation.type,
@@ -326,35 +326,35 @@ class PerformanceMetricsCollector {
    */
   getAggregatedMetrics(filters = {}) {
     if (!this.enabled) return {};
-    
+
     const { operationType, timeRange, agentId } = filters;
     const now = Date.now();
     const timeLimit = timeRange ? now - timeRange : 0;
-    
+
     const result = {
       summary: {},
       operations: {},
       performance: {},
       system: {}
     };
-    
+
     // Aggregate operation metrics
     for (const [type, operations] of this.metrics.operations) {
       if (operationType && type !== operationType) continue;
-      
-      const filteredOps = operations.filter(op => 
-        op.endTime >= timeLimit && 
+
+      const filteredOps = operations.filter(op =>
+        op.endTime >= timeLimit &&
         (!agentId || op.agentId === agentId)
       );
-      
+
       if (filteredOps.length === 0) continue;
-      
+
       const completed = filteredOps.filter(op => op.status === 'COMPLETED');
       const failed = filteredOps.filter(op => op.status === 'FAILED');
-      
+
       const durations = completed.map(op => op.duration);
       const memoryDeltas = completed.map(op => op.memoryDelta);
-      
+
       result.operations[type] = {
         total: filteredOps.length,
         completed: completed.length,
@@ -366,7 +366,7 @@ class PerformanceMetricsCollector {
         avgMemoryDelta: memoryDeltas.length > 0 ? memoryDeltas.reduce((a, b) => a + b, 0) / memoryDeltas.length : 0
       };
     }
-    
+
     // System metrics summary
     for (const [event, metrics] of this.metrics.system) {
       const recentMetrics = metrics.filter(m => m.timestamp >= timeLimit);
@@ -376,7 +376,7 @@ class PerformanceMetricsCollector {
         avgCpu: recentMetrics.length > 0 ? recentMetrics.reduce((sum, m) => sum + (m.cpu || 0), 0) / recentMetrics.length : 0
       };
     }
-    
+
     return result;
   }
 
@@ -403,7 +403,7 @@ class PerformanceMetricsCollector {
    */
   startPeriodicFlush() {
     if (this.flushTimer) return;
-    
+
     this.flushTimer = setInterval(() => {
       this.flushMetrics();
     }, this.flushInterval);
@@ -424,16 +424,16 @@ class PerformanceMetricsCollector {
    */
   flushMetrics() {
     if (!this.enabled) return;
-    
+
     const metrics = this.getAggregatedMetrics();
-    
+
     if (this.logger) {
       this.logger.debug('Flushing performance metrics', {
         operationsCount: Object.keys(metrics.operations).length,
         systemEventsCount: Object.keys(metrics.system).length
       });
     }
-    
+
     // Could integrate with external metrics systems here
     // For now, just log summary
   }
@@ -460,7 +460,7 @@ class PerformanceMetricsCollector {
     this.metrics.system.clear();
     this.metrics.performance.clear();
     this.activeOperations.clear();
-    
+
     if (this.logger) {
       this.logger.info('Reset all performance metrics');
     }

package/src/agents/SecurityValidationService.js

@@ -1,6 +1,6 @@
 /**
  * Security Validation Service
- * 
+ *
  * Provides security validation for downloaded packages and files.
  * Follows constitutional requirements: <555 lines, test-first approach.
  */
@@ -32,7 +32,7 @@ class SecurityValidationService {
       /\.vbs$/i, // VBScript files
       /\.ps1$/i  // PowerShell scripts (require validation)
     ];
-    
+
     // Known malicious signatures (simplified)
     this.maliciousSignatures = [
       'eval(',
@@ -93,7 +93,7 @@ class SecurityValidationService {
       // File extension validation
       const ext = path.extname(filePath).toLowerCase();
       result.security.fileExtension = ext;
-      
+
       if (!this.allowedExtensions.includes(ext)) {
         result.warnings.push(`File extension '${ext}' is not in the allowed list`);
       }
@@ -108,13 +108,13 @@ class SecurityValidationService {
 
       // Read file for content validation
       const content = await fs.readFile(filePath);
-      
+
       // Calculate file hash
       result.security.fileHash = this.calculateHash(content);
-      
+
       // Content validation
       await this.validateContent(content, result);
-      
+
       // Permission validation (Windows specific)
       if (process.platform === 'win32') {
         await this.validateFilePermissions(filePath, result);
@@ -135,7 +135,7 @@ class SecurityValidationService {
     } catch (error) {
       result.valid = false;
       result.errors.push(`Validation error: ${error.message}`);
-      
+
       if (this.logger) {
         this.logger.error('File security validation failed', {
           filePath,
@@ -154,7 +154,7 @@ class SecurityValidationService {
    */
   async validateContent(content, result) {
     const contentString = content.toString('utf8', 0, Math.min(content.length, 1024 * 1024)); // First 1MB
-    
+
     // Check for malicious signatures
     for (const signature of this.maliciousSignatures) {
       if (contentString.includes(signature)) {
@@ -211,7 +211,7 @@ class SecurityValidationService {
     try {
       // Check if file is writable (potential security risk)
       await fs.access(filePath, fs.constants.W_OK);
-      
+
       // Check file attributes (Windows specific)
       const stats = await fs.stat(filePath);
       result.security.permissions = {
@@ -272,7 +272,7 @@ class SecurityValidationService {
     } catch (error) {
       result.valid = false;
       result.errors.push(`Signature validation error: ${error.message}`);
-      
+
       if (this.logger) {
         this.logger.error('Package signature validation failed', {
           filePath,
@@ -299,7 +299,7 @@ class SecurityValidationService {
 
     try {
       const urlObj = new URL(url);
-      
+
       // Check protocol
       if (!['https:', 'http:'].includes(urlObj.protocol)) {
         result.valid = false;
@@ -386,14 +386,14 @@ class SecurityValidationService {
     for (const filePath of filePaths) {
       const fileResult = await this.validateFile(filePath, context);
       results.files.push(fileResult);
-      
+
       if (fileResult.valid) {
         results.summary.valid++;
       } else {
         results.summary.invalid++;
         results.valid = false;
       }
-      
+
       results.summary.warnings += fileResult.warnings.length;
       results.summary.errors += fileResult.errors.length;
     }
@@ -469,19 +469,19 @@ class SecurityValidationService {
     if (result.security.maliciousContent) {
       return 'HIGH';
     }
-    
+
     if (result.errors.length > 0) {
       return 'HIGH';
     }
-    
+
     if (result.warnings.length > 3) {
       return 'MEDIUM';
     }
-    
+
     if (result.warnings.length > 0) {
       return 'LOW';
     }
-    
+
     return 'LOW';
   }