@onlineapps/conn-orch-orchestrator 1.0.9 → 1.0.10

package/package.json

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

package/src/WorkflowOrchestrator.js

@@ -91,32 +91,6 @@ class WorkflowOrchestrator {
         return { skipped: true, reason: 'wrong_service' };
       }
 
-      // Publish workflow.progress to monitoring.workflow using unified helper
-      const { publishToMonitoringWorkflow } = require('@onlineapps/mq-client-core').monitoring;
-      try {
-        const stepIndex = cookbookDef.steps.findIndex(s => s.id === current_step);
-        console.log(`[WorkflowOrchestrator] [PROGRESS] Publishing progress for ${workflow_id}, step: ${current_step}, index: ${stepIndex}`);
-        await publishToMonitoringWorkflow(this.mqClient, {
-          event_type: 'progress',
-          workflow_id: workflow_id,
-          service_name: serviceName,
-          step_index: stepIndex >= 0 ? stepIndex : 0,
-          step_id: current_step,
-          cookbook: cookbookDef,
-          context: enrichedContext,
-          status: 'in_progress',
-          timestamp: new Date().toISOString()
-        }, this.logger, { workflow_id, step_id: current_step });
-        console.log(`[WorkflowOrchestrator] [PROGRESS] ✓ Progress published for ${workflow_id}`);
-      } catch (monitoringError) {
-        // Don't fail workflow if monitoring publish fails (helper already logs)
-        console.error(`[WorkflowOrchestrator] [PROGRESS] ✗ Failed to publish progress:`, monitoringError);
-        this.logger.warn('Failed to publish workflow.progress to monitoring', {
-          workflow_id,
-          error: monitoringError.message || String(monitoringError)
-        });
-      }
-
       // Check cache if available
       let result;
       if (this.cache && step.type === 'task') {
@@ -158,6 +132,32 @@ class WorkflowOrchestrator {
         steps: existingSteps
       };
 
+      // Publish workflow.progress AFTER execution (so we have step output)
+      const { publishToMonitoringWorkflow } = require('@onlineapps/mq-client-core').monitoring;
+      try {
+        console.log(`[WorkflowOrchestrator] [PROGRESS] Publishing progress for ${workflow_id}, step: ${current_step}, index: ${currentIndex}`);
+        await publishToMonitoringWorkflow(this.mqClient, {
+          event_type: 'progress',
+          workflow_id: workflow_id,
+          service_name: serviceName,
+          step_index: currentIndex >= 0 ? currentIndex : 0,
+          step_id: current_step,
+          cookbook: cookbookDef,
+          context: updatedContext,  // Context WITH step output
+          output: result,           // Step output directly
+          status: 'completed',      // Step completed (not in_progress)
+          timestamp: new Date().toISOString()
+        }, this.logger, { workflow_id, step_id: current_step });
+        console.log(`[WorkflowOrchestrator] [PROGRESS] ✓ Progress published for ${workflow_id}`);
+      } catch (monitoringError) {
+        // Don't fail workflow if monitoring publish fails
+        console.error(`[WorkflowOrchestrator] [PROGRESS] ✗ Failed to publish progress:`, monitoringError);
+        this.logger.warn('Failed to publish workflow.progress to monitoring', {
+          workflow_id,
+          error: monitoringError.message || String(monitoringError)
+        });
+      }
+
       // Find next step (currentIndex already defined above)
       const nextStep = cookbookDef.steps[currentIndex + 1];