atiaomar1978-hub commented on PR #26145: URL: https://github.com/apache/camel/pull/26145#issuecomment-5560207415
Review follow-up (Bugbot + Grok): **Fixed in latest commits:** - TUI doctor now probes Ollama **once** per popup open (cached `OllamaDoctorSupport.Status`) - Replaced flaky test that assumed no local Ollama on `:11434` with injectable `detect(LlmClient)` + HTTP-server tests - AI provider shows **WARN** when Ollama is running but no models are pulled (`ollama pull` hint) - Truncated long model lists / hints to fit the doctor popup width - Generic `ollama serve` hint (not brew-only) **Deferred / out of scope for this ticket:** - Tighter Ollama reachability check in `LlmClient.isEndpointReachable()` (broader change) - Doctor probe timeout tuning on TUI event thread - Docs / upgrade guide (can follow in a doc PR if desired) Tests: `OllamaDoctorSupportTest` (7), `DoctorTest` (+3 Ollama), `DoctorPopupOllamaTest` (7) — all green locally. _AI-generated on behalf of atiaomar1978-hub_ -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
