Skip to content

Bluetooth control in Cedar Aim #7

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
oakamil wants to merge 11 commits into
base: main
Choose a base branch
from
Open

Bluetooth control in Cedar Aim #7

oakamil wants to merge 11 commits into from

Conversation

@oakamil
Copy link
Contributor

@oakamil oakamil commented Jan 27, 2026

Currently only allowed for DIY devices. A few notes:

  • Currently selected device to control is stored in a local shared preference
  • Selecting another device to control is possible if we can't establish connection for 5 seconds
  • WiFi control is only allowed on 192.168.4.1
  • Bluetooth control allows any paired device at the moment. It's possible to filter devices using the Cedar control UUID, but there are edge cases that would be hard to deal with since the service UUIDs are only pulled at the time of pairing
  • Cedar Aim will try to handle various edge cases - missing permissions, Bluetooth off, etc. But if user repeatedly denies permission they will have to go to their phone settings to grant them. Cedar Aim currently doesn't walk them through that edge case

Omair Kamil and others added 10 commits January 6, 2026 22:34
Implement a picture-in-picture mode on Android for guidance.
b8ec980
Revert "Implement a picture-in-picture mode on Android for guidance."
1df7328 
This reverts commit b8ec980.
@smroid
update proto compilation and deps
bee4274
Merge branch 'bluetooth' of https://github.com/oakamil/cedar-aim
b4b4a8a
More Bluetooth control implementation including basic device selection.
6bf59e8
Allow 5s for connection to establish before showing error.
0da5fc0 
Also update colors for device selection dialog to match theme.
Restore formatting of unchanged sections in client_main.dart.
9998fca
Check permissions before trying to connect using flutter_blue_classic.
024191f 
There's a bug in that flutter library that will cause a crash in some cases when permissions are missing.
Merge remote-tracking branch 'origin/main' into bluetooth
18a182a
More Bluetooth control updates.
7c1f791 
- Ensure correct permissions are requested based on Android SDK
- Try to turn on Bluetooth if it's off
- Don't cleanup unless 5 seconds have passed since connection start to reduce calls on expensive operations
- Assorted fixes
@oakamil
Copy link
Contributor Author

oakamil commented Jan 27, 2026

On filtering devices based on Cedar UUID, this isn't supported by the flutter_blue_classic package and might need custom platform channels to do. So I don't have plans to add that at this time.

Remove isDIY check for device selection handling flow.
ff671df 
Prior to connecting to a Cedar device it's unknown if the device isDIY or not.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@oakamil @smroid