Skip to content

Updates platform feature selection for non-local platforms#635

Open
hmeerlo wants to merge 1 commit into
developfrom
feature/ble-advertisements
Open

Updates platform feature selection for non-local platforms#635
hmeerlo wants to merge 1 commit into
developfrom
feature/ble-advertisements

Conversation

@hmeerlo

@hmeerlo hmeerlo commented May 28, 2026

Copy link
Copy Markdown
Contributor

Ensures feature list defaults to camera streaming and BLE advertisements when running on non-local platforms, improving compatibility and clarity of reported capabilities. Prevents reliance on unavailable local feature detection outside local environments.

Ensures feature list defaults to camera streaming and BLE advertisements
when running on non-local platforms, improving compatibility and clarity of
reported capabilities. Prevents reliance on unavailable local feature detection
outside local environments.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adjusts how homeyPlatformFeatures is reported when creating the remote client, avoiding local feature detection on non-local platforms and providing a consistent default capability set.

Changes:

  • Conditionally uses HomeyLibUtil.getPlatformLocalFeatures(homey.model) only when homey.platform === 'local'.
  • Defaults non-local platforms to ['camera-streaming', 'ble-advertisements'] for reported platform features.

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

Comment thread lib/App.js
@hmeerlo hmeerlo requested a review from jeroenwienk May 31, 2026 17:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants