node-red-contrib-uos-nats 1.3.71 → 1.3.72

package/lib/payloads.js

@@ -222,6 +222,13 @@ export function decodeVariableList(list, whitelistIds = null) {
     if (!list)
         return [];
     const result = [];
+
+    // Optimization: Reuse holders to reduce GC pressure
+    const holderInt64 = new VariableValueInt64();
+    const holderFloat64 = new VariableValueFloat64();
+    const holderBoolean = new VariableValueBoolean();
+    const holderString = new VariableValueString();
+
     for (let i = 0; i < list.itemsLength(); i += 1) {
         const item = list.items(i);
         if (!item)
@@ -236,27 +243,23 @@ export function decodeVariableList(list, whitelistIds = null) {
         let decoded;
         switch (item.valueType()) {
             case VariableValue.Int64: {
-                const holder = new VariableValueInt64();
-                item.value(holder);
-                decoded = Number(holder.value());
+                item.value(holderInt64);
+                decoded = Number(holderInt64.value());
                 break;
             }
             case VariableValue.Float64: {
-                const holder = new VariableValueFloat64();
-                item.value(holder);
-                decoded = holder.value();
+                item.value(holderFloat64);
+                decoded = holderFloat64.value();
                 break;
             }
             case VariableValue.Boolean: {
-                const holder = new VariableValueBoolean();
-                item.value(holder);
-                decoded = holder.value();
+                item.value(holderBoolean);
+                decoded = holderBoolean.value();
                 break;
             }
             case VariableValue.String: {
-                const holder = new VariableValueString();
-                item.value(holder);
-                decoded = holder.value();
+                item.value(holderString);
+                decoded = holderString.value();
                 break;
             }
             default:

package/nodes/datahub-input.js

@@ -74,12 +74,14 @@ module.exports = function (RED) {
     // Pre-populate raw map with manual definitions
     this.manualDefs.forEach(d => defMap.set(d.id, { ...d, type: 'MANUAL', dataType: 'UNKNOWN', access: 'READ_ONLY' }));
 
+    // Optimization: Use Set for O(1) lookups
+    const variableSet = new Set(this.variables.map(v => normalizeKey(v)));
+
     const shouldInclude = (key) => {
-      if (!this.variables.length) {
+      if (variableSet.size === 0) {
         return true;
       }
-      const needle = normalizeKey(key);
-      return this.variables.includes(needle);
+      return variableSet.has(normalizeKey(key));
     };
 
     const processStates = (states) => {
@@ -101,6 +103,12 @@ module.exports = function (RED) {
         this.warnOnce('Filtering active but Variable Definitions failed to load (API Error). Names cannot be resolved. Try using "Name:ID" format to manually map variables.');
       }
 
+      // Optimization: redundant check removed if whitelist is active, but keeping as safeguard 
+      // If we used a whitelist, we know we only have relevant IDs.
+      // However, keeping strict check is safer for "Name-based" consistency.
+      // But we can optimize: if mapped.length is same as whitelist size, we are good?
+      // Actually, let's keep it simple: Filter-on-Decode handles the heavy lifting (byte level).
+      // This JS filter is now cheap (O(1) lookup). We'll keep it for correctness in case of aliasing/manual IDs.
       return mapped.filter((state) => shouldInclude(state.key));
     };
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "node-red-contrib-uos-nats",
-  "version": "1.3.71",
+  "version": "1.3.72",
   "description": "Node-RED nodes for Weidmüller u-OS Data Hub. Read, write, and provide variables via NATS protocol with OAuth2 authentication. Features: Variable Key resolution, custom icons, example flows, and provider definition caching.",
   "author": {
     "name": "IoTUeli",