@onlineapps/service-wrapper 2.0.50 → 2.0.52

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@onlineapps/service-wrapper",
-  "version": "2.0.50",
+  "version": "2.0.52",
   "description": "Thin orchestration layer for microservices - delegates all infrastructure concerns to specialized connectors",
   "main": "src/index.js",
   "scripts": {
@@ -11,7 +11,9 @@
     "test:coverage": "jest --coverage",
     "test:mocked": "node test/run-tests.js",
     "docs": "jsdoc2md --files src/**/*.js > API.md",
-    "docs:html": "jsdoc -c jsdoc.json -d docs/html"
+    "docs:html": "jsdoc -c jsdoc.json -d docs/html",
+    "prepublishOnly": "cd $(git rev-parse --show-toplevel) && bash scripts/pre-publish-compatibility-check.sh shared/connector/service-wrapper",
+    "postpublish": "cd $(git rev-parse --show-toplevel) && bash scripts/update-manifest-from-npm.sh && bash scripts/update-all-services.sh"
   },
   "keywords": [
     "microservices",
@@ -33,7 +35,7 @@
     "@onlineapps/conn-orch-orchestrator": "^1.0.1",
     "@onlineapps/conn-orch-registry": "^1.1.25",
     "@onlineapps/conn-orch-validator": "^2.0.0",
-    "@onlineapps/service-common": "^1.0.0",
+    "@onlineapps/service-common": "^1.0.4",
     "@onlineapps/monitoring-core": "^1.0.0"
   },
   "devDependencies": {

package/src/ServiceWrapper.js

@@ -153,7 +153,8 @@ class ServiceWrapper {
    */
   async _waitForInfrastructureGate(context) {
     const { waitForInfrastructureReady } = require('@onlineapps/service-common');
-    const redisUrl = this.config.wrapper?.cache?.url || process.env.REDIS_URL || 'redis://api_node_cache:6379';
+    const { requireEnv } = require('@onlineapps/service-common');
+    const redisUrl = this.config.wrapper?.cache?.url || requireEnv('REDIS_URL', 'Redis connection URL');
     const maxWait = parseInt(process.env.INFRASTRUCTURE_HEALTH_WAIT_MAX_TIME) || 300000;
     const checkInterval = parseInt(process.env.INFRASTRUCTURE_HEALTH_WAIT_CHECK_INTERVAL) || 5000;
 
@@ -562,6 +563,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 +689,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 +1250,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,