Skip to content

Updated text to clarify LLM connection issues #1151

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
pavangudiwada wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Updated text to clarify LLM connection issues #1151

pavangudiwada wants to merge 4 commits into from
+6 −2

Conversation

@pavangudiwada
Copy link
Contributor

@pavangudiwada pavangudiwada commented Dec 1, 2025

Purely UX change

  1. Instead of logging "No llm models were loaded"(missed in text) -> loading Tools -> Raising exception(image below), raise an exception immediately.
image 2. Updated text to provide context on what could be the issue and how to fix it.

Result

image

@pavangudiwada pavangudiwada changed the title Missing models text Updated text to clarify missing model error Dec 1, 2025
@pavangudiwada pavangudiwada changed the title Updated text to clarify missing model error Updated text to clarify LLM connection issues Dec 1, 2025
@moshemorad
Copy link
Collaborator

moshemorad commented Dec 7, 2025

@pavangudiwada can you check tests please?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@moshemorad moshemorad moshemorad approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@pavangudiwada @moshemorad