@onlineapps/service-wrapper 2.0.50 → 2.0.51

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@onlineapps/service-wrapper",
-  "version": "2.0.50",
+  "version": "2.0.51",
   "description": "Thin orchestration layer for microservices - delegates all infrastructure concerns to specialized connectors",
   "main": "src/index.js",
   "scripts": {

package/src/ServiceWrapper.js

@@ -562,6 +562,57 @@ class ServiceWrapper {
     }
   }
 
+  /**
+   * Run health check before workflow step (debug mode only)
+   * @private
+   * @param {Object} message - Workflow message
+   */
+  async _runHealthCheckBeforeWorkflowStep(message) {
+    if (!this.mqClient || !this.mqClient._transport) {
+      return;
+    }
+    
+    const transport = this.mqClient._transport;
+    
+    // Check if transport has performHealthCheck method (public API)
+    if (typeof transport.performHealthCheck === 'function') {
+      try {
+        const health = await transport.performHealthCheck();
+        
+        this.logger?.debug('[ServiceWrapper] [DEBUG] Health check before workflow step', {
+          workflow_id: message.workflow_id || message.workflowId,
+          current_step: message.current_step,
+          health: {
+            healthy: health.healthy,
+            issues: health.issues,
+            consumers: {
+              active: health.consumers.active,
+              tracked: health.consumers.tracked
+            },
+            queues: {
+              checked: health.queues.checked,
+              missing: health.queues.missing,
+              noConsumer: health.queues.noConsumer
+            },
+            channels: health.channels
+          }
+        });
+        
+        if (!health.healthy) {
+          this.logger?.warn('[ServiceWrapper] [DEBUG] Health check FAILED before workflow step', {
+            workflow_id: message.workflow_id || message.workflowId,
+            issues: health.issues
+          });
+        }
+      } catch (err) {
+        this.logger?.warn('[ServiceWrapper] [DEBUG] Health check error before workflow step', {
+          error: err.message,
+          workflow_id: message.workflow_id || message.workflowId
+        });
+      }
+    }
+  }
+
   /**
    * Register MQ channel close hooks and health monitoring
    * Must be called AFTER monitoring is initialized
@@ -637,9 +688,10 @@ class ServiceWrapper {
     const serviceName = this.config.service?.name || 'unnamed-service';
 
     // Health monitoring and channel close hooks configuration
+    // NOTE: Health monitoring is NOT run periodically - it's triggered manually before workflow steps in debug mode
     const healthCheckConfig = {
-      healthCheckInterval: this.config.wrapper?.mq?.healthCheckInterval || 30000, // 30s
-      healthCheckEnabled: this.config.wrapper?.mq?.healthCheckEnabled !== false, // Default: true
+      healthCheckInterval: this.config.wrapper?.mq?.healthCheckInterval || 30000, // Not used when healthCheckEnabled=false
+      healthCheckEnabled: false, // Disable periodic checks - we'll run manually before workflow steps
       criticalHealthShutdown: this.config.wrapper?.mq?.criticalHealthShutdown !== false, // Default: true
       criticalHealthShutdownDelay: this.config.wrapper?.mq?.criticalHealthShutdownDelay || 60000, // 60s
       // Health reporting callbacks
@@ -1197,6 +1249,11 @@ class ServiceWrapper {
           throw new Error('Workflow message must have current_step field or cookbook with at least one step');
         }
         
+        // DEBUG MODE: Run health check before workflow step
+        if (process.env.DEBUG === 'true' || process.env.NODE_ENV === 'development') {
+          await this._runHealthCheckBeforeWorkflowStep(message);
+        }
+        
         // Normalize message format: Gateway sends workflowId, orchestrator expects workflow_id
         const normalizedMessage = {
           ...message,