@onlineapps/conn-infra-mq 1.1.42 → 1.1.43

package/package.json

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

package/src/transports/rabbitmqClient.js

@@ -221,48 +221,11 @@ class RabbitMQClient extends EventEmitter {
           // 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 - use assertQueue() directly (idempotent, won't cause 406)
-          // assertQueue() will use existing queue even if args differ, avoiding channel closure
-          // Recreate queueChannel if it's closed
-          if (!this._queueChannel || this._queueChannel.closed) {
-            if (!this._connection) {
-              throw new Error('Cannot assertQueue: connection is not available');
-            }
-            console.warn('[RabbitMQClient] Queue channel is closed, recreating...');
-            try {
-              if (this._queueChannel) {
-                try {
-                  await this._queueChannel.close();
-                } catch (err) {
-                  // Ignore errors when closing already-closed channel
-                }
-              }
-              this._queueChannel = await this._connection.createChannel();
-              this._queueChannel._createdAt = new Date().toISOString();
-              this._queueChannel._closeReason = null;
-              this._queueChannel._lastOperation = null;
-              this._queueChannel.on('error', (err) => {
-                console.error('[RabbitMQClient] Queue channel error:', err.message);
-                this._queueChannel._closeReason = `Error: ${err.message} (code: ${err.code})`;
-              });
-              this._queueChannel.on('close', () => {
-                console.error('[RabbitMQClient] Queue channel closed');
-              });
-            } catch (recreateErr) {
-              console.warn(`[RabbitMQClient] Failed to recreate queue channel, proceeding anyway:`, recreateErr.message);
-            }
-          }
-          
-          const queueOptions = this._getQueueOptions(queue);
-          try {
-            if (this._queueChannel && !this._queueChannel.closed) {
-              await this._queueChannel.assertQueue(queue, queueOptions);
-            }
-          } catch (assertErr) {
-            // If assertQueue fails (shouldn't happen), log and proceed anyway
-            // The queue likely exists with different args - publish will work or fail clearly
-            console.warn(`[RabbitMQClient] assertQueue failed for ${queue}, proceeding anyway:`, assertErr.message);
-          }
+          // Infrastructure queue - skip assertQueue() to avoid 406 channel closure
+          // Infrastructure queues are created by monitoring consumer at startup
+          // If queue doesn't exist, publish() will fail with a clear error
+          // This matches the pattern used for business queues - early creation, no runtime checks
+          console.log(`[RabbitMQClient] DEBUG: Infrastructure queue ${queue}, skipping assertQueue() - queue should already exist (created by monitoring consumer)`);
         }
         // Publish using ConfirmChannel (for publisher confirms)
         this._channel.sendToQueue(queue, buffer, { persistent, headers, routingKey });