Skip to content

feat(api): Add chat_dashboard support to V2 API#2726

Merged
chenliang15405 merged 1 commit into
eosphoros-ai:mainfrom
WangzJi:feature/api_v2-support-chat_dashboard
May 23, 2025
Merged

feat(api): Add chat_dashboard support to V2 API#2726
chenliang15405 merged 1 commit into
eosphoros-ai:mainfrom
WangzJi:feature/api_v2-support-chat_dashboard

Conversation

@WangzJi
Copy link
Copy Markdown
Contributor

@WangzJi WangzJi commented May 23, 2025

  • Add CHAT_DASHBOARD to ChatMode enum
  • Enable chat_dashboard mode in V2 chat completions API
  • Fix database config lookup to handle missing configurations gracefully

Description

This PR implements support for chat_dashboard mode in the v2 API, ensuring that the stream parameter works correctly in this mode. The implementation follows the same functionality as in v1 API but adapts it to the v2 API architecture and design patterns.

This PR addresses the issue where the dashboard mode was only available in v1 API, while project maintainers recommend using v2 API for new features.

Fixes #2697 #2699

How Has This Been Tested?

  1. Setting up a local instance of DB-GPT
  2. Making API calls to the v2 endpoint with chat_dashboard mode enabled
  3. Verifying that the stream parameter works correctly in dashboard mode

Snapshots:

image image

Checklist:

  • My code follows the style guidelines of this project
  • I have already rebased the commits and make the commit message conform to the project standard.
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • Any dependent changes have been merged and published in downstream modules

- Add CHAT_DASHBOARD to ChatMode enum
- Enable chat_dashboard mode in V2 chat completions API
- Fix database config lookup to handle missing configurations gracefully
@github-actions github-actions Bot added the enhancement New feature or request label May 23, 2025
Copy link
Copy Markdown
Member

@fangyinc fangyinc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

Copy link
Copy Markdown
Collaborator

@chenliang15405 chenliang15405 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@chenliang15405 chenliang15405 merged commit a10535d into eosphoros-ai:main May 23, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] [Dashboard] Fix stream parameter not working in chat_dashboard mode for API v1

3 participants