node-opcua-address-space 2.167.0 → 2.168.0

package/dist/src/namespace_impl.js

@@ -15,17 +15,14 @@ const node_opcua_assert_1 = require("node-opcua-assert");
 const node_opcua_basic_types_1 = require("node-opcua-basic-types");
 const node_opcua_data_access_1 = require("node-opcua-data-access");
 const node_opcua_data_model_1 = require("node-opcua-data-model");
-const node_opcua_data_model_2 = require("node-opcua-data-model");
-const node_opcua_data_model_3 = require("node-opcua-data-model");
-const node_opcua_data_model_4 = require("node-opcua-data-model");
 const node_opcua_debug_1 = require("node-opcua-debug");
 const node_opcua_nodeid_1 = require("node-opcua-nodeid");
-const node_opcua_nodeid_2 = require("node-opcua-nodeid");
 const node_opcua_status_code_1 = require("node-opcua-status-code");
 const node_opcua_types_1 = require("node-opcua-types");
 const node_opcua_utils_1 = require("node-opcua-utils");
 const node_opcua_variant_1 = require("node-opcua-variant");
 const address_space_change_event_tools_1 = require("./address_space_change_event_tools");
+const alarms_and_conditions_1 = require("./alarms_and_conditions");
 const ua_condition_impl_1 = require("./alarms_and_conditions/ua_condition_impl");
 const ua_discrete_alarm_impl_1 = require("./alarms_and_conditions/ua_discrete_alarm_impl");
 const ua_exclusive_deviation_alarm_impl_1 = require("./alarms_and_conditions/ua_exclusive_deviation_alarm_impl");
@@ -33,27 +30,26 @@ const ua_exclusive_limit_alarm_impl_1 = require("./alarms_and_conditions/ua_excl
 const ua_limit_alarm_impl_1 = require("./alarms_and_conditions/ua_limit_alarm_impl");
 const ua_non_exclusive_deviation_alarm_impl_1 = require("./alarms_and_conditions/ua_non_exclusive_deviation_alarm_impl");
 const ua_non_exclusive_limit_alarm_impl_1 = require("./alarms_and_conditions/ua_non_exclusive_limit_alarm_impl");
-const alarms_and_conditions_1 = require("./alarms_and_conditions");
 const ua_off_normal_alarm_impl_1 = require("./alarms_and_conditions/ua_off_normal_alarm_impl");
+const base_node_impl_1 = require("./base_node_impl");
 const add_dataItem_stuff_1 = require("./data_access/add_dataItem_stuff");
 const ua_multistate_discrete_impl_1 = require("./data_access/ua_multistate_discrete_impl");
-// state machine
-const ua_two_state_variable_1 = require("./state_machine/ua_two_state_variable");
+const ua_multistate_value_discrete_impl_1 = require("./data_access/ua_multistate_value_discrete_impl");
+const ua_two_state_discrete_impl_1 = require("./data_access/ua_two_state_discrete_impl");
 //
 const namespace_private_1 = require("./namespace_private");
-const base_node_impl_1 = require("./base_node_impl");
-const ua_variable_impl_1 = require("./ua_variable_impl");
 const nodeid_manager_1 = require("./nodeid_manager");
-const ua_two_state_discrete_impl_1 = require("./data_access/ua_two_state_discrete_impl");
 const role_permissions_1 = require("./role_permissions");
-const ua_object_impl_1 = require("./ua_object_impl");
+// state machine
+const ua_two_state_variable_1 = require("./state_machine/ua_two_state_variable");
 const ua_data_type_impl_1 = require("./ua_data_type_impl");
-const ua_object_type_impl_1 = require("./ua_object_type_impl");
 const ua_method_impl_1 = require("./ua_method_impl");
-const ua_variable_type_impl_1 = require("./ua_variable_type_impl");
+const ua_object_impl_1 = require("./ua_object_impl");
+const ua_object_type_impl_1 = require("./ua_object_type_impl");
 const ua_reference_type_impl_1 = require("./ua_reference_type_impl");
+const ua_variable_impl_1 = require("./ua_variable_impl");
+const ua_variable_type_impl_1 = require("./ua_variable_type_impl");
 const ua_view_impl_1 = require("./ua_view_impl");
-const ua_multistate_value_discrete_impl_1 = require("./data_access/ua_multistate_value_discrete_impl");
 function _makeHashKey(nodeId) {
     switch (nodeId.identifierType) {
         case node_opcua_nodeid_1.NodeIdType.STRING:
@@ -76,16 +72,16 @@ const regExp1 = /^(s|i|b|g)=/;
 const regExpNamespaceDotBrowseName = /^[0-9]+:(.*)/;
 function detachNode(node) {
     const addressSpace = node.addressSpace;
-    const nonHierarchicalReferences = node.findReferencesEx("NonHierarchicalReferences", node_opcua_data_model_3.BrowseDirection.Inverse);
+    const nonHierarchicalReferences = node.findReferencesEx("NonHierarchicalReferences", node_opcua_data_model_1.BrowseDirection.Inverse);
     for (const ref of nonHierarchicalReferences) {
         (0, node_opcua_assert_1.assert)(!ref.isForward);
-        ref.node.removeReference({
+        ref.node?.removeReference({
             isForward: !ref.isForward,
             nodeId: node.nodeId,
             referenceType: ref.referenceType
         });
     }
-    const nonHierarchicalReferencesF = node.findReferencesEx("NonHierarchicalReferences", node_opcua_data_model_3.BrowseDirection.Forward);
+    const nonHierarchicalReferencesF = node.findReferencesEx("NonHierarchicalReferences", node_opcua_data_model_1.BrowseDirection.Forward);
     for (const ref of nonHierarchicalReferencesF) {
         if (!ref.node) {
             // could be a special case of a frequently use target node such as ModellingRule_Mandatory that do not back trace
@@ -93,14 +89,14 @@ function detachNode(node) {
             continue;
         }
         (0, node_opcua_assert_1.assert)(ref.isForward);
-        ref.node.removeReference({
+        ref.node?.removeReference({
             isForward: !ref.isForward,
             nodeId: node.nodeId,
             referenceType: ref.referenceType
         });
     }
     // remove reversed Hierarchical references
-    const hierarchicalReferences = node.findReferencesEx("HierarchicalReferences", node_opcua_data_model_3.BrowseDirection.Inverse);
+    const hierarchicalReferences = node.findReferencesEx("HierarchicalReferences", node_opcua_data_model_1.BrowseDirection.Inverse);
     for (const ref of hierarchicalReferences) {
         (0, node_opcua_assert_1.assert)(!ref.isForward);
         const parent = addressSpace.findNode(ref.nodeId);
@@ -112,6 +108,21 @@ function detachNode(node) {
     }
     node.unpropagate_back_references();
 }
+function toNodeId(nodeId) {
+    if (!nodeId) {
+        return undefined;
+    }
+    if (nodeId instanceof node_opcua_nodeid_1.NodeId) {
+        return nodeId;
+    }
+    if (typeof nodeId === "string") {
+        return (0, node_opcua_nodeid_1.coerceNodeId)(nodeId);
+    }
+    if (typeof nodeId === "number") {
+        return (0, node_opcua_nodeid_1.coerceNodeId)(nodeId);
+    }
+    return nodeId.nodeId;
+}
 /**
  *
  * @constructor
@@ -147,7 +158,7 @@ class NamespaceImpl {
     constructor(options) {
         // c8 ignore next
         if (!(typeof options.namespaceUri === "string")) {
-            throw new Error("NamespaceImpl constructor: namespaceUri must exists and be a string : got " + options.namespaceUri);
+            throw new Error(`NamespaceImpl constructor: namespaceUri must exists and be a string : got ${options.namespaceUri}`);
         }
         // c8 ignore next
         if (typeof options.index !== "number") {
@@ -223,7 +234,7 @@ class NamespaceImpl {
     findNode(nodeId) {
         if (typeof nodeId === "string") {
             if (nodeId.match(regExp1)) {
-                nodeId = "ns=" + this.index + ";" + nodeId;
+                nodeId = `ns=${this.index};${nodeId}`;
             }
         }
         nodeId = (0, node_opcua_nodeid_1.resolveNodeId)(nodeId);
@@ -270,7 +281,7 @@ class NamespaceImpl {
     findReferenceTypeFromInverseName(inverseName) {
         (0, node_opcua_assert_1.assert)(typeof inverseName === "string");
         const node = this._referenceTypeMapInv.get(inverseName);
-        (0, node_opcua_assert_1.assert)(!node || (node.nodeClass === node_opcua_data_model_4.NodeClass.ReferenceType && node.inverseName.text === inverseName));
+        (0, node_opcua_assert_1.assert)(!node || (node.nodeClass === node_opcua_data_model_1.NodeClass.ReferenceType && node.inverseName.text === inverseName));
         return node ? node : null;
     }
     /**
@@ -281,7 +292,7 @@ class NamespaceImpl {
      */
     addAlias(alias_name, nodeId) {
         (0, node_opcua_assert_1.assert)(typeof alias_name === "string");
-        (0, node_opcua_assert_1.assert)(nodeId instanceof node_opcua_nodeid_2.NodeId);
+        (0, node_opcua_assert_1.assert)(nodeId instanceof node_opcua_nodeid_1.NodeId);
         (0, node_opcua_assert_1.assert)(nodeId.namespace === this.index);
         this._aliases.set(alias_name, nodeId);
     }
@@ -303,10 +314,10 @@ class NamespaceImpl {
      *
      */
     addObjectType(options) {
-        (0, node_opcua_assert_1.assert)(!Object.prototype.hasOwnProperty.call(options, "dataType"), "an objectType should not have a dataType");
-        (0, node_opcua_assert_1.assert)(!Object.prototype.hasOwnProperty.call(options, "valueRank"), "an objectType should not have a valueRank");
-        (0, node_opcua_assert_1.assert)(!Object.prototype.hasOwnProperty.call(options, "arrayDimensions"), "an objectType should not have a arrayDimensions");
-        return this._addObjectOrVariableType(options, "BaseObjectType", node_opcua_data_model_4.NodeClass.ObjectType);
+        (0, node_opcua_assert_1.assert)(!Object.hasOwn(options, "dataType"), "an objectType should not have a dataType");
+        (0, node_opcua_assert_1.assert)(!Object.hasOwn(options, "valueRank"), "an objectType should not have a valueRank");
+        (0, node_opcua_assert_1.assert)(!Object.hasOwn(options, "arrayDimensions"), "an objectType should not have a arrayDimensions");
+        return this._addObjectOrVariableType(options, "BaseObjectType", node_opcua_data_model_1.NodeClass.ObjectType);
     }
     /**
      * add a new Variable type to the address space
@@ -324,17 +335,17 @@ class NamespaceImpl {
      *
      */
     addVariableType(options) {
-        (0, node_opcua_assert_1.assert)(!Object.prototype.hasOwnProperty.call(options, "arrayDimension"), "Do you mean ArrayDimensions ?");
+        (0, node_opcua_assert_1.assert)(!Object.hasOwn(options, "arrayDimension"), "Do you mean ArrayDimensions ?");
         // dataType
         options.dataType = options.dataType || "Int32";
         options.dataType = this.addressSpace._coerce_DataType(options.dataType);
         // valueRank/ arrayDimensions
         (0, node_opcua_variant_1.verifyRankAndDimensions)(options);
         // arrayDimensions
-        const variableType = this._addObjectOrVariableType(options, "BaseVariableType", node_opcua_data_model_4.NodeClass.VariableType);
+        const variableType = this._addObjectOrVariableType(options, "BaseVariableType", node_opcua_data_model_1.NodeClass.VariableType);
         variableType.dataType = options.dataType;
         variableType.valueRank = options.valueRank || 0;
-        variableType.arrayDimensions = options.arrayDimensions;
+        variableType.arrayDimensions = options.arrayDimensions || [];
         return variableType;
     }
     /**
@@ -342,7 +353,7 @@ class NamespaceImpl {
      */
     addVariable(options) {
         (0, node_opcua_assert_1.assert)(arguments.length === 1, "Invalid arguments IAddressSpace#addVariable now takes only one argument.");
-        if (Object.prototype.hasOwnProperty.call(options, "propertyOf") && options.propertyOf) {
+        if (Object.hasOwn(options, "propertyOf") && options.propertyOf) {
             (0, node_opcua_assert_1.assert)(!options.typeDefinition || options.typeDefinition === "PropertyType");
             options.typeDefinition = options.typeDefinition || "PropertyType";
         }
@@ -352,41 +363,39 @@ class NamespaceImpl {
         return this._addVariable(options);
     }
     addView(options) {
-        (0, node_opcua_assert_1.assert)(arguments.length === 1, "Namespace#addView expecting a single argument");
-        (0, node_opcua_assert_1.assert)(options);
-        (0, node_opcua_assert_1.assert)(Object.prototype.hasOwnProperty.call(options, "browseName"));
-        (0, node_opcua_assert_1.assert)(Object.prototype.hasOwnProperty.call(options, "organizedBy"));
+        (0, node_opcua_assert_1.assert)(Object.hasOwn(options, "browseName"));
+        (0, node_opcua_assert_1.assert)(Object.hasOwn(options, "organizedBy"));
         const browseName = options.browseName;
         (0, node_opcua_assert_1.assert)(typeof browseName === "string");
         const addressSpace = this.addressSpace;
-        const baseDataVariableTypeId = addressSpace.findVariableType("BaseDataVariableType").nodeId;
+        const baseDataVariableTypeId = addressSpace.findVariableType("BaseDataVariableType")?.nodeId;
         // ------------------------------------------ TypeDefinition
-        const typeDefinition = options.typeDefinition || baseDataVariableTypeId;
+        const _typeDefinition = options.typeDefinition || baseDataVariableTypeId;
         options.references = options.references || [];
         options.references.push({
             isForward: true,
-            nodeId: typeDefinition,
+            nodeId: toNodeId(_typeDefinition) || node_opcua_nodeid_1.NodeId.nullNodeId,
             referenceType: "HasTypeDefinition"
         });
         const createOptions = options;
         (0, node_opcua_assert_1.assert)(!createOptions.nodeClass);
-        createOptions.nodeClass = node_opcua_data_model_4.NodeClass.View;
+        createOptions.nodeClass = node_opcua_data_model_1.NodeClass.View;
         const view = this.createNode(createOptions);
-        (0, node_opcua_assert_1.assert)(view.nodeId instanceof node_opcua_nodeid_2.NodeId);
-        (0, node_opcua_assert_1.assert)(view.nodeClass === node_opcua_data_model_4.NodeClass.View);
+        (0, node_opcua_assert_1.assert)(view.nodeId instanceof node_opcua_nodeid_1.NodeId);
+        (0, node_opcua_assert_1.assert)(view.nodeClass === node_opcua_data_model_1.NodeClass.View);
         return view;
     }
     addObject(options1) {
         const options = options1;
-        (0, node_opcua_assert_1.assert)(!options.nodeClass || options.nodeClass === node_opcua_data_model_4.NodeClass.Object);
-        options.nodeClass = node_opcua_data_model_4.NodeClass.Object;
+        (0, node_opcua_assert_1.assert)(!options.nodeClass || options.nodeClass === node_opcua_data_model_1.NodeClass.Object);
+        options.nodeClass = node_opcua_data_model_1.NodeClass.Object;
         const typeDefinition = options.typeDefinition || "BaseObjectType";
         options.references = options.references || [];
         options.references.push({ referenceType: "HasTypeDefinition", isForward: true, nodeId: typeDefinition });
         options.eventNotifier = +(options.eventNotifier || 0);
         const obj = this.createNode(options);
         (0, node_opcua_assert_1.assert)(obj instanceof ua_object_impl_1.UAObjectImpl);
-        (0, node_opcua_assert_1.assert)(obj.nodeClass === node_opcua_data_model_4.NodeClass.Object);
+        (0, node_opcua_assert_1.assert)(obj.nodeClass === node_opcua_data_model_1.NodeClass.Object);
         return obj;
     }
     /**
@@ -407,7 +416,7 @@ class NamespaceImpl {
         (0, node_opcua_assert_1.assert)(!options.typeDefinition, "addFolder does not expect typeDefinition to be defined ");
         const typeDefinition = addressSpace._coerceTypeDefinition("FolderType");
         parentFolder = addressSpace._coerceFolder(parentFolder);
-        options.nodeClass = node_opcua_data_model_4.NodeClass.Object;
+        options.nodeClass = node_opcua_data_model_1.NodeClass.Object;
         options.references = [
             { referenceType: "HasTypeDefinition", isForward: true, nodeId: typeDefinition },
             { referenceType: "Organizes", isForward: false, nodeId: parentFolder.nodeId }
@@ -425,11 +434,11 @@ class NamespaceImpl {
     addReferenceType(options) {
         const addressSpace = this.addressSpace;
         const options1 = options;
-        options1.nodeClass = node_opcua_data_model_4.NodeClass.ReferenceType;
+        options1.nodeClass = node_opcua_data_model_1.NodeClass.ReferenceType;
         options1.references = options.references || [];
         options1.nodeId = options.nodeId;
         if (options.subtypeOf) {
-            const subtypeOfNodeId = addressSpace._coerceType(options.subtypeOf, "References", node_opcua_data_model_4.NodeClass.ReferenceType);
+            const subtypeOfNodeId = addressSpace._coerceType(options.subtypeOf, "References", node_opcua_data_model_1.NodeClass.ReferenceType);
             (0, node_opcua_assert_1.assert)(subtypeOfNodeId);
             options1.references.push({
                 isForward: false,
@@ -450,11 +459,11 @@ class NamespaceImpl {
 
      */
     createDataType(options) {
-        (0, node_opcua_assert_1.assert)(Object.prototype.hasOwnProperty.call(options, "isAbstract"), "must provide isAbstract");
-        (0, node_opcua_assert_1.assert)(!Object.prototype.hasOwnProperty.call(options, "nodeClass"));
-        (0, node_opcua_assert_1.assert)(Object.prototype.hasOwnProperty.call(options, "browseName"), "must provide a browseName");
+        (0, node_opcua_assert_1.assert)(Object.hasOwn(options, "isAbstract"), "must provide isAbstract");
+        (0, node_opcua_assert_1.assert)(!Object.hasOwn(options, "nodeClass"));
+        (0, node_opcua_assert_1.assert)(Object.hasOwn(options, "browseName"), "must provide a browseName");
         const options1 = options;
-        options1.nodeClass = node_opcua_data_model_4.NodeClass.DataType;
+        options1.nodeClass = node_opcua_data_model_1.NodeClass.DataType;
         options1.references = options.references || [];
         if (options1.references.length === 0) {
             if (!options1.subtypeOf) {
@@ -492,7 +501,7 @@ class NamespaceImpl {
         addressSpace.modelChangeTransaction(() => {
             (0, node_opcua_assert_1.assert)(isNonEmptyQualifiedName(options.browseName));
             // xx assert(Object.prototype.hasOwnProperty.call(options,"browseName") && options.browseName.length > 0);
-            (0, node_opcua_assert_1.assert)(Object.prototype.hasOwnProperty.call(options, "nodeClass"));
+            (0, node_opcua_assert_1.assert)(Object.hasOwn(options, "nodeClass"));
             options.references = addressSpace.normalizeReferenceTypes(options.references);
             const references = _copy_references(options.references);
             _handle_hierarchy_parent(addressSpace, references, options);
@@ -500,7 +509,7 @@ class NamespaceImpl {
             (0, namespace_private_1.UANamespace_process_modelling_rule)(references, options.modellingRule);
             options.references = references;
             node = this.internalCreateNode(options);
-            (0, node_opcua_assert_1.assert)(node.nodeId instanceof node_opcua_nodeid_2.NodeId);
+            (0, node_opcua_assert_1.assert)(node.nodeId instanceof node_opcua_nodeid_1.NodeId);
             node.propagate_back_references();
             node.install_extra_properties();
             _create_node_version_if_needed(node, options);
@@ -518,13 +527,13 @@ class NamespaceImpl {
      */
     deleteNode(nodeOrNodeId) {
         let node = null;
-        let nodeId = new node_opcua_nodeid_2.NodeId();
-        if (nodeOrNodeId instanceof node_opcua_nodeid_2.NodeId) {
+        let nodeId = new node_opcua_nodeid_1.NodeId();
+        if (nodeOrNodeId instanceof node_opcua_nodeid_1.NodeId) {
             nodeId = nodeOrNodeId;
             node = this.findNode(nodeId);
             // c8 ignore next
             if (!node) {
-                throw new Error(" deleteNode : cannot find node with nodeId" + nodeId.toString());
+                throw new Error(` deleteNode : cannot find node with nodeId${nodeId.toString()}`);
             }
         }
         else if (nodeOrNodeId instanceof base_node_impl_1.BaseNodeImpl) {
@@ -542,7 +551,7 @@ class NamespaceImpl {
                 throw new Error("this node doesn't belong to this namespace");
             }
             // notify parent that node is being removed
-            const hierarchicalReferences = node.findReferencesEx("HierarchicalReferences", node_opcua_data_model_3.BrowseDirection.Inverse);
+            const hierarchicalReferences = node.findReferencesEx("HierarchicalReferences", node_opcua_data_model_1.BrowseDirection.Inverse);
             for (const ref of hierarchicalReferences) {
                 (0, node_opcua_assert_1.assert)(!ref.isForward);
                 const parent = addressSpace.findNode(ref.nodeId);
@@ -555,10 +564,10 @@ class NamespaceImpl {
             }
             // recursively delete all nodes below in the hierarchy of nodes
             // TODO : a better idea would be to extract any references of type "HasChild"
-            const components = node.findReferencesEx("HasComponent", node_opcua_data_model_3.BrowseDirection.Forward);
-            const properties = node.findReferencesEx("HasProperty", node_opcua_data_model_3.BrowseDirection.Forward);
+            const components = node.findReferencesEx("HasComponent", node_opcua_data_model_1.BrowseDirection.Forward);
+            const properties = node.findReferencesEx("HasProperty", node_opcua_data_model_1.BrowseDirection.Forward);
             // TODO: shall we delete nodes pointed by "Organizes" links here ?
-            const subFolders = node.findReferencesEx("Organizes", node_opcua_data_model_3.BrowseDirection.Forward);
+            const subFolders = node.findReferencesEx("Organizes", node_opcua_data_model_1.BrowseDirection.Forward);
             for (const r of components) {
                 deleteNodePointedByReference(r);
             }
@@ -584,10 +593,10 @@ class NamespaceImpl {
             referenceTypeIds: {}
         };
         for (const referenceType of this._referenceTypeMap.values()) {
-            standardNodeIds.referenceTypeIds[referenceType.browseName.name] = referenceType.nodeId.toString();
+            standardNodeIds.referenceTypeIds[referenceType?.browseName?.name] = referenceType.nodeId.toString();
         }
         for (const objectType of this._objectTypeMap.values()) {
-            standardNodeIds.objectTypeIds[objectType.browseName.name] = objectType.nodeId.toString();
+            standardNodeIds.objectTypeIds[objectType?.browseName?.name] = objectType.nodeId.toString();
         }
         return standardNodeIds;
     }
@@ -618,7 +627,7 @@ class NamespaceImpl {
         options.subtypeOf = options.subtypeOf || "BaseEventType";
         // are eventType always abstract ?? No => Condition can be instantiated!
         // but, by default is abstract is true
-        options.isAbstract = Object.prototype.hasOwnProperty.call(options, "isAbstract") ? !!options.isAbstract : true;
+        options.isAbstract = Object.hasOwn(options, "isAbstract") ? !!options.isAbstract : true;
         return this.addObjectType(options);
     }
     // ---------------------------------------------------------------------------------------------------
@@ -676,7 +685,7 @@ class NamespaceImpl {
      */
     addAnalogDataItem(options) {
         const addressSpace = this.addressSpace;
-        (0, node_opcua_assert_1.assert)(Object.prototype.hasOwnProperty.call(options, "engineeringUnitsRange"), "expecting engineeringUnitsRange");
+        (0, node_opcua_assert_1.assert)(Object.hasOwn(options, "engineeringUnitsRange"), "expecting engineeringUnitsRange");
         const dataType = options.dataType || "Number";
         const analogItemType = addressSpace.findVariableType("AnalogItemType");
         if (!analogItemType) {
@@ -710,7 +719,7 @@ class NamespaceImpl {
         (0, node_opcua_assert_1.assert)(euRange.readValue().value.value instanceof node_opcua_types_1.Range);
         const handler = variable.handle_semantic_changed.bind(variable);
         euRange.on("value_changed", handler);
-        if (Object.prototype.hasOwnProperty.call(options, "instrumentRange")) {
+        if (Object.hasOwn(options, "instrumentRange")) {
             const instrumentRange = this.addVariable({
                 accessLevel: "CurrentRead | CurrentWrite",
                 browseName: { name: "InstrumentRange", namespaceIndex: 0 },
@@ -727,7 +736,7 @@ class NamespaceImpl {
             instrumentRange.on("value_changed", handler);
         }
         variable.acceptValueOutOfRange = options.acceptValueOutOfRange;
-        if (Object.prototype.hasOwnProperty.call(options, "engineeringUnits")) {
+        if (Object.hasOwn(options, "engineeringUnits")) {
             const engineeringUnits = new node_opcua_types_1.EUInformation(options.engineeringUnits);
             (0, node_opcua_assert_1.assert)(engineeringUnits instanceof node_opcua_types_1.EUInformation, "expecting engineering units");
             // EngineeringUnits  specifies the units for the   DataItem‟s value (e.g., degree, hertz, seconds).   The
@@ -828,26 +837,17 @@ class NamespaceImpl {
      * @param options.value
      */
     addYArrayItem(options) {
-        (0, node_opcua_assert_1.assert)(Object.prototype.hasOwnProperty.call(options, "engineeringUnitsRange"), "expecting engineeringUnitsRange");
-        (0, node_opcua_assert_1.assert)(Object.prototype.hasOwnProperty.call(options, "axisScaleType"), "expecting axisScaleType");
+        (0, node_opcua_assert_1.assert)(Object.hasOwn(options, "engineeringUnitsRange"), "expecting engineeringUnitsRange");
+        (0, node_opcua_assert_1.assert)(Object.hasOwn(options, "axisScaleType"), "expecting axisScaleType");
         (0, node_opcua_assert_1.assert)(options.xAxisDefinition !== null && typeof options.xAxisDefinition === "object", "expecting a xAxisDefinition");
         const addressSpace = this.addressSpace;
         const YArrayItemType = addressSpace.findVariableType("YArrayItemType");
         if (!YArrayItemType) {
             throw new Error("expecting YArrayItemType to be defined , check nodeset xml file");
         }
-        function toNodeId(options) {
-            if (!options) {
-                return (0, node_opcua_nodeid_1.resolveNodeId)(node_opcua_variant_1.DataType.Float);
-            }
-            if (Object.prototype.hasOwnProperty.call(options, "nodeId") || options instanceof base_node_impl_1.BaseNodeImpl) {
-                return options.nodeId;
-            }
-            return (0, node_opcua_nodeid_1.resolveNodeId)(options);
-        }
-        const dataType = toNodeId(options.dataType);
+        const dataType = toNodeId(options.dataType) || toNodeId(options.value?.dataType);
         const optionals = [];
-        if (Object.prototype.hasOwnProperty.call(options, "instrumentRange")) {
+        if (Object.hasOwn(options, "instrumentRange")) {
             optionals.push("InstrumentRange");
         }
         const variable = YArrayItemType.instantiate({
@@ -866,7 +866,7 @@ class NamespaceImpl {
             dataType: node_opcua_variant_1.DataType.ExtensionObject,
             value: new node_opcua_types_1.Range(options.engineeringUnitsRange)
         }));
-        if (Object.prototype.hasOwnProperty.call(options, "instrumentRange") && variable.instrumentRange) {
+        if (Object.hasOwn(options, "instrumentRange") && variable.instrumentRange) {
             variable.instrumentRange.setValueFromSource(new node_opcua_variant_1.Variant({
                 dataType: node_opcua_variant_1.DataType.ExtensionObject,
                 value: new node_opcua_types_1.Range(options.instrumentRange)
@@ -902,7 +902,7 @@ class NamespaceImpl {
     addMethod(parentObject, options) {
         const addressSpace = this.addressSpace;
         (0, node_opcua_assert_1.assert)(parentObject !== null && typeof parentObject === "object" && parentObject instanceof base_node_impl_1.BaseNodeImpl, "expecting a valid parent object");
-        (0, node_opcua_assert_1.assert)(Object.prototype.hasOwnProperty.call(options, "browseName"));
+        (0, node_opcua_assert_1.assert)(Object.hasOwn(options, "browseName"));
         options.componentOf = parentObject;
         const method = this._addMethod(options);
         const propertyTypeId = addressSpace._coerce_VariableTypeIds("PropertyType");
@@ -1028,18 +1028,17 @@ class NamespaceImpl {
         (0, node_opcua_assert_1.assert)(typeof options.browseName === "string");
         (0, node_opcua_assert_1.assert)(Array.isArray(options.enumeration));
         const addressSpace = this.addressSpace;
-        let definition;
         const enumerationType = addressSpace.findDataType("Enumeration");
-        (0, node_opcua_assert_1.assert)(enumerationType.nodeId instanceof node_opcua_nodeid_2.NodeId);
+        (0, node_opcua_assert_1.assert)(enumerationType.nodeId instanceof node_opcua_nodeid_1.NodeId);
         (0, node_opcua_assert_1.assert)(enumerationType instanceof ua_data_type_impl_1.UADataTypeImpl);
         const references = [{ referenceType: "HasSubtype", isForward: false, nodeId: enumerationType.nodeId }];
         const opts = {
             browseName: options.browseName,
-            definition,
+            definition: undefined,
             description: (0, node_opcua_data_model_1.coerceLocalizedText)(options.description) || null,
             displayName: options.displayName || null,
             isAbstract: false,
-            nodeClass: node_opcua_data_model_4.NodeClass.DataType,
+            nodeClass: node_opcua_data_model_1.NodeClass.DataType,
             references
         };
         const enumType = this.internalCreateNode(opts); //  as UAEnumeration;
@@ -1047,7 +1046,7 @@ class NamespaceImpl {
         if (typeof options.enumeration[0] === "string") {
             const enumeration = options.enumeration;
             // enumeration is a array of string
-            definition = enumeration.map((str, index) => (0, node_opcua_data_model_1.coerceLocalizedText)(str));
+            const definition = enumeration.map((str, _index) => (0, node_opcua_data_model_1.coerceLocalizedText)(str));
             const value = new node_opcua_variant_1.Variant({
                 arrayType: node_opcua_variant_1.VariantArrayType.Array,
                 dataType: node_opcua_variant_1.DataType.LocalizedText,
@@ -1083,7 +1082,7 @@ class NamespaceImpl {
         else {
             const enumeration = options.enumeration;
             // construct the definition object
-            definition = enumeration.map((enumItem) => {
+            const definition = enumeration.map((enumItem) => {
                 return new node_opcua_types_1.EnumValueType({
                     description: (0, node_opcua_data_model_1.coerceLocalizedText)(enumItem.description),
                     displayName: (0, node_opcua_data_model_1.coerceLocalizedText)(enumItem.displayName),
@@ -1106,7 +1105,7 @@ class NamespaceImpl {
             });
             (0, node_opcua_assert_1.assert)(enumValues.browseName.toString() === "EnumValues");
             enumType.$fullDefinition = new node_opcua_types_1.EnumDefinition({
-                fields: enumeration.map((x, index) => new node_opcua_types_1.EnumField({
+                fields: enumeration.map((x, _index) => new node_opcua_types_1.EnumField({
                     name: x.displayName.toString(),
                     description: x.description || "",
                     value: (0, node_opcua_basic_types_1.coerceInt64)(x.value)
@@ -1147,12 +1146,12 @@ class NamespaceImpl {
     // State and Transition
     // -------------------------------------------------------------------------
     /**
-    */
+     */
     addState(component, stateName, stateNumber, isInitialState) {
         const addressSpace = this.addressSpace;
         isInitialState = !!isInitialState;
         const _component = component;
-        (0, node_opcua_assert_1.assert)(_component.nodeClass === node_opcua_data_model_4.NodeClass.Object || _component.nodeClass === node_opcua_data_model_4.NodeClass.ObjectType);
+        (0, node_opcua_assert_1.assert)(_component.nodeClass === node_opcua_data_model_1.NodeClass.Object || _component.nodeClass === node_opcua_data_model_1.NodeClass.ObjectType);
         const initialStateType = addressSpace.findObjectType("InitialStateType");
         const stateType = addressSpace.findObjectType("StateType");
         let state;
@@ -1180,27 +1179,27 @@ class NamespaceImpl {
     addTransition(component, fromState, toState, transitionNumber, browseName) {
         const addressSpace = this.addressSpace;
         const _component = component;
-        (0, node_opcua_assert_1.assert)(_component.nodeClass === node_opcua_data_model_4.NodeClass.Object || _component.nodeClass === node_opcua_data_model_4.NodeClass.ObjectType);
+        (0, node_opcua_assert_1.assert)(_component.nodeClass === node_opcua_data_model_1.NodeClass.Object || _component.nodeClass === node_opcua_data_model_1.NodeClass.ObjectType);
         (0, node_opcua_assert_1.assert)(typeof fromState === "string");
         (0, node_opcua_assert_1.assert)(typeof toState === "string");
-        (0, node_opcua_assert_1.assert)(isFinite(transitionNumber));
+        (0, node_opcua_assert_1.assert)(Number.isFinite(transitionNumber));
         const fromStateNode = _component.getComponentByName(fromState);
         // c8 ignore next
         if (!fromStateNode) {
-            throw new Error("Cannot find state with name " + fromState);
+            throw new Error(`Cannot find state with name ${fromState}`);
         }
-        (0, node_opcua_assert_1.assert)(fromStateNode.browseName.name.toString() === fromState);
+        (0, node_opcua_assert_1.assert)(fromStateNode.browseName.name?.toString() === fromState);
         const toStateNode = _component.getComponentByName(toState);
         // c8 ignore next
         if (!toStateNode) {
-            throw new Error("Cannot find state with name " + toState);
+            throw new Error(`Cannot find state with name ${toState}`);
         }
-        (0, node_opcua_assert_1.assert)(toStateNode.browseName.name.toString() === toState);
+        (0, node_opcua_assert_1.assert)(toStateNode.browseName.name?.toString() === toState);
         const transitionType = addressSpace.findObjectType("TransitionType");
         if (!transitionType) {
             throw new Error("Cannot find TransitionType");
         }
-        browseName = browseName || fromState + "To" + toState; //  "Transition";
+        browseName = browseName || `${fromState}To${toState}`; //  "Transition";
         const transition = transitionType.instantiate({
             browseName,
             componentOf: _component
@@ -1291,7 +1290,7 @@ class NamespaceImpl {
     }
     _register(node) {
         (0, node_opcua_assert_1.assert)(node instanceof base_node_impl_1.BaseNodeImpl, "Expecting a instance of BaseNode in _register");
-        (0, node_opcua_assert_1.assert)(node.nodeId instanceof node_opcua_nodeid_2.NodeId, "Expecting a NodeId");
+        (0, node_opcua_assert_1.assert)(node.nodeId instanceof node_opcua_nodeid_1.NodeId, "Expecting a NodeId");
         // c8 ignore next
         if (node.nodeId.namespace !== this.index) {
             throw new Error("node must belong to this namespace : " +
@@ -1303,7 +1302,7 @@ class NamespaceImpl {
                 this.index);
         }
         (0, node_opcua_assert_1.assert)(node.nodeId.namespace === this.index, "node must belongs to this namespace");
-        (0, node_opcua_assert_1.assert)(Object.prototype.hasOwnProperty.call(node, "browseName"), "Node must have a browseName");
+        (0, node_opcua_assert_1.assert)(Object.hasOwn(node, "browseName"), "Node must have a browseName");
         const hashKey = _makeHashKey(node.nodeId);
         // c8 ignore next
         if (this._nodeid_index.has(hashKey)) {
@@ -1327,22 +1326,22 @@ class NamespaceImpl {
         }
         this._nodeid_index.set(hashKey, node);
         switch (node.nodeClass) {
-            case node_opcua_data_model_4.NodeClass.ObjectType:
+            case node_opcua_data_model_1.NodeClass.ObjectType:
                 this._registerObjectType(node);
                 break;
-            case node_opcua_data_model_4.NodeClass.VariableType:
+            case node_opcua_data_model_1.NodeClass.VariableType:
                 this._registerVariableType(node);
                 break;
-            case node_opcua_data_model_4.NodeClass.ReferenceType:
+            case node_opcua_data_model_1.NodeClass.ReferenceType:
                 this._registerReferenceType(node);
                 break;
-            case node_opcua_data_model_4.NodeClass.DataType:
+            case node_opcua_data_model_1.NodeClass.DataType:
                 this._registerDataType(node);
                 break;
-            case node_opcua_data_model_4.NodeClass.Object:
-            case node_opcua_data_model_4.NodeClass.Variable:
-            case node_opcua_data_model_4.NodeClass.Method:
-            case node_opcua_data_model_4.NodeClass.View:
+            case node_opcua_data_model_1.NodeClass.Object:
+            case node_opcua_data_model_1.NodeClass.Variable:
+            case node_opcua_data_model_1.NodeClass.Method:
+            case node_opcua_data_model_1.NodeClass.View:
                 break;
             default:
                 // tslint:disable-next-line:no-console
@@ -1360,7 +1359,7 @@ class NamespaceImpl {
         // xx assert(options.browseName instanceof QualifiedName
         // ? (options.browseName.namespaceIndex === this.index): true,
         // "Expecting browseName to have the same namespaceIndex as the namespace");
-        options.description = (0, node_opcua_data_model_1.coerceLocalizedText)(options.description);
+        options.description = (0, node_opcua_data_model_1.coerceLocalizedText)(options.description) || "";
         // browseName adjustment
         if (typeof options.browseName === "string") {
             const match = options.browseName.match(regExpNamespaceDotBrowseName);
@@ -1381,31 +1380,32 @@ class NamespaceImpl {
                         " please refer to the NodeOPCUA documentation");
                 }
             }
-            options.browseName = new node_opcua_data_model_2.QualifiedName({ name: options.browseName, namespaceIndex: this.index });
+            options.browseName = new node_opcua_data_model_1.QualifiedName({ name: options.browseName, namespaceIndex: this.index });
         }
-        else if (!(options.browseName instanceof node_opcua_data_model_2.QualifiedName)) {
-            options.browseName = new node_opcua_data_model_2.QualifiedName(options.browseName);
+        else if (!(options.browseName instanceof node_opcua_data_model_1.QualifiedName)) {
+            options.browseName = new node_opcua_data_model_1.QualifiedName(options.browseName);
         }
-        (0, node_opcua_assert_1.assert)(options.browseName instanceof node_opcua_data_model_2.QualifiedName, "Expecting options.browseName to be instanceof  QualifiedName ");
+        (0, node_opcua_assert_1.assert)(options.browseName instanceof node_opcua_data_model_1.QualifiedName, "Expecting options.browseName to be instanceof  QualifiedName ");
         // ------------- set display name
         if (!options.displayName) {
             (0, node_opcua_assert_1.assert)(typeof options.browseName.name === "string");
-            options.displayName = (0, node_opcua_data_model_1.coerceLocalizedText)(options.browseName.name);
+            options.displayName = (0, node_opcua_data_model_1.coerceLocalizedText)(options.browseName.name || "");
         }
-        if (!options.nodeClass || options.nodeClass == undefined) {
+        if (!options.nodeClass || options.nodeClass === undefined) {
             throw new Error("nodeclass must be specified");
         }
         // --- nodeId adjustment
         options.nodeId = this.constructNodeId(options);
         (0, node_opcua_debug_1.dumpIf)(!options.nodeId, options); // missing node Id
-        (0, node_opcua_assert_1.assert)(options.nodeId instanceof node_opcua_nodeid_2.NodeId);
+        (0, node_opcua_assert_1.assert)(options.nodeId instanceof node_opcua_nodeid_1.NodeId);
         // assert(options.browseName.namespaceIndex === this.index,"Expecting browseName to have
         // the same namespaceIndex as the namespace");
-        const Constructor = _constructors_map[node_opcua_data_model_4.NodeClass[options.nodeClass]];
+        const Constructor = _constructors_map[node_opcua_data_model_1.NodeClass[options.nodeClass]];
         if (!Constructor) {
-            throw new Error(" missing constructor for NodeClass " + node_opcua_data_model_4.NodeClass[options.nodeClass]);
+            throw new Error(` missing constructor for NodeClass ${node_opcua_data_model_1.NodeClass[options.nodeClass]}`);
         }
         options.addressSpace = this.addressSpace;
+        // biome-ignore lint/suspicious/noExplicitAny: to fix later
         const node = new Constructor(options);
         this._register(node);
         // object shall now be registered
@@ -1420,19 +1420,19 @@ class NamespaceImpl {
         const hashKey = _makeHashKey(node.nodeId);
         // c8 ignore next
         if (!this._nodeid_index.has(hashKey)) {
-            throw new Error("deleteNode : nodeId " + node.nodeId.displayText() + " is not registered " + node.nodeId.toString());
+            throw new Error(`deleteNode : nodeId ${node.nodeId.displayText()} is not registered ${node.nodeId.toString()}`);
         }
         switch (node.nodeClass) {
-            case node_opcua_data_model_4.NodeClass.ObjectType:
+            case node_opcua_data_model_1.NodeClass.ObjectType:
                 this._unregisterObjectType(node);
                 break;
-            case node_opcua_data_model_4.NodeClass.VariableType:
+            case node_opcua_data_model_1.NodeClass.VariableType:
                 this._unregisterVariableType(node);
                 break;
-            case node_opcua_data_model_4.NodeClass.Object:
-            case node_opcua_data_model_4.NodeClass.Variable:
-            case node_opcua_data_model_4.NodeClass.Method:
-            case node_opcua_data_model_4.NodeClass.View:
+            case node_opcua_data_model_1.NodeClass.Object:
+            case node_opcua_data_model_1.NodeClass.Variable:
+            case node_opcua_data_model_1.NodeClass.Method:
+            case node_opcua_data_model_1.NodeClass.View:
                 break;
             default:
                 // tslint:disable:no-console
@@ -1447,19 +1447,19 @@ class NamespaceImpl {
     _addObjectOrVariableType(options1, topMostBaseType, nodeClass) {
         const addressSpace = this.addressSpace;
         (0, node_opcua_assert_1.assert)(typeof topMostBaseType === "string");
-        (0, node_opcua_assert_1.assert)(nodeClass === node_opcua_data_model_4.NodeClass.ObjectType || nodeClass === node_opcua_data_model_4.NodeClass.VariableType);
+        (0, node_opcua_assert_1.assert)(nodeClass === node_opcua_data_model_1.NodeClass.ObjectType || nodeClass === node_opcua_data_model_1.NodeClass.VariableType);
         const options = options1;
         (0, node_opcua_assert_1.assert)(!options.nodeClass);
         (0, node_opcua_assert_1.assert)(options.browseName);
         (0, node_opcua_assert_1.assert)(typeof options.browseName === "string");
-        if (Object.prototype.hasOwnProperty.call(options, "references")) {
+        if (Object.hasOwn(options, "references")) {
             throw new Error("options.references should not be provided, use options.subtypeOf instead");
         }
         const references = [];
         function process_subtypeOf_options(options2, references1) {
             // check common misspelling mistake
             (0, node_opcua_assert_1.assert)(!options2.subTypeOf, "misspell error : it should be 'subtypeOf' instead");
-            if (Object.prototype.hasOwnProperty.call(options2, "hasTypeDefinition")) {
+            if (Object.hasOwn(options2, "hasTypeDefinition")) {
                 throw new Error("hasTypeDefinition option is invalid. Do you mean typeDefinition instead ?");
             }
             (0, node_opcua_assert_1.assert)(!options2.typeDefinition, " do you mean subtypeOf ?");
@@ -1506,38 +1506,38 @@ class NamespaceImpl {
     // }
     _registerObjectType(node) {
         (0, node_opcua_assert_1.assert)(this.index === node.nodeId.namespace);
-        const key = node.browseName.name;
+        const key = node.browseName.name || "";
         if (this._objectTypeMap.has(key)) {
-            throw new Error(" UAObjectType already declared " + node.browseName.toString() + "  " + node.nodeId.toString());
+            throw new Error(` UAObjectType already declared ${node.browseName.toString()}  ${node.nodeId.toString()}`);
         }
         this._objectTypeMap.set(key, node);
     }
     _registerVariableType(node) {
         (0, node_opcua_assert_1.assert)(this.index === node.nodeId.namespace);
-        const key = node.browseName.name;
+        const key = node.browseName.name || "";
         (0, node_opcua_assert_1.assert)(!this._variableTypeMap.has(key), " UAVariableType already declared");
         this._variableTypeMap.set(key, node);
     }
     _registerReferenceType(node) {
         (0, node_opcua_assert_1.assert)(this.index === node.nodeId.namespace);
-        (0, node_opcua_assert_1.assert)(node.browseName instanceof node_opcua_data_model_2.QualifiedName);
-        const key = node.browseName.name;
+        (0, node_opcua_assert_1.assert)(node.browseName instanceof node_opcua_data_model_1.QualifiedName);
+        const key = node.browseName.name || "";
         this._referenceTypeMap.set(key, node);
-        this._referenceTypeMapInv.set(node.inverseName.text, node);
+        this._referenceTypeMapInv.set(node.inverseName.text || "", node);
     }
     _registerDataType(node) {
         (0, node_opcua_assert_1.assert)(this.index === node.nodeId.namespace);
-        const key = node.browseName.name;
-        (0, node_opcua_assert_1.assert)(node.browseName instanceof node_opcua_data_model_2.QualifiedName);
+        const key = node.browseName.name || "";
+        (0, node_opcua_assert_1.assert)(node.browseName instanceof node_opcua_data_model_1.QualifiedName);
         (0, node_opcua_assert_1.assert)(!this._dataTypeMap.has(key), " DataType already declared");
         this._dataTypeMap.set(key, node);
     }
     _unregisterObjectType(node) {
-        const key = node.browseName.name;
+        const key = node.browseName.name || "";
         this._objectTypeMap.delete(key);
     }
     _unregisterVariableType(node) {
-        const key = node.browseName.name;
+        const key = node.browseName.name || "";
         this._variableTypeMap.delete(key);
     }
     /**
@@ -1550,33 +1550,33 @@ class NamespaceImpl {
             throw new Error("cannot find BaseDataVariableType");
         }
         const baseDataVariableTypeId = baseDataVariableType.nodeId;
-        (0, node_opcua_assert_1.assert)(Object.prototype.hasOwnProperty.call(options, "browseName"), "options.browseName must be provided");
-        (0, node_opcua_assert_1.assert)(Object.prototype.hasOwnProperty.call(options, "dataType"), "options.dataType must be provided");
+        (0, node_opcua_assert_1.assert)(Object.hasOwn(options, "browseName"), "options.browseName must be provided");
+        (0, node_opcua_assert_1.assert)(Object.hasOwn(options, "dataType"), "options.dataType must be provided");
         options.historizing = !!options.historizing;
         // c8 ignore next
-        if (Object.prototype.hasOwnProperty.call(options, "hasTypeDefinition")) {
+        if (Object.hasOwn(options, "hasTypeDefinition")) {
             throw new Error("hasTypeDefinition option is invalid. Do you mean typeDefinition instead ?");
         }
         // ------------------------------------------ TypeDefinition
         let typeDefinition = options.typeDefinition || baseDataVariableTypeId;
         if (typeDefinition instanceof base_node_impl_1.BaseNodeImpl) {
             // c8 ignore next
-            if (typeDefinition.nodeClass !== node_opcua_data_model_4.NodeClass.VariableType) {
-                const message = `invalid typeDefinition expecting a VariableType got ${node_opcua_data_model_4.NodeClass[typeDefinition.nodeClass]}`;
+            if (typeDefinition.nodeClass !== node_opcua_data_model_1.NodeClass.VariableType) {
+                const message = `invalid typeDefinition expecting a VariableType got ${node_opcua_data_model_1.NodeClass[typeDefinition.nodeClass]}`;
                 errorLog(message);
                 throw new Error(message);
             }
         }
         typeDefinition = addressSpace._coerce_VariableTypeIds(typeDefinition);
-        (0, node_opcua_assert_1.assert)(typeDefinition instanceof node_opcua_nodeid_2.NodeId);
+        (0, node_opcua_assert_1.assert)(typeDefinition instanceof node_opcua_nodeid_1.NodeId);
         // ------------------------------------------ DataType
-        options.dataType = addressSpace._coerce_DataType(options.dataType);
+        options.dataType = addressSpace._coerce_DataType(options.dataType || node_opcua_variant_1.DataType.Null);
         options.valueRank = (0, node_opcua_utils_1.isNullOrUndefined)(options.valueRank)
             ? options.arrayDimensions
                 ? options.arrayDimensions.length
                 : -1
             : options.valueRank;
-        (0, node_opcua_assert_1.assert)(typeof options.valueRank === "number" && isFinite(options.valueRank));
+        (0, node_opcua_assert_1.assert)(typeof options.valueRank === "number" && Number.isFinite(options.valueRank));
         options.arrayDimensions = options.arrayDimensions || null;
         (0, node_opcua_assert_1.assert)(options.arrayDimensions === null || Array.isArray(options.arrayDimensions));
         // -----------------------------------------------------
@@ -1603,8 +1603,8 @@ class NamespaceImpl {
                 referenceType: "HasTypeDefinition"
             }
         ]);
-        (0, node_opcua_assert_1.assert)(!options.nodeClass || options.nodeClass === node_opcua_data_model_4.NodeClass.Variable);
-        options.nodeClass = node_opcua_data_model_4.NodeClass.Variable;
+        (0, node_opcua_assert_1.assert)(!options.nodeClass || options.nodeClass === node_opcua_data_model_1.NodeClass.Variable);
+        options.nodeClass = node_opcua_data_model_1.NodeClass.Variable;
         options.references = references;
         const variable = this.createNode(options);
         return variable;
@@ -1625,7 +1625,7 @@ class NamespaceImpl {
             displayName: options.displayName,
             eventNotifier: +options.eventNotifier,
             isAbstract: false,
-            nodeClass: node_opcua_data_model_4.NodeClass.Method,
+            nodeClass: node_opcua_data_model_1.NodeClass.Method,
             nodeId: options.nodeId,
             references,
             rolePermissions: options.rolePermissions
@@ -1661,7 +1661,7 @@ function _coerce_parent(addressSpace, value, coerceFunc) {
         if (typeof value === "string") {
             value = coerceFunc.call(addressSpace, value);
         }
-        if (value instanceof node_opcua_nodeid_2.NodeId) {
+        if (value instanceof node_opcua_nodeid_1.NodeId) {
             value = addressSpace.findNode(value);
         }
     }
@@ -1699,7 +1699,7 @@ function _handle_hierarchy_parent(addressSpace, references, options) {
         (0, node_opcua_assert_1.assert)(!options.componentOf);
         (0, node_opcua_assert_1.assert)(!options.propertyOf);
         (0, node_opcua_assert_1.assert)(!options.organizedBy);
-        (0, node_opcua_assert_1.assert)(options.addInOf.nodeClass === node_opcua_data_model_4.NodeClass.Object || options.addInOf.nodeClass === node_opcua_data_model_4.NodeClass.ObjectType, "addInOf must be of nodeClass Object or ObjectType");
+        (0, node_opcua_assert_1.assert)(options.addInOf.nodeClass === node_opcua_data_model_1.NodeClass.Object || options.addInOf.nodeClass === node_opcua_data_model_1.NodeClass.ObjectType, "addInOf must be of nodeClass Object or ObjectType");
         references.push({
             isForward: false,
             nodeId: options.addInOf.nodeId,
@@ -1741,7 +1741,7 @@ function _handle_hierarchy_parent(addressSpace, references, options) {
     }
     if (options.encodingOf) {
         // parent must be a DataType
-        (0, node_opcua_assert_1.assert)(options.encodingOf.nodeClass === node_opcua_data_model_4.NodeClass.DataType, "encodingOf must be toward a DataType");
+        (0, node_opcua_assert_1.assert)(options.encodingOf.nodeClass === node_opcua_data_model_1.NodeClass.DataType, "encodingOf must be toward a DataType");
         references.push({
             isForward: false,
             nodeId: options.encodingOf.nodeId,
@@ -1750,10 +1750,10 @@ function _handle_hierarchy_parent(addressSpace, references, options) {
     }
 }
 function _copy_reference(reference) {
-    (0, node_opcua_assert_1.assert)(Object.prototype.hasOwnProperty.call(reference, "referenceType"));
-    (0, node_opcua_assert_1.assert)(Object.prototype.hasOwnProperty.call(reference, "isForward"));
-    (0, node_opcua_assert_1.assert)(Object.prototype.hasOwnProperty.call(reference, "nodeId"));
-    (0, node_opcua_assert_1.assert)(reference.nodeId instanceof node_opcua_nodeid_2.NodeId);
+    (0, node_opcua_assert_1.assert)(Object.hasOwn(reference, "referenceType"));
+    (0, node_opcua_assert_1.assert)(Object.hasOwn(reference, "isForward"));
+    (0, node_opcua_assert_1.assert)(Object.hasOwn(reference, "nodeId"));
+    (0, node_opcua_assert_1.assert)(reference.nodeId instanceof node_opcua_nodeid_1.NodeId);
     return {
         isForward: reference.isForward,
         nodeId: reference.nodeId,
@@ -1771,16 +1771,16 @@ function isNonEmptyQualifiedName(browseName) {
     if (typeof browseName === "string") {
         return browseName.length >= 0;
     }
-    if (!(browseName instanceof node_opcua_data_model_2.QualifiedName)) {
-        browseName = new node_opcua_data_model_2.QualifiedName(browseName);
+    if (!(browseName instanceof node_opcua_data_model_1.QualifiedName)) {
+        browseName = new node_opcua_data_model_1.QualifiedName(browseName);
     }
-    (0, node_opcua_assert_1.assert)(browseName instanceof node_opcua_data_model_2.QualifiedName);
-    return browseName.name.length > 0;
+    (0, node_opcua_assert_1.assert)(browseName instanceof node_opcua_data_model_1.QualifiedName);
+    return (browseName.name?.length || 0) > 0;
 }
 function _create_node_version_if_needed(node, options) {
     (0, node_opcua_assert_1.assert)(options);
-    if (typeof options.nodeVersion == "string") {
-        (0, node_opcua_assert_1.assert)(node.nodeClass === node_opcua_data_model_4.NodeClass.Variable || node.nodeClass === node_opcua_data_model_4.NodeClass.Object);
+    if (typeof options.nodeVersion === "string") {
+        (0, node_opcua_assert_1.assert)(node.nodeClass === node_opcua_data_model_1.NodeClass.Variable || node.nodeClass === node_opcua_data_model_1.NodeClass.Object);
         // c8 ignore next
         if (node.getNodeVersion()) {
             return; // already exists