node-opcua-address-space 2.170.1 → 2.172.0

package/dist/src/ua_variable_impl.js

@@ -10,7 +10,7 @@ exports.adjust_userAccessLevel = adjust_userAccessLevel;
 /**
  * @module node-opcua-address-space
  */
-const util_1 = require("util");
+const node_util_1 = require("node:util");
 const chalk_1 = __importDefault(require("chalk"));
 const node_opcua_address_space_base_1 = require("node-opcua-address-space-base");
 const node_opcua_assert_1 = require("node-opcua-assert");
@@ -24,17 +24,16 @@ const node_opcua_numeric_range_1 = require("node-opcua-numeric-range");
 const node_opcua_service_write_1 = require("node-opcua-service-write");
 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 session_context_1 = require("../source/session_context");
 const multiform_func_1 = require("../source/helpers/multiform_func");
+const session_context_1 = require("../source/session_context");
+const apply_condition_refresh_1 = require("./apply_condition_refresh");
 const base_node_impl_1 = require("./base_node_impl");
 const base_node_private_1 = require("./base_node_private");
-const ua_data_type_impl_1 = require("./ua_data_type_impl");
-const apply_condition_refresh_1 = require("./apply_condition_refresh");
-const ua_variable_impl_ext_obj_1 = require("./ua_variable_impl_ext_obj");
 const adjust_datavalue_status_code_1 = require("./data_access/adjust_datavalue_status_code");
 const get_basic_datatype_1 = require("./get_basic_datatype");
+const ua_data_type_impl_1 = require("./ua_data_type_impl");
+const ua_variable_impl_ext_obj_1 = require("./ua_variable_impl_ext_obj");
 const validate_data_type_correctness_1 = require("./validate_data_type_correctness");
 const debugLog = (0, node_opcua_debug_1.make_debugLog)(__filename);
 const warningLog = (0, node_opcua_debug_1.make_warningLog)(__filename);
@@ -42,10 +41,9 @@ const doDebug = (0, node_opcua_debug_1.checkDebugFlag)(__filename);
 const errorLog = (0, node_opcua_debug_1.make_errorLog)(__filename);
 const plainChalk = new Proxy(chalk_1.default, { get: () => (s) => s });
 function adjust_accessLevel(accessLevel) {
-    accessLevel = (0, node_opcua_utils_1.isNullOrUndefined)(accessLevel) ? "CurrentRead | CurrentWrite" : accessLevel;
-    accessLevel = (0, node_opcua_data_model_1.makeAccessLevelFlag)(accessLevel);
-    (0, node_opcua_assert_1.assert)(isFinite(accessLevel));
-    return accessLevel;
+    const flag = (0, node_opcua_data_model_1.makeAccessLevelFlag)(accessLevel ?? "CurrentRead | CurrentWrite");
+    (0, node_opcua_assert_1.assert)(Number.isFinite(flag));
+    return flag;
 }
 function adjust_userAccessLevel(userAccessLevel, accessLevel) {
     if (userAccessLevel === undefined) {
@@ -56,7 +54,7 @@ function adjust_userAccessLevel(userAccessLevel, accessLevel) {
     return (0, node_opcua_data_model_1.makeAccessLevelFlag)(accessLevel & userAccessLevel);
 }
 function adjust_samplingInterval(minimumSamplingInterval) {
-    (0, node_opcua_assert_1.assert)(isFinite(minimumSamplingInterval));
+    (0, node_opcua_assert_1.assert)(Number.isFinite(minimumSamplingInterval));
     if (minimumSamplingInterval < 0) {
         return -1; // only -1 is a valid negative value for samplingInterval and means "unspecified"
     }
@@ -66,8 +64,7 @@ function is_Variant(v) {
     return v instanceof node_opcua_variant_1.Variant;
 }
 function is_StatusCode(v) {
-    return (v &&
-        v.constructor &&
+    return !!(v?.constructor &&
         (v instanceof node_opcua_status_code_1.StatusCode ||
             v.constructor.name === "ConstantStatusCode" ||
             v.constructor.name === "StatusCode" ||
@@ -109,6 +106,13 @@ function default_func(dataValue1, callback1) {
  *
  */
 class UAVariableImpl extends base_node_impl_1.BaseNodeImpl {
+    // -------------- UAvaraibleImplExArray
+    $$variableType;
+    $$dataType = undefined;
+    $$getElementBrowseName;
+    $$extensionObjectArray;
+    $$indexPropertyName;
+    // --------------
     nodeClass = node_opcua_data_model_1.NodeClass.Variable;
     dataType;
     _basicDataType;
@@ -152,8 +156,8 @@ class UAVariableImpl extends base_node_impl_1.BaseNodeImpl {
     constructor(options) {
         super(options);
         (0, node_opcua_variant_1.verifyRankAndDimensions)(options);
-        this.valueRank = options.valueRank;
-        this.arrayDimensions = options.arrayDimensions;
+        this.valueRank = options.valueRank ?? -1;
+        this.arrayDimensions = options.arrayDimensions ?? null;
         this.dataType = this.resolveNodeId(options.dataType); // DataType (NodeId)
         this.accessLevel = adjust_accessLevel(options.accessLevel);
         this.userAccessLevel = adjust_userAccessLevel(options.userAccessLevel, this.accessLevel);
@@ -193,7 +197,7 @@ class UAVariableImpl extends base_node_impl_1.BaseNodeImpl {
         }
         return this.checkAccessLevelPrivate(context, accessLevel);
     }
-    isReadable(context) {
+    isReadable(_context) {
         return (this.accessLevel & node_opcua_data_model_1.AccessLevelFlag.CurrentRead) === node_opcua_data_model_1.AccessLevelFlag.CurrentRead;
     }
     isUserReadable(context) {
@@ -205,7 +209,7 @@ class UAVariableImpl extends base_node_impl_1.BaseNodeImpl {
         }
         return this.checkAccessLevelPrivate(context, node_opcua_data_model_1.AccessLevelFlag.CurrentRead);
     }
-    isWritable(context) {
+    isWritable(_context) {
         return (this.accessLevel & node_opcua_data_model_1.AccessLevelFlag.CurrentWrite) === node_opcua_data_model_1.AccessLevelFlag.CurrentWrite;
     }
     isUserWritable(context) {
@@ -281,7 +285,7 @@ class UAVariableImpl extends base_node_impl_1.BaseNodeImpl {
         if (this._timestamped_get_func) {
             if (this._timestamped_get_func.length === 0) {
                 const dataValueOrPromise = this._timestamped_get_func();
-                if (!Object.prototype.hasOwnProperty.call(dataValueOrPromise.constructor.prototype, "then")) {
+                if (!Object.hasOwn(dataValueOrPromise.constructor.prototype, "then")) {
                     if (dataValueOrPromise !== this.$dataValue) {
                         // we may have a problem here if we use a getter that returns a dataValue that is a ExtensionObject
                         if (dataValueOrPromise.value?.dataType === node_opcua_variant_1.DataType.ExtensionObject) {
@@ -335,7 +339,7 @@ class UAVariableImpl extends base_node_impl_1.BaseNodeImpl {
             return false;
         const dataTypeNode = this.addressSpace.findDataType(this.dataType);
         if (!dataTypeNode) {
-            throw new Error(" Cannot find  DataType  " + this.dataType.toString() + " in standard address Space");
+            throw new Error(` Cannot find  DataType  ${this.dataType.toString()} in standard address Space`);
         }
         const structureNode = this.addressSpace.findDataType("Structure");
         if (!structureNode) {
@@ -412,17 +416,17 @@ class UAVariableImpl extends base_node_impl_1.BaseNodeImpl {
     writeEnumValue(value) {
         const enumInfo = this._getEnumerationInfo();
         if (typeof value === "string") {
-            if (!Object.prototype.hasOwnProperty.call(enumInfo.nameIndex, value)) {
+            if (!Object.hasOwn(enumInfo.nameIndex, value)) {
                 const possibleValues = Object.keys(enumInfo.nameIndex).join(",");
-                throw new Error("UAVariable#writeEnumValue: cannot find value " + value + " in [" + possibleValues + "]");
+                throw new Error(`UAVariable#writeEnumValue: cannot find value ${value} in [${possibleValues}]`);
             }
             const valueIndex = enumInfo.nameIndex[value].value;
             value = valueIndex;
         }
-        if (isFinite(value)) {
+        if (Number.isFinite(value)) {
             const possibleValues = Object.keys(enumInfo.nameIndex).join(",");
             if (!enumInfo.valueIndex[value]) {
-                throw new Error("UAVariable#writeEnumValue : value out of range " + value + " in [" + possibleValues + "]");
+                throw new Error(`UAVariable#writeEnumValue : value out of range ${value} in [${possibleValues}]`);
             }
             this.setValueFromSource({
                 dataType: node_opcua_variant_1.DataType.Int32,
@@ -438,7 +442,7 @@ class UAVariableImpl extends base_node_impl_1.BaseNodeImpl {
         (0, node_opcua_assert_1.assert)(context instanceof session_context_1.SessionContext);
         const options = {};
         if (attributeId !== node_opcua_data_model_1.AttributeIds.Value) {
-            if (indexRange && indexRange.isDefined()) {
+            if (indexRange?.isDefined()) {
                 options.statusCode = node_opcua_status_code_1.StatusCodes.BadIndexRangeNoData;
                 return new node_opcua_data_value_1.DataValue(options);
             }
@@ -479,7 +483,7 @@ class UAVariableImpl extends base_node_impl_1.BaseNodeImpl {
     verifyVariantCompatibility(variant) {
         try {
             // c8 ignore next
-            if (Object.prototype.hasOwnProperty.call(variant, "value")) {
+            if (Object.hasOwn(variant, "value")) {
                 if (variant.dataType === null || variant.dataType === undefined) {
                     throw new Error("Variant must provide a valid dataType : variant = " +
                         variant.toString() +
@@ -639,30 +643,30 @@ class UAVariableImpl extends base_node_impl_1.BaseNodeImpl {
         indexRange = node_opcua_numeric_range_1.NumericRange.coerce(indexRange);
         // test write permission
         if (!this.isWritable(context)) {
-            return callback(null, node_opcua_status_code_1.StatusCodes.BadNotWritable);
+            return callback?.(null, node_opcua_status_code_1.StatusCodes.BadNotWritable);
         }
         if (!this.checkPermissionPrivate(context, node_opcua_types_1.PermissionType.Write)) {
             return new node_opcua_data_value_1.DataValue({ statusCode: node_opcua_status_code_1.StatusCodes.BadUserAccessDenied });
         }
         if (!this.isUserWritable(context)) {
-            return callback(null, node_opcua_status_code_1.StatusCodes.BadWriteNotSupported);
+            return callback?.(null, node_opcua_status_code_1.StatusCodes.BadWriteNotSupported);
         }
         // adjust special case
         const variant = adjustVariant2.call(this, dataValue.value);
         const statusCode = this.checkVariantCompatibility(variant);
         if (statusCode.isNot(node_opcua_status_code_1.StatusCodes.Good)) {
-            return callback(null, statusCode);
+            return callback?.(null, statusCode);
         }
         // adjust dataValue.statusCode based on InstrumentRange and EngineeringUnits
         const statusCode2 = this.adjustDataValueStatusCode(dataValue);
         if (statusCode2.isNotGood()) {
-            return callback(null, statusCode2);
+            return callback?.(null, statusCode2);
         }
         const write_func = this._timestamped_set_func || default_func;
         if (!write_func) {
-            warningLog(" warning " + this.nodeId.toString() + " " + this.browseName.toString() + " has no setter. \n");
+            warningLog(` warning ${this.nodeId.toString()} ${this.browseName.toString()} has no setter. \n`);
             warningLog("Please make sure to bind the variable or to pass a valid value: new Variant({}) during construction time");
-            return callback(null, node_opcua_status_code_1.StatusCodes.BadNotWritable);
+            return callback?.(null, node_opcua_status_code_1.StatusCodes.BadNotWritable);
         }
         (0, node_opcua_assert_1.assert)(write_func);
         write_func.call(this, dataValue, (err, statusCode1) => {
@@ -670,7 +674,7 @@ class UAVariableImpl extends base_node_impl_1.BaseNodeImpl {
                 dataValue && this.verifyVariantCompatibility(dataValue.value);
                 if (indexRange && !indexRange.isEmpty()) {
                     if (!indexRange.isValid()) {
-                        return callback(null, node_opcua_status_code_1.StatusCodes.BadIndexRangeInvalid);
+                        return callback?.(null, node_opcua_status_code_1.StatusCodes.BadIndexRangeInvalid);
                     }
                     const newArrayOrMatrix = dataValue.value.value;
                     if (dataValue.value.arrayType === node_opcua_variant_1.VariantArrayType.Array) {
@@ -682,7 +686,7 @@ class UAVariableImpl extends base_node_impl_1.BaseNodeImpl {
                         const destArr = this.$dataValue.value.value;
                         const result = indexRange.set_values(destArr, newArrayOrMatrix);
                         if (result.statusCode.isNot(node_opcua_status_code_1.StatusCodes.Good)) {
-                            return callback(null, result.statusCode);
+                            return callback?.(null, result.statusCode);
                         }
                         dataValue.value.value = result.array;
                         // scrap original array so we detect range
@@ -692,7 +696,7 @@ class UAVariableImpl extends base_node_impl_1.BaseNodeImpl {
                         const dimensions = this.$dataValue.value.dimensions;
                         if (this.$dataValue.value.arrayType !== node_opcua_variant_1.VariantArrayType.Matrix || !dimensions) {
                             // not a matrix !
-                            return callback(null, node_opcua_status_code_1.StatusCodes.BadTypeMismatch);
+                            return callback?.(null, node_opcua_status_code_1.StatusCodes.BadTypeMismatch);
                         }
                         const matrix = this.$dataValue.value.value;
                         const result = indexRange.set_values_matrix({
@@ -700,7 +704,7 @@ class UAVariableImpl extends base_node_impl_1.BaseNodeImpl {
                             dimensions
                         }, newArrayOrMatrix);
                         if (result.statusCode.isNot(node_opcua_status_code_1.StatusCodes.Good)) {
-                            return callback(null, result.statusCode);
+                            return callback?.(null, result.statusCode);
                         }
                         dataValue.value.dimensions = this.$dataValue.value.dimensions;
                         dataValue.value.value = result.matrix;
@@ -708,20 +712,20 @@ class UAVariableImpl extends base_node_impl_1.BaseNodeImpl {
                         this.$dataValue.value.value = null;
                     }
                     else {
-                        return callback(null, node_opcua_status_code_1.StatusCodes.BadTypeMismatch);
+                        return callback?.(null, node_opcua_status_code_1.StatusCodes.BadTypeMismatch);
                     }
                 }
                 try {
                     this._internal_set_dataValue(dataValue, indexRange);
                 }
                 catch (err) {
-                    if (util_1.types.isNativeError(err)) {
+                    if (node_util_1.types.isNativeError(err)) {
                         warningLog(err.message);
                     }
-                    return callback(null, node_opcua_status_code_1.StatusCodes.BadInternalError);
+                    return callback?.(null, node_opcua_status_code_1.StatusCodes.BadInternalError);
                 }
             }
-            callback(err || null, statusCode1);
+            callback?.(err || null, statusCode1);
         });
     }
     writeAttribute(context, writeValueOptions, callback) {
@@ -737,7 +741,7 @@ class UAVariableImpl extends base_node_impl_1.BaseNodeImpl {
         (0, node_opcua_assert_1.assert)(context instanceof session_context_1.SessionContext);
         (0, node_opcua_assert_1.assert)(writeValue instanceof node_opcua_service_write_1.WriteValue);
         (0, node_opcua_assert_1.assert)(writeValue.value instanceof node_opcua_data_value_1.DataValue);
-        (0, node_opcua_assert_1.assert)(writeValue.value.value instanceof node_opcua_variant_1.Variant);
+        (0, node_opcua_assert_1.assert)(writeValue.value?.value instanceof node_opcua_variant_1.Variant);
         (0, node_opcua_assert_1.assert)(typeof callback === "function");
         // Spec 1.0.2 Part 4 page 58
         // If the SourceTimestamp or the ServerTimestamp is specified, the Server shall
@@ -748,7 +752,7 @@ class UAVariableImpl extends base_node_impl_1.BaseNodeImpl {
                 this.writeValue(context, writeValue.value, writeValue.indexRange, callback);
                 break;
             case node_opcua_data_model_1.AttributeIds.Historizing:
-                if (writeValue.value.value.dataType !== node_opcua_variant_1.DataType.Boolean) {
+                if (writeValue.value?.value.dataType !== node_opcua_variant_1.DataType.Boolean) {
                     return callback(null, node_opcua_status_code_1.StatusCodes.BadTypeMismatch);
                 }
                 if (!this.checkPermissionPrivate(context, node_opcua_types_1.PermissionType.WriteHistorizing)) {
@@ -763,7 +767,7 @@ class UAVariableImpl extends base_node_impl_1.BaseNodeImpl {
                 }
                 // check if user is allowed to do that !
                 // TODO
-                this.historizing = !!writeValue.value.value.value; // yes ! indeed !
+                this.historizing = !!writeValue.value?.value.value; // yes ! indeed !
                 return callback(null, node_opcua_status_code_1.StatusCodes.Good);
             default:
                 super.writeAttribute(context, writeValue, callback);
@@ -784,7 +788,7 @@ class UAVariableImpl extends base_node_impl_1.BaseNodeImpl {
         try {
             this.verifyVariantCompatibility(value);
         }
-        catch (err) {
+        catch (_err) {
             return node_opcua_status_code_1.StatusCodes.BadTypeMismatch;
         }
         return node_opcua_status_code_1.StatusCodes.Good;
@@ -941,7 +945,7 @@ class UAVariableImpl extends base_node_impl_1.BaseNodeImpl {
         }
         // post conditions
         (0, node_opcua_assert_1.assert)(typeof this._timestamped_set_func === "function");
-        (0, node_opcua_assert_1.assert)(this._timestamped_set_func.length === 2, "expecting 2 parameters");
+        (0, node_opcua_assert_1.assert)(this._timestamped_set_func?.length === 2, "expecting 2 parameters");
     }
     readValueAsync(context, callback) {
         (0, node_opcua_assert_1.assert)(typeof callback === "function");
@@ -953,7 +957,7 @@ class UAVariableImpl extends base_node_impl_1.BaseNodeImpl {
             return;
         }
         if (this.isDisposed()) {
-            return callback(null, new node_opcua_data_value_1.DataValue({ statusCode: node_opcua_status_code_1.StatusCodes.BadNodeIdUnknown }));
+            return callback?.(null, new node_opcua_data_value_1.DataValue({ statusCode: node_opcua_status_code_1.StatusCodes.BadNodeIdUnknown }));
         }
         const readImmediate = (innerCallback) => {
             (0, node_opcua_assert_1.assert)(this.$dataValue instanceof node_opcua_data_value_1.DataValue);
@@ -987,7 +991,7 @@ class UAVariableImpl extends base_node_impl_1.BaseNodeImpl {
             // now call all pending callbacks
             const callbacks = this.__waiting_callbacks || [];
             this.__waiting_callbacks = [];
-            const n = callbacks.length;
+            const _n = callbacks.length;
             for (const callback1 of callbacks) {
                 callback1.call(this, err, dataValue);
             }
@@ -999,7 +1003,7 @@ class UAVariableImpl extends base_node_impl_1.BaseNodeImpl {
             // c8 ignore next
             if (doDebug) {
                 debugLog(chalk_1.default.red("func readValueAsync has failed "));
-                if (util_1.types.isNativeError(err)) {
+                if (node_util_1.types.isNativeError(err)) {
                     debugLog(" stack", err.stack);
                 }
             }
@@ -1025,7 +1029,7 @@ class UAVariableImpl extends base_node_impl_1.BaseNodeImpl {
             userAccessLevel: this.userAccessLevel,
             valueRank: this.valueRank
         };
-        const newVariable = (0, base_node_private_1._clone)(this, UAVariableImpl, options, optionalFilter || node_opcua_address_space_base_1.defaultCloneFilter, extraInfo || (0, node_opcua_address_space_base_1.makeDefaultCloneExtraInfo)(this));
+        const newVariable = (0, base_node_private_1._clone)(this, (UAVariableImpl), options, optionalFilter || node_opcua_address_space_base_1.defaultCloneFilter, extraInfo || (0, node_opcua_address_space_base_1.makeDefaultCloneExtraInfo)(this));
         newVariable.bindVariable();
         (0, node_opcua_assert_1.assert)(typeof newVariable._timestamped_set_func === "function");
         (0, node_opcua_assert_1.assert)(newVariable.dataType === this.dataType);
@@ -1057,7 +1061,7 @@ class UAVariableImpl extends base_node_impl_1.BaseNodeImpl {
         const dt = addressSpace.findNode(this.dataType);
         // c8 ignore next
         if (!dt) {
-            throw new Error("getDataTypeNode: cannot find dataType " + this.dataType.toString());
+            throw new Error(`getDataTypeNode: cannot find dataType ${this.dataType.toString()}`);
         }
         return dt;
     }
@@ -1084,14 +1088,14 @@ class UAVariableImpl extends base_node_impl_1.BaseNodeImpl {
         const Constructor = addressSpace.getExtensionObjectConstructor(this.dataType);
         if (this.valueRank === -1) {
             /** Scalar */
-            if (extObj instanceof Array) {
+            if (Array.isArray(extObj)) {
                 return false;
             }
             return checkExtensionObjectIsCorrectScalar.call(this, extObj);
         }
         else if (this.valueRank >= 1) {
             /** array */
-            if (!(extObj instanceof Array)) {
+            if (!Array.isArray(extObj)) {
                 // let's coerce this scalar into an 1-element array if it is a valid extension object
                 if (checkExtensionObjectIsCorrectScalar.call(this, extObj)) {
                     warningLog(`warning: checkExtensionObjectIsCorrect : expecting a array but got a scalar (value rank of '${this.browseName.toString()}' is 1)\nautomatic conversion from scalar to array with 1 element is taking place.`);
@@ -1114,7 +1118,7 @@ class UAVariableImpl extends base_node_impl_1.BaseNodeImpl {
         }
         function checkExtensionObjectIsCorrectScalar(extObj) {
             // c8 ignore next
-            if (!(extObj && extObj.constructor)) {
+            if (!extObj?.constructor) {
                 errorLog(extObj);
                 throw new Error("expecting an valid extension object");
             }
@@ -1137,7 +1141,7 @@ class UAVariableImpl extends base_node_impl_1.BaseNodeImpl {
         function checkExtensionObjectIsCorrectArray(extObjArray) {
             // c8 ignore next
             for (const extObj of extObjArray) {
-                if (!(extObj && extObj.constructor)) {
+                if (!extObj?.constructor) {
                     errorLog(extObj);
                     throw new Error("expecting an valid extension object");
                 }
@@ -1194,13 +1198,13 @@ class UAVariableImpl extends base_node_impl_1.BaseNodeImpl {
             optionalExtensionObject = [optionalExtensionObject];
         }
         if (optionalExtensionObject) {
-            if (optionalExtensionObject instanceof Array) {
+            if (Array.isArray(optionalExtensionObject)) {
                 (0, node_opcua_assert_1.assert)(this.valueRank >= 1, "bindExtensionObject: expecting an Array of Matrix variable here");
                 return (0, ua_variable_impl_ext_obj_1._bindExtensionObjectArrayOrMatrix)(this, optionalExtensionObject, options);
             }
             else {
                 if (this.valueRank !== -1 && this.valueRank !== 0) {
-                    throw new Error("bindExtensionObject: expecting an Scalar variable here but got value rank " + this.valueRank);
+                    throw new Error(`bindExtensionObject: expecting an Scalar variable here but got value rank ${this.valueRank}`);
                 }
                 return (0, ua_variable_impl_ext_obj_1._bindExtensionObject)(this, optionalExtensionObject, options);
             }
@@ -1274,7 +1278,10 @@ class UAVariableImpl extends base_node_impl_1.BaseNodeImpl {
             if (dv) {
                 if (dv.statusCode)
                     statusStr = dv.statusCode.toString();
-                if (dv.value && dv.value.value !== undefined && dv.value.value !== null && !(dv.value.value instanceof node_opcua_extension_object_1.OpaqueStructure)) {
+                if (dv.value &&
+                    dv.value.value !== undefined &&
+                    dv.value.value !== null &&
+                    !(dv.value.value instanceof node_opcua_extension_object_1.OpaqueStructure)) {
                     const v = dv.value.value;
                     valueStr = typeof v === "object" ? Object.prototype.toString.call(v) : String(v);
                     if (valueStr.length > 80)
@@ -1311,17 +1318,17 @@ class UAVariableImpl extends base_node_impl_1.BaseNodeImpl {
         (0, node_opcua_assert_1.assert)(context instanceof session_context_1.SessionContext);
         (0, node_opcua_assert_1.assert)(typeof callback === "function");
         if (typeof this._historyRead !== "function") {
-            return callback(null, new node_opcua_types_1.HistoryReadResult({ statusCode: node_opcua_status_code_1.StatusCodes.BadNotReadable }));
+            return callback?.(null, new node_opcua_types_1.HistoryReadResult({ statusCode: node_opcua_status_code_1.StatusCodes.BadNotReadable }));
         }
         this._historyRead(context, historyReadDetails, indexRange, dataEncoding, continuationData, callback);
     }
-    _historyReadRaw(context, historyReadRawModifiedDetails, indexRange, dataEncoding, continuationData, callback) {
+    _historyReadRaw(_context, _historyReadRawModifiedDetails, _indexRange, _dataEncoding, _continuationData, _callback) {
         throw new Error("");
     }
-    _historyReadRawModify(context, historyReadRawModifiedDetails, indexRange, dataEncoding, continuationData, callback) {
+    _historyReadRawModify(_context, _historyReadRawModifiedDetails, _indexRange, _dataEncoding, _continuationData, _callback) {
         throw new Error("");
     }
-    _historyRead(context, historyReadDetails, indexRange, dataEncoding, continuationData, callback) {
+    _historyRead(context, _historyReadDetails, _indexRange, _dataEncoding, _continuationData, callback) {
         if (!this.checkPermissionPrivate(context, node_opcua_types_1.PermissionType.ReadHistory)) {
             const result = new node_opcua_types_1.HistoryReadResult({
                 statusCode: node_opcua_status_code_1.StatusCodes.BadUserAccessDenied
@@ -1339,20 +1346,20 @@ class UAVariableImpl extends base_node_impl_1.BaseNodeImpl {
         });
         callback(null, result);
     }
-    _historyPush(newDataValue) {
+    _historyPush(_newDataValue) {
         throw new Error("");
     }
-    _historyReadRawAsync(historyReadRawModifiedDetails, maxNumberToExtract, isReversed, reverseDataValue, callback) {
+    _historyReadRawAsync(_historyReadRawModifiedDetails, _maxNumberToExtract, _isReversed, _reverseDataValue, _callback) {
         throw new Error("");
     }
-    _historyReadModify(context, historyReadRawModifiedDetails, indexRange, dataEncoding, continuationData, callback) {
+    _historyReadModify(_context, _historyReadRawModifiedDetails, _indexRange, _dataEncoding, _continuationData, _callback) {
         throw new Error("");
     }
-    _update_startOfOnlineArchive(newDate) {
+    _update_startOfOnlineArchive(_newDate) {
         // please install
         throw new Error("");
     }
-    _update_startOfArchive(newDate) {
+    _update_startOfArchive(_newDate) {
         throw new Error("");
     }
     _validate_DataType(variantDataType) {
@@ -1389,7 +1396,7 @@ class UAVariableImpl extends base_node_impl_1.BaseNodeImpl {
             // c8 ignore next
             if (!this.checkExtensionObjectIsCorrect(dataValue.value.value)) {
                 warningLog(dataValue.toString());
-                throw new Error("Invalid Extension Object on nodeId =" + this.nodeId.toString());
+                throw new Error(`Invalid Extension Object on nodeId =${this.nodeId.toString()}`);
             }
         }
         this.verifyVariantCompatibility(dataValue.value);
@@ -1459,7 +1466,7 @@ class UAVariableImpl extends base_node_impl_1.BaseNodeImpl {
     _conditionRefresh(_cache) {
         apply_condition_refresh_1.apply_condition_refresh.call(this, _cache);
     }
-    handle_semantic_changed() {
+    handle_semantic_changed(_dataValue) {
         this.semantic_version = this.semantic_version + 1;
         this.emit("semantic_changed");
     }
@@ -1588,7 +1595,7 @@ function check_valid_array(dataType, array) {
 function unsetFlag(flags, mask) {
     return flags & ~mask;
 }
-function setFlag(flags, mask) {
+function _setFlag(flags, mask) {
     return flags | mask;
 }
 function _calculateEffectiveUserAccessLevelFromPermission(node, context, userAccessLevel) {
@@ -1620,7 +1627,7 @@ function _calculateEffectiveUserAccessLevelFromPermission(node, context, userAcc
 }
 function adjustVariant2(variant) {
     // convert Variant( Scalar|ByteString) =>  Variant(Array|ByteArray)
-    const addressSpace = this.addressSpace;
+    const _addressSpace = this.addressSpace;
     const basicType = this.getBasicDataType();
     variant = (0, node_opcua_variant_1.adjustVariant)(variant, this.valueRank, basicType);
     return variant;
@@ -1680,13 +1687,15 @@ function _Variable_bind_with_async_refresh(options) {
     (0, node_opcua_assert_1.assert)(!options.timestamped_get, "a getter shall not be specified when refreshFunc is set");
     (0, node_opcua_assert_1.assert)(!this.refreshFunc);
     this.refreshFunc = options.refreshFunc;
-    // TO DO : REVISIT THIS ASSUMPTION
-    if (false && this.minimumSamplingInterval === 0) {
-        // when a getter /timestamped_getter or async_getter is provided
-        // samplingInterval cannot be 0, as the item value must be scanned to be updated.
-        this.minimumSamplingInterval = _default_minimumSamplingInterval; // MonitoredItem.minimumSamplingInterval;
-        debugLog("adapting minimumSamplingInterval on " + this.browseName.toString() + " to " + this.minimumSamplingInterval);
-    }
+    /*
+        // TO DO : REVISIT THIS ASSUMPTION
+        if (false && this.minimumSamplingInterval === 0) {
+            // when a getter /timestamped_getter or async_getter is provided
+            // samplingInterval cannot be 0, as the item value must be scanned to be updated.
+            this.minimumSamplingInterval = _default_minimumSamplingInterval; // MonitoredItem.minimumSamplingInterval;
+            debugLog(`adapting minimumSamplingInterval on ${this.browseName.toString()} to ${this.minimumSamplingInterval}`);
+        }
+            */
 }
 // variation 2
 function _Variable_bind_with_timestamped_get(options) {
@@ -1730,7 +1739,7 @@ function _Variable_bind_with_timestamped_get(options) {
 function _Variable_bind_with_simple_get(options) {
     (0, node_opcua_assert_1.assert)(this instanceof UAVariableImpl);
     (0, node_opcua_assert_1.assert)(typeof options.get === "function", "should specify get function");
-    (0, node_opcua_assert_1.assert)(options.get.length === 0, "get function should not have arguments");
+    (0, node_opcua_assert_1.assert)(options.get?.length === 0, "get function should not have arguments");
     (0, node_opcua_assert_1.assert)(!options.timestamped_get, "should not specify a timestamped_get function when get is specified");
     (0, node_opcua_assert_1.assert)(!this._timestamped_get_func);
     (0, node_opcua_assert_1.assert)(!this._get_func);
@@ -1746,9 +1755,7 @@ function _Variable_bind_with_simple_get(options) {
             return new node_opcua_data_value_1.DataValue({ statusCode: value });
         }
         else {
-            if (!this.$dataValue ||
-                !this.$dataValue.statusCode.isGoodish() ||
-                !(0, node_opcua_variant_1.sameVariant)(this.$dataValue.value, value)) {
+            if (!this.$dataValue?.statusCode.isGoodish() || !(0, node_opcua_variant_1.sameVariant)(this.$dataValue.value, value)) {
                 // rebuilding artificially timestamps with current clock as they are not provided
                 // by the underlying getter function
                 const { timestamp: sourceTimestamp, picoseconds: sourcePicoseconds } = (0, node_opcua_date_time_1.getCurrentClock)();
@@ -1782,7 +1789,7 @@ function _Variable_bind_with_simple_set(options) {
                 errorLog(chalk_1.default.yellow("StatusCode.Good is assumed"));
                 return callback(err, node_opcua_status_code_1.StatusCodes.Good, timestamped_value);
             }
-            if (statusCode && statusCode.isNotGood()) {
+            if (statusCode?.isNotGood()) {
                 // record the value but still record the statusCode !
                 timestamped_value.statusCode = statusCode;
             }
@@ -1848,7 +1855,7 @@ function bind_getter(options) {
         _Variable_bind_with_async_refresh.call(this, { refreshFunc: options.refreshFunc });
     }
     else {
-        (0, node_opcua_assert_1.assert)(!Object.prototype.hasOwnProperty.call(options, "set"), "getter is missing : a getter must be provided if a setter is provided");
+        (0, node_opcua_assert_1.assert)(!Object.hasOwn(options, "set"), "getter is missing : a getter must be provided if a setter is provided");
         // xx bind_variant.call(this,options);
         if (options.dataType !== undefined) {
             // if (options.dataType !== DataType.ExtensionObject) {
@@ -1858,6 +1865,15 @@ function bind_getter(options) {
     }
 }
 class UAVariableImplT extends UAVariableImpl {
+    readValue(context, indexRange, dataEncoding) {
+        return super.readValue(context, indexRange, dataEncoding);
+    }
+    readValueAsync(context, callback) {
+        return super.readValueAsync(context, callback);
+    }
+    writeValue(context, dataValue, indexRangeOrCallback, callback) {
+        return super.writeValue(context, dataValue, indexRangeOrCallback, callback);
+    }
 }
 exports.UAVariableImplT = UAVariableImplT;
 function changeUAVariableDataType(uaVariable, newDataType, valueLike) {
@@ -1866,7 +1882,7 @@ function changeUAVariableDataType(uaVariable, newDataType, valueLike) {
     const newDataTypeNode = addressSpace.findNode(newDataType);
     // c8 ignore next
     if (!newDataTypeNode || !(newDataTypeNode instanceof ua_data_type_impl_1.UADataTypeImpl)) {
-        throw new Error("Cannot find newDataTypeNode " + newDataType.toString());
+        throw new Error(`Cannot find newDataTypeNode ${newDataType.toString()}`);
     }
     const newBaseDataType = newDataTypeNode.basicDataType;
     // c8 ignore next