fix: app picks model automatically edge cases

[louis-jan]

Describe Your Changes

Fixed an issue where deleted models were not handled gracefully. However, there are still edge cases where users could change tool settings or delete a new thread, causing the app to automatically assign a new model to older threads with deleted models.

Fixes Issues

• QA: Jan 0.5.12 Release Sign-off #4291

Changes made

This pull request includes several changes to improve the handling of model selection and thread management in the web application. The most important changes include refactoring the code to simplify model selection and updating the thread deletion logic to use a new hook.

Improvements to model selection:

• web/containers/ModelDropdown/index.tsx: Simplified the logic for finding and setting the selected model by removing unnecessary conditional checks.
• web/screens/Thread/ThreadRightPanel/Tools/index.tsx: Refactored the model selection logic to directly set the selected model without additional checks.

Enhancements to thread management:

• web/hooks/useDeleteThread.ts: Added useSetActiveThread hook to manage the active thread state and updated the cleanThread function to use this new hook. [1] [2] [3]

[github-actions]

This is the build for this pull request. You can download it from the Artifacts section here: Build URL.

[github-actions]

Barecheck - Code coverage report

Total: 68.75%

Your code coverage diff: 0.15% ▴

Uncovered files and lines

File	Lines
web/containers/ModelDropdown/index.tsx	119-120, 126, 137, 147-148, 151, 153-154, 156-157, 159-162, 164, 198-201, 203, 205, 224, 228, 237, 244, 247-248, 284-286, 290, 314-316, 318, 322, 325-330, 337-338, 349, 371, 394, 415, 424, 430, 438-439, 442-447, 450, 453, 505-506, 508, 534, 542, 544, 569, 574-575, 577, 581-583, 585-586, 588, 598-601, 629, 637, 639
web/hooks/useDeleteThread.ts	71-72
web/screens/Thread/ThreadRightPanel/Tools/index.tsx	22-27, 29, 33-36, 38, 47, 49-50, 72, 74-75, 98, 128, 219