Improve Mentra Live pairing UX (OS-1615) by nic-olo · Pull Request #3224 · Mentra-Community/MentraOS

nic-olo · 2026-06-22T08:34:41Z

Summary

Add Mentra Live pairing flow: prep guide, pairing-mode scan filter, auto-connect, and required media wipe on ownership transfer.
Extend Bluetooth SDK (iOS/Android) with pairing_info, wipe_media_result, and pairing transfer commands.
Add wipe_media handler on asg_client to erase gallery during ownership transfer.

Test plan

Flash BES firmware from companion PR and pair a factory-reset Mentra Live (auto pairing mode on boot).
Hold power+camera 10s on bonded glasses; confirm LED/voice and discoverability in scan.
Verify auto-connect on scan and 15s timeout + Try Again when no unit found.
Transfer ownership: confirm wipe prompt, successful wipe + finalize, and decline aborts pairing.
Confirm bonded phone reconnect still works via connectDefault() without scan list.

Made with Cursor

cloudflare-workers-and-pages · 2026-06-22T08:35:44Z

Deploying mentra-store-dev with Cloudflare Pages

Latest commit:	`628fb03`
Status:	✅ Deploy successful!
Preview URL:	https://f193e28b.augmentos-appstore-2.pages.dev
Branch Preview URL:	https://nicolo-os-1615-improve-mentr.augmentos-appstore-2.pages.dev

View logs

cloudflare-workers-and-pages · 2026-06-22T08:37:12Z

Deploying dev-augmentos-console with Cloudflare Pages

Latest commit:	`628fb03`
Status:	✅ Deploy successful!
Preview URL:	https://263f100c.dev-augmentos-console.pages.dev
Branch Preview URL:	https://nicolo-os-1615-improve-mentr.dev-augmentos-console.pages.dev

View logs

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

cloudflare-workers-and-pages · 2026-06-22T08:38:36Z

Deploying prod-augmentos-account with Cloudflare Pages

Latest commit:	`628fb03`
Status:	✅ Deploy successful!
Preview URL:	https://2a9c50ca.augmentos-e84.pages.dev
Branch Preview URL:	https://nicolo-os-1615-improve-mentr.augmentos-e84.pages.dev

View logs

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

cloudflare-workers-and-pages · 2026-06-22T08:38:57Z

Deploying mentra-live-ota-site with Cloudflare Pages

Latest commit:	`628fb03`
Status:	✅ Deploy successful!
Preview URL:	https://a622ce47.mentra-live-ota-site.pages.dev
Branch Preview URL:	https://nicolo-os-1615-improve-mentr.mentra-live-ota-site.pages.dev

View logs

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

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

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

github-actions · 2026-07-03T06:17:48Z

bugbot run

cubic-dev-ai

1 issue found across 4 files (changes from recent commits).

Prompt for AI agents (unresolved issues)


Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="mobile/src/app/pairing/scan.tsx">

<violation number="1" location="mobile/src/app/pairing/scan.tsx:169">
P2: Pressing 'Try Again' on iOS navigates two screens back. goBack() in handleTryAgain triggers the beforeRemove listener registered by focusEffectPreventBack(..., iosDontPreventBack=true), which also calls goBack(). The listener should handle cleanup (disconnect/forget) without an additional back navigation, or handleTryAgain should restart the scan in place instead of navigating.</violation>
</file>

<file name="mobile/modules/bluetooth-sdk/ios/Source/DeviceManager.swift">

<violation number="1" location="mobile/modules/bluetooth-sdk/ios/Source/DeviceManager.swift:1210">
P1: Pairing finalize/abort and wipe-media commands silently fail when the connected glasses aren't Mentra Live, because these methods use `(sgc as? MentraLive)?.sendXxx()` — an optional cast that silently discards the command when the cast fails. This is inconsistent with other MentraLive-only commands (`sendGalleryMode`, `sendCameraFovSetting`, etc.) that use `liveSgc()` which throws `BluetoothSdkError(unsupported_device)`. The risk is highest for `sendPairingFinalize` and `sendPairingAbort` (fire-and-forget with no pending-response timeout — the caller gets zero feedback), and for `sendWipeMediaForPairing` the failure only surfaces after a timeout. If the app is coordinating a pairing flow across steps, the silent failure means the JS layer thinks the command was sent while the glasses never received it.

Consider using the `liveSgc()` pattern like the rest of the file, or at minimum adding a `guard` with a Bridge.log warning.</violation>
</file>

<file name="mobile/modules/bluetooth-sdk/android/src/main/java/com/mentra/bluetoothsdk/BluetoothSdkModule.kt">

<violation number="1" location="mobile/modules/bluetooth-sdk/android/src/main/java/com/mentra/bluetoothsdk/BluetoothSdkModule.kt:473">
P2: Android `wipeMediaForPairing` uses blocking `CountDownLatch.await()` inside `PendingResponse`, which holds the Expo AsyncFunction worker thread until timeout if the glasses never respond (e.g., disconnect mid-wipe). The iOS side uses cancellable Swift structured concurrency. Consider wrapping the call in a coroutine with a cooperative cancellation scope so the thread can be released promptly on disconnect or abort.</violation>
</file>

<file name="agents/OS-1615-mentra-live-pairing-ux-plan.md">

<violation number="1" location="agents/OS-1615-mentra-live-pairing-ux-plan.md:16">
P1: According to linked Linear issue OS-1615, BT classic radio lifecycle hardening is a required part of this work, but this plan’s stated end state and implementation steps omit it. If implementation follows this doc as written, the always-on classic discoverable/connectable exposure called out in the issue can remain unaddressed. Consider adding explicit BT classic lifecycle tasks and acceptance criteria to this plan.</violation>

<violation number="2" location="agents/OS-1615-mentra-live-pairing-ux-plan.md:185">
P2: The pairing gesture is defined as a 10-second power+camera hold, but later sections still instruct a 5-tap camera flow. That inconsistency can lead to mismatched firmware/app behavior and incorrect user guidance during pairing. Align all references to one gesture path (or explicitly document both if intentional) so scan timeout copy, prep instructions, and implementation order all match.</violation>
</file>

_{Tip: Review your code locally with the cubic CLI to iterate faster.

Re-trigger cubic}

github-actions · 2026-07-03T07:56:42Z

bugbot run

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

github-actions · 2026-07-03T09:16:16Z

bugbot run

cursor

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

There are 7 total unresolved issues (including 6 from previous reviews).

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 5066707. Configure here.}

cursor · 2026-07-03T09:29:17Z

+      if (!connectingRef.current) {
+        setScanTimedOut(true)
+        void BluetoothSdk.stopScan()
+      }


Timeout hides device list

Medium Severity

The 15s Mentra Live scan timer sets scanTimedOut whenever connectingRef is false, without checking whether devices were already discovered. The UI then shows “No glasses found” and hides the multi-device picker.

Additional Locations (1)

mobile/src/app/pairing/scan.tsx#L198-L203

^{Reviewed by Cursor Bugbot for commit 5066707. Configure here.}

- scan.tsx: reset connectingRef on permission denial so 15s timeout still fires - loading.tsx: dedicated effect catches late pairing_info and cancels pending navigation - loading.tsx: in-flight guard on checkGalleryAndHandleOwnershipTransfer - loading.tsx: back/cancel aborts pairing transfer when ownership transfer is in progress - en.ts: align noGlassesFoundHint copy with 5x click gesture - WipeMediaCommandHandler: children==null returns false (wipe_media_result.success=false) - MentraLive.swift: forget() clears PREFS_DEVICE_NAME to prevent pairing filter bypass on rescan Co-authored-by: Cursor <cursoragent@cursor.com>

cubic-dev-ai

1 issue found across 4 files (changes from recent commits).

Prompt for AI agents (unresolved issues)


Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="mobile/src/app/pairing/loading.tsx">

<violation number="1" location="mobile/src/app/pairing/loading.tsx:268">
P2: If the user exits this screen during the 1-second delay before success navigation (for example, by tapping the header back button or swiping back on iOS), the pending `setTimeout` is never cleared on unmount. When it eventually fires, `replace('/pairing/success')` runs from a stale context and can unexpectedly jump the user to the success screen. Consider adding a cleanup effect that clears `navigationTimerRef.current` when the component unmounts.</violation>
</file>

_{Tip: Review your code locally with the cubic CLI to iterate faster.

Re-trigger cubic}

cubic-dev-ai · 2026-07-03T09:48:42Z

+
    hasNavigatedRef.current = true
-    setTimeout(() => {
+    navigationTimerRef.current = setTimeout(() => {


P2: If the user exits this screen during the 1-second delay before success navigation (for example, by tapping the header back button or swiping back on iOS), the pending setTimeout is never cleared on unmount. When it eventually fires, replace('/pairing/success') runs from a stale context and can unexpectedly jump the user to the success screen. Consider adding a cleanup effect that clears navigationTimerRef.current when the component unmounts.

Prompt for AI agents

Check if this issue is valid — if so, understand the root cause and fix it. At mobile/src/app/pairing/loading.tsx, line 268: <comment>If the user exits this screen during the 1-second delay before success navigation (for example, by tapping the header back button or swiping back on iOS), the pending `setTimeout` is never cleared on unmount. When it eventually fires, `replace('/pairing/success')` runs from a stale context and can unexpectedly jump the user to the success screen. Consider adding a cleanup effect that clears `navigationTimerRef.current` when the component unmounts.</comment> <file context> @@ -234,14 +258,14 @@ export default function GlassesPairingLoadingScreen() { hasNavigatedRef.current = true - setTimeout(() => { + navigationTimerRef.current = setTimeout(() => { replace("/pairing/success", {deviceModel: deviceModel}) }, 1000) </file context>

github-actions · 2026-07-03T10:14:36Z

bugbot run

nic-olo requested review from AryanFP, PhilippeFerreiraDeSousa and aisraelov as code owners June 22, 2026 08:34

cursor Bot reviewed Jun 22, 2026

View reviewed changes

Comment thread mobile/modules/bluetooth-sdk/ios/Source/sgcs/MentraLive.swift Outdated

Comment thread mobile/src/app/pairing/loading.tsx

Comment thread mobile/src/app/pairing/scan.tsx

Comment thread ...t/app/src/main/java/com/mentra/asg_client/service/core/handlers/WipeMediaCommandHandler.java

nic-olo and others added 4 commits June 22, 2026 16:37

Add WipeMediaCommandHandler for ownership-transfer media wipe

3211405

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

Register wipe_media handler in CommandProcessor

f5ffcdb

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

Add pairing transfer types and SDK methods to BluetoothSdk.types

9258865

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

Expose wipe and pairing transfer methods on native BluetoothSdk module

28c27f9

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

nic-olo and others added 4 commits June 22, 2026 16:38

Export pairing events and transfer methods from public BluetoothSdk

f7fb0b6

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

Add WipeMediaResultEvent to Android Bluetooth SDK events

e8c6d08

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

Add pairing_info and wipe_media_result bridge events on Android

c6761a0

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

Route pairing transfer commands through DeviceManager on Android

fac56c8

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

nic-olo and others added 14 commits June 22, 2026 16:39

Handle pairing transfer promises in MentraBluetoothSdk on Android

21bac94

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

Add pairing transfer async functions to Android BluetoothSdkModule

35dd999

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

Filter pairing-mode adv and handle pairing messages in MentraLive And…

23d71d8

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

Add pairing_info and wipe_media_result bridge events on iOS

e60f6be

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

Route pairing transfer commands through DeviceManager on iOS

ee0a0b4

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

Handle pairing transfer promises in MentraBluetoothSDK on iOS

1c08570

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

Add WipeMediaResultEvent model for iOS Bluetooth SDK

276c69d

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

Filter pairing-mode adv and handle pairing messages in MentraLive iOS

e299138

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

Add pairing transfer async functions to iOS BluetoothSdkModule

2de0004

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

Re-export pairing types from island module

af8e1fc

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

Add Mentra Live two-step pairing prep guide

0f59447

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

Auto-connect first pairing-mode unit with 15s scan timeout

fad68ea

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

Require media wipe confirmation during Mentra Live ownership transfer

bca1971

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

Add i18n strings for Mentra Live pairing UX

640ab30

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

fix(pr-agent): round 2 [pr-agent-fix]

172e79d

cubic-dev-ai Bot reviewed Jul 3, 2026

View reviewed changes

Comment thread mobile/src/app/pairing/loading.tsx

github-actions Bot added agent-in-progress and removed agent-in-progress labels Jul 3, 2026

fix(pr-agent): round 3 [pr-agent-fix]

60f9a37

cursor Bot reviewed Jul 3, 2026

View reviewed changes

Comment thread mobile/src/app/pairing/loading.tsx

github-actions Bot removed the agent-in-progress label Jul 3, 2026

Only prompt wipe during ownership transfer if gallery has content

3cb54ba

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

github-actions Bot added the agent-in-progress label Jul 3, 2026

github-actions Bot removed the agent-in-progress label Jul 3, 2026

cursor Bot reviewed Jul 3, 2026

View reviewed changes

Comment thread mobile/src/app/pairing/loading.tsx

github-actions Bot added the agent-in-progress label Jul 3, 2026

fix(pr-agent): round 4 [pr-agent-fix]

5066707

cursor Bot reviewed Jul 3, 2026

View reviewed changes

github-actions Bot added agent-in-progress and removed agent-in-progress labels Jul 3, 2026

cubic-dev-ai Bot reviewed Jul 3, 2026

View reviewed changes

github-actions Bot added agent-in-progress and removed agent-in-progress labels Jul 3, 2026

aisraelov mentioned this pull request Jul 7, 2026

Overhaul Mentra Live Bluetooth connection & pairing (UX + security) #3131

Open

Uh oh!

Conversation

nic-olo commented Jun 22, 2026

Summary

Test plan

Uh oh!

cloudflare-workers-and-pages Bot commented Jun 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Deploying mentra-store-dev with Cloudflare Pages

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

cloudflare-workers-and-pages Bot commented Jun 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Deploying dev-augmentos-console with Cloudflare Pages

Uh oh!

cloudflare-workers-and-pages Bot commented Jun 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Deploying prod-augmentos-account with Cloudflare Pages

Uh oh!

cloudflare-workers-and-pages Bot commented Jun 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Deploying mentra-live-ota-site with Cloudflare Pages

Uh oh!

github-actions Bot commented Jul 3, 2026

Uh oh!

cubic-dev-ai Bot left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

github-actions Bot commented Jul 3, 2026

Uh oh!

Uh oh!

github-actions Bot commented Jul 3, 2026

Uh oh!

Uh oh!

cursor Bot left a comment

Choose a reason for hiding this comment

Uh oh!

cursor Bot Jul 3, 2026

Choose a reason for hiding this comment

Timeout hides device list

Uh oh!

cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

cubic-dev-ai Bot Jul 3, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot commented Jul 3, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

cloudflare-workers-and-pages Bot commented Jun 22, 2026 •

edited

Loading

cloudflare-workers-and-pages Bot commented Jun 22, 2026 •

edited

Loading

cloudflare-workers-and-pages Bot commented Jun 22, 2026 •

edited

Loading

cloudflare-workers-and-pages Bot commented Jun 22, 2026 •

edited

Loading

cubic-dev-ai Bot left a comment •

edited

Loading