Skip to content

fix: populate manufacturer data entries in scan results - #50

Merged
sheehanmunim merged 1 commit into
munimtechnologies:mainfrom
PlayerData:fix/manufacturer-data-company-id
Oct 8, 2026
Merged

sheehanmunim merged 1 commit into
munimtechnologies:mainfrom
PlayerData:fix/manufacturer-data-company-id

Conversation

@saty9

@saty9 saty9 commented Oct 8, 2026 •

Copy link
Copy Markdown
Contributor

BLEDevice declares manufacturerDataEntries, but scan results never set it. Android reported only the first manufacturer entry, dropping its company identifier, while iOS reported CoreBluetooth's raw blob with the little endian company identifier still prefixed.

Report every entry as { companyId, data } with the company identifier parsed out on both platforms. manufacturerData is left untouched for existing callers and deprecated.

Fixes "iOS: the deviceFound payload differs from Android (serviceData and the manufacturer-data prefix)."

BLEDevice declares manufacturerDataEntries, but scan results never set it.
Android reported only the first manufacturer entry, dropping its company
identifier, while iOS reported CoreBluetooth's raw blob with the little
endian company identifier still prefixed.

Report every entry as { companyId, data } with the company identifier
parsed out on both platforms. manufacturerData is left untouched for
existing callers and deprecated.
@sheehanmunim
sheehanmunim merged commit 480d6d8 into munimtechnologies:main Oct 8, 2026
3 checks passed
sheehanmunim pushed a commit that referenced this pull request Oct 8, 2026
## [0.9.1](v0.9.0...v0.9.1) (2026-10-08)

### 🐛 Bug Fixes

* iOS GATT promise mix-ups and crashes, Android short UUIDs, plugin scan permission ([#49](#49)) ([943959d](943959d))
* populate manufacturer data entries in scan results ([#50](#50)) ([480d6d8](480d6d8))

### 📚 Documentation

* move Installation up and list Expo first ([b31f5fd](b31f5fd))
@sheehanmunim

Copy link
Copy Markdown
Member

🎉 This PR is included in version 0.9.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants