@@ -1682,14 +1682,15 @@ object MG4Hardware {
     /**
      * Niveau d'un chauffage à crans (sièges chauffants).
      *
-     * La voiture ne sait qu'AVANCER d'un cran : écrire 1 dans la propriété est un appui, comme
-     * sur l'écran d'origine. Pour un niveau précis, [SeatHeat.reach] lit UNE fois, envoie le
-     * nombre de crans qui convient, puis vérifie sans jamais recliquer.
+     * La voiture ne connaît qu'une commande, « un cran » : écrire 1 dans la propriété est un
+     * appui, comme sur l'écran d'origine, et chaque cran fait DESCENDRE — éteint → 3 → 2 → 1 →
+     * éteint (établi sur SWI133 le 2026-10-09, voir [SeatHeat]). Pour un niveau précis,
+     * [SeatHeat.reach] lit UNE fois, envoie le nombre de crans qui convient, puis vérifie sans
+     * jamais recliquer. Monter d'un niveau coûte donc trois crans, en passant par « éteint ».
      *
-     * L'ancienne boucle (lire, cliquer, relire dans les 0,7 s, recliquer) envoyait un cran de
-     * trop dès que la voiture tardait à annoncer le nouveau niveau, puis refaisait tout le tour :
-     * « 1 → 2 » donnait 2, 3, éteint, 1, 2 (constaté sur SWI133 le 2026-10-09). Et sans niveau
-     * lisible, elle cliquait à l'aveugle pendant 7 secondes.
+     * L'ancienne boucle (lire, cliquer, relire, recliquer jusqu'à lire la cible) arrivait au même
+     * résultat sans connaître le sens, mais cliquait à l'aveugle pendant 7 secondes quand le
+     * niveau était illisible, et pouvait recliquer sur une lecture en retard.
      *
      * ⚠️ Bloquant (0,6 s par cran, puis jusqu'à 3 s de vérification) → hors du thread principal.
      */
@@ -1714,8 +1715,9 @@ object MG4Hardware {
         }
 
     /**
-     * UN cran de plus, sans viser de niveau : c'est la commande même de la voiture, et tout ce
-     * qu'il faut pour « niveau suivant » (pop-up HVAC, raccourci siège chauffant).
+     * UN cran, sans viser de niveau : c'est la commande même de la voiture, et tout ce qu'il faut
+     * pour « niveau suivant » (pop-up HVAC, raccourci siège chauffant). Le niveau obtenu est
+     * [SeatHeat.next] : éteint → 3 → 2 → 1 → éteint.
      */
     private fun stepHvacLevel(propId: Int, areaId: Int): Boolean =
         synchronized(hvacClickLock(propId)) { clickHvac(propId, areaId) }
@@ -2078,7 +2080,7 @@ object MG4Hardware {
         return setHvacLevelWithToggle(PROP_SEAT_HEAT_R, AREA_HVAC, level)
     }
 
-    /** Siège chauffant gauche : niveau suivant (éteint → 1 → 2 → 3 → éteint), en UN cran. */
+    /** Siège chauffant gauche : niveau suivant (éteint → 3 → 2 → 1 → éteint), en UN cran. */
     fun stepSeatHeatLeft(): Boolean {
         if (logEnabled) AppLogger.i(TAG, "stepSeatHeatLeft → un cran")
         return stepHvacLevel(PROP_SEAT_HEAT_L, AREA_HVAC)
 
@@ -96,20 +96,12 @@ object HvacPopup {
             .coerceIn(reglages.fanMin, reglages.fanMax)
 
     // ── Sièges chauffants (firmwares équipés seulement) ──────────────────────
-    private const val NIVEAU_SIEGE_MAX = 3
-
-    /**
-     * Niveau d'un siège chauffant après un appui : éteint → 1 → 2 → 3 → éteint, comme le
-     * raccourci. null si le niveau est illisible — partir d'un 0 supposé alors que le siège est
-     * à 3 ferait DESCENDRE le chauffage.
-     */
-    fun niveauSiegeSuivant(actuel: Int?): Int? =
-        actuel?.let { (it.coerceIn(0, NIVEAU_SIEGE_MAX) + 1) % (NIVEAU_SIEGE_MAX + 1) }
+    // L'ordre des niveaux et le compte des crans vivent dans [SeatHeat].
 
     /** Niveau d'un siège en pastilles pleines et vides : « ● ● ○ » pour 2. */
     fun pastilles(niveau: Int): String {
-        val plein = niveau.coerceIn(0, NIVEAU_SIEGE_MAX)
-        return (1..NIVEAU_SIEGE_MAX).joinToString(" ") { if (it <= plein) "●" else "○" }
+        val plein = niveau.coerceIn(0, SeatHeat.MAX_LEVEL)
+        return (1..SeatHeat.MAX_LEVEL).joinToString(" ") { if (it <= plein) "●" else "○" }
     }
 
     // ── Taille de la fenêtre ─────────────────────────────────────────────────
 
@@ -1,39 +1,54 @@
 package com.mg4.control.model
 
 /**
- * Sièges chauffants — atteindre un niveau sur une voiture qui ne sait qu'AVANCER d'un cran.
+ * Sièges chauffants — atteindre un niveau sur une voiture qui ne connaît qu'une commande : « un
+ * cran ».
  *
- * La commande du véhicule est un appui : éteint → 1 → 2 → 3 → éteint, toujours dans ce sens
- * (l'écran clim d'origine envoie 1 à chaque clic). Et la voiture annonce son nouveau niveau avec
- * retard. Lire, cliquer, relire, recliquer ne marche donc pas : un cran part en trop dès que la
- * lecture tarde, et il faut refaire tout le tour — « 1 → 2 » donnait 2, 3, éteint, 1, 2
- * (constaté sur SWI133 le 2026-10-09).
+ * L'écran clim d'origine envoie 1 à chaque clic et c'est la voiture qui choisit le niveau
+ * suivant ; le sens n'est écrit nulle part dans le firmware. Il a été établi sur véhicule
+ * (SWI133, 2026-10-09) : **chaque cran fait DESCENDRE** — éteint → 3 → 2 → 1 → éteint. Sept
+ * essais sur les boutons de niveau, faits alors que le compte supposait à tort que le cran
+ * montait, l'ont montré sans ambiguïté (demander 1 depuis éteint donnait 3, demander 2 depuis 3
+ * éteignait le siège…) ; ils sont rejoués dans les tests.
  *
- * D'où la règle : on lit UNE fois, on compte, on envoie, et on ne reclique jamais d'après une
- * relecture. Sans vue ni véhicule : la lecture, le cran et l'attente sont fournis par l'appelant.
+ * Conséquence : descendre d'un niveau coûte un cran, MONTER d'un niveau en coûte trois, en
+ * passant par « éteint ». Aucun compte ne peut y échapper.
+ *
+ * La voiture annonce aussi son nouveau niveau avec retard : on lit donc UNE fois, on compte, on
+ * envoie, puis on vérifie sans jamais recliquer. Sans vue ni véhicule : la lecture, le cran et
+ * l'attente sont fournis par l'appelant.
  */
 object SeatHeat {
 
     const val MAX_LEVEL = 3
+    private const val CYCLE = MAX_LEVEL + 1
 
     /** Écart entre deux crans : trop rapprochés, la voiture pourrait en ignorer un. */
     const val CLICK_INTERVAL_MS = 600L
 
-    /** Délai après lequel le niveau LU reflète un cran qui vient de partir. */
-    const val READBACK_MS = 1_200L
+    /**
+     * Délai après lequel le niveau LU reflète un cran qui vient de partir. Valeur prudente, non
+     * mesurée : compter d'après une lecture en retard enverrait un cran de trop ou de moins.
+     */
+    const val READBACK_MS = 2_000L
 
     private const val VERIFY_POLL_MS = 250L
     private const val VERIFY_TIMEOUT_MS = 3_000L
 
+    /**
+     * Niveau où un cran amène la voiture : éteint → 3 → 2 → 1 → éteint. C'est ce qu'il faut
+     * annoncer après un appui (pop-up HVAC, raccourci) ; null si le niveau de départ est illisible.
+     */
+    fun next(actuel: Int?): Int? =
+        actuel?.let { (it.coerceIn(0, MAX_LEVEL) + CYCLE - 1) % CYCLE }
+
     /**
      * Nombre de crans à envoyer pour passer de [actuel] à [cible] ; null si l'un des deux n'est
-     * pas un niveau (lecture en échec, cible inconnue). Descendre d'un niveau en coûte trois :
-     * la voiture ne recule pas.
+     * pas un niveau (lecture en échec, cible inconnue).
      */
     fun steps(actuel: Int, cible: Int): Int? {
         if (actuel !in 0..MAX_LEVEL || cible !in 0..MAX_LEVEL) return null
-        val cycle = MAX_LEVEL + 1
-        return ((cible - actuel) % cycle + cycle) % cycle
+        return ((actuel - cible) % CYCLE + CYCLE) % CYCLE
     }
 
     /**
@@ -43,7 +58,10 @@ object SeatHeat {
      * Rend false sans rien envoyer si le niveau est illisible — envoyer des crans à l'aveugle
      * ferait tourner le siège sans fin — et false aussi si la voiture n'a pas annoncé [cible] au
      * bout de la vérification. Dans ce cas on n'insiste pas : avec une lecture en retard, un cran
-     * de rattrapage serait justement le cran de trop.
+     * de rattrapage partirait d'un niveau faux.
+     *
+     * La vérification ne peut pas être trompée par un niveau traversé en route : une série compte
+     * au plus trois crans, donc quatre niveaux distincts, et [cible] est le dernier.
      *
      * @param lire     niveau annoncé par la voiture (négatif = illisible)
      * @param cran     envoie UN cran
 
@@ -25,6 +25,7 @@ import com.mg4.control.debug.AppLogger
 import com.mg4.control.hardware.MG4Hardware
 import com.mg4.control.model.AirFlow
 import com.mg4.control.model.HvacPopup
+import com.mg4.control.model.SeatHeat
 import com.mg4.control.util.FirmwareInfo
 import com.mg4.control.util.LocaleHelper
 import java.util.Locale
@@ -336,16 +337,16 @@ object HvacPopupOverlay {
     }
 
     /**
-     * Siège chauffant : un appui = niveau suivant (éteint → 1 → 2 → 3 → éteint), comme le
-     * raccourci — et donc UN cran envoyé, puisque c'est la commande même de la voiture. Viser le
-     * niveau par la boucle de recherche faisait faire un tour complet au siège (2, 3, éteint, 1,
-     * 2 pour passer de 1 à 2). Deux appuis rapprochés envoient deux crans, espacés par le matériel.
+     * Siège chauffant : un appui = UN cran envoyé, la commande même de la voiture. Le niveau
+     * suivant est donc le sien — éteint → 3 → 2 → 1 → éteint ([SeatHeat.next]) — et c'est lui
+     * qu'on affiche aussitôt, sans attendre la relecture. Deux appuis rapprochés envoient deux
+     * crans, espacés par le matériel.
      */
     private fun cyclerSiege(gauche: Boolean) {
         val c = chauffages ?: return
         relancerDelai()
         val actuel = if (gauche) c.siegeGauche else c.siegeDroit
-        val suivant = HvacPopup.niveauSiegeSuivant(actuel) ?: return
+        val suivant = SeatHeat.next(actuel) ?: return
         val cote = if (gauche) "gauche" else "droit"
         AppLogger.i(TAG, "Pop-up HVAC — siège chauffant $cote : $actuel → $suivant")
         chauffages = if (gauche) c.copy(siegeGauche = suivant) else c.copy(siegeDroit = suivant)
 
@@ -39,6 +39,7 @@ import com.mg4.control.hardware.MG4Hardware.AebMode
 import com.mg4.control.hardware.MG4Hardware.Swi68Mode
 import com.mg4.control.model.HvacPopup
 import com.mg4.control.model.RegenLevel
+import com.mg4.control.model.SeatHeat
 import com.mg4.control.model.WindowCommand
 import com.mg4.control.profile.ActiveProfile
 import com.mg4.control.profile.ProfileApplier
@@ -824,11 +825,10 @@ class MG4ControlService : Service() {
                     AppLogger.w(TAG, "SHORTCUT siège chauffant — niveau illisible, aucune action")
                     return
                 }
-                val suivant = ExternalApi.cycleStep(actuel, 0, 3, 1)
+                // Un appui = UN cran, la commande même de la voiture. Le niveau suivant est le
+                // sien : éteint → 3 → 2 → 1 → éteint (établi sur véhicule, voir SeatHeat).
                 AppLogger.i(TAG, "SHORTCUT siège chauffant ${if (gauche) "gauche" else "droit"} " +
-                    ": $actuel → $suivant (un cran)")
-                // « Niveau suivant » est la commande même de la voiture : UN cran suffit. Viser
-                // `suivant` par la recherche de niveau faisait faire un tour complet au siège.
+                    ": $actuel → ${SeatHeat.next(actuel)} (un cran)")
                 if (gauche) MG4Hardware.stepSeatHeatLeft()
                 else        MG4Hardware.stepSeatHeatRight()
             }
 
@@ -477,11 +477,20 @@ class DashboardFragment : Fragment() {
                 if (!isRefreshing)
                     CoroutineScope(Dispatchers.IO).launch { MG4Hardware.setSteeringHeat(checked) }
             }
+            // Au clic la rangée montre le niveau DEMANDÉ ; une fois la commande terminée on la
+            // repeint d'après la voiture. Sans ça elle n'était relue qu'à l'ouverture de l'onglet,
+            // et restait fausse dès que le siège n'était pas là où on l'attendait.
             seatLeftButtons?.let { setupSeatButtons(it) { level ->
-                CoroutineScope(Dispatchers.IO).launch { MG4Hardware.setSeatHeatLeft(level) }
+                CoroutineScope(Dispatchers.IO).launch {
+                    MG4Hardware.setSeatHeatLeft(level)
+                    repeindreSieges()
+                }
             } }
             seatRightButtons?.let { setupSeatButtons(it) { level ->
-                CoroutineScope(Dispatchers.IO).launch { MG4Hardware.setSeatHeatRight(level) }
+                CoroutineScope(Dispatchers.IO).launch {
+                    MG4Hardware.setSeatHeatRight(level)
+                    repeindreSieges()
+                }
             } }
         }
 
@@ -1096,6 +1105,22 @@ class DashboardFragment : Fragment() {
         }
     }
 
+    /**
+     * Repeint les deux rangées de sièges d'après la voiture. À appeler hors du thread principal,
+     * une fois la commande terminée : [MG4Hardware.setSeatHeatLeft] ne rend la main qu'après
+     * avoir vu la voiture annoncer le niveau (ou y avoir renoncé), la lecture est donc à jour.
+     * Un niveau illisible laisse la rangée telle quelle.
+     */
+    private suspend fun repeindreSieges() {
+        val gauche = MG4Hardware.getSeatHeatLeftOrNull()
+        val droit  = MG4Hardware.getSeatHeatRightOrNull()
+        withContext(Dispatchers.Main) {
+            if (!isAdded) return@withContext
+            gauche?.let { niveau -> seatLeftButtons?.let { applySeatUI(it, niveau) } }
+            droit?.let { niveau -> seatRightButtons?.let { applySeatUI(it, niveau) } }
+        }
+    }
+
     private fun setupSeatButtons(buttons: List<Button>, onLevel: (Int) -> Unit) {
         buttons.forEachIndexed { index, btn ->
             btn.setOnClickListener {
 
@@ -148,27 +148,6 @@ class HvacPopupTest {
 
     // ── Sièges chauffants (firmwares équipés seulement) ──────────────────────
 
-    @Test
-    fun `un appui sur un siege passe au niveau suivant puis l eteint`() {
-        // Éteint → 1 → 2 → 3 → éteint : comme le raccourci « siège chauffant ».
-        assertEquals(1, HvacPopup.niveauSiegeSuivant(0))
-        assertEquals(2, HvacPopup.niveauSiegeSuivant(1))
-        assertEquals(3, HvacPopup.niveauSiegeSuivant(2))
-        assertEquals(0, HvacPopup.niveauSiegeSuivant(3))
-    }
-
-    @Test
-    fun `un niveau de siege illisible ne se regle pas`() {
-        // Partir d'un 0 supposé alors que le siège est à 3 ferait DESCENDRE le chauffage.
-        assertNull(HvacPopup.niveauSiegeSuivant(null))
-    }
-
-    @Test
-    fun `un niveau de siege hors echelle repart de l echelle`() {
-        assertEquals(0, HvacPopup.niveauSiegeSuivant(7))
-        assertEquals(1, HvacPopup.niveauSiegeSuivant(-1))
-    }
-
     @Test
     fun `les pastilles montrent le niveau du siege`() {
         assertEquals("○ ○ ○", HvacPopup.pastilles(0))