firmware: make "Swap Buttons" a select (match Motor Direction UI)

Per the sibling install-orientation setting, convert the toggle from a template
switch to a template select with named states ("Normal" / "Swapped ↔"), sitting
right under Motor Direction in Setup. Like Motor Direction it holds no state of
its own: it reads/writes a persisted global (global_btn_swap) and never fights
the restored value. Behaviour is identical to the switch version.

Verified: esphome config valid, compiles clean, OTA-flashed to VAL3000 @ .58 --
device boots with 'Swap Buttons' select showing "Normal".

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

Author: daniel-frenkel

firmware/common/ropener-product.yaml

@@ -82,6 +82,9 @@ globals:
   # whichever button is currently the close button ever increments its counter.
   - { id: b2_tap_count,   type: int,      restore_value: no, initial_value: '0' }
   - { id: b2_last_tap_ms, type: uint32_t, restore_value: no, initial_value: '0' }
+  # Persisted button-swap state. Like global_stepper_direction, it is the single
+  # source of truth behind a stateless "Swap Buttons" select (see the select block).
+  - { id: global_btn_swap, type: bool, restore_value: yes, initial_value: 'false' }
 
 # --- Stall handling (homing) ------------------------------------------------
 # The Ropener homes sensorlessly: drive at the closed end until StallGuard fires,
@@ -231,7 +234,7 @@ script:
 
   # Hold-to-run engage (armed on press, fired after hold_run_ms).
   # Hold-to-run for each button. The MOTION depends on the button's current role
-  # (set by the "Swap Buttons" switch): the close button drives close, the open
+  # (set by the "Swap Buttons" select): the close button drives close, the open
   # button drives open. Button 1 is the close button when swap is OFF; Button 2 is
   # the close button when swap is ON.
   - id: b1_hold_check
@@ -245,7 +248,7 @@ script:
             - globals.set: { id: b1_hold_active, value: "true" }
             - globals.set: { id: b1_consumed, value: "true" }
             - if:
-                condition: { lambda: 'return !id(btn_swap).state;' }
+                condition: { lambda: 'return !id(global_btn_swap);' }
                 then:
                   - cover.close: ropener_cover
                   - logger.log: "Button 1 held: closing while held"
@@ -263,7 +266,7 @@ script:
             - globals.set: { id: b2_hold_active, value: "true" }
             - globals.set: { id: b2_consumed, value: "true" }
             - if:
-                condition: { lambda: 'return id(btn_swap).state;' }
+                condition: { lambda: 'return id(global_btn_swap);' }
                 then:
                   - cover.close: ropener_cover
                   - logger.log: "Button 2 held: closing while held"
@@ -369,7 +372,7 @@ binary_sensor:
                       else:
                         # Idle, single button. Behaviour depends on Button 1's role.
                         - if:
-                            condition: { lambda: 'return !id(btn_swap).state;' }
+                            condition: { lambda: 'return !id(global_btn_swap);' }
                             then:
                               # CLOSE button: count taps (7 -> homing), defer the close.
                               - script.stop: b1_tap_commit
@@ -414,7 +417,7 @@ binary_sensor:
                   # Tap: the close button defers (allows 7-tap chaining); the open
                   # button opens immediately.
                   - if:
-                      condition: { lambda: 'return !id(btn_swap).state;' }
+                      condition: { lambda: 'return !id(global_btn_swap);' }
                       then:
                         - script.execute: b1_tap_commit
                       else:
@@ -461,7 +464,7 @@ binary_sensor:
                       else:
                         # Idle, single button. Behaviour depends on Button 2's role.
                         - if:
-                            condition: { lambda: 'return id(btn_swap).state;' }
+                            condition: { lambda: 'return id(global_btn_swap);' }
                             then:
                               # CLOSE button (swap ON): count taps (7 -> homing), defer close.
                               - script.stop: b2_tap_commit
@@ -506,7 +509,7 @@ binary_sensor:
                   # Tap: the close button defers (allows 7-tap chaining); the open
                   # button opens immediately.
                   - if:
-                      condition: { lambda: 'return id(btn_swap).state;' }
+                      condition: { lambda: 'return id(global_btn_swap);' }
                       then:
                         - script.execute: b2_tap_commit
                       else:
@@ -581,6 +584,36 @@ select:
           global_speed->update();
           global_preferences->sync();
 
+  # Swap the two physical buttons' open/close roles -- same select UI as Motor
+  # Direction, and its sibling install-orientation setting (sits right under it in
+  # Setup). "Normal" = Button 1 closes, Button 2 opens (the shipped layout).
+  # "Swapped" = Button 1 opens, Button 2 closes -- for a mirror-image install (e.g.
+  # pull-rope on the left) where the flipped unit makes the buttons feel reversed.
+  # The 7-tap homing gesture and the deferred single-tap always follow the CLOSE
+  # button. Like Motor Direction it holds NO state of its own: it reads and writes
+  # the persisted global_btn_swap, so it never fights the restored value. Only
+  # remaps the on-device buttons; the web UI, Home Assistant, and the schedule
+  # open/close are unaffected. (A left-side install usually wants Motor Direction =
+  # Reversed too.)
+  - platform: template
+    name: "Swap Buttons"
+    id: sel_btn_swap
+    icon: "mdi:swap-horizontal"
+    entity_category: CONFIG
+    web_server: { sorting_weight: 2, sorting_group_id: group_setup }
+    update_interval: 1s
+    options:
+      - "Normal"
+      - "Swapped ↔"
+    lambda: |-
+      if (id(global_btn_swap)) return std::string("Swapped ↔");
+      return std::string("Normal");
+    set_action:
+      - lambda: |-
+          id(global_btn_swap) = (x.find("Swap") != std::string::npos);
+          global_btn_swap->update();
+          global_preferences->sync();
+
 # --- Rename core tuning entities to Ropener's existing entity_ids -------------
 number:
   # Raw IRUN (register 1-31) is meaningless to users -- the friendly "Motor
@@ -743,24 +776,6 @@ switch:
     optimistic: true
     restore_mode: RESTORE_DEFAULT_OFF
 
-  # Swap the two physical buttons' open/close roles. Default OFF = Button 1 closes,
-  # Button 2 opens (the shipped layout). ON = Button 1 opens, Button 2 closes -- for
-  # a mirror-image install (e.g. the pull-rope on the left instead of the right)
-  # where the unit sits flipped and the buttons otherwise feel reversed. The 7-tap
-  # homing gesture and the deferred single-tap always follow the CLOSE button, so
-  # homing lives on whichever button closes. This only remaps the on-device buttons;
-  # the web UI, Home Assistant, and the schedule open/close are unaffected. (Motor
-  # Direction, a separate setting, reverses the motor itself -- a left-side install
-  # usually wants both.)
-  - platform: template
-    name: "Swap Buttons"
-    id: btn_swap
-    icon: "mdi:swap-horizontal"
-    entity_category: CONFIG
-    web_server: { sorting_weight: 3, sorting_group_id: group_setup }
-    optimistic: true
-    restore_mode: RESTORE_DEFAULT_OFF
-
 text_sensor:
   # Surfaces StallGuard's arming state, which is otherwise invisible. The
   # failure it exists to catch: if TCOOLTHRS is below the cruise TSTEP for the