Fix color parsing and pulse effect implementation

- Add case-insensitive hex prefix handling (0x/0X) in color parsing
- Prioritize hex color parsing before named color lookups
- Remove broken HTML5 color parsers that had typo
- Fix pulse effect to use correct gradient parameters and manager method
- Add missing logger import to udev_rules subcommand

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>

Author: JnyJny

src/busylight/color.py

@@ -32,9 +32,17 @@ def parse_color_string(value: str, scale: float = 1.0) -> tuple[int, int, int]:
     """
     scale = max(0.0, min(scale, 1.0))
 
-    if value.startswith("0x"):
+    if value.startswith("0x") or value.startswith("0X"):
         value = f"#{value[2:]}"
 
+    if value.startswith("#"):
+        try:
+            rgb = webcolors.hex_to_rgb(value)
+            return scale_color(rgb, scale)
+        except ValueError as error:
+            logger.debug(f"No match found for {value} -> {error}")
+            raise ColorLookupError(f"No color mapping for {value}") from error
+
     for spec in [
         webcolors.CSS3,
         webcolors.CSS2,
@@ -47,16 +55,6 @@ def parse_color_string(value: str, scale: float = 1.0) -> tuple[int, int, int]:
         except ValueError as error:
             logger.info(f"name_to_rgb[{spec}] {value} ->  {error}")
 
-    for parse_color in [
-        webcolors.html5_parse_legacy_color,
-        webcolors.html5_parse_simple_color,
-    ]:
-        try:
-            rgb = parse_color(value)
-            return scale_color(rgb, scale)
-        except ValueError as error:
-            loggerinfo(f"[parse] {value} -> {error}")
-
     raise ColorLookupError(f"No color mapping for {value}")
 
 

src/busylight/subcommands/pulse.py

@@ -41,14 +41,10 @@ def pulse_lights(
 
     logger.info("Applying pulse effect.")
 
-    effect = Effects.for_name("gradient")(
-        speed.duty_cycle / 4,
-        scale=ctx.obj.dim,
-        count=count,
-    )
+    effect = Effects.for_name("gradient")(color, speed.duty_cycle / 16, 8, count=count)
 
     try:
-        ctx.obj.manager.apply(effect, ctx.obj.lights, timeout=ctx.obj.timeout)
+        ctx.obj.manager.apply_effect(effect, ctx.obj.lights, timeout=ctx.obj.timeout)
     except (KeyboardInterrupt, TimeoutError):
         ctx.obj.manager.off(ctx.obj.lights)
     except NoLightsFoundError:

src/busylight/subcommands/udev_rules.py

@@ -2,6 +2,7 @@
 
 import typer
 from busylight_core import Light
+from loguru import logger
 
 udev_rules_cli = typer.Typer(help="Generate udev rules for supported lights.")