Add Broadlink infrared emitter support#168889
Merged
abmantis merged 5 commits intoApr 29, 2026
Merged
Conversation
home-assistant
Bot
added
cla-signed
has-tests
integration: broadlink
new-feature
new-platform
Top 100
Contributor
|
Hey there @Danielhiversen, @felipediel, @L-I-Am, @eifinger, mind taking a look at this pull request as it has been labeled with an integration ( Code owner commandsCode owners of
|
8 tasks
Contributor
There was a problem hiding this comment.
Pull request overview
Adds Broadlink RM-series IR emitter support to Home Assistant’s native infrared platform, enabling Broadlink devices to transmit infrared_protocols commands through a dedicated infrared entity.
Changes:
- Add a new Broadlink
infraredplatform entity that encodes raw timings into Broadlink IR packets and sends them viasend_data. - Forward
Platform.INFRAREDonly for IR-capable Broadlink RM device types. - Add translations and tests covering entity creation, command sending, and error translation behavior.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
homeassistant/components/broadlink/infrared.py |
Implements Broadlink InfraredEntity that converts timings to Broadlink packets and transmits them. |
homeassistant/components/broadlink/const.py |
Adds Platform.INFRARED to the forwarded platforms for RM device types. |
homeassistant/components/broadlink/strings.json |
Adds entity name translation and a translated exception message for send failures. |
tests/components/broadlink/test_infrared.py |
Adds tests for IR entity setup, send behavior, and error translation mapping. |
YuvalWS
force-pushed
the
broadlink-infrared-followup
branch
from
0319c03 to
7a44b47
Compare
joostlek
reviewed
Apr 23, 2026
| def __init__(self, device: BroadlinkDevice) -> None: | ||
| """Initialize the entity.""" | ||
| super().__init__(device) | ||
| self._attr_unique_id = f"{device.unique_id}-infrared" |
Member
There was a problem hiding this comment.
we don't need to add infrared as unique ids are unique per integration pet platform
abmantis
requested changes
Apr 28, 2026
Member
abmantis
left a comment
abmantis
left a comment
There was a problem hiding this comment.
LGTM, just some minor suggestions
| ] | ||
| assert len(infrared_entities) == 1 | ||
| assert infrared_entities[0].unique_id == device.mac | ||
| assert mock_setup.api.auth.call_count == 1 |
Member
There was a problem hiding this comment.
Suggested change
| assert mock_setup.api.auth.call_count == 1 |
is this related to the infrared platform?
Contributor
|
Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍 |
abmantis
reviewed
Apr 28, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Breaking change
Proposed change
Adds infrared transmitter support for Broadlink RM devices (RM4MINI, RM4PRO, RMMINI, RMMINIB, RMPRO) to Home Assistant's native infrared platform.
Type of change
Additional information
Reintroduces Broadlink infrared emitter support on the native
infraredplatform. Follow-up to #168385 (reverted in #168717) that addresses all
review feedback before re-landing.
Changes vs. the reverted PR:
Drop
BroadlinkIRCommand. The speculative subclass had no UI/YAMLpath and couldn't be constructed by standard consumers
Rewrite tests against the public dispatcher. Tests now call
homeassistant.components.infrared.async_send_commandand assert onmock_setup.api.send_data, instead of reaching into integrationinternals
Remove
"dependencies": ["infrared"]from the manifest. OnlyIR-capable RM models get
Platform.INFRAREDforwarded viaDOMAINS_AND_TYPES, so every non-IR Broadlink device no longer paysfor loading the infrared base integration (feedback from Copilot).
Tested on: TCL 49C2US TV via Broadlink RM4 remote (power, volume, source commands verified working)
This PR fixes or closes issue: fixes #
This PR is related to issue: This PR is a follow up to previous PR which was reverted
Link to documentation pull request: Document Broadlink Infrared integration support home-assistant.io#44968
Link to developer documentation pull request:
Link to frontend pull request:
Checklist
ruff format homeassistant tests)If user exposed functionality or configuration variables are added/changed:
If the code communicates with devices, web services, or third-party tools:
Updated and included derived files by running:
python3 -m script.hassfest.requirements_all.txt.Updated by running
python3 -m script.gen_requirements_all.To help with the load of incoming pull requests: