make hhd remove leftover udev rules when starting/exiting

Author: antheas

pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "hhd"
-version = "0.2.4"
+version = "0.2.5"
 authors = [
   { name="Kapenekakis Antheas", email="[email protected]" },
 ]

src/hhd/controller/lib/hide.py

@@ -85,3 +85,12 @@ def unhide_gamepad(devpath: str):
         return reload_children(parent)
     except Exception:
         return False
+
+
+def unhide_all():
+    for rule in os.listdir("/run/udev/rules.d/"):
+        if rule.startswith("95-hhd-devhide"):
+            try:
+                os.remove(os.path.join("/run/udev/rules.d/", rule))
+            except Exception:
+                pass

src/hhd/controller/physical/evdev.py

@@ -2,8 +2,6 @@
 import logging
 import os
 import re
-import select
-import stat
 import subprocess
 from typing import Mapping, Sequence, TypeVar, cast, Collection
 
@@ -13,7 +11,7 @@
 from hhd.controller import Axis, Button, Consumer, Event, Producer, can_read
 from hhd.controller.base import Event
 from hhd.controller.lib.common import hexify, matches_patterns
-from hhd.controller.lib.hide import hide_gamepad, unhide_gamepad
+from hhd.controller.lib.hide import hide_gamepad, unhide_gamepad, unhide_all
 from hhd.controller.const import AbsAxis, GamepadButton, KeyboardButton
 
 logger = logging.getLogger(__name__)

src/hhd/device/legion_go/base.py

@@ -11,7 +11,7 @@
 from hhd.controller.base import Multiplexer
 from hhd.controller.lib.hid import enumerate_unique
 from hhd.controller.physical.evdev import B as EC
-from hhd.controller.physical.evdev import GenericGamepadEvdev
+from hhd.controller.physical.evdev import GenericGamepadEvdev, unhide_all
 from hhd.controller.physical.hidraw import GenericGamepadHidraw
 from hhd.controller.physical.imu import AccelImu, GyroImu
 from hhd.controller.virtual.ds5 import DualSense5Edge, TouchpadCorrectionType
@@ -51,6 +51,8 @@
 
 
 def plugin_run(conf: Config, emit: Emitter, context: Context, should_exit: TEvent):
+    # Remove leftover udev rules
+    unhide_all()
     if (gyro_fix := conf.get("gyro_fix", False)) and conf["gyro"].to(bool):
         gyro_fixer = GyroFixer(int(gyro_fix) if int(gyro_fix) > 10 else 100)
     else:
@@ -110,6 +112,8 @@ def plugin_run(conf: Config, emit: Emitter, context: Context, should_exit: TEven
         finally:
             if gyro_fixer:
                 gyro_fixer.close()
+            # Remove leftover udev rules
+            unhide_all()
 
 
 def controller_loop_rest(mode: str, pid: int, conf: Config, should_exit: TEvent):