tsvesync 1.0.25 → 1.0.27

package/dist/lib/vesync.js

@@ -373,6 +373,7 @@ class VeSync {
             }
             else {
                 logger_1.logger.error('No devices found in response');
+                logger_1.logger.error(response.code, response.msg);
             }
         }
         catch (error) {

package/dist/lib/vesyncBaseDevice.d.ts

@@ -35,11 +35,11 @@ export declare abstract class VeSyncBaseDevice {
     /**
      * Get device details
      */
-    abstract getDetails(): Promise<void>;
+    abstract getDetails(): Promise<Boolean>;
     /**
      * Update device details
      */
-    update(): Promise<void>;
+    update(): Promise<Boolean>;
     /**
      * Call API with proper headers
      */

package/dist/lib/vesyncBaseDevice.js

@@ -84,7 +84,7 @@ class VeSyncBaseDevice {
      * Update device details
      */
     async update() {
-        await this.getDetails();
+        return await this.getDetails();
     }
     /**
      * Call API with proper headers

package/dist/lib/vesyncBulb.d.ts

@@ -31,11 +31,11 @@ export declare abstract class VeSyncBulb extends VeSyncBaseDevice {
     /**
      * Get bulb details
      */
-    getDetails(): Promise<void>;
+    getDetails(): Promise<Boolean>;
     /**
-     * Update bulb status
+     * Update bulb details
      */
-    update(): Promise<void>;
+    update(): Promise<Boolean>;
     /**
      * Turn bulb on
      */

package/dist/lib/vesyncBulb.js

@@ -74,8 +74,9 @@ class VeSyncBulb extends vesyncBaseDevice_1.VeSyncBaseDevice {
                 getLightStatus: 'get'
             }
         };
-        const [response] = await this.callApi(endpoint, 'post', body, helpers_1.Helpers.reqHeaders(this.manager));
-        if ((response === null || response === void 0 ? void 0 : response.code) === 0 && (response === null || response === void 0 ? void 0 : response.result)) {
+        const [response, statusCode] = await this.callApi(endpoint, 'post', body, helpers_1.Helpers.reqHeaders(this.manager));
+        const success = this.checkResponse([response, statusCode], 'getDetails');
+        if (success && (response === null || response === void 0 ? void 0 : response.result)) {
             this.deviceStatus = response.result.enabled ? 'on' : 'off';
             this.brightness = response.result.brightness || this.brightness;
             if (this.features.includes('color_temp')) {
@@ -97,17 +98,16 @@ class VeSyncBulb extends vesyncBaseDevice_1.VeSyncBaseDevice {
             }
             logger_1.logger.debug(`[${this.deviceName}] Successfully retrieved bulb details`);
         }
-        else {
-            logger_1.logger.error(`[${this.deviceName}] Failed to get bulb details: ${JSON.stringify(response)}`);
-        }
+        return success;
     }
     /**
-     * Update bulb status
+     * Update bulb details
      */
     async update() {
         logger_1.logger.debug(`[${this.deviceName}] Updating bulb information`);
-        await this.getDetails();
+        const success = await this.getDetails();
         logger_1.logger.info(`[${this.deviceName}] Successfully updated bulb information`);
+        return success;
     }
     /**
      * Turn bulb on

package/dist/lib/vesyncFan.d.ts

@@ -71,7 +71,7 @@ export declare abstract class VeSyncFan extends VeSyncBaseDevice {
      * Return JSON details for fan
      */
     displayJSON(): string;
-    abstract getDetails(): Promise<void>;
+    abstract getDetails(): Promise<Boolean>;
     abstract turnOn(): Promise<boolean>;
     abstract turnOff(): Promise<boolean>;
     abstract changeFanSpeed(speed: number): Promise<boolean>;

package/dist/lib/vesyncFanImpl.d.ts

@@ -15,7 +15,7 @@ export declare class VeSyncAirBypass extends VeSyncFan {
     /**
      * Get device details
      */
-    getDetails(): Promise<void>;
+    getDetails(): Promise<Boolean>;
     /**
      * Turn device on
      */
@@ -93,7 +93,7 @@ export declare class VeSyncHumidifier extends VeSyncFan {
     /**
      * Get device details
      */
-    getDetails(): Promise<void>;
+    getDetails(): Promise<Boolean>;
     /**
      * Turn device on
      */

package/dist/lib/vesyncFanImpl.js

@@ -35,7 +35,8 @@ class VeSyncAirBypass extends vesyncFan_1.VeSyncFan {
                 source: 'APP'
             }
         }, helpers_1.Helpers.reqHeaderBypass());
-        if (this.checkResponse([response, status], 'getDetails')) {
+        const success = this.checkResponse([response, status], 'getDetails');
+        if (success) {
             const result = (_a = response === null || response === void 0 ? void 0 : response.result) === null || _a === void 0 ? void 0 : _a.result;
             if (result) {
                 // Update device status
@@ -56,9 +57,7 @@ class VeSyncAirBypass extends vesyncFan_1.VeSyncFan {
                 logger_1.logger.debug(`Successfully got details for device: ${this.deviceName}`);
             }
         }
-        else {
-            logger_1.logger.error(`Failed to get details for device: ${this.deviceName}`);
-        }
+        return success;
     }
     /**
      * Turn device on
@@ -398,6 +397,7 @@ class VeSyncHumidifier extends vesyncFan_1.VeSyncFan {
      * Get device details
      */
     async getDetails() {
+        var _a;
         logger_1.logger.debug(`Getting details for device: ${this.deviceName}`);
         const [response, status] = await this.callApi('/cloud/v2/deviceManaged/bypassV2', 'post', {
             ...helpers_1.Helpers.reqBody(this.manager, 'bypassV2'),
@@ -410,13 +410,11 @@ class VeSyncHumidifier extends vesyncFan_1.VeSyncFan {
             }
         }, helpers_1.Helpers.reqHeaderBypass());
         const success = this.checkResponse([response, status], 'getDetails');
-        if (success) {
+        if (success && ((_a = response === null || response === void 0 ? void 0 : response.result) === null || _a === void 0 ? void 0 : _a.result)) {
             this.details = response.result.result;
             logger_1.logger.debug(`Successfully got details for device: ${this.deviceName}`);
         }
-        else {
-            logger_1.logger.error(`Failed to get details for device: ${this.deviceName}`);
-        }
+        return success;
     }
     /**
      * Turn device on

package/dist/lib/vesyncOutlet.d.ts

@@ -21,7 +21,7 @@ export declare abstract class VeSyncOutlet extends VeSyncBaseDevice {
     /**
      * Get outlet details
      */
-    getDetails(): Promise<void>;
+    getDetails(): Promise<Boolean>;
     /**
      * Update outlet energy data
      */
@@ -53,7 +53,7 @@ export declare abstract class VeSyncOutlet extends VeSyncBaseDevice {
     /**
      * Update outlet details and energy info
      */
-    update(): Promise<void>;
+    update(): Promise<Boolean>;
     /**
      * Check if outlet has nightlight feature
      */

package/dist/lib/vesyncOutlet.js

@@ -66,15 +66,15 @@ class VeSyncOutlet extends vesyncBaseDevice_1.VeSyncBaseDevice {
         else {
             url = '/v1/device/devicedetail';
         }
-        const [response] = await this.callApi(url, this.deviceType === 'wifi-switch-1.3' ? 'get' : 'post', this.deviceType === 'wifi-switch-1.3' ? null : body, helpers_1.Helpers.reqHeaders(this.manager));
+        const [response, statusCode] = await this.callApi(url, this.deviceType === 'wifi-switch-1.3' ? 'get' : 'post', this.deviceType === 'wifi-switch-1.3' ? null : body, helpers_1.Helpers.reqHeaders(this.manager));
         if (!response) {
             logger_1.logger.debug(`[${this.deviceName}] No response received from API`);
-            return;
+            return false;
         }
         // Handle error responses
         if (response.error) {
             logger_1.logger.debug(`[${this.deviceName}] Failed to get outlet details: ${JSON.stringify(response)}`);
-            return;
+            return false;
         }
         // Handle successful responses
         if (response.code === 0) {
@@ -102,7 +102,7 @@ class VeSyncOutlet extends vesyncBaseDevice_1.VeSyncBaseDevice {
                 if (subDevice) {
                     this.deviceStatus = subDevice.subDeviceStatus;
                     logger_1.logger.debug(`[${this.deviceName}] Successfully retrieved sub-device status: ${this.deviceStatus}`);
-                    return;
+                    return true;
                 }
             }
             // Handle regular devices
@@ -117,12 +117,14 @@ class VeSyncOutlet extends vesyncBaseDevice_1.VeSyncBaseDevice {
             }
             else {
                 logger_1.logger.debug(`[${this.deviceName}] Device status not found in response: ${JSON.stringify(response)}`);
-                return;
+                return false;
             }
             logger_1.logger.debug(`[${this.deviceName}] Successfully retrieved outlet details`);
+            return true;
         }
         else {
             logger_1.logger.debug(`[${this.deviceName}] Failed to get outlet details: ${JSON.stringify(response)}`);
+            return false;
         }
     }
     /**
@@ -313,11 +315,12 @@ class VeSyncOutlet extends vesyncBaseDevice_1.VeSyncBaseDevice {
      */
     async update() {
         logger_1.logger.debug(`[${this.deviceName}] Updating outlet information`);
-        await this.getDetails();
-        if (this.features.includes('energy')) {
+        const success = await this.getDetails();
+        if (success && this.features.includes('energy')) {
             await this.updateEnergy();
         }
         logger_1.logger.info(`[${this.deviceName}] Successfully updated outlet information`);
+        return success;
     }
     /**
      * Check if outlet has nightlight feature

package/dist/lib/vesyncSwitch.d.ts

@@ -33,13 +33,16 @@ export declare abstract class VeSyncSwitch extends VeSyncBaseDevice {
      * Return JSON details for switch
      */
     displayJSON(): string;
-    abstract getDetails(): Promise<void>;
+    /**
+     * Get switch details
+     */
+    abstract getDetails(): Promise<Boolean>;
+    /**
+     * Update switch details
+     */
+    update(): Promise<Boolean>;
     abstract getConfig(): Promise<void>;
     abstract turnOn(): Promise<boolean>;
     abstract turnOff(): Promise<boolean>;
-    /**
-     * Update device details
-     */
-    update(): Promise<void>;
 }
 export {};

package/dist/lib/vesyncSwitch.js

@@ -69,10 +69,13 @@ class VeSyncSwitch extends vesyncBaseDevice_1.VeSyncBaseDevice {
         }, null, 4);
     }
     /**
-     * Update device details
+     * Update switch details
      */
     async update() {
-        await this.getDetails();
+        logger_1.logger.debug(`[${this.deviceName}] Updating switch information`);
+        const success = await this.getDetails();
+        logger_1.logger.debug(`[${this.deviceName}] Successfully updated switch information`);
+        return success;
     }
 }
 exports.VeSyncSwitch = VeSyncSwitch;

package/dist/lib/vesyncSwitchImpl.d.ts

@@ -11,7 +11,7 @@ export declare class VeSyncWallSwitch extends VeSyncSwitch {
     /**
      * Get wall switch details
      */
-    getDetails(): Promise<void>;
+    getDetails(): Promise<Boolean>;
     /**
      * Get switch device configuration info
      */
@@ -37,7 +37,7 @@ export declare class VeSyncDimmerSwitch extends VeSyncSwitch {
     /**
      * Get dimmer switch details
      */
-    getDetails(): Promise<void>;
+    getDetails(): Promise<Boolean>;
     /**
      * Get dimmer switch configuration info
      */

package/dist/lib/vesyncSwitchImpl.js

@@ -25,17 +25,16 @@ class VeSyncWallSwitch extends vesyncSwitch_1.VeSyncSwitch {
             uuid: this.cid,
             method: 'devicedetail'
         };
-        const [response] = await this.callApi('/inwallswitch/v1/device/devicedetail', 'post', body, helpers_1.Helpers.reqHeaders(this.manager));
-        if ((response === null || response === void 0 ? void 0 : response.code) === 0 && (response === null || response === void 0 ? void 0 : response.result)) {
+        const [response, statusCode] = await this.callApi('/inwallswitch/v1/device/devicedetail', 'post', body, helpers_1.Helpers.reqHeaders(this.manager));
+        const success = this.checkResponse([response, statusCode], 'getDetails');
+        if (success && (response === null || response === void 0 ? void 0 : response.result)) {
             const result = response.result;
             this.deviceStatus = result.deviceStatus || this.deviceStatus;
             this.details.active_time = result.activeTime || 0;
             this.connectionStatus = result.connectionStatus || this.connectionStatus;
             logger_1.logger.debug(`Successfully got details for device: ${this.deviceName}`);
         }
-        else {
-            logger_1.logger.error(`Failed to get details for device: ${this.deviceName}`);
-        }
+        return success;
     }
     /**
      * Get switch device configuration info
@@ -118,8 +117,9 @@ class VeSyncDimmerSwitch extends vesyncSwitch_1.VeSyncSwitch {
             uuid: this.cid,
             method: 'devicedetail'
         };
-        const [response] = await this.callApi('/dimmer/v1/device/devicedetail', 'post', body, helpers_1.Helpers.reqHeaders(this.manager));
-        if ((response === null || response === void 0 ? void 0 : response.code) === 0 && (response === null || response === void 0 ? void 0 : response.result)) {
+        const [response, statusCode] = await this.callApi('/dimmer/v1/device/devicedetail', 'post', body, helpers_1.Helpers.reqHeaders(this.manager));
+        const success = this.checkResponse([response, statusCode], 'getDetails');
+        if (success && (response === null || response === void 0 ? void 0 : response.result)) {
             const result = response.result;
             this.deviceStatus = result.deviceStatus || this.deviceStatus;
             this.details.active_time = result.activeTime || 0;
@@ -136,9 +136,7 @@ class VeSyncDimmerSwitch extends vesyncSwitch_1.VeSyncSwitch {
             }
             logger_1.logger.debug(`Successfully got details for device: ${this.deviceName}`);
         }
-        else {
-            logger_1.logger.error(`Failed to get details for device: ${this.deviceName}`);
-        }
+        return success;
     }
     /**
      * Get dimmer switch configuration info

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "tsvesync",
-  "version": "1.0.25",
+  "version": "1.0.27",
   "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

@@ -456,6 +456,7 @@ export class VeSync {
                 logger.debug('\n=== End of Device Discovery ===\n');
             } else {
                 logger.error('No devices found in response');
+                logger.error(response.code, response.msg);
             }
         } catch (error) {
             logger.error('Unable to get device list:', error);

package/src/lib/vesyncBaseDevice.ts

@@ -105,13 +105,13 @@ export abstract class VeSyncBaseDevice {
     /**
      * Get device details
      */
-    abstract getDetails(): Promise<void>;
+    abstract getDetails(): Promise<Boolean>;
 
     /**
      * Update device details
      */
-    async update(): Promise<void> {
-        await this.getDetails();
+    async update(): Promise<Boolean> {
+        return await this.getDetails();
     }
 
     /**

package/src/lib/vesyncBulb.ts

@@ -71,7 +71,7 @@ export abstract class VeSyncBulb extends VeSyncBaseDevice {
     /**
      * Get bulb details
      */
-    async getDetails(): Promise<void> {
+    async getDetails(): Promise<Boolean> {
         logger.debug(`[${this.deviceName}] Getting bulb details`);
         
         const isV2Device = this.deviceType === 'XYD0001';
@@ -94,14 +94,15 @@ export abstract class VeSyncBulb extends VeSyncBaseDevice {
             }
         };
 
-        const [response] = await this.callApi(
+        const [response, statusCode] = await this.callApi(
             endpoint,
             'post',
             body,
             Helpers.reqHeaders(this.manager)
         );
 
-        if (response?.code === 0 && response?.result) {
+        const success = this.checkResponse([response, statusCode], 'getDetails');
+        if (success && response?.result) {
             this.deviceStatus = response.result.enabled ? 'on' : 'off';
             this.brightness = response.result.brightness || this.brightness;
             if (this.features.includes('color_temp')) {
@@ -121,18 +122,18 @@ export abstract class VeSyncBulb extends VeSyncBaseDevice {
                 }
             }
             logger.debug(`[${this.deviceName}] Successfully retrieved bulb details`);
-        } else {
-            logger.error(`[${this.deviceName}] Failed to get bulb details: ${JSON.stringify(response)}`);
         }
+        return success;
     }
 
     /**
-     * Update bulb status
+     * Update bulb details
      */
-    async update(): Promise<void> {
+    async update(): Promise<Boolean> {
         logger.debug(`[${this.deviceName}] Updating bulb information`);
-        await this.getDetails();
+        const success = await this.getDetails();
         logger.info(`[${this.deviceName}] Successfully updated bulb information`);
+        return success;
     }
 
     /**

package/src/lib/vesyncFan.ts

@@ -437,7 +437,7 @@ export abstract class VeSyncFan extends VeSyncBaseDevice {
         }, null, 4);
     }
 
-    abstract getDetails(): Promise<void>;
+    abstract getDetails(): Promise<Boolean>;
     abstract turnOn(): Promise<boolean>;
     abstract turnOff(): Promise<boolean>;
     abstract changeFanSpeed(speed: number): Promise<boolean>;

package/src/lib/vesyncFanImpl.ts

@@ -24,7 +24,7 @@ export class VeSyncAirBypass extends VeSyncFan {
     /**
      * Get device details
      */
-    async getDetails(): Promise<void> {
+    async getDetails(): Promise<Boolean> {
         logger.debug(`Getting details for device: ${this.deviceName}`);
         const [response, status] = await this.callApi(
             '/cloud/v2/deviceManaged/bypassV2',
@@ -42,7 +42,8 @@ export class VeSyncAirBypass extends VeSyncFan {
             Helpers.reqHeaderBypass()
         );
 
-        if (this.checkResponse([response, status], 'getDetails')) {
+        const success = this.checkResponse([response, status], 'getDetails');
+        if (success) {
             const result = response?.result?.result;
             if (result) {
                 // Update device status
@@ -64,9 +65,8 @@ export class VeSyncAirBypass extends VeSyncFan {
                 }
                 logger.debug(`Successfully got details for device: ${this.deviceName}`);
             }
-        } else {
-            logger.error(`Failed to get details for device: ${this.deviceName}`);
         }
+        return success;
     }
 
     /**
@@ -488,7 +488,7 @@ export class VeSyncHumidifier extends VeSyncFan {
     /**
      * Get device details
      */
-    async getDetails(): Promise<void> {
+    async getDetails(): Promise<Boolean> {
         logger.debug(`Getting details for device: ${this.deviceName}`);
         const [response, status] = await this.callApi(
             '/cloud/v2/deviceManaged/bypassV2',
@@ -507,12 +507,11 @@ export class VeSyncHumidifier extends VeSyncFan {
         );
 
         const success = this.checkResponse([response, status], 'getDetails');
-        if (success) {
+        if (success && response?.result?.result) {
             this.details = response.result.result;
             logger.debug(`Successfully got details for device: ${this.deviceName}`);
-        } else {
-            logger.error(`Failed to get details for device: ${this.deviceName}`);
         }
+        return success;
     }
 
     /**

package/src/lib/vesyncOutlet.ts

@@ -56,7 +56,7 @@ export abstract class VeSyncOutlet extends VeSyncBaseDevice {
     /**
      * Get outlet details
      */
-    async getDetails(): Promise<void> {
+    async getDetails(): Promise<Boolean> {
         logger.debug(`[${this.deviceName}] Getting outlet details`);
         
         const body = {
@@ -77,7 +77,7 @@ export abstract class VeSyncOutlet extends VeSyncBaseDevice {
             url = '/v1/device/devicedetail';
         }
 
-        const [response] = await this.callApi(
+        const [response, statusCode] = await this.callApi(
             url,
             this.deviceType === 'wifi-switch-1.3' ? 'get' : 'post',
             this.deviceType === 'wifi-switch-1.3' ? null : body,
@@ -86,13 +86,13 @@ export abstract class VeSyncOutlet extends VeSyncBaseDevice {
 
         if (!response) {
             logger.debug(`[${this.deviceName}] No response received from API`);
-            return;
+            return false;
         }
 
         // Handle error responses
         if (response.error) {
             logger.debug(`[${this.deviceName}] Failed to get outlet details: ${JSON.stringify(response)}`);
-            return;
+            return false;
         }
 
         // Handle successful responses
@@ -126,7 +126,7 @@ export abstract class VeSyncOutlet extends VeSyncBaseDevice {
                 if (subDevice) {
                     this.deviceStatus = subDevice.subDeviceStatus;
                     logger.debug(`[${this.deviceName}] Successfully retrieved sub-device status: ${this.deviceStatus}`);
-                    return;
+                    return true;
                 }
             }
 
@@ -139,11 +139,13 @@ export abstract class VeSyncOutlet extends VeSyncBaseDevice {
                 this.deviceStatus = response.power === 'on' ? 'on' : 'off';
             } else {
                 logger.debug(`[${this.deviceName}] Device status not found in response: ${JSON.stringify(response)}`);
-                return;
+                return false;
             }
             logger.debug(`[${this.deviceName}] Successfully retrieved outlet details`);
+            return true;
         } else {
             logger.debug(`[${this.deviceName}] Failed to get outlet details: ${JSON.stringify(response)}`);
+            return false;
         }
     }
 
@@ -364,13 +366,14 @@ export abstract class VeSyncOutlet extends VeSyncBaseDevice {
     /**
      * Update outlet details and energy info
      */
-    async update(): Promise<void> {
+    async update(): Promise<Boolean> {
         logger.debug(`[${this.deviceName}] Updating outlet information`);
-        await this.getDetails();
-        if (this.features.includes('energy')) {
+        const success = await this.getDetails();
+        if (success && this.features.includes('energy')) {
             await this.updateEnergy();
         }
         logger.info(`[${this.deviceName}] Successfully updated outlet information`);
+        return success;
     }
 
     /**

package/src/lib/vesyncSwitch.ts

@@ -81,15 +81,22 @@ export abstract class VeSyncSwitch extends VeSyncBaseDevice {
         }, null, 4);
     }
 
-    abstract getDetails(): Promise<void>;
-    abstract getConfig(): Promise<void>;
-    abstract turnOn(): Promise<boolean>;
-    abstract turnOff(): Promise<boolean>;
+    /**
+     * Get switch details
+     */
+    abstract getDetails(): Promise<Boolean>;
 
     /**
-     * Update device details
+     * Update switch details
      */
-    async update(): Promise<void> {
-        await this.getDetails();
+    async update(): Promise<Boolean> {
+        logger.debug(`[${this.deviceName}] Updating switch information`);
+        const success = await this.getDetails();
+        logger.debug(`[${this.deviceName}] Successfully updated switch information`);
+        return success;
     }
+
+    abstract getConfig(): Promise<void>;
+    abstract turnOn(): Promise<boolean>;
+    abstract turnOff(): Promise<boolean>;
 } 

package/src/lib/vesyncSwitchImpl.ts

@@ -19,7 +19,7 @@ export class VeSyncWallSwitch extends VeSyncSwitch {
     /**
      * Get wall switch details
      */
-    async getDetails(): Promise<void> {
+    async getDetails(): Promise<Boolean> {
         logger.debug(`Getting details for device: ${this.deviceName}`);
         const body = {
             ...Helpers.reqBody(this.manager, 'devicedetail'),
@@ -27,22 +27,22 @@ export class VeSyncWallSwitch extends VeSyncSwitch {
             method: 'devicedetail'
         };
 
-        const [response] = await this.callApi(
+        const [response, statusCode] = await this.callApi(
             '/inwallswitch/v1/device/devicedetail',
             'post',
             body,
             Helpers.reqHeaders(this.manager)
         );
 
-        if (response?.code === 0 && response?.result) {
+        const success = this.checkResponse([response, statusCode], 'getDetails');
+        if (success && response?.result) {
             const result = response.result;
             this.deviceStatus = result.deviceStatus || this.deviceStatus;
             this.details.active_time = result.activeTime || 0;
             this.connectionStatus = result.connectionStatus || this.connectionStatus;
             logger.debug(`Successfully got details for device: ${this.deviceName}`);
-        } else {
-            logger.error(`Failed to get details for device: ${this.deviceName}`);
         }
+        return success;
     }
 
     /**
@@ -143,7 +143,7 @@ export class VeSyncDimmerSwitch extends VeSyncSwitch {
     /**
      * Get dimmer switch details
      */
-    async getDetails(): Promise<void> {
+    async getDetails(): Promise<Boolean> {
         logger.debug(`Getting details for device: ${this.deviceName}`);
         const body = {
             ...Helpers.reqBody(this.manager, 'devicedetail'),
@@ -151,14 +151,15 @@ export class VeSyncDimmerSwitch extends VeSyncSwitch {
             method: 'devicedetail'
         };
 
-        const [response] = await this.callApi(
+        const [response, statusCode] = await this.callApi(
             '/dimmer/v1/device/devicedetail',
             'post',
             body,
             Helpers.reqHeaders(this.manager)
         );
 
-        if (response?.code === 0 && response?.result) {
+        const success = this.checkResponse([response, statusCode], 'getDetails');
+        if (success && response?.result) {
             const result = response.result;
             this.deviceStatus = result.deviceStatus || this.deviceStatus;
             this.details.active_time = result.activeTime || 0;
@@ -174,9 +175,8 @@ export class VeSyncDimmerSwitch extends VeSyncSwitch {
                 };
             }
             logger.debug(`Successfully got details for device: ${this.deviceName}`);
-        } else {
-            logger.error(`Failed to get details for device: ${this.deviceName}`);
         }
+        return success;
     }
 
     /**