fix(S3): Cold Extraction brach während der Vorbenetzung ab (5.3.3)

Die Stillstands-Erkennung lief bereits während der Vorbenetzung mit. In dieser
Phase sättigt der Puck aber erst, in der Tasse kommt naturgemäß nichts an — bei
einer Vorbenetzung länger als die 15 s Stillstands-Zeit wurde deshalb jeder kalte
Bezug mit "kein Zulauf" abgebrochen. Mit dem bisherigen Standard von 30 s wäre das
ausnahmslos passiert; die Funktion war mit aktiver Waage unbenutzbar. Die
Erkennung startet jetzt erst mit dem Hauptbezug.

Dazu der Standard der Vorbenetzung von 30 s auf 10 s: Die Pumpe läuft in dieser
Phase bewusst ungepulst auf voller Leistung. Heiß kommt der erste Tropfen nach
etwa 7 s, kalt (rund dreifache Viskosität) entsprechend später — 30 s wären längst
voller Bezug bei vollem Druck gewesen, und die Hälfte des Pumpen-Laufzeitbudgets
wäre vor dem eigentlichen Bezug verbraucht. Bestehende Installationen behalten
ihren gespeicherten Wert.

Die Beschriftung auf /Brew-Control sagt jetzt ausdrücklich, dass die Pumpe während
der Vorbenetzung durchläuft und erst im Hauptbezug gepulst wird.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

Changelog.txt

@@ -1,3 +1,19 @@
+Version 5.3.3:
+- Bug-Fix (Cold Extraction, hätte die Funktion mit Waage komplett unbrauchbar gemacht): Die
+  Stillstands-Erkennung lief bereits während der Vorbenetzung mit. In dieser Phase sättigt der Puck
+  aber erst, es kommt naturgemäß nichts in der Tasse an - bei einer Vorbenetzung länger als die
+  15 s Stillstands-Zeit wurde deshalb JEDER kalte Bezug mit „kein Zulauf" abgebrochen. Mit dem
+  bisherigen Standard von 30 s Vorbenetzung wäre das immer passiert. Die Erkennung startet jetzt
+  erst mit dem Hauptbezug.
+- Standard der Vorbenetzung von 30 s auf 10 s gesenkt. In dieser Phase läuft die Pumpe bewusst
+  ungepulst auf voller Leistung; bei heißem Bezug kommt der erste Tropfen nach etwa 7 s, kalt
+  (rund dreifache Viskosität) entsprechend später. 30 s wären also längst voller Bezug bei vollem
+  Druck gewesen, und die Hälfte des Pumpen-Laufzeitbudgets wäre vor dem eigentlichen Bezug
+  verbraucht. Bestehende Installationen behalten ihren gespeicherten Wert - dort ggf. auf
+  /Brew-Control nachziehen.
+- Die Beschriftung auf /Brew-Control sagt jetzt ausdrücklich, dass die Pumpe während der
+  Vorbenetzung durchläuft und erst im Hauptbezug gepulst wird.
+
 Version 5.3.2:
 - Cold Extraction sendet den Sperrgrund jetzt als Code („cxBlock") statt als Klartext („cxMessage"),
   dazu eine transiente Meldung als Code („cxNotice") und die Vorbenetzungsdauer („cxPreInf").

Dual-PID.ino

@@ -369,7 +369,7 @@ Adafruit_SH1106G display = Adafruit_SH1106G(128, 64, &Wire);
  * Firmware-Informationen
  ************************************************************************************/
 
-String version = "5.3.2";
+String version = "5.3.3";
 String versionHersteller = "Thomas Müller";
 String versionHerstellerMail =
   "<a href='mailto:thomas@mueller.black' class='info-link'>thomas@mueller.black</a>";
@@ -782,7 +782,7 @@ bool flowGuardPumpPulseActive = false;
  ************************************************************************************/
 bool coldExtractionEnabled = false;             // Funktion in der Web-UI freigeschaltet
 float coldExtractionMaxTempC = 30.0f;           // Freigabeschwelle Wasserkessel
-float coldExtractionPreInfusionSeconds = 30.0f; // Vorbenetzung (Pumpe laeuft ungepulst)
+float coldExtractionPreInfusionSeconds = 10.0f; // Vorbenetzung (Pumpe laeuft ungepulst)
 uint16_t coldExtractionPulsePeriodMs = 2000;    // Pulsperiode der Pumpe im Hauptbezug
 float coldExtractionDutyPercent = 30.0f;        // Pumpen-Duty im Hauptbezug
 float coldExtractionTargetGrams = 60.0f;        // Zielgewicht (ca. 1:3 auf 18 g)
@@ -792,7 +792,11 @@ float coldExtractionPumpRestSec = 20.0f;        // Dauer der Zwangspause
 
 const bool defaultColdExtractionEnabled = false;
 const float defaultColdExtractionMaxTempC = 30.0f;
-const float defaultColdExtractionPreInfusionSeconds = 30.0f;
+// 10 s statt der urspruenglichen 30 s: Die Pumpe laeuft in dieser Phase ungepulst auf
+// voller Leistung. Heiss kommt der erste Tropfen nach ~7 s, kalt (rund dreifache
+// Viskositaet) entsprechend spaeter - 30 s waeren laengst voller Bezug bei vollem Druck
+// gewesen, und die Haelfte des Pumpen-Laufzeitbudgets waere vor dem Hauptbezug verbraucht.
+const float defaultColdExtractionPreInfusionSeconds = 10.0f;
 const uint16_t defaultColdExtractionPulsePeriodMs = 2000;
 const float defaultColdExtractionDutyPercent = 30.0f;
 const float defaultColdExtractionTargetGrams = 60.0f;
@@ -4202,8 +4206,12 @@ bool coldExtractionStallDetected(float netWeight) {
         coldExtractionStallSinceMs = 0;
         return false;
     }
-    if (coldExtractionPumpResting) {
-        coldExtractionStallSinceMs = 0;         // Pause ist kein Stillstand
+    // Waehrend der Vorbenetzung saettigt der Puck erst - da kommt naturgemaess nichts in
+    // der Tasse an. Ohne diese Ausnahme wuerde die Erkennung jeden kalten Bezug abbrechen,
+    // sobald die Vorbenetzung laenger als COLDEX_STALL_TIMEOUT_MS dauert. Der Zaehler
+    // startet damit erst mit dem Hauptbezug.
+    if (coldExtractionPumpResting || currentPreInfusionState == PI_PRE_BREW) {
+        coldExtractionStallSinceMs = 0;         // Pause/Vorbenetzung ist kein Stillstand
         coldExtractionStallLastWeight = netWeight;
         return false;
     }
@@ -7144,7 +7152,7 @@ static const char brewControlCX_ToggleContainerEnd[] PROGMEM = R"rawliteral(><sp
 static const char brewControlCX_Fields[] PROGMEM = R"rawliteral(
     <label for='cxMaxTemp'>Freigabe bis Wassertemperatur (&deg;C):</label>
     <input type='number' min='10.0' max='40.0' step='0.5' id='cxMaxTemp' name='cxMaxTemp' value='{CX_MAX_TEMP}' required>
-    <label for='cxPreInf'>Vorbenetzung (Sekunden):</label>
+    <label for='cxPreInf'>Vorbenetzung (Sekunden, Pumpe l&auml;uft dabei durchgehend):</label>
     <input type='number' min='0.0' max='120.0' step='1.0' id='cxPreInf' name='cxPreInf' value='{CX_PREINF}' required>
     <label for='cxPulseMs'>Pulsperiode (ms):</label>
     <input type='number' min='200' max='5000' step='50' id='cxPulseMs' name='cxPulseMs' value='{CX_PULSE_MS}' required>
@@ -7161,7 +7169,9 @@ static const char brewControlCX_Fields[] PROGMEM = R"rawliteral(
     <small class="toggle-description-input">(Die Maschine hat keinen Bypass um den Wasserkessel: Kaltes Wasser kommt nur bei kaltem
     Kessel am Puck an. Deshalb ist die kalte Extraktion nur unterhalb der Freigabetemperatur m&ouml;glich, praktisch also
     aus dem kalten Zustand heraus. Solange der Modus aktiv ist, heizt der Wasserkreis nicht - der Dampfkreis bleibt
-    unber&uuml;hrt und darf weiter heizen. Die Pumpenlaufzeit wird kumuliert gez&auml;hlt; nach der eingestellten
+    unber&uuml;hrt und darf weiter heizen. W&auml;hrend der Vorbenetzung l&auml;uft die Pumpe durchgehend auf voller
+    Leistung (kein Pulsen) - sinnvoll ist eine Dauer knapp unterhalb des ersten Tropfens; gepulst wird erst im
+    Hauptbezug. Die Pumpenlaufzeit wird kumuliert gez&auml;hlt; nach der eingestellten
     Laufzeit legt die Steuerung eine Zwangspause ein. Der Modus selbst wird auf dem Dashboard bzw. am Touch-Display
     ein- und ausgeschaltet und bleibt &uuml;ber Standby und Neustart hinweg aktiv, bis er dort wieder ausgeschaltet
     wird - solange heizt der Wasserkreis nicht.)</small>

JC_Display_Firmware/Changelog.txt

@@ -1,3 +1,18 @@
+Version 1.1.1:
+- Compiler-Warnungen des P4-Builds aufgeräumt (rein technisch, keine Funktionsänderung):
+  - „LV_FS_DEFAULT_DRIVE_LETTER is deprecated": Die Option heißt ab LVGL 9.3
+    LV_FS_DEFAULT_DRIVER_LETTER. Der alte Name funktionierte zwar noch, löste aber in
+    lv_api_map_v9_1.h ein #warning aus - und zwar in JEDER Übersetzungseinheit, die lvgl.h
+    einbindet. Das allein waren rund 60 Warnungen pro Build, die alles andere zugedeckt haben.
+    In lv_conf.h umbenannt.
+  - „bitwise operation between different enumeration types is deprecated": LV_PART_* und
+    LV_STATE_* sind in LVGL 9 zwei verschiedene Enum-Typen; sie direkt mit | zu verknüpfen ist
+    in C++20 abgekündigt. Neues Makro TH_SEL(part, state) in theme.h castet beide auf
+    lv_style_selector_t (ohnehin uint32_t). Betraf 5 Stellen in ui.cpp und theme.h.
+- Bug-Fix: Der Puffer für die Cold-Extraction-Infozeile auf der Brew-Seite war mit 192 Bytes
+  zu klein - der Text ist rund 205 Zeichen lang (das „°" zählt als zwei Bytes), die letzte
+  Zeile wurde also abgeschnitten. Jetzt 256 Bytes.
+
 Version 1.1.0:
 - Cold Extraction am Display bedienbar (braucht S3-Firmware ab 5.3.2):
   - Neuer Schalter „Cold Extraction" auf der Seite „Temperaturen" unter „Modi", darunter eine

JC_Display_Firmware/config.h

@@ -65,7 +65,7 @@
 // -------------------------------------------------------------------------------------
 //  Firmware
 // -------------------------------------------------------------------------------------
-#define DISPLAY_FW_VERSION       "1.1.0"   // Firmware-Stand der P4-Display-Steuerung (Info-Seite)
+#define DISPLAY_FW_VERSION       "1.1.1"   // Firmware-Stand der P4-Display-Steuerung (Info-Seite)
 
 // -------------------------------------------------------------------------------------
 //  Protokoll

JC_Display_Firmware/lv_conf.h

@@ -727,7 +727,10 @@
 /*File system interfaces for common APIs */
 
 /*Setting a default driver letter allows skipping the driver prefix in filepaths*/
-#define LV_FS_DEFAULT_DRIVE_LETTER '\0'
+/* Ab LVGL 9.3 heisst die Option LV_FS_DEFAULT_DRIVER_LETTER. Der alte Name wird zwar noch
+   uebersetzt, loest aber in lv_api_map_v9_1.h ein #warning aus - und zwar in JEDER
+   Uebersetzungseinheit, die lvgl.h einbindet (~60 Warnungen pro Build). */
+#define LV_FS_DEFAULT_DRIVER_LETTER '\0'
 
 /*API for fopen, fread, etc*/
 #define LV_USE_FS_STDIO 0

JC_Display_Firmware/theme.h

@@ -25,6 +25,11 @@
 #define COL_BORDER    lv_color_hex(0x555555)
 #define COL_INPUT_BG  lv_color_hex(0x101010)
 
+// Style-Selektor aus Part + State. LV_PART_* und LV_STATE_* sind in LVGL 9 zwei
+// verschiedene Enum-Typen; sie direkt mit | zu verknuepfen ist in C++20 deprecated
+// (-Wdeprecated-enum-enum-conversion). Der Zieltyp ist ohnehin uint32_t.
+#define TH_SEL(part, state) ((lv_style_selector_t)(part) | (lv_style_selector_t)(state))
+
 // ---- Karte (dunkel, abgerundet) ----
 static inline lv_obj_t* th_card(lv_obj_t* parent) {
     lv_obj_t* c = lv_obj_create(parent);
@@ -61,7 +66,7 @@ static inline lv_obj_t* th_textarea(lv_obj_t* parent, bool oneLine) {
     lv_textarea_set_one_line(ta, oneLine);
     lv_obj_set_style_bg_color(ta, COL_INPUT_BG, 0);
     lv_obj_set_style_border_color(ta, COL_BORDER, 0);
-    lv_obj_set_style_border_color(ta, COL_ACCENT, LV_PART_MAIN | LV_STATE_FOCUSED);
+    lv_obj_set_style_border_color(ta, COL_ACCENT, TH_SEL(LV_PART_MAIN, LV_STATE_FOCUSED));
     lv_obj_set_style_border_width(ta, 1, 0);
     lv_obj_set_style_radius(ta, 8, 0);
     lv_obj_set_style_text_color(ta, COL_TEXT, 0);

JC_Display_Firmware/ui.cpp

@@ -389,7 +389,7 @@ static lv_obj_t* row(lv_obj_t* parent, const char* label) {
 static lv_obj_t* add_switch(lv_obj_t* parent, const char* label) {
     lv_obj_t* r = row(parent, label);
     lv_obj_t* sw = lv_switch_create(r);
-    lv_obj_set_style_bg_color(sw, COL_ACCENT, LV_PART_INDICATOR | LV_STATE_CHECKED);
+    lv_obj_set_style_bg_color(sw, COL_ACCENT, TH_SEL(LV_PART_INDICATOR, LV_STATE_CHECKED));
     return sw;
 }
 static lv_obj_t* add_num(lv_obj_t* parent, const char* label) {
@@ -2348,7 +2348,7 @@ static void build_wifi(lv_obj_t* p) {
     // Aktionszeile: Scan + gewaehltes Netz
     lv_obj_t* ar = row(p, nullptr);
     g_btnScan = th_accent_button(ar, LV_SYMBOL_REFRESH " Netzwerke suchen");
-    lv_obj_set_style_bg_opa(g_btnScan, LV_OPA_40, LV_PART_MAIN | LV_STATE_DISABLED);
+    lv_obj_set_style_bg_opa(g_btnScan, LV_OPA_40, TH_SEL(LV_PART_MAIN, LV_STATE_DISABLED));
     lv_obj_add_event_cb(g_btnScan, wifi_scan_cb, LV_EVENT_CLICKED, nullptr);
     lblWifiSel = lv_label_create(ar);
     lv_label_set_text(lblWifiSel, "Netz: -");
@@ -2585,13 +2585,13 @@ static void build_profiles(lv_obj_t* p) {
     lv_obj_set_style_text_color(profDetailLbl, COL_TEXT_DIM, 0);
     profLoadBtn = th_accent_button(dcard, "Profil laden");
     lv_obj_set_width(profLoadBtn, LV_PCT(100));
-    lv_obj_set_style_bg_opa(profLoadBtn, LV_OPA_40, LV_PART_MAIN | LV_STATE_DISABLED);
+    lv_obj_set_style_bg_opa(profLoadBtn, LV_OPA_40, TH_SEL(LV_PART_MAIN, LV_STATE_DISABLED));
     lv_obj_add_state(profLoadBtn, LV_STATE_DISABLED);
     lv_obj_add_event_cb(profLoadBtn, prof_do_load_cb, LV_EVENT_CLICKED, nullptr);
 
     profDelBtn = th_button(dcard, "Profil löschen", COL_DANGER, COL_TEXT);
     lv_obj_set_width(profDelBtn, LV_PCT(100));
-    lv_obj_set_style_bg_opa(profDelBtn, LV_OPA_40, LV_PART_MAIN | LV_STATE_DISABLED);
+    lv_obj_set_style_bg_opa(profDelBtn, LV_OPA_40, TH_SEL(LV_PART_MAIN, LV_STATE_DISABLED));
     lv_obj_add_state(profDelBtn, LV_STATE_DISABLED);
     lv_obj_add_event_cb(profDelBtn, prof_delete_cb, LV_EVENT_CLICKED, nullptr);
 }
@@ -3870,7 +3870,7 @@ void ui_update(const MachineState& st) {
     }
     if (lblBrewColdEx) {
         if (st.cxEnabled) {
-            char b[192];
+            char b[256];   // der Text ist ~205 Zeichen lang (das ° zaehlt als 2 Bytes)
             snprintf(b, sizeof(b),
                      "Cold Extraction (nur Anzeige, Einstellung in der Web-UI):\n"
                      "Freigabe < %.1f °C  |  Vorbenetzung %.0f s  |  Ziel %.0f g  |  max. %.0f s\n"