-
Notifications
You must be signed in to change notification settings - Fork 4k
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
feat: convert agent prompts into structured Jinja2 templates #3360
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.
Cool stuff! LGTM
Co-authored-by: Engel Nyst <[email protected]>
|
||
|
||
class CodeActAgent(Agent): | ||
VERSION = '1.8' | ||
VERSION = '1.9' |
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.
Just to note, it's not obvious to me from the diff (looking on GitHub) that the prompt as sent to the llm really did change. But 1.9 is ok anyway, it's probably overdue.
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.
OK, it did change, though not a lot.
This looks great! |
What is the problem that this fixes or functionality that this introduces? Does it fix any open issues?
This is part of the new micro-agent proposal: #3444
Give a summary of what the PR does, explaining any non-trivial design decisions
PromptManager
that takes in a prompt directory,agentskill
documentations, and optionally a micro-agent name. It will use Jinja2 template to generate the system message and first user message for CodeAct agent. See [Agent Framework] New microagents implementation #3444 for more details.Other references