Skip to content

Document required BLE GATT services (#42)#44

Closed
ilouzl wants to merge 2 commits into
masterfrom
cursor/ble-gatt-services-docs-ca48
Closed

Document required BLE GATT services (#42)#44
ilouzl wants to merge 2 commits into
masterfrom
cursor/ble-gatt-services-docs-ca48

Conversation

@ilouzl

@ilouzl ilouzl commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Summary

Firmware-verified BLE GATT reference for #42, checked against Bitbucket tap_rd/ble-mcu-hw-v3-sdk-14.0 branch TAP_XR_develop.

Re-evaluation vs first pass

Finding First pass After TAP_XR_develop
Core SDK UUIDs (Tap + NUS RX/TX + mouse/UI/air) Correct Confirmed in ble_tap_service.c / Nordic ble_nus
Required for this SDK Tap svc + Tap data + NUS + NUS RX Still correct (FW always exposes them; SDK minimum unchanged)
Device services Only DIS + Battery listed Also HID, buttonless DFU, Tap, NUS
Tap-service chars Mentioned name/data-request briefly Full map: settings 0002, name 0003, mapping 0007/0008, request 000b, Kneron 000c/000d
Advertising Implied Tap UUID in adv Tap UUID is in scan response; primary adv is HID
Char properties Simplified Match FW (notify/write/write_wo_resp/read as coded)

Required by this SDK

  • Tap service c3ff0001-… (scan response)
  • Tap data c3ff0005-… (notify)
  • NUS 6e400001-… + NUS RX 6e400002-… (write mode)

Test plan

  • Diff UUIDs against ble_tap_service.c / TAP_BASE_UUID on TAP_XR_develop
  • Confirm services_init() registers DFU, BAS, HID, Tap, NUS (+ DIS via mediator)
  • Skim README section for clarity
  • Optional: GATT dump from a TapXR on current FW

cursoragent and others added 2 commits July 21, 2026 18:03
Add a BLE GATT reference section to the README listing the Tap and NUS
services this SDK requires, plus optional feature characteristics, aligned
with the other official Tap SDKs and internal BLE API docs (#42).

Co-authored-by: Liron Ilouz <ilouzl@users.noreply.github.com>
Re-check UUIDs and properties from ble-mcu-hw-v3-sdk-14.0, expand the
README with the full firmware service/characteristic map, and note that
the Tap UUID is advertised in the scan response (#42).

Co-authored-by: Cursor <cursoragent@cursor.com>
@ilouzl

ilouzl commented Jul 21, 2026

Copy link
Copy Markdown
Contributor Author

This was a mistake. referred to the wrong issue

@ilouzl ilouzl closed this Jul 21, 2026
@ilouzl
ilouzl deleted the cursor/ble-gatt-services-docs-ca48 branch July 21, 2026 20:28
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