main <- dev #511
Merged
main <- dev #511
Conversation
This file contains 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
Signed-off-by: Diwank Singh Tomer <[email protected]>
fix(agents-api): fix create_agent and create_or_update_agent query
fix(llm-proxy): Update docker image to main-v1.46.2
Signed-off-by: Diwank Singh Tomer <[email protected]>
…t-endpoint Add custom api key support to chat endpoint
Signed-off-by: Diwank Singh Tomer <[email protected]>
overriden -> overridden
![ellipsis-dev[bot]](https://avatars.githubusercontent.com/in/64358?s=60&v=4){kind=small}
There was a problem hiding this comment.
👍 Looks good to me! Reviewed everything up to db6594f in 36 seconds
More details
- Looked at
1574lines of code in63files - Skipped
1files when reviewing. - Skipped posting
1drafted comments based on config settings.
1. docs/js-sdk-docs/modules/api.md:202
- Draft comment:
| `tools?` | [`Tool`](api.md#tool)[] \| ``null`` | (Advanced) List of tools that are provided in addition to agent's default set of tools. Functions of same name in agent set are overridden |
- Reason this comment was not posted:
Confidence changes required:10%
The word 'overridden' is misspelled as 'overriden'. This is a simple typo that should be corrected for clarity and professionalism.
Workflow ID: wflow_B6q2RVb89KsdjyfK
You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
feat: enhance agents API and session management, update Docker configurations
Summary:
This pull request enhances the agents API, session management, and task execution, updates Docker configurations, and improves error handling and security across the codebase.
Key points:
X-Custom-Api-Keyheader togenerateendpoint inendpoints.tspfor custom API key support.create_or_update_session.pyto returnResourceUpdatedResponseinstead ofResourceCreatedResponse.create_agent.py,create_user.py) to useuuid.UUIDinstead ofpydantic.UUID4.LITELLM_SALT_KEYto.env.examplefor additional security.acompletioninlitellm.pyto support custom API keys.search_docs_by_embedding.pyto useunique(snippet_data)instead ofcollect(snippet).docker-compose.ymlinllm-proxyto uselitellm-database:main-v1.46.6image.scheduler/docker-compose.ymlto use environment variables for database configuration.api.md("overriden" to "overridden").AssertionErrorhandling increate_or_update_session.pyandcreate_session.pyfor better error management.Generated with ❤️ by ellipsis.dev