fix(sensor): replace magic number with constant for sensor read interval

Author: tyeth

.agents/skills/add-sensor-component-v1/SKILL.md

@@ -642,7 +642,7 @@ protected:
 
   bool _readSensor() {
     unsigned long now = millis();
-    if (_lastRead != 0 && now - _lastRead < 1000)
+    if (_lastRead != 0 && now - _lastRead < ONE_SECOND_IN_MILLIS)
       return true; // recently read, use cached value
     if (!_tmp119->getEvent(&_cachedTemp))
       return false;

src/components/i2c/drivers/WipperSnapper_I2C_Driver_APDS9999.h

@@ -86,7 +86,7 @@ class WipperSnapper_I2C_Driver_APDS9999 : public WipperSnapper_I2C_Driver {
   bool ReadSensorData() {
     unsigned long now = millis();
     // Don't read sensor more than once per second
-    if (_lastRead != 0 && (now - _lastRead < 1000))
+    if (_lastRead != 0 && (now - _lastRead < ONE_SECOND_IN_MILLIS))
       return true;
 
     uint32_t r, g, b, ir;

src/components/i2c/drivers/WipperSnapper_I2C_Driver_SCD30.h

@@ -59,7 +59,7 @@ class WipperSnapper_I2C_Driver_SCD30 : public WipperSnapper_I2C_Driver {
       @returns  True if the sensor was recently read, False otherwise.
   */
   bool HasBeenReadInLastSecond() {
-    return _lastRead != 0 && millis() - _lastRead < 1000;
+    return _lastRead != 0 && millis() - _lastRead < ONE_SECOND_IN_MILLIS;
   }
 
   /*******************************************************************************/

src/components/i2c/drivers/WipperSnapper_I2C_Driver_SCD4X.h

@@ -74,7 +74,7 @@ class WipperSnapper_I2C_Driver_SCD4X : public WipperSnapper_I2C_Driver {
   */
   /*******************************************************************************/
   bool HasBeenReadInLastSecond() {
-    return _lastRead != 0 && millis() - _lastRead < 1000;
+    return _lastRead != 0 && millis() - _lastRead < ONE_SECOND_IN_MILLIS;
   }
 
   /*******************************************************************************/

src/components/i2c/drivers/WipperSnapper_I2C_Driver_SEN5X.h

@@ -82,7 +82,7 @@ class WipperSnapper_I2C_Driver_SEN5X : public WipperSnapper_I2C_Driver {
       @returns  True if the sensor was recently read, False otherwise.
   */
   bool HasBeenReadInLastSecond() {
-    return _lastRead != 0 && millis() - _lastRead < 1000;
+    return _lastRead != 0 && millis() - _lastRead < ONE_SECOND_IN_MILLIS;
   }
 
   /*******************************************************************************/

src/components/i2c/drivers/WipperSnapper_I2C_Driver_SEN6X.h

@@ -85,7 +85,7 @@ class WipperSnapper_I2C_Driver_SEN6X : public WipperSnapper_I2C_Driver {
       @returns  True if the sensor was recently read, False otherwise.
   */
   bool HasBeenReadInLastSecond() {
-    return _lastRead != 0 && millis() - _lastRead < 1000;
+    return _lastRead != 0 && millis() - _lastRead < ONE_SECOND_IN_MILLIS;
   }
 
   /*******************************************************************************/

src/components/i2c/drivers/WipperSnapper_I2C_Driver_TMP119.h

@@ -94,7 +94,7 @@ class WipperSnapper_I2C_Driver_TMP119 : public WipperSnapper_I2C_Driver {
   */
   /*******************************************************************************/
   bool _readSensor() {
-    if (_lastRead != 0 && millis() - _lastRead < 1000)
+    if (_lastRead != 0 && millis() - _lastRead < ONE_SECOND_IN_MILLIS)
       return true; // use cached value
     if (!_tmp119->getEvent(&_cachedTemp))
       return false;