@onlineapps/mq-client-core 1.0.62 → 1.0.64

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@onlineapps/mq-client-core",
-  "version": "1.0.62",
+  "version": "1.0.64",
   "description": "Core MQ client library for RabbitMQ - shared by infrastructure services and connectors",
   "main": "src/index.js",
   "scripts": {
@@ -21,8 +21,7 @@
     "amqplib": "^0.10.3",
     "ajv": "^8.12.0",
     "lodash.merge": "^4.6.2",
-    "@onlineapps/runtime-config": "1.0.1",
-    "@onlineapps/infrastructure-tools": "^1.0.23"
+    "@onlineapps/runtime-config": "1.0.2"
   },
   "devDependencies": {
     "jest": "^29.7.0"

package/src/transports/rabbitmqClient.js

@@ -19,21 +19,31 @@ const RecoveryWorker = require('../workers/RecoveryWorker');
 const PublishMonitor = require('../monitoring/PublishMonitor');
 const runtimeCfg = require('../config');
 
-// LAZY LOAD: Avoid circular dependency with infrastructure-tools
-// infrastructure-tools imports mq-client-core, so we can't import it at top-level
-let _createInfraLogger = null;
-function getCreateInfraLogger() {
-  if (!_createInfraLogger) {
+/**
+ * Local structured logger (module-owned) to avoid dependency cycles.
+ *
+ * IMPORTANT:
+ * - mq-client-core is a low-level library used by infrastructure-tools.
+ * - Therefore mq-client-core MUST NOT depend on infrastructure-tools.
+ */
+function createInfraLogger(service, layer) {
+  const prefix = (contextId, action) => `[${contextId}:${service}:${layer}:${action}]`;
+
+  const toJson = (data) => {
     try {
-      _createInfraLogger = require('@onlineapps/infrastructure-tools').createInfraLogger;
+      return JSON.stringify(data);
     } catch (e) {
-      // Fallback: simple console logger if infrastructure-tools not available
-      _createInfraLogger = (service, layer) => ({
-        log: (action, data) => console.log(`[${service}:${layer}:${action}]`, JSON.stringify(data))
-      });
+      return JSON.stringify({ error: 'failed_to_serialize', message: e.message });
     }
-  }
-  return _createInfraLogger;
+  };
+
+  return {
+    input: (contextId, action, data) => console.log(prefix(contextId, action), toJson(data)),
+    output: (contextId, action, data) => console.log(prefix(contextId, action), toJson(data)),
+    process: (contextId, action, data) => console.log(prefix(contextId, action), toJson(data)),
+    lifecycle: (contextId, action, data) => console.log(prefix(contextId, action), toJson(data)),
+    fail: (contextId, action, data) => console.error(prefix(contextId, action), toJson(data)),
+  };
 }
 
 class RabbitMQClient extends EventEmitter {
@@ -119,8 +129,8 @@ class RabbitMQClient extends EventEmitter {
     this._criticalHealthShutdownDelay = this._config.criticalHealthShutdownDelay || 60000; // Default: 60s delay before shutdown
     this._criticalHealthStartTime = null; // Track when critical health started
 
-    // Create structured logger for infrastructure logging (lazy-loaded)
-    this._log = getCreateInfraLogger()('mq-client-core', 'transport');
+    // Create structured logger for infrastructure logging (module-owned)
+    this._log = createInfraLogger('mq-client-core', 'transport');
     
     // Publish layer (retry + buffer)
     this._publishLayer = new PublishLayer({