-
Notifications
You must be signed in to change notification settings - Fork 315
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
[LIVE-12648] Crash because of invalid DeviceModelId #6912
[LIVE-12648] Crash because of invalid DeviceModelId #6912
Conversation
The latest updates on your projects. Learn more about Vercel for Git βοΈ 5 Ignored Deployments
|
b2c1f57
to
de9d42e
Compare
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.
π
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.
Very good. From what we saw together it definitely make things more robust.
And shipped with tests π
chore: changesets
3f637a2
to
cf58a90
Compare
β Checklist
npx changeset
was attached.π Description
The store has some bad data in it because we are doing bad type assertions of some objects into the
Device
type.It's currently causing crashes at different levels of LLM, on Android.
The solution proposed here is:
as Device
type assertions. If an object is incorrect (here, because it does not contain device model id), it is actually unusable so it should be ignored. We could not reproduce the issue but it appears that in some rare cases, the "device discovery" of the Android USB transport sends event with such invalid objects.β Context
π§ Checklist for the PR Reviewers