Skip to content

Add missing ble deps#207

Merged
Apollon77 merged 6 commits intomainfrom
add-ble-deps
Feb 4, 2026
Merged

Add missing ble deps#207
Apollon77 merged 6 commits intomainfrom
add-ble-deps

Conversation

@Apollon77
Copy link
Collaborator

addresses #180

Copilot AI review requested due to automatic review settings February 4, 2026 08:56
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the standalone Docker image so that BLE can function correctly out of the box, addressing the missing native dependencies that prevented @stoprocent/noble from being installed/loaded.

Changes:

  • Extend the matterjs-server Dockerfile to install system build tools and Bluetooth libraries required for BLE support.
  • Document the fix in the changelog, noting that missing Docker package dependencies for BLE have been added.

Reviewed changes

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

File Description
docker/matterjs-server/Dockerfile Adds build tools and Bluetooth-related system libraries before npm install so BLE native addons (e.g. @stoprocent/noble) can compile and be available at runtime.
CHANGELOG.md Records the BLE Docker dependency fix in the project’s change log.

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

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

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


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

@aptalca
Copy link
Contributor

aptalca commented Feb 4, 2026

Please see #209, which will update this PR

@Apollon77
Copy link
Collaborator Author

Thanks a lot. included and I will merge. I will comment the issue when it got release, but next nightly tomorrow morning shouod already have it included (@dev tag)

@Apollon77 Apollon77 enabled auto-merge (squash) February 4, 2026 23:15
@Apollon77 Apollon77 merged commit 65ab326 into main Feb 4, 2026
11 checks passed
@Apollon77 Apollon77 deleted the add-ble-deps branch February 4, 2026 23:16
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.

2 participants