@onlineapps/service-wrapper 2.1.111 → 2.1.113

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@onlineapps/service-wrapper",
-  "version": "2.1.111",
+  "version": "2.1.113",
   "description": "Thin orchestration layer for microservices - delegates all infrastructure concerns to specialized connectors",
   "main": "src/index.js",
   "scripts": {
@@ -25,16 +25,16 @@
   "license": "MIT",
   "dependencies": {
     "@onlineapps/conn-base-cache": "1.0.8",
-    "@onlineapps/conn-base-monitoring": "1.0.9",
-    "@onlineapps/conn-infra-error-handler": "1.0.8",
+    "@onlineapps/conn-base-monitoring": "1.0.10",
+    "@onlineapps/conn-infra-error-handler": "1.0.9",
     "@onlineapps/conn-infra-mq": "1.1.67",
     "@onlineapps/conn-orch-api-mapper": "1.0.30",
     "@onlineapps/conn-orch-cookbook": "2.0.35",
-    "@onlineapps/conn-orch-orchestrator": "1.0.102",
+    "@onlineapps/conn-orch-orchestrator": "1.0.103",
     "@onlineapps/conn-orch-registry": "1.1.52",
     "@onlineapps/conn-orch-validator": "2.0.29",
-    "@onlineapps/monitoring-core": "1.0.20",
-    "@onlineapps/service-common": "1.0.15",
+    "@onlineapps/monitoring-core": "1.0.21",
+    "@onlineapps/service-common": "1.0.17",
     "@onlineapps/runtime-config": "1.0.2"
   },
   "devDependencies": {

package/src/ServiceWrapper.js

@@ -973,18 +973,37 @@ class ServiceWrapper {
       throw new Error(`Failed to register service ${serviceName}: ${error.message}`);
     }
 
-    // Start heartbeat
+    // Start heartbeat with fail-fast on persistent MQ failure
     const heartbeatInterval = this.config.wrapper?.registry?.heartbeatInterval;
     if (typeof heartbeatInterval !== 'number' || Number.isNaN(heartbeatInterval) || heartbeatInterval <= 0) {
       throw new Error(`[ServiceWrapper] Invalid configuration - wrapper.registry.heartbeatInterval must be a positive number, got: ${heartbeatInterval}`);
     }
     this.logger?.info(`[ServiceWrapper] Heartbeat interval set to ${heartbeatInterval}ms`);
 
+    const MAX_CONSECUTIVE_FAILURES = this.config.wrapper?.registry?.maxHeartbeatFailures || 5;
+    let consecutiveFailures = 0;
+
     this.heartbeatTimer = setInterval(async () => {
       try {
         await this.registryClient.sendHeartbeat(serviceName);
+        consecutiveFailures = 0;
       } catch (error) {
-        this.logger?.error('Heartbeat failed', { error: error.message, stack: error.stack });
+        consecutiveFailures++;
+        this.logger?.error('Heartbeat failed', {
+          error: error.message,
+          consecutiveFailures,
+          maxBeforeExit: MAX_CONSECUTIVE_FAILURES,
+          stack: error.stack
+        });
+
+        if (consecutiveFailures >= MAX_CONSECUTIVE_FAILURES) {
+          this.logger?.error(
+            `[ServiceWrapper] FATAL: ${consecutiveFailures} consecutive heartbeat failures. ` +
+            `MQ channel is dead and not recovering. Exiting for Docker restart.`,
+            { consecutiveFailures, serviceName }
+          );
+          process.exit(1);
+        }
       }
     }, heartbeatInterval);
   }
@@ -1059,20 +1078,32 @@ class ServiceWrapper {
   _setupHealthChecks() {
     const healthEndpoint = this.config.wrapper?.health?.endpoint || '/health';
 
-    this.app.get(healthEndpoint, (req, res) => {
+    this.app.get(healthEndpoint, async (req, res) => {
       const health = {
         status: 'healthy',
         service: this.config.service?.name || 'unnamed-service',
         timestamp: new Date().toISOString(),
         components: {
           http: 'healthy',
-          mq: this.mqClient?.isConnected() ? 'healthy' : 'unhealthy',
+          mq: 'disabled',
           registry: this.registryClient ? 'healthy' : 'disabled',
           cache: this.cacheConnector ? 'healthy' : 'disabled'
         }
       };
 
-      // Determine overall status
+      if (this.mqClient) {
+        try {
+          const mqHealth = await this.mqClient.performHealthCheck();
+          health.components.mq = mqHealth.healthy ? 'healthy' : 'unhealthy';
+          if (!mqHealth.healthy) {
+            health.mqIssues = mqHealth.issues;
+          }
+        } catch (e) {
+          health.components.mq = 'unhealthy';
+          health.mqIssues = [e.message];
+        }
+      }
+
       const statuses = Object.values(health.components);
       if (statuses.includes('unhealthy')) {
         health.status = 'unhealthy';