Commit 19b0cf6
committed
Add OH_LLM_MODEL_KIND env var to enable DB-backed verified models
Wires up the SaaS verified-models DI injector so the OpenHands app reads
its verified model list from the database (maintained via the admin API)
instead of falling back to the hardcoded list shipped in openhands-sdk.
Before OpenHands/OpenHands#14237, enterprise/saas_server.py called
override_llm_models_dependency() unconditionally. That PR replaced the
mechanism with a generic LLMModelServiceInjector whose default is
DefaultLLMModelService (hardcoded). The SaaS variant
(SaaSLLMModelServiceInjector) is now only registered when
OH_LLM_MODEL_KIND points at it. Production was therefore silently
serving the hardcoded model list and admin-API changes did not show up
in app.all-hands.dev.
Mirrors the pattern already used for OH_APP_CONVERSATION_INFO_KIND.
Fixes #677
Co-authored-by: openhands <openhands@all-hands.dev>1 parent ed0435b commit 19b0cf6
2 files changed
Lines changed: 3 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
2 | 2 | | |
3 | 3 | | |
4 | 4 | | |
5 | | - | |
| 5 | + | |
6 | 6 | | |
7 | 7 | | |
8 | 8 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
529 | 529 | | |
530 | 530 | | |
531 | 531 | | |
| 532 | + | |
| 533 | + | |
532 | 534 | | |
533 | 535 | | |
534 | 536 | | |
| |||
0 commit comments