@onlineapps/cookbook-router 3.0.0 → 3.0.1

package/README.md

@@ -84,6 +84,23 @@ amqplib message properties. One object fed both until d.396d, which meant the
 queue was declared with `persistent` and the message published with `durable` —
 neither of which the receiving side reads.
 
+A queue limit — how long a message lives, how many the queue holds — is one
+entry of `arguments`, written in the broker's own vocabulary: `x-message-ttl`,
+`x-max-length`. This package translates nothing: `queueOptions` and the per-call
+`options` of `ensureQueue()` are merged and handed to `assertQueue()` as they
+are, so a friendlier spelling such as `maxLength` arrives at the client as a key
+it does not read, and the client refuses it by name rather than dropping it:
+
+    ensureQueue('quotes.workflow', {
+      arguments: { 'x-max-length': 1000, 'x-message-ttl': 60000 }
+    })
+    → assertQueue('quotes.workflow', {
+        durable: true,
+        arguments: { 'x-max-length': 1000, 'x-message-ttl': 60000 }
+      })
+
+A queue asked for without limits carries no `arguments` key at all.
+
 A key not in this table is not read by anything here.
 
 ## Queue naming

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@onlineapps/cookbook-router",
-  "version": "3.0.0",
+  "version": "3.0.1",
   "description": "Message routing for cookbook workflows - handles service discovery and queue routing",
   "oa": {
     "category": "orchestration"

package/src/queueManager.js

@@ -111,7 +111,11 @@ class QueueManager {
    * Ensure queue exists
    * @param {string} queueName - Queue name
    * @param {Object} options - Queue options, as `mqClient.assertQueue()` declares
-   *   them (`durable`, `arguments`, `exclusive`, `autoDelete`)
+   *   them (`durable`, `arguments`, `exclusive`, `autoDelete`). A limit is one
+   *   entry of `arguments`, in the broker's own vocabulary — `x-max-length`,
+   *   `x-message-ttl`. This class translates no friendlier spelling into one:
+   *   `maxLength` is a key the client does not read, and since d.396c it says so
+   *   by name instead of dropping it.
    * @returns {Promise<boolean>}
    */
   async ensureQueue(queueName, options = {}) {