[hue] Fix firmware availability issues

[andrewfg]

Resolves #20702
Depends on #20707

Signed-off-by: Andrew Fiddian-Green software@whitebear.ch

[Copilot]

Pull request overview

Fixes Hue API v2 bridge firmware update availability signaling so the trigger channel matches the documented channel group and avoids repeated hourly trigger events.

Changes:

• Prefix the update-ready trigger channel ID with the software channel group (software#update-ready).
• Add a handler-side guard to only trigger the “update ready” event once per runtime.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File	Description
bundles/org.openhab.binding.hue/src/main/java/org/openhab/binding/hue/internal/HueBindingConstants.java	Adjustes the update-ready channel ID to include the software group separator.
bundles/org.openhab.binding.hue/src/main/java/org/openhab/binding/hue/internal/handler/Clip2BridgeHandler.java	Prevents repeated triggering of the update-ready event by tracking whether a notification was already sent.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[jlaur]

Thanks! I have not yet had time to test this, but did a quick review for now.

[Copilot]

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)

bundles/org.openhab.binding.hue/src/main/java/org/openhab/binding/hue/internal/handler/Clip2BridgeHandler.java:1

• softwareUpdateReadyNotificationSent is unintentionally reset to false when status == READY_TO_INSTALL and the notification was already sent. In that case, newSentFlagValue stays false and the assignment at the end clears the flag, causing the trigger to fire again on the next refresh. Fix by ensuring the “sent” flag remains true for as long as the status remains READY_TO_INSTALL (e.g., initialize newSentFlagValue from wasSentFlagValue, or explicitly set newSentFlagValue = wasSentFlagValue || status == READY_TO_INSTALL and only call triggerChannel on the transition from not-sent to sent).

/*

[jlaur]

Tested and reviewed, LGTM.