PlaudDeviceConnection._parseAudioChunk - RangeError crash

[aaravgarg]

Summary

• Add bounds check before payload.sublist(9, 9 + length) to prevent RangeError
• Return null when payload is truncated or corrupted instead of crashing

Crash Stats

• Events: 14
• Users affected: 1
• Crashlytics: View in Console

Test plan

• Verify Plaud audio streaming still works correctly
• Test with corrupted BLE payload (should return null, not crash)

🤖 Generated with Claude Code

[gemini-code-assist]

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

[beastoin]

Hey @aaravgarg! Thanks for this crash fix (PlaudDevice RangeError). Good write-up with Crashlytics data.

We're tidying up stale PRs — this one has been open 8 days with an untested test plan (all checkboxes unchecked) and no screenshot/video demo of the fix working.

Could you please run the test plan, check off the items, and add a quick screenshot or recording? Feel free to reopen once that evidence is added. Happy to review!

[github-actions]

Hey @aaravgarg 👋

Thank you so much for taking the time to contribute to Omi! We truly appreciate you putting in the effort to submit this pull request.

After careful review, we've decided not to merge this particular PR. Please don't take this personally — we genuinely try to merge as many contributions as possible, but sometimes we have to make tough calls based on:

• Project standards — Ensuring consistency across the codebase
• User needs — Making sure changes align with what our users need
• Code best practices — Maintaining code quality and maintainability
• Project direction — Keeping aligned with our roadmap and vision

Your contribution is still valuable to us, and we'd love to see you contribute again in the future! If you'd like feedback on how to improve this PR or want to discuss alternative approaches, please don't hesitate to reach out.

Thank you for being part of the Omi community! 💜