@halsystems/red-bacnet 2.2.1 → 2.2.2-alpha.0

package/CHANGELOG.md

@@ -1,5 +1,14 @@
 # Changelog
 
+## [2.2.2-alpha.0]
+### Added
+- `CoalescedJobQueue` added `onStart` and `onStop` hooks
+- auto sync to `red-bacnet-public` repo
+
+### Fixed
+- `discover device` fixed `#IAmHandler` memory leak issue
+- `CICD` Fixed broken pipeline due to package registry caching / propagation issue
+
 ## [2.2.1]
 ### Fixed
 - Fixed unhandled promise rejections in BACnet `jobs` that could hang the job queue or crash Node-RED; errors are now reported instead.

package/common/core/constant.js

@@ -31,7 +31,5 @@ module.exports = {
 
     // job error
     ERR_CONSTRUCT_ABSTRACT_JOB: 'Cannot construct abstract Job class',
-    ERR_ONSTART_NOT_IMPLEMENTED: 'OnStart method is not implemented.',
-    ERR_ONSTOP_NOT_IMPLEMENTED: 'OnStop method is not implemented.',
     ERR_EXECUTE_NOT_IMPLEMENTED: 'Execute method is not implemented.',
 }

package/common/job/core.js

@@ -3,8 +3,7 @@ require('./_alias.js');
 
 const { print } = require('@root/common/core/util.js');
 const {
-    ERR_CONSTRUCT_ABSTRACT_JOB, ERR_ONSTART_NOT_IMPLEMENTED,
-    ERR_ONSTOP_NOT_IMPLEMENTED, ERR_EXECUTE_NOT_IMPLEMENTED
+    ERR_CONSTRUCT_ABSTRACT_JOB, ERR_EXECUTE_NOT_IMPLEMENTED
 } = require('@root/common/core/constant.js')
 
 /**
@@ -51,10 +50,8 @@ class BaseJob {
      *
      * @async
      * @returns {Promise<void>}
-     * @throws {Error} If not implemented by subclass.
      */
     async onStart() {
-        throw new Error(ERR_ONSTART_NOT_IMPLEMENTED);
     }
 
     /**
@@ -79,10 +76,8 @@ class BaseJob {
      *
      * @async
      * @returns {Promise<void>}
-     * @throws {Error} If not implemented by subclass.
      */
     async onStop() {
-        throw new Error(ERR_ONSTOP_NOT_IMPLEMENTED);
     }
 }
 
@@ -160,9 +155,12 @@ class CoalescedJobQueue {
 
                 const job = this.queue.splice(jobIndex, 1)[0];
                 try {
+                    await job.task.onStart();
                     await job.task.execute();
                 } catch (error) {
                     console.error(`Error executing job ${job.id}:`, error);
+                } finally {
+                    await job.task.onStop();
                 }
             }
             await new Promise(resolve => setTimeout(resolve, this.delay));

package/common/job/discover_device.js

@@ -22,6 +22,7 @@ module.exports = {
         knownDevices = new Set();
         discoverList = [];
         discoveredDevices = [];
+        #boundIAmHandler;
 
         /**
          * Construct a DiscoverDeviceJob.
@@ -46,17 +47,17 @@ module.exports = {
             this.highLimit = highLimit
             this.whoIsTimeout = whoIsTimeout
             this.name = name
+            this.#boundIAmHandler = this.#iAmHandler.bind(this)
         }
 
         async onStart() {
-            this.client.on(EVENT_I_AM, this.#iAmHandler.bind(this))
+            this.client.on(EVENT_I_AM, this.#boundIAmHandler)
         }
         async onStop() {
-            this.client.removeListener(EVENT_I_AM, this.#iAmHandler.bind(this))
+            this.client.removeListener(EVENT_I_AM, this.#boundIAmHandler)
         }
 
         async execute() {
-            // discover device job will be reused
             this.knownDevices.clear();
             this.discoverList = [];
             this.discoveredDevices = [];

package/common/job/global_queue_manager.js

@@ -72,7 +72,7 @@ module.exports = {
     getQueue() {
         if (!globalQueue) {
             stopping = false;
-            globalQueue = new CoalescedJobQueue(100, true, 5);
+            globalQueue = new CoalescedJobQueue(50, true, 5);
         }
 
         if (!globalQueue.isRunning && !stopping) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@halsystems/red-bacnet",
-  "version": "2.2.1",
+  "version": "2.2.2-alpha.0",
   "publishConfig": {
     "registry": "https://registry.npmjs.org/",
     "access": "public"

package/red-bacnet/discover_device.js

@@ -9,7 +9,7 @@ const { DiscoverDeviceJob } = require('@root/common/job/discover_device.js')
 const { getQueue } = require('@root/common/job/global_queue_manager.js')
 const { generateUniqueJobId } = require('@root/common/func.js')
 const {
-    EVENT_UPDATE_STATUS, EVENT_ERROR, EVENT_INPUT, EVENT_OUTPUT, EVENT_CLOSE
+    EVENT_UPDATE_STATUS, EVENT_ERROR, EVENT_INPUT, EVENT_OUTPUT
 } = require('@root/common/core/constant.js')
 
 
@@ -37,18 +37,6 @@ module.exports = function (RED) {
 
             // configure job queue - use global queue
             this.job = getQueue();
-
-            // init task
-            this.task = new DiscoverDeviceJob(
-                this.client,
-                this.#eventEmitter,
-                null,
-                this.network,
-                this.lowLimit,
-                this.highLimit,
-                this.whoIsTimeout,
-            );
-            this.task.onStart();
         }
 
         #subscribeListeners() {
@@ -57,11 +45,6 @@ module.exports = function (RED) {
             */
             // @ts-ignore
             this.on(EVENT_INPUT, async function (msg) {
-                /*
-                BACnet iAm handler is causing memory leak due to unable to detach after use
-                as a result, workaround is introduce before strategy to reuse discoverDeviceJob instead
-                of creating new one every time onInput is triggered
-                */
                 const jobId = (typeof msg.id === 'string' || typeof msg.id === 'number') ? msg.id : this.jobId;
 
                 if (this.job.queue.map(item => item.id).includes(jobId)) {
@@ -70,14 +53,19 @@ module.exports = function (RED) {
                     return;
                 }
 
-                this.task.network = (msg.network === undefined) ? this.network : msg.network;
-                this.task.lowLimit = (msg.lowLimit === undefined) ? this.lowLimit : msg.lowLimit;
-                this.task.highLimit = (msg.highLimit === undefined) ? this.highLimit : msg.highLimit;
-                this.task.msg = msg;
+                const task = new DiscoverDeviceJob(
+                    this.client,
+                    this.#eventEmitter,
+                    msg,
+                    this.network,
+                    this.lowLimit,
+                    this.highLimit,
+                    this.whoIsTimeout,
+                );
 
                 this.job.addJob({
                     id: jobId,
-                    task: this.task,
+                    task: task,
                     priority: Number.isFinite(msg.priority) ? msg.priority : 5,
                 });
 
@@ -85,11 +73,6 @@ module.exports = function (RED) {
                 this.status({ fill: 'yellow', shape: 'dot', text: `in queue` });
             });
 
-            // @ts-ignore
-            this.on(EVENT_CLOSE, () => {
-                this.task.onStop();
-            })
-
             this.#eventEmitter.on(EVENT_OUTPUT, (msg) => {
                 // @ts-ignore
                 this.send(msg);