tsvesync 1.0.20 → 1.0.22

package/dist/lib/vesync.js

@@ -291,37 +291,49 @@ class VeSync {
      * Process devices from API response
      */
     processDevices(deviceList) {
-        // Clear existing devices
-        for (const category of Object.keys(this._devList)) {
-            this._devList[category].length = 0;
-        }
-        // Process each device
-        deviceList.forEach(dev => {
-            const [category, device] = objectFactory(dev, this);
-            // Handle outdoor plug sub-devices
-            if (dev.deviceType === 'ESO15-TB' && dev.subDevices) {
-                dev.subDevices.forEach((subDev) => {
-                    const subDeviceDetails = {
-                        ...dev,
-                        deviceName: subDev.subDeviceName,
-                        deviceStatus: subDev.subDeviceStatus,
-                        subDeviceNo: subDev.subDeviceNo,
-                        isSubDevice: true,
-                        parentCid: dev.cid,
-                        cid: `${dev.cid}_${subDev.subDeviceNo}`
-                    };
-                    const [subCategory, subDevice] = objectFactory(subDeviceDetails, this);
-                    if (subDevice && subCategory in this._devList) {
-                        this._devList[subCategory].push(subDevice);
-                    }
-                });
+        try {
+            // Clear existing devices
+            for (const category of Object.keys(this._devList)) {
+                this._devList[category].length = 0;
             }
-            else if (device && category in this._devList) {
-                this._devList[category].push(device);
+            if (!deviceList || deviceList.length === 0) {
+                logger_1.logger.warn('No devices found in API response');
+                return false;
             }
-        });
-        // Update device list reference
-        this.devices = Object.values(this._devList).flat();
+            // Process each device
+            deviceList.forEach(dev => {
+                const [category, device] = objectFactory(dev, this);
+                // Handle outdoor plug sub-devices
+                if (dev.deviceType === 'ESO15-TB' && dev.subDevices) {
+                    dev.subDevices.forEach((subDev) => {
+                        const subDeviceDetails = {
+                            ...dev,
+                            deviceName: subDev.subDeviceName,
+                            deviceStatus: subDev.subDeviceStatus,
+                            subDeviceNo: subDev.subDeviceNo,
+                            isSubDevice: true,
+                            parentCid: dev.cid,
+                            cid: `${dev.cid}_${subDev.subDeviceNo}`
+                        };
+                        const [subCategory, subDevice] = objectFactory(subDeviceDetails, this);
+                        if (subDevice && subCategory in this._devList) {
+                            this._devList[subCategory].push(subDevice);
+                        }
+                    });
+                }
+                else if (device && category in this._devList) {
+                    this._devList[category].push(device);
+                }
+            });
+            // Update device list reference
+            this.devices = Object.values(this._devList).flat();
+            // Return true if we processed at least one device successfully
+            return this.devices.length > 0;
+        }
+        catch (error) {
+            logger_1.logger.error('Error processing devices:', error);
+            return false;
+        }
     }
     /**
      * Get list of VeSync devices
@@ -333,12 +345,12 @@ class VeSync {
             return false;
         }
         this._inProcess = true;
-        let procReturn = false;
+        let success = false;
         try {
             const [response] = await helpers_1.Helpers.callApi('/cloud/v2/deviceManaged/devices', 'post', helpers_1.Helpers.reqBody(this, 'devicelist'), helpers_1.Helpers.reqHeaders(this));
             if ((_a = response === null || response === void 0 ? void 0 : response.result) === null || _a === void 0 ? void 0 : _a.list) {
                 const deviceList = response.result.list;
-                this.processDevices(deviceList);
+                success = this.processDevices(deviceList);
                 // Log device discovery results
                 logger_1.logger.debug('\n=== Device Discovery Summary ===');
                 logger_1.logger.debug(`Total devices processed: ${deviceList.length}`);
@@ -376,7 +388,7 @@ class VeSync {
             logger_1.logger.error('Unable to get device list:', error);
         }
         this._inProcess = false;
-        return procReturn;
+        return success;
     }
     /**
      * Login to VeSync server

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "tsvesync",
-  "version": "1.0.20",
+  "version": "1.0.22",
   "description": "A TypeScript library for interacting with VeSync smart home devices",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",

package/src/lib/vesync.ts

@@ -358,40 +358,53 @@ export class VeSync {
     /**
      * Process devices from API response
      */
-    private processDevices(deviceList: any[]): void {
-        // Clear existing devices
-        for (const category of Object.keys(this._devList)) {
-            this._devList[category].length = 0;
-        }
+    private processDevices(deviceList: any[]): boolean {
+        try {
+            // Clear existing devices
+            for (const category of Object.keys(this._devList)) {
+                this._devList[category].length = 0;
+            }
 
-        // Process each device
-        deviceList.forEach(dev => {
-            const [category, device] = objectFactory(dev, this);
-            
-            // Handle outdoor plug sub-devices
-            if (dev.deviceType === 'ESO15-TB' && dev.subDevices) {
-                dev.subDevices.forEach((subDev: any) => {
-                    const subDeviceDetails = {
-                        ...dev,
-                        deviceName: subDev.subDeviceName,
-                        deviceStatus: subDev.subDeviceStatus,
-                        subDeviceNo: subDev.subDeviceNo,
-                        isSubDevice: true,
-                        parentCid: dev.cid,
-                        cid: `${dev.cid}_${subDev.subDeviceNo}`
-                    };
-                    const [subCategory, subDevice] = objectFactory(subDeviceDetails, this);
-                    if (subDevice && subCategory in this._devList) {
-                        this._devList[subCategory].push(subDevice);
-                    }
-                });
-            } else if (device && category in this._devList) {
-                this._devList[category].push(device);
+            if (!deviceList || deviceList.length === 0) {
+                logger.warn('No devices found in API response');
+                return false;
             }
-        });
 
-        // Update device list reference
-        this.devices = Object.values(this._devList).flat();
+            // Process each device
+            deviceList.forEach(dev => {
+                const [category, device] = objectFactory(dev, this);
+                
+                // Handle outdoor plug sub-devices
+                if (dev.deviceType === 'ESO15-TB' && dev.subDevices) {
+                    dev.subDevices.forEach((subDev: any) => {
+                        const subDeviceDetails = {
+                            ...dev,
+                            deviceName: subDev.subDeviceName,
+                            deviceStatus: subDev.subDeviceStatus,
+                            subDeviceNo: subDev.subDeviceNo,
+                            isSubDevice: true,
+                            parentCid: dev.cid,
+                            cid: `${dev.cid}_${subDev.subDeviceNo}`
+                        };
+                        const [subCategory, subDevice] = objectFactory(subDeviceDetails, this);
+                        if (subDevice && subCategory in this._devList) {
+                            this._devList[subCategory].push(subDevice);
+                        }
+                    });
+                } else if (device && category in this._devList) {
+                    this._devList[category].push(device);
+                }
+            });
+
+            // Update device list reference
+            this.devices = Object.values(this._devList).flat();
+
+            // Return true if we processed at least one device successfully
+            return this.devices.length > 0;
+        } catch (error) {
+            logger.error('Error processing devices:', error);
+            return false;
+        }
     }
 
     /**
@@ -404,7 +417,7 @@ export class VeSync {
         }
 
         this._inProcess = true;
-        let procReturn = false;
+        let success = false;
 
         try {
             const [response] = await Helpers.callApi(
@@ -416,7 +429,7 @@ export class VeSync {
 
             if (response?.result?.list) {
                 const deviceList = response.result.list;
-                this.processDevices(deviceList);
+                success = this.processDevices(deviceList);
 
                 // Log device discovery results
                 logger.debug('\n=== Device Discovery Summary ===');
@@ -458,7 +471,7 @@ export class VeSync {
         }
 
         this._inProcess = false;
-        return procReturn;
+        return success;
     }
 
     /**