node-red-contrib-uos-nats 0.2.54 → 0.2.56

package/lib/payloads.js

@@ -288,28 +288,26 @@ function buildVariableList(defs, states, fingerprint) {
         variables.push(encodeVariableState(def, state));
     }
     // Always provide a baseTimestamp to satisfy strict schema requirements
-    // Use current time (now) for the transmission timestamp, NOT the variable timestamp.
-    // This ensures Heartbeat packets are fresh and not discarded by DataHub.
-    let baseTimestamp = nowNs();
-    /*
-    const firstId = Object.keys(states)[0];
-    if (firstId && states[firstId] && states[firstId].timestamp) {
-        const totalNs = BigInt(states[firstId].timestamp);
-        const secs = totalNs / 1_000_000_000n;
-        const nanos = Number(totalNs % 1_000_000_000n);
-        baseTimestamp = new TimestampT(secs, nanos);
+    // Use the timestamp of the first item if available, otherwise NOW.
+    // This allows relative timestamps in the list to be non-negative if we tracked deltas,
+    // but mostly ensures logical consistency (Base <= Item).
+    let baseTimestamp = null;
+    if (variables.length > 0 && variables[0].timestamp) {
+        baseTimestamp = variables[0].timestamp;
+    } else {
+        baseTimestamp = nowNs();
     }
-    */
 
     const list = new VariableListT(fingerprint, baseTimestamp, variables);
-    // console.log(`[Payloads] Built VariableList with baseTimestamp: ${baseTimestamp ? 'SET' : 'NULL'}`);
     return list;
 }
 function encodeVariableState(def, state) {
     const varT = new VariableT();
     varT.id = def.id;
     varT.quality = VariableQuality[state?.quality?.toUpperCase?.()] ?? VariableQuality[DEFAULT_QUALITY];
-    if (state?.timestamp !== undefined) {
+
+    // Logic matches Sample: use state.timestamp or now
+    if (state?.timestamp) {
         const totalNs = BigInt(state.timestamp);
         const secs = totalNs / 1_000_000_000n;
         const nanos = Number(totalNs % 1_000_000_000n);

package/nodes/datahub-output.js

@@ -84,7 +84,7 @@ module.exports = function (RED) {
         id: nextId += 1,
         key: normalized,
         dataType,
-        access: 'READWRITE',
+        access: 'READ_ONLY',
       };
       defMap.set(normalized, def);
       definitions.push(def);
@@ -175,7 +175,10 @@ module.exports = function (RED) {
         nc = await connection.acquire();
         console.log('[DataHub Output] NATS acquired.');
 
-        await sendDefinitionUpdate(payloads, subjects);
+        // Only publish definition if we have one. Empty definitions might be rejected?
+        if (definitions.length > 0) {
+          await sendDefinitionUpdate(payloads, subjects);
+        }
 
         // Listen for Variable READ requests
         sub = nc.subscribe(subjects.readVariablesQuery(this.providerId), {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "node-red-contrib-uos-nats",
-  "version": "0.2.54",
+  "version": "0.2.56",
   "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",