@onlineapps/conn-orch-orchestrator 1.0.33 → 1.0.34

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@onlineapps/conn-orch-orchestrator",
-  "version": "1.0.33",
+  "version": "1.0.34",
   "description": "Workflow orchestration connector for OA Drive - handles message routing and workflow execution",
   "main": "src/index.js",
   "scripts": {

package/src/WorkflowOrchestrator.js

@@ -61,6 +61,25 @@ class WorkflowOrchestrator {
     return step?.step_id || step?.id;
   }
 
+  /**
+   * Get steps as array (supports both V1 array and V2 object formats)
+   * V2 FORMAT: steps is an object keyed by step_id
+   * V1 FORMAT (deprecated): steps is an array
+   * @private
+   * @param {Object} cookbookDef - Cookbook definition
+   * @returns {Array} Array of step objects
+   */
+  _getStepsArray(cookbookDef) {
+    if (!cookbookDef?.steps) return [];
+    if (Array.isArray(cookbookDef.steps)) {
+      // V1 format (deprecated) - steps is already an array
+      this.logger.warn('Using deprecated V1 cookbook format (steps as array). Please migrate to V2 (steps as object).');
+      return cookbookDef.steps;
+    }
+    // V2 format - steps is an object keyed by step_id
+    return Object.values(cookbookDef.steps);
+  }
+
   /**
    * Process a workflow message
    * @async
@@ -97,7 +116,9 @@ class WorkflowOrchestrator {
       };
 
       // Find current step (supports both V1 'id' and V2 'step_id')
-      const step = cookbookDef.steps.find(s => this._getStepId(s) === current_step);
+      // V2 FORMAT: steps is an object keyed by step_id
+      const stepsArray = this._getStepsArray(cookbookDef);
+      const step = stepsArray.find(s => this._getStepId(s) === current_step);
       if (!step) {
         throw new Error(`Step not found: ${current_step}`);
       }
@@ -135,13 +156,14 @@ class WorkflowOrchestrator {
 
       // Update context with result - steps as ARRAY (consistent with cookbook.steps)
       // Each step preserves its definition (id/step_id, type, service, operation, input) and adds output
-      const currentIndex = cookbookDef.steps.findIndex(s => this._getStepId(s) === current_step);
-      const stepDefinition = cookbookDef.steps[currentIndex];
+      // stepsArray is already defined above via _getStepsArray()
+      const currentIndex = stepsArray.findIndex(s => this._getStepId(s) === current_step);
+      const stepDefinition = stepsArray[currentIndex];
       
       // Initialize steps array from cookbook if not present
       const existingSteps = Array.isArray(enrichedContext.steps) 
         ? [...enrichedContext.steps] 
-        : cookbookDef.steps.map(s => ({ ...s }));  // Deep copy of step definitions
+        : stepsArray.map(s => ({ ...s }));  // Deep copy of step definitions
       
       // Update the current step with output (preserve id, type, service, operation, input)
       existingSteps[currentIndex] = {
@@ -182,8 +204,8 @@ class WorkflowOrchestrator {
         });
       }
 
-      // Find next step (currentIndex already defined above)
-      const nextStep = cookbookDef.steps[currentIndex + 1];
+      // Find next step (currentIndex and stepsArray already defined above)
+      const nextStep = stepsArray[currentIndex + 1];
 
       if (nextStep) {
         // Route to next step