fix: expand boards if available on detected port #2178
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Motivation
When multiple boards are discovered on a detected port, the board selector component is broken in the firmware uploader dialog. This PR moves the board inference logic from the UI to the model and allows selecting a board+port pair even before confirming the board in the board list.
This PR pins the CLI version to
0.34.0-rc.1
to overcome the flaky tests mentioned at #2176 (comment).Change description
Other information
I could reproduce the bug following the steps in #2175. I could fix the bug, but the firmware update fails at the CLI.
I connect my Arduino Nano RP2040 board and start the firmware update, and get all these board list watch logs and the stderr of the executable:
click to see the `board list watch` events and the error log from the firmware uploader CLI
Closes #2175
Reviewer checklist