@enyo-energy/sunspec-sdk 0.0.49 → 0.0.51

package/dist/cjs/sunspec-devices.cjs

@@ -335,7 +335,7 @@ class SunspecInverter extends BaseSunspecDevice {
             const dcStrings = this.mapMPPTToStrings(mpptDataList);
             if (inverterData) {
                 const pvPowerW = dcStrings.reduce((sum, s) => sum + (s.powerW || 0), 0);
-                console.log(`Got PV Power from DC strings: ${pvPowerW}W`);
+                console.debug(`Got PV Power from DC strings: ${pvPowerW}W`);
                 const inverterMessage = {
                     id: (0, node_crypto_1.randomUUID)(),
                     message: enyo_data_bus_value_js_1.EnyoDataBusMessageEnum.InverterValuesUpdateV1,
@@ -425,19 +425,22 @@ class SunspecInverter extends BaseSunspecDevice {
     }
     /**
      * Map a set bit in the Evt2 register. Reserved in the SunSpec spec, so
-     * the default emits a generic code with no translation. Override for
-     * vendor-specific Evt2 semantics.
+     * the default returns `undefined` — Evt2 bits are not treated as errors.
+     * Override in a vendor-specific subclass to opt specific bits in as
+     * errors.
      */
-    mapEvt2Bit(bit) {
-        return { code: `inverter_event2_bit_${bit}` };
+    mapEvt2Bit(_bit) {
+        return undefined;
     }
     /**
      * Map a set bit in one of the four vendor-specific event registers.
-     * Override in a vendor-specific subclass to translate vendor bits.
-     * Return `undefined` to drop a bit entirely.
+     * Vendor bits are not standardized and many are informational rather
+     * than faults, so the default returns `undefined` and no error code is
+     * emitted. Override in a vendor-specific subclass to opt specific bits
+     * in as errors.
      */
-    mapVendorEventBit(registerIndex, bit) {
-        return { code: `inverter_vendor_event${registerIndex}_bit_${bit}` };
+    mapVendorEventBit(_registerIndex, _bit) {
+        return undefined;
     }
     hasErrorSetChanged(newCodeIds) {
         const prev = this.errorState.activeCodes;

package/dist/cjs/sunspec-devices.d.cts

@@ -98,16 +98,19 @@ export declare class SunspecInverter extends BaseSunspecDevice {
     protected mapEvt1Bit(bit: number): EnyoApplianceErrorCode | undefined;
     /**
      * Map a set bit in the Evt2 register. Reserved in the SunSpec spec, so
-     * the default emits a generic code with no translation. Override for
-     * vendor-specific Evt2 semantics.
+     * the default returns `undefined` — Evt2 bits are not treated as errors.
+     * Override in a vendor-specific subclass to opt specific bits in as
+     * errors.
      */
-    protected mapEvt2Bit(bit: number): EnyoApplianceErrorCode | undefined;
+    protected mapEvt2Bit(_bit: number): EnyoApplianceErrorCode | undefined;
     /**
      * Map a set bit in one of the four vendor-specific event registers.
-     * Override in a vendor-specific subclass to translate vendor bits.
-     * Return `undefined` to drop a bit entirely.
+     * Vendor bits are not standardized and many are informational rather
+     * than faults, so the default returns `undefined` and no error code is
+     * emitted. Override in a vendor-specific subclass to opt specific bits
+     * in as errors.
      */
-    protected mapVendorEventBit(registerIndex: 1 | 2 | 3 | 4, bit: number): EnyoApplianceErrorCode | undefined;
+    protected mapVendorEventBit(_registerIndex: 1 | 2 | 3 | 4, _bit: number): EnyoApplianceErrorCode | undefined;
     private hasErrorSetChanged;
     private buildStatusMessage;
     private storageKey;

package/dist/cjs/sunspec-modbus-client.cjs

@@ -36,6 +36,9 @@ class SunspecModbusClient {
     openCount = 0;
     closeCount = 0;
     currentlyOpen = 0;
+    // Serializes connection-state transitions so concurrent callers cannot open a second
+    // TCP socket while the first is still alive (some Modbus devices allow only one client).
+    operationChain = Promise.resolve();
     constructor(energyApp) {
         this.energyApp = energyApp;
         this.connectionHealth = new EnergyAppModbusConnectionHealth_js_1.EnergyAppModbusConnectionHealth();
@@ -49,25 +52,23 @@ class SunspecModbusClient {
      * @param secondaryHost Optional secondary host (ipAddress) for fallback during reconnection
      */
     async connect(host, port = 502, unitId = 1, secondaryHost) {
-        if (this.connected) {
-            await this.disconnect();
-        }
-        // Store connection parameters for potential reconnection
-        this.connectionParams = { primaryHost: host, secondaryHost, port, unitId };
-        this.modbusClient = await this.energyApp.useModbus().connect({
-            host,
-            port,
-            unitId,
-            timeout: 5000
+        return this.withConnectionLock(async () => {
+            const sameParams = this.connected
+                && this.connectionParams !== null
+                && this.connectionParams.primaryHost === host
+                && this.connectionParams.port === port
+                && this.connectionParams.unitId === unitId;
+            if (sameParams) {
+                console.debug(`connect(): already connected to ${host}:${port} unit ${unitId}, skipping`);
+                return;
+            }
+            if (this.connected || this.modbusClient) {
+                await this._closeSocket();
+            }
+            this.connectionParams = { primaryHost: host, secondaryHost, port, unitId };
+            await this._openSocket(host, port, unitId);
+            console.log(`Connected to Sunspec device at ${host}:${port} unit ${unitId} (opens=${this.openCount}, closes=${this.closeCount}, current=${this.currentlyOpen})`);
         });
-        // Create fault-tolerant reader with connection health monitoring
-        if (this.modbusClient) {
-            this.faultTolerantReader = new EnergyAppModbusFaultTolerantReader_js_1.EnergyAppModbusFaultTolerantReader(this.modbusClient, this.connectionHealth);
-        }
-        this.connected = true;
-        this.connectionHealth.recordSuccess();
-        this.recordOpen();
-        console.log(`Connected to Sunspec device at ${host}:${port} unit ${unitId} (opens=${this.openCount}, closes=${this.closeCount}, current=${this.currentlyOpen})`);
     }
     /**
      * Disconnect from Modbus device.
@@ -76,18 +77,16 @@ class SunspecModbusClient {
      * They will be overwritten by the next connect() call anyway.
      */
     async disconnect() {
-        if (this.modbusClient && this.connected) {
-            await this.modbusClient.disconnect();
-            this.modbusClient = null;
-            this.faultTolerantReader = null;
-            this.connected = false;
-            this.discoveredModels.clear();
-            this.recordClose();
+        return this.withConnectionLock(async () => {
+            if (!this.modbusClient && !this.connected) {
+                return;
+            }
             const host = this.connectionParams?.primaryHost ?? 'unknown';
             const port = this.connectionParams?.port ?? 0;
             const unitId = this.connectionParams?.unitId ?? 0;
+            await this._closeSocket();
             console.log(`Disconnected from Sunspec device at ${host}:${port} unit ${unitId} (opens=${this.openCount}, closes=${this.closeCount}, current=${this.currentlyOpen})`);
-        }
+        });
     }
     /**
      * Reconnect using stored connection parameters
@@ -95,78 +94,122 @@ class SunspecModbusClient {
      * Returns true if reconnection was successful, false otherwise
      */
     async reconnect() {
-        if (!this.connectionParams) {
-            console.error('Cannot reconnect: no connection parameters stored. Call connect() first.');
+        return this.withConnectionLock(async () => {
+            if (!this.connectionParams) {
+                console.error('Cannot reconnect: no connection parameters stored. Call connect() first.');
+                return false;
+            }
+            const { primaryHost, secondaryHost, port, unitId } = this.connectionParams;
+            console.log(`Attempting to reconnect to primary host ${primaryHost}:${port} unit ${unitId}...`);
+            const primarySuccess = await this.attemptConnection(primaryHost, port, unitId);
+            if (primarySuccess) {
+                console.log(`Successfully reconnected to primary host ${primaryHost}:${port} unit ${unitId}`);
+                return true;
+            }
+            if (secondaryHost && secondaryHost !== primaryHost) {
+                console.log(`Primary host failed, attempting secondary host ${secondaryHost}:${port} unit ${unitId}...`);
+                const secondarySuccess = await this.attemptConnection(secondaryHost, port, unitId);
+                if (secondarySuccess) {
+                    console.log(`Successfully reconnected to secondary host ${secondaryHost}:${port} unit ${unitId}`);
+                    return true;
+                }
+            }
+            console.error(`Reconnection failed to all available hosts`);
             return false;
+        });
+    }
+    /**
+     * Attempt to establish a connection to a specific host. Caller must hold the connection lock.
+     * Returns true if successful, false otherwise.
+     */
+    async attemptConnection(host, port, unitId) {
+        if (this.connected || this.modbusClient) {
+            await this._closeSocket();
         }
-        const { primaryHost, secondaryHost, port, unitId } = this.connectionParams;
-        // Try primary host first
-        console.log(`Attempting to reconnect to primary host ${primaryHost}:${port} unit ${unitId}...`);
-        const primarySuccess = await this.attemptConnection(primaryHost, port, unitId);
-        if (primarySuccess) {
-            console.log(`Successfully reconnected to primary host ${primaryHost}:${port} unit ${unitId}`);
+        try {
+            await this._openSocket(host, port, unitId);
+            console.log(`Connection attempt to ${host}:${port} unit ${unitId} succeeded (opens=${this.openCount}, closes=${this.closeCount}, current=${this.currentlyOpen})`);
             return true;
         }
-        // If primary failed and secondary is available, try secondary
-        if (secondaryHost && secondaryHost !== primaryHost) {
-            console.log(`Primary host failed, attempting secondary host ${secondaryHost}:${port} unit ${unitId}...`);
-            const secondarySuccess = await this.attemptConnection(secondaryHost, port, unitId);
-            if (secondarySuccess) {
-                console.log(`Successfully reconnected to secondary host ${secondaryHost}:${port} unit ${unitId}`);
-                return true;
-            }
+        catch (error) {
+            console.error(`Connection attempt to ${host}:${port} failed: ${error}`);
+            return false;
         }
-        console.error(`Reconnection failed to all available hosts`);
-        this.connected = false;
-        return false;
     }
     /**
-     * Attempt to establish a connection to a specific host
-     * Returns true if successful, false otherwise
+     * Open a new Modbus socket and wire up the fault-tolerant reader. On any failure, the
+     * just-opened socket (if any) is closed so we never leak. Caller must hold the connection
+     * lock and ensure no prior socket is open (use _closeSocket first).
      */
-    async attemptConnection(host, port, unitId) {
+    async _openSocket(host, port, unitId) {
+        let candidate = null;
         try {
-            // Disconnect existing connection if any
-            if (this.modbusClient) {
-                const wasConnected = this.connected;
-                try {
-                    await this.modbusClient.disconnect();
-                }
-                catch (e) {
-                    // Ignore disconnect errors during reconnection
-                }
-                this.modbusClient = null;
-                this.faultTolerantReader = null;
-                if (wasConnected) {
-                    this.connected = false;
-                    this.recordClose();
-                }
-            }
-            // Attempt connection
-            this.modbusClient = await this.energyApp.useModbus().connect({
+            candidate = await this.energyApp.useModbus().connect({
                 host,
                 port,
                 unitId,
                 timeout: 5000
             });
-            // Create fault-tolerant reader with connection health monitoring
-            if (this.modbusClient) {
-                this.faultTolerantReader = new EnergyAppModbusFaultTolerantReader_js_1.EnergyAppModbusFaultTolerantReader(this.modbusClient, this.connectionHealth);
+            if (!candidate) {
+                throw new Error(`useModbus().connect returned null for ${host}:${port} unit ${unitId}`);
             }
+            this.faultTolerantReader = new EnergyAppModbusFaultTolerantReader_js_1.EnergyAppModbusFaultTolerantReader(candidate, this.connectionHealth);
+            this.modbusClient = candidate;
             this.connected = true;
             this.connectionHealth.recordSuccess();
             this.recordOpen();
-            console.log(`Connection attempt to ${host}:${port} unit ${unitId} succeeded (opens=${this.openCount}, closes=${this.closeCount}, current=${this.currentlyOpen})`);
-            return true;
         }
-        catch (error) {
-            console.error(`Connection attempt to ${host}:${port} failed: ${error}`);
-            return false;
+        catch (err) {
+            if (candidate) {
+                try {
+                    await candidate.disconnect();
+                }
+                catch { /* ignore */ }
+            }
+            this.modbusClient = null;
+            this.faultTolerantReader = null;
+            this.connected = false;
+            throw err;
         }
     }
+    /**
+     * Close the current Modbus socket if any. Idempotent. Caller must hold the connection lock.
+     */
+    async _closeSocket() {
+        if (!this.modbusClient && !this.connected)
+            return;
+        const client = this.modbusClient;
+        const wasConnected = this.connected;
+        this.modbusClient = null;
+        this.faultTolerantReader = null;
+        this.connected = false;
+        this.discoveredModels.clear();
+        if (client) {
+            try {
+                await client.disconnect();
+            }
+            catch { /* ignore */ }
+        }
+        if (wasConnected)
+            this.recordClose();
+    }
+    /**
+     * Run `fn` with exclusive access to connection-state transitions. Subsequent calls queue
+     * behind any in-flight one. A rejected `fn` does not poison the chain for later callers.
+     */
+    withConnectionLock(fn) {
+        const run = this.operationChain.then(fn, fn);
+        this.operationChain = run.catch(() => undefined);
+        return run;
+    }
     recordOpen() {
         this.openCount++;
         this.currentlyOpen++;
+        if (this.currentlyOpen > 1) {
+            console.error(`SunspecModbusClient: currentlyOpen=${this.currentlyOpen} after open ` +
+                `(opens=${this.openCount}, closes=${this.closeCount}). ` +
+                `This indicates a code path bypassing the connection lock — please investigate.`);
+        }
     }
     recordClose() {
         this.closeCount++;
@@ -516,7 +559,7 @@ class SunspecModbusClient {
     async readInverterData() {
         const model = this.findModel(sunspec_interfaces_js_1.SunspecModelId.Inverter3Phase);
         if (!model) {
-            console.log('Inverter model 103 not found, trying single phase model 101');
+            console.debug('Inverter model 103 not found, trying single phase model 101');
             const singlePhaseModel = this.findModel(sunspec_interfaces_js_1.SunspecModelId.InverterSinglePhase);
             if (!singlePhaseModel) {
                 console.error('No inverter model found');
@@ -525,10 +568,10 @@ class SunspecModbusClient {
             console.warn('IMPORTANT: Working with single-phase inverter, but 3-phase is expected!');
             return this.readSinglePhaseInverterData(singlePhaseModel);
         }
-        console.log(`Found 3-phase inverter model 103 at address ${model.address} with length ${model.length}`);
+        console.debug(`Found 3-phase inverter model 103 at address ${model.address} with length ${model.length}`);
         try {
             // Read entire model block in a single Modbus call
-            console.log(`Reading Inverter Data from Model ${model.id} at base address: ${model.address}`);
+            console.debug(`Reading Inverter Data from Model ${model.id} at base address: ${model.address}`);
             const buffer = await this.readModelBlock(model);
             // Extract all scale factors from buffer
             const scaleFactors = this.extractInverterScaleFactors(buffer);
@@ -612,7 +655,7 @@ class SunspecModbusClient {
      */
     async readSinglePhaseInverterData(model) {
         try {
-            console.log(`Reading Single-Phase Inverter Data from Model 101 at base address: ${model.address}`);
+            console.debug(`Reading Single-Phase Inverter Data from Model 101 at base address: ${model.address}`);
             // Read entire model block in a single Modbus call
             const buffer = await this.readModelBlock(model);
             // Extract scale factors from buffer
@@ -766,7 +809,7 @@ class SunspecModbusClient {
     async readMPPTScaleFactors() {
         const model = this.findModel(sunspec_interfaces_js_1.SunspecModelId.MPPT);
         if (!model) {
-            console.log('MPPT model 160 not found');
+            console.debug('MPPT model 160 not found');
             return null;
         }
         try {
@@ -800,7 +843,7 @@ class SunspecModbusClient {
             this.isUnimplementedValue(dcCurrentRaw, 'uint16') &&
             this.isUnimplementedValue(dcVoltageRaw, 'uint16') &&
             this.isUnimplementedValue(dcPowerRaw, 'uint16')) {
-            console.log(`MPPT module ${moduleId} appears to be unconnected (all values are 0xFFFF)`);
+            console.debug(`MPPT module ${moduleId} appears to be unconnected (all values are 0xFFFF)`);
             return null;
         }
         const temperatureScaleFactor = -1;
@@ -838,7 +881,7 @@ class SunspecModbusClient {
     async readMPPTData(moduleId = 1) {
         const model = this.findModel(sunspec_interfaces_js_1.SunspecModelId.MPPT);
         if (!model) {
-            console.log('MPPT model 160 not found');
+            console.debug('MPPT model 160 not found');
             return null;
         }
         try {
@@ -859,7 +902,7 @@ class SunspecModbusClient {
         const mpptData = [];
         const model = this.findModel(sunspec_interfaces_js_1.SunspecModelId.MPPT);
         if (!model) {
-            console.log('MPPT model 160 not found');
+            console.debug('MPPT model 160 not found');
             return [];
         }
         try {
@@ -871,16 +914,16 @@ class SunspecModbusClient {
             const count = this.extractValue(buffer, 8, 'uint16');
             if (!this.isUnimplementedValue(count, 'uint16') && count > 0 && count <= 20) {
                 moduleCount = count;
-                console.log(`MPPT module count from register 8: ${moduleCount}`);
+                console.debug(`MPPT module count from register 8: ${moduleCount}`);
             }
             else {
-                console.log(`Invalid or unimplemented module count (${count}), using default: ${moduleCount}`);
+                console.debug(`Invalid or unimplemented module count (${count}), using default: ${moduleCount}`);
             }
             // Extract each MPPT module from the same buffer
             for (let i = 1; i <= moduleCount; i++) {
                 try {
                     const data = this.extractMPPTModuleData(buffer, model, i, scaleFactors);
-                    console.log(`MPPT ${i} has id ${data?.id} (${data?.stringId}) with ${data?.dcPower}W`);
+                    console.debug(`MPPT ${i} has id ${data?.id} (${data?.stringId}) with ${data?.dcPower}W`);
                     if (data &&
                         (data.dcCurrent !== undefined ||
                             data.dcVoltage !== undefined ||
@@ -1028,10 +1071,10 @@ class SunspecModbusClient {
     async readBatteryBaseData() {
         const model = this.findModel(sunspec_interfaces_js_1.SunspecModelId.BatteryBase);
         if (!model) {
-            console.log('Battery Base model 802 not found');
+            console.debug('Battery Base model 802 not found');
             return null;
         }
-        console.log(`Reading Battery Base Data from Model 802 at base address: ${model.address}`);
+        console.debug(`Reading Battery Base Data from Model 802 at base address: ${model.address}`);
         try {
             // Read entire model block in a single Modbus call
             const buffer = await this.readModelBlock(model);
@@ -1174,14 +1217,14 @@ class SunspecModbusClient {
             model = this.findModel(sunspec_interfaces_js_1.SunspecModelId.BatteryControl);
         }
         if (!model) {
-            console.log('No battery model found');
+            console.debug('No battery model found');
             return null;
         }
-        console.log(`Reading Battery Data from Model ${model.id} at base address: ${model.address}`);
+        console.debug(`Reading Battery Data from Model ${model.id} at base address: ${model.address}`);
         try {
             if (model.id === 124) {
                 // Model 124: Basic Storage Controls
-                console.log('Using Model 124 (Basic Storage Controls)');
+                console.debug('Using Model 124 (Basic Storage Controls)');
                 // Read entire model block in a single Modbus call
                 const buffer = await this.readModelBlock(model);
                 // Extract scale factors from buffer (offsets 18-25)
@@ -1271,19 +1314,19 @@ class SunspecModbusClient {
                 // Calculate charge/discharge power if rates are available
                 if (data.inWRte !== undefined && data.wChaMax !== undefined) {
                     data.chargePower = (data.inWRte / 100) * data.wChaMax;
-                    console.log(`Calculated Charge Power: (inWRte: ${data.inWRte}, wChaMax: ${data.wChaMax}) ${data.chargePower?.toFixed(2)} W`);
+                    console.debug(`Calculated Charge Power: (inWRte: ${data.inWRte}, wChaMax: ${data.wChaMax}) ${data.chargePower?.toFixed(2)} W`);
                 }
                 if (data.outWRte !== undefined && data.wChaMax !== undefined) {
                     // Assuming WDisChaMax is similar to WChaMax for simplicity
                     data.dischargePower = Math.abs((data.outWRte / 100) * data.wChaMax);
-                    console.log(`Calculated Discharge Power (inWRte: ${data.outWRte}, wChaMax: ${data.wChaMax}): ${data.dischargePower?.toFixed(2)} W`);
+                    console.debug(`Calculated Discharge Power (inWRte: ${data.outWRte}, wChaMax: ${data.wChaMax}): ${data.dischargePower?.toFixed(2)} W`);
                 }
                 console.debug('[Model 124] Battery Data:', data);
                 return data;
             }
             else if (model.id === 802) {
                 // Model 802: Battery Base
-                console.log('Using Model 802 (Battery Base)');
+                console.debug('Using Model 802 (Battery Base)');
                 const baseData = await this.readBatteryBaseData();
                 if (!baseData) {
                     return null;
@@ -1320,7 +1363,7 @@ class SunspecModbusClient {
             }
             else {
                 // Handle other battery models (803) if needed
-                console.log(`Battery Model ${model.id} reading not yet implemented`);
+                console.debug(`Battery Model ${model.id} reading not yet implemented`);
                 return {
                     blockNumber: model.id,
                     blockAddress: model.address,
@@ -1492,13 +1535,13 @@ class SunspecModbusClient {
             model = this.findModel(sunspec_interfaces_js_1.SunspecModelId.MeterSinglePhase);
         }
         if (!model) {
-            console.log('No meter model found');
+            console.debug('No meter model found');
             return null;
         }
-        console.log(`Reading Meter Data from Model ${model.id} at base address: ${model.address}`);
+        console.debug(`Reading Meter Data from Model ${model.id} at base address: ${model.address}`);
         try {
             // Different meter models have different register offsets
-            console.log(`Meter is Model ${model.id}`);
+            console.debug(`Meter is Model ${model.id}`);
             let powerOffset;
             let powerSFOffset;
             let freqOffset;
@@ -1508,7 +1551,7 @@ class SunspecModbusClient {
             let energySFOffset;
             switch (model.id) {
                 case 203: // 3-phase meter (Delta)
-                    console.log('Using Model 203 (3-Phase Meter Delta) offsets');
+                    console.debug('Using Model 203 (3-Phase Meter Delta) offsets');
                     powerOffset = 18; // W - Total Real Power (int16)
                     powerSFOffset = 22; // W_SF - Power scale factor
                     freqOffset = 16; // Hz - Frequency (uint16)
@@ -1518,7 +1561,7 @@ class SunspecModbusClient {
                     energySFOffset = 54; // TotWh_SF - Total Energy scale factor
                     break;
                 case 204: // 3-phase meter (Wye)
-                    console.log('Using Model 204 (3-Phase Meter Wye) offsets');
+                    console.debug('Using Model 204 (3-Phase Meter Wye) offsets');
                     powerOffset = 18; // W - Total Real Power (int16)
                     powerSFOffset = 22; // W_SF - Power scale factor
                     freqOffset = 16; // Hz - Frequency (uint16)
@@ -1529,7 +1572,7 @@ class SunspecModbusClient {
                     break;
                 case 201: // Single-phase meter
                 default:
-                    console.log('Using Model 201 (Single-Phase Meter) offsets');
+                    console.debug('Using Model 201 (Single-Phase Meter) offsets');
                     powerOffset = 18; // W - Total Real Power (int16)
                     powerSFOffset = 22; // W_SF - Power scale factor
                     freqOffset = 16; // Hz - Frequency (uint16)
@@ -1651,7 +1694,7 @@ class SunspecModbusClient {
     async readInverterSettings() {
         const model = this.findModel(sunspec_interfaces_js_1.SunspecModelId.Settings);
         if (!model) {
-            console.log('Settings model 121 not found');
+            console.debug('Settings model 121 not found');
             return null;
         }
         try {

package/dist/cjs/sunspec-modbus-client.d.cts

@@ -32,6 +32,7 @@ export declare class SunspecModbusClient {
     private openCount;
     private closeCount;
     private currentlyOpen;
+    private operationChain;
     constructor(energyApp: EnergyApp);
     /**
      * Connect to Modbus device
@@ -55,10 +56,25 @@ export declare class SunspecModbusClient {
      */
     reconnect(): Promise<boolean>;
     /**
-     * Attempt to establish a connection to a specific host
-     * Returns true if successful, false otherwise
+     * Attempt to establish a connection to a specific host. Caller must hold the connection lock.
+     * Returns true if successful, false otherwise.
      */
     private attemptConnection;
+    /**
+     * Open a new Modbus socket and wire up the fault-tolerant reader. On any failure, the
+     * just-opened socket (if any) is closed so we never leak. Caller must hold the connection
+     * lock and ensure no prior socket is open (use _closeSocket first).
+     */
+    private _openSocket;
+    /**
+     * Close the current Modbus socket if any. Idempotent. Caller must hold the connection lock.
+     */
+    private _closeSocket;
+    /**
+     * Run `fn` with exclusive access to connection-state transitions. Subsequent calls queue
+     * behind any in-flight one. A rejected `fn` does not poison the chain for later callers.
+     */
+    private withConnectionLock;
     private recordOpen;
     private recordClose;
     /**

package/dist/cjs/version.cjs

@@ -9,7 +9,7 @@ exports.getSdkVersion = getSdkVersion;
 /**
  * Current version of the enyo Energy App SDK.
  */
-exports.SDK_VERSION = '0.0.49';
+exports.SDK_VERSION = '0.0.51';
 /**
  * Gets the current SDK version.
  * @returns The semantic version string of the SDK

package/dist/cjs/version.d.cts

@@ -5,7 +5,7 @@
 /**
  * Current version of the enyo Energy App SDK.
  */
-export declare const SDK_VERSION = "0.0.49";
+export declare const SDK_VERSION = "0.0.51";
 /**
  * Gets the current SDK version.
  * @returns The semantic version string of the SDK

package/dist/sunspec-devices.d.ts

@@ -98,16 +98,19 @@ export declare class SunspecInverter extends BaseSunspecDevice {
     protected mapEvt1Bit(bit: number): EnyoApplianceErrorCode | undefined;
     /**
      * Map a set bit in the Evt2 register. Reserved in the SunSpec spec, so
-     * the default emits a generic code with no translation. Override for
-     * vendor-specific Evt2 semantics.
+     * the default returns `undefined` — Evt2 bits are not treated as errors.
+     * Override in a vendor-specific subclass to opt specific bits in as
+     * errors.
      */
-    protected mapEvt2Bit(bit: number): EnyoApplianceErrorCode | undefined;
+    protected mapEvt2Bit(_bit: number): EnyoApplianceErrorCode | undefined;
     /**
      * Map a set bit in one of the four vendor-specific event registers.
-     * Override in a vendor-specific subclass to translate vendor bits.
-     * Return `undefined` to drop a bit entirely.
+     * Vendor bits are not standardized and many are informational rather
+     * than faults, so the default returns `undefined` and no error code is
+     * emitted. Override in a vendor-specific subclass to opt specific bits
+     * in as errors.
      */
-    protected mapVendorEventBit(registerIndex: 1 | 2 | 3 | 4, bit: number): EnyoApplianceErrorCode | undefined;
+    protected mapVendorEventBit(_registerIndex: 1 | 2 | 3 | 4, _bit: number): EnyoApplianceErrorCode | undefined;
     private hasErrorSetChanged;
     private buildStatusMessage;
     private storageKey;

package/dist/sunspec-devices.js

@@ -331,7 +331,7 @@ export class SunspecInverter extends BaseSunspecDevice {
             const dcStrings = this.mapMPPTToStrings(mpptDataList);
             if (inverterData) {
                 const pvPowerW = dcStrings.reduce((sum, s) => sum + (s.powerW || 0), 0);
-                console.log(`Got PV Power from DC strings: ${pvPowerW}W`);
+                console.debug(`Got PV Power from DC strings: ${pvPowerW}W`);
                 const inverterMessage = {
                     id: randomUUID(),
                     message: EnyoDataBusMessageEnum.InverterValuesUpdateV1,
@@ -421,19 +421,22 @@ export class SunspecInverter extends BaseSunspecDevice {
     }
     /**
      * Map a set bit in the Evt2 register. Reserved in the SunSpec spec, so
-     * the default emits a generic code with no translation. Override for
-     * vendor-specific Evt2 semantics.
+     * the default returns `undefined` — Evt2 bits are not treated as errors.
+     * Override in a vendor-specific subclass to opt specific bits in as
+     * errors.
      */
-    mapEvt2Bit(bit) {
-        return { code: `inverter_event2_bit_${bit}` };
+    mapEvt2Bit(_bit) {
+        return undefined;
     }
     /**
      * Map a set bit in one of the four vendor-specific event registers.
-     * Override in a vendor-specific subclass to translate vendor bits.
-     * Return `undefined` to drop a bit entirely.
+     * Vendor bits are not standardized and many are informational rather
+     * than faults, so the default returns `undefined` and no error code is
+     * emitted. Override in a vendor-specific subclass to opt specific bits
+     * in as errors.
      */
-    mapVendorEventBit(registerIndex, bit) {
-        return { code: `inverter_vendor_event${registerIndex}_bit_${bit}` };
+    mapVendorEventBit(_registerIndex, _bit) {
+        return undefined;
     }
     hasErrorSetChanged(newCodeIds) {
         const prev = this.errorState.activeCodes;

package/dist/sunspec-modbus-client.d.ts

@@ -32,6 +32,7 @@ export declare class SunspecModbusClient {
     private openCount;
     private closeCount;
     private currentlyOpen;
+    private operationChain;
     constructor(energyApp: EnergyApp);
     /**
      * Connect to Modbus device
@@ -55,10 +56,25 @@ export declare class SunspecModbusClient {
      */
     reconnect(): Promise<boolean>;
     /**
-     * Attempt to establish a connection to a specific host
-     * Returns true if successful, false otherwise
+     * Attempt to establish a connection to a specific host. Caller must hold the connection lock.
+     * Returns true if successful, false otherwise.
      */
     private attemptConnection;
+    /**
+     * Open a new Modbus socket and wire up the fault-tolerant reader. On any failure, the
+     * just-opened socket (if any) is closed so we never leak. Caller must hold the connection
+     * lock and ensure no prior socket is open (use _closeSocket first).
+     */
+    private _openSocket;
+    /**
+     * Close the current Modbus socket if any. Idempotent. Caller must hold the connection lock.
+     */
+    private _closeSocket;
+    /**
+     * Run `fn` with exclusive access to connection-state transitions. Subsequent calls queue
+     * behind any in-flight one. A rejected `fn` does not poison the chain for later callers.
+     */
+    private withConnectionLock;
     private recordOpen;
     private recordClose;
     /**

package/dist/sunspec-modbus-client.js

@@ -33,6 +33,9 @@ export class SunspecModbusClient {
     openCount = 0;
     closeCount = 0;
     currentlyOpen = 0;
+    // Serializes connection-state transitions so concurrent callers cannot open a second
+    // TCP socket while the first is still alive (some Modbus devices allow only one client).
+    operationChain = Promise.resolve();
     constructor(energyApp) {
         this.energyApp = energyApp;
         this.connectionHealth = new EnergyAppModbusConnectionHealth();
@@ -46,25 +49,23 @@ export class SunspecModbusClient {
      * @param secondaryHost Optional secondary host (ipAddress) for fallback during reconnection
      */
     async connect(host, port = 502, unitId = 1, secondaryHost) {
-        if (this.connected) {
-            await this.disconnect();
-        }
-        // Store connection parameters for potential reconnection
-        this.connectionParams = { primaryHost: host, secondaryHost, port, unitId };
-        this.modbusClient = await this.energyApp.useModbus().connect({
-            host,
-            port,
-            unitId,
-            timeout: 5000
+        return this.withConnectionLock(async () => {
+            const sameParams = this.connected
+                && this.connectionParams !== null
+                && this.connectionParams.primaryHost === host
+                && this.connectionParams.port === port
+                && this.connectionParams.unitId === unitId;
+            if (sameParams) {
+                console.debug(`connect(): already connected to ${host}:${port} unit ${unitId}, skipping`);
+                return;
+            }
+            if (this.connected || this.modbusClient) {
+                await this._closeSocket();
+            }
+            this.connectionParams = { primaryHost: host, secondaryHost, port, unitId };
+            await this._openSocket(host, port, unitId);
+            console.log(`Connected to Sunspec device at ${host}:${port} unit ${unitId} (opens=${this.openCount}, closes=${this.closeCount}, current=${this.currentlyOpen})`);
         });
-        // Create fault-tolerant reader with connection health monitoring
-        if (this.modbusClient) {
-            this.faultTolerantReader = new EnergyAppModbusFaultTolerantReader(this.modbusClient, this.connectionHealth);
-        }
-        this.connected = true;
-        this.connectionHealth.recordSuccess();
-        this.recordOpen();
-        console.log(`Connected to Sunspec device at ${host}:${port} unit ${unitId} (opens=${this.openCount}, closes=${this.closeCount}, current=${this.currentlyOpen})`);
     }
     /**
      * Disconnect from Modbus device.
@@ -73,18 +74,16 @@ export class SunspecModbusClient {
      * They will be overwritten by the next connect() call anyway.
      */
     async disconnect() {
-        if (this.modbusClient && this.connected) {
-            await this.modbusClient.disconnect();
-            this.modbusClient = null;
-            this.faultTolerantReader = null;
-            this.connected = false;
-            this.discoveredModels.clear();
-            this.recordClose();
+        return this.withConnectionLock(async () => {
+            if (!this.modbusClient && !this.connected) {
+                return;
+            }
             const host = this.connectionParams?.primaryHost ?? 'unknown';
             const port = this.connectionParams?.port ?? 0;
             const unitId = this.connectionParams?.unitId ?? 0;
+            await this._closeSocket();
             console.log(`Disconnected from Sunspec device at ${host}:${port} unit ${unitId} (opens=${this.openCount}, closes=${this.closeCount}, current=${this.currentlyOpen})`);
-        }
+        });
     }
     /**
      * Reconnect using stored connection parameters
@@ -92,78 +91,122 @@ export class SunspecModbusClient {
      * Returns true if reconnection was successful, false otherwise
      */
     async reconnect() {
-        if (!this.connectionParams) {
-            console.error('Cannot reconnect: no connection parameters stored. Call connect() first.');
+        return this.withConnectionLock(async () => {
+            if (!this.connectionParams) {
+                console.error('Cannot reconnect: no connection parameters stored. Call connect() first.');
+                return false;
+            }
+            const { primaryHost, secondaryHost, port, unitId } = this.connectionParams;
+            console.log(`Attempting to reconnect to primary host ${primaryHost}:${port} unit ${unitId}...`);
+            const primarySuccess = await this.attemptConnection(primaryHost, port, unitId);
+            if (primarySuccess) {
+                console.log(`Successfully reconnected to primary host ${primaryHost}:${port} unit ${unitId}`);
+                return true;
+            }
+            if (secondaryHost && secondaryHost !== primaryHost) {
+                console.log(`Primary host failed, attempting secondary host ${secondaryHost}:${port} unit ${unitId}...`);
+                const secondarySuccess = await this.attemptConnection(secondaryHost, port, unitId);
+                if (secondarySuccess) {
+                    console.log(`Successfully reconnected to secondary host ${secondaryHost}:${port} unit ${unitId}`);
+                    return true;
+                }
+            }
+            console.error(`Reconnection failed to all available hosts`);
             return false;
+        });
+    }
+    /**
+     * Attempt to establish a connection to a specific host. Caller must hold the connection lock.
+     * Returns true if successful, false otherwise.
+     */
+    async attemptConnection(host, port, unitId) {
+        if (this.connected || this.modbusClient) {
+            await this._closeSocket();
         }
-        const { primaryHost, secondaryHost, port, unitId } = this.connectionParams;
-        // Try primary host first
-        console.log(`Attempting to reconnect to primary host ${primaryHost}:${port} unit ${unitId}...`);
-        const primarySuccess = await this.attemptConnection(primaryHost, port, unitId);
-        if (primarySuccess) {
-            console.log(`Successfully reconnected to primary host ${primaryHost}:${port} unit ${unitId}`);
+        try {
+            await this._openSocket(host, port, unitId);
+            console.log(`Connection attempt to ${host}:${port} unit ${unitId} succeeded (opens=${this.openCount}, closes=${this.closeCount}, current=${this.currentlyOpen})`);
             return true;
         }
-        // If primary failed and secondary is available, try secondary
-        if (secondaryHost && secondaryHost !== primaryHost) {
-            console.log(`Primary host failed, attempting secondary host ${secondaryHost}:${port} unit ${unitId}...`);
-            const secondarySuccess = await this.attemptConnection(secondaryHost, port, unitId);
-            if (secondarySuccess) {
-                console.log(`Successfully reconnected to secondary host ${secondaryHost}:${port} unit ${unitId}`);
-                return true;
-            }
+        catch (error) {
+            console.error(`Connection attempt to ${host}:${port} failed: ${error}`);
+            return false;
         }
-        console.error(`Reconnection failed to all available hosts`);
-        this.connected = false;
-        return false;
     }
     /**
-     * Attempt to establish a connection to a specific host
-     * Returns true if successful, false otherwise
+     * Open a new Modbus socket and wire up the fault-tolerant reader. On any failure, the
+     * just-opened socket (if any) is closed so we never leak. Caller must hold the connection
+     * lock and ensure no prior socket is open (use _closeSocket first).
      */
-    async attemptConnection(host, port, unitId) {
+    async _openSocket(host, port, unitId) {
+        let candidate = null;
         try {
-            // Disconnect existing connection if any
-            if (this.modbusClient) {
-                const wasConnected = this.connected;
-                try {
-                    await this.modbusClient.disconnect();
-                }
-                catch (e) {
-                    // Ignore disconnect errors during reconnection
-                }
-                this.modbusClient = null;
-                this.faultTolerantReader = null;
-                if (wasConnected) {
-                    this.connected = false;
-                    this.recordClose();
-                }
-            }
-            // Attempt connection
-            this.modbusClient = await this.energyApp.useModbus().connect({
+            candidate = await this.energyApp.useModbus().connect({
                 host,
                 port,
                 unitId,
                 timeout: 5000
             });
-            // Create fault-tolerant reader with connection health monitoring
-            if (this.modbusClient) {
-                this.faultTolerantReader = new EnergyAppModbusFaultTolerantReader(this.modbusClient, this.connectionHealth);
+            if (!candidate) {
+                throw new Error(`useModbus().connect returned null for ${host}:${port} unit ${unitId}`);
             }
+            this.faultTolerantReader = new EnergyAppModbusFaultTolerantReader(candidate, this.connectionHealth);
+            this.modbusClient = candidate;
             this.connected = true;
             this.connectionHealth.recordSuccess();
             this.recordOpen();
-            console.log(`Connection attempt to ${host}:${port} unit ${unitId} succeeded (opens=${this.openCount}, closes=${this.closeCount}, current=${this.currentlyOpen})`);
-            return true;
         }
-        catch (error) {
-            console.error(`Connection attempt to ${host}:${port} failed: ${error}`);
-            return false;
+        catch (err) {
+            if (candidate) {
+                try {
+                    await candidate.disconnect();
+                }
+                catch { /* ignore */ }
+            }
+            this.modbusClient = null;
+            this.faultTolerantReader = null;
+            this.connected = false;
+            throw err;
         }
     }
+    /**
+     * Close the current Modbus socket if any. Idempotent. Caller must hold the connection lock.
+     */
+    async _closeSocket() {
+        if (!this.modbusClient && !this.connected)
+            return;
+        const client = this.modbusClient;
+        const wasConnected = this.connected;
+        this.modbusClient = null;
+        this.faultTolerantReader = null;
+        this.connected = false;
+        this.discoveredModels.clear();
+        if (client) {
+            try {
+                await client.disconnect();
+            }
+            catch { /* ignore */ }
+        }
+        if (wasConnected)
+            this.recordClose();
+    }
+    /**
+     * Run `fn` with exclusive access to connection-state transitions. Subsequent calls queue
+     * behind any in-flight one. A rejected `fn` does not poison the chain for later callers.
+     */
+    withConnectionLock(fn) {
+        const run = this.operationChain.then(fn, fn);
+        this.operationChain = run.catch(() => undefined);
+        return run;
+    }
     recordOpen() {
         this.openCount++;
         this.currentlyOpen++;
+        if (this.currentlyOpen > 1) {
+            console.error(`SunspecModbusClient: currentlyOpen=${this.currentlyOpen} after open ` +
+                `(opens=${this.openCount}, closes=${this.closeCount}). ` +
+                `This indicates a code path bypassing the connection lock — please investigate.`);
+        }
     }
     recordClose() {
         this.closeCount++;
@@ -513,7 +556,7 @@ export class SunspecModbusClient {
     async readInverterData() {
         const model = this.findModel(SunspecModelId.Inverter3Phase);
         if (!model) {
-            console.log('Inverter model 103 not found, trying single phase model 101');
+            console.debug('Inverter model 103 not found, trying single phase model 101');
             const singlePhaseModel = this.findModel(SunspecModelId.InverterSinglePhase);
             if (!singlePhaseModel) {
                 console.error('No inverter model found');
@@ -522,10 +565,10 @@ export class SunspecModbusClient {
             console.warn('IMPORTANT: Working with single-phase inverter, but 3-phase is expected!');
             return this.readSinglePhaseInverterData(singlePhaseModel);
         }
-        console.log(`Found 3-phase inverter model 103 at address ${model.address} with length ${model.length}`);
+        console.debug(`Found 3-phase inverter model 103 at address ${model.address} with length ${model.length}`);
         try {
             // Read entire model block in a single Modbus call
-            console.log(`Reading Inverter Data from Model ${model.id} at base address: ${model.address}`);
+            console.debug(`Reading Inverter Data from Model ${model.id} at base address: ${model.address}`);
             const buffer = await this.readModelBlock(model);
             // Extract all scale factors from buffer
             const scaleFactors = this.extractInverterScaleFactors(buffer);
@@ -609,7 +652,7 @@ export class SunspecModbusClient {
      */
     async readSinglePhaseInverterData(model) {
         try {
-            console.log(`Reading Single-Phase Inverter Data from Model 101 at base address: ${model.address}`);
+            console.debug(`Reading Single-Phase Inverter Data from Model 101 at base address: ${model.address}`);
             // Read entire model block in a single Modbus call
             const buffer = await this.readModelBlock(model);
             // Extract scale factors from buffer
@@ -763,7 +806,7 @@ export class SunspecModbusClient {
     async readMPPTScaleFactors() {
         const model = this.findModel(SunspecModelId.MPPT);
         if (!model) {
-            console.log('MPPT model 160 not found');
+            console.debug('MPPT model 160 not found');
             return null;
         }
         try {
@@ -797,7 +840,7 @@ export class SunspecModbusClient {
             this.isUnimplementedValue(dcCurrentRaw, 'uint16') &&
             this.isUnimplementedValue(dcVoltageRaw, 'uint16') &&
             this.isUnimplementedValue(dcPowerRaw, 'uint16')) {
-            console.log(`MPPT module ${moduleId} appears to be unconnected (all values are 0xFFFF)`);
+            console.debug(`MPPT module ${moduleId} appears to be unconnected (all values are 0xFFFF)`);
             return null;
         }
         const temperatureScaleFactor = -1;
@@ -835,7 +878,7 @@ export class SunspecModbusClient {
     async readMPPTData(moduleId = 1) {
         const model = this.findModel(SunspecModelId.MPPT);
         if (!model) {
-            console.log('MPPT model 160 not found');
+            console.debug('MPPT model 160 not found');
             return null;
         }
         try {
@@ -856,7 +899,7 @@ export class SunspecModbusClient {
         const mpptData = [];
         const model = this.findModel(SunspecModelId.MPPT);
         if (!model) {
-            console.log('MPPT model 160 not found');
+            console.debug('MPPT model 160 not found');
             return [];
         }
         try {
@@ -868,16 +911,16 @@ export class SunspecModbusClient {
             const count = this.extractValue(buffer, 8, 'uint16');
             if (!this.isUnimplementedValue(count, 'uint16') && count > 0 && count <= 20) {
                 moduleCount = count;
-                console.log(`MPPT module count from register 8: ${moduleCount}`);
+                console.debug(`MPPT module count from register 8: ${moduleCount}`);
             }
             else {
-                console.log(`Invalid or unimplemented module count (${count}), using default: ${moduleCount}`);
+                console.debug(`Invalid or unimplemented module count (${count}), using default: ${moduleCount}`);
             }
             // Extract each MPPT module from the same buffer
             for (let i = 1; i <= moduleCount; i++) {
                 try {
                     const data = this.extractMPPTModuleData(buffer, model, i, scaleFactors);
-                    console.log(`MPPT ${i} has id ${data?.id} (${data?.stringId}) with ${data?.dcPower}W`);
+                    console.debug(`MPPT ${i} has id ${data?.id} (${data?.stringId}) with ${data?.dcPower}W`);
                     if (data &&
                         (data.dcCurrent !== undefined ||
                             data.dcVoltage !== undefined ||
@@ -1025,10 +1068,10 @@ export class SunspecModbusClient {
     async readBatteryBaseData() {
         const model = this.findModel(SunspecModelId.BatteryBase);
         if (!model) {
-            console.log('Battery Base model 802 not found');
+            console.debug('Battery Base model 802 not found');
             return null;
         }
-        console.log(`Reading Battery Base Data from Model 802 at base address: ${model.address}`);
+        console.debug(`Reading Battery Base Data from Model 802 at base address: ${model.address}`);
         try {
             // Read entire model block in a single Modbus call
             const buffer = await this.readModelBlock(model);
@@ -1171,14 +1214,14 @@ export class SunspecModbusClient {
             model = this.findModel(SunspecModelId.BatteryControl);
         }
         if (!model) {
-            console.log('No battery model found');
+            console.debug('No battery model found');
             return null;
         }
-        console.log(`Reading Battery Data from Model ${model.id} at base address: ${model.address}`);
+        console.debug(`Reading Battery Data from Model ${model.id} at base address: ${model.address}`);
         try {
             if (model.id === 124) {
                 // Model 124: Basic Storage Controls
-                console.log('Using Model 124 (Basic Storage Controls)');
+                console.debug('Using Model 124 (Basic Storage Controls)');
                 // Read entire model block in a single Modbus call
                 const buffer = await this.readModelBlock(model);
                 // Extract scale factors from buffer (offsets 18-25)
@@ -1268,19 +1311,19 @@ export class SunspecModbusClient {
                 // Calculate charge/discharge power if rates are available
                 if (data.inWRte !== undefined && data.wChaMax !== undefined) {
                     data.chargePower = (data.inWRte / 100) * data.wChaMax;
-                    console.log(`Calculated Charge Power: (inWRte: ${data.inWRte}, wChaMax: ${data.wChaMax}) ${data.chargePower?.toFixed(2)} W`);
+                    console.debug(`Calculated Charge Power: (inWRte: ${data.inWRte}, wChaMax: ${data.wChaMax}) ${data.chargePower?.toFixed(2)} W`);
                 }
                 if (data.outWRte !== undefined && data.wChaMax !== undefined) {
                     // Assuming WDisChaMax is similar to WChaMax for simplicity
                     data.dischargePower = Math.abs((data.outWRte / 100) * data.wChaMax);
-                    console.log(`Calculated Discharge Power (inWRte: ${data.outWRte}, wChaMax: ${data.wChaMax}): ${data.dischargePower?.toFixed(2)} W`);
+                    console.debug(`Calculated Discharge Power (inWRte: ${data.outWRte}, wChaMax: ${data.wChaMax}): ${data.dischargePower?.toFixed(2)} W`);
                 }
                 console.debug('[Model 124] Battery Data:', data);
                 return data;
             }
             else if (model.id === 802) {
                 // Model 802: Battery Base
-                console.log('Using Model 802 (Battery Base)');
+                console.debug('Using Model 802 (Battery Base)');
                 const baseData = await this.readBatteryBaseData();
                 if (!baseData) {
                     return null;
@@ -1317,7 +1360,7 @@ export class SunspecModbusClient {
             }
             else {
                 // Handle other battery models (803) if needed
-                console.log(`Battery Model ${model.id} reading not yet implemented`);
+                console.debug(`Battery Model ${model.id} reading not yet implemented`);
                 return {
                     blockNumber: model.id,
                     blockAddress: model.address,
@@ -1489,13 +1532,13 @@ export class SunspecModbusClient {
             model = this.findModel(SunspecModelId.MeterSinglePhase);
         }
         if (!model) {
-            console.log('No meter model found');
+            console.debug('No meter model found');
             return null;
         }
-        console.log(`Reading Meter Data from Model ${model.id} at base address: ${model.address}`);
+        console.debug(`Reading Meter Data from Model ${model.id} at base address: ${model.address}`);
         try {
             // Different meter models have different register offsets
-            console.log(`Meter is Model ${model.id}`);
+            console.debug(`Meter is Model ${model.id}`);
             let powerOffset;
             let powerSFOffset;
             let freqOffset;
@@ -1505,7 +1548,7 @@ export class SunspecModbusClient {
             let energySFOffset;
             switch (model.id) {
                 case 203: // 3-phase meter (Delta)
-                    console.log('Using Model 203 (3-Phase Meter Delta) offsets');
+                    console.debug('Using Model 203 (3-Phase Meter Delta) offsets');
                     powerOffset = 18; // W - Total Real Power (int16)
                     powerSFOffset = 22; // W_SF - Power scale factor
                     freqOffset = 16; // Hz - Frequency (uint16)
@@ -1515,7 +1558,7 @@ export class SunspecModbusClient {
                     energySFOffset = 54; // TotWh_SF - Total Energy scale factor
                     break;
                 case 204: // 3-phase meter (Wye)
-                    console.log('Using Model 204 (3-Phase Meter Wye) offsets');
+                    console.debug('Using Model 204 (3-Phase Meter Wye) offsets');
                     powerOffset = 18; // W - Total Real Power (int16)
                     powerSFOffset = 22; // W_SF - Power scale factor
                     freqOffset = 16; // Hz - Frequency (uint16)
@@ -1526,7 +1569,7 @@ export class SunspecModbusClient {
                     break;
                 case 201: // Single-phase meter
                 default:
-                    console.log('Using Model 201 (Single-Phase Meter) offsets');
+                    console.debug('Using Model 201 (Single-Phase Meter) offsets');
                     powerOffset = 18; // W - Total Real Power (int16)
                     powerSFOffset = 22; // W_SF - Power scale factor
                     freqOffset = 16; // Hz - Frequency (uint16)
@@ -1648,7 +1691,7 @@ export class SunspecModbusClient {
     async readInverterSettings() {
         const model = this.findModel(SunspecModelId.Settings);
         if (!model) {
-            console.log('Settings model 121 not found');
+            console.debug('Settings model 121 not found');
             return null;
         }
         try {

package/dist/version.d.ts

@@ -5,7 +5,7 @@
 /**
  * Current version of the enyo Energy App SDK.
  */
-export declare const SDK_VERSION = "0.0.49";
+export declare const SDK_VERSION = "0.0.51";
 /**
  * Gets the current SDK version.
  * @returns The semantic version string of the SDK

package/dist/version.js

@@ -5,7 +5,7 @@
 /**
  * Current version of the enyo Energy App SDK.
  */
-export const SDK_VERSION = '0.0.49';
+export const SDK_VERSION = '0.0.51';
 /**
  * Gets the current SDK version.
  * @returns The semantic version string of the SDK

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@enyo-energy/sunspec-sdk",
-  "version": "0.0.49",
+  "version": "0.0.51",
   "description": "enyo Energy Sunspec SDK",
   "type": "module",
   "main": "dist/index.js",