devika icon indicating copy to clipboard operation
devika copied to clipboard

Add support for custom OpenAI API Base URL and DuckDuckGoSearch

Open its-arun opened this issue 11 months ago • 2 comments

Added support to change OpenAI API Base URL which fixes #22 #33 Added support to use DuckDuckGoSearch as a Bing Alternative fixed #65

its-arun avatar Mar 23 '24 05:03 its-arun

Code Review Agent Run Status

  • AI Based Review: Successful

Code Review Overview

  • Summary: This PR introduces the ability to configure the OpenAI API Base URL and adds support for DuckDuckGo as an alternative search engine to Bing. These changes allow for greater flexibility in using different search engines and interacting with potentially different OpenAI API endpoints, which could be beneficial for users in regions where Bing is less effective or for those who prefer DuckDuckGo's privacy-focused approach. Additionally, the update enhances the project's configurability and adaptability to different environments and user preferences.
  • Code change type: Feature Addition, Configuration Changes
  • Unit tests added: False
  • Estimated effort to review (1-5, lower is better): 2 - The changes are straightforward and well-structured, but thorough testing is necessary to ensure the new functionality does not introduce regressions or errors.

>>See detailed code suggestions<<
The Bito AI Code Review Agent successfully reviewed 6 files and discovered 20 issues. Please review these issues along with suggested fixes in the Changed Files.

High-level Feedback

The PR is a valuable addition to the project, enhancing its flexibility and usability. However, it's crucial to ensure that these new features are thoroughly tested, especially since they involve external APIs and user preferences that might vary widely. Including unit tests for these new functionalities could significantly improve the PR's quality and reliability.

BitoAgent avatar Mar 25 '24 22:03 BitoAgent

Add this please

ibndias avatar Mar 27 '24 01:03 ibndias

We really need change OpenAI API Base URL, please merge it

ShiFangJuMie avatar Apr 02 '24 18:04 ShiFangJuMie