Skip to content

feat: drive WLED Wi-Fi adapters over WebSocket - #2

Merged
GOG1071 merged 1 commit into
mainfrom
feat/wled-websocket-transport
Oct 27, 2026
Merged

GOG1071 merged 1 commit into
mainfrom
feat/wled-websocket-transport

Conversation

@GOG1071

@GOG1071 GOG1071 commented Oct 26, 2026

Copy link
Copy Markdown
Owner

Summary

  • When port is a host/IP instead of a /dev path, frames are sent as per-LED JSON over the WLED WebSocket (/ws) instead of serial.
  • Effect that is running before connect is restored on close.
  • skydimo info shows the link type.
  • Adds the websockets>=12 dependency (pyproject.toml, install.sh).

Why WebSocket and not UDP

The vendor WLED firmware has every realtime UDP port closed (verified on the device). UDP DRGB, DDP, E1.31 and Art-Net therefore cannot work against it. The WebSocket is the realtime path that does.

Test plan

  • 38 unit tests pass.
  • Live-verified on real hardware at 192.168.1.38 (144 LEDs): red, green, blue and white, chase, and effect restore on disconnect.

Note for reviewers

led_count must be set to the strip's real LED count. For this device that is 144.

🤖 Generated with Claude Code

When the configured port is a host/IP instead of a /dev path, frames are sent as per-LED JSON over the WLED WebSocket (/ws) rather than serial.

The vendor WLED firmware has every realtime UDP port closed (verified on hardware), so UDP DRGB/DDP/E1.31/Art-Net cannot work; the WebSocket is the only realtime path that does.

- device.py: WebSocket transport alongside serial; effect that is running before connect is restored on close
- cli.py: 'skydimo info' shows the link type
- pyproject.toml, install.sh: add websockets>=12 dependency
- README.md: document Wi-Fi setup; led_count must match the strip's real LED count
- tests/test_core.py: unit tests for the transport

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@GOG1071
GOG1071 merged commit 2e7f592 into main Oct 27, 2026
2 of 6 checks passed
@GOG1071
GOG1071 deleted the feat/wled-websocket-transport branch October 27, 2026 01:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant