-
Notifications
You must be signed in to change notification settings - Fork 210
Ya/rewrite nex sgc #1822
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.
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
Ya/rewrite nex sgc #1822
Conversation
Deploying mentra-live-ota-site with
|
| Latest commit: |
b16d3ad
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://50e2a5fa.mentra-live-ota-site.pages.dev |
| Branch Preview URL: | https://ya-rewrite-nex-sgc.mentra-live-ota-site.pages.dev |
Deploying mentra-store-dev with
|
| Latest commit: |
b16d3ad
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://96c4712f.augmentos-appstore-2.pages.dev |
| Branch Preview URL: | https://ya-rewrite-nex-sgc.augmentos-appstore-2.pages.dev |
Deploying dev-augmentos-console with
|
| Latest commit: |
b16d3ad
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://87bf9abb.dev-augmentos-console.pages.dev |
| Branch Preview URL: | https://ya-rewrite-nex-sgc.dev-augmentos-console.pages.dev |
Deploying prod-augmentos-account with
|
| Latest commit: |
b16d3ad
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://44d055bb.augmentos-e84.pages.dev |
| Branch Preview URL: | https://ya-rewrite-nex-sgc.augmentos-e84.pages.dev |
|
@codex review |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: b8c05132d3
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
mobile/modules/core/android/src/main/java/com/mentra/core/sgcs/MentraNex.kt
Show resolved
Hide resolved
The version query packet was being generated but never actually sent to the glasses, causing glasses_protobuf_version to stay unset after connection. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
|
| GitGuardian id | GitGuardian status | Secret | Commit | Filename | |
|---|---|---|---|---|---|
| 22988493 | Triggered | Generic High Entropy Secret | b16d3ad | .github/workflows/china-deploy-static-websites-dev.yaml | View secret |
🛠 Guidelines to remediate hardcoded secrets
- Understand the implications of revoking this secret by investigating where it is used in your code.
- Replace and store your secret safely. Learn here the best practices.
- Revoke and rotate this secret.
- If possible, rewrite git history. Rewriting git history is not a trivial act. You might completely break other contributing developers' workflow and you risk accidentally deleting legitimate data.
To avoid such incidents in the future consider
- following these best practices for managing and storing secrets including API keys and other credentials
- install secret detection on pre-commit to catch secret before it leaves your machine and ease remediation.
🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.
|
@codex review |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: b16d3ad808
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
mobile/modules/core/android/src/main/java/com/mentra/core/sgcs/MentraNex.kt
Outdated
Show resolved
Hide resolved
mobile/modules/core/android/src/main/java/com/mentra/core/utils/NexSGCUtils.kt
Show resolved
Hide resolved
|
@codex review |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 983765f5a9
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| val bmp = BitmapFactory.decodeByteArray(bmpData, 0, bmpData.size) | ||
|
|
||
| displayBitmapImageForNexGlasses(bmpData, bmp.width, bmp.height) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Handle null bitmap decode before using width/height
In MentraNex.displayBitmap, BitmapFactory.decodeByteArray can return null for malformed/unsupported image data, but the code immediately dereferences bmp.width/bmp.height. This will crash the app whenever callers pass invalid base64 or non-decodable bytes. Add a null check (as other SGCs do) and return false instead of proceeding.
Useful? React with 👍 / 👎.
| private fun reSendImageMissingChunks(missingChunksIndexList: List<Int>) { | ||
| if (!isImageSendProgressing || currentImageChunks.isEmpty() || missingChunksIndexList.isEmpty()) { | ||
| return |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Allow image resend when transfer reports missing chunks
reSendImageMissingChunks exits unless isImageSendProgressing is true, but this flag is never set to true when an image transfer starts (it’s only cleared). As a result, when the glasses return INCOMPLETE, the missing chunks are never resent and the image remains incomplete. Set the flag when starting a transfer or remove the guard so the resend path can run.
Useful? React with 👍 / 👎.
No description provided.