Issue When Sending Siri Commands To Nest Thermostat Through Homebridge-Nest #132

index.js

@@ -1,6 +1,10 @@
 var nest = require('unofficial-nest-api');
 var NestConnection = require('./lib/nest-connection.js');
 var inherits = require('util').inherits;
+var fs = require('fs');
+var readline = require('readline');
+let CONFIG_PATH = process.env.HOME +'lastMode.conf'
+var previousMode;
 
 var Service, Characteristic, Accessory, uuid, Away;
 var DeviceAccessory, ThermostatAccessory, ProtectAccessory, CamAccessory;
@@ -301,15 +305,35 @@ function NestThermostatAccessory(log, name, device, deviceId, initialData, struc
 		.on('get', function (callback) {
 			var targetTemp = this.getTargetTemperature();
 			this.log("Target temperature for " + this.name + " is: " + targetTemp);
+			var targetHeatingCooling = this.getTargetHeatingCooling();
+			this.log("Target heating for " + this.name + " is: " + targetHeatingCooling);
+			if (targetHeatingCooling != 'Off') {
+				fs.writeFile(CONFIG_PATH, targetHeatingCooling, encoding='utf8', function(err) {
+					if(err) {
+						return this.log(err);
+					}
+					this.previousMode = targetHeatingCooling;
+					this.log("config was saved!");
+				});
+			}
 			if (callback) callback(null, targetTemp);
 		}.bind(this))
-		.on('set', this.setTargetTemperature.bind(this));
+		.on('set', this.setTargetTemperature.bind(this));	
 
 	this.getService(Service.Thermostat)
 		.getCharacteristic(Characteristic.TargetHeatingCoolingState)
 		.on('get', function (callback) {
 			var targetHeatingCooling = this.getTargetHeatingCooling();
 			this.log("Target heating for " + this.name + " is: " + targetHeatingCooling);
+			if (targetHeatingCooling != 'Off') {
+				fs.writeFile(CONFIG_PATH, targetHeatingCooling, encoding='utf8', function(err) {
+					if(err) {
+						return this.log(err);
+					}
+					this.previousMode = targetHeatingCooling;
+					this.log("config was saved!");
+				});
+			}
 			if (callback) callback(null, targetHeatingCooling);
 		}.bind(this))
 		.on('set', this.setTargetHeatingCooling.bind(this));
@@ -427,8 +451,21 @@ NestThermostatAccessory.prototype.setTargetHeatingCooling = function (targetHeat
 	}
 
 	this.log("Setting target heating cooling for " + this.name + " to: " + targetTemperatureType);
-	nest.setTargetTemperatureType(this.deviceId, targetTemperatureType);
+	try {  
+		var data = fs.readFileSync(CONFIG_PATH, 'utf8');
+		this.previousMode = data;		
+		this.log(data);
+	} catch(e) {
+		this.log('Error:', e.stack);
+	}
 
+	if (this.getCurrentHeatingCooling() == this.previousMode){
+		nest.setTargetTemperatureType(this.deviceId, this.getTargetHeatingCooling());		
+	}
+	else {
+		nest.setTargetTemperatureType(this.deviceId, this.previousMode);				
+	}
+
 	if (callback) callback(null, targetTemperatureType);
 };
 

lib/nest-thermostat-accessory.js

@@ -2,6 +2,8 @@
  * Created by kraigm on 12/15/15.
  */
 
+var Promise = require('bluebird');
+var debounce = require('lodash.debounce');
 var inherits = require('util').inherits;
 var Accessory, Service, Characteristic, uuid;
 var NestDeviceAccessory = require('./nest-device-accessory')();
@@ -179,7 +181,15 @@ NestThermostatAccessory.prototype.setTargetHeatingCooling = function (targetHeat
 		.asCallback(callback);
 };
 
-NestThermostatAccessory.prototype.setTargetTemperature = function (targetTemperature, callback) {
+NestThermostatAccessory.prototype.setTargetTemperature = function(targetTemperature, callback) {
+	this.log('Trying to set temperature ' + targetTemperature);
+	this.setTargetTemperatureDebounced(targetTemperature, function() {
+		this.log('Temperature set to ' + targetTemperature);
+	}.bind(this));
+	return Promise.resolve().asCallback(callback);
+};
+
+NestThermostatAccessory.prototype.setTargetTemperatureDebounced = debounce(function (targetTemperature, callback) {
 	var usesFahrenheit = this.usesFahrenheit();
 	if (usesFahrenheit) {
 		// Convert to Fahrenheit and round to nearest integer
@@ -213,7 +223,7 @@ NestThermostatAccessory.prototype.setTargetTemperature = function (targetTempera
 	return this.cancelAutoAway()
 		.then(this.updateDevicePropertyAsync.bind(this, key, targetTemperature, prop + "target temperature"))
 		.asCallback(callback);
-};
+}, 5000);
 
 NestThermostatAccessory.prototype.usesFahrenheit = function () {
 	return this.getTemperatureUnits() == Characteristic.TemperatureDisplayUnits.FAHRENHEIT;

package.json

@@ -17,8 +17,9 @@
     "homebridge": ">=0.2.5"
   },
   "dependencies": {
-    "firebase": "^2.3.2",
     "bluebird": "^3.2.2",
+    "firebase": "^2.3.2",
+    "lodash.debounce": "^4.0.8",
     "request-promise": "^1.0.2",
     "unofficial-nest-api": "git+https://github.com/kraigm/unofficial_nodejs_nest.git#3cbd337adc32fab3b481659b38d86f9fcd6a9c02"
   }