feat(chat model): add Volcengine chat model node#5928
feat(chat model): add Volcengine chat model node#5928yuyol wants to merge 4 commits intoFlowiseAI:mainfrom
Conversation
Summary of ChangesHello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request introduces native support for Volcengine's large language models within Flowise. By adding a dedicated chat model node, users can now seamlessly connect to and leverage Volcengine's powerful LLM services, such as the Doubao models, directly within their AI pipelines. This expansion broadens the range of available LLM providers, particularly benefiting users within the Chinese developer ecosystem. Highlights
Changelog
Activity
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
There was a problem hiding this comment.
Code Review
This pull request introduces a new chat model node for Volcengine. A critical security flaw was identified and its severity upgraded to 'high' due to the server-side context of API key leakage, with a suggested fix provided. Additionally, a high-severity issue with the temperature parameter's unsafe handling was noted, and a robust code suggestion has been provided. Despite these, the implementation correctly uses the @langchain/openai package, sets up credentials, and adds new models, generally adhering to existing patterns.
There was a problem hiding this comment.
Code Review
This pull request introduces support for Volcengine chat models by adding a new ChatVolcengine node. The implementation correctly leverages the existing ChatOpenAI component for OpenAI-compatible APIs. The changes include a new credential type, the node implementation, and model definitions in models.json. My review identified a couple of minor issues: the use of a deprecated property and an inconsistent default model name. I've provided suggestions to address these points to improve the code's correctness, maintainability, and cost-effectiveness. Overall, this is a well-implemented feature addition.
packages/components/nodes/chatmodels/ChatVolcengine/ChatVolcengine.ts
Outdated
Show resolved
Hide resolved
packages/components/nodes/chatmodels/ChatVolcengine/ChatVolcengine.ts
Outdated
Show resolved
Hide resolved
|
Hi maintainers, this PR currently has workflows awaiting approval. |
Add Volcengine Chat Model Node
Overview
This PR adds support for Volcengine Chat Models as a new chat model node in Flowise.
Users can now connect to Volcengine LLM services and use them directly inside Flowise workflows, similar to existing providers such as OpenAI, Anthropic, and Google.
Why
Volcengine provides powerful LLM services (e.g. Doubao models) that are widely used in the Chinese developer ecosystem.
Adding native support for Volcengine enables Flowise users to easily integrate these models into their AI pipelines.
What’s included
How to test
Start Flowise locally
Add the Volcengine Chat Model node to a flow
Configure:
Connect it to a Prompt node or Chatflow
Run the flow and verify responses from Volcengine
Screenshots
Notes