@onlineapps/conn-orch-orchestrator 2.0.1 → 2.0.2

package/package.json

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

package/src/WorkflowOrchestrator.js

@@ -173,23 +173,29 @@ class WorkflowOrchestrator {
   async processWorkflowMessage(message, serviceName) {
     const { workflow_id, cookbook: cookbookDef, current_step } = message;
     const startTime = Date.now();
-
-    // Fail-fast: correlation_id MUST be present on every workflow message.
-    // A workflow without a correlation_id is malformed per RFC §5.8
-    // (ContextBuilder._validateMqMessage rejects empty strings too).
-    // The gateway must publish it; if it does not, that is a gateway bug
-    // we want to surface loudly rather than silently produce 500s inside
-    // invokeOperation's ContextBuilder.
+    // correlation_id is required by RFC §5.8 — validated below inside the
+    // try block so malformed messages flow through the retry/DLQ pipeline
+    // and a `failed` monitoring event is emitted (no silent redelivery).
     const correlationId = this._extractCorrelationId(message);
-    if (typeof correlationId !== 'string' || correlationId.length === 0) {
-      throw new Error(
-        '[WorkflowOrchestrator] message.correlation_id is required - ' +
-        `Expected non-empty string on inbound workflow message (workflow_id='${workflow_id}', step='${current_step}'). ` +
-        'Fix: ensure publisher (gateway / router) sets correlation_id on message body per RFC §5.8.'
-      );
-    }
 
     try {
+      // Fail-fast: correlation_id MUST be present on every workflow message.
+      // (ContextBuilder._validateMqMessage rejects empty strings too.)
+      // Gateway is expected to publish it; if it does not, that is a gateway
+      // bug we want to surface loudly rather than silently produce 500s
+      // inside invokeOperation's ContextBuilder.
+      if (typeof correlationId !== 'string' || correlationId.length === 0) {
+        const err = new Error(
+          '[WorkflowOrchestrator] message.correlation_id is required - ' +
+          `Expected non-empty string on inbound workflow message (workflow_id='${workflow_id}', step='${current_step}'). ` +
+          'Fix: ensure publisher (gateway / router) sets correlation_id on message body per RFC §5.8.'
+        );
+        err.errorCode = 'MISSING_CORRELATION_ID';
+        err.statusCode = 400;
+        err.type = 'VALIDATION';
+        throw err;
+      }
+
       // Validate cookbook structure
       this.cookbook.validateCookbook(cookbookDef);