v4.2.5: fix LED style mismatch for PM=49 (LF10 product, LF8 layout)

resolve_style_id() name lookup matched style 7 (116 LEDs) instead of
style 5 (93 LEDs) because PM=49's product name is "LF10" but its LED
layout is style 5 ("LF8"). Now stores style_id from HID probe directly
on DeviceInfo — no reverse name lookup needed.

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

Author: Lexonight1

pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "trcc-linux"
-version = "4.2.4"
+version = "4.2.5"
 description = "Linux implementation of Thermalright LCD Control Center"
 readme = "README.md"
 license = "GPL-3.0-or-later"

src/trcc/__version__.py

@@ -1,6 +1,6 @@
 """TRCC Linux version information."""
 
-__version__ = "4.2.4"
+__version__ = "4.2.5"
 __version_info__ = tuple(int(x) for x in __version__.split("."))
 
 # Version history:
@@ -155,3 +155,7 @@
 #          resolution via handshake. Fixes HID devices like Assassin Spirit
 #          120 Vision ARGB (PM=36, 240x240) getting wrong-sized frames.
 #          CLI commands also handshake before sending. 2308 tests.
+# 4.2.5  - Fix LED style mismatch for PM=49 (LF10 product, LF8 layout):
+#          resolve_style_id() name lookup matched style 7 (116 LEDs) instead
+#          of style 5 (93 LEDs). Now stores style_id from probe directly on
+#          DeviceInfo — no reverse name lookup. 2308 tests.

src/trcc/adapters/device/scsi.py

@@ -252,6 +252,7 @@ def find_lcd_devices() -> List[Dict]:
 
             model = dev.model
             button_image = dev.button_image
+            led_style_id = None
 
             # All LED devices share PID 0x8001 — probe via HID handshake
             # to discover the real model (AX120, PA120, LC1, etc.).
@@ -262,6 +263,7 @@ def find_lcd_devices() -> List[Dict]:
                                            usb_path=dev.usb_path)
                     if info and info.model_name:
                         model = info.model_name
+                        led_style_id = info.style.style_id if info.style else None
                         btn = PmRegistry.get_button_image(info.pm, info.sub_type)
                         if btn:
                             button_image = btn
@@ -275,6 +277,7 @@ def find_lcd_devices() -> List[Dict]:
                 'vendor': dev.vendor_name,
                 'product': dev.product_name,
                 'model': model,
+                'led_style_id': led_style_id,
                 'button_image': button_image,
                 'vid': dev.vid,
                 'pid': dev.pid,

src/trcc/core/models.py

@@ -169,6 +169,7 @@ class DeviceInfo:
     protocol: str = "scsi"  # "scsi" or "hid"
     device_type: int = 1  # 1=SCSI, 2=HID Type 2 ("H"), 3=HID Type 3 ("ALi")
     implementation: str = "generic"  # e.g. "thermalright_lcd_v1", "hid_type2", "hid_led"
+    led_style_id: Optional[int] = None  # LED style from probe (avoids name-based lookup)
 
     # State
     connected: bool = True

src/trcc/qt_components/qt_app_mvc.py

@@ -1021,6 +1021,7 @@ def _on_device_widget_clicked(self, device_info: dict):
             protocol=device_info.get('protocol', 'scsi'),
             device_type=device_info.get('device_type', 1),
             implementation=implementation,
+            led_style_id=device_info.get('led_style_id'),
         )
 
         if implementation == 'hid_led':
@@ -1725,7 +1726,9 @@ def _show_led_view(self, device: DeviceInfo):
 
         from ..services.led import LEDService
         model = device.model or ''
-        led_style = LEDService.resolve_style_id(model)
+        # Use probe-resolved style_id directly (avoids name collision:
+        # PM=49 "LF10" shares style 5 with "LF8", but name lookup hits style 7)
+        led_style = device.led_style_id or LEDService.resolve_style_id(model)
 
         # Initialize controller for this device
         self._led_controller.initialize(device, led_style)
@@ -2072,6 +2075,7 @@ def _on_device_poll(self):
                     protocol=d.get('protocol', 'scsi'),
                     device_type=d.get('device_type', 1),
                     implementation=implementation,
+                    led_style_id=d.get('led_style_id'),
                 )
                 if implementation == 'hid_led':
                     self._show_led_view(device)