iobroker.flowers 0.3.2 → 0.3.4

package/README.md

@@ -7,6 +7,14 @@
 
 Monitor indoor plants via soil moisture, temperature and battery sensors with Telegram notifications.
 
+This adapter works with **any sensor** already integrated in ioBroker (Zigbee, Wi-Fi, Bluetooth, Z-Wave, etc.) — no specific hardware required. Popular compatible sensors:
+
+- [Xiaomi Mi Flora / HHCC Plant Sensor](https://www.mi.com/global/mi-flora) — Bluetooth soil moisture + temperature + light
+- [Zigbee soil moisture sensors](https://www.zigbee2mqtt.io/supported-devices/#s=soil) (e.g. Tuya TS0601, MOES) — via ioBroker Zigbee adapter
+- Any sensor exposing humidity/temperature/battery states in ioBroker
+
+Notifications are sent via the [ioBroker Telegram adapter](https://github.com/ioBroker/ioBroker.telegram).
+
 ## Features
 
 - Monitor multiple plants with individual sensor assignments
@@ -75,6 +83,16 @@ Only one watering cycle runs at a time per plant. Configure the duration in Sett
 
 ## Changelog
 
+### 0.3.4 (2026-03-31)
+- (sadam6752-tech) Add unit tests for MonitorService, NotificationManager and messages (106 tests total)
+- (sadam6752-tech) Update README with links to compatible devices and Telegram adapter
+- (sadam6752-tech) Remove mocha from devDependencies (already included in @iobroker/testing)
+
+### 0.3.3 (2026-03-30)
+- (sadam6752-tech) Fix object hierarchy: create device/channel parent objects before states
+- (sadam6752-tech) Use correct state roles: value.humidity, value.temperature, value.battery
+- (sadam6752-tech) Improve unload: null timers after clearing, guard monitor null check
+
 ### 0.3.2 (2026-03-30)
 - (sadam6752-tech) Custom profiles: users can create own plant profiles in Profiles tab
 - (sadam6752-tech) Custom profile field in Plants table for direct profile name entry

package/io-package.json

@@ -1,8 +1,34 @@
 {
   "common": {
     "name": "flowers",
-    "version": "0.3.2",
+    "version": "0.3.4",
     "news": {
+      "0.3.4": {
+        "en": "Add unit tests for MonitorService, NotificationManager and messages; update README with device links; remove mocha from devDependencies",
+        "de": "Unit-Tests für MonitorService, NotificationManager und messages hinzugefügt; README mit Gerätelinks aktualisiert; mocha aus devDependencies entfernt",
+        "ru": "Добавлены unit-тесты для MonitorService, NotificationManager и messages; обновлён README со ссылками на устройства; удалён mocha из devDependencies",
+        "fr": "Tests unitaires ajoutés pour MonitorService, NotificationManager et messages; README mis à jour avec liens appareils; mocha supprimé des devDependencies",
+        "it": "Aggiunti unit test per MonitorService, NotificationManager e messages; README aggiornato con link dispositivi; rimosso mocha da devDependencies",
+        "es": "Tests unitarios añadidos para MonitorService, NotificationManager y messages; README actualizado con enlaces de dispositivos; mocha eliminado de devDependencies",
+        "pl": "Dodano testy jednostkowe dla MonitorService, NotificationManager i messages; zaktualizowano README z linkami do urządzeń; usunięto mocha z devDependencies",
+        "pt": "Testes unitários adicionados para MonitorService, NotificationManager e messages; README atualizado com links de dispositivos; mocha removido de devDependencies",
+        "nl": "Unit-tests toegevoegd voor MonitorService, NotificationManager en messages; README bijgewerkt met apparaatlinks; mocha verwijderd uit devDependencies",
+        "uk": "Додано unit-тести для MonitorService, NotificationManager та messages; оновлено README з посиланнями на пристрої; видалено mocha з devDependencies",
+        "zh-cn": "为MonitorService、NotificationManager和messages添加单元测试；更新README添加设备链接；从devDependencies中删除mocha"
+      },
+      "0.3.3": {
+        "en": "Fix object hierarchy (device/channel/state), correct state roles (value.humidity, value.temperature, value.battery), improve unload cleanup",
+        "de": "Objekthierarchie korrigiert (device/channel/state), korrekte State-Rollen, verbessertes Aufräumen beim Entladen",
+        "ru": "Исправлена иерархия объектов (device/channel/state), правильные роли состояний, улучшена очистка при выгрузке",
+        "fr": "Correction hiérarchie objets (device/channel/state), rôles d'état corrects, nettoyage amélioré au déchargement",
+        "it": "Corretta gerarchia oggetti (device/channel/state), ruoli stato corretti, pulizia migliorata allo scaricamento",
+        "es": "Corrección jerarquía objetos (device/channel/state), roles de estado correctos, limpieza mejorada al descargar",
+        "pl": "Poprawiono hierarchię obiektów (device/channel/state), poprawne role stanów, ulepszone czyszczenie przy wyładowaniu",
+        "pt": "Correção hierarquia objetos (device/channel/state), funções de estado corretas, limpeza melhorada no descarregamento",
+        "nl": "Objecthiërarchie gecorrigeerd (device/channel/state), correcte state-rollen, verbeterde opruiming bij ontladen",
+        "uk": "Виправлено ієрархію об'єктів (device/channel/state), правильні ролі станів, покращено очищення при вивантаженні",
+        "zh-cn": "修复对象层次结构(device/channel/state)，正确的状态角色，改进卸载清理"
+      },
       "0.3.2": {
         "en": "Custom profiles: users can create own plant profiles in Profiles tab",
         "de": "Benutzerdefinierte Profile: eigene Pflanzenprofile in der Profiles-Registerkarte erstellen",
@@ -16,32 +42,6 @@
         "uk": "Користувацькі профілі: створення власних профілів рослин у вкладці Profiles",
         "zh-cn": "自定义配置文件：在配置文件选项卡中创建自己的植物配置文件"
       },
-      "0.3.1": {
-        "en": "Fixed all lint warnings: complete JSDoc descriptions",
-        "de": "Alle Lint-Warnungen behoben: vollständige JSDoc-Beschreibungen",
-        "ru": "Исправлены все lint-предупреждения: полные JSDoc-описания",
-        "fr": "Tous les avertissements lint corrigés: descriptions JSDoc complètes",
-        "it": "Tutti gli avvisi lint corretti: descrizioni JSDoc complete",
-        "es": "Todas las advertencias lint corregidas: descripciones JSDoc completas",
-        "pl": "Naprawiono wszystkie ostrzeżenia lint: pełne opisy JSDoc",
-        "pt": "Todos os avisos lint corrigidos: descrições JSDoc completas",
-        "nl": "Alle lint-waarschuwingen opgelost: volledige JSDoc-beschrijvingen",
-        "uk": "Виправлено всі lint-попередження: повні JSDoc-описи",
-        "zh-cn": "修复所有lint警告：完整的JSDoc描述"
-      },
-      "0.3.0": {
-        "en": "Standard CI/CD workflow, dependabot, release-script, standard test structure",
-        "de": "Standard CI/CD Workflow, Dependabot, Release-Script, Standard-Teststruktur",
-        "ru": "Стандартный CI/CD workflow, dependabot, release-script, стандартная структура тестов",
-        "fr": "Workflow CI/CD standard, dependabot, release-script, structure de tests standard",
-        "it": "Workflow CI/CD standard, dependabot, release-script, struttura test standard",
-        "es": "Workflow CI/CD estándar, dependabot, release-script, estructura de tests estándar",
-        "pl": "Standardowy workflow CI/CD, dependabot, release-script, standardowa struktura testów",
-        "pt": "Workflow CI/CD padrão, dependabot, release-script, estrutura de testes padrão",
-        "nl": "Standaard CI/CD workflow, dependabot, release-script, standaard teststructuur",
-        "uk": "Стандартний CI/CD workflow, dependabot, release-script, стандартна структура тестів",
-        "zh-cn": "标准CI/CD工作流、dependabot、release-script、标准测试结构"
-      },
       "0.2.9": {
         "en": "Automatic watering support; UI column width fixes",
         "de": "Automatische Bewässerung; UI-Spaltenbreiten korrigiert",

package/lib/monitor-service.js

@@ -1,383 +1,409 @@
-"use strict";
-
-const profiles = require("./plant-profiles.json");
-const { msg } = require("./messages");
-
-/**
- * MonitorService — подписка на датчики, проверка порогов, генерация алертов.
- */
-class MonitorService {
-  /**
-   * @param {object} adapter - ioBroker adapter instance
-   * @param {object} notificationManager - NotificationManager instance
-   * @param {string} lang - system language code
-   */
-  constructor(adapter, notificationManager, lang) {
-    this.adapter = adapter;
-    this.notif = notificationManager;
-    this.lang = lang || "en";
-    this.sensorValues = {};
-    this.lastSeen = {};
-    // active watering timers: key = plant.name, value = true if watering in progress
-    this._wateringActive = {};
-  }
-
-  /**
-   * Get effective thresholds for a plant (plant overrides profile).
-   *
-   * @param {object} plant - plant configuration object
-   * @returns {object} effective threshold values
-   */
-  _getThresholds(plant) {
-    // First check custom profiles from config, then built-in profiles
-    const customProfiles = this.adapter.config.customProfiles || [];
-    const profileName = plant.customProfile || plant.profile;
-    const customProfile = customProfiles.find((p) => p.name === profileName);
-    const profile =
-      customProfile || profiles[profileName] || profiles["Custom"];
-    const override = (val, fallback) => {
-      if (val === null || val === undefined || val === "") {
-        return fallback;
-      }
-      const n = parseFloat(val);
-      return isNaN(n) ? fallback : n;
-    };
-    return {
-      humidityMin: override(plant.humidityMin, profile.humidityMin),
-      humidityMax: override(plant.humidityMax, profile.humidityMax),
-      temperatureMin: override(plant.temperatureMin, profile.temperatureMin),
-      temperatureMax: override(plant.temperatureMax, profile.temperatureMax),
-      batteryMin: override(plant.batteryMin, profile.batteryMin),
-    };
-  }
-
-  /**
-   * Subscribe to all sensor states for all enabled plants.
-   */
-  async subscribeAll() {
-    const plants = this.adapter.config.plants || [];
-    for (const plant of plants) {
-      if (!plant.enabled) {
-        continue;
-      }
-      for (const attr of [
-        "sensorHumidity",
-        "sensorTemperature",
-        "sensorBattery",
-      ]) {
-        const stateId = plant[attr];
-        if (stateId) {
-          await this.adapter.subscribeForeignStatesAsync(stateId);
-          this.adapter.log.debug(`MonitorService: subscribed to ${stateId}`);
-          // Read initial value so checkAll() works immediately
-          try {
-            const state = await this.adapter.getForeignStateAsync(stateId);
-            if (state && state.val !== null && state.val !== undefined) {
-              this.sensorValues[stateId] = {
-                val: state.val,
-                ts: state.ts || Date.now(),
-              };
-              this.lastSeen[plant.name] = Date.now();
-              this.adapter.log.debug(
-                `MonitorService: initial value ${stateId} = ${state.val}`,
-              );
-            }
-          } catch (err) {
-            this.adapter.log.warn(
-              `MonitorService: could not read initial value for ${stateId}: ${err.message}`,
-            );
-          }
-        }
-      }
-    }
-  }
-
-  /**
-   * Unsubscribe from all sensor states.
-   */
-  async unsubscribeAll() {
-    const plants = this.adapter.config.plants || [];
-    for (const plant of plants) {
-      for (const attr of [
-        "sensorHumidity",
-        "sensorTemperature",
-        "sensorBattery",
-      ]) {
-        const stateId = plant[attr];
-        if (stateId) {
-          await this.adapter.unsubscribeForeignStatesAsync(stateId);
-        }
-      }
-    }
-  }
-
-  /**
-   * Handle incoming state change from subscribed sensor.
-   *
-   * @param {string} id - state id
-   * @param {object} state - ioBroker state object
-   */
-  async onStateChange(id, state) {
-    if (!state || state.val === null || state.val === undefined) {
-      return;
-    }
-
-    // Ensure numeric value (sensors may send strings)
-    const numVal = parseFloat(state.val);
-    if (isNaN(numVal)) {
-      return;
-    }
-    this.sensorValues[id] = { val: numVal, ts: state.ts || Date.now() };
-
-    const plants = this.adapter.config.plants || [];
-    for (const plant of plants) {
-      if (!plant.enabled) {
-        continue;
-      }
-      if (
-        plant.sensorHumidity === id ||
-        plant.sensorTemperature === id ||
-        plant.sensorBattery === id
-      ) {
-        this.lastSeen[plant.name] = Date.now();
-        await this._checkPlant(plant);
-      }
-    }
-  }
-
-  /**
-   * Check all plants against current sensor values (called on interval).
-   */
-  async checkAll() {
-    const plants = this.adapter.config.plants || [];
-    for (const plant of plants) {
-      if (!plant.enabled) {
-        continue;
-      }
-      await this._checkPlant(plant);
-      await this._checkOffline(plant);
-    }
-  }
-
-  /**
-   * Check a single plant's thresholds and send alerts if needed.
-   *
-   * @param {object} plant - plant configuration object
-   */
-  async _checkPlant(plant) {
-    const thresholds = this._getThresholds(plant);
-    const label = `${plant.name}${plant.location ? ` (${plant.location})` : ""}`;
-    this.adapter.log.debug(
-      `MonitorService: checking plant "${plant.name}", thresholds: hum ${thresholds.humidityMin}-${thresholds.humidityMax}%`,
-    );
-
-    // Humidity
-    if (plant.sensorHumidity) {
-      const entry = this.sensorValues[plant.sensorHumidity];
-      if (entry != null) {
-        const val = entry.val;
-        this.adapter.log.debug(
-          `MonitorService: ${plant.name} humidity=${val}% (min=${thresholds.humidityMin}, max=${thresholds.humidityMax})`,
-        );
-        if (val < thresholds.humidityMin) {
-          await this.notif.send(
-            msg("humidity_low", this.lang, label, val, thresholds.humidityMin),
-            `${plant.name}:humidity_low`,
-          );
-          await this._startWatering(plant);
-        } else if (val > thresholds.humidityMax) {
-          await this.notif.send(
-            msg("humidity_high", this.lang, label, val, thresholds.humidityMax),
-            `${plant.name}:humidity_high`,
-          );
-        }
-        await this._updateState(
-          `plants.${this._safeName(plant.name)}.humidity`,
-          val,
-          "%",
-        );
-      }
-    }
-
-    // Temperature
-    if (plant.sensorTemperature) {
-      const entry = this.sensorValues[plant.sensorTemperature];
-      if (entry != null) {
-        const val = entry.val;
-        if (val < thresholds.temperatureMin) {
-          await this.notif.send(
-            msg("temp_low", this.lang, label, val, thresholds.temperatureMin),
-            `${plant.name}:temp_low`,
-          );
-        } else if (val > thresholds.temperatureMax) {
-          await this.notif.send(
-            msg("temp_high", this.lang, label, val, thresholds.temperatureMax),
-            `${plant.name}:temp_high`,
-          );
-        }
-        await this._updateState(
-          `plants.${this._safeName(plant.name)}.temperature`,
-          val,
-          "°C",
-        );
-      }
-    }
-
-    // Battery
-    if (plant.sensorBattery) {
-      const entry = this.sensorValues[plant.sensorBattery];
-      if (entry != null) {
-        const val = entry.val;
-        if (val < thresholds.batteryMin) {
-          await this.notif.send(
-            msg("battery_low", this.lang, label, val, thresholds.batteryMin),
-            `${plant.name}:battery_low`,
-          );
-        }
-        await this._updateState(
-          `plants.${this._safeName(plant.name)}.battery`,
-          val,
-          "%",
-        );
-      }
-    }
-  }
-
-  /**
-   * Check if a plant sensor has gone offline.
-   *
-   * @param {object} plant - plant configuration object
-   */
-  async _checkOffline(plant) {
-    const offlineHours = parseInt(this.adapter.config.offlineThreshold) || 3;
-    const last = this.lastSeen[plant.name];
-    if (!last) {
-      return;
-    }
-    const diffHours = (Date.now() - last) / (1000 * 60 * 60);
-    if (diffHours >= offlineHours) {
-      const label = `${plant.name}${plant.location ? ` (${plant.location})` : ""}`;
-      await this.notif.send(
-        msg("offline", this.lang, label, Math.round(diffHours)),
-        `${plant.name}:offline`,
-      );
-    }
-  }
-
-  /**
-   * Start automatic watering for a plant if configured and not already active.
-   *
-   * @param {object} plant - plant configuration object
-   */
-  async _startWatering(plant) {
-    if (!plant.sensorWatering) {
-      return;
-    }
-    if (this._wateringActive[plant.name]) {
-      this.adapter.log.debug(
-        `MonitorService: watering already active for "${plant.name}", skipping`,
-      );
-      return;
-    }
-
-    const durationMin = parseInt(this.adapter.config.wateringDuration) || 3;
-    const durationMs = durationMin * 60 * 1000;
-
-    try {
-      await this.adapter.setForeignStateAsync(plant.sensorWatering, {
-        val: true,
-        ack: false,
-      });
-      this._wateringActive[plant.name] = true;
-      this.adapter.log.info(
-        `MonitorService: watering started for "${plant.name}" (${durationMin} min)`,
-      );
-
-      this.adapter.setTimeout(async () => {
-        try {
-          await this.adapter.setForeignStateAsync(plant.sensorWatering, {
-            val: false,
-            ack: false,
-          });
-          this.adapter.log.info(
-            `MonitorService: watering stopped for "${plant.name}"`,
-          );
-        } catch (err) {
-          this.adapter.log.warn(
-            `MonitorService: failed to stop watering for "${plant.name}": ${err.message}`,
-          );
-        }
-        this._wateringActive[plant.name] = false;
-      }, durationMs);
-    } catch (err) {
-      this.adapter.log.warn(
-        `MonitorService: failed to start watering for "${plant.name}": ${err.message}`,
-      );
-      this._wateringActive[plant.name] = false;
-    }
-  }
-
-  /**
-   * Update or create a state for a plant sensor value.
-   *
-   * @param {string} id - relative state id
-   * @param {number} val - sensor value
-   * @param {string} unit - unit of measurement
-   */
-  async _updateState(id, val, unit) {
-    try {
-      await this.adapter.extendObjectAsync(id, {
-        type: "state",
-        common: {
-          name: id,
-          type: "number",
-          role: "value",
-          unit,
-          read: true,
-          write: false,
-        },
-        native: {},
-      });
-      await this.adapter.setStateAsync(id, { val, ack: true });
-    } catch (err) {
-      this.adapter.log.warn(
-        `MonitorService: failed to update state ${id}: ${err.message}`,
-      );
-    }
-  }
-
-  /**
-   * Convert plant name to safe state id segment.
-   *
-   * @param {string} name - plant name
-   * @returns {string} sanitized name safe for use as state id
-   */
-  _safeName(name) {
-    return name.replace(/[^a-zA-Z0-9_]/g, "_").toLowerCase();
-  }
-
-  /**
-   * Get current sensor values for all enabled plants (for reports).
-   *
-   * @returns {Array<object>} array of { plant, humidity, temperature, battery }
-   */
-  getPlantStates() {
-    const plants = this.adapter.config.plants || [];
-    return plants
-      .filter((p) => p.enabled)
-      .map((plant) => ({
-        plant,
-        humidity: plant.sensorHumidity
-          ? (this.sensorValues[plant.sensorHumidity]?.val ?? null)
-          : null,
-        temperature: plant.sensorTemperature
-          ? (this.sensorValues[plant.sensorTemperature]?.val ?? null)
-          : null,
-        battery: plant.sensorBattery
-          ? (this.sensorValues[plant.sensorBattery]?.val ?? null)
-          : null,
-      }));
-  }
-}
-
-module.exports = MonitorService;
+"use strict";
+
+const profiles = require("./plant-profiles.json");
+const { msg } = require("./messages");
+
+/**
+ * MonitorService — подписка на датчики, проверка порогов, генерация алертов.
+ */
+class MonitorService {
+  /**
+   * @param {object} adapter - ioBroker adapter instance
+   * @param {object} notificationManager - NotificationManager instance
+   * @param {string} lang - system language code
+   */
+  constructor(adapter, notificationManager, lang) {
+    this.adapter = adapter;
+    this.notif = notificationManager;
+    this.lang = lang || "en";
+    this.sensorValues = {};
+    this.lastSeen = {};
+    // active watering timers: key = plant.name, value = true if watering in progress
+    this._wateringActive = {};
+  }
+
+  /**
+   * Get effective thresholds for a plant (plant overrides profile).
+   *
+   * @param {object} plant - plant configuration object
+   * @returns {object} effective threshold values
+   */
+  _getThresholds(plant) {
+    // First check custom profiles from config, then built-in profiles
+    const customProfiles = this.adapter.config.customProfiles || [];
+    const profileName = plant.customProfile || plant.profile;
+    const customProfile = customProfiles.find((p) => p.name === profileName);
+    const profile =
+      customProfile || profiles[profileName] || profiles["Custom"];
+    const override = (val, fallback) => {
+      if (val === null || val === undefined || val === "") {
+        return fallback;
+      }
+      const n = parseFloat(val);
+      return isNaN(n) ? fallback : n;
+    };
+    return {
+      humidityMin: override(plant.humidityMin, profile.humidityMin),
+      humidityMax: override(plant.humidityMax, profile.humidityMax),
+      temperatureMin: override(plant.temperatureMin, profile.temperatureMin),
+      temperatureMax: override(plant.temperatureMax, profile.temperatureMax),
+      batteryMin: override(plant.batteryMin, profile.batteryMin),
+    };
+  }
+
+  /**
+   * Subscribe to all sensor states for all enabled plants.
+   */
+  async subscribeAll() {
+    const plants = this.adapter.config.plants || [];
+    for (const plant of plants) {
+      if (!plant.enabled) {
+        continue;
+      }
+      for (const attr of [
+        "sensorHumidity",
+        "sensorTemperature",
+        "sensorBattery",
+      ]) {
+        const stateId = plant[attr];
+        if (stateId) {
+          await this.adapter.subscribeForeignStatesAsync(stateId);
+          this.adapter.log.debug(`MonitorService: subscribed to ${stateId}`);
+          // Read initial value so checkAll() works immediately
+          try {
+            const state = await this.adapter.getForeignStateAsync(stateId);
+            if (state && state.val !== null && state.val !== undefined) {
+              this.sensorValues[stateId] = {
+                val: state.val,
+                ts: state.ts || Date.now(),
+              };
+              this.lastSeen[plant.name] = Date.now();
+              this.adapter.log.debug(
+                `MonitorService: initial value ${stateId} = ${state.val}`,
+              );
+            }
+          } catch (err) {
+            this.adapter.log.warn(
+              `MonitorService: could not read initial value for ${stateId}: ${err.message}`,
+            );
+          }
+        }
+      }
+    }
+  }
+
+  /**
+   * Unsubscribe from all sensor states.
+   */
+  async unsubscribeAll() {
+    const plants = this.adapter.config.plants || [];
+    for (const plant of plants) {
+      for (const attr of [
+        "sensorHumidity",
+        "sensorTemperature",
+        "sensorBattery",
+      ]) {
+        const stateId = plant[attr];
+        if (stateId) {
+          await this.adapter.unsubscribeForeignStatesAsync(stateId);
+        }
+      }
+    }
+  }
+
+  /**
+   * Handle incoming state change from subscribed sensor.
+   *
+   * @param {string} id - state id
+   * @param {object} state - ioBroker state object
+   */
+  async onStateChange(id, state) {
+    if (!state || state.val === null || state.val === undefined) {
+      return;
+    }
+
+    // Ensure numeric value (sensors may send strings)
+    const numVal = parseFloat(state.val);
+    if (isNaN(numVal)) {
+      return;
+    }
+    this.sensorValues[id] = { val: numVal, ts: state.ts || Date.now() };
+
+    const plants = this.adapter.config.plants || [];
+    for (const plant of plants) {
+      if (!plant.enabled) {
+        continue;
+      }
+      if (
+        plant.sensorHumidity === id ||
+        plant.sensorTemperature === id ||
+        plant.sensorBattery === id
+      ) {
+        this.lastSeen[plant.name] = Date.now();
+        await this._checkPlant(plant);
+      }
+    }
+  }
+
+  /**
+   * Check all plants against current sensor values (called on interval).
+   */
+  async checkAll() {
+    const plants = this.adapter.config.plants || [];
+    for (const plant of plants) {
+      if (!plant.enabled) {
+        continue;
+      }
+      await this._checkPlant(plant);
+      await this._checkOffline(plant);
+    }
+  }
+
+  /**
+   * Check a single plant's thresholds and send alerts if needed.
+   *
+   * @param {object} plant - plant configuration object
+   */
+  async _checkPlant(plant) {
+    const thresholds = this._getThresholds(plant);
+    const label = `${plant.name}${plant.location ? ` (${plant.location})` : ""}`;
+    this.adapter.log.debug(
+      `MonitorService: checking plant "${plant.name}", thresholds: hum ${thresholds.humidityMin}-${thresholds.humidityMax}%`,
+    );
+
+    // Humidity
+    if (plant.sensorHumidity) {
+      const entry = this.sensorValues[plant.sensorHumidity];
+      if (entry != null) {
+        const val = entry.val;
+        this.adapter.log.debug(
+          `MonitorService: ${plant.name} humidity=${val}% (min=${thresholds.humidityMin}, max=${thresholds.humidityMax})`,
+        );
+        if (val < thresholds.humidityMin) {
+          await this.notif.send(
+            msg("humidity_low", this.lang, label, val, thresholds.humidityMin),
+            `${plant.name}:humidity_low`,
+          );
+          await this._startWatering(plant);
+        } else if (val > thresholds.humidityMax) {
+          await this.notif.send(
+            msg("humidity_high", this.lang, label, val, thresholds.humidityMax),
+            `${plant.name}:humidity_high`,
+          );
+        }
+        await this._updateState(
+          `plants.${this._safeName(plant.name)}.humidity`,
+          val,
+          "%",
+        );
+      }
+    }
+
+    // Temperature
+    if (plant.sensorTemperature) {
+      const entry = this.sensorValues[plant.sensorTemperature];
+      if (entry != null) {
+        const val = entry.val;
+        if (val < thresholds.temperatureMin) {
+          await this.notif.send(
+            msg("temp_low", this.lang, label, val, thresholds.temperatureMin),
+            `${plant.name}:temp_low`,
+          );
+        } else if (val > thresholds.temperatureMax) {
+          await this.notif.send(
+            msg("temp_high", this.lang, label, val, thresholds.temperatureMax),
+            `${plant.name}:temp_high`,
+          );
+        }
+        await this._updateState(
+          `plants.${this._safeName(plant.name)}.temperature`,
+          val,
+          "°C",
+        );
+      }
+    }
+
+    // Battery
+    if (plant.sensorBattery) {
+      const entry = this.sensorValues[plant.sensorBattery];
+      if (entry != null) {
+        const val = entry.val;
+        if (val < thresholds.batteryMin) {
+          await this.notif.send(
+            msg("battery_low", this.lang, label, val, thresholds.batteryMin),
+            `${plant.name}:battery_low`,
+          );
+        }
+        await this._updateState(
+          `plants.${this._safeName(plant.name)}.battery`,
+          val,
+          "%",
+        );
+      }
+    }
+  }
+
+  /**
+   * Check if a plant sensor has gone offline.
+   *
+   * @param {object} plant - plant configuration object
+   */
+  async _checkOffline(plant) {
+    const offlineHours = parseInt(this.adapter.config.offlineThreshold) || 3;
+    const last = this.lastSeen[plant.name];
+    if (!last) {
+      return;
+    }
+    const diffHours = (Date.now() - last) / (1000 * 60 * 60);
+    if (diffHours >= offlineHours) {
+      const label = `${plant.name}${plant.location ? ` (${plant.location})` : ""}`;
+      await this.notif.send(
+        msg("offline", this.lang, label, Math.round(diffHours)),
+        `${plant.name}:offline`,
+      );
+    }
+  }
+
+  /**
+   * Start automatic watering for a plant if configured and not already active.
+   *
+   * @param {object} plant - plant configuration object
+   */
+  async _startWatering(plant) {
+    if (!plant.sensorWatering) {
+      return;
+    }
+    if (this._wateringActive[plant.name]) {
+      this.adapter.log.debug(
+        `MonitorService: watering already active for "${plant.name}", skipping`,
+      );
+      return;
+    }
+
+    const durationMin = parseInt(this.adapter.config.wateringDuration) || 3;
+    const durationMs = durationMin * 60 * 1000;
+
+    try {
+      await this.adapter.setForeignStateAsync(plant.sensorWatering, {
+        val: true,
+        ack: false,
+      });
+      this._wateringActive[plant.name] = true;
+      this.adapter.log.info(
+        `MonitorService: watering started for "${plant.name}" (${durationMin} min)`,
+      );
+
+      this.adapter.setTimeout(async () => {
+        try {
+          await this.adapter.setForeignStateAsync(plant.sensorWatering, {
+            val: false,
+            ack: false,
+          });
+          this.adapter.log.info(
+            `MonitorService: watering stopped for "${plant.name}"`,
+          );
+        } catch (err) {
+          this.adapter.log.warn(
+            `MonitorService: failed to stop watering for "${plant.name}": ${err.message}`,
+          );
+        }
+        this._wateringActive[plant.name] = false;
+      }, durationMs);
+    } catch (err) {
+      this.adapter.log.warn(
+        `MonitorService: failed to start watering for "${plant.name}": ${err.message}`,
+      );
+      this._wateringActive[plant.name] = false;
+    }
+  }
+
+  /**
+   * Update or create a state for a plant sensor value.
+   * Ensures parent device and channel objects exist first.
+   *
+   * @param {string} id - relative state id (e.g. plants.ficus.humidity)
+   * @param {number} val - sensor value
+   * @param {string} unit - unit of measurement
+   */
+  async _updateState(id, val, unit) {
+    try {
+      // Ensure parent objects exist: plants (channel) → plants.<name> (device)
+      const parts = id.split(".");
+      if (parts.length === 3) {
+        // parts = ['plants', '<safeName>', '<sensor>']
+        await this.adapter.extendObjectAsync(parts[0], {
+          type: "channel",
+          common: { name: "Plants" },
+          native: {},
+        });
+        await this.adapter.extendObjectAsync(`${parts[0]}.${parts[1]}`, {
+          type: "device",
+          common: { name: parts[1] },
+          native: {},
+        });
+      }
+
+      // Determine correct role
+      const roleMap = {
+        humidity: "value.humidity",
+        temperature: "value.temperature",
+        battery: "value.battery",
+      };
+      const sensor = parts[parts.length - 1];
+      const role = roleMap[sensor] || "value";
+
+      await this.adapter.extendObjectAsync(id, {
+        type: "state",
+        common: {
+          name: sensor,
+          type: "number",
+          role,
+          unit,
+          read: true,
+          write: false,
+        },
+        native: {},
+      });
+      await this.adapter.setStateAsync(id, { val, ack: true });
+    } catch (err) {
+      this.adapter.log.warn(
+        `MonitorService: failed to update state ${id}: ${err.message}`,
+      );
+    }
+  }
+
+  /**
+   * Convert plant name to safe state id segment.
+   *
+   * @param {string} name - plant name
+   * @returns {string} sanitized name safe for use as state id
+   */
+  _safeName(name) {
+    return name.replace(/[^a-zA-Z0-9_]/g, "_").toLowerCase();
+  }
+
+  /**
+   * Get current sensor values for all enabled plants (for reports).
+   *
+   * @returns {Array<object>} array of { plant, humidity, temperature, battery }
+   */
+  getPlantStates() {
+    const plants = this.adapter.config.plants || [];
+    return plants
+      .filter((p) => p.enabled)
+      .map((plant) => ({
+        plant,
+        humidity: plant.sensorHumidity
+          ? (this.sensorValues[plant.sensorHumidity]?.val ?? null)
+          : null,
+        temperature: plant.sensorTemperature
+          ? (this.sensorValues[plant.sensorTemperature]?.val ?? null)
+          : null,
+        battery: plant.sensorBattery
+          ? (this.sensorValues[plant.sensorBattery]?.val ?? null)
+          : null,
+      }));
+  }
+}
+
+module.exports = MonitorService;

package/main.js

@@ -114,14 +114,20 @@ class FlowersAdapter extends utils.Adapter {
     try {
       if (this._checkTimer) {
         this.clearInterval(this._checkTimer);
+        this._checkTimer = null;
       }
       if (this._dailyReportTimer) {
         this.clearTimeout(this._dailyReportTimer);
+        this._dailyReportTimer = null;
       }
       if (this._weeklyReportTimer) {
         this.clearTimeout(this._weeklyReportTimer);
+        this._weeklyReportTimer = null;
+      }
+      if (this.monitor) {
+        this.monitor.unsubscribeAll().catch(() => {});
+        this.monitor = null;
       }
-      this.monitor.unsubscribeAll().catch(() => {});
       this.setStateAsync("info.connection", { val: false, ack: true }).catch(
         () => {},
       );

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "iobroker.flowers",
-  "version": "0.3.2",
+  "version": "0.3.4",
   "description": "ioBroker adapter for monitoring indoor plants via soil moisture, temperature and battery sensors with Telegram notifications",
   "author": {
     "name": "sadam6752-tech",
@@ -35,8 +35,7 @@
     "@iobroker/dev-server": "^0.8.0",
     "@iobroker/eslint-config": "^2.2.0",
     "@iobroker/testing": "^5.2.2",
-    "eslint": "^9.17.0",
-    "mocha": "^11.7.5"
+    "eslint": "^9.17.0"
   },
   "main": "main.js",
   "files": [