@onlineapps/service-wrapper 2.0.38 → 2.0.39

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@onlineapps/service-wrapper",
-  "version": "2.0.38",
+  "version": "2.0.39",
   "description": "Thin orchestration layer for microservices - delegates all infrastructure concerns to specialized connectors",
   "main": "src/index.js",
   "scripts": {
@@ -33,7 +33,7 @@
     "@onlineapps/conn-orch-orchestrator": "^1.0.1",
     "@onlineapps/conn-orch-registry": "^1.1.16",
     "@onlineapps/conn-orch-validator": "^2.0.0",
-    "@onlineapps/infrastructure-tools": "^1.0.3",
+    "@onlineapps/service-common": "^1.0.0",
     "@onlineapps/monitoring-core": "^1.0.0"
   },
   "devDependencies": {

package/src/ServiceWrapper.js

@@ -414,7 +414,6 @@ class ServiceWrapper {
     const maxRetries = parseInt(process.env.INFRASTRUCTURE_VERIFY_MAX_RETRIES) || 12; // 12 * 5s = 60s max
     const baseDelay = parseInt(process.env.INFRASTRUCTURE_VERIFY_BASE_DELAY) || 5000; // 5 seconds
     const maxDelay = parseInt(process.env.INFRASTRUCTURE_VERIFY_MAX_DELAY) || 30000; // 30 seconds max
-    const redisUrl = this.config.wrapper?.cache?.url || process.env.REDIS_URL || 'redis://api_node_cache:6379';
     
     // Required infrastructure queues that must exist before business queues can be created
     const requiredInfrastructureQueues = [
@@ -438,20 +437,24 @@ class ServiceWrapper {
     
     for (let attempt = 1; attempt <= maxRetries; attempt++) {
       try {
-        // Step 1: Check infrastructure health via Redis
+        // Step 1: Check infrastructure health via Redis (using service-common)
+        // ServiceWrapper uses service-common (shared utility) to maintain architectural separation
         this.logger?.info(`[InfrastructureVerify] Attempt ${attempt}/${maxRetries}: Checking infrastructure health...`);
         
-        const { waitForInfrastructureReady } = require('@onlineapps/infrastructure-tools');
-        const waitMaxTime = 10000; // 10 seconds per attempt
-        const waitCheckInterval = 2000; // Check every 2 seconds
+        const redisUrl = this.config.wrapper?.cache?.url || process.env.REDIS_URL || 'redis://api_node_cache:6379';
         
         try {
+          const { waitForInfrastructureReady } = require('@onlineapps/service-common');
+          const waitMaxTime = 10000; // 10 seconds per attempt
+          const waitCheckInterval = 2000; // Check every 2 seconds
+          
           await waitForInfrastructureReady({
             redisUrl: redisUrl,
             maxWait: waitMaxTime,
             checkInterval: waitCheckInterval,
             logger: {
-              log: (msg) => this.logger?.info(`[InfrastructureVerify] ${msg}`)
+              log: (msg) => this.logger?.info(`[InfrastructureVerify] ${msg}`),
+              info: (msg) => this.logger?.info(`[InfrastructureVerify] ${msg}`)
             }
           });
           this.logger?.info('[InfrastructureVerify] ✓ All infrastructure services are UP');