@onlineapps/mq-client-core 1.0.31 → 1.0.32

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@onlineapps/mq-client-core",
-  "version": "1.0.31",
+  "version": "1.0.32",
   "description": "Core MQ client library for RabbitMQ - shared by infrastructure services and connectors",
   "main": "src/index.js",
   "scripts": {

package/src/transports/rabbitmqClient.js

@@ -32,8 +32,9 @@ class RabbitMQClient extends EventEmitter {
     );
 
     this._connection = null;
-    this._channel = null;
-    this._queueChannel = null;
+    this._channel = null; // ConfirmChannel for publish operations
+    this._queueChannel = null; // Regular channel for queue operations (assertQueue, checkQueue)
+    this._consumerChannel = null; // Dedicated channel for consume operations
   }
 
   /**
@@ -109,6 +110,18 @@ class RabbitMQClient extends EventEmitter {
       this._queueChannel.on('close', () => {
         console.warn('[RabbitMQClient] Queue channel closed');
       });
+      
+      // Create a dedicated channel for consume operations
+      // This prevents channel conflicts between publish (ConfirmChannel) and consume operations
+      this._consumerChannel = await this._connection.createChannel();
+      this._consumerChannel.on('error', (err) => {
+        console.warn('[RabbitMQClient] Consumer channel error:', err.message);
+        this.emit('error', err);
+      });
+      this._consumerChannel.on('close', () => {
+        console.warn('[RabbitMQClient] Consumer channel closed');
+        this.emit('error', new Error('RabbitMQ consumer channel closed unexpectedly'));
+      });
     } catch (err) {
       // Cleanup partially created resources
       if (this._connection) {
@@ -128,6 +141,14 @@ class RabbitMQClient extends EventEmitter {
    * @returns {Promise<void>}
    */
   async disconnect() {
+    try {
+      if (this._consumerChannel) {
+        await this._consumerChannel.close();
+        this._consumerChannel = null;
+      }
+    } catch (err) {
+      console.warn('[RabbitMQClient] Error closing consumer channel:', err.message);
+    }
     try {
       if (this._queueChannel) {
         await this._queueChannel.close();
@@ -270,8 +291,40 @@ class RabbitMQClient extends EventEmitter {
    * @throws {Error} If consume setup fails or channel is not available.
    */
   async consume(queue, onMessage, options = {}) {
-    if (!this._channel) {
-      throw new Error('Cannot consume: channel is not initialized');
+    // Use dedicated consumer channel instead of ConfirmChannel
+    // ConfirmChannel is optimized for publish operations, not consume
+    if (!this._consumerChannel) {
+      // Recreate consumer channel if closed
+      if (this._connection && !this._connection.closed) {
+        this._consumerChannel = await this._connection.createChannel();
+        this._consumerChannel.on('error', (err) => {
+          console.warn('[RabbitMQClient] Consumer channel error:', err.message);
+          this.emit('error', err);
+        });
+        this._consumerChannel.on('close', () => {
+          console.warn('[RabbitMQClient] Consumer channel closed');
+          this.emit('error', new Error('RabbitMQ consumer channel closed unexpectedly'));
+        });
+      } else {
+        throw new Error('Cannot consume: consumer channel is not initialized and connection is closed');
+      }
+    }
+    
+    if (this._consumerChannel.closed) {
+      // Recreate consumer channel if closed
+      if (this._connection && !this._connection.closed) {
+        this._consumerChannel = await this._connection.createChannel();
+        this._consumerChannel.on('error', (err) => {
+          console.warn('[RabbitMQClient] Consumer channel error:', err.message);
+          this.emit('error', err);
+        });
+        this._consumerChannel.on('close', () => {
+          console.warn('[RabbitMQClient] Consumer channel closed');
+          this.emit('error', new Error('RabbitMQ consumer channel closed unexpectedly'));
+        });
+      } else {
+        throw new Error('Cannot consume: consumer channel is closed and connection is closed');
+      }
     }
 
     const durable = options.durable !== undefined ? options.durable : this._config.durable;
@@ -368,32 +421,21 @@ class RabbitMQClient extends EventEmitter {
         }
       }
       }
-      // Set prefetch if provided
+      // Set prefetch if provided (on consumer channel)
       if (typeof prefetch === 'number') {
-        this._channel.prefetch(prefetch);
+        this._consumerChannel.prefetch(prefetch);
       }
 
       // CRITICAL: Log before calling amqplib's consume()
       // amqplib's consume() may internally call assertQueue() without parameters if queue doesn't exist
       // This would create queue with default arguments (no TTL), causing 406 errors later
-      console.log(`[RabbitMQClient] [mq-client-core] [CONSUMER] About to call amqplib's channel.consume(${queue})`);
+      console.log(`[RabbitMQClient] [mq-client-core] [CONSUMER] About to call amqplib's consumerChannel.consume(${queue})`);
       console.log(`[RabbitMQClient] [mq-client-core] [CONSUMER] ⚠ WARNING: amqplib's consume() may internally call assertQueue() WITHOUT parameters if queue doesn't exist`);
       console.log(`[RabbitMQClient] [mq-client-core] [CONSUMER] ⚠ WARNING: We already asserted queue with correct parameters above - this should prevent auto-creation`);
       
-      // CRITICAL: Wrap amqplib's channel.consume() to intercept any internal assertQueue() calls
-      // This will help us identify if amqplib is creating the queue without TTL
-      const originalConsume = this._channel.consume.bind(this._channel);
-      const originalAssertQueue = this._channel.assertQueue.bind(this._channel);
-      
-      // Intercept assertQueue() calls to log them
-      this._channel.assertQueue = async function(queueName, options) {
-        console.log(`[RabbitMQClient] [mq-client-core] [INTERCEPT] assertQueue() called for: ${queueName}`);
-        console.log(`[RabbitMQClient] [mq-client-core] [INTERCEPT] Options:`, JSON.stringify(options || {}, null, 2));
-        console.log(`[RabbitMQClient] [mq-client-core] [INTERCEPT] Stack trace:`, new Error().stack.split('\n').slice(1, 10).join('\n'));
-        return originalAssertQueue.call(this, queueName, options);
-      };
-      
-      await this._channel.consume(
+      // Use dedicated consumer channel for consume operations
+      // This prevents conflicts with ConfirmChannel used for publish operations
+      const consumeResult = await this._consumerChannel.consume(
         queue,
         async (msg) => {
           if (msg === null) {
@@ -402,15 +444,18 @@ class RabbitMQClient extends EventEmitter {
           try {
             await onMessage(msg);
             if (!noAck) {
-              this._channel.ack(msg);
+              this._consumerChannel.ack(msg);
             }
           } catch (handlerErr) {
             // Negative acknowledge and requeue by default
-            this._channel.nack(msg, false, true);
+            this._consumerChannel.nack(msg, false, true);
           }
         },
         { noAck }
       );
+      
+      // Return consumer tag for cancellation
+      return consumeResult.consumerTag;
     } catch (err) {
       this.emit('error', err);
       throw err;