@enyo-energy/sunspec-sdk 0.0.61 → 0.0.62

package/dist/cjs/connection-retry-manager.cjs

@@ -6,11 +6,12 @@
  * Called on each readData() cycle; determines whether enough time has elapsed
  * to attempt a reconnection based on the current retry phase.
  *
- * Default schedule:
+ * Default schedule (infinite retry, max 15m backoff):
  * - Phase 1: every 10s for 1 minute
  * - Phase 2: every 30s for 2 minutes
  * - Phase 3: every 1m for 5 minutes
- * - Phase 4: every 5m forever
+ * - Phase 4: every 5m for 10 minutes
+ * - Phase 5: every 15m forever
  */
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.ConnectionRetryManager = void 0;

package/dist/cjs/connection-retry-manager.d.cts

@@ -5,11 +5,12 @@
  * Called on each readData() cycle; determines whether enough time has elapsed
  * to attempt a reconnection based on the current retry phase.
  *
- * Default schedule:
+ * Default schedule (infinite retry, max 15m backoff):
  * - Phase 1: every 10s for 1 minute
  * - Phase 2: every 30s for 2 minutes
  * - Phase 3: every 1m for 5 minutes
- * - Phase 4: every 5m forever
+ * - Phase 4: every 5m for 10 minutes
+ * - Phase 5: every 15m forever
  */
 import { IRetryConfig, IRetryPhase } from './sunspec-interfaces.cjs';
 export declare class ConnectionRetryManager {

package/dist/cjs/sunspec-devices.cjs

@@ -187,6 +187,24 @@ class BaseSunspecDevice {
     async onConnectionFailure(_consecutiveFailures) {
         // Default: no-op. SunspecInverter overrides to publish a faulted status.
     }
+    /**
+     * Detect a silent read failure: the SDK's per-model readers swallow modbus errors and
+     * return null/[] rather than throwing, so a readData() cycle can complete "successfully"
+     * with no usable data while the underlying connection is broken. Once
+     * `connectionHealth.isHealthy()` flips to false, treat the device as offline so the
+     * appliance state is updated and the retry manager starts the backoff schedule.
+     *
+     * Returns true if the device was marked offline.
+     */
+    async markOfflineIfUnhealthy() {
+        if (this.sunspecClient.isHealthy(this.unitId)) {
+            return false;
+        }
+        console.warn(`${this.constructor.name} ${this.applianceId}: connection unhealthy after read cycle ` +
+            `(consecutiveFailures=${this.sunspecClient.getConnectionHealth().getConsecutiveFailures()}) — marking offline`);
+        await this.markOffline();
+        return true;
+    }
     /**
      * Mark the device as offline: close the underlying socket so the next readData()
      * cycle sees isConnected() === false and tryReconnect() can establish a fresh
@@ -338,6 +356,11 @@ class SunspecInverter extends BaseSunspecDevice {
             const mpptDataList = await this.sunspecClient.readAllMPPTData(this.unitId);
             const inverterSettings = await this.sunspecClient.readInverterSettings(this.unitId);
             const dcStrings = this.mapMPPTToStrings(mpptDataList);
+            // SDK readers swallow modbus errors and return null/[]; detect that here so the
+            // appliance is marked offline and the retry manager starts its backoff.
+            if (await this.markOfflineIfUnhealthy()) {
+                return messages;
+            }
             if (inverterData) {
                 const pvPowerW = dcStrings.reduce((sum, s) => sum + (s.powerW || 0), 0);
                 console.debug(`Got PV Power from DC strings: ${pvPowerW}W`);
@@ -806,6 +829,11 @@ class SunspecBattery extends BaseSunspecDevice {
             const batteryData = await this.sunspecClient.readBatteryData(this.unitId);
             const mpptDataList = await this.sunspecClient.readAllMPPTData(this.unitId);
             const mpptBatteryPowerW = this.extractBatteryPowerFromMPPT(mpptDataList);
+            // SDK readers swallow modbus errors and return null/[]; detect that here so the
+            // appliance is marked offline and the retry manager starts its backoff.
+            if (await this.markOfflineIfUnhealthy()) {
+                return messages;
+            }
             if (batteryData) {
                 const advancedBatteryModel = this.sunspecClient.findModel(this.unitId, 801);
                 const batteryBaseModel = this.sunspecClient.findModel(this.unitId, sunspec_interfaces_js_1.SunspecModelId.BatteryBase);
@@ -1291,6 +1319,11 @@ class SunspecMeter extends BaseSunspecDevice {
         try {
             // Read meter data
             const meterData = await this.sunspecClient.readMeterData(this.unitId);
+            // SDK readers swallow modbus errors and return null; detect that here so the
+            // appliance is marked offline and the retry manager starts its backoff.
+            if (await this.markOfflineIfUnhealthy()) {
+                return messages;
+            }
             if (meterData) {
                 const meterMessage = {
                     id: (0, node_crypto_1.randomUUID)(),

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

@@ -60,6 +60,16 @@ export declare abstract class BaseSunspecDevice {
      * per failed attempt with the running consecutive-failure count.
      */
     protected onConnectionFailure(_consecutiveFailures: number): Promise<void>;
+    /**
+     * Detect a silent read failure: the SDK's per-model readers swallow modbus errors and
+     * return null/[] rather than throwing, so a readData() cycle can complete "successfully"
+     * with no usable data while the underlying connection is broken. Once
+     * `connectionHealth.isHealthy()` flips to false, treat the device as offline so the
+     * appliance state is updated and the retry manager starts the backoff schedule.
+     *
+     * Returns true if the device was marked offline.
+     */
+    protected markOfflineIfUnhealthy(): Promise<boolean>;
     /**
      * Mark the device as offline: close the underlying socket so the next readData()
      * cycle sees isConnected() === false and tryReconnect() can establish a fresh

package/dist/cjs/sunspec-interfaces.cjs

@@ -9,7 +9,8 @@ exports.DEFAULT_RETRY_CONFIG = {
         { intervalMs: 10_000, durationMs: 60_000 }, // Phase 1: every 10s for 1 minute
         { intervalMs: 30_000, durationMs: 120_000 }, // Phase 2: every 30s for 2 minutes
         { intervalMs: 60_000, durationMs: 300_000 }, // Phase 3: every 1m for 5 minutes
-        { intervalMs: 300_000, durationMs: 0 }, // Phase 4: every 5m forever
+        { intervalMs: 300_000, durationMs: 600_000 }, // Phase 4: every 5m for 10 minutes
+        { intervalMs: 900_000, durationMs: 0 }, // Phase 5: every 15m forever
     ]
 };
 /**

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.61';
+exports.SDK_VERSION = '0.0.62';
 /**
  * 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.61";
+export declare const SDK_VERSION = "0.0.62";
 /**
  * Gets the current SDK version.
  * @returns The semantic version string of the SDK

package/dist/connection-retry-manager.d.ts

@@ -5,11 +5,12 @@
  * Called on each readData() cycle; determines whether enough time has elapsed
  * to attempt a reconnection based on the current retry phase.
  *
- * Default schedule:
+ * Default schedule (infinite retry, max 15m backoff):
  * - Phase 1: every 10s for 1 minute
  * - Phase 2: every 30s for 2 minutes
  * - Phase 3: every 1m for 5 minutes
- * - Phase 4: every 5m forever
+ * - Phase 4: every 5m for 10 minutes
+ * - Phase 5: every 15m forever
  */
 import { IRetryConfig, IRetryPhase } from './sunspec-interfaces.js';
 export declare class ConnectionRetryManager {

package/dist/connection-retry-manager.js

@@ -5,11 +5,12 @@
  * Called on each readData() cycle; determines whether enough time has elapsed
  * to attempt a reconnection based on the current retry phase.
  *
- * Default schedule:
+ * Default schedule (infinite retry, max 15m backoff):
  * - Phase 1: every 10s for 1 minute
  * - Phase 2: every 30s for 2 minutes
  * - Phase 3: every 1m for 5 minutes
- * - Phase 4: every 5m forever
+ * - Phase 4: every 5m for 10 minutes
+ * - Phase 5: every 15m forever
  */
 import { DEFAULT_RETRY_CONFIG } from './sunspec-interfaces.js';
 export class ConnectionRetryManager {

package/dist/sunspec-devices.d.ts

@@ -60,6 +60,16 @@ export declare abstract class BaseSunspecDevice {
      * per failed attempt with the running consecutive-failure count.
      */
     protected onConnectionFailure(_consecutiveFailures: number): Promise<void>;
+    /**
+     * Detect a silent read failure: the SDK's per-model readers swallow modbus errors and
+     * return null/[] rather than throwing, so a readData() cycle can complete "successfully"
+     * with no usable data while the underlying connection is broken. Once
+     * `connectionHealth.isHealthy()` flips to false, treat the device as offline so the
+     * appliance state is updated and the retry manager starts the backoff schedule.
+     *
+     * Returns true if the device was marked offline.
+     */
+    protected markOfflineIfUnhealthy(): Promise<boolean>;
     /**
      * Mark the device as offline: close the underlying socket so the next readData()
      * cycle sees isConnected() === false and tryReconnect() can establish a fresh

package/dist/sunspec-devices.js

@@ -184,6 +184,24 @@ export class BaseSunspecDevice {
     async onConnectionFailure(_consecutiveFailures) {
         // Default: no-op. SunspecInverter overrides to publish a faulted status.
     }
+    /**
+     * Detect a silent read failure: the SDK's per-model readers swallow modbus errors and
+     * return null/[] rather than throwing, so a readData() cycle can complete "successfully"
+     * with no usable data while the underlying connection is broken. Once
+     * `connectionHealth.isHealthy()` flips to false, treat the device as offline so the
+     * appliance state is updated and the retry manager starts the backoff schedule.
+     *
+     * Returns true if the device was marked offline.
+     */
+    async markOfflineIfUnhealthy() {
+        if (this.sunspecClient.isHealthy(this.unitId)) {
+            return false;
+        }
+        console.warn(`${this.constructor.name} ${this.applianceId}: connection unhealthy after read cycle ` +
+            `(consecutiveFailures=${this.sunspecClient.getConnectionHealth().getConsecutiveFailures()}) — marking offline`);
+        await this.markOffline();
+        return true;
+    }
     /**
      * Mark the device as offline: close the underlying socket so the next readData()
      * cycle sees isConnected() === false and tryReconnect() can establish a fresh
@@ -334,6 +352,11 @@ export class SunspecInverter extends BaseSunspecDevice {
             const mpptDataList = await this.sunspecClient.readAllMPPTData(this.unitId);
             const inverterSettings = await this.sunspecClient.readInverterSettings(this.unitId);
             const dcStrings = this.mapMPPTToStrings(mpptDataList);
+            // SDK readers swallow modbus errors and return null/[]; detect that here so the
+            // appliance is marked offline and the retry manager starts its backoff.
+            if (await this.markOfflineIfUnhealthy()) {
+                return messages;
+            }
             if (inverterData) {
                 const pvPowerW = dcStrings.reduce((sum, s) => sum + (s.powerW || 0), 0);
                 console.debug(`Got PV Power from DC strings: ${pvPowerW}W`);
@@ -801,6 +824,11 @@ export class SunspecBattery extends BaseSunspecDevice {
             const batteryData = await this.sunspecClient.readBatteryData(this.unitId);
             const mpptDataList = await this.sunspecClient.readAllMPPTData(this.unitId);
             const mpptBatteryPowerW = this.extractBatteryPowerFromMPPT(mpptDataList);
+            // SDK readers swallow modbus errors and return null/[]; detect that here so the
+            // appliance is marked offline and the retry manager starts its backoff.
+            if (await this.markOfflineIfUnhealthy()) {
+                return messages;
+            }
             if (batteryData) {
                 const advancedBatteryModel = this.sunspecClient.findModel(this.unitId, 801);
                 const batteryBaseModel = this.sunspecClient.findModel(this.unitId, SunspecModelId.BatteryBase);
@@ -1285,6 +1313,11 @@ export class SunspecMeter extends BaseSunspecDevice {
         try {
             // Read meter data
             const meterData = await this.sunspecClient.readMeterData(this.unitId);
+            // SDK readers swallow modbus errors and return null; detect that here so the
+            // appliance is marked offline and the retry manager starts its backoff.
+            if (await this.markOfflineIfUnhealthy()) {
+                return messages;
+            }
             if (meterData) {
                 const meterMessage = {
                     id: randomUUID(),

package/dist/sunspec-interfaces.js

@@ -6,7 +6,8 @@ export const DEFAULT_RETRY_CONFIG = {
         { intervalMs: 10_000, durationMs: 60_000 }, // Phase 1: every 10s for 1 minute
         { intervalMs: 30_000, durationMs: 120_000 }, // Phase 2: every 30s for 2 minutes
         { intervalMs: 60_000, durationMs: 300_000 }, // Phase 3: every 1m for 5 minutes
-        { intervalMs: 300_000, durationMs: 0 }, // Phase 4: every 5m forever
+        { intervalMs: 300_000, durationMs: 600_000 }, // Phase 4: every 5m for 10 minutes
+        { intervalMs: 900_000, durationMs: 0 }, // Phase 5: every 15m forever
     ]
 };
 /**

package/dist/version.d.ts

@@ -5,7 +5,7 @@
 /**
  * Current version of the enyo Energy App SDK.
  */
-export declare const SDK_VERSION = "0.0.61";
+export declare const SDK_VERSION = "0.0.62";
 /**
  * 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.61';
+export const SDK_VERSION = '0.0.62';
 /**
  * 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.61",
+  "version": "0.0.62",
   "description": "enyo Energy Sunspec SDK",
   "type": "module",
   "main": "dist/index.js",