fix(tui): implement models:fetch-and-pick async handler

[echobt]

Summary

The /models command (and aliases /lm, /list-models) was broken because it dispatched to 'models:fetch-and-pick' async handler but that handler was never implemented in handle_async_command().

Changes

Added the missing case in handle_async_command() that opens the ModelPicker modal, which fetches models from the provider_manager and displays them.

Fix

• Added "models:fetch-and-pick" case in handle_async_command() that calls handle_open_modal(ModalType::ModelPicker).await
• The existing ModalType::ModelPicker infrastructure handles model fetching and display

Testing

• cargo check -p cortex-tui passes
• cargo clippy -p cortex-tui passes with no warnings

What was happening

When running /models, /lm, or /list-models, the command fell through to the default case showing "Command not yet implemented" toast.

What happens now

The model picker modal opens showing available models fetched from the provider manager.

[greptile-apps]

Greptile Overview

Greptile Summary

This PR fixes a broken command by adding the missing async handler for 'models:fetch-and-pick'. The /models, /lm, and /list-models commands were dispatching to this handler but it wasn't implemented, causing them to fall through to the default case.

Changes:

• Added "models:fetch-and-pick" case in handle_async_command() that calls handle_open_modal(ModalType::ModelPicker).await
• The existing ModalType::ModelPicker infrastructure already handles model fetching and display (lines 209-242)

Impact:

• Commands /models, /lm, /list-models now properly open the model picker modal
• No breaking changes or new functionality added - purely a bug fix

Confidence Score: 5/5

• This PR is safe to merge with no risk
• The change is a minimal, straightforward bug fix that adds a missing case to a match statement. The handler simply delegates to existing, well-tested infrastructure (handle_open_modal with ModalType::ModelPicker). The fix is aligned with how other async commands are handled in the same function.
• No files require special attention

Important Files Changed

Filename	Overview
src/cortex-tui/src/runner/event_loop/commands.rs	Added missing "models:fetch-and-pick" async handler that opens ModelPicker modal

Sequence Diagram

sequenceDiagram
participant User
participant CommandExecutor
participant EventLoop
participant ModalHandler
participant ProviderManager
participant UI
User->>CommandExecutor: /models (or /lm, /list-models)
CommandExecutor->>EventLoop: CommandResult::Async("models:fetch-and-pick")
EventLoop->>EventLoop: handle_async_command("models:fetch-and-pick")
EventLoop->>ModalHandler: handle_open_modal(ModalType::ModelPicker)
ModalHandler->>ProviderManager: available_models()
ProviderManager-->>ModalHandler: Vec<Model>
ModalHandler->>ProviderManager: current_model()
ProviderManager-->>ModalHandler: String
ModalHandler->>UI: build_model_selector(models, current)
ModalHandler->>EventLoop: enter_interactive_mode(interactive)
EventLoop-->>User: Display ModelPicker modal

Loading

[greptile-apps]

_{1 file reviewed, no comments}

_{Edit Code Review Agent Settings | Greptile}