@onlineapps/infrastructure-tools 1.0.2 → 1.0.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@onlineapps/infrastructure-tools",
-  "version": "1.0.2",
+  "version": "1.0.3",
   "description": "Infrastructure orchestration utilities for OA Drive infrastructure services (health tracking, queue initialization, service discovery)",
   "main": "src/index.js",
   "scripts": {

package/src/orchestration/waitForInfrastructureReady.js

@@ -43,19 +43,26 @@ async function waitForInfrastructureReady(options = {}) {
   const logger = options.logger || console;
 
   // Universal logger adapter (supports both console and winston)
-  // Winston logger expects object with message property, console.log accepts string
+  // Winston logger can accept string, but to be safe, we'll always use object format for winston
   const log = (message) => {
-    if (typeof logger.info === 'function' && typeof logger.log !== 'function') {
-      // Likely winston logger - use object format
-      logger.info({ message: String(message) });
+    const messageStr = String(message);
+    
+    // Check if logger is winston (has child method or format property)
+    const isWinston = typeof logger.child === 'function' || logger.format !== undefined;
+    
+    if (typeof logger.info === 'function') {
+      if (isWinston) {
+        // Winston logger - always use object format to avoid property conflicts
+        logger.info({ message: messageStr });
+      } else {
+        // Other logger with info method - use string
+        logger.info(messageStr);
+      }
     } else if (typeof logger.log === 'function') {
       // console.log or similar
-      logger.log(message);
-    } else if (typeof logger.info === 'function') {
-      // Fallback for other loggers
-      logger.info(message);
+      logger.log(messageStr);
     } else {
-      console.log(message);
+      console.log(messageStr);
     }
   };