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
Fix micro-agents schema bug #1424
Conversation
This reverts commit 94f3b4e.
I don't think micro-agents are ready for integration testing yet, so I didn't add them to CI testing. |
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.
LGTM!
I've been debating whether to add the Agent
suffix to everything...
Aha, I had a guess you intentionally did so. But it looks a bit inconsistent as in there's no criteria which shall be an agent and which shall not be. To keep the consistency I added "Agent" as a suffix to each micro-agent. |
Some of the micro-agents'
agent.yaml
have schema definition errors, e.g.RepoExplorerAgent
. This PR fixes these bugs. ManagerAgent, however, is not added to integration tests as I can barely get it working with the task in the test:Write a shell script 'hello.sh' that prints 'hello'
. Looks like micro-agents are not ready yet and may need more prompt tuning.You can verify this fix locally by running any task that might trigger
RepoExplorerAgent
.Fixes #1423