@seriousme/opifex 1.10.1 → 1.11.1

package/dist/utils/mod.js.map

@@ -1 +1 @@
-{"version":3,"file":"mod.js","sourceRoot":"","sources":["../../utils/mod.ts"],"names":[],"mappings":"AAAA;;;GAGG;AACH,OAAO,EAAE,MAAM,EAAE,MAAM,aAAa,CAAC;AACrC,OAAO,EAAE,QAAQ,EAAE,MAAM,eAAe,CAAC;AACzC,OAAO,EAAE,MAAM,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,aAAa,CAAC;AACvD,OAAO,EAAE,UAAU,EAAE,MAAM,iBAAiB,CAAC;AAC7C,OAAO,EAAE,QAAQ,EAAE,MAAM,eAAe,CAAC"}
+{"version":3,"file":"mod.js","sourceRoot":"","sources":["../../utils/mod.ts"],"names":[],"mappings":"AAAA;;;GAGG;AACH,OAAO,EAAE,MAAM,EAAE,MAAM,aAAa,CAAC;AACrC,OAAO,EAAE,QAAQ,EAAE,MAAM,eAAe,CAAC;AACzC,OAAO,EAAE,MAAM,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,aAAa,CAAC;AACvD,OAAO,EAAE,qBAAqB,EAAE,MAAM,4BAA4B,CAAC"}

package/dist/utils/queue.d.ts

@@ -0,0 +1,43 @@
+/**
+ * A simple queue implementation using an array.
+ * This implementation optimizes for O(1) enqueue and dequeue operations
+ * by avoiding array shifts, which are O(n).
+ *
+ * Usage:
+ * ```ts
+ * const queue = new ArrayQueue<number>();
+ * queue.enqueue(1);
+ * queue.enqueue(2);
+ * console.log(queue.dequeue()); // 1
+ * console.log(queue.size()); // 1
+ * console.log(queue.isEmpty()); // false
+ * queue.dequeue();
+ * console.log(queue.isEmpty()); // true
+ * ```
+ */
+export declare class ArrayQueue<T> {
+    private items;
+    private headIndex;
+    /**
+     * size is the difference between the actual array length and the index offset
+     * @returns the number of items in the queue
+     */
+    size(): number;
+    /**
+     * isEmpty checks if the queue has no items
+     * @returns true if the queue is empty
+     */
+    isEmpty(): boolean;
+    /**
+     * enqueue adds an item to the end of the queue
+     *  O(1) - Push to the end is highly optimized in V8.
+     */
+    enqueue(data: T): void;
+    /**
+     * dequeue removes and returns the item at the front of the queue
+     * O(1) - Avoids array shifting by incrementing the index.
+     * @returns the item at the front of the queue or undefined if the queue is empty
+     */
+    dequeue(): T | undefined;
+}
+//# sourceMappingURL=queue.d.ts.map

package/dist/utils/queue.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"queue.d.ts","sourceRoot":"","sources":["../../utils/queue.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;GAgBG;AAEH,qBAAa,UAAU,CAAC,CAAC;IACvB,OAAO,CAAC,KAAK,CAAW;IACxB,OAAO,CAAC,SAAS,CAAa;IAE9B;;;OAGG;IACI,IAAI,IAAI,MAAM;IAGrB;;;OAGG;IACI,OAAO,IAAI,OAAO;IAGzB;;;OAGG;IACI,OAAO,CAAC,IAAI,EAAE,CAAC,GAAG,IAAI;IAG7B;;;;OAIG;IACI,OAAO,IAAI,CAAC,GAAG,SAAS;CAiBhC"}

package/dist/utils/queue.js

@@ -0,0 +1,63 @@
+/**
+ * A simple queue implementation using an array.
+ * This implementation optimizes for O(1) enqueue and dequeue operations
+ * by avoiding array shifts, which are O(n).
+ *
+ * Usage:
+ * ```ts
+ * const queue = new ArrayQueue<number>();
+ * queue.enqueue(1);
+ * queue.enqueue(2);
+ * console.log(queue.dequeue()); // 1
+ * console.log(queue.size()); // 1
+ * console.log(queue.isEmpty()); // false
+ * queue.dequeue();
+ * console.log(queue.isEmpty()); // true
+ * ```
+ */
+export class ArrayQueue {
+    items = [];
+    headIndex = 0; // Tracks the front of the queue
+    /**
+     * size is the difference between the actual array length and the index offset
+     * @returns the number of items in the queue
+     */
+    size() {
+        return this.items.length - this.headIndex;
+    }
+    /**
+     * isEmpty checks if the queue has no items
+     * @returns true if the queue is empty
+     */
+    isEmpty() {
+        return this.size() === 0;
+    }
+    /**
+     * enqueue adds an item to the end of the queue
+     *  O(1) - Push to the end is highly optimized in V8.
+     */
+    enqueue(data) {
+        this.items.push(data);
+    }
+    /**
+     * dequeue removes and returns the item at the front of the queue
+     * O(1) - Avoids array shifting by incrementing the index.
+     * @returns the item at the front of the queue or undefined if the queue is empty
+     */
+    dequeue() {
+        if (this.isEmpty()) {
+            return undefined;
+        }
+        const item = this.items[this.headIndex];
+        this.headIndex++; // Move the "front" marker forward
+        // Optimization: Periodically "clean up" the dead space
+        // when the empty space (headIndex) exceeds a threshold.
+        // This amortizes the cost, keeping average time closer to O(1).
+        if (this.headIndex > 2000 && this.headIndex > this.items.length / 4) {
+            this.items = this.items.slice(this.headIndex);
+            this.headIndex = 0;
+        }
+        return item;
+    }
+}
+//# sourceMappingURL=queue.js.map

package/dist/utils/queue.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"queue.js","sourceRoot":"","sources":["../../utils/queue.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;GAgBG;AAEH,MAAM,OAAO,UAAU;IACb,KAAK,GAAQ,EAAE,CAAC;IAChB,SAAS,GAAW,CAAC,CAAC,CAAC,gCAAgC;IAE/D;;;OAGG;IACI,IAAI;QACT,OAAO,IAAI,CAAC,KAAK,CAAC,MAAM,GAAG,IAAI,CAAC,SAAS,CAAC;IAC5C,CAAC;IACD;;;OAGG;IACI,OAAO;QACZ,OAAO,IAAI,CAAC,IAAI,EAAE,KAAK,CAAC,CAAC;IAC3B,CAAC;IACD;;;OAGG;IACI,OAAO,CAAC,IAAO;QACpB,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IACxB,CAAC;IACD;;;;OAIG;IACI,OAAO;QACZ,IAAI,IAAI,CAAC,OAAO,EAAE,EAAE,CAAC;YACnB,OAAO,SAAS,CAAC;QACnB,CAAC;QAED,MAAM,IAAI,GAAG,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;QACxC,IAAI,CAAC,SAAS,EAAE,CAAC,CAAC,kCAAkC;QAEpD,uDAAuD;QACvD,wDAAwD;QACxD,gEAAgE;QAChE,IAAI,IAAI,CAAC,SAAS,GAAG,IAAI,IAAI,IAAI,CAAC,SAAS,GAAG,IAAI,CAAC,KAAK,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YACpE,IAAI,CAAC,KAAK,GAAG,IAAI,CAAC,KAAK,CAAC,KAAK,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;YAC9C,IAAI,CAAC,SAAS,GAAG,CAAC,CAAC;QACrB,CAAC;QACD,OAAO,IAAI,CAAC;IACd,CAAC;CACF"}

package/dist/utils/utils.d.ts

@@ -0,0 +1,2 @@
+export declare const noop: () => void;
+//# sourceMappingURL=utils.d.ts.map

package/dist/utils/utils.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"utils.d.ts","sourceRoot":"","sources":["../../utils/utils.ts"],"names":[],"mappings":"AAKA,eAAO,MAAM,IAAI,YAAW,CAAC"}

package/dist/utils/utils.js

@@ -0,0 +1,6 @@
+/*
+ * This file contains utilities used throughout the project too small to
+ * deserve their own dedicated files.
+ */
+export const noop = () => { };
+//# sourceMappingURL=utils.js.map

package/dist/utils/utils.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"utils.js","sourceRoot":"","sources":["../../utils/utils.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,MAAM,CAAC,MAAM,IAAI,GAAG,GAAG,EAAE,GAAE,CAAC,CAAC"}

package/examples/tcpServer.test.ts

@@ -72,7 +72,6 @@ test("Test pubSub using client and server", async function () {
 
   await sleep(100);
   logger.verbose(`Disconnect client`);
-  //client.closeMessages();
   await client.disconnect();
 
   logger.verbose(`Check completeness`);

package/mqttConn/mqttConn.test.ts

@@ -56,7 +56,7 @@ test("MqttConn should act as asyncIterator", async () => {
   );
   const mqttConn = new MqttConn({ conn, protocolLevel: 4 });
 
-  const packets = [];
+  const packets: AnyPacket[] = [];
   for await (const packet of mqttConn) {
     packets.push(packet);
   }
@@ -71,7 +71,7 @@ test("MqttConn should close on malformed length", async () => {
   const conn = makeDummySockConn([new Uint8Array([1, 175])], new Uint8Array());
   const mqttConn = new MqttConn({ conn });
 
-  const packets = [];
+  const packets: AnyPacket[] = [];
   for await (const packet of mqttConn) {
     packets.push(packet);
   }
@@ -89,7 +89,7 @@ test("MqttConn should close on failed packets", async () => {
   const conn = makeDummySockConn([connect, brokenPublish], new Uint8Array());
   const mqttConn = new MqttConn({ conn });
 
-  const packets = [];
+  const packets: AnyPacket[] = [];
   for await (const packet of mqttConn) {
     packets.push(packet);
   }
@@ -105,7 +105,8 @@ test("MqttConn should close on packets too large", async () => {
 
   const conn = makeDummySockConn([connect], new Uint8Array());
   const mqttConn = new MqttConn({ conn, maxIncomingPacketSize: 20 });
-  const packets = [];
+
+  const packets: AnyPacket[] = [];
   for await (const packet of mqttConn) {
     packets.push(packet);
   }

package/mqttConn/mqttConn.ts

@@ -41,8 +41,6 @@ export interface IMqttConn extends AsyncIterable<AnyPacket> {
   readonly isClosed: boolean;
   /** Reason for connection closure if any */
   readonly reason: string | undefined;
-  /** Async iterator for receiving packets */
-  [Symbol.asyncIterator](): AsyncIterableIterator<AnyPacket>;
   /** Send an MQTT packet */
   send(data: AnyPacket): Promise<void>;
   /** Close the connection */
@@ -166,27 +164,33 @@ export class MqttConn implements IMqttConn {
   }
 
   /**
-   * Async iterator for receiving packets
-   * @yields MQTT packets
+   * Returns incoming MQTT packages until the connection is closed. Note that
+   * the connection will close automatically upon receiving an invalid packet.
+   *
+   * @returns a Promise that resolves to AnyPacket if the connection is open
+   *          and working nominally, undefined otherwise.
    */
-  async *[Symbol.asyncIterator](): AsyncIterableIterator<AnyPacket> {
-    while (!this._isClosed) {
-      try {
-        yield await readPacket(
-          this.conn,
-          this.codecOpts,
-        );
-      } catch (err) {
+  #receive(): Promise<AnyPacket | undefined> {
+    if (!this._isClosed) {
+      return readPacket(this.conn, this.codecOpts).catch((err) => {
         if (err instanceof Error) {
           if (err.name === "PartialReadError") {
             err.message = MqttConnError.UnexpectedEof;
           }
           this._reason = err.message;
         }
-        // packet too large, malformed packet or connection closed
+        // packet too large, malformed packet or connection closed.
         this.close();
-        break;
-      }
+        return undefined;
+      });
+    }
+    return Promise.resolve(undefined);
+  }
+
+  async *[Symbol.asyncIterator](): AsyncIterator<AnyPacket> {
+    let packet: AnyPacket | undefined;
+    while ((packet = await this.#receive()) !== undefined) {
+      yield packet;
     }
   }
 

package/mqttPacket/auth.ts

@@ -5,13 +5,6 @@ import { Decoder, DecoderError, hasEmptyFlags } from "./decoder.ts";
 import type { TReasonCode } from "./ReasonCode.ts";
 import type { AuthProperties } from "./Properties.ts";
 
-/**
- * An AUTH packet is sent from Client to Server or Server to Client as part of an extended
- * authentication exchange, such as challenge / response authentication.
- * It is a Protocol Error for the Client or Server to send an AUTH packet
- * if the CONNECT packet did not contain the same Authentication Method.
- */
-
 // AuthPacket does not exist on protocol levels < 5
 export type AuthPacketV5 = {
   type: TPacketType;
@@ -20,6 +13,12 @@ export type AuthPacketV5 = {
   properties?: AuthProperties;
 };
 
+/**
+ * An AUTH packet is sent from Client to Server or Server to Client as part of an extended
+ * authentication exchange, such as challenge / response authentication.
+ * It is a Protocol Error for the Client or Server to send an AUTH packet
+ * if the CONNECT packet did not contain the same Authentication Method.
+ */
 export type AuthPacket = AuthPacketV5;
 
 export const auth: {

package/mqttPacket/connack.ts

@@ -17,10 +17,6 @@ import { AuthenticationResultByNumber } from "./AuthenticationResult.ts";
 import type { TReasonCode } from "./ReasonCode.ts";
 import type { ConnackProperties } from "./Properties.ts";
 
-/**
- * ConnackPacket is sent from the server to the client in response to a connect packet.
- * It indicates that the connect is accepted.
- */
 export type ConnackPacketV4 = {
   type: TPacketType;
   protocolLevel: ProtocolLevelNoV5;
@@ -36,6 +32,10 @@ export type ConnackPacketV5 = {
   properties?: ConnackProperties;
 };
 
+/**
+ * ConnackPacket is sent from the server to the client in response to a connect packet.
+ * It indicates that the connect is accepted.
+ */
 export type ConnackPacket = ConnackPacketV4 | ConnackPacketV5;
 
 export const connack: {

package/mqttPacket/connect.ts

@@ -20,9 +20,6 @@ import {
   hasEmptyFlags,
 } from "./decoder.ts";
 
-/**
- * ConnectPacket is sent from the client to the server to initiate a connection.
- */
 export type ConnectPacketV4 = {
   type: TPacketType;
   protocolName?: string;
@@ -61,6 +58,9 @@ export type ConnectPacketV5 = {
   properties?: ConnectProperties;
 };
 
+/**
+ * ConnectPacket is sent from the client to the server to initiate a connection.
+ */
 export type ConnectPacket = ConnectPacketV4 | ConnectPacketV5;
 
 function invalidProtocolName(version: number, name: string): boolean {

package/mqttPacket/disconnect.ts

@@ -10,10 +10,6 @@ import type { DisconnectProperties } from "./Properties.ts";
 import { Encoder } from "./encoder.ts";
 import { ReasonCode } from "./ReasonCode.ts";
 
-/**
- * DisconnectPacket is the final control packet sent from the client to the server.
- * It indicates that the client is disconnecting cleanly.
- */
 export type DisconnectPacketv4 = {
   type: TPacketType;
   protocolLevel: ProtocolLevelNoV5;
@@ -26,6 +22,10 @@ export type DisconnectPacketv5 = {
   properties?: DisconnectProperties;
 };
 
+/**
+ * DisconnectPacket is the final control packet sent from the client to the server.
+ * It indicates that the client is disconnecting cleanly.
+ */
 export type DisconnectPacket = DisconnectPacketv4 | DisconnectPacketv5;
 
 const DISCONNECT_PACKET = new Uint8Array([PacketType.disconnect << 4, 0]);

package/mqttPacket/mod.ts

@@ -209,6 +209,7 @@ export function decodePayload(
 /**
  * @function decode
  * @description Decodes a complete MQTT packet from binary format into a packet object
+ *
  * @param {Uint8Array} buffer - The binary buffer containing the complete MQTT packet
  * @param {CodecOpts} codecOpts - the options for the decoder
  * @returns {AnyPacket} The decoded MQTT packet object

package/mqttPacket/protocolLevels.ts

@@ -4,10 +4,10 @@
 
 import type { ProtocolLevel } from "./types.ts";
 
-// MQTT Protocol levels
+/** available MQTT Protocol levels */
 export const MQTTLevel = {
   unknown: undefined as ProtocolLevel,
   v3: 3 as ProtocolLevel,
   v4: 4 as ProtocolLevel,
   v5: 5 as ProtocolLevel,
-};
+} as const;

package/mqttPacket/pubblishAcks.ts

@@ -10,14 +10,6 @@ import { PacketType } from "./PacketType.ts";
 import { Decoder, DecoderError } from "./decoder.ts";
 import { Encoder } from "./encoder.ts";
 
-/**
- * PubackPacket is sent to indicate publish complete (QoS 1)
- * Pubrec is sent to indicate publish received (QoS 2)
- * Pubrel is sent to indicate publish release (QoS 2)
- * Pubcomp is sent to indicate publish complete (QoS 2)
- *
- * all 4 packets are identical except for packet type
- */
 export type AckPacketV4<T> = {
   type: TPacketType;
   protocolLevel: ProtocolLevelNoV5;
@@ -31,11 +23,25 @@ export type AckPacketV5<T> = {
   reasonCode?: TReasonCode;
   properties?: PubackProperties;
 };
-
+/**
+ * all 4 ack packets are identical except for packet type and flags of Pubrel
+ */
 export type AckPacket<T> = AckPacketV4<T> | AckPacketV5<T>;
+/**
+ * PubackPacket is sent to indicate publish complete (QoS 1)
+ */
 export type PubackPacket = AckPacket<typeof PacketType.puback>;
+/**
+ * Pubrec is sent to indicate publish received (QoS 2)
+ */
 export type PubrecPacket = AckPacket<typeof PacketType.pubrec>;
+/**
+ * Pubrel is sent to indicate publish release (QoS 2)
+ */
 export type PubrelPacket = AckPacket<typeof PacketType.pubrel>;
+/**
+ * Pubcomp is sent to indicate publish complete (QoS 2)
+ */
 export type PubcompPacket = AckPacket<typeof PacketType.pubcomp>;
 
 export type AnyAckPacket =

package/mqttPacket/suback.ts

@@ -12,10 +12,6 @@ import { Encoder } from "./encoder.ts";
 import { Decoder, DecoderError } from "./decoder.ts";
 const validReturnCodes = [0x00, 0x01, 0x02, 0x80];
 
-/**
- * SubackPacket is sent by the server to the client to confirm receipt and
- * processing of a SubscribePacket.
- */
 export type SubackPacketV4 = {
   type: TPacketType;
   protocolLevel: ProtocolLevelNoV5;
@@ -31,6 +27,10 @@ export type SubackPacketV5 = {
   reasonCodes: Array<TReasonCode>;
 };
 
+/**
+ * SubackPacket is sent by the server to the client to confirm receipt and
+ * processing of a SubscribePacket.
+ */
 export type SubackPacket = SubackPacketV4 | SubackPacketV5;
 
 export const suback: {

package/mqttPacket/subscribe.ts

@@ -13,9 +13,6 @@ import { BitMask } from "./BitMask.ts";
 import { Encoder } from "./encoder.ts";
 import { booleanFlag, Decoder, DecoderError } from "./decoder.ts";
 
-/**
- * SubscribePacket is sent from client to server to subscribe to topics
- */
 export type SubscribePacketV4 = {
   type: TPacketType;
   protocolLevel: ProtocolLevelNoV5;
@@ -31,6 +28,9 @@ export type SubscribePacketV5 = {
   subscriptions: Array<SubscriptionV5>;
 };
 
+/**
+ * SubscribePacket is sent from client to server to subscribe to topics
+ */
 export type SubscribePacket = SubscribePacketV4 | SubscribePacketV5;
 /**
  * The topic to subscribe to and the associated QoS

package/mqttPacket/types.ts

@@ -73,6 +73,9 @@ export type ReturnCodes = number[];
  */
 export type ClientId = string;
 
+/**
+ * helper type to limit retain handling values
+ */
 export type TRetainHandling =
   typeof RetainHandling[keyof typeof RetainHandling];
 

package/mqttPacket/unsuback.ts

@@ -10,10 +10,6 @@ import { PacketType } from "./PacketType.ts";
 import { Decoder, hasEmptyFlags } from "./decoder.ts";
 import { Encoder } from "./encoder.ts";
 
-/**
- * UnsubackPacket is sent by the server to the client to confirm receipt
- * of an UnsubscribePacket.
- */
 export type UnsubackPacketV4 = {
   type: TPacketType;
   protocolLevel: ProtocolLevelNoV5;
@@ -28,6 +24,10 @@ export type UnsubackPacketV5 = {
   reasonCodes: Array<TReasonCode>;
 };
 
+/**
+ * UnsubackPacket is sent by the server to the client to confirm receipt
+ * of an UnsubscribePacket.
+ */
 export type UnsubackPacket = UnsubackPacketV4 | UnsubackPacketV5;
 
 export const unsuback: {

package/mqttPacket/unsubscribe.ts

@@ -11,9 +11,6 @@ import { PacketType } from "./PacketType.ts";
 import { Encoder } from "./encoder.ts";
 import { Decoder, DecoderError } from "./decoder.ts";
 
-/**
- * UnsubscribePacket is sent from client to server to unsubscribe from topics
- */
 export type UnsubscribePacketV4 = {
   type: TPacketType;
   protocolLevel: ProtocolLevelNoV5;
@@ -29,6 +26,9 @@ export type UnsubscribePacketV5 = {
   topicFilters: TopicFilter[];
 };
 
+/**
+ * UnsubscribePacket is sent from client to server to unsubscribe from topics
+ */
 export type UnsubscribePacket = UnsubscribePacketV4 | UnsubscribePacketV5;
 
 export const unsubscribe: {

package/mqttPacket/validators.ts

@@ -7,14 +7,29 @@ const invalidTopicRegEx = new RegExp(/^$|\+|#|\x00|\uFFFD/);
 // deno-lint-ignore no-control-regex
 const invalidTopicFilterRegEx = new RegExp(/^$|#.|[^\/]\+|\+[^\/]|\x00|\uFFFD/);
 
+/**
+ * check for invalid UTF-8 characters
+ * @param value string to test
+ * @returns true if the string contains invalid UTF-8 characters
+ */
 export function invalidUTF8(value: string): boolean {
   return invalidUTF8regEx.test(value);
 }
 
+/**
+ * check for invalid topic characters
+ * @param value
+ * @returns true if the topic is invalid
+ */
 export function invalidTopic(value: Topic): boolean {
   return invalidTopicRegEx.test(value);
 }
 
+/**
+ * check for invalid topic filter characters
+ * @param value topic filter to test
+ * @returns true if the topic filter is invalid
+ */
 export function invalidTopicFilter(value: TopicFilter): boolean {
   return invalidTopicFilterRegEx.test(value);
 }

package/node/tcpClient.ts

@@ -96,6 +96,6 @@ export class TcpClient extends Client {
     if (protocol === "mqtt:") {
       return this.connectMQTT(hostname, port);
     }
-    throw `Unsupported protocol: ${protocol}`;
+    throw new Error(`Unsupported protocol: ${protocol}`);
   }
 }

package/node/tcpServer.test.ts

@@ -72,7 +72,6 @@ test("Test pubSub using client and server", async function () {
 
   await sleep(100);
   logger.verbose(`Disconnect client`);
-  //client.closeMessages();
   await client.disconnect();
 
   logger.verbose(`Check completeness`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@seriousme/opifex",
-  "version": "1.10.1",
+  "version": "1.11.1",
   "description": "MQTT client & server for Deno & NodeJS",
   "type": "module",
   "scripts": {

package/persistence/memory/memoryPersistence.ts

@@ -134,7 +134,7 @@ export class MemoryPersistence implements IPersistence {
       newPacket.qos = qos;
       //  logger.debug(`publish ${topic} to client ${clientId}`);
       const client = this.clientList.get(clientId);
-      client?.handler(packet);
+      client?.handler(newPacket);
     }
   }
 

package/server/test/handleConnect.test.ts

@@ -1,12 +1,15 @@
 import assert from "node:assert/strict";
 import { test } from "node:test";
-import { makeDummyQueueSockConn } from "../../dev_utils/mod.ts";
+import {
+  makeDummyQueueSockConn,
+  resolveNextTick,
+} from "../../dev_utils/mod.ts";
 import {
   type AnyPacket,
   AuthenticationResult,
   PacketType,
 } from "../../mqttPacket/mod.ts";
-import { AsyncQueue, nextTick } from "../../utils/mod.ts";
+import { BufferedAsyncIterable } from "../../utils/mod.ts";
 import { MqttConn, MQTTLevel } from "../deps.ts";
 import { MqttServer } from "../mod.ts";
 import { handlers } from "./test-handlers.ts";
@@ -34,11 +37,11 @@ const disconnectPacket: AnyPacket = {
 const mqttServer = new MqttServer({ handlers });
 
 function startServer(): {
-  reader: AsyncIterableIterator<AnyPacket>;
+  reader: AsyncIterator<AnyPacket>;
   mqttConn: MqttConn;
 } {
-  const reader = new AsyncQueue<Uint8Array>();
-  const writer = new AsyncQueue<Uint8Array>();
+  const reader = new BufferedAsyncIterable<Uint8Array>();
+  const writer = new BufferedAsyncIterable<Uint8Array>();
 
   const outputConn = makeDummyQueueSockConn(writer, reader);
   const mqttConn = new MqttConn({ conn: outputConn });
@@ -66,7 +69,7 @@ test("Authentication with valid username and password works", async () => {
     );
   }
   mqttConn.send(disconnectPacket);
-  await nextTick();
+  await resolveNextTick();
   assert.deepStrictEqual(
     mqttConn.isClosed,
     true,
@@ -92,7 +95,7 @@ test("Authentication with invalid username fails", async () => {
       "Expected badUsernameOrPassword",
     );
   }
-  await nextTick();
+  await resolveNextTick();
   assert.deepStrictEqual(
     mqttConn.isClosed,
     true,
@@ -118,7 +121,7 @@ test("Authentication with invalid password fails", async () => {
       "Expected badUsernameOrPassword",
     );
   }
-  await nextTick();
+  await resolveNextTick();
   assert.deepStrictEqual(
     mqttConn.isClosed,
     true,