@onlineapps/conn-infra-mq 1.1.28 → 1.1.29

package/package.json

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

package/src/transports/rabbitmqClient.js

@@ -154,19 +154,29 @@ 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);
-        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`);
+        // CRITICAL: For business queues, do NOT use checkQueue() - it closes channel on 404!
+        // Business queues should already be created by setupServiceQueues() BEFORE publish is called
+        // If queue doesn't exist, that's a programming error - just proceed to publish
+        // publish() will fail with a clear error if queue doesn't exist
+        const isBusinessQueue = queueConfig.isBusinessQueue(queue);
+        
+        if (isBusinessQueue) {
+          // Business queue - should already exist, skip checkQueue() to avoid channel closure
+          console.log(`[RabbitMQClient] DEBUG: Business queue ${queue}, skipping checkQueue() in publish() - queue should already exist`);
+        } else {
+          // Infrastructure queue - check if it exists
+          const queueOptions = this._getQueueOptions(queue);
+          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)