feat(assistant): add optional API URL field to LLM configuration#248
Merged
ntthanh2603 merged 2 commits intomainfrom Jan 15, 2026
Merged
feat(assistant): add optional API URL field to LLM configuration#248ntthanh2603 merged 2 commits intomainfrom
ntthanh2603 merged 2 commits intomainfrom
Conversation
- Add optional apiUrl field to LLM form and manager interfaces - Include new API URL input field in LLM configuration form - Update LLM configuration handlers to support custom API endpoints - Enable users to specify custom API URLs for LLM providers This change allows users to configure custom API endpoints for their LLM providers, providing flexibility for self-hosted or alternative API implementations.
1 task
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
Add missing apiUrl field to the expected object in AiAssistantService test. This ensures the test assertion matches the updated service interface structure.
l1ttps
approved these changes
Jan 15, 2026
Member
l1ttps
left a comment
There was a problem hiding this comment.
Thanks @ntthanh2603 for this PR! This looks great - adding the optional API URL field to LLM configurations provides much-needed flexibility for users who want to connect to self-hosted or alternative LLM endpoints. The implementation is comprehensive, covering the frontend form, backend DTOs, proto definitions, and service logic. I appreciate the clean integration across the full stack and the removal of unused code paths. The dependency update to pgvector/postgres:pg17 is also a nice bonus for vector database capabilities.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This change allows users to configure custom API endpoints for their LLM providers, providing flexibility for self-hosted or alternative API implementations.