musubi-sdd 5.0.0 → 5.6.1

package/src/monitoring/release-manager.js

@@ -1,11 +1,21 @@
 /**
  * Release Manager - Release coordination and management
- * 
+ *
  * Provides release management capabilities:
  * - Release planning and tracking
  * - Feature flag management
  * - Rollback procedures
  * - Release notes generation
+ *
+ * Part of MUSUBI v5.0.0 - Production Readiness
+ *
+ * @module monitoring/release-manager
+ * @version 1.0.0
+ *
+ * @traceability
+ * - Requirement: REQ-P5-003 (Release Automation)
+ * - Design: docs/design/tdd-musubi-v5.0.0.md#3.3
+ * - Test: tests/monitoring/release-manager.test.js
  */
 
 const { EventEmitter } = require('events');
@@ -22,7 +32,7 @@ const ReleaseState = {
   PRODUCTION: 'production',
   ROLLBACK: 'rollback',
   COMPLETED: 'completed',
-  CANCELLED: 'cancelled'
+  CANCELLED: 'cancelled',
 };
 
 /**
@@ -33,7 +43,7 @@ const ReleaseType = {
   MINOR: 'minor',
   PATCH: 'patch',
   HOTFIX: 'hotfix',
-  CANARY: 'canary'
+  CANARY: 'canary',
 };
 
 /**
@@ -43,7 +53,7 @@ const FeatureFlagStatus = {
   ENABLED: 'enabled',
   DISABLED: 'disabled',
   PERCENTAGE: 'percentage',
-  USER_LIST: 'user-list'
+  USER_LIST: 'user-list',
 };
 
 /**
@@ -61,35 +71,35 @@ class Release {
     this.createdAt = options.createdAt || new Date();
     this.updatedAt = options.updatedAt || new Date();
     this.completedAt = null;
-    
+
     this.features = options.features || [];
     this.bugFixes = options.bugFixes || [];
     this.breakingChanges = options.breakingChanges || [];
     this.dependencies = options.dependencies || [];
-    
+
     this.rolloutStrategy = options.rolloutStrategy || {
       type: 'percentage',
       stages: [
         { percentage: 1, duration: '1h' },
         { percentage: 10, duration: '2h' },
         { percentage: 50, duration: '4h' },
-        { percentage: 100, duration: null }
-      ]
+        { percentage: 100, duration: null },
+      ],
     };
-    
+
     this.rollbackPlan = options.rollbackPlan || {
       automatic: true,
       triggers: ['error_rate > 5%', 'latency_p99 > 2s'],
-      procedure: []
+      procedure: [],
     };
-    
+
     this.metrics = {
       errorsBefore: null,
       errorsAfter: null,
       latencyBefore: null,
-      latencyAfter: null
+      latencyAfter: null,
     };
-    
+
     this.history = [];
     this._addHistory('created', { version: this.version });
   }
@@ -101,13 +111,22 @@ class Release {
     const validTransitions = {
       [ReleaseState.PLANNING]: [ReleaseState.DEVELOPMENT, ReleaseState.CANCELLED],
       [ReleaseState.DEVELOPMENT]: [ReleaseState.TESTING, ReleaseState.CANCELLED],
-      [ReleaseState.TESTING]: [ReleaseState.STAGING, ReleaseState.DEVELOPMENT, ReleaseState.CANCELLED],
-      [ReleaseState.STAGING]: [ReleaseState.CANARY, ReleaseState.PRODUCTION, ReleaseState.TESTING, ReleaseState.CANCELLED],
+      [ReleaseState.TESTING]: [
+        ReleaseState.STAGING,
+        ReleaseState.DEVELOPMENT,
+        ReleaseState.CANCELLED,
+      ],
+      [ReleaseState.STAGING]: [
+        ReleaseState.CANARY,
+        ReleaseState.PRODUCTION,
+        ReleaseState.TESTING,
+        ReleaseState.CANCELLED,
+      ],
       [ReleaseState.CANARY]: [ReleaseState.PRODUCTION, ReleaseState.ROLLBACK],
       [ReleaseState.PRODUCTION]: [ReleaseState.COMPLETED, ReleaseState.ROLLBACK],
       [ReleaseState.ROLLBACK]: [ReleaseState.TESTING, ReleaseState.CANCELLED],
       [ReleaseState.COMPLETED]: [],
-      [ReleaseState.CANCELLED]: []
+      [ReleaseState.CANCELLED]: [],
     };
 
     const allowed = validTransitions[this.state] || [];
@@ -118,15 +137,15 @@ class Release {
     const previousState = this.state;
     this.state = newState;
     this.updatedAt = new Date();
-    
+
     if (newState === ReleaseState.COMPLETED) {
       this.completedAt = new Date();
     }
 
-    this._addHistory('transition', { 
-      from: previousState, 
+    this._addHistory('transition', {
+      from: previousState,
       to: newState,
-      ...metadata 
+      ...metadata,
     });
 
     return this;
@@ -141,7 +160,7 @@ class Release {
       title: feature.title,
       description: feature.description || '',
       jiraId: feature.jiraId || null,
-      breaking: feature.breaking || false
+      breaking: feature.breaking || false,
     });
     this._addHistory('featureAdded', { feature: feature.title });
     return this;
@@ -156,7 +175,7 @@ class Release {
       title: bugFix.title,
       description: bugFix.description || '',
       jiraId: bugFix.jiraId || null,
-      severity: bugFix.severity || 'medium'
+      severity: bugFix.severity || 'medium',
     });
     this._addHistory('bugFixAdded', { bugFix: bugFix.title });
     return this;
@@ -171,7 +190,7 @@ class Release {
       date: this.completedAt || new Date(),
       features: this.features,
       bugFixes: this.bugFixes,
-      breakingChanges: this.breakingChanges
+      breakingChanges: this.breakingChanges,
     };
 
     if (format === 'markdown') {
@@ -237,7 +256,7 @@ class Release {
     this.history.push({
       action,
       timestamp: new Date(),
-      data
+      data,
     });
   }
 
@@ -266,7 +285,7 @@ class Release {
       breakingChanges: this.breakingChanges,
       rolloutStrategy: this.rolloutStrategy,
       rollbackPlan: this.rollbackPlan,
-      history: this.history
+      history: this.history,
     };
   }
 }
@@ -327,18 +346,19 @@ class FeatureFlag {
     switch (this.status) {
       case FeatureFlagStatus.ENABLED:
         return true;
-      
+
       case FeatureFlagStatus.DISABLED:
         return false;
-      
+
       case FeatureFlagStatus.USER_LIST:
         return this.userList.includes(userId);
-      
-      case FeatureFlagStatus.PERCENTAGE:
+
+      case FeatureFlagStatus.PERCENTAGE: {
         // Consistent hashing based on userId
         const hash = this._hashString(`${this.key}:${userId}`);
-        return (hash % 100) < this.percentage;
-      
+        return hash % 100 < this.percentage;
+      }
+
       default:
         return false;
     }
@@ -352,7 +372,7 @@ class FeatureFlag {
     let hash = 0;
     for (let i = 0; i < str.length; i++) {
       const char = str.charCodeAt(i);
-      hash = ((hash << 5) - hash) + char;
+      hash = (hash << 5) - hash + char;
       hash = hash & hash;
     }
     return Math.abs(hash);
@@ -368,7 +388,7 @@ class FeatureFlag {
       userList: this.userList,
       createdAt: this.createdAt,
       updatedAt: this.updatedAt,
-      metadata: this.metadata
+      metadata: this.metadata,
     };
   }
 }
@@ -383,7 +403,7 @@ class ReleaseManager extends EventEmitter {
     this.featureFlags = new Map();
     this.options = {
       autoGenerateNotes: options.autoGenerateNotes !== false,
-      ...options
+      ...options,
     };
   }
 
@@ -421,14 +441,14 @@ class ReleaseManager extends EventEmitter {
    */
   listReleases(filter = {}) {
     let releases = [...this.releases.values()];
-    
+
     if (filter.state) {
       releases = releases.filter(r => r.state === filter.state);
     }
     if (filter.type) {
       releases = releases.filter(r => r.type === filter.type);
     }
-    
+
     return releases.map(r => r.toJSON());
   }
 
@@ -440,7 +460,7 @@ class ReleaseManager extends EventEmitter {
     if (!release) {
       throw new Error(`Release not found: ${releaseId}`);
     }
-    
+
     release.transitionTo(newState, metadata);
     this.emit('releaseTransitioned', { release, newState });
     return release;
@@ -476,7 +496,7 @@ class ReleaseManager extends EventEmitter {
   isFeatureEnabled(flagKey, userId = null) {
     const flag = this.featureFlags.get(flagKey);
     if (!flag) return false;
-    
+
     if (userId) {
       return flag.isEnabledFor(userId);
     }
@@ -531,41 +551,41 @@ class ReleaseManager extends EventEmitter {
           order: 1,
           action: 'notify',
           description: 'Notify team of rollback initiation',
-          command: null
+          command: null,
         },
         {
           order: 2,
           action: 'disable-flags',
           description: 'Disable all new feature flags',
-          command: 'musubi release disable-flags --version ' + release.version
+          command: 'musubi release disable-flags --version ' + release.version,
         },
         {
           order: 3,
           action: 'scale-down',
           description: 'Scale down new deployment',
-          command: 'kubectl scale deployment app-v' + release.version + ' --replicas=0'
+          command: 'kubectl scale deployment app-v' + release.version + ' --replicas=0',
         },
         {
           order: 4,
           action: 'traffic-shift',
           description: 'Shift traffic to previous version',
-          command: 'kubectl rollout undo deployment/app'
+          command: 'kubectl rollout undo deployment/app',
         },
         {
           order: 5,
           action: 'verify',
           description: 'Verify rollback success',
-          command: 'curl -f http://app/health'
+          command: 'curl -f http://app/health',
         },
         {
           order: 6,
           action: 'notify-complete',
           description: 'Notify team of rollback completion',
-          command: null
-        }
+          command: null,
+        },
       ],
       automaticTriggers: release.rollbackPlan.triggers,
-      estimatedDuration: '5-10 minutes'
+      estimatedDuration: '5-10 minutes',
     };
   }
 
@@ -583,7 +603,7 @@ class ReleaseManager extends EventEmitter {
       totalFeatureFlags: flags.length,
       enabledFlags: flags.filter(f => f.status === FeatureFlagStatus.ENABLED).length,
       disabledFlags: flags.filter(f => f.status === FeatureFlagStatus.DISABLED).length,
-      percentageFlags: flags.filter(f => f.status === FeatureFlagStatus.PERCENTAGE).length
+      percentageFlags: flags.filter(f => f.status === FeatureFlagStatus.PERCENTAGE).length,
     };
   }
 
@@ -611,12 +631,12 @@ module.exports = {
   Release,
   FeatureFlag,
   ReleaseManager,
-  
+
   // Constants
   ReleaseState,
   ReleaseType,
   FeatureFlagStatus,
-  
+
   // Factory
-  createReleaseManager
+  createReleaseManager,
 };

package/src/orchestration/agent-skill-binding.js

@@ -1,7 +1,7 @@
 /**
  * Agent-Skill Binding - Dynamic capability-based skill assignment
  * Sprint 3.3: Skill System Architecture
- * 
+ *
  * Features:
  * - Agent capability scoring
  * - Dynamic skill-agent matching
@@ -19,7 +19,7 @@ const AgentStatus = {
   AVAILABLE: 'available',
   BUSY: 'busy',
   OFFLINE: 'offline',
-  MAINTENANCE: 'maintenance'
+  MAINTENANCE: 'maintenance',
 };
 
 /**
@@ -40,22 +40,22 @@ class AgentDefinition {
 
   validate() {
     const errors = [];
-    
+
     if (!this.id) {
       errors.push('Agent ID is required');
     }
-    
+
     if (!this.name) {
       errors.push('Agent name is required');
     }
-    
+
     if (!Array.isArray(this.capabilities)) {
       errors.push('Capabilities must be an array');
     }
-    
+
     return {
       valid: errors.length === 0,
-      errors
+      errors,
     };
   }
 
@@ -69,7 +69,7 @@ class AgentDefinition {
       maxConcurrentTasks: this.maxConcurrentTasks,
       priority: this.priority,
       tags: this.tags,
-      metadata: this.metadata
+      metadata: this.metadata,
     };
   }
 }
@@ -93,16 +93,12 @@ class BindingRecord {
   updateStats(success, executionTime) {
     this.executionCount++;
     const successWeight = success ? 1 : 0;
-    this.successRate = (
-      (this.successRate * (this.executionCount - 1) + successWeight) / 
-      this.executionCount
-    );
-    this.averageExecutionTime = (
-      (this.averageExecutionTime * (this.executionCount - 1) + executionTime) / 
-      this.executionCount
-    );
+    this.successRate =
+      (this.successRate * (this.executionCount - 1) + successWeight) / this.executionCount;
+    this.averageExecutionTime =
+      (this.averageExecutionTime * (this.executionCount - 1) + executionTime) / this.executionCount;
     this.lastExecutedAt = new Date().toISOString();
-    
+
     // Update affinity based on performance
     this.affinity = this._calculateAffinity();
   }
@@ -111,10 +107,10 @@ class BindingRecord {
     // Affinity increases with success rate and execution count
     const successFactor = this.successRate * 50;
     const experienceFactor = Math.min(this.executionCount / 10, 30);
-    const recencyFactor = this.lastExecutedAt 
+    const recencyFactor = this.lastExecutedAt
       ? Math.max(0, 20 - (Date.now() - new Date(this.lastExecutedAt).getTime()) / 86400000)
       : 0;
-    
+
     return Math.round(successFactor + experienceFactor + recencyFactor);
   }
 
@@ -127,7 +123,7 @@ class BindingRecord {
       executionCount: this.executionCount,
       successRate: this.successRate,
       averageExecutionTime: this.averageExecutionTime,
-      lastExecutedAt: this.lastExecutedAt
+      lastExecutedAt: this.lastExecutedAt,
     };
   }
 }
@@ -215,13 +211,13 @@ class AgentSkillBinding extends EventEmitter {
     this.agentLoad = new Map();
     this.bindings = new Map(); // Map<agentId, Map<skillId, BindingRecord>>
     this.matcher = new CapabilityMatcher();
-    
+
     // Options
     this.options = {
       autoBinding: options.autoBinding !== false,
       minMatchScore: options.minMatchScore || 50,
       enableLoadBalancing: options.enableLoadBalancing !== false,
-      affinityWeight: options.affinityWeight || 0.3
+      affinityWeight: options.affinityWeight || 0.3,
     };
 
     // Listen to skill registry events
@@ -238,9 +234,7 @@ class AgentSkillBinding extends EventEmitter {
    * Register an agent
    */
   registerAgent(agentDef) {
-    const agent = agentDef instanceof AgentDefinition 
-      ? agentDef 
-      : new AgentDefinition(agentDef);
+    const agent = agentDef instanceof AgentDefinition ? agentDef : new AgentDefinition(agentDef);
 
     const validation = agent.validate();
     if (!validation.valid) {
@@ -262,7 +256,7 @@ class AgentSkillBinding extends EventEmitter {
     }
 
     this.emit('agent-registered', { agentId: agent.id, agent });
-    
+
     return agent;
   }
 
@@ -280,7 +274,7 @@ class AgentSkillBinding extends EventEmitter {
     this.bindings.delete(agentId);
 
     this.emit('agent-unregistered', { agentId });
-    
+
     return true;
   }
 
@@ -348,19 +342,19 @@ class AgentSkillBinding extends EventEmitter {
 
     // Check permissions
     if (skill.permissions && skill.permissions.length > 0) {
-      const hasPermission = skill.permissions.every(
-        p => agent.permissions.includes(p)
-      );
+      const hasPermission = skill.permissions.every(p => agent.permissions.includes(p));
       if (!hasPermission) {
         throw new Error(`Agent '${agentId}' lacks required permissions for skill '${skillId}'`);
       }
     }
 
     // Calculate score
-    const score = options.score || this.matcher.calculateScore(
-      agent.capabilities,
-      skill.tags // Use tags as capability requirements
-    );
+    const score =
+      options.score ||
+      this.matcher.calculateScore(
+        agent.capabilities,
+        skill.tags // Use tags as capability requirements
+      );
 
     const record = new BindingRecord(agentId, skillId, { score });
     this.bindings.get(agentId).set(skillId, record);
@@ -405,7 +399,7 @@ class AgentSkillBinding extends EventEmitter {
   /**
    * Find best agent for a skill
    */
-  findBestAgentForSkill(skillId, options = {}) {
+  findBestAgentForSkill(skillId, _options = {}) {
     const skill = this.skillRegistry?.getSkill(skillId);
     if (!skill) {
       return null;
@@ -427,9 +421,7 @@ class AgentSkillBinding extends EventEmitter {
 
       // Check permissions
       if (skill.permissions && skill.permissions.length > 0) {
-        const hasPermission = skill.permissions.every(
-          p => agent.permissions.includes(p)
-        );
+        const hasPermission = skill.permissions.every(p => agent.permissions.includes(p));
         if (!hasPermission) {
           continue;
         }
@@ -454,7 +446,7 @@ class AgentSkillBinding extends EventEmitter {
           agent,
           binding,
           finalScore,
-          load
+          load,
         });
       }
     }
@@ -487,7 +479,7 @@ class AgentSkillBinding extends EventEmitter {
           agent,
           score,
           status: this.agentStatus.get(agentId),
-          load: this.agentLoad.get(agentId) || 0
+          load: this.agentLoad.get(agentId) || 0,
         });
       }
     }
@@ -516,7 +508,7 @@ class AgentSkillBinding extends EventEmitter {
     }
 
     this.emit('agent-acquired', { agentId, currentLoad: load + 1 });
-    
+
     return true;
   }
 
@@ -541,7 +533,7 @@ class AgentSkillBinding extends EventEmitter {
     }
 
     this.emit('agent-released', { agentId, currentLoad: load - 1 });
-    
+
     return true;
   }
 
@@ -578,7 +570,7 @@ class AgentSkillBinding extends EventEmitter {
       [AgentStatus.AVAILABLE]: 0,
       [AgentStatus.BUSY]: 0,
       [AgentStatus.OFFLINE]: 0,
-      [AgentStatus.MAINTENANCE]: 0
+      [AgentStatus.MAINTENANCE]: 0,
     };
 
     let totalBindings = 0;
@@ -586,8 +578,8 @@ class AgentSkillBinding extends EventEmitter {
 
     for (const [agentId, status] of this.agentStatus) {
       statusCounts[status]++;
-      totalBindings += (this.bindings.get(agentId)?.size || 0);
-      totalLoad += (this.agentLoad.get(agentId) || 0);
+      totalBindings += this.bindings.get(agentId)?.size || 0;
+      totalLoad += this.agentLoad.get(agentId) || 0;
     }
 
     return {
@@ -595,7 +587,7 @@ class AgentSkillBinding extends EventEmitter {
       statusCounts,
       totalBindings,
       totalLoad,
-      averageLoad: this.agents.size > 0 ? totalLoad / this.agents.size : 0
+      averageLoad: this.agents.size > 0 ? totalLoad / this.agents.size : 0,
     };
   }
 
@@ -651,5 +643,5 @@ module.exports = {
   AgentDefinition,
   BindingRecord,
   CapabilityMatcher,
-  AgentStatus
+  AgentStatus,
 };