fix(ideapad-conservation-mode): use bind action in udev rule instead of add|change to avoid boot race (#61)

ideapad-conservation-mode/conservation_mode.luau

@@ -62,7 +62,7 @@ local function readState()
 end
 
 local function render(on, enabled)
-  shortcut.setLabel(noctalia.tr(if on then "shortcut.on" else "shortcut.off"))
+  shortcut.setLabel(noctalia.tr("shortcut.label"))
   shortcut.setIcon("battery-charging", "battery")
   shortcut.setActive(on)
   shortcut.setEnabled(enabled)

ideapad-conservation-mode/plugin.toml

@@ -1,6 +1,6 @@
 id           = "lux/ideapad-conservation-mode"
 name         = "IdeaPad Conservation Mode"
-version      = "0.3.0"
+version      = "0.3.1"
 plugin_api = 3
 author       = "lux"
 license      = "MIT"

ideapad-conservation-mode/scripts/setup-permissions.sh

@@ -41,11 +41,11 @@ usermod -aG "$GROUP_NAME" "$TARGET_USER"
 
 echo "Writing $RULE_FILE..."
 cat >"$RULE_FILE" <<EOF
-ACTION=="add|change", SUBSYSTEM=="platform", DRIVER=="ideapad_acpi", RUN+="$CHGRP_BIN $GROUP_NAME /sys%p/conservation_mode", RUN+="$CHMOD_BIN 664 /sys%p/conservation_mode"
+ACTION=="bind", SUBSYSTEM=="platform", DRIVER=="ideapad_acpi", RUN+="$CHGRP_BIN $GROUP_NAME /sys%p/conservation_mode", RUN+="$CHMOD_BIN 664 /sys%p/conservation_mode"
 EOF
 
 echo "Reloading udev rules..."
 udevadm control --reload-rules
-udevadm trigger --subsystem-match=platform
+udevadm trigger --action=bind --subsystem-match=platform
 
 echo "Done."

ideapad-conservation-mode/translations/en.json

@@ -12,8 +12,7 @@
   },
   "shortcut": {
     "na": "Conservation N/A",
-    "off": "Conservation",
+    "label": "Conservation"
-    "on": "Conservation"
   },
   "title": "Conservation mode"
 }