Open
Conversation
…h to different LLM providers and configure additional settings like proxy and CA bundle 📝 (package.json): update devDependencies to include required types packages for follow-redirects, and add dependencies for http-proxy-agent and https-proxy-agent ♻️ (config.ts): refactor config.ts to include new configuration options for HTTP proxy, CA bundle, timeout, verify SSL, client certificate, and API version 🚀 (azure.ts): update azure.ts to use the new OpenAI package and pass the HTTP agent to handle proxy settings ♻️ (groq.ts): refactor groq.ts to use OpenAI package instead of the deprecated @azure/openai package ♻️ (openAi.ts): refactor openAi.ts to use OpenAI package instead of the deprecated @azure/openai package, and pass the HTTP agent to handle proxy settings 🚀 (set_missing_default_values.ts): add new migration to set missing default values in global config file ♻️ (engine.ts): refactor engine.ts to include API version in the default config object 🔧 (httpAgent.ts): add new httpAgent utility to handle HTTP settings including proxy, CA bundle, timeout, verify SSL, and client certificate
…it from Docker build ⬆️ (package.json): upgrade version from 3.2.2 to 3.2.3 in package.json ♻️ (httpAgent.ts): refactor code in getHttpAgent function to improve readability and maintainability
di-sukharev
reviewed
Nov 19, 2024
| export class AzureEngine implements AiEngine { | ||
| config: AzureAiEngineConfig; | ||
| client: AzureOpenAIClient; | ||
| client: OpenAI; |
| import migration00 from './00_use_single_api_key_and_url'; | ||
| import migration01 from './01_remove_obsolete_config_keys_from_global_file'; | ||
| import migration02 from './02_set_missing_default_values'; | ||
| import migration03 from './03_set_missing_default_values'; |
Owner
|
@ByJacob nice job, left few comments, also pls change the base branch to dev (we first merge there) and solve conflicts (there are few). to solve the |
Owner
|
@ByJacob pls let me know when you fix conflicts, i will merge this as soon as you are ready |
* 378: fix hook env (di-sukharev#402) * fix(prepare-commit-msg-hook): update error handling to provide clearer instructions for setting API keys and improve user guidance * Fix: a bug that causes an error when pushing without setting git remote (di-sukharev#396) * update deploy commands * feat(cli): add context flag for providing additional commit message input * Fix [Bug]: punycode` module is deprecated di-sukharev#426 (di-sukharev#433) Signed-off-by: Tiger Kaovilai <passawit.kaovilai@gmail.com> * npm audit fix (di-sukharev#432) Signed-off-by: Tiger Kaovilai <passawit.kaovilai@gmail.com> * Feat: Add an option to `Don't push` when there are multiple git remotes (di-sukharev#434) --------- Co-authored-by: GPT8 <57486732+di-sukharev@users.noreply.github.com> * feat(engine): add support for MLX AI provider (di-sukharev#437) --------- Signed-off-by: Emmanuel Ferdman <emmanuelferdman@gmail.com> Co-authored-by: Emmanuel Ferdman <emmanuelferdman@gmail.com> * feat(config, engine): add support for Mistral AI provider and engine (di-sukharev#436) * docs(CONTRIBUTING.md): update `TODO.md` reference (di-sukharev#435) Signed-off-by: Emmanuel Ferdman <emmanuelferdman@gmail.com> * feat(config, engine): add support for Mistral AI provider and engine * ``` feat(package): add mistralai and zod dependencies ``` * fix: recreate package-lock.json with node20 * fix: recreate package-lock.json with node v20.18.1 based on branch dev --------- Signed-off-by: Emmanuel Ferdman <emmanuelferdman@gmail.com> Co-authored-by: Emmanuel Ferdman <emmanuelferdman@gmail.com> Co-authored-by: pedro-valentim <> --------- Signed-off-by: Tiger Kaovilai <passawit.kaovilai@gmail.com> Signed-off-by: Emmanuel Ferdman <emmanuelferdman@gmail.com> Co-authored-by: Takanori Matsumoto <matscube@gmail.com> Co-authored-by: BILLY Maxime <ozeliurs@gmail.com> Co-authored-by: Welington Sampaio <welington.sampaio@icloud.com> Co-authored-by: Tiger Kaovilai <passawit.kaovilai@gmail.com> Co-authored-by: albi ⚡️ <sigismondi.alberto@gmail.com> Co-authored-by: Emmanuel Ferdman <emmanuelferdman@gmail.com> Co-authored-by: Pedro Valentim Silva Leite <18179935+pedro-valentim@users.noreply.github.com>
Owner
|
@ByJacob dude? |
Contributor
Author
Hi i merge with dev branch |
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.
Pull Request Description 🚀
Overview
This pull request updates the Azure library and introduces new features for enhanced configuration options.
Features
Documentation
Notes
Thank you! 🙏