node-opcua-address-space 2.89.0 → 2.90.0

package/dist/src/ua_view_impl.js

@@ -13,15 +13,15 @@ const node_opcua_address_space_base_1 = require("node-opcua-address-space-base")
 const session_context_1 = require("../source/session_context");
 const base_node_impl_1 = require("./base_node_impl");
 class UAViewImpl extends base_node_impl_1.BaseNodeImpl {
-    get eventNotifier() {
-        return this._eventNotifier;
-    }
     constructor(options) {
         super(options);
         this.nodeClass = node_opcua_data_model_1.NodeClass.View;
         this.containsNoLoops = !!options.containsNoLoops;
         this._eventNotifier = node_opcua_address_space_base_1.EventNotifierFlags.None;
     }
+    get eventNotifier() {
+        return this._eventNotifier;
+    }
     readAttribute(context, attributeId) {
         context = context || session_context_1.SessionContext.defaultContext;
         const options = {};

package/dist/src/ua_view_impl.js.map

@@ -1 +1 @@
-{"version":3,"file":"ua_view_impl.js","sourceRoot":"","sources":["../../src/ua_view_impl.ts"],"names":[],"mappings":";;;AAAA;;GAEG;AACH,iEAAkD;AAClD,iEAAqD;AACrD,iEAAiE;AACjE,mEAAqD;AACrD,2DAA8C;AAC9C,iFAA4F;AAE5F,+DAA2D;AAC3D,qDAAyE;AAMzE,MAAa,UAAW,SAAQ,6BAAY;IAIxC,IAAW,aAAa;QACpB,OAAO,IAAI,CAAC,cAAc,CAAC;IAC/B,CAAC;IAED,YAAY,OAA4B;QACpC,KAAK,CAAC,OAAO,CAAC,CAAC;QARH,cAAS,GAAG,iCAAS,CAAC,IAAI,CAAC;QASvC,IAAI,CAAC,eAAe,GAAG,CAAC,CAAC,OAAO,CAAC,eAAe,CAAC;QACjD,IAAI,CAAC,cAAc,GAAG,kDAAkB,CAAC,IAAI,CAAC;IAClD,CAAC;IAEM,aAAa,CAAC,OAA+B,EAAE,WAAyB;QAC3E,OAAO,GAAG,OAAO,IAAI,gCAAc,CAAC,cAAc,CAAC;QAEnD,MAAM,OAAO,GAAkB,EAAE,CAAC;QAElC,QAAQ,WAAW,EAAE;YACjB,KAAK,oCAAY,CAAC,aAAa;gBAC3B,OAAO,CAAC,KAAK,GAAG,EAAE,QAAQ,EAAE,6BAAQ,CAAC,IAAI,EAAE,KAAK,EAAE,IAAI,CAAC,aAAa,EAAE,CAAC;gBACvE,OAAO,CAAC,UAAU,GAAG,oCAAW,CAAC,IAAI,CAAC;gBACtC,MAAM;YAEV,KAAK,oCAAY,CAAC,eAAe;gBAC7B,OAAO,CAAC,KAAK,GAAG,EAAE,QAAQ,EAAE,6BAAQ,CAAC,OAAO,EAAE,KAAK,EAAE,IAAI,CAAC,eAAe,EAAE,CAAC;gBAC5E,OAAO,CAAC,UAAU,GAAG,oCAAW,CAAC,IAAI,CAAC;gBACtC,MAAM;YAEV;gBACI,OAAO,KAAK,CAAC,aAAa,CAAC,OAAO,EAAE,WAAW,CAAC,CAAC;SACxD;QACD,OAAO,IAAI,iCAAS,CAAC,OAAO,CAAC,CAAC;IAClC,CAAC;CACJ;AAnCD,gCAmCC"}
+{"version":3,"file":"ua_view_impl.js","sourceRoot":"","sources":["../../src/ua_view_impl.ts"],"names":[],"mappings":";;;AAAA;;GAEG;AACH,iEAAkD;AAClD,iEAAqD;AACrD,iEAAiE;AACjE,mEAAqD;AACrD,2DAA8C;AAC9C,iFAA4F;AAE5F,+DAA2D;AAC3D,qDAAyE;AAMzE,MAAa,UAAW,SAAQ,6BAAY;IAQxC,YAAY,OAA4B;QACpC,KAAK,CAAC,OAAO,CAAC,CAAC;QARH,cAAS,GAAG,iCAAS,CAAC,IAAI,CAAC;QASvC,IAAI,CAAC,eAAe,GAAG,CAAC,CAAC,OAAO,CAAC,eAAe,CAAC;QACjD,IAAI,CAAC,cAAc,GAAG,kDAAkB,CAAC,IAAI,CAAC;IAClD,CAAC;IARD,IAAW,aAAa;QACpB,OAAO,IAAI,CAAC,cAAc,CAAC;IAC/B,CAAC;IAQM,aAAa,CAAC,OAA+B,EAAE,WAAyB;QAC3E,OAAO,GAAG,OAAO,IAAI,gCAAc,CAAC,cAAc,CAAC;QAEnD,MAAM,OAAO,GAAkB,EAAE,CAAC;QAElC,QAAQ,WAAW,EAAE;YACjB,KAAK,oCAAY,CAAC,aAAa;gBAC3B,OAAO,CAAC,KAAK,GAAG,EAAE,QAAQ,EAAE,6BAAQ,CAAC,IAAI,EAAE,KAAK,EAAE,IAAI,CAAC,aAAa,EAAE,CAAC;gBACvE,OAAO,CAAC,UAAU,GAAG,oCAAW,CAAC,IAAI,CAAC;gBACtC,MAAM;YAEV,KAAK,oCAAY,CAAC,eAAe;gBAC7B,OAAO,CAAC,KAAK,GAAG,EAAE,QAAQ,EAAE,6BAAQ,CAAC,OAAO,EAAE,KAAK,EAAE,IAAI,CAAC,eAAe,EAAE,CAAC;gBAC5E,OAAO,CAAC,UAAU,GAAG,oCAAW,CAAC,IAAI,CAAC;gBACtC,MAAM;YAEV;gBACI,OAAO,KAAK,CAAC,aAAa,CAAC,OAAO,EAAE,WAAW,CAAC,CAAC;SACxD;QACD,OAAO,IAAI,iCAAS,CAAC,OAAO,CAAC,CAAC;IAClC,CAAC;CACJ;AAnCD,gCAmCC"}

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "node-opcua-address-space",
-    "version": "2.89.0",
+    "version": "2.90.0",
     "description": "pure nodejs OPCUA SDK - module -address-space",
     "main": "./dist/src/index_current.js",
     "types": "./dist/source/index.d.ts",
@@ -22,36 +22,36 @@
         "chalk": "4.1.2",
         "dequeue": "^1.0.5",
         "lodash": "4.17.21",
-        "node-opcua-address-space-base": "2.89.0",
+        "node-opcua-address-space-base": "2.90.0",
         "node-opcua-assert": "2.88.0",
-        "node-opcua-basic-types": "2.89.0",
+        "node-opcua-basic-types": "2.90.0",
         "node-opcua-binary-stream": "2.89.0",
-        "node-opcua-client-dynamic-extension-object": "2.89.0",
+        "node-opcua-client-dynamic-extension-object": "2.90.0",
         "node-opcua-constants": "2.88.0",
         "node-opcua-crypto": "^2.1.2",
-        "node-opcua-data-access": "2.89.0",
-        "node-opcua-data-model": "2.89.0",
-        "node-opcua-data-value": "2.89.0",
-        "node-opcua-date-time": "2.89.0",
-        "node-opcua-debug": "2.88.0",
+        "node-opcua-data-access": "2.90.0",
+        "node-opcua-data-model": "2.90.0",
+        "node-opcua-data-value": "2.90.0",
+        "node-opcua-date-time": "2.90.0",
+        "node-opcua-debug": "2.90.0",
         "node-opcua-enum": "2.88.0",
-        "node-opcua-extension-object": "2.89.0",
-        "node-opcua-factory": "2.89.0",
-        "node-opcua-nodeid": "2.88.0",
-        "node-opcua-nodeset-ua": "2.89.0",
-        "node-opcua-numeric-range": "2.89.0",
-        "node-opcua-object-registry": "2.88.0",
-        "node-opcua-pseudo-session": "2.89.0",
-        "node-opcua-service-browse": "2.89.0",
-        "node-opcua-service-call": "2.89.0",
-        "node-opcua-service-history": "2.89.0",
-        "node-opcua-service-translate-browse-path": "2.89.0",
-        "node-opcua-service-write": "2.89.0",
+        "node-opcua-extension-object": "2.90.0",
+        "node-opcua-factory": "2.90.0",
+        "node-opcua-nodeid": "2.90.0",
+        "node-opcua-nodeset-ua": "2.90.0",
+        "node-opcua-numeric-range": "2.90.0",
+        "node-opcua-object-registry": "2.90.0",
+        "node-opcua-pseudo-session": "2.90.0",
+        "node-opcua-service-browse": "2.90.0",
+        "node-opcua-service-call": "2.90.0",
+        "node-opcua-service-history": "2.90.0",
+        "node-opcua-service-translate-browse-path": "2.90.0",
+        "node-opcua-service-write": "2.90.0",
         "node-opcua-status-code": "2.89.0",
-        "node-opcua-types": "2.89.0",
+        "node-opcua-types": "2.90.0",
         "node-opcua-utils": "2.88.0",
-        "node-opcua-variant": "2.89.0",
-        "node-opcua-xml2json": "2.88.0",
+        "node-opcua-variant": "2.90.0",
+        "node-opcua-xml2json": "2.90.0",
         "semver": "^7.3.8",
         "set-prototype-of": "^1.0.0",
         "thenify": "^3.3.1",
@@ -60,11 +60,11 @@
     "devDependencies": {
         "mocha": "^10.2.0",
         "node-opcua-benchmarker": "2.88.0",
-        "node-opcua-leak-detector": "2.88.0",
+        "node-opcua-leak-detector": "2.90.0",
         "node-opcua-nodesets": "2.88.0",
-        "node-opcua-packet-analyzer": "2.89.0",
-        "node-opcua-service-filter": "2.89.0",
-        "node-opcua-test-fixtures": "2.89.0",
+        "node-opcua-packet-analyzer": "2.90.0",
+        "node-opcua-service-filter": "2.90.0",
+        "node-opcua-test-fixtures": "2.90.0",
         "should": "^13.2.3",
         "sinon": "^15.0.1",
         "source-map-support": "^0.5.21"
@@ -84,5 +84,5 @@
         "internet of things"
     ],
     "homepage": "http://node-opcua.github.io/",
-    "gitHead": "93d5502a1b71afbb5e5fa56846615c412b63cc1b"
+    "gitHead": "d55f9150c3ffa5b86fbe1a0baba69a2543c3090a"
 }

package/src/base_node_private.ts

@@ -418,7 +418,7 @@ export function VariableOrVariableType_toString(this: UAVariableType | UAVariabl
         options.add(options.padding + chalk.yellow("          dataType            : ") + this.dataType + "  " + n);
     }
     if (this.nodeClass === NodeClass.Variable) {
-        const _dataValue = (<WithDataValue>this).$dataValue as DataValue | undefined;
+        const _dataValue = (<WithDataValue>this).$dataValue;
         if (_dataValue) {
             options.add(
                 options.padding +
@@ -579,9 +579,9 @@ function _extractInterfaces2(typeDefinitionNode: UAObjectType | UAVariableType,
     //         MachineToolIdentificationType
     //
     //
-    // IMachineTagNameplateType            -subTypeOf-> ITagNameplateType
-    // IMachineVendorNamePlateType         -subTypeOf-> IMachineryItemVendorNamePlateType
-    // IMachineryItemVendorNamePlateType   -subTypeOf-> IVendorNameplateType
+    // IMachineTagNameplateType            -subtypeOf-> ITagNameplateType
+    // IMachineVendorNamePlateType         -subtypeOf-> IMachineryItemVendorNamePlateType
+    // IMachineryItemVendorNamePlateType   -subtypeOf-> IVendorNameplateType
     const interfacesRef = typeDefinitionNode.findReferencesEx("HasInterface", BrowseDirection.Forward);
     const interfaces = interfacesRef.map((r) => addressSpace.findNode(r.nodeId) as UAInterface);
 
@@ -662,9 +662,9 @@ function _crap_extractInterfaces(typeDefinitionNode: UAObjectType | UAVariableTy
     //         MachineToolIdentificationType
     //
     //
-    // IMachineTagNameplateType            -subTypeOf-> ITagNameplateType
-    // IMachineVendorNamePlateType         -subTypeOf-> IMachineryItemVendorNamePlateType
-    // IMachineryItemVendorNamePlateType   -subTypeOf-> IVendorNameplateType
+    // IMachineTagNameplateType            -subtypeOf-> ITagNameplateType
+    // IMachineVendorNamePlateType         -subtypeOf-> IMachineryItemVendorNamePlateType
+    // IMachineryItemVendorNamePlateType   -subtypeOf-> IVendorNameplateType
     const interfacesRef = typeDefinitionNode.findReferencesEx("HasInterface", BrowseDirection.Forward);
     const interfaces = interfacesRef.map((r) => r.node! as UAInterface);
     for (const iface of interfaces) {

package/src/extension_object_array_node.ts

@@ -43,6 +43,10 @@ function removeElementByIndex<T extends ExtensionObject>(uaArrayVariableNode: UA
 
     // remove element from global array (inefficient)
     uaArrayVariableNode.$$extensionObjectArray.splice(elementIndex, 1);
+    if(uaArrayVariableNode.$$extensionObjectArray !== uaArrayVariableNode.$dataValue.value.value) {
+    //    throw new Error("internal error");
+    }
+    uaArrayVariableNode.touchValue();
 
     // remove matching component
     const node = uaArrayVariableNode.getComponentByName(browseName);
@@ -170,12 +174,13 @@ export function bindExtObjArrayNode<T extends ExtensionObject>(
     // verify that an object with same doesn't already exist
     dataType = addressSpace.findDataType(variableType.dataType)! as UADataType;
     assert(dataType!.isSupertypeOf(structure), "expecting a structure (= ExtensionObject) here ");
-
+    assert(!uaArrayVariableNode.$$extensionObjectArray, "UAVariable ExtensionObject array already bounded");
     uaArrayVariableNode.$$dataType = dataType;
     uaArrayVariableNode.$$extensionObjectArray = [];
     uaArrayVariableNode.$$indexPropertyName = indexPropertyName;
-
     uaArrayVariableNode.$$getElementBrowseName = _getElementBrowseName;
+    uaArrayVariableNode.$dataValue.value.value =  uaArrayVariableNode.$$extensionObjectArray;
+    uaArrayVariableNode.$dataValue.value.arrayType = VariantArrayType.Array;
 
     const bindOptions: any = {
         get: getExtObjArrayNodeValue,
@@ -246,8 +251,12 @@ export function addElement<T extends ExtensionObject>(
         elVar.bindExtensionObject(extensionObject, { force: true });
     }
 
+    if(uaArrayVariableNode.$$extensionObjectArray !== uaArrayVariableNode.$dataValue.value.value) {
+    //    throw new Error("internal error");
+    }
     // also add the value inside
     uaArrayVariableNode.$$extensionObjectArray.push(elVar.$extensionObject);
+    uaArrayVariableNode.touchValue();
 
     return elVar;
 }

package/src/namespace_impl.ts

@@ -579,10 +579,11 @@ export class NamespaceImpl implements NamespacePrivate {
     public addReferenceType(options: AddReferenceTypeOptions): UAReferenceType {
         const addressSpace = this.addressSpace;
 
-        const options1 = options as any;
+        const options1 = options as CreateNodeOptions;
         options1.nodeClass = NodeClass.ReferenceType;
-        options1.references = options1.references || [];
-
+        options1.references = options.references || [];
+        options1.nodeId = options.nodeId;
+        
         if (options.subtypeOf) {
             const subtypeOfNodeId = addressSpace._coerceType(options.subtypeOf, "References", NodeClass.ReferenceType);
 

package/src/nodeset_tools/nodeset_to_xml.ts

@@ -1273,9 +1273,14 @@ function dumpReferenceType(xw: XmlWriter, referenceType: UAReferenceType) {
 
     dumpCommonAttributes(xw, referenceType);
 
+    const isSymmetric = (!referenceType.inverseName || referenceType.inverseName?.text === referenceType.browseName?.name);
+    if (isSymmetric) {
+        xw.writeAttribute("Symmetric", "true");
+    }
+
     dumpCommonElements(xw, referenceType);
 
-    if (referenceType.inverseName /* LocalizedText*/) {
+    if (!isSymmetric) {
         xw.startElement("InverseName");
         xw.text(referenceType.inverseName!.text || "");
         xw.endElement();
@@ -1389,9 +1394,9 @@ NamespaceImpl.prototype.toNodeset2XML = function (this: NamespaceImpl) {
     xw.startElement("NamespaceUris");
 
     // let's sort the dependencies in the same order as the translation table
-    const sortedDependencies = dependency.sort((a,b)=> translationTable[a.index] > translationTable[b.index] ? 1: -1);
-    
-    doDebug && console.log(sortedDependencies.map((a)=>a.index + " + "+ a.namespaceUri).join("\n"));
+    const sortedDependencies = dependency.sort((a, b) => translationTable[a.index] > translationTable[b.index] ? 1 : -1);
+
+    doDebug && console.log(sortedDependencies.map((a) => a.index + " + " + a.namespaceUri).join("\n"));
     doDebug && console.log("translation table ", translationTable);
 
     for (const depend of sortedDependencies) {

package/src/ua_variable_impl.ts

@@ -86,6 +86,8 @@ import { apply_condition_refresh, ConditionRefreshCache } from "./apply_conditio
 import {
     extractPartialData,
     incrementElement,
+    propagateTouchValueDownward,
+    propagateTouchValueDownwardArray,
     propagateTouchValueUpward,
     setExtensionObjectPartialValue,
     _bindExtensionObject,
@@ -524,6 +526,7 @@ export class UAVariableImpl extends BaseNodeImpl implements UAVariable {
 
     public isExtensionObject(): boolean {
         // DataType must be one of Structure
+        if (this.dataType.isEmpty()) return false;
         const dataTypeNode = this.addressSpace.findDataType(this.dataType) as UADataType;
         if (!dataTypeNode) {
             throw new Error(" Cannot find  DataType  " + this.dataType.toString() + " in standard address Space");
@@ -572,9 +575,9 @@ export class UAVariableImpl extends BaseNodeImpl implements UAVariable {
             dataValue.serverPicoseconds = 0;
             return callback(null, dataValue);
         }
-
         try {
             this.refreshFunc.call(this, (err: Error | null, dataValue?: DataValueLike) => {
+                // istanbul ignore next
                 if (err || !dataValue) {
                     errorLog(
                         "-------------- refresh call failed",
@@ -819,7 +822,6 @@ export class UAVariableImpl extends BaseNodeImpl implements UAVariable {
                     errorLog(dataValue.toString());
                     this.checkExtensionObjectIsCorrect(dataValue.value.value);
                 }
-                this.$dataValue = dataValue;
                 // ----------------------------------
                 if (this.$extensionObject) {
                     // we have an extension object already bound to this node
@@ -827,6 +829,8 @@ export class UAVariableImpl extends BaseNodeImpl implements UAVariable {
                     // const ext = dataValue.value.value;
                     this._internal_set_dataValue(dataValue);
                     return;
+                } else {
+                    this.$dataValue = dataValue;
                 }
             } else {
                 this._internal_set_dataValue(dataValue);
@@ -1283,6 +1287,7 @@ export class UAVariableImpl extends BaseNodeImpl implements UAVariable {
             const n = callbacks.length;
             for (const callback1 of callbacks) {
                 callback1.call(this, err, dataValue);
+
             }
         };
 
@@ -1712,7 +1717,7 @@ export class UAVariableImpl extends BaseNodeImpl implements UAVariable {
                 throw new Error("Invalid Extension Object on nodeId =" + this.nodeId.toString());
             }
         }
-   
+
         this.verifyVariantCompatibility(dataValue.value);
 
         this._inner_replace_dataValue(dataValue, indexRange);
@@ -1722,23 +1727,54 @@ export class UAVariableImpl extends BaseNodeImpl implements UAVariable {
      * @private
      */
     public _inner_replace_dataValue(dataValue: DataValue, indexRange?: NumericRange | null) {
-        const old_dataValue = this.$dataValue;
 
-        this.$dataValue = dataValue;
-        this.$dataValue.statusCode = this.$dataValue.statusCode || StatusCodes.Good;
+        assert(this.$dataValue.value instanceof Variant);
+        const old_dataValue = this.$dataValue.clone();
+
+        if (this.$$extensionObjectArray && dataValue.value.arrayType !== VariantArrayType.Scalar) {
+            // we have a bounded array or matrix
+            assert(Array.isArray(dataValue.value.value));
+            if (this.$$extensionObjectArray !== this.$dataValue.value.value) {
+                throw new Error("internal error");
+            }
+            this.$$extensionObjectArray = dataValue.value.value;
+            this.$dataValue.value.value = dataValue.value.value;
+
+            this.$dataValue.statusCode = dataValue.statusCode || StatusCodes.Good;
+            this.$dataValue.serverTimestamp = dataValue.serverTimestamp;
+            this.$dataValue.serverPicoseconds = dataValue.serverPicoseconds;
+            this.$dataValue.sourceTimestamp = dataValue.sourceTimestamp;
+            this.$dataValue.sourcePicoseconds = dataValue.sourcePicoseconds;
 
+        } else {
+            this.$dataValue = dataValue;
+            this.$dataValue.statusCode = this.$dataValue.statusCode || StatusCodes.Good;
+        }
         // repair missing timestamps
+        const now = new Date();
         if (!dataValue.serverTimestamp) {
-            this.$dataValue.serverTimestamp = old_dataValue.serverTimestamp;
-            this.$dataValue.serverPicoseconds = old_dataValue.serverPicoseconds;
+            this.$dataValue.serverTimestamp = old_dataValue.serverTimestamp || now;
+            this.$dataValue.serverPicoseconds = old_dataValue.serverPicoseconds || 0;
         }
         if (!dataValue.sourceTimestamp) {
-            this.$dataValue.sourceTimestamp = old_dataValue.sourceTimestamp;
-            this.$dataValue.sourcePicoseconds = old_dataValue.sourcePicoseconds;
+            this.$dataValue.sourceTimestamp = old_dataValue.sourceTimestamp || now;
+            this.$dataValue.sourcePicoseconds = old_dataValue.sourcePicoseconds || 0;
         }
 
         if (!sameDataValue(old_dataValue, dataValue)) {
-            this.emit("value_changed", this.$dataValue, indexRange);
+            if (this.getBasicDataType() === DataType.ExtensionObject) {
+                const preciseClock = coerceClock(this.$dataValue.sourceTimestamp, this.$dataValue.sourcePicoseconds);
+                const cache: Set<UAVariable> = new Set();
+                if (this.$$extensionObjectArray) {
+                    this.touchValue(preciseClock);
+                    propagateTouchValueDownwardArray(this, preciseClock, cache);
+                } else {
+                    this.touchValue(preciseClock);
+                    propagateTouchValueDownward(this, preciseClock, cache);
+                }
+            } else {
+                this.emit("value_changed", this.$dataValue, indexRange);
+            }
         }
     }
 
@@ -2067,7 +2103,7 @@ function _Variable_bind_with_simple_get(this: UAVariableImpl, options: GetterOpt
     this._get_func = options.get;
 
     const timestamped_get_func_from__Variable_bind_with_simple_get = () => {
-        const value = this._get_func();
+        const value: Variant | StatusCode = this._get_func();
 
         /* istanbul ignore next */
         if (!is_Variant_or_StatusCode(value)) {
@@ -2081,10 +2117,10 @@ function _Variable_bind_with_simple_get(this: UAVariableImpl, options: GetterOpt
             );
         }
         if (is_StatusCode(value)) {
-            return new DataValue({ statusCode: value });
+            return new DataValue({ statusCode: value as StatusCode });
         } else {
-            if (!this.$dataValue || !this.$dataValue.statusCode.isGoodish() || !sameVariant(this.$dataValue.value, value)) {
-                this.setValueFromSource(value, StatusCodes.Good);
+            if (!this.$dataValue || !this.$dataValue.statusCode.isGoodish() || !sameVariant(this.$dataValue.value, value as Variant)) {
+                this._inner_replace_dataValue(new DataValue({ value }));
             }
             return this.$dataValue;
         }

package/src/ua_variable_impl_ext_obj.ts

@@ -104,12 +104,12 @@ export function _touchValue(property: UAVariableImpl, now: PreciseClock): void {
     property.$dataValue.serverTimestamp = now.timestamp;
     property.$dataValue.serverPicoseconds = now.picoseconds;
     property.$dataValue.statusCode = StatusCodes.Good;
-    if (property.minimumSamplingInterval === 0) {
-        if (property.listenerCount("value_changed") > 0) {
-            const clonedDataValue = property.readValue();
-            property.emit("value_changed", clonedDataValue);
-        }
+    // if (property.minimumSamplingInterval === 0) {
+    if (property.listenerCount("value_changed") > 0) {
+        const clonedDataValue = property.readValue();
+        property.emit("value_changed", clonedDataValue);
     }
+    // }
 }
 
 export function propagateTouchValueUpward(self: UAVariableImpl, now: PreciseClock, cache?: Set<UAVariable>): void {
@@ -126,7 +126,7 @@ export function propagateTouchValueUpward(self: UAVariableImpl, now: PreciseCloc
     }
 }
 
-function propagateTouchValueDownward(self: UAVariableImpl, now: PreciseClock, cache?: Set<UAVariable>): void {
+export function propagateTouchValueDownward(self: UAVariableImpl, now: PreciseClock, cache?: Set<UAVariable>): void {
     if (!self.isExtensionObject()) return;
     // also propagate changes to embeded variables
     const dataTypeNode = self.getDataTypeNode();
@@ -336,7 +336,7 @@ function _installFields2(uaVariable: UAVariableImpl, { get, set }: {
         propertyNode.$dataValue.serverPicoseconds = uaVariable.$dataValue.serverPicoseconds;
         propertyNode.$dataValue.value.dataType = propertyNode.dataTypeObj.basicDataType;
         propertyNode.$dataValue.value.arrayType = propertyNode.valueRank === -1 ? VariantArrayType.Scalar : (propertyNode.valueRank === 1 ? VariantArrayType.Array : VariantArrayType.Matrix)
-        propertyNode.$dataValue.value.dimensions =propertyNode.valueRank > 1 ?  propertyNode.arrayDimensions : null;
+        propertyNode.$dataValue.value.dimensions = propertyNode.valueRank > 1 ? propertyNode.arrayDimensions : null;
 
         const fieldName = field.name!;
         installDataValueGetter(propertyNode, () => get(fieldName));
@@ -398,7 +398,7 @@ function isVariableContainingExtensionObject(uaVariable: UAVariableImpl): boolea
 function _innerBindExtensionObjectScalar(uaVariable: UAVariableImpl,
     { get, set }: {
         get: () => ExtensionObject;
-        set: (value: ExtensionObject, sourceTimestamp: PreciseClock) => void;
+        set: (value: ExtensionObject, sourceTimestamp: PreciseClock, cache: Set<UAVariableImpl>) => void;
     },
     options?: BindExtensionObjectOptions
 ) {
@@ -417,7 +417,8 @@ function _innerBindExtensionObjectScalar(uaVariable: UAVariableImpl,
         /** */
         const ext = dataValue.value.value;
         const sourceTime = coerceClock(dataValue.sourceTimestamp, dataValue.sourcePicoseconds);
-        set(ext, sourceTime);
+        const cache = new Set<UAVariableImpl>();
+        set(ext, sourceTime, cache);
     }
 
     _installFields2(uaVariable, {
@@ -607,6 +608,21 @@ export function _bindExtensionObjectArrayOrMatrix(
     uaVariable.$dataValue.value.dataType = DataType.ExtensionObject;
     uaVariable.$dataValue.value.value = uaVariable.$$extensionObjectArray;
 
+
+    // make sure uaVariable.$dataValue cannot be inadvertantly changed from this point onward
+    const $dataValue = uaVariable.$dataValue;
+    Object.defineProperty(uaVariable, "$dataValue", {
+        get(): DataValue {
+            return $dataValue;
+        },
+        set() {
+            throw new Error("$dataValue is now sealed , you should not change internal $dataValue!");
+        },
+        //    writable: true,
+        enumerable: true,
+        configurable: true,
+    });
+
     uaVariable.bindVariable({
         get: () => uaVariable.$dataValue.value
     }, true);
@@ -634,7 +650,6 @@ export function _bindExtensionObjectArrayOrMatrix(
             }) as UAVariableImpl;
         }
 
-        uaElement.$dataValue.value.dataType = DataType.ExtensionObject;
         uaElement.$dataValue.statusCode = StatusCodes.Good;
         uaElement.$dataValue.sourceTimestamp = uaVariable.$dataValue.sourceTimestamp;
         uaElement.$dataValue.sourcePicoseconds = uaVariable.$dataValue.sourcePicoseconds;
@@ -645,16 +660,21 @@ export function _bindExtensionObjectArrayOrMatrix(
 
         {
             const capturedIndex = i;
+            const capturedUaElement = uaElement as UAVariableImpl;
             _innerBindExtensionObjectScalar(uaElement,
                 {
                     get: () => uaVariable.$$extensionObjectArray[capturedIndex],
-                    set: (newValue: ExtensionObject, sourceTimestamp: PreciseClock) => {
-                        uaVariable.$$extensionObjectArray[capturedIndex] = newValue;
-                        uaVariable.touchValue();
-                        propagateTouchValueDownward(uaVariable, sourceTimestamp);
-                        propagateTouchValueUpward(uaVariable, sourceTimestamp);
-                    },
+                    set: (newValue: ExtensionObject, sourceTimestamp: PreciseClock, cache: Set<UAVariableImpl>) => {
 
+                        assert(!isProxy(uaVariable.$$extensionObjectArray[capturedIndex]));
+                        uaVariable.$$extensionObjectArray[capturedIndex] = newValue;
+                        if (uaVariable.$$extensionObjectArray !== uaVariable.$dataValue.value.value) {
+                            console.log("uaVariable", uaVariable.nodeId.toString());
+                            console.log("Houston! We have a problem ");
+                        }
+                        propagateTouchValueDownward(capturedUaElement, sourceTimestamp, cache);
+                        propagateTouchValueUpward(capturedUaElement, sourceTimestamp, cache);
+                    }
                 }, { ...options, force: true });
 
         }
@@ -717,3 +737,23 @@ export function extractPartialData(path: string | string[], extensionObject: Ext
     c1[name] = c2[name];
     return partialData;
 }
+
+export function propagateTouchValueDownwardArray(uaVariable: UAVariableImpl, now: PreciseClock, cache: Set<UAVariable>) {
+
+    if (!uaVariable.$$extensionObjectArray) return;
+    const arrayDimensions = uaVariable.arrayDimensions || [];
+    const totalLength = uaVariable.$$extensionObjectArray.length;
+
+    const indexIterator = new IndexIterator(arrayDimensions);
+    for (let i = 0; i < totalLength; i++) {
+
+        const index = indexIterator.next();
+
+        const { browseName, nodeId } = composeBrowseNameAndNodeId(uaVariable, index);
+        const uaElement = uaVariable.getComponentByName(browseName) as UAVariableImpl | null;
+        if (uaElement?.nodeClass === NodeClass.Variable) {
+            uaElement.touchValue(now);
+            propagateTouchValueDownward(uaElement, now, cache);
+        }
+    }
+}