22e31a0d41
Closes #37617 We're already using `get` in a bunch of places, this PR updates the remaining spots to follow the same pattern. Note that the `ix` we read in `render_match` can sometimes be stale. The likely reason is that we run the match-update logic asynchronously (see [here](138117e0b1/crates/picker/src/picker.rs (L643))). That means it's possible to render items after the list's [data update](138117e0b1/crates/picker/src/picker.rs (L652)) but before the [list reset](138117e0b1/crates/picker/src/picker.rs (L662)), in which case the `ix` can be greater than that of our updated data. Release Notes: - Fixed crash when filtering MCP tools.