-
Notifications
You must be signed in to change notification settings - Fork 86
Add Nexa local LLM support #19
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.
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
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -63,6 +63,8 @@ def _detect_provider(self) -> str: | |
| model_lower = (self.model or "").lower() | ||
| base_lower = (self.base_url or "").lower() | ||
|
|
||
| if "nexa" in model_lower or "nexa" in base_lower or "11434" in base_lower: | ||
| return "nexa" | ||
|
Comment on lines
+66
to
+67
|
||
| if any(k in model_lower for k in ["claude", "anthropic"]): | ||
| return "anthropic" | ||
| if "anthropic" in base_lower: | ||
|
|
||
| Original file line number | Diff line number | Diff line change | ||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| @@ -0,0 +1,20 @@ | ||||||||||||||
| services: | ||||||||||||||
| mirofish: | ||||||||||||||
| depends_on: {} | ||||||||||||||
| networks: | ||||||||||||||
| - default | ||||||||||||||
| environment: | ||||||||||||||
|
Comment on lines
+1
to
+6
|
||||||||||||||
| LLM_PROVIDER: openai | ||||||||||||||
| LLM_BASE_URL: http://host.docker.internal:11434/v1 | ||||||||||||||
| LLM_API_KEY: nexa | ||||||||||||||
|
Comment on lines
+7
to
+9
|
||||||||||||||
| LLM_PROVIDER: openai | |
| LLM_BASE_URL: http://host.docker.internal:11434/v1 | |
| LLM_API_KEY: nexa | |
| LLM_PROVIDER: nexa | |
| LLM_BASE_URL: http://host.docker.internal:11434/v1 | |
| NEXA_API_KEY: nexa |
Copilot
AI
Apr 2, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LLM_BASE_URL uses host.docker.internal, which isn’t available by default on many Linux Docker setups. If this compose config is meant to be cross-platform, add an extra_hosts: ["host.docker.internal:host-gateway"] mapping (or document the platform requirement / use a different host reference).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Config.validate()comparescls.LLM_PROVIDERwithout normalizing case. If a user setsLLM_PROVIDER=NEXA(or other mixed-case), validation won’t treat it as the exemptnexaprovider and may produce a misleading “LLM_API_KEY not configured” error. Consider lowercasingLLM_PROVIDERwhen reading it (or invalidate()) to match howLLMClientnormalizes providers.