-
Notifications
You must be signed in to change notification settings - Fork 0
add deep agent #110
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add deep agent #110
Conversation
There was a problem hiding this 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 adds a new "deep agent" capability to the template, which provides a sophisticated research agent framework with sub-agent orchestration, file management, and task tracking features. The implementation is based on the langchain-ai/deepagents project.
Key changes:
- Adds a complete deep agent core framework with state management, tools, and sub-agent coordination
- Implements a research-focused agent that can conduct thorough research and generate detailed reports
- Integrates the new agent into the LangGraph configuration for deployment
Reviewed Changes
Copilot reviewed 9 out of 9 changed files in this pull request and generated 2 comments.
File | Description |
---|---|
template_langgraph/agents/demo_agents/research_deep_agent.py | Main entry point defining research and critique sub-agents with detailed prompts |
template_langgraph/agents/demo_agents/deep_agent_core/ | Core framework providing state management, tools, and agent orchestration |
langgraph.json | Registers the new deep agent in the application configuration |
docs/references.md | Adds documentation references to the deep agents project |
Comments suppressed due to low confidence (1)
template_langgraph/agents/demo_agents/deep_agent_core/tools.py:14
- The parameter names 'l' and 'r' are ambiguous single-letter variables. Consider renaming to 'left' and 'right' or 'current' and 'new' for better readability.
from template_langgraph.agents.demo_agents.deep_agent_core.state import DeepAgentState, Todo
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
|
||
critique_sub_agent = { | ||
"name": "critique-agent", | ||
"description": "Used to critique the final report. Give this agent some infomration about how you want it to critique the report.", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is a spelling error in 'infomration'. It should be 'information'.
"description": "Used to critique the final report. Give this agent some infomration about how you want it to critique the report.", | |
"description": "Used to critique the final report. Give this agent some information about how you want it to critique the report.", |
Copilot uses AI. Check for mistakes.
- `description` (used by the main agent to decide whether to call the sub agent) | ||
- `prompt` (used as the system prompt in the subagent) | ||
- (optional) `tools` | ||
state_schema: The schema of the deep agent. Should subclass from template_langgraph.agents.demo_agents.deep_agent_coretate |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is a typo in the docstring: 'deep_agent_coretate' should be 'deep_agent_core.state'.
state_schema: The schema of the deep agent. Should subclass from template_langgraph.agents.demo_agents.deep_agent_coretate | |
state_schema: The schema of the deep agent. Should subclass from template_langgraph.agents.demo_agents.deep_agent_core.state |
Copilot uses AI. Check for mistakes.
What this PR does / why we need it
Which issue(s) this PR fixes:
Fixes #86
Does this introduce a breaking change?
Pull Request Type
What kind of change does this Pull Request introduce?
How to Test
What to Check
Verify that the following are valid
Additional Information