@onlineapps/conn-orch-orchestrator 1.0.26 → 1.0.28

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@onlineapps/conn-orch-orchestrator",
-  "version": "1.0.26",
+  "version": "1.0.28",
   "description": "Workflow orchestration connector for OA Drive - handles message routing and workflow execution",
   "main": "src/index.js",
   "scripts": {
@@ -24,7 +24,7 @@
     "@onlineapps/conn-infra-mq": "^1.1.0",
     "@onlineapps/conn-orch-registry": "^1.1.4",
     "@onlineapps/conn-orch-cookbook": "^2.0.0",
-    "@onlineapps/conn-orch-api-mapper": "^1.0.6"
+    "@onlineapps/conn-orch-api-mapper": "^1.0.8"
   },
   "devDependencies": {
     "jest": "^29.5.0",

package/src/WorkflowOrchestrator.js

@@ -89,10 +89,10 @@ class WorkflowOrchestrator {
         delivery: context?.delivery || cookbookDef?.delivery || { handler: 'none' }
       };
 
-      // Find current step (V2: step_id only)
-      const step = cookbookDef.steps.find(s => s.step_id === current_step);
+      // Find current step (V2: steps is an object keyed by step_id)
+      const step = cookbookDef.steps[current_step];
       if (!step) {
-        throw new Error(`Step not found: ${current_step}. Available steps: ${cookbookDef.steps.map(s => s.step_id).join(', ')}`);
+        throw new Error(`Step not found: ${current_step}. Available steps: ${Object.keys(cookbookDef.steps).join(', ')}`);
       }
       
       // Get step_id (V2)
@@ -131,19 +131,16 @@ class WorkflowOrchestrator {
 
       // Update context with result - steps as OBJECT (V2 format, keyed by step_id)
       // Each step preserves its definition (step_id, type, service, operation, input) and adds output
-      const currentIndex = cookbookDef.steps.findIndex(s => s.step_id === current_step);
-      const stepDefinition = cookbookDef.steps[currentIndex];
+      const stepDefinition = cookbookDef.steps[current_step];
       
       // Initialize steps object from cookbook if not present
-      // V2: steps is an object keyed by step_id, not an array
+      // V2: steps is an object keyed by step_id
       let existingSteps = enrichedContext.steps || {};
       
       // Initialize from cookbook if empty
       if (Object.keys(existingSteps).length === 0) {
-        cookbookDef.steps.forEach(s => {
-          if (s.step_id) {
-            existingSteps[s.step_id] = { ...s };
-          }
+        Object.entries(cookbookDef.steps).forEach(([key, s]) => {
+          existingSteps[key] = { ...s };
         });
       }
       
@@ -155,11 +152,32 @@ class WorkflowOrchestrator {
         completed_at: new Date().toISOString()
       };
       
+      // === STRUCTURED LOG: Context update ===
+      console.log(`[WorkflowOrchestrator:CONTEXT_UPDATE] ${JSON.stringify({
+        layer: 'orchestrator',
+        workflow_id: workflow_id,
+        step_id: stepId,
+        stored_output: {
+          _descriptor: result?._descriptor,
+          type: result?.type,
+          storage_ref: result?.storage_ref,
+          outputKeys: result && typeof result === 'object' ? Object.keys(result) : null
+        },
+        timestamp: new Date().toISOString()
+      })}`);
+
+      
       const updatedContext = {
         ...enrichedContext,
         steps: existingSteps  // Object keyed by step_id
       };
 
+      // V2: Get step order from object keys (JavaScript preserves insertion order for string keys)
+      const stepKeys = Object.keys(cookbookDef.steps);
+      const currentIndex = stepKeys.indexOf(current_step);
+      const nextStepKey = stepKeys[currentIndex + 1];
+      const nextStep = nextStepKey ? cookbookDef.steps[nextStepKey] : null;
+
       // Publish workflow.progress AFTER execution (so we have step output)
       const { publishToMonitoringWorkflow } = require('@onlineapps/mq-client-core').monitoring;
       try {
@@ -186,9 +204,6 @@ class WorkflowOrchestrator {
         });
       }
 
-      // Find next step (currentIndex already defined above)
-      const nextStep = cookbookDef.steps[currentIndex + 1];
-
       if (nextStep) {
         // Route to next step
         const nextStepId = nextStep.step_id;
@@ -335,6 +350,21 @@ class WorkflowOrchestrator {
       context
     );
 
+    // === STRUCTURED LOG: Step result ===
+    console.log(`[WorkflowOrchestrator:STEP_RESULT] ${JSON.stringify({
+      layer: 'orchestrator',
+      step_id: step.step_id,
+      service: step.service,
+      operation: step.operation,
+      result: {
+        _descriptor: result?._descriptor,
+        type: result?.type,
+        storage_ref: result?.storage_ref,
+        resultKeys: result && typeof result === 'object' ? Object.keys(result) : null
+      },
+      timestamp: new Date().toISOString()
+    })}`);
+
     this.logger.info(`Task step executed`, {
       step: step.step_id,
       service: step.service,
@@ -722,22 +752,36 @@ class WorkflowOrchestrator {
       throw new Error(`FAIL-FAST: Invalid version format '${version}'. Must be 2.x.x (e.g., 2.0.0)`);
     }
 
-    // Check steps
-    if (!cookbook.steps || !Array.isArray(cookbook.steps) || cookbook.steps.length === 0) {
+    // Check steps - V2 FORMAT: steps MUST be an OBJECT (keyed by step_id), NOT an array!
+    if (!cookbook.steps || typeof cookbook.steps !== 'object') {
+      throw new Error('FAIL-FAST: Cookbook must have steps object');
+    }
+    
+    // V1 format (array) is BANNED!
+    if (Array.isArray(cookbook.steps)) {
+      throw new Error('FAIL-FAST: V1 FORMAT BANNED! Cookbook steps must be an OBJECT (keyed by step_id), NOT an array!');
+    }
+    
+    const stepKeys = Object.keys(cookbook.steps);
+    if (stepKeys.length === 0) {
       throw new Error('FAIL-FAST: Cookbook must have at least one step');
     }
 
     // Validate each step has step_id (V2) not id (V1)
     const invalidSteps = [];
-    cookbook.steps.forEach((step, index) => {
+    stepKeys.forEach((stepKey) => {
+      const step = cookbook.steps[stepKey];
       if (step.id && !step.step_id) {
-        invalidSteps.push(`Step ${index}: has 'id' (V1) but missing 'step_id' (V2)`);
+        invalidSteps.push(`Step '${stepKey}': has 'id' (V1) but missing 'step_id' (V2)`);
       }
       if (!step.step_id) {
-        invalidSteps.push(`Step ${index}: missing required 'step_id'`);
+        invalidSteps.push(`Step '${stepKey}': missing required 'step_id'`);
+      }
+      if (step.step_id !== stepKey) {
+        invalidSteps.push(`Step '${stepKey}': step_id '${step.step_id}' does not match key '${stepKey}'`);
       }
       if (!step.type) {
-        invalidSteps.push(`Step ${index} (${step.step_id || 'unknown'}): missing required 'type'`);
+        invalidSteps.push(`Step '${stepKey}': missing required 'type'`);
       }
     });
 
@@ -747,7 +791,7 @@ class WorkflowOrchestrator {
 
     this.logger.info('[WorkflowOrchestrator] V2 format validation passed', {
       version: cookbook.version,
-      steps: cookbook.steps.map(s => s.step_id)
+      steps: Object.keys(cookbook.steps)
     });
   }
 }