-
-
Notifications
You must be signed in to change notification settings - Fork 18.9k
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 agent flow tool call errors #3981
base: main
Are you sure you want to change the base?
Fix agent flow tool call errors #3981
Conversation
@@ -164,15 +187,44 @@ export class MySQLSaver extends BaseCheckpointSaver implements MemoryMethods { | |||
} | |||
} | |||
|
|||
private async cleanupOldMessages(dataSource: DataSource, threadId: string, limit: number = 100): Promise<void> { |
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.
Can we have the option for user to enable/disable this. For example we can put this to the this.inputs
:
{
label: 'Auto Clear Messages',
name: 'autoClearMessages',
type: 'boolean',
description: 'Automatically clear all but the last 100 messages for this thread',
additionalParams: true,
optional: true
},
if (tools.some((tool) => tool.name === toolCall.name)) { | ||
return toolNode.name | ||
try { | ||
// Track tool calls that need responses |
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.
can we hold on to this first, as we are revamping the agentflow and there's some fixes we have put in place as well
thank you so much @refugedesigns ! would you be able to leave the changes on the |
Ok Henry, will revamp and send another pull request. |
No description provided.