comq 0.15.0 → 0.15.1

package/CHANGELOG.md

@@ -1,3 +1,11 @@
+## [0.15.1](https://github.com/toa-io/comq/compare/v0.15.0...v0.15.1) (2026-08-02)
+
+
+### Bug Fixes
+
+* skip end control when reply stream buffer overflows ([654e046](https://github.com/toa-io/comq/commit/654e046bdcbfbbbbbb789e5bb032bc16564221d1)), closes [#control](https://github.com/toa-io/comq/issues/control) [#reply](https://github.com/toa-io/comq/issues/reply)
+* skip end control when reply stream buffer overflows ([#237](https://github.com/toa-io/comq/issues/237)) ([c34d87f](https://github.com/toa-io/comq/commit/c34d87f6d219ce6c8509d989117eadd55a33ca86))
+
 # [0.15.0](https://github.com/toa-io/comq/compare/v0.14.0...v0.15.0) (2026-08-01)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "comq",
-  "version": "0.15.0",
+  "version": "0.15.1",
   "description": "ComQ",
   "author": {
     "name": "Artem Gurtovoi",

package/source/.io/ReplyStream.js

@@ -26,6 +26,8 @@ class ReplyStream extends Readable {
 
   #buffered = 0
 
+  #maxBufferSize
+
   /** @type {Map<number, unknown>} */
   #queue = new Map()
 
@@ -39,6 +41,7 @@ class ReplyStream extends Readable {
     this.#emitter = request.emitter
     this.#correlationId = request.properties.correlationId
     this.#idleInterval = global['COMQ_TESTING_IDLE_INTERVAL'] || IDLE_INTERVAL
+    this.#maxBufferSize = global['COMQ_TESTING_MAX_BUFFER_SIZE'] || MAX_BUFFER_SIZE
     this.#reply = reply
 
     this.#emitter.on(this.#correlationId, this.arrange.bind(this))
@@ -48,7 +51,9 @@ class ReplyStream extends Readable {
     this._clear()
     this.push(null)
 
-    void this.#reply(this.#control, control.end)
+    if (this.#control !== undefined)
+      void this.#reply(this.#control, control.end)
+
     super._destroy(error, callback)
   }
 
@@ -95,7 +100,11 @@ class ReplyStream extends Readable {
   }
 
   _buffer (payload, properties) {
-    if (this.#buffered > MAX_BUFFER_SIZE) this.destroy()
+    if (this.#buffered > this.#maxBufferSize) {
+      this.destroy()
+
+      return
+    }
 
     this.#buffered++
     this.#queue.set(properties.headers.index, { payload, properties })

package/test/ReplyStream.test.js

@@ -0,0 +1,45 @@
+'use strict'
+
+const { EventEmitter } = require('node:events')
+const { once } = require('node:events')
+const { ReplyStream } = require('../source/.io/ReplyStream')
+
+/** @type {jest.Mock} */
+let reply
+
+/** @type {ReplyStream} */
+let stream
+
+beforeEach(() => {
+  global.COMQ_TESTING_MAX_BUFFER_SIZE = 3
+  global.COMQ_TESTING_IDLE_INTERVAL = 60_000
+
+  reply = jest.fn().mockResolvedValue(true)
+
+  const request = {
+    emitter: new EventEmitter(),
+    properties: { correlationId: 'test-correlation' }
+  }
+
+  stream = new ReplyStream(request, reply)
+})
+
+afterEach(() => {
+  delete global.COMQ_TESTING_MAX_BUFFER_SIZE
+  delete global.COMQ_TESTING_IDLE_INTERVAL
+
+  stream.destroy()
+})
+
+it('should destroy without calling reply when buffer overflows before control.ok', async () => {
+  const closed = once(stream, 'close')
+
+  for (let index = 1; index <= 5; index++) {
+    stream.arrange(index, { headers: { index } })
+  }
+
+  await closed
+
+  expect(stream.destroyed).toBe(true)
+  expect(reply).not.toHaveBeenCalled()
+})