@onlineapps/conn-infra-mq 1.1.14 → 1.1.16

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@onlineapps/conn-infra-mq",
-    "version": "1.1.14",
+    "version": "1.1.16",
     "description": "A promise-based, broker-agnostic client for sending and receiving messages via RabbitMQ",
     "main": "src/index.js",
     "repository": {

package/src/layers/QueueManager.js

@@ -36,7 +36,9 @@ class QueueManager {
     if (!transport || !transport.channel) {
       throw new Error('MQ client not connected');
     }
-    const channel = transport.channel;
+    // Use queueChannel for queue operations to avoid RPC reply queue issues
+    // queueChannel is a regular channel, not ConfirmChannel
+    const channel = transport.queueChannel || transport.channel;
 
     let queueOptions;
 
@@ -186,7 +188,8 @@ class QueueManager {
     if (!transport || !transport.channel) {
       throw new Error('MQ client not connected');
     }
-    const channel = transport.channel;
+    // Use queueChannel for queue operations to avoid RPC reply queue issues
+    const channel = transport.queueChannel || transport.channel;
 
     const queues = {};
 
@@ -300,7 +303,8 @@ class QueueManager {
     if (!transport || !transport.channel) {
       throw new Error('MQ client not connected');
     }
-    const channel = transport.channel;
+    // Use queueChannel for queue operations
+    const channel = transport.queueChannel || transport.channel;
 
     this.managedQueues.delete(queueName);
     return channel.deleteQueue(queueName, options);
@@ -315,7 +319,8 @@ class QueueManager {
     if (!transport || !transport.channel) {
       throw new Error('MQ client not connected');
     }
-    const channel = transport.channel;
+    // Use queueChannel for queue operations
+    const channel = transport.queueChannel || transport.channel;
 
     return channel.purgeQueue(queueName);
   }
@@ -329,7 +334,8 @@ class QueueManager {
     if (!transport || !transport.channel) {
       throw new Error('MQ client not connected');
     }
-    const channel = transport.channel;
+    // Use queueChannel for queue operations
+    const channel = transport.queueChannel || transport.channel;
 
     const result = await channel.checkQueue(queueName);
     return {
@@ -350,7 +356,8 @@ class QueueManager {
     if (!transport || !transport.channel) {
       throw new Error('MQ client not connected');
     }
-    const channel = transport.channel;
+    // Exchange operations can use queueChannel (regular channel) - no RPC issues
+    const channel = transport.queueChannel || transport.channel;
 
     return channel.assertExchange(exchangeName, type, {
       durable: options.durable !== false,
@@ -369,7 +376,8 @@ class QueueManager {
     if (!transport || !transport.channel) {
       throw new Error('MQ client not connected');
     }
-    const channel = transport.channel;
+    // Bind operations can use queueChannel (regular channel) - no RPC issues
+    const channel = transport.queueChannel || transport.channel;
 
     return channel.bindQueue(queue, exchange, routingKey);
   }

package/src/transports/rabbitmqClient.js

@@ -38,10 +38,19 @@ class RabbitMQClient extends EventEmitter {
 
   /**
    * Getter for channel - provides compatibility with QueueManager
+   * Returns ConfirmChannel for publish operations
    */
   get channel() {
     return this._channel;
   }
+  
+  /**
+   * Getter for queue channel - regular channel for queue operations
+   * Use this for assertQueue, checkQueue to avoid RPC reply queue issues
+   */
+  get queueChannel() {
+    return this._queueChannel || this._channel; // Fallback to main channel if queueChannel not available
+  }
 
   /**
    * Connects to RabbitMQ server and creates a confirm channel.
@@ -95,6 +104,15 @@ class RabbitMQClient extends EventEmitter {
    * @returns {Promise<void>}
    */
   async disconnect() {
+    try {
+      if (this._queueChannel) {
+        await this._queueChannel.close();
+        this._queueChannel = null;
+      }
+    } catch (err) {
+      // Log but don't emit - queue channel errors are less critical
+      console.warn('[RabbitMQClient] Error closing queue channel:', err.message);
+    }
     try {
       if (this._channel) {
         await this._channel.close();
@@ -134,9 +152,22 @@ class RabbitMQClient extends EventEmitter {
     try {
       // Ensure queue exists if publishing directly to queue and using default exchange
       if (!exchange) {
+        // Use queueChannel (regular channel) for queue operations to avoid RPC reply queue issues
         // Use central config for infrastructure queues
         const queueOptions = this._getQueueOptions(queue);
-        await this._channel.assertQueue(queue, queueOptions);
+        try {
+          await this._queueChannel.checkQueue(queue);
+          // Queue exists - proceed to publish
+        } catch (checkErr) {
+          // If queue doesn't exist (404), create it
+          if (checkErr.code === 404) {
+            await this._queueChannel.assertQueue(queue, queueOptions);
+          } else {
+            // Other error (including 406) - queue exists with different args, proceed
+            console.warn(`[RabbitMQClient] Queue ${queue} exists with different arguments, using as-is`);
+          }
+        }
+        // Publish using ConfirmChannel (for publisher confirms)
         this._channel.sendToQueue(queue, buffer, { persistent, headers, routingKey });
       } else {
         // If exchange is specified, assert exchange and publish to it
@@ -177,9 +208,10 @@ class RabbitMQClient extends EventEmitter {
         
         if (isInfraQueue) {
           // Infrastructure queue - should already exist, created by infrastructure initialization
+          // Use queueChannel (regular channel) for queue operations to avoid RPC reply queue issues
           // Only check if it exists, don't try to create or assert with arguments
           try {
-            await this._channel.checkQueue(queue);
+            await this._queueChannel.checkQueue(queue);
             // Queue exists - proceed to consume
           } catch (checkErr) {
             if (checkErr.code === 404) {
@@ -191,13 +223,14 @@ class RabbitMQClient extends EventEmitter {
           }
         } else {
           // Business queue - may need to be created, but use unified config
+          // Use queueChannel (regular channel) for queue operations to avoid RPC reply queue issues
           const queueOptions = this._getQueueOptions(queue, { durable });
           
           // Try to assert queue with our config
           // If it fails with 406 (PRECONDITION-FAILED), queue exists with different args - use it as-is
           // IMPORTANT: Don't try to re-assert after 406, as it will close the channel
           try {
-            await this._channel.assertQueue(queue, queueOptions);
+            await this._queueChannel.assertQueue(queue, queueOptions);
           } catch (assertErr) {
             // If queue exists with different arguments (406), use it as-is without re-asserting
             if (assertErr.code === 406) {