musubi-sdd 5.1.0 → 5.6.1

package/src/orchestration/workflow-orchestrator.js

@@ -1,6 +1,6 @@
 /**
  * WorkflowOrchestrator - Complex multi-pattern workflow execution
- * 
+ *
  * Enables end-to-end workflows combining multiple orchestration patterns
  * for complete SDD lifecycle (Research → Monitoring)
  */
@@ -11,12 +11,12 @@ const { PatternType, ExecutionContext, ExecutionStatus } = require('./orchestrat
  * Workflow step types
  */
 const StepType = {
-  SKILL: 'skill',           // Single skill execution
-  PATTERN: 'pattern',       // Execute an orchestration pattern
+  SKILL: 'skill', // Single skill execution
+  PATTERN: 'pattern', // Execute an orchestration pattern
   CONDITIONAL: 'conditional', // Branch based on condition
-  PARALLEL: 'parallel',     // Parallel steps
+  PARALLEL: 'parallel', // Parallel steps
   CHECKPOINT: 'checkpoint', // Save state checkpoint
-  HUMAN_GATE: 'human-gate'  // Require human approval
+  HUMAN_GATE: 'human-gate', // Require human approval
 };
 
 /**
@@ -28,7 +28,7 @@ const WorkflowState = {
   PAUSED: 'paused',
   COMPLETED: 'completed',
   FAILED: 'failed',
-  CANCELLED: 'cancelled'
+  CANCELLED: 'cancelled',
 };
 
 /**
@@ -43,7 +43,7 @@ class WorkflowOrchestrator {
       maxRetries: options.maxRetries || 3,
       retryDelay: options.retryDelay || 1000,
       timeout: options.timeout || 300000, // 5 minutes default
-      ...options
+      ...options,
     };
 
     this.workflows = new Map();
@@ -68,7 +68,7 @@ class WorkflowOrchestrator {
       inputs: definition.inputs || [],
       outputs: definition.outputs || [],
       onError: definition.onError || 'stop',
-      metadata: definition.metadata || {}
+      metadata: definition.metadata || {},
     });
 
     return this;
@@ -106,7 +106,7 @@ class WorkflowOrchestrator {
 
     const executionId = this._generateId();
     const startTime = Date.now();
-    
+
     const execution = {
       id: executionId,
       workflow: workflowName,
@@ -119,7 +119,7 @@ class WorkflowOrchestrator {
       checkpoints: [],
       startTime,
       endTime: null,
-      error: null
+      error: null,
     };
 
     this.activeExecutions.set(executionId, execution);
@@ -127,7 +127,7 @@ class WorkflowOrchestrator {
     this.engine.emit('workflowStarted', {
       executionId,
       workflow: workflowName,
-      input
+      input,
     });
 
     try {
@@ -140,17 +140,17 @@ class WorkflowOrchestrator {
           executionId,
           stepIndex: i,
           step: step.name || step.type,
-          totalSteps: workflow.steps.length
+          totalSteps: workflow.steps.length,
         });
 
         const stepResult = await this._executeStep(step, execution, options);
-        
+
         execution.stepResults.push({
           step: step.name || `step-${i}`,
           type: step.type,
           status: stepResult.status,
           output: stepResult.output,
-          duration: stepResult.duration
+          duration: stepResult.duration,
         });
 
         // Merge step output into context
@@ -158,7 +158,7 @@ class WorkflowOrchestrator {
           execution.context = {
             ...execution.context,
             ...stepResult.output,
-            [`step_${i}_result`]: stepResult.output
+            [`step_${i}_result`]: stepResult.output,
           };
         }
 
@@ -166,7 +166,7 @@ class WorkflowOrchestrator {
           executionId,
           stepIndex: i,
           step: step.name || step.type,
-          result: stepResult
+          result: stepResult,
         });
 
         // Handle step failure
@@ -188,7 +188,7 @@ class WorkflowOrchestrator {
             executionId,
             state: WorkflowState.PAUSED,
             resumeFrom: i + 1,
-            context: execution.context
+            context: execution.context,
           };
         }
       }
@@ -205,16 +205,15 @@ class WorkflowOrchestrator {
         output: execution.output,
         stepResults: execution.stepResults,
         duration: execution.endTime - startTime,
-        summary: this._createSummary(execution)
+        summary: this._createSummary(execution),
       };
 
       this.engine.emit('workflowCompleted', {
         executionId,
-        result
+        result,
       });
 
       return result;
-
     } catch (error) {
       execution.state = WorkflowState.FAILED;
       execution.endTime = Date.now();
@@ -223,7 +222,7 @@ class WorkflowOrchestrator {
       this.engine.emit('workflowFailed', {
         executionId,
         error,
-        stepResults: execution.stepResults
+        stepResults: execution.stepResults,
       });
 
       throw error;
@@ -274,14 +273,13 @@ class WorkflowOrchestrator {
       return {
         status: ExecutionStatus.COMPLETED,
         output,
-        duration: Date.now() - startTime
+        duration: Date.now() - startTime,
       };
-
     } catch (error) {
       return {
         status: ExecutionStatus.FAILED,
         error: error.message,
-        duration: Date.now() - startTime
+        duration: Date.now() - startTime,
       };
     }
   }
@@ -293,7 +291,7 @@ class WorkflowOrchestrator {
   async _executeSkillStep(step, execution) {
     const input = this._resolveInput(step.input, execution.context);
     const skill = this.engine.getSkill(step.skill);
-    
+
     if (!skill) {
       throw new Error(`Unknown skill: ${step.skill}`);
     }
@@ -301,7 +299,7 @@ class WorkflowOrchestrator {
     const parentContext = new ExecutionContext({
       task: `Workflow skill: ${step.skill}`,
       skill: step.skill,
-      input
+      input,
     });
 
     return await this.engine.executeSkill(step.skill, input, parentContext);
@@ -313,9 +311,9 @@ class WorkflowOrchestrator {
    */
   async _executePatternStep(step, execution) {
     const input = this._resolveInput(step.input, execution.context);
-    
+
     const context = await this.engine.execute(step.pattern, {
-      input: { ...input, ...step.config }
+      input: { ...input, ...step.config },
     });
 
     return context.output;
@@ -327,9 +325,9 @@ class WorkflowOrchestrator {
    */
   async _executeConditionalStep(step, execution) {
     const condition = this._evaluateCondition(step.condition, execution.context);
-    
+
     const branchSteps = condition ? step.then : step.else;
-    
+
     if (branchSteps && branchSteps.length > 0) {
       for (const branchStep of branchSteps) {
         const result = await this._executeStep(branchStep, execution, {});
@@ -348,16 +346,14 @@ class WorkflowOrchestrator {
    */
   async _executeParallelStep(step, execution) {
     const results = await Promise.allSettled(
-      step.steps.map(subStep => 
-        this._executeStep(subStep, execution, {})
-      )
+      step.steps.map(subStep => this._executeStep(subStep, execution, {}))
     );
 
     const outputs = results.map((r, i) => ({
       step: step.steps[i].name || `parallel-${i}`,
       status: r.status === 'fulfilled' ? r.value.status : ExecutionStatus.FAILED,
       output: r.status === 'fulfilled' ? r.value.output : null,
-      error: r.status === 'rejected' ? r.reason.message : null
+      error: r.status === 'rejected' ? r.reason.message : null,
     }));
 
     return { parallelResults: outputs };
@@ -372,7 +368,7 @@ class WorkflowOrchestrator {
       id: this._generateId(),
       name: step.name || 'checkpoint',
       timestamp: Date.now(),
-      context: { ...execution.context }
+      context: { ...execution.context },
     };
 
     execution.checkpoints.push(checkpoint);
@@ -411,7 +407,7 @@ class WorkflowOrchestrator {
   _resolveInput(input, context) {
     if (!input) return context;
     if (typeof input === 'function') return input(context);
-    
+
     const resolved = {};
     for (const [key, value] of Object.entries(input)) {
       if (typeof value === 'string' && value.startsWith('$')) {
@@ -454,8 +450,9 @@ class WorkflowOrchestrator {
    */
   _resolveTemplate(template, context) {
     if (typeof template !== 'string') return template;
-    return template.replace(/\$\{([^}]+)\}/g, (_, path) => 
-      this._getValueByPath(context, path) || ''
+    return template.replace(
+      /\$\{([^}]+)\}/g,
+      (_, path) => this._getValueByPath(context, path) || ''
     );
   }
 
@@ -465,7 +462,7 @@ class WorkflowOrchestrator {
    */
   _extractOutputs(context, outputs) {
     if (!outputs || outputs.length === 0) return context;
-    
+
     const result = {};
     for (const output of outputs) {
       result[output] = context[output];
@@ -478,7 +475,9 @@ class WorkflowOrchestrator {
    * @private
    */
   _createSummary(execution) {
-    const completed = execution.stepResults.filter(s => s.status === ExecutionStatus.COMPLETED).length;
+    const completed = execution.stepResults.filter(
+      s => s.status === ExecutionStatus.COMPLETED
+    ).length;
     const failed = execution.stepResults.filter(s => s.status === ExecutionStatus.FAILED).length;
     const total = execution.stepResults.length;
 
@@ -486,9 +485,9 @@ class WorkflowOrchestrator {
       totalSteps: total,
       completedSteps: completed,
       failedSteps: failed,
-      successRate: total > 0 ? (completed / total * 100).toFixed(1) + '%' : '0%',
+      successRate: total > 0 ? ((completed / total) * 100).toFixed(1) + '%' : '0%',
       duration: execution.endTime - execution.startTime,
-      checkpoints: execution.checkpoints.length
+      checkpoints: execution.checkpoints.length,
     };
   }
 
@@ -504,7 +503,7 @@ class WorkflowOrchestrator {
         workflow: execution.workflow,
         currentStep: execution.currentStep,
         context: execution.context,
-        timestamp: Date.now()
+        timestamp: Date.now(),
       };
       this.options.checkpointStorage.set(checkpoint.id, checkpoint);
       return checkpoint;
@@ -532,7 +531,7 @@ class WorkflowOrchestrator {
     const remainingSteps = workflow.steps.slice(checkpoint.currentStep);
     const resumedWorkflow = {
       ...workflow,
-      steps: remainingSteps
+      steps: remainingSteps,
     };
 
     // Temporarily register resumed workflow
@@ -596,22 +595,22 @@ const SDDWorkflowTemplates = {
         name: 'requirements',
         type: StepType.SKILL,
         skill: 'requirements-analyst',
-        input: { feature: '$featureName' }
+        input: { feature: '$featureName' },
       },
       {
         name: 'design',
         type: StepType.SKILL,
         skill: 'software-architect',
-        input: { requirements: '$step_0_result' }
+        input: { requirements: '$step_0_result' },
       },
       {
         name: 'tasks',
         type: StepType.SKILL,
         skill: 'task-planner',
-        input: { design: '$step_1_result' }
-      }
+        input: { design: '$step_1_result' },
+      },
     ],
-    outputs: ['requirements', 'design', 'tasks']
+    outputs: ['requirements', 'design', 'tasks'],
   },
 
   /**
@@ -625,17 +624,17 @@ const SDDWorkflowTemplates = {
         name: 'research',
         type: StepType.PATTERN,
         pattern: PatternType.AUTO,
-        input: { task: 'Research: $featureName' }
+        input: { task: 'Research: $featureName' },
       },
       {
         name: 'requirements',
         type: StepType.SKILL,
-        skill: 'requirements-analyst'
+        skill: 'requirements-analyst',
       },
       {
         name: 'review-requirements',
         type: StepType.HUMAN_GATE,
-        question: 'Please review the requirements for ${featureName}'
+        question: 'Please review the requirements for ${featureName}',
       },
       {
         name: 'design',
@@ -643,16 +642,16 @@ const SDDWorkflowTemplates = {
         pattern: PatternType.GROUP_CHAT,
         config: {
           participants: ['software-architect', 'security-reviewer', 'ux-designer'],
-          topic: 'Design review'
-        }
+          topic: 'Design review',
+        },
       },
       {
         name: 'implementation',
         type: StepType.PATTERN,
         pattern: PatternType.SEQUENTIAL,
         config: {
-          skills: ['code-generator', 'test-engineer']
-        }
+          skills: ['code-generator', 'test-engineer'],
+        },
       },
       {
         name: 'validation',
@@ -662,22 +661,22 @@ const SDDWorkflowTemplates = {
           tasks: [
             { skill: 'code-reviewer' },
             { skill: 'security-reviewer' },
-            { skill: 'accessibility-specialist' }
-          ]
-        }
+            { skill: 'accessibility-specialist' },
+          ],
+        },
       },
       {
         name: 'checkpoint',
         type: StepType.CHECKPOINT,
-        name: 'pre-deployment'
+        checkpoint: 'pre-deployment',
       },
       {
         name: 'deploy-approval',
         type: StepType.HUMAN_GATE,
-        question: 'Approve deployment for ${featureName}?'
-      }
+        question: 'Approve deployment for ${featureName}?',
+      },
     ],
-    outputs: ['requirements', 'design', 'code', 'tests', 'reviews']
+    outputs: ['requirements', 'design', 'code', 'tests', 'reviews'],
   },
 
   /**
@@ -694,29 +693,29 @@ const SDDWorkflowTemplates = {
           {
             type: StepType.SKILL,
             skill: 'code-reviewer',
-            input: { focus: 'quality' }
+            input: { focus: 'quality' },
           },
           {
             type: StepType.SKILL,
             skill: 'security-reviewer',
-            input: { focus: 'security' }
+            input: { focus: 'security' },
           },
           {
             type: StepType.SKILL,
             skill: 'performance-engineer',
-            input: { focus: 'performance' }
-          }
-        ]
+            input: { focus: 'performance' },
+          },
+        ],
       },
       {
         name: 'consolidate',
         type: StepType.SKILL,
         skill: 'documentation-writer',
-        input: { reviews: '$parallelResults' }
-      }
+        input: { reviews: '$parallelResults' },
+      },
     ],
-    outputs: ['reviews', 'summary']
-  }
+    outputs: ['reviews', 'summary'],
+  },
 };
 
 /**
@@ -734,5 +733,5 @@ module.exports = {
   StepType,
   WorkflowState,
   SDDWorkflowTemplates,
-  createWorkflowOrchestrator
+  createWorkflowOrchestrator,
 };

package/src/phase4-integration.js

@@ -10,9 +10,17 @@
 const mcpIntegration = require('./integrations/mcp');
 
 // Sprint 4.2: Codebase Intelligence
-const { RepositoryMap, createRepositoryMap, generateRepositoryMap } = require('./analyzers/repository-map');
+const {
+  RepositoryMap,
+  createRepositoryMap,
+  generateRepositoryMap,
+} = require('./analyzers/repository-map');
 const { ASTExtractor, createASTExtractor } = require('./analyzers/ast-extractor');
-const { ContextOptimizer, createContextOptimizer, optimizeContext } = require('./analyzers/context-optimizer');
+const {
+  ContextOptimizer,
+  createContextOptimizer,
+  optimizeContext,
+} = require('./analyzers/context-optimizer');
 
 // Sprint 4.3: Agentic Reasoning
 const {
@@ -26,7 +34,7 @@ const {
   STEP_TYPE,
   TASK_STATUS,
   PLAN_STATUS,
-  PRIORITY
+  PRIORITY,
 } = require('./orchestration/reasoning');
 
 // Sprint 4.4: Agentic Features
@@ -42,7 +50,7 @@ const {
   TEMPLATES,
   SEVERITY,
   CATEGORY,
-  DEFAULT_RULES
+  DEFAULT_RULES,
 } = require('./agents/agentic');
 
 /**
@@ -53,10 +61,10 @@ const phase4 = {
   // Version
   version: '4.0.0',
   phase: 4,
-  
+
   // Sprint 4.1: MCP Integration
   mcp: mcpIntegration,
-  
+
   // Sprint 4.2: Codebase Intelligence
   codebase: {
     RepositoryMap,
@@ -66,9 +74,9 @@ const phase4 = {
     createASTExtractor,
     ContextOptimizer,
     createContextOptimizer,
-    optimizeContext
+    optimizeContext,
   },
-  
+
   // Sprint 4.3: Agentic Reasoning
   reasoning: {
     ReasoningEngine,
@@ -81,9 +89,9 @@ const phase4 = {
     STEP_TYPE,
     TASK_STATUS,
     PLAN_STATUS,
-    PRIORITY
+    PRIORITY,
   },
-  
+
   // Sprint 4.4: Agentic Features
   agentic: {
     CodeGenerator,
@@ -97,8 +105,8 @@ const phase4 = {
     TEMPLATES,
     SEVERITY,
     CATEGORY,
-    DEFAULT_RULES
-  }
+    DEFAULT_RULES,
+  },
 };
 
 /**
@@ -112,16 +120,16 @@ function createIntegratedAgent(options = {}) {
     repositoryMap: createRepositoryMap(options.repositoryMap),
     astExtractor: createASTExtractor(options.astExtractor),
     contextOptimizer: createContextOptimizer(options.contextOptimizer),
-    
+
     // Agentic Reasoning
     reasoningEngine: createReasoningEngine(options.reasoning),
     planningEngine: createPlanningEngine(options.planning),
     selfCorrection: createSelfCorrection(options.selfCorrection),
-    
+
     // Agentic Features
     codeGenerator: createCodeGenerator(options.codeGenerator),
     codeReviewer: createCodeReviewer(options.codeReviewer),
-    
+
     /**
      * Analyze repository and generate context
      * @param {string} rootPath - Repository root path
@@ -131,10 +139,10 @@ function createIntegratedAgent(options = {}) {
       const map = await this.repositoryMap.generate(rootPath);
       return {
         map,
-        stats: this.repositoryMap.getStats()
+        stats: this.repositoryMap.getStats(),
       };
     },
-    
+
     /**
      * Generate code with reasoning
      * @param {string} description - What to generate
@@ -144,50 +152,49 @@ function createIntegratedAgent(options = {}) {
     async generateWithReasoning(description, options = {}) {
       // Plan the generation
       const plan = this.planningEngine.createPlan(`Generate: ${description}`, {
-        strategy: 'incremental'
+        strategy: 'incremental',
       });
-      
+
       // Reason about the implementation
-      const reasoning = this.reasoningEngine.reason(
-        `How to implement: ${description}`,
-        { type: 'decomposition' }
-      );
-      
+      const reasoning = this.reasoningEngine.reason(`How to implement: ${description}`, {
+        type: 'decomposition',
+      });
+
       // Generate the code
       const generated = await this.codeGenerator.generate({
         description,
-        ...options
+        ...options,
       });
-      
+
       // Review the generated code
       const review = this.codeReviewer.review(generated.code, options);
-      
+
       // Self-correct if needed
       if (review.score < 70) {
         const correction = this.selfCorrection.analyzeError({
           error: new Error('Code quality below threshold'),
-          context: { review, generated }
+          context: { review, generated },
         });
-        
+
         return {
           generated,
           review,
           correction,
           reasoning: reasoning.result,
           plan: plan.id,
-          needsImprovement: true
+          needsImprovement: true,
         };
       }
-      
+
       return {
         generated,
         review,
         reasoning: reasoning.result,
         plan: plan.id,
-        needsImprovement: false
+        needsImprovement: false,
       };
     },
-    
+
     /**
      * Get agent statistics
      * @returns {Object} Agent statistics
@@ -198,9 +205,9 @@ function createIntegratedAgent(options = {}) {
         reasoning: this.reasoningEngine.getStats(),
         planning: this.planningEngine.getStats(),
         codeGenerator: this.codeGenerator.getStats(),
-        codeReviewer: this.codeReviewer.getStats()
+        codeReviewer: this.codeReviewer.getStats(),
       };
-    }
+    },
   };
 }
 
@@ -208,7 +215,7 @@ module.exports = {
   ...phase4,
   phase4,
   createIntegratedAgent,
-  
+
   // Direct exports for convenience
   RepositoryMap,
   ASTExtractor,
@@ -218,7 +225,7 @@ module.exports = {
   SelfCorrection,
   CodeGenerator,
   CodeReviewer,
-  
+
   // Factory functions
   createRepositoryMap,
   createASTExtractor,
@@ -228,13 +235,13 @@ module.exports = {
   createSelfCorrection,
   createCodeGenerator,
   createCodeReviewer,
-  
+
   // Helper functions
   generateRepositoryMap,
   optimizeContext,
   generateCode,
   reviewCode,
-  
+
   // Enums
   STRATEGY,
   STEP_TYPE,
@@ -244,5 +251,5 @@ module.exports = {
   GEN_MODE,
   LANGUAGE,
   SEVERITY,
-  CATEGORY
+  CATEGORY,
 };