iobroker.laundrylens 0.4.35 → 0.4.36

package/README.md

@@ -132,6 +132,10 @@ This also works inside the conditional `[...]` blocks: `[🌡️ Outside: {state
 
 ### **WORK IN PROGRESS**
 
+### 0.4.36 (2026-10-03)
+- Fix: follow-up to 0.4.34's `needsFeedback` fix, reported live - deleting a cycle (e.g. an unconfirmed one someone just discards without confirming/correcting it first) could also leave `needsFeedback` stuck on `true` forever, since only `confirmCycle`/`correctCycle` had been covered. `clearAllData` and `importConfig` had the same gap (both can also replace `cycleHistory`/profiles wholesale). All three now also re-sync `needsFeedback`; `clearAllData`/`importConfig` additionally re-sync `availablePrograms`
+- Covered by 3 new test cases extending `tests/test_programs_and_feedback_sync.js`
+
 ### 0.4.35 (2026-10-03)
 - Fix: reported live - `phaseText` could get stuck showing an earlier phase (e.g. "Aufheizen"/heating) while `phase` had correctly moved on (e.g. to "dryer_drying"). Root cause: `phase` was already written on every `_onTime()` tick (which fires frequently throughout a running cycle), but `phaseText` (the human-readable, localized, emoji-carrying label) was only ever written in `_onManagerState()` - which only runs on actual state transitions (off/starting/running/paused/ending), not on every phase change within a single long "running" period. Both data points now update together on every tick
 - Covered by a new regression test (`tests/test_phase_text_tick_update.js`)

package/io-package.json

@@ -1,8 +1,21 @@
 {
     "common": {
         "name": "laundrylens",
-        "version": "0.4.35",
+        "version": "0.4.36",
         "news": {
+            "0.4.36": {
+                "en": "Follow-up to 0.4.34's needsFeedback fix: deleting a cycle (e.g. an unconfirmed one someone just discards without confirming/correcting it first), clearing all data, or importing a config could also leave needsFeedback stuck on true, since only confirmCycle/correctCycle had been covered. Now also re-synced after deleteCycle, clearAllData and importConfig (the latter two also re-sync availablePrograms, since they can add/replace profiles too).",
+                "de": "Folgefix zum needsFeedback-Fix aus 0.4.34: Das Löschen eines Zyklus (z. B. eines unbestätigten, den man einfach verwirft, ohne ihn vorher zu bestätigen/korrigieren), das Löschen aller Daten oder der Import einer Konfiguration konnten needsFeedback ebenfalls dauerhaft auf true belassen, da bisher nur confirmCycle/correctCycle abgedeckt waren. Wird jetzt auch nach deleteCycle, clearAllData und importConfig synchronisiert (die letzten beiden synchronisieren zusätzlich availablePrograms, da sie auch Programme hinzufügen/ersetzen können).",
+                "ru": "Дополнение к исправлению needsFeedback из 0.4.34: удаление цикла (например, неподтверждённого, который просто отклоняют без предварительного подтверждения/исправления), очистка всех данных или импорт конфигурации тоже могли оставить needsFeedback застрявшим на true, поскольку ранее были покрыты только confirmCycle/correctCycle. Теперь также синхронизируется после deleteCycle, clearAllData и importConfig (последние два дополнительно синхронизируют availablePrograms, так как они тоже могут добавлять/заменять программы).",
+                "pt": "Acompanhamento da correção do needsFeedback em 0.4.34: excluir um ciclo (ex. um não confirmado que alguém apenas descarta sem confirmar/corrigir antes), limpar todos os dados ou importar uma configuração também podiam deixar needsFeedback preso em true, já que só confirmCycle/correctCycle haviam sido cobertos. Agora também é ressincronizado após deleteCycle, clearAllData e importConfig (os dois últimos também ressincronizam availablePrograms, já que podem adicionar/substituir programas).",
+                "nl": "Vervolg op de needsFeedback-fix van 0.4.34: het verwijderen van een cyclus (bijv. een onbevestigde die iemand gewoon weggooit zonder deze eerst te bevestigen/corrigeren), alle data wissen, of een configuratie importeren konden needsFeedback ook vast laten zitten op true, aangezien alleen confirmCycle/correctCycle waren afgedekt. Wordt nu ook opnieuw gesynchroniseerd na deleteCycle, clearAllData en importConfig (de laatste twee synchroniseren ook availablePrograms, omdat ze ook programma's kunnen toevoegen/vervangen).",
+                "fr": "Suite de la correction needsFeedback de la 0.4.29 : supprimer un cycle (par ex. un cycle non confirmé que l'on écarte simplement sans le confirmer/corriger au préalable), effacer toutes les données ou importer une configuration pouvaient aussi laisser needsFeedback bloqué sur true, car seuls confirmCycle/correctCycle étaient couverts. Désormais resynchronisé aussi après deleteCycle, clearAllData et importConfig (ces deux derniers resynchronisent aussi availablePrograms, car ils peuvent également ajouter/remplacer des programmes).",
+                "it": "Seguito della correzione di needsFeedback della 0.4.34: eliminare un ciclo (es. uno non confermato che viene semplicemente scartato senza prima confermarlo/correggerlo), cancellare tutti i dati o importare una configurazione potevano anch'essi lasciare needsFeedback bloccato su true, poiché erano coperti solo confirmCycle/correctCycle. Ora risincronizzato anche dopo deleteCycle, clearAllData e importConfig (questi ultimi due risincronizzano anche availablePrograms, poiché possono anche aggiungere/sostituire programmi).",
+                "es": "Seguimiento de la corrección de needsFeedback de 0.4.34: eliminar un ciclo (p. ej. uno no confirmado que simplemente se descarta sin confirmarlo/corregirlo antes), borrar todos los datos o importar una configuración también podían dejar needsFeedback atascado en true, ya que solo confirmCycle/correctCycle estaban cubiertos. Ahora también se resincroniza tras deleteCycle, clearAllData e importConfig (estos dos últimos también resincronizan availablePrograms, ya que pueden añadir/sustituir programas).",
+                "pl": "Kontynuacja poprawki needsFeedback z 0.4.34: usunięcie cyklu (np. niepotwierdzonego, który ktoś po prostu odrzuca bez wcześniejszego potwierdzenia/poprawienia), wyczyszczenie wszystkich danych lub import konfiguracji mogły również pozostawić needsFeedback na stałe na true, ponieważ wcześniej objęto tylko confirmCycle/correctCycle. Teraz synchronizowane również po deleteCycle, clearAllData i importConfig (te dwa ostatnie synchronizują też availablePrograms, ponieważ mogą dodawać/zastępować programy).",
+                "uk": "Продовження виправлення needsFeedback з 0.4.34: видалення циклу (наприклад, непідтвердженого, який просто відхиляють без попереднього підтвердження/виправлення), очищення всіх даних або імпорт конфігурації також могли залишати needsFeedback застряглим на true, оскільки раніше були охоплені лише confirmCycle/correctCycle. Тепер також синхронізується після deleteCycle, clearAllData та importConfig (останні два також синхронізують availablePrograms, оскільки вони можуть додавати/замінювати програми).",
+                "zh-cn": "0.4.34 needsFeedback 修复的后续：删除某个周期（例如直接丢弃一个未确认的周期，而不先确认/更正它）、清除所有数据或导入配置，也可能导致 needsFeedback 一直卡在 true，因为此前只覆盖了 confirmCycle/correctCycle。现在 deleteCycle、clearAllData 和 importConfig 之后也会重新同步（后两者还会同步 availablePrograms，因为它们也可能添加/替换程序）。"
+            },
             "0.4.35": {
                 "en": "Fix: phaseText could get stuck showing an earlier phase (e.g. \"Aufheizen\"/heating) while phase correctly moved on (e.g. to \"dryer_drying\") - phase was already updated on every tick, but phaseText (the human-readable, localized label) was only ever written at actual state transitions, not on every phase change within a single running cycle. Both now update together on every tick.",
                 "de": "Fix: phaseText konnte auf einer älteren Phase hängen bleiben (z. B. \"Aufheizen\"), während phase korrekt weiterlief (z. B. zu \"dryer_drying\") - phase wurde bereits bei jedem Tick aktualisiert, phaseText (das menschenlesbare, übersetzte Label) aber nur bei tatsächlichen Zustandsübergängen, nicht bei jedem Phasenwechsel innerhalb eines laufenden Zyklus. Beide werden jetzt gemeinsam bei jedem Tick aktualisiert.",
@@ -80,19 +93,6 @@
                 "pl": "Poprawka: elapsedTime deklaruje role \"value.interval\", ktora wymaga wartosci w sekundach, ale wszedzie byla obliczana i przechowywana w minutach (unit definicji stanu, jednorazowa migracja dla istniejacych instalacji, okresowa aktualizacja w main.js oraz zrzut na zywo odczytywany przez zakladke admina przez getStatus). Wszystkie cztery miejsca uzywaja teraz spojnie sekund, tak jak timeRemaining/totalDuration. Poprawka: usunieto ponownie devDependency sinon - jest juz dostarczana tranzytywnie przez @iobroker/testing, wiec bezposrednie jej wymienianie bylo zbedne (checker przegladu E0063). Poprawka: timery falszywego adaptera w kolejnym pliku testowym dostosowano do juz ustalonego w tym zestawie wzorca aliasow natywnych setTimeout/setInterval (checker przegladu E5004/E5005).",
                 "uk": "Виправлення: elapsedTime оголошує роль \"value.interval\", яка вимагає значення в секундах, але воно всюди обчислювалося і зберігалося в хвилинах (unit у визначенні стану, одноразова міграція для наявних інсталяцій, періодичне оновлення в main.js та живий знімок, який вкладка адміністратора читає через getStatus). Тепер усі чотири місця узгоджено використовують секунди, як timeRemaining/totalDuration. Виправлення: devDependency sinon знову видалено - вона вже надається транзитивно через @iobroker/testing, тому пряме зазначення було зайвим (чекер перегляду E0063). Виправлення: таймери фіктивного адаптера ще одного тестового файлу переведено на вже усталений у цьому наборі шаблон нативних псевдонімів setTimeout/setInterval (чекер перегляду E5004/E5005).",
                 "zh-cn": "修复：elapsedTime 声明的角色为「value.interval」，该角色要求数值以秒为单位，但此前在所有位置都以分钟计算和存储（状态定义的 unit、面向现有安装的一次性迁移、main.js 中的周期性更新，以及管理标签页通过 getStatus 读取的实时快照）。现在这四处已统一改为秒，与 timeRemaining/totalDuration 保持一致。修复：再次移除了 sinon 这一开发依赖——它已经通过 @iobroker/testing 间接提供，直接列出属于多余（审查检查器 E0063）。修复：另一个测试文件中伪造适配器的计时器已改为本测试套件中已确立的原生 setTimeout/setInterval 别名写法（审查检查器 E5004/E5005）。"
-            },
-            "0.4.28": {
-                "en": "Dependency/repo-hygiene fixes reported by the ioBroker repository review: bumped @iobroker/testing to the current minimum (6.2.2). Replaced the GitHub noreply author email (not accepted by the review checker) with a real contact address in package.json, io-package.json and the README/LICENSE copyright lines. Reworked a test file's fake-adapter timers to use the same native-setTimeout/setInterval alias pattern already used everywhere else in the suite, so the checker's plain-timer-usage check no longer misidentifies it (the mock's job is exactly to forward to the real timer under the adapter.setTimeout name - behavior is unchanged).",
-                "de": "Abhaengigkeits-/Repo-Hygiene-Fixes aus dem ioBroker-Repository-Review: @iobroker/testing auf die aktuell geforderte Mindestversion (6.2.2) angehoben. Die GitHub-Noreply-Autoren-E-Mail (vom Review-Checker nicht akzeptiert) in package.json, io-package.json sowie den Copyright-Zeilen von README/LICENSE durch eine echte Kontaktadresse ersetzt. Die Fake-Adapter-Timer einer Testdatei auf dasselbe native-setTimeout/setInterval-Alias-Muster umgestellt, das im Rest der Testsuite bereits ueberall verwendet wird, damit die Checker-Pruefung auf rohe Timer-Aufrufe die Datei nicht mehr faelschlich meldet (die Mock-Funktion leitet ohnehin nur unter dem Namen adapter.setTimeout an den echten Timer weiter - Verhalten unveraendert).",
-                "ru": "Исправления зависимостей и гигиены репозитория по итогам ревью репозитория ioBroker: @iobroker/testing обновлён до текущего минимума (6.2.2). Адрес электронной почты автора GitHub noreply (не принимаемый чекером ревью) заменён на реальный контактный адрес в package.json, io-package.json и строках авторских прав README/LICENSE. Таймеры фиктивного адаптера в одном тестовом файле переведены на тот же шаблон нативных алиасов setTimeout/setInterval, который уже используется во всём остальном наборе тестов, чтобы проверка на использование простых таймеров больше не срабатывала ошибочно (мок и так лишь перенаправляет вызов под именем adapter.setTimeout к настоящему таймеру - поведение не изменилось).",
-                "pt": "Correcoes de dependencias/higiene do repositorio apontadas pela revisao do repositorio ioBroker: @iobroker/testing atualizado para o minimo atual (6.2.2). O e-mail de autor noreply do GitHub (nao aceito pelo verificador de revisao) foi substituido por um endereco de contato real no package.json, io-package.json e nas linhas de copyright do README/LICENSE. Os temporizadores do adaptador simulado de um arquivo de teste foram ajustados para usar o mesmo padrao de alias nativo setTimeout/setInterval ja usado em todo o restante da suite, para que a verificacao de uso de temporizadores simples do checker deixe de identifica-lo incorretamente (o mock apenas encaminha para o temporizador real sob o nome adapter.setTimeout - o comportamento nao muda).",
-                "nl": "Dependency-/repo-hygienefixes gemeld door de ioBroker-repository-review: @iobroker/testing bijgewerkt naar het huidige minimum (6.2.2). Het GitHub-noreply-auteurs-e-mailadres (niet geaccepteerd door de review-checker) vervangen door een echt contactadres in package.json, io-package.json en de copyright-regels van README/LICENSE. De nep-adaptertimers van een testbestand omgezet naar hetzelfde native-setTimeout/setInterval-aliaspatroon dat al overal elders in de testsuite wordt gebruikt, zodat de checkercontrole op kaal timergebruik het bestand niet langer ten onrechte meldt (de mock geeft toch al enkel door onder de naam adapter.setTimeout aan de echte timer - gedrag blijft ongewijzigd).",
-                "fr": "Corrections de dependances/hygiene du depot signalees par la revue du depot ioBroker : @iobroker/testing mis a jour vers le minimum actuel (6.2.2). L'adresse e-mail d'auteur noreply de GitHub (non acceptee par le verificateur de revue) remplacee par une veritable adresse de contact dans package.json, io-package.json ainsi que les lignes de copyright du README/LICENSE. Les timers du faux adaptateur d'un fichier de test reorganises pour utiliser le meme modele d'alias natif setTimeout/setInterval deja utilise partout ailleurs dans la suite, afin que la verification des appels de timer bruts par le checker ne le signale plus a tort (le mock ne fait que relayer vers le vrai timer sous le nom adapter.setTimeout - le comportement reste inchange).",
-                "it": "Correzioni di dipendenze/pulizia del repository segnalate dalla revisione del repository ioBroker: @iobroker/testing aggiornato al minimo attuale (6.2.2). L'indirizzo email dell'autore noreply di GitHub (non accettato dal checker di revisione) sostituito con un indirizzo di contatto reale in package.json, io-package.json e nelle righe di copyright di README/LICENSE. I timer del finto adattatore di un file di test riorganizzati per usare lo stesso schema di alias nativo setTimeout/setInterval gia usato ovunque nel resto della suite, cosi il controllo del checker sull'uso di timer semplici non lo segnala piu erroneamente (il mock si limita comunque a inoltrare al timer reale con il nome adapter.setTimeout - il comportamento resta invariato).",
-                "es": "Correcciones de dependencias/higiene del repositorio senaladas por la revision del repositorio ioBroker: @iobroker/testing actualizado al minimo actual (6.2.2). La direccion de correo del autor noreply de GitHub (no aceptada por el verificador de revision) reemplazada por una direccion de contacto real en package.json, io-package.json y las lineas de copyright de README/LICENSE. Los temporizadores del adaptador simulado de un archivo de prueba reorganizados para usar el mismo patron de alias nativo setTimeout/setInterval ya usado en el resto de la suite, para que la verificacion del checker sobre el uso de temporizadores simples deje de detectarlo por error (el mock de todos modos solo reenvia al temporizador real bajo el nombre adapter.setTimeout - el comportamiento no cambia).",
-                "pl": "Poprawki zaleznosci/higieny repozytorium zgloszone przez przeglad repozytorium ioBroker: zaktualizowano @iobroker/testing do aktualnego minimum (6.2.2). Adres e-mail autora typu noreply z GitHub (nieakceptowany przez checker przegladu) zastapiono prawdziwym adresem kontaktowym w package.json, io-package.json oraz w liniach praw autorskich README/LICENSE. Timery falszywego adaptera w jednym pliku testowym przebudowano tak, aby uzywaly tego samego wzorca aliasow natywnych setTimeout/setInterval, ktory jest juz uzywany wszedzie indziej w zestawie testow, dzieki czemu kontrola checkera dotyczaca golych wywolan timerow juz blednie tego nie zglasza (mock i tak jedynie przekazuje wywolanie do prawdziwego timera pod nazwa adapter.setTimeout - zachowanie sie nie zmienia).",
-                "uk": "Виправлення залежностей і гігієни репозиторію, виявлені під час перегляду репозиторію ioBroker: @iobroker/testing оновлено до поточного мінімуму (6.2.2). Адресу електронної пошти автора GitHub noreply (яку не приймає чекер перегляду) замінено на справжню контактну адресу в package.json, io-package.json та рядках авторських прав README/LICENSE. Таймери фіктивного адаптера в одному тестовому файлі переведено на той самий шаблон нативних псевдонімів setTimeout/setInterval, який уже використовується всюди в решті набору тестів, щоб перевірка чекера на використання простих таймерів більше не позначала це помилково (мок і так лише передає виклик до справжнього таймера під назвою adapter.setTimeout - поведінка не змінюється).",
-                "zh-cn": "根据 ioBroker 仓库审查报告修复的依赖/仓库卫生问题：将 @iobroker/testing 升级到当前要求的最低版本（6.2.2）。将审查检查器不接受的 GitHub noreply 作者邮箱，替换为 package.json、io-package.json 以及 README/LICENSE 版权行中的真实联系邮箱。将某测试文件中伪造适配器的计时器改为使用测试套件其余部分早已统一采用的原生 setTimeout/setInterval 别名写法，使检查器对「直接使用原生计时器」的检测不再误报该文件（该模拟函数本来就只是以 adapter.setTimeout 之名转发给真实计时器——行为未发生变化）。"
             }
         },
         "titleLang": {

package/main.js

@@ -1153,6 +1153,10 @@ class WashdataAdapter extends utils.Adapter {
           mgr.traceStore.deleteTrace(cycleId);
           await mgr._saveState();
           await mgr.traceStore.save();
+          // Deleting a cycle - even an unconfirmed one, without ever
+          // confirming/correcting it first - changes whether any cycle
+          // still needs feedback, exactly like confirming one does.
+          await this._updateNeedsFeedback(obj.message.deviceId, mgr);
           respond({ ok: true });
           break;
         }
@@ -1171,6 +1175,7 @@ class WashdataAdapter extends utils.Adapter {
             mgr._saveState(),
           ]);
           await this._updateOverrideStates(obj.message.deviceId, mgr);
+          await this._updateNeedsFeedback(obj.message.deviceId, mgr);
           respond({ ok: true });
           break;
         }
@@ -1203,6 +1208,11 @@ class WashdataAdapter extends utils.Adapter {
             importedTraces = Object.keys(data.traces).length;
           }
 
+          // Imported data can add profiles and/or replace cycleHistory
+          // wholesale - same as any other action that changes either.
+          await this._updateOverrideStates(obj.message.deviceId, mgr);
+          await this._updateNeedsFeedback(obj.message.deviceId, mgr);
+
           respond({
             ok: true,
             imported: data.profiles.length,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "iobroker.laundrylens",
-  "version": "0.4.35",
+  "version": "0.4.36",
   "description": "ioBroker adapter that detects washing machine and dryer cycles via smart plug power measurement, matches them against learned programs and estimates remaining time",
   "author": {
     "name": "backfisch88",