Conversation
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (46)
📝 WalkthroughWalkthroughThis pull request introduces debug method stubs across mobile platforms, implements a G2 glasses dashboard menu configuration system with synchronization, adds controller connection management, defines a mini-app selection event, and removes microphone-related SDK APIs while making various UI improvements. Changes
Sequence Diagram(s)sequenceDiagram
actor User
participant Settings as GlassesMenuScreen
participant Store as GlassesStore / AppletStore
participant Util as glassesMenu Util
participant Module as CoreModule
participant SGC as SGC (Native)
participant Hardware as G2 Hardware
User->>Settings: Open glasses menu config
Settings->>Store: Load saved glasses_menu_apps
Settings->>Util: getDefaultMenuApps() if none saved
Settings->>Settings: Render menu items, allow drag/reorder
User->>Settings: Reorder or add/remove items
Settings->>Store: setSavedMenuApps()
Settings->>Util: syncDashboardMenu()
Util->>Store: Get saved menu items
Util->>Store: Resolve applet running state
Util->>Module: updateCore({dashboard_menu_apps: items})
Module->>SGC: setDashboardMenu(items)
SGC->>Hardware: Send menu config
Hardware-->>SGC: ACK
SGC-->>Module: Complete
Module-->>Util: Sync complete
Estimated code review effort🎯 4 (Complex) | ⏱️ ~60 minutes Suggested reviewers
Poem
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ast-grep (0.42.1)mobile/modules/core/android/src/main/java/com/mentra/core/sgcs/MentraLive.javaThanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
📋 PR Review Helper📱 Mobile App Build✅ Ready to test! (commit 🕶️ ASG Client Build⏳ Waiting for build... 🔀 Test Locallygh pr checkout 2463 |
…to g2-continued-work
Deploying mentra-live-ota-site with
|
| Latest commit: |
939f400
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://606dfd2c.mentra-live-ota-site.pages.dev |
| Branch Preview URL: | https://g2-continued-work.mentra-live-ota-site.pages.dev |
Deploying mentra-store-dev with
|
| Latest commit: |
939f400
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://f3c3604c.augmentos-appstore-2.pages.dev |
| Branch Preview URL: | https://g2-continued-work.augmentos-appstore-2.pages.dev |
Deploying dev-augmentos-console with
|
| Latest commit: |
939f400
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://4e439df9.dev-augmentos-console.pages.dev |
| Branch Preview URL: | https://g2-continued-work.dev-augmentos-console.pages.dev |
Deploying prod-augmentos-account with
|
| Latest commit: |
939f400
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://e4f6cf9c.augmentos-e84.pages.dev |
| Branch Preview URL: | https://g2-continued-work.augmentos-e84.pages.dev |
…to g2-continued-work
…to g2-continued-work
… into g2-continued-work
…to g2-continued-work
…to g2-continued-work
Summary by CodeRabbit
Release Notes
New Features
Improvements
Removals
setMicState()from WebView SDKrequestAudio()subscription from events API