Skip to content

Conversation

@yhjun1026
Copy link
Collaborator

Description

Vis architecture modification, component packages can be dynamically extended; Agent architecture extension, supporting the binding of any conversational target for dialogue; Model selection strategy code optimization.

How Has This Been Tested?

Snapshots:

Include snapshots for easier review.

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

@Aries-ckt
Copy link
Collaborator

hi, Can you explain how to use it and show a screenshot of the specific effect?

@fangyinc fangyinc requested a review from Copilot April 27, 2025 03:03
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR optimizes the agent planning and execution architecture by renaming key fields (e.g. from "plan" to "reasoning_engine") and correcting task identifier usage across planning, messaging, and memory modules. Key changes include updates to the diff structure in auto plan and planner agent, modifications in field names and error messages, and the addition of a new thinking method in agent classes.

Reviewed Changes

Copilot reviewed 70 out of 72 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
packages/dbgpt-core/src/dbgpt/agent/core/plan/auto/team_auto_plan.py Renames terminology in comments and variable usage for clarity.
packages/dbgpt-core/src/dbgpt/agent/core/plan/auto/planner_agent.py Updates docstrings and error messages to refer to "reasoning_engine" consistently.
packages/dbgpt-core/src/dbgpt/agent/core/plan/auto/plan_action.py Replaces task numbering with task id references and updates output messaging.
packages/dbgpt-core/src/dbgpt/agent/core/memory/gpts/default_gpts_memory.py Adjusts query conditions to use new field names for task ids.
packages/dbgpt-core/src/dbgpt/agent/core/memory/gpts/base.py Adds new fields and updates conversion method but contains a field mapping issue.
Remaining files Update references to new vis protocols, message fields, and resource handling with consistent naming.
Files not reviewed (2)
  • assets/schema/dbgpt.sql: Language not supported
  • assets/schema/upgrade/v0_7_2/upgrade_to_v0.7.2.sql: Language not supported

"""Create a GptsPlan object from a dictionary."""
return GptsPlan(
conv_id=d["conv_id"],
conv_round=d["conv_id"],
Copy link

Copilot AI Apr 27, 2025

Choose a reason for hiding this comment

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

The 'conv_round' field is incorrectly assigned the value of 'd["conv_id"]' instead of the expected 'd["conv_round"]'. Please update the assignment to correctly retrieve the conversation round.

Suggested change
conv_round=d["conv_id"],
conv_round=d["conv_round"],

Copilot uses AI. Check for mistakes.
@fangyinc fangyinc changed the title Feat/agent optimize feat(agent): Agent optimize Apr 27, 2025
@github-actions github-actions bot added agent Module: agent enhancement New feature or request labels Apr 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agent Module: agent enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants