Update Bluetooth to v3.0.0 - #1106
Conversation
Issues FoundNo issues with your PR description were found. Next Steps
Thank you for your contribution! If you need any help, please reach out on our Discord server. ❤️ |
Plugin Testing ReportI manually installed the plugin at the commit referenced by this PR's submodule. Installed Plugins
Specifications
IssuesHas the following major blocking issue(s): N/A SummaryTested with a few different Bluetooth devices that report their battery life
Devices tested with:
|
Testing report — Bluetooth v3.0.0Hardware: Steam Machine (not a Deck), SteamOS Stable, Decky Loader 3.2.6, BlueZ 5.83, single adapter Works
Issues1. The device row doesn't update after you connect or disconnect it. In export async function toggleDeviceConnection(device: Device): Promise<void> {
controller.toggleDeviceConnection(device.mac, device.connected).catch(console.error);
}The inner promise isn't returned, so the 2. D-Bus error replies are never checked. 3.
4. The battery readout won't fire for PlayStation controllers. A 5. 6. Hardcoded 7. Minor: with no paired devices the list renders as an empty gap — no "no devices" message. One loose end: this turned up once in the plugin log during connect testing and I could not establish what produced it, so take it as an observation rather than a diagnosis. Not testedA device that actually exposes |
Update Bluetooth to v3.0.0
bluethoothctlreplacement withD-BusPlugin used subprocess calls to
bluetoothctlsince the beginning but it was always a bit unreliable because it conflicted with Steam. Then Steam sharedSteamClient.System.Bluetoothwhich was much more reliable but it did not last long as they removed it.Fed up with this and getting issues opened I finally took some time and switched the project to using
dbuswith thejeepneypython library instead.Devices now display properly and also got the added bonus to retrieve the current battery level if available.
Full changelog
Task Checklist
Developer
Plugin
Backend
Community
Testing