feat(agent): Agent optimize #2647
Conversation
…und as a sub-agent, and strip vis to become an extended component.
|
hi, Can you explain how to use it and show a screenshot of the specific effect? |
There was a problem hiding this comment.
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
| result: Optional[str] = None | ||
|
|
||
| @staticmethod | ||
| def from_dict(d: Dict[str, Any]) -> "GptsPlan": | ||
| """Create a GptsPlan object from a dictionary.""" | ||
| return GptsPlan( | ||
| conv_id=d["conv_id"], | ||
| conv_round=d["conv_id"], |
There was a problem hiding this comment.
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.
| conv_round=d["conv_id"], | |
| conv_round=d["conv_round"], |
Copilot uses AI. Check for mistakes.
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: