Skip to content
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

Address "Please read manifesto first" errors #234

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

smendola
Copy link

Running agency-swarm genesis was giving errors "Please read manifesto".
The fix offered here is to add an explicit instruction to AgentCreator to do so.
I added it as Step 0 so as to not have to renumber all the other steps and make a big diff out of a small change.
If you prefer, I will renumber starting at 1.

@@ -15,6 +15,7 @@
You are an agent that builds responsive web applications using Next.js and Material-UI (MUI). You must use the tools provided to navigate directories, read, write, modify files, and execute terminal commands.

### Primary Instructions:
0. Begin by reading the manifesto with the ReadManifesto tool
Copy link
Collaborator

Choose a reason for hiding this comment

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

I believe it's the wrong place to put this. web_developer_example_instructions are examples.

Copy link
Author

Choose a reason for hiding this comment

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

TY. So then where would it go?

Copy link
Collaborator

@bonk1t bonk1t Feb 18, 2025

Choose a reason for hiding this comment

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

The issue is not clear to me; we already have this instruction. The error you see is because an LLM hallucinates, and we raise it in the tool. This is expected behavior. The LLM must simply continue by calling that tool.
I suggest closing this PR for now.

Copy link
Author

Choose a reason for hiding this comment

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

My experience was that the genesis command fails; the AI does not recover from this error. The error is generated many times during the generation process, too. Adding this instruction fixes it.

Nevertheless, if you don't want the fix, I will close the PR.

Copy link
Collaborator

Choose a reason for hiding this comment

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

@smendola we already have this instruction:

1. First, read the manifesto using `ReadManifesto` tool if you have not already done so. This file contains the agency manifesto that describes the agency's purpose and goals.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants