musubi-sdd 5.0.0 → 5.6.1

package/src/orchestration/orchestration-engine.js

@@ -1,6 +1,6 @@
 /**
  * OrchestrationEngine - Core engine for multi-skill orchestration
- * 
+ *
  * Implements ag2-inspired patterns for skill coordination:
  * - Auto: Automatic skill selection based on task
  * - Sequential: Linear skill execution
@@ -25,7 +25,7 @@ const PatternType = {
   SWARM: 'swarm',
   HUMAN_IN_LOOP: 'human-in-loop',
   HANDOFF: 'handoff',
-  TRIAGE: 'triage'
+  TRIAGE: 'triage',
 };
 
 /**
@@ -37,7 +37,7 @@ const ExecutionStatus = {
   COMPLETED: 'completed',
   FAILED: 'failed',
   CANCELLED: 'cancelled',
-  WAITING_FOR_HUMAN: 'waiting-for-human'
+  WAITING_FOR_HUMAN: 'waiting-for-human',
 };
 
 /**
@@ -47,7 +47,7 @@ const Priority = {
   P0: 0, // Critical - must complete first
   P1: 1, // High - important
   P2: 2, // Medium - nice to have
-  P3: 3  // Low - future enhancement
+  P3: 3, // Low - future enhancement
 };
 
 /**
@@ -117,11 +117,11 @@ class ExecutionContext {
       input: this.input,
       output: this.output,
       error: this.error,
-      children: this.children.map(c => c.toJSON ? c.toJSON() : c),
+      children: this.children.map(c => (c.toJSON ? c.toJSON() : c)),
       metadata: this.metadata,
       startTime: this.startTime,
       endTime: this.endTime,
-      duration: this.getDuration()
+      duration: this.getDuration(),
     };
   }
 }
@@ -140,7 +140,7 @@ class OrchestrationEngine extends EventEmitter {
       timeout: options.timeout || 300000, // 5 minutes default
       retryCount: options.retryCount || 3,
       retryDelay: options.retryDelay || 1000,
-      ...options
+      ...options,
     };
     this.skillResolver = options.skillResolver || null;
     this.humanGate = options.humanGate || null;
@@ -227,8 +227,8 @@ class OrchestrationEngine extends EventEmitter {
       input: options.input || {},
       metadata: {
         pattern: patternName,
-        ...options.metadata
-      }
+        ...options.metadata,
+      },
     });
 
     this.activeContexts.set(context.id, context);
@@ -236,12 +236,12 @@ class OrchestrationEngine extends EventEmitter {
 
     try {
       context.start();
-      
+
       const result = await this._executeWithTimeout(
         () => pattern.execute(context, this),
         this.config.timeout
       );
-      
+
       context.complete(result);
       this.emit('executionCompleted', context);
     } catch (error) {
@@ -294,7 +294,7 @@ class OrchestrationEngine extends EventEmitter {
       skill: skillName,
       input,
       parentId: parentContext?.id,
-      metadata: { directExecution: true }
+      metadata: { directExecution: true },
     });
 
     if (parentContext) {
@@ -305,7 +305,7 @@ class OrchestrationEngine extends EventEmitter {
 
     try {
       context.start();
-      
+
       let result;
       if (typeof skill.execute === 'function') {
         result = await skill.execute(input, this);
@@ -314,7 +314,7 @@ class OrchestrationEngine extends EventEmitter {
       } else {
         throw new Error(`Skill '${skillName}' is not executable`);
       }
-      
+
       context.complete(result);
       this.emit('skillExecutionCompleted', { skillName, context });
       return result;
@@ -334,7 +334,7 @@ class OrchestrationEngine extends EventEmitter {
     if (this.skillResolver) {
       return this.skillResolver.resolve(task, this.listSkills());
     }
-    
+
     // Default: return first matching skill based on keywords
     const taskLower = task.toLowerCase();
     for (const [name, skill] of this.skills) {
@@ -343,7 +343,7 @@ class OrchestrationEngine extends EventEmitter {
         return name;
       }
     }
-    
+
     return null;
   }
 
@@ -356,11 +356,11 @@ class OrchestrationEngine extends EventEmitter {
   async requestHumanValidation(context, question) {
     context.waitForHuman();
     this.emit('humanValidationRequested', { context, question });
-    
+
     if (this.humanGate) {
       return this.humanGate.request(question, context);
     }
-    
+
     // Default: auto-approve
     return { approved: true, feedback: 'Auto-approved (no human gate configured)' };
   }
@@ -375,7 +375,7 @@ class OrchestrationEngine extends EventEmitter {
       activeExecutions: contexts.length,
       patterns: this.listPatterns(),
       skills: this.listSkills(),
-      contexts: contexts.map(c => c.toJSON())
+      contexts: contexts.map(c => c.toJSON()),
     };
   }
 
@@ -396,7 +396,7 @@ class OrchestrationEngine extends EventEmitter {
    * Cancel all active executions
    */
   cancelAll() {
-    for (const [id, context] of this.activeContexts) {
+    for (const [_id, context] of this.activeContexts) {
       context.cancel();
       this.emit('executionCancelled', context);
     }
@@ -409,5 +409,5 @@ module.exports = {
   ExecutionContext,
   PatternType,
   ExecutionStatus,
-  Priority
+  Priority,
 };

package/src/orchestration/pattern-registry.js

@@ -1,6 +1,6 @@
 /**
  * PatternRegistry - Registry for orchestration patterns
- * 
+ *
  * Manages pattern lifecycle and provides pattern discovery
  */
 
@@ -26,10 +26,13 @@ class PatternMetadata {
   matches(criteria = {}) {
     if (criteria.type && this.type !== criteria.type) return false;
     if (criteria.complexity && this.complexity !== criteria.complexity) return false;
-    if (criteria.supportsParallel !== undefined && 
-        this.supportsParallel !== criteria.supportsParallel) return false;
-    if (criteria.requiresHuman !== undefined && 
-        this.requiresHuman !== criteria.requiresHuman) return false;
+    if (
+      criteria.supportsParallel !== undefined &&
+      this.supportsParallel !== criteria.supportsParallel
+    )
+      return false;
+    if (criteria.requiresHuman !== undefined && this.requiresHuman !== criteria.requiresHuman)
+      return false;
     if (criteria.tags && criteria.tags.length > 0) {
       if (!criteria.tags.some(t => this.tags.includes(t))) return false;
     }
@@ -47,7 +50,7 @@ class PatternMetadata {
       useCases: this.useCases,
       complexity: this.complexity,
       supportsParallel: this.supportsParallel,
-      requiresHuman: this.requiresHuman
+      requiresHuman: this.requiresHuman,
     };
   }
 }
@@ -66,7 +69,7 @@ class BasePattern {
    * @param {OrchestrationEngine} engine - Orchestration engine
    * @returns {Promise<any>} Execution result
    */
-  async execute(context, engine) {
+  async execute(_context, _engine) {
     throw new Error('Pattern must implement execute method');
   }
 
@@ -76,7 +79,7 @@ class BasePattern {
    * @param {OrchestrationEngine} engine - Orchestration engine
    * @returns {object} Validation result
    */
-  validate(context, engine) {
+  validate(_context, _engine) {
     return { valid: true, errors: [] };
   }
 
@@ -127,11 +130,14 @@ class PatternRegistry {
 
     // Extract or create metadata
     const patternMeta = pattern.metadata || pattern.getMetadata?.() || {};
-    this.metadata.set(name, new PatternMetadata({
+    this.metadata.set(
       name,
-      ...patternMeta,
-      ...metadata
-    }));
+      new PatternMetadata({
+        name,
+        ...patternMeta,
+        ...metadata,
+      })
+    );
 
     return this;
   }
@@ -188,18 +194,18 @@ class PatternRegistry {
    */
   find(criteria = {}) {
     const results = [];
-    
+
     for (const [name, pattern] of this.patterns) {
       const meta = this.metadata.get(name);
       if (meta && meta.matches(criteria)) {
         results.push({
           name,
           pattern,
-          metadata: meta
+          metadata: meta,
         });
       }
     }
-    
+
     return results;
   }
 
@@ -210,49 +216,59 @@ class PatternRegistry {
    */
   findBestPattern(task) {
     const taskLower = task.toLowerCase();
-    
+
     // Check for parallel keywords
-    if (taskLower.includes('parallel') || 
-        taskLower.includes('concurrent') ||
-        taskLower.includes('simultaneous')) {
+    if (
+      taskLower.includes('parallel') ||
+      taskLower.includes('concurrent') ||
+      taskLower.includes('simultaneous')
+    ) {
       const swarm = this.find({ supportsParallel: true });
       if (swarm.length > 0) return swarm[0].name;
     }
-    
+
     // Check for sequential keywords
-    if (taskLower.includes('sequential') || 
-        taskLower.includes('step by step') ||
-        taskLower.includes('one by one')) {
+    if (
+      taskLower.includes('sequential') ||
+      taskLower.includes('step by step') ||
+      taskLower.includes('one by one')
+    ) {
       if (this.has(PatternType.SEQUENTIAL)) return PatternType.SEQUENTIAL;
     }
-    
+
     // Check for collaboration keywords
-    if (taskLower.includes('discuss') || 
-        taskLower.includes('review') ||
-        taskLower.includes('collaborate')) {
+    if (
+      taskLower.includes('discuss') ||
+      taskLower.includes('review') ||
+      taskLower.includes('collaborate')
+    ) {
       if (this.has(PatternType.GROUP_CHAT)) return PatternType.GROUP_CHAT;
     }
-    
+
     // Check for nested/hierarchical keywords
-    if (taskLower.includes('break down') || 
-        taskLower.includes('decompose') ||
-        taskLower.includes('hierarchical')) {
+    if (
+      taskLower.includes('break down') ||
+      taskLower.includes('decompose') ||
+      taskLower.includes('hierarchical')
+    ) {
       if (this.has(PatternType.NESTED)) return PatternType.NESTED;
     }
-    
+
     // Check for human validation keywords
-    if (taskLower.includes('validate') || 
-        taskLower.includes('approve') ||
-        taskLower.includes('review')) {
+    if (
+      taskLower.includes('validate') ||
+      taskLower.includes('approve') ||
+      taskLower.includes('review')
+    ) {
       if (this.has(PatternType.HUMAN_IN_LOOP)) return PatternType.HUMAN_IN_LOOP;
     }
-    
+
     // Default to auto pattern
     if (this.has(PatternType.AUTO)) return PatternType.AUTO;
-    
+
     // Fall back to sequential
     if (this.has(PatternType.SEQUENTIAL)) return PatternType.SEQUENTIAL;
-    
+
     // Return first available pattern
     const patterns = this.list();
     return patterns.length > 0 ? patterns[0] : null;
@@ -277,18 +293,17 @@ class PatternRegistry {
       total: this.patterns.size,
       patterns: [],
       byType: {},
-      byComplexity: {}
+      byComplexity: {},
     };
 
-    for (const [name, meta] of this.metadata) {
+    for (const [_name, meta] of this.metadata) {
       summary.patterns.push(meta.toJSON());
-      
+
       // Count by type
       summary.byType[meta.type] = (summary.byType[meta.type] || 0) + 1;
-      
+
       // Count by complexity
-      summary.byComplexity[meta.complexity] = 
-        (summary.byComplexity[meta.complexity] || 0) + 1;
+      summary.byComplexity[meta.complexity] = (summary.byComplexity[meta.complexity] || 0) + 1;
     }
 
     return summary;
@@ -315,5 +330,5 @@ module.exports = {
   PatternRegistry,
   PatternMetadata,
   BasePattern,
-  createDefaultRegistry
+  createDefaultRegistry,
 };

package/src/orchestration/patterns/auto.js

@@ -1,6 +1,6 @@
 /**
  * AutoPattern - Automatic skill selection pattern
- * 
+ *
  * Analyzes the task and automatically selects the most appropriate
  * skill to execute. Uses skill metadata and keywords for matching.
  */
@@ -12,10 +12,10 @@ const { PatternType, ExecutionContext, ExecutionStatus } = require('../orchestra
  * Skill match confidence levels
  */
 const ConfidenceLevel = {
-  HIGH: 'high',     // >= 0.8
+  HIGH: 'high', // >= 0.8
   MEDIUM: 'medium', // >= 0.5
-  LOW: 'low',       // >= 0.3
-  NONE: 'none'      // < 0.3
+  LOW: 'low', // >= 0.3
+  NONE: 'none', // < 0.3
 };
 
 /**
@@ -32,11 +32,11 @@ class AutoPattern extends BasePattern {
       useCases: [
         'Dynamic task routing',
         'Intelligent skill selection',
-        'Natural language task processing'
+        'Natural language task processing',
       ],
       complexity: 'medium',
       supportsParallel: false,
-      requiresHuman: false
+      requiresHuman: false,
     });
 
     this.options = {
@@ -44,7 +44,7 @@ class AutoPattern extends BasePattern {
       fallbackSkill: options.fallbackSkill || null,
       multiMatch: options.multiMatch || false, // Execute multiple matching skills
       maxMatches: options.maxMatches || 3,
-      ...options
+      ...options,
     };
 
     // Skill category keywords for classification
@@ -58,7 +58,7 @@ class AutoPattern extends BasePattern {
       security: ['security', 'vulnerability', 'authentication', 'authorization'],
       performance: ['performance', 'optimize', 'benchmark', 'profiling'],
       analysis: ['analyze', 'review', 'assess', 'evaluate', 'audit'],
-      research: ['research', 'investigate', 'explore', 'study']
+      research: ['research', 'investigate', 'explore', 'study'],
     };
   }
 
@@ -81,7 +81,7 @@ class AutoPattern extends BasePattern {
 
     return {
       valid: errors.length === 0,
-      errors
+      errors,
     };
   }
 
@@ -102,16 +102,16 @@ class AutoPattern extends BasePattern {
 
     engine.emit('autoPatternStarted', {
       context,
-      task
+      task,
     });
 
     // Find matching skills
     const matches = this._findMatchingSkills(task, engine);
-    
+
     engine.emit('autoPatternMatched', {
       context,
       task,
-      matches: matches.map(m => ({ skill: m.skill, confidence: m.confidence }))
+      matches: matches.map(m => ({ skill: m.skill, confidence: m.confidence })),
     });
 
     if (matches.length === 0) {
@@ -119,18 +119,15 @@ class AutoPattern extends BasePattern {
       if (this.options.fallbackSkill && engine.getSkill(this.options.fallbackSkill)) {
         engine.emit('autoPatternFallback', {
           context,
-          fallbackSkill: this.options.fallbackSkill
+          fallbackSkill: this.options.fallbackSkill,
+        });
+
+        return this._executeSingleSkill(this.options.fallbackSkill, input, context, engine, {
+          confidence: 0,
+          reason: 'fallback',
         });
-        
-        return this._executeSingleSkill(
-          this.options.fallbackSkill, 
-          input, 
-          context, 
-          engine,
-          { confidence: 0, reason: 'fallback' }
-        );
       }
-      
+
       throw new Error(`No matching skill found for task: ${task}`);
     }
 
@@ -141,13 +138,7 @@ class AutoPattern extends BasePattern {
 
     // Execute best match
     const bestMatch = matches[0];
-    return this._executeSingleSkill(
-      bestMatch.skill,
-      input,
-      context,
-      engine,
-      bestMatch
-    );
+    return this._executeSingleSkill(bestMatch.skill, input, context, engine, bestMatch);
   }
 
   /**
@@ -162,13 +153,13 @@ class AutoPattern extends BasePattern {
     for (const skillName of engine.listSkills()) {
       const skill = engine.getSkill(skillName);
       const score = this._calculateMatchScore(taskLower, words, skillName, skill);
-      
+
       if (score >= this.options.minConfidence) {
         matches.push({
           skill: skillName,
           confidence: score,
           confidenceLevel: this._getConfidenceLevel(score),
-          matchedKeywords: this._getMatchedKeywords(taskLower, skill)
+          matchedKeywords: this._getMatchedKeywords(taskLower, skill),
         });
       }
     }
@@ -204,9 +195,7 @@ class AutoPattern extends BasePattern {
     const keywords = skill?.keywords || [];
     if (keywords.length > 0) {
       maxScore += 0.4;
-      const matchedKeywords = keywords.filter(k => 
-        taskLower.includes(k.toLowerCase())
-      );
+      const matchedKeywords = keywords.filter(k => taskLower.includes(k.toLowerCase()));
       score += 0.4 * (matchedKeywords.length / keywords.length);
     }
 
@@ -279,8 +268,8 @@ class AutoPattern extends BasePattern {
         pattern: PatternType.AUTO,
         confidence: matchInfo.confidence,
         confidenceLevel: matchInfo.confidenceLevel,
-        matchedKeywords: matchInfo.matchedKeywords
-      }
+        matchedKeywords: matchInfo.matchedKeywords,
+      },
     });
 
     context.children.push(stepContext);
@@ -294,7 +283,7 @@ class AutoPattern extends BasePattern {
         context,
         selectedSkill: skillName,
         confidence: matchInfo.confidence,
-        output
+        output,
       });
 
       return {
@@ -302,9 +291,8 @@ class AutoPattern extends BasePattern {
         confidence: matchInfo.confidence,
         confidenceLevel: matchInfo.confidenceLevel,
         output,
-        multiMatch: false
+        multiMatch: false,
       };
-
     } catch (error) {
       stepContext.fail(error);
       throw error;
@@ -327,8 +315,8 @@ class AutoPattern extends BasePattern {
         metadata: {
           pattern: PatternType.AUTO,
           confidence: match.confidence,
-          multiMatch: true
-        }
+          multiMatch: true,
+        },
       });
 
       context.children.push(stepContext);
@@ -342,16 +330,15 @@ class AutoPattern extends BasePattern {
           skill: match.skill,
           confidence: match.confidence,
           status: ExecutionStatus.COMPLETED,
-          output
+          output,
         });
-
       } catch (error) {
         stepContext.fail(error);
         results.push({
           skill: match.skill,
           confidence: match.confidence,
           status: ExecutionStatus.FAILED,
-          error: error.message
+          error: error.message,
         });
       }
     }
@@ -359,13 +346,13 @@ class AutoPattern extends BasePattern {
     engine.emit('autoPatternCompleted', {
       context,
       results,
-      multiMatch: true
+      multiMatch: true,
     });
 
     return {
       selectedSkills: matches.map(m => m.skill),
       results,
-      multiMatch: true
+      multiMatch: true,
     };
   }
 }
@@ -382,5 +369,5 @@ function createAutoPattern(options = {}) {
 module.exports = {
   AutoPattern,
   ConfidenceLevel,
-  createAutoPattern
+  createAutoPattern,
 };