Skip to content

Commit 8916f54

Browse files
committed
Remove 128-bit UUID from PM5 advertising to fix visibility
128-bit UUIDs are too large for the BLE advertising packet (31 bytes max). PM5 will now advertise with device name only. Apps will discover the PM5 services after establishing a connection. https://claude.ai/code/session_01XWBgRPWze8Mb7DEEm4oXiF
1 parent 896a4be commit 8916f54

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

src/android/src/BleAdvertiser.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -77,15 +77,15 @@ public static void startAdvertisingRowerPM5(Context context) {
7777
.setConnectable(true)
7878
.build();
7979

80-
// PM5 advertising - use discovery service UUID
81-
// Note: 128-bit UUIDs take more space, so we only advertise the discovery UUID
80+
// PM5 advertising - only use device name, no service UUID
81+
// 128-bit UUIDs are too large for advertising packet (31 bytes max)
82+
// Apps like Mywhoosh should discover services after connection
8283
AdvertiseData advertiseData = new AdvertiseData.Builder()
8384
.setIncludeDeviceName(true)
84-
.addServiceUuid(new ParcelUuid(PM5_DISCOVERY_SERVICE_UUID))
8585
.build();
8686

8787
if (advertiser != null) {
88-
QLog.d("BleAdvertiser", "Starting PM5 advertising with UUID: " + PM5_DISCOVERY_SERVICE_UUID.toString());
88+
QLog.d("BleAdvertiser", "Starting PM5 advertising (name only, no UUID)");
8989
advertiser.startAdvertising(settings, advertiseData, advertiseCallback);
9090
}
9191
}

0 commit comments

Comments
 (0)