feat(model connection): remove manual model creation#1551
feat(model connection): remove manual model creation#1551AnishSarkar22 wants to merge 6 commits into
Conversation
|
@AnishSarkar22 is attempting to deploy a commit to the Rohan Verma's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
… provider connect dialogs - Introduced `isRefreshDisabled` prop to `ModelsSelectionPanel` to control refresh button state. - Updated `ProviderConnectDialog` to conditionally enable refresh functionality based on model support and submission status.
Description
Motivation and Context
FIX #
Screenshots
API Changes
Change Type
Testing Performed
Checklist
High-level PR Summary
This PR fixes an issue with LM Studio integration by providing a dummy API key value (
"not-needed") when no API key is configured. LM Studio accepts unauthenticated local requests, but the LiteLLM OpenAI-compatible adapter requires anapi_keyparameter to be present. The fix includes a conditional check that supplies the placeholder value specifically for thelm_studioprovider when the API key is empty, along with a corresponding unit test to verify the behavior.⏱️ Estimated Review Time: 5-15 minutes
💡 Review Order Suggestion
surfsense_backend/tests/unit/services/test_model_connections.pysurfsense_backend/app/services/model_resolver.py